* fix: align first-run project identity
* test(cli): canonicalize tmpDir with realpathSync for macOS
The new "writes a flat local config and returns the global project
identity" test failed on macOS because os.tmpdir() returns /var/... while
the project path stored in the global registry is canonicalized to
/private/var/... by registerProjectInGlobalConfig and loadConfig. Wrap
the comparison operand in realpathSync to match the prior art in the
PR's own integration test (cli-first-run-config.integration.test.ts:31).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* chore(cli): address PR review feedback
Three small follow-ups raised by reviewers on #2039:
- Remove stale findFreePort mock in autoCreateConfig test — the function
no longer calls findFreePort, so the mock setup was dead code
- Drop misleading "⚠ Could not register project in global config" warning
printed immediately before rethrowing. The yellow warning visually
signalled a recoverable-and-continuing flow, but the function now
throws fatally. Let the thrown error propagate to the caller for
presentation; the registerProjectInGlobalConfig error message is
descriptive enough on its own
- Add trailing newline to cli-first-run-config.integration.test.ts
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---------
Co-authored-by: suraj-markup <sk9261712674@gmail.com>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2fa27412ea