Repository navigation
feat(shell): describe RDD sidebar states by what is happening - #1508
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 configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (6)
Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe review sidebar now uses revised labels for review states, includes singular or plural file counts in candidate scope, and omits the scope row when scope is unavailable. Tests and task documentation cover these display rules. ChangesReview sidebar display
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Feature Merge Risk: ⚪ Minimal · up to The changes update sidebar wording and scope display without changing review-state routing. No material merge risk was established. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 5 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches 💡 1📝 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 |
Refs #1438
Summary
🌹 RDDStatus group now says what is happening and who acts, instead of internal state names (for exampleclosed→✓ Approved,consent→Needs your consent,approved→Approved · finalizing…).first.ts +N files, and is omitted when the candidate scope is unknown instead of showing "Candidate scope unavailable".REVIEW_SCOPE_UNAVAILABLEstays as the internal sentinel.Labels
Changes
lib/review-sidebar-state.ts+N filesscope wordinglib/shell-bar.tstests/review-sidebar-state.test.ts,tests/shell-bar.test.ts,tests/gentle-shell.test.tsodd/tasks/rdd-sidebar-labels.mdTest plan
mainafter feat(shell): add an RDD toggle to visual Sections #1507 (the RDD Sections toggle); the combined RDD group keeps both the visibility check and the scope omission.node --experimental-strip-types --test tests/*.test.ts: 3935 pass, 0 fail, 43 skippednode scripts/check-types.mjs: 188 recorded diagnostics, no regressionsnode scripts/check-provider-contract.mjs: passed;tests/runtime-harness.mjs: exit 0review-f837dc6e7581e272and the integration test fixreview-10c89d8cccd0283e, both approvedChecklist
Refs #1438)type:*labelCo-Authored-BytrailersSummary by CodeRabbit