Skip to content

fix(js): prevent crash when terminating task using the @nx/js:swc executor#33845

Merged
leosvelperez merged 1 commit into
masterfrom
issue-31938
Dec 12, 2025
Merged

fix(js): prevent crash when terminating task using the @nx/js:swc executor#33845
leosvelperez merged 1 commit into
masterfrom
issue-31938

Conversation

@leosvelperez

Copy link
Copy Markdown
Member

Current Behavior

When terminating a task using the @nx/js:swc executor before it ever got to execute its post-compilation logic, a TypeError: disposeFn is not a function error is thrown.

Expected Behavior

Terminating a task using the @nx/js:swc executor should not error.

Related Issue(s)

Fixes #31938

@leosvelperez leosvelperez self-assigned this Dec 12, 2025
@leosvelperez leosvelperez requested a review from a team as a code owner December 12, 2025 17:06
@vercel

vercel Bot commented Dec 12, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
nx-dev Ready Ready Preview Dec 12, 2025 5:11pm

@netlify

netlify Bot commented Dec 12, 2025

Copy link
Copy Markdown

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit c07b587
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/693c4bac6564030008134ed2
😎 Deploy Preview https://deploy-preview-33845--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

nx-cloud Bot commented Dec 12, 2025

Copy link
Copy Markdown
Contributor

View your CI Pipeline Execution ↗ for commit c07b587

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

☁️ Nx Cloud last updated this comment at 2025-12-12 17:58:08 UTC

@nx-cloud nx-cloud Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Important

A new CI pipeline execution was requested that may update the conclusion below...

Nx Cloud has identified a possible root cause for your failed CI:

Our analysis shows the failing e2e test doesn't use the @nx/js:swc executor that was modified in this PR. The timeout occurs during environment-dependent setup operations (pnpm install, git commands) and cannot be logically connected to the PR's changes to swc's signal handling. This appears to be an environmental issue rather than a code change problem.

No code changes were suggested for this issue.

If the issue was transient, you can trigger a rerun:

Rerun CI

Nx Cloud View detailed reasoning on Nx Cloud ↗


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

@leosvelperez leosvelperez enabled auto-merge (squash) December 12, 2025 17:51
@leosvelperez leosvelperez merged commit 80523ef into master Dec 12, 2025
21 of 22 checks passed
@leosvelperez leosvelperez deleted the issue-31938 branch December 12, 2025 17:58
FrozenPandaz pushed a commit that referenced this pull request Dec 15, 2025
…xecutor (#33845)

## Current Behavior

When terminating a task using the `@nx/js:swc` executor before it ever
got to execute its post-compilation logic, a `TypeError: disposeFn is
not a function` error is thrown.

## Expected Behavior

Terminating a task using the `@nx/js:swc` executor should not error.

## Related Issue(s)

Fixes #31938

(cherry picked from commit 80523ef)
@github-actions

Copy link
Copy Markdown
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Dec 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

disposeFn is not a function when terminating @nx/js:swc with watch mode

2 participants