Skip to content

fix: route existing PR body updates through file_pr#686

Merged
brickfrog merged 2 commits into
mainfrom
main.gh-shim-gap-1783615200832-2935042-0
Jul 9, 2026
Merged

fix: route existing PR body updates through file_pr#686
brickfrog merged 2 commits into
mainfrom
main.gh-shim-gap-1783615200832-2935042-0

Conversation

@brickfrog

Copy link
Copy Markdown
Owner

Fixes choir-q6z0. Adds leaf prompt guidance that PR lifecycle writes are restricted to Choir tools/human paths across gh and GitHub App/connector surfaces, and makes file_pr update an existing open PR body via injected gh pr edit --body while logging edit failures non-fatally.\n\nVerification: moon test --target native && moon run src/bin/choir_lint --target native (pass: 2264 tests); moon fmt --check (pass).

Verification

Generated by Choir from commands executed in the leaf workspace.

  • moon test --target native
    • exit: 0
    • head: 49c2241
    • output tail:
----- BEG4
IN MOON TEST RESULT -----
{"type":"start","file":"exec_test.mbt","index":9}
----- END MOON TEST RESULT -----
Total tests: 2264, passed: 2264, failed: 0.
  • moon run src/bin/choir_lint --target native

@brickfrog

Copy link
Copy Markdown
Owner Author

Choir: GitHub Actions rollup reports all checks successful on the latest head — re-check review/merge policy on GitHub (not an approval or merge signal by itself).

@brickfrog

Copy link
Copy Markdown
Owner Author

Choir: GitHub Actions rollup reports all checks successful on the latest head — re-check review/merge policy on GitHub (not an approval or merge signal by itself).

@brickfrog brickfrog merged commit d49a22c into main Jul 9, 2026
1 check passed
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