fix(odd-phase): label CI status polling as checking - #1512
Conversation
gh pr checks and gh run view/watch/list verify work, so they infer checking instead of exploring.
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 2 remain after this review. 📝 WalkthroughWalkthroughGitHub pull request checks and run view, watch, and list commands now infer the checking phase. Tests cover these commands and confirm that pull request and issue inspection commands infer the exploring phase. ChangesGitHub command phase inference
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The change limits checking labels to the intended CI status commands; other run commands retain their prior behavior. No material merge risk remains. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
PR type
type:bug)Summary
The Gentle Shell working label showed exploring while the agent was waiting on CI (for example
sleep 30; gh pr checks 1511 --watch | tail), becausegh pr checksandgh run *were classified as read-only inspection.gh pr checksandgh run view|watch|listnow infer checking.gh pr|issue|repo|release view|list|diff|statusstill infer exploring.gh runsubcommands (for examplererun,cancel) now leave the label unchanged instead of reading as exploration.Verification
exploring), then GREEN.node --experimental-strip-types --test tests/odd-phase-inference.test.ts tests/odd-phase.test.ts: 48/48 passed.node scripts/check-types.mjs: 188 baseline diagnostics, no regressions.node scripts/build-runtime-modules.mjs --check: generated modules match sources.Summary by CodeRabbit