Clean up eslint config for v10 upgrade

- Remove unused globals (__BASE_PATH__, __PATH_PREFIX__, graphql) — not referenced anywhere in codebase
- Remove globals package and languageOptions.globals — only TS files are linted, and TypeScript handles undefined variable checking
- Remove all formatting rules (comma-dangle, quotes, semi, max-len, etc.) — Prettier handles formatting
- Remove content/** max-len override — base max-len rule was removed, making this a no-op
- Remove eslint-config-prettier — nothing left for it to disable after formatting rules removed
- Remove eslint-plugin-security — plugin was registered but zero rules were enabled, effectively a no-op
- Remove duplicate parser declarations — tseslint.configs.recommended already sets the parser
- Remove duplicate await-thenable rule — already included in tseslint.configs.recommended
- Remove settings.polyfills — config for eslint-plugin-compat which is not a dependency
- Remove settings['import/parsers'] (2 instances) — config for eslint-plugin-import which is not a dependency
- Remove parserOptions.requireConfigFile — babel-eslint option, not used by typescript-eslint parser
- Remove parserOptions.ecmaFeatures.modules — legacy option, redundant with sourceType: 'module'
- Remove ecmaVersion and sourceType — ESLint 10 defaults to 'latest' and 'module'
- Remove sourceType: 'script' on .ts file block — was incorrect for TypeScript modules
- Remove **/node_modules from ignores — ESLint ignores node_modules by default
- Remove .scss import sort group — no .scss imports exist in codebase
- Remove dead eslint-disable-next-line max-len comments — rule no longer exists

Author: dhogan8

bin/_headers.config.ts

@@ -22,7 +22,6 @@ const config: HeadersConfig = {
             'https://status.maxmind.com',
             'https://www.maxmind.com',
 
-            // eslint-disable-next-line max-len
             // https://knowledge.hubspot.com/domains-and-urls/ssl-and-domain-security-in-hubspot#content-security-policy
 
             // HubSpot API
@@ -33,7 +32,6 @@ const config: HeadersConfig = {
 
             'https://*.googleapis.com',
 
-            // eslint-disable-next-line max-len
             // https://developers.google.com/tag-platform/security/guides/csp#google_analytics_4_google_analytics
             'https://*.google-analytics.com',
             'https://*.analytics.google.com',
@@ -57,7 +55,6 @@ const config: HeadersConfig = {
           'frame-src': [
             "'self'",
 
-            // eslint-disable-next-line max-len
             // https://knowledge.hubspot.com/domains-and-urls/ssl-and-domain-security-in-hubspot#content-security-policy
 
             // HubSpot calls-to-action (pop-ups) and chatflows
@@ -76,7 +73,6 @@ const config: HeadersConfig = {
             "'report-sample'",
             "'unsafe-inline'",
 
-            // eslint-disable-next-line max-len
             // https://knowledge.hubspot.com/domains-and-urls/ssl-and-domain-security-in-hubspot#content-security-policy
 
             // HubSpot tracking code

eslint.config.mjs

@@ -1,134 +1,25 @@
 import js from '@eslint/js';
-import prettier from 'eslint-config-prettier';
-import security from 'eslint-plugin-security';
 import simpleImportSort from 'eslint-plugin-simple-import-sort';
-import globals from 'globals';
 import tseslint from 'typescript-eslint';
 
 export default tseslint.config(
   js.configs.recommended,
   ...tseslint.configs.recommended,
-  prettier,
   {
     ignores: [
       '**/*.md',
       '**/.cache',
-      '**/node_modules',
       '**/public',
       'eslint.config.mjs',
       'bin/format-rawhtml.ts'
     ],
   },
   {
     plugins: {
-      security,
       'simple-import-sort': simpleImportSort,
     },
 
-    languageOptions: {
-      globals: {
-        ...globals.browser,
-        ...globals.node,
-        __BASE_PATH__: true,
-        __PATH_PREFIX__: true,
-        graphql: true,
-      },
-
-      parser: tseslint.parser,
-      ecmaVersion: 6,
-      sourceType: 'module',
-
-      parserOptions: {
-        ecmaFeatures: {
-          modules: true,
-        },
-
-        requireConfigFile: false,
-      },
-    },
-
-    settings: {
-      'import/parsers': {
-        '@typescript-eslint/parser': [
-          '.ts',
-        ],
-      },
-
-      polyfills: [
-        'CustomEvent',
-        'fetch',
-        'IntersectionObserver',
-        'Promise.all',
-        'Promise.race',
-        'Promise.resolve',
-        'URLSearchParams',
-      ],
-    },
-
     rules: {
-      'comma-dangle': [
-        'warn',
-        'always-multiline',
-      ],
-      'eol-last': [
-        'warn',
-        'always',
-      ],
-
-      'max-len': [
-        'warn',
-        {
-          code: 100,
-        },
-      ],
-
-      'no-trailing-spaces': 'warn',
-
-      'object-curly-newline': [
-        'warn',
-        {
-          ExportDeclaration: 'never',
-
-          ImportDeclaration: {
-            multiline: true,
-          },
-
-          ObjectExpression: {
-            minProperties: 1,
-            multiline: true,
-          },
-
-          ObjectPattern: {
-            multiline: true,
-          },
-        },
-      ],
-
-      'object-curly-spacing': [
-        'warn',
-        'always',
-        {
-          objectsInObjects: false,
-        },
-      ],
-
-      'object-property-newline': 'warn',
-      'quote-props': [
-        'warn',
-        'as-needed',
-      ],
-      quotes: [
-        'warn',
-        'single',
-        {
-          avoidEscape: true,
-        },
-      ],
-      semi: [
-        1,
-        'always',
-      ],
-
       'simple-import-sort/imports': [
         'warn',
         {
@@ -145,56 +36,27 @@ export default tseslint.config(
             [
               '^\\.',
             ],
-            [
-              '\\.scss$',
-            ],
           ],
         },
       ],
       '@typescript-eslint/no-unused-expressions': ['error', {
-        allowTernary: true
-    }],
+        allowTernary: true,
+      }],
     },
-  },
-  {
-  files: [
-    'content/**',
-  ],
-
-  rules: {
-    'max-len': [
-      0,
-    ],
-  },
-  },
-  {
+  }, {
   files: [
     '**/*.ts',
   ],
 
   languageOptions: {
-    parser: tseslint.parser,
-    ecmaVersion: 5,
-    sourceType: 'script',
-
     parserOptions: {
       project: [
         'tsconfig.json',
       ],
     },
   },
 
-  settings: {
-    'import/parsers': {
-      '@typescript-eslint/parser': [
-        '.ts',
-      ],
-    },
-  },
-
   rules: {
-    '@typescript-eslint/await-thenable': 'error',
-
     '@typescript-eslint/naming-convention': ['error',
       {
         selector: 'default', format: ['camelCase'],

package.json

@@ -11,10 +11,7 @@
     "baseline-browser-mapping": "^2.10.0",
     "cspell": "^9.7.0",
     "eslint": "^10.0.3",
-    "eslint-config-prettier": "^10.1.8",
-    "eslint-plugin-security": "^4.0.0",
     "eslint-plugin-simple-import-sort": "^12.1.1",
-    "globals": "^17.4.0",
     "npm-run-all": "^4.1.5",
     "prettier": "^3.8.1",
     "prettier-plugin-go-template": "^0.0.15",