Skip to content

feat(linter): add initial oxlint plugin, docs, and e2e coverage#34838

Draft
juristr wants to merge 9 commits intomasterfrom
alabaster-stetson
Draft

feat(linter): add initial oxlint plugin, docs, and e2e coverage#34838
juristr wants to merge 9 commits intomasterfrom
alabaster-stetson

Conversation

@juristr
Copy link
Copy Markdown
Member

@juristr juristr commented Mar 14, 2026

This PR adds the documentation for the experimental @nx/oxlint plugin. The plugin implementation lives in nx-labs.

Related Issue(s)

Plugin implementation: nrwl/nx-labs#441

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 14, 2026

Deploy Preview for nx-dev ready!

Name Link
🔨 Latest commit 6cf9a90
🔍 Latest deploy log https://app.netlify.com/projects/nx-dev/deploys/69bd64cf7e058f000879d51b
😎 Deploy Preview https://deploy-preview-34838--nx-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 14, 2026

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit 6cf9a90
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/69bd64cf23367a000888e19c
😎 Deploy Preview https://deploy-preview-34838--nx-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Mar 14, 2026

View your CI Pipeline Execution ↗ for commit 6cf9a90

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 5m 25s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 7s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 8s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 1s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-03-20 15:26:26 UTC

juristr added 3 commits March 16, 2026 11:39
updates @nx/oxlint inferred command generation and adds regression coverage for vite config loading.
- fix heading case in oxlint introduction docs (vale errors)
- add oxlint to pluginToTechnology mapping for auto-generated generator/executor pages
- add knowledge-base/oxlint index page to resolve broken link
- remove default maxWarnings: -1 from executor schema (caused CLI parse error)
- use --max-warnings=N format to handle negative values safely
nx-cloud[bot]

This comment was marked as outdated.

oxlint depends on @nx/eslint-plugin at runtime for boundaries-plugin bridge.
pnpm resolves the workspace symlink which ESLint can't load without a build.
nx-cloud[bot]

This comment was marked as outdated.

nx-cloud[bot]

This comment was marked as outdated.

Comment thread astro-docs/src/content/docs/features/enforce-module-boundaries.mdoc Outdated
@pumano
Copy link
Copy Markdown
Contributor

pumano commented Mar 18, 2026

Wow! Would love to get this landed!

@juristr
Copy link
Copy Markdown
Member Author

juristr commented Mar 20, 2026

We're going to test this out first as a labs extension :)

follow along here: nrwl/nx-labs#441 and on socials when we're releasing this for you to try out

@juristr juristr closed this Mar 20, 2026
@juristr juristr reopened this Mar 20, 2026
juristr and others added 2 commits March 20, 2026 15:55
implementation moved to https://github.com/nrwl/nx-labs
this PR now contains only the astro-docs for the experimental oxlint plugin
implementation moved to https://github.com/nrwl/nx-labs
this PR now contains only the astro-docs for the experimental oxlint plugin [Self-Healing CI Rerun]
Copy link
Copy Markdown
Contributor

@nx-cloud nx-cloud Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nx Cloud has identified a flaky task in your failed CI:

🔂 Since the failure was identified as flaky, we triggered a CI rerun by adding an empty commit to this branch.

Nx Cloud View detailed reasoning in Nx Cloud ↗

🔔 Heads up, your workspace has pending recommendations ↗ to auto-apply fixes for similar failures.


🎓 Learn more about Self-Healing CI on nx.dev

@andres-ito1
Copy link
Copy Markdown

hey @juristr! Im pumped about this work. Any idea when this work will land into oxlint?

@pumano
Copy link
Copy Markdown
Contributor

pumano commented Apr 15, 2026

warm reminder for @juristr. We really need it)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants