Skip to content

refactor: quiet the Context rail and sessions row for everyday use - #649

Merged
pedronauck merged 1 commit into
mainfrom
fix/context-rail-quiet
Sep 15, 2026
Merged

pedronauck merged 1 commit into
mainfrom
fix/context-rail-quiet

Conversation

@pedronauck

@pedronauck pedronauck commented Sep 15, 2026 •

Copy link
Copy Markdown
Member

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 gate passes locally; this PR is delivered only after its required CI checks are green — local execution waived by explicit CI-only user instruction.
  • New or changed behavior is covered by tests, or I explained above why not
  • If an agent wrote or co-wrote this, I named it above and verified the result myself — authorship is named; no human verification is claimed.

@vercel

vercel Bot commented Sep 15, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
compozy-site Ready Ready Preview Sep 15, 2026 1:13pm UTC

Request Review

@coderabbitai

coderabbitai Bot commented Sep 15, 2026 •

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 4e19d92c-fad9-4898-ac14-c5643ea2e2d6

📥 Commits

Reviewing files that changed from the base of the PR and between 6b081d4 and 327f336.

⛔ Files ignored due to path filters (5)
  • docs/_memory/change-impact.md is excluded by !**/*.md
  • docs/qa/scenarios/ET-web-session-context-meter.md is excluded by !**/*.md
  • docs/qa/scenarios/ET-web-session-context-sidebar.md is excluded by !**/*.md
  • docs/qa/scenarios/ET-web-session-sidebar-threads.md is excluded by !**/*.md
  • web/src/systems/session/components/stories/session-inspector.stories.tsx is excluded by !**/*.stories.tsx
📒 Files selected for processing (10)
  • web/src/systems/session/components/__tests__/session-inspector.test.tsx
  • web/src/systems/session/components/session-context-control.tsx
  • web/src/systems/session/components/session-context-injected-section.tsx
  • web/src/systems/session/components/session-context-meter-section.tsx
  • web/src/systems/session/components/session-context-turns-section.tsx
  • web/src/systems/session/components/session-inspector-section.tsx
  • web/src/systems/session/components/session-inspector.tsx
  • web/src/systems/session/components/session-list/session-list-row.tsx
  • web/src/systems/session/lib/context-format.ts
  • web/src/systems/session/lib/session-context-view.ts

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


Walkthrough

The 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.

Changes

Session inspector display and disclosure updates

Layer / File(s) Summary
Quiet context status display
web/src/systems/session/lib/session-context-view.ts, web/src/systems/session/lib/context-format.ts, web/src/systems/session/components/session-context-*.tsx
Reported timestamps and summary lines were removed. Context chips are optional. Stale context renders a fixed warning.
Shared disclosure sections
web/src/systems/session/components/session-inspector-section.tsx, web/src/systems/session/components/session-context-injected-section.tsx, web/src/systems/session/components/session-context-turns-section.tsx
A shared disclosure header now renders the chevron, label, metadata, and expanded state. Injected context and turn content use collapsible sections.
Inspector integration and validation
web/src/systems/session/components/session-inspector.tsx, web/src/systems/session/components/__tests__/session-inspector.test.tsx, web/src/systems/session/components/session-list/session-list-row.tsx
SessionInspector forwards turnsDefaultOpen. Tests cover collapsed and expanded turn content. Current sessions use the selected-row background style.

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
Loading
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/context-rail-quiet

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@greptile-apps

greptile-apps Bot commented Sep 15, 2026 •

Copy link
Copy Markdown

Greptile Summary

This PR quiets routine session-context presentation while preserving caveat and compaction signals.

  • Removes routine provenance chips, turn identifiers, clocks, and the meter policy line.
  • Folds turn history behind a disclosure whose heading shows the turn count.
  • Reuses the disclosure heading for injected CompozyOS context.
  • Removes the current-session accent bar while retaining selected tint and accessibility semantics.
  • Updates owning unit tests, stories, QA scenarios, and the cross-surface impact audit.

Confidence Score: 5/5

The PR appears safe to merge, with no actionable correctness, accessibility, security, or repository-rule failures identified.

Routine context details are removed consistently across view models, components, tests, stories, and QA documentation; caveat states and compaction information remain available, turn details remain keyboard-accessible through the disclosure, and current-session rows retain programmatic semantics.

Important Files Changed

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]
Loading

Reviews (3): Last reviewed commit: "refactor: quiet the Context rail and ses..." | Re-trigger Greptile

@github-actions

github-actions Bot commented Sep 15, 2026 •

Copy link
Copy Markdown
Contributor

React Doctor found no new issues. 🎉

Reviewed by React Doctor for commit 327f336.

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]>
@pedronauck
pedronauck force-pushed the fix/context-rail-quiet branch from f3e6fa0 to 327f336 Compare September 15, 2026 13:09
@pedronauck
pedronauck merged commit e7e4a7e into main Sep 15, 2026
18 of 20 checks passed
@pedronauck
pedronauck deleted the fix/context-rail-quiet branch September 15, 2026 13:10
@pedronauck pedronauck mentioned this pull request Sep 15, 2026
@pedronauck

Copy link
Copy Markdown
Member Author

Consolidated integration disposition

Squash merge: e7e4a7e329499a56fa11b364ed453a1e9d5447f7. Integrated main: e7e4a7e329499a56fa11b364ed453a1e9d5447f7.

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.

This branch was successfully deployed

1 active deployment
Preview — 327f336d Deployed Sep 15, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant