Skip to content

fix(live-room): restore mouse wheel scrollback#400

Merged
spytensor merged 2 commits into
mainfrom
chore/project-truth-alignment
May 27, 2026
Merged

fix(live-room): restore mouse wheel scrollback#400
spytensor merged 2 commits into
mainfrom
chore/project-truth-alignment

Conversation

@spytensor
Copy link
Copy Markdown
Owner

Summary

  • Align active project truth sources after v0.9.19: AGENTS.md, TASKS.md, PR/issue templates, development docs, and source-repo ignore rules.
  • Add scripts/check-project-truth.py plus a CI project truth job to catch future version/tracker/path drift.
  • Fix the live-room mouse-wheel history regression by enabling mouse capture by default again, with COREROOM_MOUSE_CAPTURE=0 as the opt-out.

Architecture impact

  • No change — follows the existing live-room scrollback and CoreRoom truth/evidence model.
  • Refines an open question.
  • Amends a locked decision.

Linked issues

none — direct user-scoped maintenance and bugfix request in CoreRoom thread th-19e67d13314-7.

Acceptance criteria

  • Active worker guidance no longer names v0.8/v0.8.0 - CoreRoom Console Data Plane Tracker #238 as current work.
  • PR and issue templates reference the current .coreroom/ state path and no-active-tracker policy.
  • Source repo ignores local .coreroom/ dogfood state while preserving legacy .coderoom/ ignore.
  • Project truth drift check runs in CI.
  • Mouse wheel events are captured by default so Room history scrolls inside the TUI.
  • Users can opt out of mouse capture with COREROOM_MOUSE_CAPTURE=0.

Test plan

  • python3 scripts/check-project-truth.py
  • python3 -m py_compile scripts/check-project-truth.py
  • cargo fmt --all -- --check
  • cargo test --lib write_enter_commands
  • cargo test --lib mouse_capture
  • cargo test --lib mouse_wheel
  • cargo test --lib page_keys_scroll_by_half_a_viewport
  • cargo test --lib scrolled_back
  • cargo clippy --all-targets --all-features -- -D warnings
  • git diff --check

Evidence packet

  • WorkOrder: none; direct user-scoped maintenance/bugfix.
  • Gate threads:
    • th-19e67817468-3 project truth alignment — closed with accepted-risk bypass for missing independent cross-model review.
    • th-19e67c99bf2-5 mouse-wheel scrollback regression — closed with accepted-risk bypass for missing independent cross-model review.
  • Changed files:
    • .github/ISSUE_TEMPLATE/bug.yml
    • .github/PULL_REQUEST_TEMPLATE.md
    • .github/workflows/ci.yml
    • .gitignore
    • AGENTS.md
    • CHANGELOG.md
    • README.md
    • TASKS.md
    • docs/DEVELOPMENT.md
    • scripts/check-project-truth.py
    • src/console_room_runtime.rs
  • Snapshot/fixture evidence: not applicable; no rendered layout/view-model fixture changed.
  • Root cause for bugfix: v0.9.18 made mouse capture opt-in, so wheel events stopped reaching crossterm Event::Mouse and handle_mouse never adjusted Room scroll_offset.
  • Risks: mouse capture can make terminal-native text selection less convenient; documented opt-out is COREROOM_MOUSE_CAPTURE=0.
  • Rollback: revert this PR; keyboard scrollback remains available either way.
  • Unverified items: manual real-terminal wheel smoke after merge is still recommended.

Tracker update

  • Tracker issue: none; AGENTS.md currently says Primary tracker: none.
  • Issue checkbox updated:
    • yes
    • no, reason: no active tracker / no linked issue.
  • Milestone AC updated, if applicable:
    • yes
    • no, reason: no active milestone.
  • Evidence Ledger row updated:
    • yes
    • no, reason: no active tracker.

Risk and rollback

  • Risk: default mouse capture restores wheel scrolling but changes copy/selection ergonomics in some terminals.
  • Rollback: revert the PR or set COREROOM_MOUSE_CAPTURE=0 for a session.

Out of scope

  • No release version bump.
  • No GitHub milestone closure or tracker editing.
  • No redesign of scrollback layout, working cards, or keyboard navigation.

@spytensor spytensor merged commit 2a35ae7 into main May 27, 2026
6 checks passed
@spytensor spytensor deleted the chore/project-truth-alignment branch May 27, 2026 05:07
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