Skip to content

fix(core): clean up daemon workspace data directory on nx reset --onl…#34174

Merged
MaxKless merged 2 commits intomasterfrom
claude/slack-fix-nx-reset-daemon-cleanup-lltqu
Jan 22, 2026
Merged

fix(core): clean up daemon workspace data directory on nx reset --onl…#34174
MaxKless merged 2 commits intomasterfrom
claude/slack-fix-nx-reset-daemon-cleanup-lltqu

Conversation

@MaxKless
Copy link
Copy Markdown
Contributor

Previously, nx reset --onlyDaemon would only stop the daemon process but not clean up the daemon files in .nx/workspace-data/d. This change ensures the daemon workspace data directory is also removed when using the --onlyDaemon flag, consistent with the behavior of a full reset.

…yDaemon

Previously, `nx reset --onlyDaemon` would only stop the daemon process
but not clean up the daemon files in `.nx/workspace-data/d`. This change
ensures the daemon workspace data directory is also removed when using
the `--onlyDaemon` flag, consistent with the behavior of a full reset.
@MaxKless MaxKless requested a review from a team as a code owner January 21, 2026 15:51
@MaxKless MaxKless requested a review from Cammisuli January 21, 2026 15:51
@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 21, 2026

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit 1bf0291
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/6970f80830a1b600081cdb7a
😎 Deploy Preview https://deploy-preview-34174--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.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 21, 2026

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

Project Deployment Review Updated (UTC)
nx-dev Ready Ready Preview Jan 21, 2026 4:16pm

Request Review

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Jan 21, 2026

View your CI Pipeline Execution ↗ for commit 1bf0291

Command Status Duration Result
nx affected --targets=lint,test,test-kt,build,e... ✅ Succeeded 37m 25s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 1m 47s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 9s 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 2026-01-21 18:33:30 UTC

nx-cloud[bot]

This comment was marked as outdated.

Co-authored-by: MaxKless <MaxKless@users.noreply.github.com>
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 has identified a possible root cause for your failed CI:

Our analysis indicates these failures are due to an environment_state issue rather than the PR's code changes. All 79 failing tasks show ERR_PNPM_WORKSPACE_PKG_NOT_FOUND for @jsonjoy.com/fs-core@workspace:*, which is a pnpm workspace configuration problem unrelated to the daemon reset cleanup functionality being modified in this PR.

No code changes were suggested for this issue.

Trigger a rerun:

Rerun CI

Nx Cloud View detailed reasoning on Nx Cloud ↗


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

@MaxKless MaxKless merged commit d6572f3 into master Jan 22, 2026
22 of 24 checks passed
@MaxKless MaxKless deleted the claude/slack-fix-nx-reset-daemon-cleanup-lltqu branch January 22, 2026 07:29
@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 Jan 28, 2026
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.

3 participants