Skip to content

Run local CLI rules with Oxlint - #8209

Draft
gonzaloriestra wants to merge 1 commit into
gonzalo/oxlint-03-primaryfrom
gonzalo/oxlint-04-local-rules
Draft

Run local CLI rules with Oxlint#8209
gonzaloriestra wants to merge 1 commit into
gonzalo/oxlint-03-primaryfrom
gonzalo/oxlint-04-local-rules

Conversation

@gonzaloriestra

@gonzaloriestra gonzaloriestra commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Why

This is the fourth step of the ESLint-to-Oxlint migration. The repository-local CLI rules were one of the remaining reasons normal linting still needed ESLint.

What changed

  • load the existing @shopify/eslint-plugin-cli implementation as an Oxlint JavaScript plugin
  • enable all 12 local CLI rules currently active in the flat ESLint configuration
  • make all 13 exported local rules available to Oxlint, including the currently disabled bootstrap-import rule
  • let the compatibility configuration stop running the migrated local rules in ESLint
  • make the no-inline-graphql known-failure lookup independent of pnpm installation depth so it works when Oxlint loads the workspace plugin directly

Validation

  • pnpm lint:oxlint
  • pnpm lint
  • full ESLint config against known inline-GraphQL exceptions
  • verified normal ESLint effective config disables a migrated local rule
  • node bin/run-knip-ci.js
  • git diff --check

gonzaloriestra commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

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

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

@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label Jul 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant