chore: remove unused eslint-disable comments after eslint upgrades

packages/design-system-react-native/global.d.ts

@@ -1,5 +1,4 @@
 declare module '*.svg' {
-  // eslint-disable-next-line @typescript-eslint/naming-convention
   import type * as React from 'react';
   import type { SvgProps } from 'react-native-svg';
 

packages/design-system-react/global.d.ts

@@ -4,7 +4,6 @@ declare module '@metamask/jazzicon' {
 }
 
 declare module '*.svg' {
-  // eslint-disable-next-line @typescript-eslint/naming-convention
   import type * as React from 'react';
 
   export const ReactComponent: React.FC<React.SVGProps<SVGSVGElement>>;

packages/design-system-react/scripts/create-component/create-component.test.ts

@@ -171,7 +171,6 @@ import { ComponentName } from '@metamask/design-system-react';
       `;
 
       (fs.readFile as jest.Mock).mockImplementation(async (path: string) => {
-        // eslint-disable-next-line jest/no-conditional-in-test
         if (path.endsWith('README.mdx')) {
           return Promise.resolve(mockReadmeContent);
         }
@@ -231,7 +230,6 @@ import { ComponentName } from '@metamask/design-system-react';
       `;
 
       (fs.readFile as jest.Mock).mockImplementation(async (path: string) => {
-        // eslint-disable-next-line jest/no-conditional-in-test
         if (path.endsWith('README.mdx')) {
           return Promise.resolve(mockReadmeContent);
         }

packages/design-system-react/src/components/Icon/template.test.ts

@@ -13,7 +13,7 @@ describe('template', () => {
 
     const mockTpl = (strings: TemplateStringsArray, ...values: unknown[]) => {
       return strings.reduce((result, str, i) => {
-        // eslint-disable-next-line @typescript-eslint/no-base-to-string, jest/no-conditional-in-test
+        // eslint-disable-next-line @typescript-eslint/no-base-to-string
         return result + str + String(values[i] || '');
       }, '');
     };

packages/design-system-tailwind-preset/src/colors.test.ts

@@ -50,7 +50,7 @@ describe('Color Preset', () => {
 
     // Filter out variables that are either used or explicitly ignored
     const unusedVariables = Array.from(designTokens).filter(
-      (varName) => !usedSet.has(varName) && !ignoredSet.has(varName), // eslint-disable-line jest/no-conditional-in-test -- Legitimate data processing for test setup
+      (varName) => !usedSet.has(varName) && !ignoredSet.has(varName),
     );
 
     // Expect no unused variables

packages/design-system-tailwind-preset/src/index.test.ts

@@ -34,7 +34,6 @@ describe('Tailwind Preset', () => {
     }) => Record<string, string>;
 
     const result = textColorFn({
-      // eslint-disable-next-line jest/no-conditional-in-test -- Mocking Tailwind's theme function behavior
       theme: (path: string) => (path === 'colors' ? colors : {}),
     });
     expect(result).toStrictEqual(
@@ -52,7 +51,6 @@ describe('Tailwind Preset', () => {
     }) => Record<string, string>;
 
     const result = bgColorFn({
-      // eslint-disable-next-line jest/no-conditional-in-test -- Mocking Tailwind's theme function behavior
       theme: (path: string) => (path === 'colors' ? colors : {}),
     });
     expect(result).toStrictEqual(
@@ -70,7 +68,6 @@ describe('Tailwind Preset', () => {
     }) => Record<string, string>;
 
     const result = borderColorFn({
-      // eslint-disable-next-line jest/no-conditional-in-test -- Mocking Tailwind's theme function behavior
       theme: (path: string) => (path === 'colors' ? colors : {}),
     });
     expect(result).toStrictEqual(

packages/design-system-tailwind-preset/src/typography.test.ts

@@ -125,7 +125,7 @@ describe('Typography', () => {
 
     // Filter out variables that are either used or explicitly ignored
     const unusedVariables = Array.from(designTokens).filter(
-      (varName) => !usedSet.has(varName) && !ignoredSet.has(varName), // eslint-disable-line jest/no-conditional-in-test -- Legitimate data processing for test setup
+      (varName) => !usedSet.has(varName) && !ignoredSet.has(varName),
     );
 
     // Expect no unused variables

scripts/create-package/utils.test.ts

@@ -43,15 +43,13 @@ describe('create-package/utils', () => {
     it('should read the expected monorepo files', async () => {
       (fs.promises.readFile as jest.Mock).mockImplementation(
         async (filePath: string) => {
-          // eslint-disable-next-line jest/no-conditional-in-test
           switch (path.basename(filePath)) {
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
             case MonorepoFiles.TsConfig:
               return tsConfig;
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
+
             case MonorepoFiles.TsConfigBuild:
               return tsConfigBuild;
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
+
             case MonorepoFiles.PackageJson:
               return packageJson;
             default:

scripts/create-package/utils.ts

@@ -199,19 +199,18 @@ function processTemplateContent(
 
   return content.replace(allPlaceholdersRegex, (match) => {
     switch (match) {
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
       case Placeholders.CurrentYear:
         return currentYear;
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
+
       case Placeholders.NodeVersions:
         return nodeVersions;
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
+
       case Placeholders.PackageName:
         return name;
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
+
       case Placeholders.PackageDescription:
         return description;
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
+
       case Placeholders.PackageDirectoryName:
         return packageData.directoryName;
       /* istanbul ignore next: should be impossible */