Remove ESLint and use independent Oxlint rules

[gonzaloriestra]

WHY are these changes introduced?

Remove ESLint from the CLI lint command and installed dependency tree while retaining the public CLI rules and as many existing checks as practical.

WHAT is this pull request doing?

Use native Oxlint, its type-aware checker, Prettier, and the independent rules prepared in #8725. Remove the compatibility adapter, upstream ESLint plugins, repository ESLint configuration, and associated CI/editor wiring. Workspace boundary checks read package manifests directly, removing Nx graph generation from lint startup.

Keep @shopify/eslint-plugin-cli public and preserve all 14 custom rule exports. Its configs.config entry point now contains only CLI rules and test exemptions. Consumers must supply their own shared configs and TypeScript parser. This breaking public config change has a major changeset.

Coverage audit

Against the original configuration, 243 of 301 lint rule names have replacements: 194 native counterparts, 14 retained implementations, and 35 partial independent replacements. 58 rule names have no dedicated replacement; 11 have identified coverage or overlap elsewhere in the existing toolchain. Prettier covers formatting separately. These counts do not mean 58 wholly missing capabilities, and overlap is not proof of full behavioral parity.

Existing tools cover part of this territory:

• Knip checks unresolved imports, unlisted dependencies, and unused exports/files, within its configured graph and exclusions.
• TypeScript and esbuild check resolution in their inputs. TypeScript and Prettier reject legacy octal syntax in TypeScript; Oxlint rejects it in strict ES modules. CommonJS JavaScript still has gaps.
• Existing Oxlint rules/parser catch duplicate parameters and catch-variable shadowing in tested cases. Its CLI reports unused disable comments in full lint and the PR job.
• Prettier fixes supported JSDoc indentation; TSDoc catches malformed documentation syntax.

Remaining gaps include redundant type assertions, ambiguous default imports, deprecated imports and redundant paths, import checks outside the tools' scopes, advanced JSDoc semantics, consistent returns and atomic updates, Node API metadata, specialized Shopify/style policies, and Nx cycles/tags/buildability/lazy loading. Knip excludes the plugin and e2e workspaces and selected generated files. Several original rules, including consistent returns and undeclared dependencies, applied only to JavaScript; TypeScript does not replace those scopes. The shared TS config does not enable checkJs or noImplicitReturns.

Benchmark

• Full lint: 55.26s with ESLint compatibility → 11.00s without ESLint, about 5.0× faster (80.1% less time).
• Core lint/format toolchain, including the public plugin: 19 direct + 725 transitive package instances / 290.07 MiB → 4 direct + 2 transitive instances / 45.36 MiB. That is 99.2% fewer total package instances (744 → 6), 78.9% fewer direct dependencies, 99.7% fewer transitive instances, and 84.4% less package-file size.
• This PR removes 4,048 lockfile lines (21.6%): 18,757 → 14,709. Across the complete stack, the lockfile shrinks by 3,767 lines (20.4%) from the original baseline (18,476 → 14,709).

Validation

Full lint passes on this PR and #8725, along with the uncached workspace type-check, 147 focused tests, and the coverage audit. The audit compares 1,606 files and reports no undeclared scope, option, severity, or formatting gaps. All non-e2e GitHub Actions checks pass except Breaking change detection, which flags the public plugin’s major changeset.

Checklist

• I've considered possible cross-platform impacts (Mac, Linux, Windows)
• I've considered possible documentation changes
• I've considered analytics changes to measure impact
• The change is user-facing — I've identified the correct bump type and added a changeset

[gonzaloriestra]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• Remove ESLint and use independent Oxlint rules #8211 👈 (View in Graphite)
• Close remaining Oxlint parity gaps #8210
• Run local CLI rules with Oxlint #8209
• Make Oxlint the primary linter #8208
• Add type-aware Oxlint alongside ESLint #8206
• Decouple Prettier from ESLint #8204
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[github-actions]

This PR seems inactive. If it's still relevant, please add a comment saying so. Otherwise, take no action.
→ If there's no activity within a week, then a bot will automatically close this.
Thanks for helping to improve Shopify's dev tooling and experience.

[gonzaloriestra]

@amcaplan thanks, I missed those commands. I only left the important ones now.

[amcaplan]

Looks great, thanks for doing this work!

[isaacroldan]

The package lint targets skip the CLI entry scripts.

---

🤖 Generated by the review-pr skill

[isaacroldan]

Oxlint doesn't expand this quoted glob, so the CLI entry scripts are skipped. I added debugger to bin/dev.js: this command passed, but passing bin directly failed as expected.

Can we use src bin in both lint and lint:fix, and add a check that covers a file in bin?

---

🤖 Generated by the review-pr skill

[gonzaloriestra]

Updated both lint and lint:fix to use src bin. Added an integration test for each configured command using a real temporary CLI with debugger in bin/dev.js. Both commands now report no-debugger; restoring the quoted glob makes both regression tests fail.

[github-actions]

⚠️ Potential Breaking Changes Detected

This PR contains changes that may break the existing contract.

@shopify/dev_experience — this PR contains breaking changes that require coordination for the next major release.

📦 Major Version Changesets

The following changesets request a major version bump:

Changeset	Package
remove-cli-eslint-config-dependencies.md	'@shopify/eslint-plugin-cli': major

[github-actions]

Differences in type declarations

We detected differences in the type declarations generated by Typescript for this branch compared to the baseline ('main' branch). Please, review them to ensure they are backward-compatible. Here are some important things to keep in mind:

• Some seemingly private modules might be re-exported through public modules.
• If the branch is behind main you might see odd diffs, rebase main into this branch.

New type declarations

We found no new type declarations in this PR

Existing type declarations

packages/cli-kit/dist/public/node/cli.d.ts

@@ -1,3 +1,4 @@
+import type { OptionFlag } from '@oclif/core/interfaces';
 import type { LazyCommandLoader } from './custom-oclif-loader.js';
 /**
  * IMPORTANT NOTE: Imports in this module are dynamic to ensure that "setupEnvironmentVariables" can dynamically
@@ -41,7 +42,7 @@ export declare const jsonFlag: {
     json: import("@oclif/core/interfaces").BooleanFlag<boolean>;
 };
 export declare const authAliasFlag: {
-    'auth-alias': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+    'auth-alias': OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
 };
 /**
  * Builds a `--port` flag that only accepts a valid port number. The flag parses its
@@ -55,7 +56,7 @@ export declare const portFlag: (options?: {
     description?: string;
     env?: string;
     hidden?: boolean;
-}) => import("@oclif/core/interfaces").OptionFlag<number | undefined, import("@oclif/core/interfaces").CustomOptions>;
+}) => OptionFlag<number | undefined>;
 /**
  * Marks a flag as required when the CLI cannot prompt for a value.
  *

packages/cli-kit/dist/private/node/ui/components/AutocompletePrompt.d.ts

@@ -28,5 +28,5 @@ export interface AutocompletePromptProps<T> {
      */
     searchDebounceMs?: number;
 }
-declare function AutocompletePrompt<T>({ message, choices, infoTable, onSubmit, search, hasMorePages: initialHasMorePages, abortSignal, infoMessage, groupOrder, searchDebounceMs, }: React.PropsWithChildren<AutocompletePromptProps<T>>): ReactElement | null;
+declare const AutocompletePrompt: <T>({ message, choices, infoTable, onSubmit, search, hasMorePages: initialHasMorePages, abortSignal, infoMessage, groupOrder, searchDebounceMs, }: React.PropsWithChildren<AutocompletePromptProps<T>>) => ReactElement | null;
 export { AutocompletePrompt };
\ No newline at end of file

packages/cli-kit/dist/private/node/ui/components/SelectInput.d.ts

@@ -27,5 +27,5 @@ export interface Item<T> {
     helperText?: string;
     disabled?: boolean;
 }
-declare function SelectInput<T>({ items: rawItems, initialItems, onChange, enableShortcuts, focus, emptyMessage, defaultValue, highlightedTerm, loading, errorMessage, hasMorePages, morePagesMessage, availableLines, onSubmit, inputFixedAreaRef, ref, groupOrder, }: SelectInputProps<T>): React.ReactElement | null;
+declare const SelectInput: <T>({ items: rawItems, initialItems, onChange, enableShortcuts, focus, emptyMessage, defaultValue, highlightedTerm, loading, errorMessage, hasMorePages, morePagesMessage, availableLines, onSubmit, inputFixedAreaRef, ref, groupOrder, }: SelectInputProps<T>) => React.ReactElement | null;
 export { SelectInput };
\ No newline at end of file

packages/cli-kit/dist/private/node/ui/components/SelectPrompt.d.ts

@@ -14,5 +14,5 @@ export interface SelectPromptProps<T> {
     infoMessage?: InfoMessageProps['message'];
     groupOrder?: string[];
 }
-declare function SelectPrompt<T>({ message, choices, infoTable, infoMessage, onSubmit, defaultValue, abortSignal, groupOrder, }: React.PropsWithChildren<SelectPromptProps<T>>): ReactElement | null;
+declare const SelectPrompt: <T>({ message, choices, infoTable, infoMessage, onSubmit, defaultValue, abortSignal, groupOrder, }: React.PropsWithChildren<SelectPromptProps<T>>) => ReactElement | null;
 export { SelectPrompt };
\ No newline at end of file

packages/cli-kit/dist/private/node/ui/components/Tasks.d.ts

@@ -10,5 +10,5 @@ interface TasksProps<TContext> {
     noColor?: boolean;
     noProgressBar?: boolean;
 }
-declare function Tasks<TContext>({ tasks, silent, onComplete, abortSignal, noColor, noProgressBar, }: React.PropsWithChildren<TasksProps<TContext>>): React.JSX.Element | null;
+declare const Tasks: <TContext>({ tasks, silent, onComplete, abortSignal, noColor, noProgressBar, }: React.PropsWithChildren<TasksProps<TContext>>) => React.JSX.Element | null;
 export { Tasks };
\ No newline at end of file

packages/cli-kit/dist/private/node/ui/components/Table/Table.d.ts

@@ -11,5 +11,5 @@ export interface TableProps<T extends ScalarDict> {
     rows: T[];
     columns: TableColumn<T>;
 }
-declare function Table<T extends ScalarDict>({ rows, columns: columnsConfiguration }: TableProps<T>): React.JSX.Element;
+declare const Table: <T extends ScalarDict>({ rows, columns: columnsConfiguration }: TableProps<T>) => React.JSX.Element;
 export { Table };
\ No newline at end of file