fix(core): misc daemon fixes

[Cammisuli]

Current Behavior

• Daemon does not stop while doing migrations
• DaemonClient reset does not close the socket connection
• DaemonClient out and err file handlers are not closed
• The socket path was created with the workspace root, but didnt take case sensitivity into account, (ie. On Windows, Nx Console will send c:\workspace\path, but the daemon will have C:\workspace\path) causing multiple daemons to always respawn

Expected Behavior

• The Daemon will now be reset when doing migrations
• DaemonClient will reset at the appropriate time
• The file handles for the out and err writes are closed by the client (but still kept for the daemon process)
• Daemon socket path is now case insensitive

Related Issue(s)

Fixes #15424
Fixes #15480

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated
nx-dev	✅ Ready (Inspect)	Visit Preview	💬 Add your feedback	Mar 16, 2023 at 8:50PM (UTC)

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 8700934. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

---

✅ Successfully ran 8 targets

• nx run-many -t e2e-macos --parallel=1 --exclude=e2e-detox
• nx affected --target=e2e --base=cc8907bb13f0c60d861f4b52df052b038506252f --head=870093465b478cb96d39bbba68f1327dc44a2402 --exclude=e2e-storybook,e2e-storybook-angular --parallel=1
• nx affected --target=test --base=cc8907bb13f0c60d861f4b52df052b038506252f --head=870093465b478cb96d39bbba68f1327dc44a2402 --parallel=1
• nx affected --target=build --base=cc8907bb13f0c60d861f4b52df052b038506252f --head=870093465b478cb96d39bbba68f1327dc44a2402 --parallel=3
• nx run-many -t root-lint check-imports check-commit check-lock-files depcheck --parallel=1 --no-dte
• nx affected --target=lint --base=cc8907bb13f0c60d861f4b52df052b038506252f --head=870093465b478cb96d39bbba68f1327dc44a2402 --parallel=3
• nx-cloud record -- nx format:check --base=cc8907bb13f0c60d861f4b52df052b038506252f --head=870093465b478cb96d39bbba68f1327dc44a2402
• nx documentation --no-dte

---

Sent with 💌 from NxCloud.

Stale

[github-actions]

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.