Skip to content

Use gh-aw build 1508133d7 and recompile update-pr-body#616

Merged
strawgate merged 2 commits intomainfrom
chore/use-ghaw-1508133d7
Mar 9, 2026
Merged

Use gh-aw build 1508133d7 and recompile update-pr-body#616
strawgate merged 2 commits intomainfrom
chore/use-ghaw-1508133d7

Conversation

@strawgate
Copy link
Collaborator

Summary

  • pin GH_AW_BUILD_VERSION in Makefile to 1508133d7 from strawgate/gh-aw
  • regenerate gh-aw-update-pr-body.lock.yml with that compiler build
  • include the corresponding safe-output-update-pr fragment change so lock output carries explicit target wiring for update-pull-request

Test plan

  • build local gh-aw from strawgate/gh-aw branch containing the pull_request_target triggering fix
  • run sync + compile in ai-github-actions with local binary
  • verify resulting changes are limited to Makefile, update-pr fragment, and compiled lock file

Made with Cursor

Pin the compiler build to strawgate/gh-aw commit 1508133d7 and regenerate the update-pr-body lock workflow so update_pull_request target configuration includes explicit triggering target handling.

Made-with: Cursor
@coderabbitai
Copy link

coderabbitai bot commented Mar 9, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: dcc0ea22-6881-4ff6-9923-9e4a8259de71

📥 Commits

Reviewing files that changed from the base of the PR and between 039b52c and eef4d48.

📒 Files selected for processing (2)
  • .github/workflows/gh-aw-fragments/safe-output-update-pr.md
  • .github/workflows/gh-aw-update-pr-body.lock.yml

📝 Walkthrough

Walkthrough

Three files changed: the safe-outputs.update-pull-request workflow fragment gained a target field set to ${{ inputs.target-pr-number || 'triggering' }} (fallback to "triggering"); the compiled/lock workflow (.github/workflows/gh-aw-update-pr-body.lock.yml) was updated to propagate the new target input into the safe-outputs tool/handler and documentation; the Makefile's GH_AW_BUILD_VERSION value was shortened from a full commit hash to 1508133d7.

Possibly related PRs

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/use-ghaw-1508133d7
  • 🛠️ Update Documentation: Commit on current branch
  • 🛠️ Update Documentation: Create PR

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions bot added the medium_boom Medium PR blast radius; likely benefits from human review label Mar 9, 2026
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requesting changes: the new expression quoting introduces invalid GitHub expression syntax in the compiled workflow. Evidence: .github/workflows/gh-aw-update-pr-body.lock.yml now contains ${{ inputs.target-pr-number || \"triggering\" }} (e.g., lines 686 and 1500 in the diff), which fails expression lexing (got unexpected character '\\' while lexing expression). Please switch to single-quoted expression literals in the fragment (e.g., ${{ inputs.target-pr-number || 'triggering' }}) and recompile.


What is this? | From workflow: Trigger PR Review

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.

Use single-quoted expression literal for triggering fallback so generated safe output handler config remains valid GitHub expression syntax.

Made-with: Cursor
@strawgate strawgate merged commit b94747a into main Mar 9, 2026
13 of 14 checks passed
@strawgate strawgate deleted the chore/use-ghaw-1508133d7 branch March 9, 2026 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

medium_boom Medium PR blast radius; likely benefits from human review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant