fix: Disable import-x/no-duplicates in TypeScript files (#427)

This rule had to be disabled in `core` because the autofixer is very
broken. I'm also not sure we want to strictly require one import between
type and non-type imports anyway, it's a stylistic preference.

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Disables `import-x/no-duplicates` in the TypeScript ESLint config,
updates the rules snapshot, and adds changelog entries noting the change
and related autofix behavior fix.
> 
> - **TypeScript ESLint config**:
> - Disable `import-x/no-duplicates` in
`packages/typescript/src/index.mjs` with rationale comments.
> - **Rules Snapshot**:
> - Set `import-x/no-duplicates` to `off` in
`packages/typescript/rules-snapshot.json`.
> - **Changelog**:
> - Add "Changed" entry for disabling `import-x/no-duplicates` in
`packages/typescript/CHANGELOG.md`.
> - Add "Fixed" entry noting prevention of non-type imports being
grouped under a type import during auto-fix.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
d06d2c8. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Author: Gudahtt

packages/typescript/CHANGELOG.md

@@ -14,6 +14,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Disable `@typescript-eslint/no-unnecessary-type-arguments` ([#426](https://github.com/MetaMask/eslint-config/pull/426))
   - We decided that "unnecessary" type arguments make types easier to read sometimes, so we should allow them.
 - Disable `promise/valid-params` because it's redundant in type-checked projects ([#425](https://github.com/MetaMask/eslint-config/pull/425))
+- Disable `import-x/no-duplicates` ([#427](https://github.com/MetaMask/eslint-config/pull/427))
+  - It was a style preference that we may not want, and the auto-fix was broken.
+
+### Fixed
+
+- Prevent non-type imports from being grouped under a type import upon auto-fix ([#427](https://github.com/MetaMask/eslint-config/pull/427))
+  - This was caused by `import-x/no-duplicates`, which is now disabled.
 
 ## [14.1.0]
 

packages/typescript/rules-snapshot.json

@@ -155,6 +155,7 @@
   "getter-return": "off",
   "import-x/consistent-type-specifier-style": ["error", "prefer-top-level"],
   "import-x/named": "off",
+  "import-x/no-duplicates": "off",
   "import-x/no-unresolved": "off",
   "jsdoc/check-access": "error",
   "jsdoc/check-alignment": "error",

packages/typescript/src/index.mjs

@@ -199,6 +199,11 @@ const config = createConfig({
 
     /* import-x plugin rules */
 
+    // This rule is to aggresive about combining type and non-type imports, which I'm not sure that we want.
+    // But more importantly, the auto-fixer is broken.
+    // See here for details on that bug: https://github.com/un-ts/eslint-plugin-import-x/issues/231
+    'import-x/no-duplicates': 'off',
+
     // Handled by TypeScript
     'import-x/no-unresolved': 'off',