Upgrade ESLint to v9 with flat config

- Upgrade eslint from v8 to v9.18.0 - Migrate from .eslintrc.js to eslint.config.mjs (flat config) - Upgrade typescript-eslint to v8.20.0 (unified package) - Replace eslint-plugin-import with eslint-plugin-import-x for flat config support - Add globals package for environment globals - Update eslint-plugin-bsky-internal with proper meta objects for ESLint v9 - Fix deprecated context.getScope() API usage - Update bskyembed to use flat config - Remove deprecated --ext flag from lint scripts - Configure rules to maintain previous behavior while using new ESLint version
2026-01-10 22:31:13 +00:00
parent 3b98f2f8ab
commit 0cd99c841e
21 changed files with 1727 additions and 2097 deletions
@@ -1,127 +0,0 @@
-module.exports = {
-  root: true,
-  extends: [
-    '@react-native',
-    'plugin:react/recommended',
-    'plugin:react/jsx-runtime',
-    'plugin:react-native-a11y/ios',
-    'prettier',
-  ],
-  parser: '@typescript-eslint/parser',
-  plugins: [
-    '@typescript-eslint',
-    'react',
-    'lingui',
-    'simple-import-sort',
-    'bsky-internal',
-    'eslint-plugin-react-compiler',
-    'import',
-  ],
-  rules: {
-    'react/no-unescaped-entities': 0,
-    'react/prop-types': 0,
-    'react-native/no-inline-styles': 0,
-    'bsky-internal/avoid-unwrapped-text': [
-      'error',
-      {
-        impliedTextComponents: [
-          'H1',
-          'H2',
-          'H3',
-          'H4',
-          'H5',
-          'H6',
-          'P',
-          'Admonition',
-          'Admonition.Admonition',
-          'Toast.Action',
-          'toast.Action',
-          'AgeAssuranceAdmonition',
-          'Span',
-          'StackedButton',
-        ],
-        impliedTextProps: [],
-        suggestedTextWrappers: {
-          Button: 'ButtonText',
-          'ToggleButton.Button': 'ToggleButton.ButtonText',
-          'SegmentedControl.Item': 'SegmentedControl.ItemText',
-        },
-      },
-    ],
-    'bsky-internal/use-exact-imports': 'error',
-    'bsky-internal/use-typed-gates': 'error',
-    'bsky-internal/use-prefixed-imports': 'error',
-    'simple-import-sort/imports': [
-      'error',
-      {
-        groups: [
-          // Side effect imports.
-          ['^\\u0000'],
-          // Node.js builtins prefixed with `node:`.
-          ['^node:'],
-          // Packages.
-          // Things that start with a letter (or digit or underscore), or `@` followed by a letter.
-          // React/React Native priortized, followed by expo
-          // Followed by all packages excluding unprefixed relative ones
-          [
-            '^(react\\/(.*)$)|^(react$)|^(react-native(.*)$)',
-            '^(expo(.*)$)|^(expo$)',
-            '^(?!(?:alf|components|lib|locale|logger|platform|screens|state|view)(?:$|\\/))@?\\w',
-          ],
-          // Relative imports.
-          // Ideally, anything that starts with a dot or #
-          // due to unprefixed relative imports being used, we whitelist the relative paths we use
-          // (?:$|\\/) matches end of string or /
-          [
-            '^(?:#\\/)?(?:lib|state|logger|platform|locale)(?:$|\\/)',
-            '^(?:#\\/)?view(?:$|\\/)',
-            '^(?:#\\/)?screens(?:$|\\/)',
-            '^(?:#\\/)?alf(?:$|\\/)',
-            '^(?:#\\/)?components(?:$|\\/)',
-            '^#\\/',
-            '^\\.',
-          ],
-          // anything else - hopefully we don't have any of these
-          ['^'],
-        ],
-      },
-    ],
-    'simple-import-sort/exports': 'error',
-    'react-compiler/react-compiler': 'warn',
-    'no-unused-vars': 'off',
-    '@typescript-eslint/no-unused-vars': [
-      'error',
-      {argsIgnorePattern: '^_', varsIgnorePattern: '^_.+'},
-    ],
-    '@typescript-eslint/consistent-type-imports': [
-      'warn',
-      {prefer: 'type-imports', fixStyle: 'inline-type-imports'},
-    ],
-    'import/consistent-type-specifier-style': ['warn', 'prefer-inline'],
-  },
-  ignorePatterns: [
-    '**/__mocks__/*.ts',
-    'src/platform/polyfills.ts',
-    'src/third-party',
-    'ios',
-    'android',
-    'coverage',
-    '*.lock',
-    '.husky',
-    'patches',
-    '*.html',
-    'bskyweb',
-    'bskyembed',
-    'src/locale/locales/_build/',
-    'src/locale/locales/**/*.js',
-    '*.e2e.ts',
-    '*.e2e.tsx',
-  ],
-  settings: {
-    componentWrapperFunctions: ['observer'],
-  },
-  parserOptions: {
-    sourceType: 'module',
-    ecmaVersion: 'latest',
-  },
-}
@@ -1,22 +0,0 @@
-module.exports = {
-  root: true,
-  parser: '@typescript-eslint/parser',
-  plugins: ['@typescript-eslint', 'simple-import-sort'],
-  extends: [
-    'eslint:recommended',
-    'preact',
-    'plugin:@typescript-eslint/recommended',
-    'plugin:@typescript-eslint/recommended-requiring-type-checking',
-  ],
-  rules: {
-    'simple-import-sort/imports': 'warn',
-    'simple-import-sort/exports': 'warn',
-    'no-else-return': 'off',
-  },
-  parserOptions: {
-    sourceType: 'module',
-    ecmaVersion: 'latest',
-    project: ['./tsconfig.json'],
-    tsconfigRootDir: __dirname,
-  },
-}
@@ -0,0 +1,52 @@
+// @ts-check
+import js from '@eslint/js'
+import tseslint from 'typescript-eslint'
+import simpleImportSort from 'eslint-plugin-simple-import-sort'
+import globals from 'globals'
+
+export default tseslint.config(
+  // Global ignores
+  {
+    ignores: ['dist/**', 'node_modules/**'],
+  },
+
+  // Base JS recommended rules
+  js.configs.recommended,
+
+  // TypeScript rules with type checking
+  ...tseslint.configs.recommendedTypeChecked,
+
+  // Main configuration
+  {
+    files: ['**/*.{js,jsx,ts,tsx}'],
+    plugins: {
+      'simple-import-sort': simpleImportSort,
+    },
+    languageOptions: {
+      ecmaVersion: 'latest',
+      sourceType: 'module',
+      globals: {
+        ...globals.browser,
+      },
+      parserOptions: {
+        projectService: true,
+        tsconfigRootDir: import.meta.dirname,
+      },
+    },
+    rules: {
+      'simple-import-sort/imports': 'warn',
+      'simple-import-sort/exports': 'warn',
+      'no-else-return': 'off',
+      '@typescript-eslint/no-require-imports': 'off',
+      '@typescript-eslint/no-unused-vars': [
+        'error',
+        {
+          argsIgnorePattern: '^_',
+          varsIgnorePattern: '^_',
+          caughtErrors: 'none',
+          ignoreRestSiblings: true,
+        },
+      ],
+    },
+  },
+)
@@ -7,7 +7,7 @@
    "dev-snippet": "tsc --project tsconfig.snippet.json && serve -s dist -p 3000 -n",
    "build": "tsc && vite build",
    "build-snippet": "tsc --project tsconfig.snippet.json",
-    "lint": "eslint --cache --ext .js,.jsx,.ts,.tsx src",
+    "lint": "eslint --cache src",
    "typecheck": "tsc --noEmit"
  },
  "dependencies": {
@@ -17,11 +17,13 @@
  "devDependencies": {
    "@preact/preset-vite": "^2.10.2",
    "@vitejs/plugin-legacy": "^7.0.0",
+    "@eslint/js": "^9.18.0",
    "autoprefixer": "^10.4.19",
-    "eslint": "^8.19.0",
-    "eslint-config-preact": "^1.3.0",
-    "eslint-plugin-simple-import-sort": "^12.0.0",
+    "eslint": "^9.18.0",
+    "eslint-plugin-simple-import-sort": "^12.1.1",
+    "globals": "^15.14.0",
    "postcss": "^8.4.38",
+    "typescript-eslint": "^8.20.0",
    "serve": "^14.2.5",
    "tailwindcss": "^3.4.3",
    "terser": "^5.43.1",
@@ -0,0 +1,213 @@
+// @ts-check
+import js from '@eslint/js'
+import tseslint from 'typescript-eslint'
+import react from 'eslint-plugin-react'
+import reactHooks from 'eslint-plugin-react-hooks'
+import reactNative from 'eslint-plugin-react-native'
+import reactNativeA11y from 'eslint-plugin-react-native-a11y'
+import simpleImportSort from 'eslint-plugin-simple-import-sort'
+import importX from 'eslint-plugin-import-x'
+import lingui from 'eslint-plugin-lingui'
+import reactCompiler from 'eslint-plugin-react-compiler'
+import bskyInternal from 'eslint-plugin-bsky-internal'
+import globals from 'globals'
+
+export default tseslint.config(
+  // Global ignores
+  {
+    ignores: [
+      '**/__mocks__/*.ts',
+      'src/platform/polyfills.ts',
+      'src/third-party/**',
+      'ios/**',
+      'android/**',
+      'coverage/**',
+      '*.lock',
+      '.husky/**',
+      'patches/**',
+      '*.html',
+      'bskyweb/**',
+      'bskyembed/**',
+      'src/locale/locales/_build/**',
+      'src/locale/locales/**/*.js',
+      '*.e2e.ts',
+      '*.e2e.tsx',
+      'eslint.config.mjs',
+    ],
+  },
+
+  // Base JS recommended rules
+  js.configs.recommended,
+
+  // TypeScript rules
+  ...tseslint.configs.recommended,
+
+  // Main configuration for all JS/TS/JSX/TSX files
+  {
+    files: ['**/*.{js,jsx,ts,tsx}'],
+    plugins: {
+      react,
+      'react-hooks': reactHooks,
+      'react-native': reactNative,
+      'react-native-a11y': reactNativeA11y,
+      'simple-import-sort': simpleImportSort,
+      'import-x': importX,
+      lingui,
+      'react-compiler': reactCompiler,
+      'bsky-internal': bskyInternal,
+    },
+    languageOptions: {
+      ecmaVersion: 'latest',
+      sourceType: 'module',
+      globals: {
+        ...globals.browser,
+        ...globals.node,
+        ...globals.es2021,
+      },
+      parserOptions: {
+        ecmaFeatures: {
+          jsx: true,
+        },
+      },
+    },
+    settings: {
+      react: {
+        version: 'detect',
+      },
+      componentWrapperFunctions: ['observer'],
+    },
+    rules: {
+      // React rules
+      ...react.configs.recommended.rules,
+      ...react.configs['jsx-runtime'].rules,
+      'react/no-unescaped-entities': 'off',
+      'react/prop-types': 'off',
+
+      // React Hooks rules
+      ...reactHooks.configs.recommended.rules,
+
+      // React Native rules
+      'react-native/no-inline-styles': 'off',
+
+      // React Native A11y rules (iOS focused)
+      ...reactNativeA11y.configs.ios.rules,
+
+      // Bsky internal rules
+      'bsky-internal/avoid-unwrapped-text': [
+        'error',
+        {
+          impliedTextComponents: [
+            'H1',
+            'H2',
+            'H3',
+            'H4',
+            'H5',
+            'H6',
+            'P',
+            'Admonition',
+            'Admonition.Admonition',
+            'Toast.Action',
+            'AgeAssuranceAdmonition',
+            'Span',
+            'StackedButton',
+          ],
+          impliedTextProps: [],
+          suggestedTextWrappers: {
+            Button: 'ButtonText',
+            'ToggleButton.Button': 'ToggleButton.ButtonText',
+            'SegmentedControl.Item': 'SegmentedControl.ItemText',
+          },
+        },
+      ],
+      'bsky-internal/use-exact-imports': 'error',
+      'bsky-internal/use-typed-gates': 'error',
+      'bsky-internal/use-prefixed-imports': 'error',
+
+      // Import sorting
+      'simple-import-sort/imports': [
+        'error',
+        {
+          groups: [
+            // Side effect imports.
+            ['^\\u0000'],
+            // Node.js builtins prefixed with `node:`.
+            ['^node:'],
+            // Packages.
+            // Things that start with a letter (or digit or underscore), or `@` followed by a letter.
+            // React/React Native prioritized, followed by expo
+            // Followed by all packages excluding unprefixed relative ones
+            [
+              '^(react\\/(.*)$)|^(react$)|^(react-native(.*)$)',
+              '^(expo(.*)$)|^(expo$)',
+              '^(?!(?:alf|components|lib|locale|logger|platform|screens|state|view)(?:$|\\/))@?\\w',
+            ],
+            // Relative imports.
+            // Ideally, anything that starts with a dot or #
+            // due to unprefixed relative imports being used, we whitelist the relative paths we use
+            // (?:$|\\/) matches end of string or /
+            [
+              '^(?:#\\/)?(?:lib|state|logger|platform|locale)(?:$|\\/)',
+              '^(?:#\\/)?view(?:$|\\/)',
+              '^(?:#\\/)?screens(?:$|\\/)',
+              '^(?:#\\/)?alf(?:$|\\/)',
+              '^(?:#\\/)?components(?:$|\\/)',
+              '^#\\/',
+              '^\\.',
+            ],
+            // anything else - hopefully we don't have any of these
+            ['^'],
+          ],
+        },
+      ],
+      'simple-import-sort/exports': 'error',
+
+      // React Compiler
+      'react-compiler/react-compiler': 'warn',
+
+      // TypeScript rules
+      'no-unused-vars': 'off',
+      '@typescript-eslint/no-unused-vars': [
+        'error',
+        {
+          argsIgnorePattern: '^_',
+          varsIgnorePattern: '^_',
+          caughtErrors: 'none',
+          ignoreRestSiblings: true,
+        },
+      ],
+      '@typescript-eslint/consistent-type-imports': [
+        'warn',
+        {prefer: 'type-imports', fixStyle: 'inline-type-imports'},
+      ],
+      '@typescript-eslint/no-require-imports': 'off',
+      // Maintain previous behavior - these are stricter in typescript-eslint v8
+      '@typescript-eslint/no-explicit-any': 'off',
+      '@typescript-eslint/ban-ts-comment': 'off',
+      '@typescript-eslint/no-empty-object-type': 'off',
+      '@typescript-eslint/no-unused-expressions': 'off',
+      '@typescript-eslint/no-non-null-asserted-optional-chain': 'off',
+      '@typescript-eslint/no-wrapper-object-types': 'off',
+      '@typescript-eslint/no-unsafe-function-type': 'off',
+
+      // Import rules
+      'import-x/consistent-type-specifier-style': ['warn', 'prefer-inline'],
+
+      // Turn off rules that weren't enforced in previous config
+      'no-empty-pattern': 'off',
+      'no-async-promise-executor': 'off',
+      'no-constant-binary-expression': 'warn',
+      'prefer-const': 'off',
+      'no-empty': 'off',
+      'no-unsafe-optional-chaining': 'off',
+      'no-prototype-builtins': 'off',
+      'no-var': 'off',
+      'prefer-rest-params': 'off',
+      'no-case-declarations': 'off',
+      'no-irregular-whitespace': 'off',
+      'no-useless-escape': 'off',
+      'no-sparse-arrays': 'off',
+      'no-fallthrough': 'off',
+      'no-control-regex': 'off',
+    },
+  },
+)
@@ -29,303 +29,331 @@ function getTagName(node) {
  return reversedIdentifiers.reverse().join('.')
 }

-exports.create = function create(context) {
-  const options = context.options[0] || {}
-  const impliedTextProps = options.impliedTextProps ?? []
-  const impliedTextComponents = options.impliedTextComponents ?? []
-  const suggestedTextWrappers = options.suggestedTextWrappers ?? {}
-  const textProps = [...impliedTextProps]
-  const textComponents = ['Text', ...impliedTextComponents]
+module.exports = {
+  meta: {
+    type: 'problem',
+    docs: {
+      description: 'Enforce text strings are wrapped in <Text> components',
+    },
+    schema: [
+      {
+        type: 'object',
+        properties: {
+          impliedTextComponents: {
+            type: 'array',
+            items: {type: 'string'},
+          },
+          impliedTextProps: {
+            type: 'array',
+            items: {type: 'string'},
+          },
+          suggestedTextWrappers: {
+            type: 'object',
+            additionalProperties: {type: 'string'},
+          },
+        },
+        additionalProperties: false,
+      },
+    ],
+  },
+  create(context) {
+    const options = context.options[0] || {}
+    const impliedTextProps = options.impliedTextProps ?? []
+    const impliedTextComponents = options.impliedTextComponents ?? []
+    const suggestedTextWrappers = options.suggestedTextWrappers ?? {}
+    const textProps = [...impliedTextProps]
+    const textComponents = ['Text', ...impliedTextComponents]

-  function isTextComponent(tagName) {
-    return textComponents.includes(tagName) || tagName.endsWith('Text')
-  }
+    function isTextComponent(tagName) {
+      return textComponents.includes(tagName) || tagName.endsWith('Text')
+    }

-  return {
-    JSXText(node) {
-      if (typeof node.value !== 'string' || hasOnlyLineBreak(node.value)) {
-        return
-      }
-      let parent = node.parent
-      while (parent) {
-        if (parent.type === 'JSXElement') {
-          const tagName = getTagName(parent)
-          if (isTextComponent(tagName)) {
-            // We're good.
-            return
-          }
-          if (tagName === 'Trans') {
-            // Exit and rely on the traversal for <Trans> JSXElement (code below).
-            // TODO: Maybe validate that it's present.
-            return
-          }
-          const suggestedWrapper = suggestedTextWrappers[tagName]
-          let message = `Wrap this string in <${suggestedWrapper ?? 'Text'}>.`
-          if (tagName !== 'View' && !suggestedWrapper) {
-            message +=
-              ' If <' +
-              tagName +
-              '> is guaranteed to render <Text>, ' +
-              'rename it to <' +
-              tagName +
-              'Text> or add it to impliedTextComponents.'
-          }
-          context.report({
-            node,
-            message,
-          })
+    return {
+      JSXText(node) {
+        if (typeof node.value !== 'string' || hasOnlyLineBreak(node.value)) {
          return
        }
+        let parent = node.parent
+        while (parent) {
+          if (parent.type === 'JSXElement') {
+            const tagName = getTagName(parent)
+            if (isTextComponent(tagName)) {
+              // We're good.
+              return
+            }
+            if (tagName === 'Trans') {
+              // Exit and rely on the traversal for <Trans> JSXElement (code below).
+              // TODO: Maybe validate that it's present.
+              return
+            }
+            const suggestedWrapper = suggestedTextWrappers[tagName]
+            let message = `Wrap this string in <${suggestedWrapper ?? 'Text'}>.`
+            if (tagName !== 'View' && !suggestedWrapper) {
+              message +=
+                ' If <' +
+                tagName +
+                '> is guaranteed to render <Text>, ' +
+                'rename it to <' +
+                tagName +
+                'Text> or add it to impliedTextComponents.'
+            }
+            context.report({
+              node,
+              message,
+            })
+            return
+          }

-        if (
-          parent.type === 'JSXAttribute' &&
-          parent.name.type === 'JSXIdentifier' &&
-          parent.parent.type === 'JSXOpeningElement' &&
-          parent.parent.parent.type === 'JSXElement'
-        ) {
-          const tagName = getTagName(parent.parent.parent)
-          const propName = parent.name.name
          if (
-            textProps.includes(tagName + ' ' + propName) ||
-            propName === 'text' ||
-            propName.endsWith('Text')
+            parent.type === 'JSXAttribute' &&
+            parent.name.type === 'JSXIdentifier' &&
+            parent.parent.type === 'JSXOpeningElement' &&
+            parent.parent.parent.type === 'JSXElement'
          ) {
-            // We're good.
+            const tagName = getTagName(parent.parent.parent)
+            const propName = parent.name.name
+            if (
+              textProps.includes(tagName + ' ' + propName) ||
+              propName === 'text' ||
+              propName.endsWith('Text')
+            ) {
+              // We're good.
+              return
+            }
+            const message =
+              'Wrap this string in <Text>.' +
+              ' If `' +
+              propName +
+              '` is guaranteed to be wrapped in <Text>, ' +
+              'rename it to `' +
+              propName +
+              'Text' +
+              '` or add it to impliedTextProps.'
+            context.report({
+              node,
+              message,
+            })
            return
          }
-          const message =
-            'Wrap this string in <Text>.' +
-            ' If `' +
-            propName +
-            '` is guaranteed to be wrapped in <Text>, ' +
-            'rename it to `' +
-            propName +
-            'Text' +
-            '` or add it to impliedTextProps.'
-          context.report({
-            node,
-            message,
-          })
+
+          parent = parent.parent
+          continue
+        }
+      },
+      Literal(node) {
+        if (typeof node.value !== 'string' && typeof node.value !== 'number') {
          return
        }
-
-        parent = parent.parent
-        continue
-      }
-    },
-    Literal(node) {
-      if (typeof node.value !== 'string' && typeof node.value !== 'number') {
-        return
-      }
-      let parent = node.parent
-      while (parent) {
-        if (parent.type === 'JSXElement') {
-          const tagName = getTagName(parent)
-          if (isTextComponent(tagName)) {
-            // We're good.
+        let parent = node.parent
+        while (parent) {
+          if (parent.type === 'JSXElement') {
+            const tagName = getTagName(parent)
+            if (isTextComponent(tagName)) {
+              // We're good.
+              return
+            }
+            if (tagName === 'Trans') {
+              // Exit and rely on the traversal for <Trans> JSXElement (code below).
+              // TODO: Maybe validate that it's present.
+              return
+            }
+            const suggestedWrapper = suggestedTextWrappers[tagName]
+            let message = `Wrap this string in <${suggestedWrapper ?? 'Text'}>.`
+            if (tagName !== 'View' && !suggestedWrapper) {
+              message +=
+                ' If <' +
+                tagName +
+                '> is guaranteed to render <Text>, ' +
+                'rename it to <' +
+                tagName +
+                'Text> or add it to impliedTextComponents.'
+            }
+            context.report({
+              node,
+              message,
+            })
            return
          }
-          if (tagName === 'Trans') {
-            // Exit and rely on the traversal for <Trans> JSXElement (code below).
-            // TODO: Maybe validate that it's present.
-            return
-          }
-          const suggestedWrapper = suggestedTextWrappers[tagName]
-          let message = `Wrap this string in <${suggestedWrapper ?? 'Text'}>.`
-          if (tagName !== 'View' && !suggestedWrapper) {
-            message +=
-              ' If <' +
-              tagName +
-              '> is guaranteed to render <Text>, ' +
-              'rename it to <' +
-              tagName +
-              'Text> or add it to impliedTextComponents.'
-          }
-          context.report({
-            node,
-            message,
-          })
-          return
-        }

-        if (parent.type === 'BinaryExpression' && parent.operator === '+') {
-          parent = parent.parent
-          continue
-        }
-
-        if (
-          parent.type === 'JSXExpressionContainer' ||
-          parent.type === 'LogicalExpression'
-        ) {
-          parent = parent.parent
-          continue
-        }
-
-        // Be conservative for other types.
-        return
-      }
-    },
-    TemplateLiteral(node) {
-      let parent = node.parent
-      while (parent) {
-        if (parent.type === 'JSXElement') {
-          const tagName = getTagName(parent)
-          if (isTextComponent(tagName)) {
-            // We're good.
-            return
+          if (parent.type === 'BinaryExpression' && parent.operator === '+') {
+            parent = parent.parent
+            continue
          }
-          if (tagName === 'Trans') {
-            // Exit and rely on the traversal for <Trans> JSXElement (code below).
-            // TODO: Maybe validate that it's present.
-            return
-          }
-          const suggestedWrapper = suggestedTextWrappers[tagName]
-          let message = `Wrap this string in <${suggestedWrapper ?? 'Text'}>.`
-          if (tagName !== 'View' && !suggestedWrapper) {
-            message +=
-              ' If <' +
-              tagName +
-              '> is guaranteed to render <Text>, ' +
-              'rename it to <' +
-              tagName +
-              'Text> or add it to impliedTextComponents.'
-          }
-          context.report({
-            node,
-            message,
-          })
-          return
-        }

-        if (
-          parent.type === 'CallExpression' &&
-          parent.callee.type === 'Identifier' &&
-          parent.callee.name === '_'
-        ) {
-          // This is a user-facing string, keep going up.
-          parent = parent.parent
-          continue
-        }
-
-        if (parent.type === 'BinaryExpression' && parent.operator === '+') {
-          parent = parent.parent
-          continue
-        }
-
-        if (
-          parent.type === 'JSXExpressionContainer' ||
-          parent.type === 'LogicalExpression' ||
-          parent.type === 'TaggedTemplateExpression'
-        ) {
-          parent = parent.parent
-          continue
-        }
-
-        // Be conservative for other types.
-        return
-      }
-    },
-    JSXElement(node) {
-      if (getTagName(node) !== 'Trans') {
-        return
-      }
-      let parent = node.parent
-      while (parent) {
-        if (parent.type === 'JSXElement') {
-          const tagName = getTagName(parent)
-          if (isTextComponent(tagName)) {
-            // We're good.
-            return
-          }
-          if (tagName === 'Trans') {
-            // Exit and rely on the traversal for this JSXElement.
-            // TODO: Should nested <Trans> even be allowed?
-            return
-          }
-          const suggestedWrapper = suggestedTextWrappers[tagName]
-          let message = `Wrap this <Trans> in <${suggestedWrapper ?? 'Text'}>.`
-          if (tagName !== 'View' && !suggestedWrapper) {
-            message +=
-              ' If <' +
-              tagName +
-              '> is guaranteed to render <Text>, ' +
-              'rename it to <' +
-              tagName +
-              'Text> or add it to impliedTextComponents.'
-          }
-          context.report({
-            node,
-            message,
-          })
-          return
-        }
-
-        if (
-          parent.type === 'JSXAttribute' &&
-          parent.name.type === 'JSXIdentifier' &&
-          parent.parent.type === 'JSXOpeningElement' &&
-          parent.parent.parent.type === 'JSXElement'
-        ) {
-          const tagName = getTagName(parent.parent.parent)
-          const propName = parent.name.name
          if (
-            textProps.includes(tagName + ' ' + propName) ||
-            propName === 'text' ||
-            propName.endsWith('Text')
+            parent.type === 'JSXExpressionContainer' ||
+            parent.type === 'LogicalExpression'
          ) {
-            // We're good.
-            return
+            parent = parent.parent
+            continue
          }
-          const message =
-            'Wrap this <Trans> in <Text>.' +
-            ' If `' +
-            propName +
-            '` is guaranteed to be wrapped in <Text>, ' +
-            'rename it to `' +
-            propName +
-            'Text' +
-            '` or add it to impliedTextProps.'
-          context.report({
-            node,
-            message,
-          })
+
+          // Be conservative for other types.
          return
        }
+      },
+      TemplateLiteral(node) {
+        let parent = node.parent
+        while (parent) {
+          if (parent.type === 'JSXElement') {
+            const tagName = getTagName(parent)
+            if (isTextComponent(tagName)) {
+              // We're good.
+              return
+            }
+            if (tagName === 'Trans') {
+              // Exit and rely on the traversal for <Trans> JSXElement (code below).
+              // TODO: Maybe validate that it's present.
+              return
+            }
+            const suggestedWrapper = suggestedTextWrappers[tagName]
+            let message = `Wrap this string in <${suggestedWrapper ?? 'Text'}>.`
+            if (tagName !== 'View' && !suggestedWrapper) {
+              message +=
+                ' If <' +
+                tagName +
+                '> is guaranteed to render <Text>, ' +
+                'rename it to <' +
+                tagName +
+                'Text> or add it to impliedTextComponents.'
+            }
+            context.report({
+              node,
+              message,
+            })
+            return
+          }

-        parent = parent.parent
-        continue
-      }
-    },
-    ReturnStatement(node) {
-      let fnScope = context.getScope()
-      while (fnScope && fnScope.type !== 'function') {
-        fnScope = fnScope.upper
-      }
-      if (!fnScope) {
-        return
-      }
-      const fn = fnScope.block
-      if (!fn.id || fn.id.type !== 'Identifier' || !fn.id.name) {
-        return
-      }
-      if (!/^[A-Z]\w*Text$/.test(fn.id.name)) {
-        return
-      }
-      if (!node.argument || node.argument.type !== 'JSXElement') {
-        return
-      }
-      const openingEl = node.argument.openingElement
-      if (openingEl.name.type !== 'JSXIdentifier') {
-        return
-      }
-      const returnedComponentName = openingEl.name.name
-      if (!isTextComponent(returnedComponentName)) {
-        context.report({
-          node,
-          message:
-            'Components ending with *Text must return <Text> or <SomeText>.',
-        })
-      }
-    },
-  }
+          if (
+            parent.type === 'CallExpression' &&
+            parent.callee.type === 'Identifier' &&
+            parent.callee.name === '_'
+          ) {
+            // This is a user-facing string, keep going up.
+            parent = parent.parent
+            continue
+          }
+
+          if (parent.type === 'BinaryExpression' && parent.operator === '+') {
+            parent = parent.parent
+            continue
+          }
+
+          if (
+            parent.type === 'JSXExpressionContainer' ||
+            parent.type === 'LogicalExpression' ||
+            parent.type === 'TaggedTemplateExpression'
+          ) {
+            parent = parent.parent
+            continue
+          }
+
+          // Be conservative for other types.
+          return
+        }
+      },
+      JSXElement(node) {
+        if (getTagName(node) !== 'Trans') {
+          return
+        }
+        let parent = node.parent
+        while (parent) {
+          if (parent.type === 'JSXElement') {
+            const tagName = getTagName(parent)
+            if (isTextComponent(tagName)) {
+              // We're good.
+              return
+            }
+            if (tagName === 'Trans') {
+              // Exit and rely on the traversal for this JSXElement.
+              // TODO: Should nested <Trans> even be allowed?
+              return
+            }
+            const suggestedWrapper = suggestedTextWrappers[tagName]
+            let message = `Wrap this <Trans> in <${suggestedWrapper ?? 'Text'}>.`
+            if (tagName !== 'View' && !suggestedWrapper) {
+              message +=
+                ' If <' +
+                tagName +
+                '> is guaranteed to render <Text>, ' +
+                'rename it to <' +
+                tagName +
+                'Text> or add it to impliedTextComponents.'
+            }
+            context.report({
+              node,
+              message,
+            })
+            return
+          }
+
+          if (
+            parent.type === 'JSXAttribute' &&
+            parent.name.type === 'JSXIdentifier' &&
+            parent.parent.type === 'JSXOpeningElement' &&
+            parent.parent.parent.type === 'JSXElement'
+          ) {
+            const tagName = getTagName(parent.parent.parent)
+            const propName = parent.name.name
+            if (
+              textProps.includes(tagName + ' ' + propName) ||
+              propName === 'text' ||
+              propName.endsWith('Text')
+            ) {
+              // We're good.
+              return
+            }
+            const message =
+              'Wrap this <Trans> in <Text>.' +
+              ' If `' +
+              propName +
+              '` is guaranteed to be wrapped in <Text>, ' +
+              'rename it to `' +
+              propName +
+              'Text' +
+              '` or add it to impliedTextProps.'
+            context.report({
+              node,
+              message,
+            })
+            return
+          }
+
+          parent = parent.parent
+          continue
+        }
+      },
+      ReturnStatement(node) {
+        let fnScope = context.sourceCode.getScope(node)
+        while (fnScope && fnScope.type !== 'function') {
+          fnScope = fnScope.upper
+        }
+        if (!fnScope) {
+          return
+        }
+        const fn = fnScope.block
+        if (!fn.id || fn.id.type !== 'Identifier' || !fn.id.name) {
+          return
+        }
+        if (!/^[A-Z]\w*Text$/.test(fn.id.name)) {
+          return
+        }
+        if (!node.argument || node.argument.type !== 'JSXElement') {
+          return
+        }
+        const openingEl = node.argument.openingElement
+        if (openingEl.name.type !== 'JSXIdentifier') {
+          return
+        }
+        const returnedComponentName = openingEl.name.name
+        if (!isTextComponent(returnedComponentName)) {
+          context.report({
+            node,
+            message:
+              'Components ending with *Text must return <Text> or <SomeText>.',
+          })
+        }
+      },
+    }
+  },
 }
@@ -1,6 +1,10 @@
 'use strict'

-module.exports = {
+const plugin = {
+  meta: {
+    name: 'eslint-plugin-bsky-internal',
+    version: '1.0.0',
+  },
  rules: {
    'avoid-unwrapped-text': require('./avoid-unwrapped-text'),
    'use-exact-imports': require('./use-exact-imports'),
@@ -8,3 +12,5 @@ module.exports = {
    'use-prefixed-imports': require('./use-prefixed-imports'),
  },
 }
+
+module.exports = plugin
@@ -3,20 +3,29 @@ const BANNED_IMPORTS = [
  '@fortawesome/free-solid-svg-icons',
 ]

-exports.create = function create(context) {
-  return {
-    ImportDeclaration(node) {
-      const source = node.source
-      if (typeof source.value !== 'string') {
-        return
-      }
-      if (BANNED_IMPORTS.includes(source.value)) {
-        context.report({
-          node,
-          message:
-            'Import the specific thing you want instead of the entire package',
-        })
-      }
+module.exports = {
+  meta: {
+    type: 'suggestion',
+    docs: {
+      description: 'Prevent importing entire icon packages',
    },
-  }
+    schema: [],
+  },
+  create(context) {
+    return {
+      ImportDeclaration(node) {
+        const source = node.source
+        if (typeof source.value !== 'string') {
+          return
+        }
+        if (BANNED_IMPORTS.includes(source.value)) {
+          context.report({
+            node,
+            message:
+              'Import the specific thing you want instead of the entire package',
+          })
+        }
+      },
+    }
+  },
 }
@@ -13,7 +13,11 @@ const BANNED_IMPORT_PREFIXES = [
 module.exports = {
  meta: {
    type: 'suggestion',
+    docs: {
+      description: 'Enforce using prefixed imports for internal paths',
+    },
    fixable: 'code',
+    schema: [],
  },
  create(context) {
    return {
@@ -1,31 +1,41 @@
 'use strict'

-exports.create = function create(context) {
-  return {
-    ImportSpecifier(node) {
-      if (
-        !node.local ||
-        node.local.type !== 'Identifier' ||
-        node.local.name !== 'useGate'
-      ) {
-        return
-      }
-      if (
-        node.parent.type !== 'ImportDeclaration' ||
-        !node.parent.source ||
-        node.parent.source.type !== 'Literal'
-      ) {
-        return
-      }
-      const source = node.parent.source.value
-      if (source.startsWith('statsig') || source.startsWith('@statsig')) {
-        context.report({
-          node,
-          message:
-            "Use useGate() from '#/lib/statsig/statsig' instead of the one on npm.",
-        })
-      }
-      // TODO: Verify gate() call results aren't stored in variables.
+module.exports = {
+  meta: {
+    type: 'suggestion',
+    docs: {
+      description:
+        'Enforce using internal statsig wrapper instead of npm package',
    },
-  }
+    schema: [],
+  },
+  create(context) {
+    return {
+      ImportSpecifier(node) {
+        if (
+          !node.local ||
+          node.local.type !== 'Identifier' ||
+          node.local.name !== 'useGate'
+        ) {
+          return
+        }
+        if (
+          node.parent.type !== 'ImportDeclaration' ||
+          !node.parent.source ||
+          node.parent.source.type !== 'Literal'
+        ) {
+          return
+        }
+        const source = node.parent.source.value
+        if (source.startsWith('statsig') || source.startsWith('@statsig')) {
+          context.report({
+            node,
+            message:
+              "Use useGate() from '#/lib/statsig/statsig' instead of the one on npm.",
+          })
+        }
+        // TODO: Verify gate() call results aren't stored in variables.
+      },
+    }
+  },
 }
@@ -41,7 +41,7 @@
    "test-watch": "NODE_ENV=test jest --watchAll",
    "test-ci": "NODE_ENV=test jest --ci --forceExit --reporters=default --reporters=jest-junit",
    "test-coverage": "NODE_ENV=test jest --coverage",
-    "lint": "eslint --cache --ext .js,.jsx,.ts,.tsx src",
+    "lint": "eslint --cache src",
    "lint-native": "swiftlint ./modules && ktlint ./modules",
    "lint-native:fix": "swiftlint --fix ./modules && ktlint --format ./modules",
    "typecheck": "tsc --project ./tsconfig.check.json",
@@ -248,22 +248,24 @@
    "@types/psl": "^1.1.1",
    "@types/react": "^19.1.12",
    "@types/react-dom": "^19.1.9",
-    "@typescript-eslint/eslint-plugin": "^7.18.0",
-    "@typescript-eslint/parser": "^7.18.0",
+    "@eslint/js": "^9.18.0",
+    "typescript-eslint": "^8.20.0",
    "babel-jest": "^29.7.0",
    "babel-plugin-macros": "^3.1.0",
    "babel-plugin-module-resolver": "^5.0.2",
    "babel-plugin-react-compiler": "^19.1.0-rc.3",
    "babel-preset-expo": "~54.0.0",
-    "eslint": "^8.19.0",
+    "eslint": "^9.18.0",
    "eslint-plugin-bsky-internal": "link:./eslint",
-    "eslint-plugin-ft-flow": "^2.0.3",
-    "eslint-plugin-import": "^2.31.0",
-    "eslint-plugin-lingui": "^0.2.0",
-    "eslint-plugin-react": "^7.33.2",
+    "eslint-plugin-import-x": "^4.6.1",
+    "eslint-plugin-lingui": "^0.3.0",
+    "eslint-plugin-react": "^7.37.4",
    "eslint-plugin-react-compiler": "^19.1.0-rc.2",
-    "eslint-plugin-react-native-a11y": "^3.3.0",
-    "eslint-plugin-simple-import-sort": "^12.0.0",
+    "eslint-plugin-react-hooks": "^5.1.0",
+    "eslint-plugin-react-native": "^5.0.0",
+    "eslint-plugin-react-native-a11y": "^3.4.1",
+    "eslint-plugin-simple-import-sort": "^12.1.1",
+    "globals": "^15.14.0",
    "file-loader": "6.2.0",
    "husky": "^8.0.3",
    "is-ci": "^3.0.1",
@@ -215,18 +215,18 @@ export function Controls({

  const seekLeft = useCallback(() => {
    if (!videoRef.current) return
-    // eslint-disable-next-line @typescript-eslint/no-shadow
+
    const currentTime = videoRef.current.currentTime
-    // eslint-disable-next-line @typescript-eslint/no-shadow
+
    const duration = videoRef.current.duration || 0
    onSeek(clamp(currentTime - 5, 0, duration))
  }, [onSeek, videoRef])

  const seekRight = useCallback(() => {
    if (!videoRef.current) return
-    // eslint-disable-next-line @typescript-eslint/no-shadow
+
    const currentTime = videoRef.current.currentTime
-    // eslint-disable-next-line @typescript-eslint/no-shadow
+
    const duration = videoRef.current.duration || 0
    onSeek(clamp(currentTime + 5, 0, duration))
  }, [onSeek, videoRef])
@@ -3,13 +3,13 @@ import {
  getInfoAsync,
  readDirectoryAsync,
 } from 'expo-file-system/legacy'
+import {type ImagePickerResult} from 'expo-image-picker'
 import ExpoImageCropTool, {
  type OpenCropperOptions,
 } from '@bsky.app/expo-image-crop-tool'

 import {compressIfNeeded} from './manip'
 import {type PickerImage} from './picker.shared'
-import {ImagePickerResult} from 'expo-image-picker'

 async function getFile() {
  const imagesDir = documentDirectory!
@@ -138,11 +138,10 @@ function ChatListItemReady({

  const {lastMessage, lastMessageSentAt, latestReportableMessage} =
    useMemo(() => {
-      // eslint-disable-next-line @typescript-eslint/no-shadow
      let lastMessage = _(msg`No messages yet`)
-      // eslint-disable-next-line @typescript-eslint/no-shadow
+
      let lastMessageSentAt: string | null = null
-      // eslint-disable-next-line @typescript-eslint/no-shadow
+
      let latestReportableMessage: ChatBskyConvoDefs.MessageView | undefined

      if (ChatBskyConvoDefs.isMessageView(convo.lastMessage)) {
@@ -64,7 +64,6 @@ export function PostThread({uri}: {uri: string}) {
   */
  const thread = usePostThread({anchor: uri})
  const {anchor, hasParents} = useMemo(() => {
-    // eslint-disable-next-line @typescript-eslint/no-shadow
    let hasParents = false
    for (const item of thread.data.items) {
      if (item.type === 'threadPost' && item.depth === 0) {
@@ -1,4 +1,3 @@
-/* eslint-disable no-labels */
 import {AppBskyUnspeccedDefs, type ModerationOpts} from '@atproto/api'

 import {
@@ -437,7 +437,6 @@ let PostFeed = ({
          for (const page of data.pages) {
            for (const slice of page.slices) {
              const item = slice.items.find(
-                // eslint-disable-next-line @typescript-eslint/no-shadow
                item => item.uri === slice.feedPostUri,
              )
              if (
@@ -1,9 +1,9 @@
 import {useState} from 'react'
-import {LogBox, Pressable, View, TextInput} from 'react-native'
+import {LogBox, Pressable, TextInput, View} from 'react-native'
 import {useQueryClient} from '@tanstack/react-query'

 import {BLUESKY_PROXY_HEADER} from '#/lib/constants'
-import {useSessionApi, useAgent} from '#/state/session'
+import {useAgent, useSessionApi} from '#/state/session'
 import {useLoggedOutViewControls} from '#/state/shell/logged-out'
 import {useOnboardingDispatch} from '#/state/shell/onboarding'
 import {navigate} from '../../../Navigation'
@@ -50,6 +50,8 @@ export function TestCtrls() {
  return (
    <View style={{position: 'absolute', top: 100, right: 0, zIndex: 100}}>
      <TextInput
+        accessibilityLabel="Text input field"
+        accessibilityHint="Enter proxy header"
        testID="e2eProxyHeaderInput"
        onChangeText={val => setProxyHeader(val as any)}
        onSubmitEditing={() => {
@@ -3,12 +3,10 @@ import {type AlertButton, type AlertStatic} from 'react-native'
 class WebAlert implements Pick<AlertStatic, 'alert'> {
  public alert(title: string, message?: string, buttons?: AlertButton[]): void {
    if (buttons === undefined || buttons.length === 0) {
-      // eslint-disable-next-line no-alert
      window.alert([title, message].filter(Boolean).join('\n'))
      return
    }

-    // eslint-disable-next-line no-alert
    const result = window.confirm([title, message].filter(Boolean).join('\n'))

    if (result === true) {