Skip to content

fix(workflows): simplify change-risk to use add-comment instead of PR review#443

Merged
jamesadevine merged 1 commit into
mainfrom
fix/change-risk-safe-outputs
May 7, 2026
Merged

fix(workflows): simplify change-risk to use add-comment instead of PR review#443
jamesadevine merged 1 commit into
mainfrom
fix/change-risk-safe-outputs

Conversation

@jamesadevine
Copy link
Copy Markdown
Collaborator

Summary

The change-risk agentic workflow was using submit-pull-request-review which wasn't working correctly in practice. This simplifies it to use add-comment instead, which reliably posts the risk assessment as a PR comment.

Changes:

  • Removed submit-pull-request-review safe output from frontmatter
  • Kept add-comment with hide-older-comments: true to minimize stale assessments
  • Updated agent instructions to post a comment rather than submit a formal PR review
  • Recompiled lock file with gh aw compile (v0.62.0)

Test plan

  • gh aw compile change-risk compiles cleanly with 0 errors, 0 warnings
  • Verified compiled lock file only exposes add_comment, missing_tool, missing_data, noop tools (no submit_pull_request_review)

… review

Remove submit-pull-request-review safe output and update agent
instructions to post a comment with the risk assessment instead.
Also enable hide-older-comments to minimize stale assessments.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jamesadevine jamesadevine merged commit 4f2f188 into main May 7, 2026
12 checks passed
@jamesadevine jamesadevine deleted the fix/change-risk-safe-outputs branch May 7, 2026 16:01
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