PR review by depohmel · merge-ready
feat(bin): fm-coderabbit-poll.sh — hourly host-cron CodeRabbit review babysitter
Situation
- Compliance:
pass
- Tests:
green
- Notes: compliance=pass tests=green
Triage
- Summary: Adds a new hourly host-cron script (fm-coderabbit-poll.sh) plus docs and an example queue file that watches a maintainer-curated list of GitHub PRs, requests/detects CodeRabbit reviews, and spawns a scoped one-shot crewmate to fix or decline each new actionable finding.
- Product implications: Routine internal tooling work with no changes to shared/core firstmate code paths (fm-spawn.sh interface is used as-is, not modified); the design choices (host cron over CronCreate, watermark-on-spawn-success, one-shot crewmate scope) are already explained and justified in the PR body, and the diff is additive-only (new files), so this looks like standard review rather than something needing owner deliberation.
- Recommended next step: look closer - worth a scan of the new shell script's error handling and gh API/quota assumptions (e.g. the org-wide 1-review/hour limit and the hardcoded actionable-count marker format) since it's unexercised end-to-end against a live CodeRabbit review, but nothing here looks blocking.
Recommended action
Merge - compliance and tests are green.
Your decision
Tick one box for a quick call, or reply with a slash-command (/merge, /close, /decline <reason>, /hold, /comment <text>, /request-changes <text>):
Only the repository owner can drive this decision - everyone else's edits and comments are ignored.
Decision needed - firstmate#281
PR review by depohmel ·
merge-readySituation
passgreenTriage
Recommended action
Merge - compliance and tests are green.
Your decision
Tick one box for a quick call, or reply with a slash-command (
/merge,/close,/decline <reason>,/hold,/comment <text>,/request-changes <text>):Only the repository owner can drive this decision - everyone else's edits and comments are ignored.