Preserve source and legacy suppressions during Oxlint migration

Author: gonzaloriestra

configurations/oxlint-rule-mapping.json

@@ -1,6 +1,6 @@
 {
   "@nx/enforce-module-boundaries": {
-    "replacement": "@shopify/cli/module-boundaries",
+    "replacement": "compat/nx-enforce-module-boundaries",
     "status": "partial",
     "limitation": "Relative cross-project imports and re-exports are checked. Nx graph cycles, tag constraints, lazy-loading checks and buildability checks are not retained."
   },
@@ -97,7 +97,7 @@
     "limitation": "The specialized Shopify rule is not ported."
   },
   "@shopify/strict-component-boundaries": {
-    "replacement": "@shopify/cli/module-boundaries",
+    "replacement": "compat/nx-enforce-module-boundaries",
     "status": "partial",
     "limitation": "Relative imports between projects are rejected. Resolver-based component entrypoint checks are not retained."
   },
@@ -147,9 +147,9 @@
     "limitation": "Independent syntax check covers repository conventions, not the complete upstream rule API."
   },
   "@typescript-eslint/naming-convention": {
-    "replacement": "@shopify/cli/naming-convention",
+    "replacement": "compat/typescript-eslint-naming-convention",
     "status": "partial",
-    "limitation": "Configured declaration, parameter, property and type names are checked. Type filters, all upstream selectors and modifier precedence are not implemented."
+    "limitation": "Configured declaration, parameter, property and type names are checked. Type filters, all upstream selectors and modifier precedence are not implemented. Inferred type parameters keep their existing names; naming enforcement applies to declared type parameters."
   },
   "@typescript-eslint/no-array-constructor": {
     "replacement": "no-array-constructor",
@@ -197,9 +197,9 @@
     "limitation": "Native behavior and option support may differ from ESLint."
   },
   "@typescript-eslint/no-unnecessary-type-assertion": {
-    "replacement": "typescript/no-unnecessary-type-assertion",
-    "status": "native",
-    "limitation": "Native behavior and option support may differ from ESLint."
+    "replacement": null,
+    "status": "unsupported",
+    "limitation": "Disabled during migration because the native checker flags assertions accepted by the original ESLint setup, including intentional test fixtures. Existing casts are retained; there is no exact compatibility option."
   },
   "@typescript-eslint/no-unnecessary-type-constraint": {
     "replacement": "typescript/no-unnecessary-type-constraint",
@@ -372,7 +372,7 @@
     "limitation": "Native behavior and option support may differ from ESLint."
   },
   "id-length": {
-    "replacement": "@shopify/cli/id-length",
+    "replacement": "compat/id-length",
     "status": "partial",
     "limitation": "Independent syntax check covers repository conventions, not the complete upstream rule API."
   },
@@ -452,14 +452,14 @@
     "limitation": "Native behavior and option support may differ from ESLint."
   },
   "import-x/no-named-as-default": {
-    "replacement": "import/no-named-as-default",
-    "status": "native",
-    "limitation": "Native behavior and option support may differ from ESLint."
+    "replacement": null,
+    "status": "unsupported",
+    "limitation": "Disabled during migration because native module analysis flags existing default WebSocket imports accepted by ESLint. Existing imports are retained; this check is currently lost."
   },
   "import-x/no-named-as-default-member": {
-    "replacement": "import/no-named-as-default-member",
-    "status": "native",
-    "limitation": "Native behavior and option support may differ from ESLint."
+    "replacement": null,
+    "status": "unsupported",
+    "limitation": "Disabled during migration because native module analysis flags existing Prettier and CommonJS GraphQL-loader access accepted by ESLint. Existing imports are retained; this check is currently lost."
   },
   "import-x/no-named-default": {
     "replacement": "import/no-named-default",
@@ -787,7 +787,7 @@
     "limitation": "Independent syntax check covers repository conventions, not the complete upstream rule API."
   },
   "n/prefer-global/console": {
-    "replacement": "@shopify/cli/prefer-global-console",
+    "replacement": "compat/n-prefer-global-console",
     "status": "partial",
     "limitation": "Independent syntax check covers repository conventions, not the complete upstream rule API."
   },
@@ -807,7 +807,7 @@
     "limitation": "Independent syntax check covers repository conventions, not the complete upstream rule API."
   },
   "n/prefer-global/url": {
-    "replacement": "@shopify/cli/prefer-global-url",
+    "replacement": "compat/n-prefer-global-url",
     "status": "partial",
     "limitation": "Independent syntax check covers repository conventions, not the complete upstream rule API."
   },
@@ -867,7 +867,7 @@
     "limitation": "Native behavior and option support may differ from ESLint."
   },
   "no-catch-all/no-catch-all": {
-    "replacement": "@shopify/cli/no-catch-all",
+    "replacement": "no-catch-all/no-catch-all",
     "status": "retained"
   },
   "no-catch-shadow": {
@@ -1191,7 +1191,7 @@
     "limitation": "Native behavior and option support may differ from ESLint."
   },
   "no-restricted-syntax": {
-    "replacement": "@shopify/cli/restricted-syntax",
+    "replacement": "compat/no-restricted-syntax",
     "status": "partial",
     "limitation": "Independent syntax check covers repository conventions, not the complete upstream rule API."
   },
@@ -1496,7 +1496,7 @@
     "limitation": "Native behavior and option support may differ from ESLint."
   },
   "tsdoc/syntax": {
-    "replacement": "@shopify/cli/tsdoc-syntax",
+    "replacement": "tsdoc/syntax",
     "status": "partial",
     "limitation": "Uses the same TSDoc parser with its default configuration. Per-directory tsdoc.json loading is not implemented; this repository has no such file."
   },

docs/cli/oxlint-rule-coverage.md

@@ -15,11 +15,11 @@ Consumers must configure those separately. This is a breaking public config chan
 
 The original configuration at `656de9379cb7b4e2af6fe0bc6c8d81b1a50163f7` enabled
 301 lint rule names plus `prettier/prettier`. The migration has replacements for
-246 lint rule names: 197 native counterparts, 14 retained implementations, and
+243 lint rule names: 194 native counterparts, 14 retained implementations, and
 35 partial independent replacements. Prettier covers formatting separately.
-There are 55 rule names without a dedicated replacement. Of those, 11 have
+There are 58 rule names without a dedicated replacement. Of those, 11 have
 identified coverage or overlap elsewhere in the existing toolchain. These are
-not 55 wholly missing capabilities, and overlap does not establish full parity.
+not 58 wholly missing capabilities, and overlap does not establish full parity.
 
 The independent plugin in `packages/eslint-plugin-cli/oxlint.js` adds naming,
 member ordering, identifier length, import ordering, project boundaries, unused
@@ -36,9 +36,18 @@ fixes for contiguous imports without comments or side effects. Public parameter
 return documentation checks cover directly exported functions; destructured parameter
 paths and full export reachability are not equivalent to the original JSDoc rules.
 
-Native rules can report additional cases, including unnecessary type assertions
-and ambiguous default imports. Existing code and directives are updated for these
-diagnostics. Unused imports in tests remain checked even though unused local variables
+To keep the migration small, existing type assertions and default imports are
+preserved. The native `typescript/no-unnecessary-type-assertion`,
+`import/no-named-as-default`, and `import/no-named-as-default-member` checks are
+disabled because they report existing code accepted by ESLint and have no exact
+compatibility option. These are three explicit coverage gaps, not equivalent
+replacements. Declared type parameters keep their naming checks; inferred type
+parameter names retain their existing spelling.
+
+Small dependency-free alias plugins preserve the existing `compat/*`,
+`no-catch-all/no-catch-all`, and `tsdoc/syntax` suppression names. They call the
+independent implementations and do not load ESLint or upstream plugins. Obsolete
+and unused suppressions are removed. Unused imports in tests remain checked even though unused local variables
 are allowed. Import-removal fixes differ from the original dedicated plugin.
 
 ## Coverage from other existing tools
@@ -89,9 +98,9 @@ The complete rule-by-rule mapping and limitations are in
 
 | Family | Remaining checks |
 | --- | --- |
-| Import analysis | Deprecated imports and redundant path segments; resolution, dependency, and unused-export coverage outside Knip/compiler scopes |
+| Import analysis | Deprecated imports, redundant path segments, and ambiguous default-import checks; resolution, dependency, and unused-export coverage outside Knip/compiler scopes |
 | JSDoc | Advanced tag/type validation, parameter path matching, return/throw/yield consistency, layout beyond Prettier |
-| Control flow | Consistent returns, atomic updates, early-return conventions |
+| Control flow and assertions | Consistent returns, atomic updates, early-return conventions, redundant type assertions |
 | Node APIs | Deprecated or unsupported APIs, callback conventions, promise API preferences, executable/shebang checks |
 | Shopify conventions | Image imports, context menus, module-scope constants, Twine, singular enum names |
 | Style and directives | Camel case in JavaScript, line-comment placement, statement padding, class sorting, explicit strict-directive policy, directive conventions beyond unused suppressions |