githubnext/chopinPublic

Let's compose rich plans together

Fix PR readiness pilot review and worker wakeups #400

MergedMaggieAppletonmerged 2 commits intomainfromrefs/pull/400/head
  1. MA

    ## Why

    The first #372 pilot prepared a valid conflict resolution but sent a 153 KB head-to-merge diff through a safe-output field capped at 10 KB. It never published the repair. The coordinator also did not wake when that worker finished because its dispatch used `GITHUB_TOKEN`.

    ## Change

    - Review the exact diff from Git's automatic merge tree to the proposed merge. Trusted validation checks clean paths and matches the reviewed bytes before publication. The real #372 review shrinks to 1,177 bytes. - Bound reviews to the actual 10,240-byte tool limit and send larger repairs for human review. - Dispatch workers and CI with the existing maintenance token so completion can wake the coordinator. That token needs Actions write permission.

    ## Verification

    `bun test scripts/pr-maintenance` (190 pass), `bun run ci`, `bun run types`, and `gh aw compile pr-readiness-worker` pass. The downloaded #372 proposal validates against the new review base.

    ## Review flow

    ![Diagram comparing the old 153 KB head-to-proposal review with the new 1.2 KB automatic-merge-to-proposal review](https://github.com/user-attachments/assets/58ecfc5b-a682-4aea-bea7-fb2559c7918a)

  2. MaggieAppletonauto-rebase-enabledlast month on GitHub

  3. MaggieAppletonmerged e33bbda53008last month on GitHub

  4. MaggieAppletonclosedlast month on GitHub

  5. MaggieAppletonhead-ref-deletedlast month on GitHub