Skip to content

Version Packages - #263

Merged
jwbron merged 1 commit into
mainfrom
changeset-release/main
Jul 14, 2026
Merged

Version Packages#263
jwbron merged 1 commit into
mainfrom
changeset-release/main

Conversation

@khan-actions-bot

Copy link
Copy Markdown
Contributor

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.

Releases

review@1.6.1

Patch Changes

  • 70c7560: Always submit REQUEST_CHANGES with a non-empty body. Step 6 previously instructed the opposite ("a REQUEST_CHANGES verdict carries at least one blocking inline comment, so submit it with an empty body", dating to [pipeline-dcdad92d][slice-1/12] Foundations: finding schema... #197): GitHub rejects a REQUEST_CHANGES review event with an empty body, and under gh-aw v0.81.6 the safe-output flow posts the review event separately from the inline comments, so the comments do not make it non-empty. Observed live on the v1.5.0 behavior test (Khan/webapp#40793): the reviewer computed REQUEST_CHANGES correctly for two blocking findings, the submission failed with "Review body is required for REQUEST_CHANGES", and the PR was left with the inline comments as a COMMENTED review and no blocking verdict. The latent instruction survived this long because models usually filled the body anyway. The body's first line is now always Changes requested — see inline comments., with the accountability section, note lines, and fingerprint stamp appended as before.

@khan-actions-bot
khan-actions-bot requested review from a team, jaredly and kevinb-khan and removed request for a team July 14, 2026 02:17
@jwbron
jwbron merged commit 27ac109 into main Jul 14, 2026
9 checks passed
@jwbron
jwbron deleted the changeset-release/main branch July 14, 2026 02:18
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.

2 participants