Skip to content

Fix Reviewers recheck claim, add Changes tab docs#694

Merged
selfcontained merged 1 commit into
mainfrom
agt_38bf51b676b8/job-doc-ter-cd3279c5
Jun 26, 2026
Merged

Fix Reviewers recheck claim, add Changes tab docs#694
selfcontained merged 1 commit into
mainfrom
agt_38bf51b676b8/job-doc-ter-cd3279c5

Conversation

@selfcontained

Copy link
Copy Markdown
Owner

Summary

  • Reviewers docs: Fixed inaccurate claim that "every review includes a recheck pass." Since PR Skip recheck round-trip on clean approval #679, clean approvals (approve + zero feedback) skip the round-trip — updated the paragraph to describe both paths.
  • Agents docs: Added "Changes tab" section documenting the unified diff viewer (file tree, syntax highlighting, inline commenting) introduced in v0.23.9.
  • Tips: Added ambient tip for the Changes tab to help users discover it.

Deferred to next run

  • Deep-dive the Agents docs section generally (the next_focus for the next run)
  • Backlog items unchanged

Files changed

File Change
apps/web/src/components/app/docs-sections/personas.tsx Fixed recheck-skip wording
apps/web/src/components/app/docs-sections/agents.tsx Added Changes tab section
apps/web/src/lib/tips/tips.ts Added changes-tab ambient tip

🤖 Generated by docs-audit job

The Reviewers docs-pane section claimed "Every review includes a
recheck pass" — since PR #679 clean approvals (approve + zero
feedback) skip the round-trip entirely. Updated the paragraph to
describe both paths.

Added a new "Changes tab" section to the Agents docs covering the
unified diff viewer, file tree, and inline commenting. Added a
matching ambient tip.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@selfcontained
selfcontained merged commit 2be6675 into main Jun 26, 2026
1 check passed
@selfcontained
selfcontained deleted the agt_38bf51b676b8/job-doc-ter-cd3279c5 branch June 26, 2026 04:10
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