Repository navigation
test: retain issue 728 reporter picker captures - #765
Conversation
|
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 configuration
📒 Files selected for processing (2)
Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 7 remain after this review. 📝 WalkthroughWalkthroughThe change adds a Claude question-picker capture fixture and a parameterized test. The test checks selection distance and attention-state classification for the captured excerpt and a wrapped-description variant. ChangesQuestion picker regression coverage
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Merge Risk: ⚪ Minimal · up to This test-only change adds coverage for the captured and wrapped picker prompts. No merge-blocking risk is evident from the reviewed context. 🚥 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 |
openrig-review
left a comment
There was a problem hiding this comment.
Approved at c9b4be3. Test-only: adds the #728 reporter's Claude Code 2.1.289 macOS question-picker capture (160x50, excerpt at lines 17-33 with blank rows elsewhere) and their one-line constructed wrapped-description variant as fixtures, and asserts both classify as attention / selection_prompt at their original pane positions. Every captured line matches the public comment, the fixture carries its source, and product code is unchanged. All 8 required checks pass.
— dev60-planner@v-openrig-build
Adds the #728 reporter’s Claude Code 2.1.289 macOS picker excerpt and their explicitly constructed wrapped-description variant beside the existing question-picker fixtures. Both must remain
attention / selection_prompt, with the selected option 12 and 13 nonblank lines from the bottom respectively.The excerpts preserve the published text from @gopinathrimc. Tests retain the stated position in a 160×50 pane using disclosed blank padding for the unprovided header; the inserted line is not presented as a native capture. Test-only change; no classifier behavior changes.
Validation: both inputs passed a bounded check of the unchanged main classifier. Full test-file and suite validation runs in CI.
Refs #728.
Summary by CodeRabbit