feat(install): add project-local agent setup - #4
Conversation
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fe0679a73a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Align local install assertions with the forward-slash executable paths written to generated config on Windows.
Flake ScriptedAlchemy#4: `cursor_workspace_open_hook_notifies_daemon` (and the sibling hook-notification tests sharing `assert_hook_notification`) intermittently panicked under parallel load. The fake-daemon harness bounded several waits at a tight 2s: the socket accept deadline and the `recv_timeout` for the observed request both depend on spawning and running the real `tracedecay` CLI as a child process, whose fork/exec + init can be scheduled slowly under nextest's process-per-test parallelism. Fix (test only): introduce `CLI_ROUNDTRIP_TIMEOUT` (20s) for waits gated on the external CLI and `LOCAL_READY_TIMEOUT` (10s) for in-process readiness signals, replacing the 2s bounds. Generous ceilings that still fail fast on a genuine hang (the CLI normally connects in well under a second). The local `git` helper also gains the resilient absolute-path lookup + ENOENT retry. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Summary
tokensave install --localplumbing for project-scoped agent setup.Test plan