First live run on PR #126 left the foreground shell stuck: Claude pushed the
commit itself and then did not exit, so the supervisor code after it (push +
ack + handled marker) never ran. Under cron that also holds the flock lock,
freezing every later run, and the comment never gets marked handled (so it
would be re-processed forever).
- Wrap the Claude invocation in `setsid timeout -k 30s "$CLAUDE_TIMEOUT"`
(default 30m). `setsid` detaches from the controlling terminal so a lingering
child can't stick an interactive run; `timeout` returns control to the
supervisor, which still pushes any commits (idempotent if Claude pushed) and
writes the handled marker. A timed-out run (rc=124) is logged, not fatal.
- Give this watcher its own dev port (devPort, default 3101) distinct from the
issue watcher's 3100, and reap it after each run -- no cross-watcher kill.
- Reinforce the prompt: stop any dev server and END THE TURN; never push.
Adds claudeTimeout + devPort to the example config and documents both.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
update-pr-review-watcher.sh refreshes the deployed script from the repo via a
dedicated self-update checkout (~/pr-review-watcher/.src) that never races the
issue watcher's work clone. Reads the live config for auth, never clobbers the
config (real token), and self-updates. Optional ref arg for pre-merge testing.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
