docs(watchdog): document GENIE_WATCHDOG_SKIP + auto-skip (PR-B G7 docs)#1642
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the watchdog documentation to include instructions on skipping the installation process during a genie doctor --fix command. It details the use of the GENIE_WATCHDOG_SKIP environment variable and explains the automatic skipping logic for bundled installs. The review feedback suggests refining the phrasing to more accurately distinguish between manual user opt-outs and automatic system behaviors.
|
|
||
| ## Skipping the install | ||
|
|
||
| `genie doctor --fix` runs a watchdog install precondition. Two ways to opt out: |
There was a problem hiding this comment.
The phrasing "Two ways to opt out" is slightly inaccurate because the second point (auto-skipping) is an automatic system behavior rather than a user-initiated "opt out". It would be clearer to state that the install can be skipped manually or automatically.
…coped per reviewer Lands the wish doc that scaffolds PR-A (#1634) and PR-B (#1636/#1637/#1638/ #1640/#1642), plus the 2026-05-07 PR-C draft + reviewer FIX-FIRST corrections. Why this is a separate docs commit: - The wish file was authored 2026-05-04 but only ever sat in a stash; never committed despite shipping work referencing it. This commit lands the reference document for completed + pending work in one place. - PR-C as originally drafted had three invalid premises against live 4.260507.1 (G3 amendment already implemented at scheduler-daemon.ts:1296; G9 line is on stderr not stdout; G10 design assumes binary-spawn that the HTTP probe doesn't do). Reviewer corrections folded in. - Only G8 (kill-path shadow+UUID dedup) survives intact — file path corrected to src/term-commands/agents.ts:2817 (handleWorkerKill). - G9 reframed as stderr-noise reduction (DEBUG=pgserve gating). - G10 deferred pending /trace into update.ts:362. QA dogfooding-72h artifacts (AUDIT.md, QA-PLAN.md) document the 72-h fix-audit sweep that surfaced the bugs and triggered the wish update. Refs: #1677 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2 participants
PR #1638 shipped GENIE_WATCHDOG_SKIP=1 opt-out + auto-skip for bundle installs without documentation. Adds 'Skipping the install' section to packages/watchdog/README.md (+14/-0). Markdown-only, no code changes. Wish: cli-noise-and-hygiene-cleanup PR-B G7 deliverable #3.