Skip to content

feat(oxlint): add oxlint labs plugin#441

Open
juristr wants to merge 3 commits intomainfrom
ninth-rayon
Open

feat(oxlint): add oxlint labs plugin#441
juristr wants to merge 3 commits intomainfrom
ninth-rayon

Conversation

@juristr
Copy link
Copy Markdown
Member

@juristr juristr commented Mar 20, 2026

No description provided.

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Mar 20, 2026

View your CI Pipeline Execution ↗ for commit 963bf9d

Command Status Duration Result
nx run-many -t e2e --exclude=nx-ignore-e2e ✅ Succeeded 1m 12s View ↗
nx run-many -t lint test build ✅ Succeeded 2m 3s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 2s View ↗

☁️ Nx Cloud last updated this comment at 2026-04-16 23:52:41 UTC

nx-cloud[bot]

This comment was marked as outdated.

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.

Important

At least one additional CI pipeline execution has run since the conclusion below was written and it may no longer be applicable.

Nx Cloud is proposing a fix for your failed CI:

We fixed the should run oxlint successfully e2e test by setting NX_NO_CLOUD: 'true' in the env passed to execSync. The test workspace is created with --nxCloud=skip, but the CI NX_CLOUD_ACCESS_TOKEN leaks in via process.env, causing Nx Cloud to reject the task run because the token doesn't belong to the ephemeral test workspace. Scoping the cloud opt-out to just this call prevents Nx Cloud from intercepting the lint task without affecting the rest of the test setup.

Tip

We verified this fix by re-running oxlint-e2e:e2e.

diff --git a/e2e/oxlint-e2e/src/oxlint.spec.ts b/e2e/oxlint-e2e/src/oxlint.spec.ts
index ca4337b..755b142 100644
--- a/e2e/oxlint-e2e/src/oxlint.spec.ts
+++ b/e2e/oxlint-e2e/src/oxlint.spec.ts
@@ -66,7 +66,7 @@ describe('oxlint plugin', () => {
     execSync('yarn nx run lib-a:lint', {
       cwd: projectDirectory,
       stdio: 'inherit',
-      env: process.env,
+      env: { ...process.env, NX_NO_CLOUD: 'true' },
     });
   });
 });

Apply fix via Nx Cloud  Reject fix via Nx Cloud


Or Apply changes locally with:

npx nx-cloud apply-locally e5m1-Pg8A

Apply fix locally with your editor ↗   View interactive diff ↗



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

@andres-ito1
Copy link
Copy Markdown

Hi! I was wondering if there's a plan to get this PR merged in the next few days?

@andres-ito1
Copy link
Copy Markdown

Hi! I was wondering if there's a plan to get this PR merged in the next few days?

@juristr hi friendly follow-up here :)

@juristr
Copy link
Copy Markdown
Member Author

juristr commented Apr 16, 2026

@andres-ito1 looking into it. just got swamped with other work

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.

2 participants