Skip to content

Improve guided tour coverage, focused reading and freshness - #54

Merged
rcosteira79 merged 3 commits into
mainfrom
feat/tour-coverage-and-focused-steps
Nov 13, 2026
Merged

rcosteira79 merged 3 commits into
mainfrom
feat/tour-coverage-and-focused-steps

Conversation

@rcosteira79

@rcosteira79 rcosteira79 commented Nov 13, 2026 •

Copy link
Copy Markdown
Collaborator

Tour stops now show their coverage and related code explicitly, and stop treating a surviving line number as proof that an explanation still matches the source.

  • Add a Not covered list for omitted and partially covered diff blocks, including binary and generated changes. Coverage remains separate from reviewer completion.
  • Group up to eight related ranges across files in a focused before/after view. It follows renderer, layout and wrapping preferences, opens full files for inspection/comments, and returns to the same stop without losing drafts.
  • Separate a short summary from expandable detail, add reading priority, and allow occasional notes with exact captured-source citations and bounded repair on invalid evidence.
  • Keep Resume tour floating at the bottom while paused, including tab detours, and remove the duplicate top action. Match the new detail, note and code controls to the sidebar’s 12px explanation text.
  • Persist content fingerprints, relocate unchanged blocks after line shifts, and flag changed, missing or ambiguous evidence as stale. Older sessions remain readable and explicitly unverified; supporting-file changes invalidate note freshness too.

Markdown/HTML exports retain the reading metadata and evidence. README, the attached-agent contract, and a guided-reading guide are updated, with a new screenshot.

Validation: full bun run verify on CI's Bun 1.3.14 (1,217 passed, 13 skipped), followed by focused checks for the final source-bound and navigation changes. Browser coverage exercises Shiki/Monaco, split/unified, draft retention, full-file return, uncovered PNGs, phase boundaries, and stale source. Existing sidebar/comment regressions also pass.

Freshness is deliberately conservative when diff context or boundaries change. Quote validation establishes provenance; the agent still must inspect callers, guards and tests before drawing conclusions.

The floating resume control also passed desktop/mobile scrolling, keyboard activation, and same-stop return checks in both renderers. Briefing source navigation and grouped-code regressions passed after the change; the documentation screenshot reflects the smaller controls.

@rcosteira79
rcosteira79 merged commit 639a5a0 into main Nov 13, 2026
14 checks passed
@rcosteira79
rcosteira79 deleted the feat/tour-coverage-and-focused-steps branch November 13, 2026 14:43
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