Repository navigation
refactor: quiet the Context rail and sessions row for everyday use - #649
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (5)
📒 Files selected for processing (10)
Disabled knowledge base sources:
WalkthroughThe session inspector now uses quiet context status displays, collapsible context and turn sections, configurable initial turn visibility, updated tests, and unified current-session row highlighting. ChangesSession inspector display and disclosure updates
Estimated code review effort: 3 (Moderate) | ~25 minutes Change: Other Sequence Diagram(s)sequenceDiagram
participant User
participant SessionInspector
participant SessionContextTurnsSection
participant SessionInspectorDisclosureHead
User->>SessionInspector: Opens session inspector
SessionInspector->>SessionContextTurnsSection: Passes turnsDefaultOpen
User->>SessionInspectorDisclosureHead: Clicks Turns
SessionInspectorDisclosureHead->>SessionContextTurnsSection: Toggles expanded state
SessionContextTurnsSection->>User: Displays turn rows or empty state
✨ 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 |
|
| Filename | Overview |
|---|---|
| web/src/systems/session/lib/session-context-view.ts | Removes routine provenance output while retaining caveat chips, stale status, warning policy, and meter geometry. |
| web/src/systems/session/components/session-context-turns-section.tsx | Moves turn and compaction history into a collapsed disclosure with count metadata and preserved pagination. |
| web/src/systems/session/components/session-inspector-section.tsx | Adds a shared accessible disclosure heading composed from existing UI primitives. |
| web/src/systems/session/components/session-list/session-list-row.tsx | Removes the current-row accent bar while preserving tint, font weight, and aria-current. |
| web/src/systems/session/components/tests/session-inspector.test.tsx | Updates coverage for quiet meter output, folded turns, disclosure expansion, and retained turn details. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
Context[Session context data] --> View[Context view model]
View --> Meter[Quiet meter]
View --> Tooltip[Quiet tooltip]
Context --> Turns[Turns and compaction markers]
Turns --> Fold[Collapsed Turns disclosure]
Fold --> Details[On-demand history]
Session[Session list state] --> Current[Current-row semantics]
Current --> Tint[Selected tint]
Current --> Aria[aria-current]
Reviews (3): Last reviewed commit: "refactor: quiet the Context rail and ses..." | Re-trigger Greptile
3521253 to
f3e6fa0
Compare
|
React Doctor found no new issues. 🎉 Reviewed by React Doctor for commit |
The composer tooltip and the Context meter drop their provenance chrome (`reported` chip, `as of turn <id> · HH:MM:SS · Compaction runs at 85%` line); a chip now renders only for a caveat state and the threshold stays as the bar tick. Turns ships folded behind the same chevron head as CompozyOS context, carrying the turn count as its meta. The current row in the sessions sidebar keeps the selected tint and loses its accent left bar. Co-Authored-By: Claude Fable 5.1 <[email protected]>
f3e6fa0 to
327f336
Compare
Consolidated integration dispositionSquash merge: Greptile and React Doctor reported no actionable new issues. CodeRabbit initially hit its rate limit; its automatic post-rebase attempt failed because the PR had been merged. Its walkthrough and green status are not completed-review coverage. No second review was requested, and no reviewer settings were changed. Rebase changed only impact-record placement; the production patch remained unchanged. Original author captures were visually inspected for folded Turns, warning cues and selected-row tint, separately from CI runtime evidence. All pages of general/inline reviews and review threads, including outdated and collapsed material, were collected. Existing test owners were retained. Blanket docstring quotas and invitations to change reviewer filters do not override repository conventions and were not treated as production defects. No reviewer configuration, branch protection or published main history was rewritten. Validation: integrated-main CI — passed: 26 successful jobs, with only the PR-title job skipped because this is a main push. Heavy verification ran only in GitHub CI under the explicit user override. Authorship and verification are agent-based, not human approval. React Doctor on integrated main: 69/100, zero errors, 178 baseline warnings, unchanged from preceding main. PR-diff analyses reported no new issues; this is not a zero-warning full scan. |
What & why
The Context rail shows routine usage numbers without a reported chip or turn/clock line. Caveat, stale and compaction signals remain truthful. Turns starts folded with its count and opens access to turn details and compaction markers. Current-session selection retains its tint and accessibility without an accent bar.
Original implementation authored with Claude Code; integration and remediation by Codex (GPT-6 Astra). No human verification is claimed.
Squash-merged as
e7e4a7e329499a56fa11b364ed453a1e9d5447f7. All five PRs (#645–#649) are integrated and verified on main.How you verified it
Owning session-inspector and session-list suites, plus the real-daemon session-context browser scenario, are the validation owners. Earlier local results were reported by the author; this integration executor does not claim to have rerun them. Final validation is CI-only. Greptile and React Doctor report no new issues. CodeRabbit was initially rate-limited; its automatic attempt after rebase failed because the PR had already been merged. The published walkthrough and successful status are not completed-review coverage. It produced no actionable findings. Greptile and React Doctor completed their analyses; no second review was requested.
CI-only user override: no local make gate, builds, tests, typechecks or React Doctor runs. Local work is limited to source inspection, formatting and diff checks. Final integrated-main CI passed on
e7e4a7e329499a56fa11b364ed453a1e9d5447f7: 26 successful jobs covering Go lint/build/race, frontend lint/types/tests/build, native Windows, real role/Goal integrations, and web/runtime/desktop E2E. PR-title validation is skipped on push; no applicable CI job failed or remained pending.All available general, inline, outdated and collapsed first-review findings were collected. One consolidated remediation round is authorized; no second review or final-head reapproval is requested. Generic docstring quotas do not override the repository's requirement to comment non-obvious intent rather than repeat identifiers; no reviewer configuration or check is changed.
Author-provided captures at f3e6fa0, now visually inspected by the integration executor (static image inspection, not a fresh runtime walk): plain rail, warning rail, sessions sidebar.
Impact
Web presentation only. No wire/config/native ID, hook, SDK, persistence or isolation change. Context meter/sidebar and session-sidebar QA scenarios plus impact audit are updated. Existing turn details remain accessible. No site or official skill changes.
Cross-surface audit:
docs/_memory/change-impact.md.make gatepasses locally; this PR is delivered only after its required CI checks are green — local execution waived by explicit CI-only user instruction.