Skip to content

feat(review): post one plain-language review - #1582

Merged
TheGreatAxios merged 2 commits into
mainfrom
cl-11823-post-one-human-readable-review-from-every-review-pass
Oct 30, 2026
Merged

TheGreatAxios merged 2 commits into
mainfrom
cl-11823-post-one-human-readable-review-from-every-review-pass

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • /review waits for every pass, then posts one GitHub review.
  • The body says what the branch does and what goes wrong in plain language, then lists each finding as path:line and the failure mode.
  • Passes do not post their own reviews, and the note back to the operator is that same body plus the review URL.

Verification

  • bun run check exits 0 after the edit (9552 pass, 0 fail), including lint, typecheck, dead-export guard, and build.
  • The gh pr review example is one command with --body-file. A review of the commit found no findings.

Fixes CL-11823

@linear-code

linear-code Bot commented Oct 30, 2026

Copy link
Copy Markdown

CL-11823

@corbits-triage

corbits-triage Bot commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

@TheGreatAxios TheGreatAxios changed the title docs(review): post one plain-language review feat(review): post one plain-language review Oct 30, 2026
Each pass was posting its own GitHub review, so one diff showed up as a comment and an approval. Dispatch now waits for every pass and posts a single review: plain language first, then path and failure mode.
@TheGreatAxios
TheGreatAxios force-pushed the cl-11823-post-one-human-readable-review-from-every-review-pass branch from 98c5d95 to a744ed7 Compare October 30, 2026 04:07
A review was a paragraph plus a bare path. Each finding is now one bullet: blocking, should-fix, or informational, then what goes wrong, then the line range.
@TheGreatAxios
TheGreatAxios merged commit d3c12be into main Oct 30, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant