2fa27412ea
* 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> |
||
|---|---|---|
| .. | ||
| commands | ||
| lib | ||
| options | ||
| scripts | ||
| index.test.ts | ||
| program.test.ts | ||