Skip to content

Close remaining Oxlint parity gaps - #8210

Draft
gonzaloriestra wants to merge 1 commit into
gonzalo/oxlint-04-local-rulesfrom
gonzalo/oxlint-05-parity
Draft

Close remaining Oxlint parity gaps#8210
gonzaloriestra wants to merge 1 commit into
gonzalo/oxlint-04-local-rulesfrom
gonzalo/oxlint-05-parity

Conversation

@gonzaloriestra

@gonzaloriestra gonzaloriestra commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Why

Oxlint could not type-check the UI extension packages because their legacy Node module-resolution settings are incompatible with the TypeScript 7 language service. Two compatibility rules also still ran only in ESLint even though Oxlint can load their JavaScript plugins.

What

  • Move both UI extension packages to bundler module resolution and make their source aliases explicit.
  • Include both packages in type-aware Oxlint.
  • Run the existing catch-all and TSDoc syntax rules through Oxlint JavaScript plugins.

This is the fifth PR in the ESLint-to-Oxlint stack. ESLint remains as the compatibility linter until the final cleanup PR.

Test

  • pnpm lint
  • pnpm exec tsc --project packages/ui-extensions-test-utils/tsconfig.json --noEmit
  • pnpm exec tsc --project packages/ui-extensions-server-kit/tsconfig.json --noEmit
  • node bin/run-knip-ci.js

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