Skip to content

Commit 4f2f188

Browse files
fix(workflows): simplify change-risk to use add-comment instead of PR review (#443)
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>
1 parent 3bddf9e commit 4f2f188

2 files changed

Lines changed: 8 additions & 31 deletions

File tree

.github/workflows/change-risk.lock.yml

Lines changed: 5 additions & 24 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/change-risk.md

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,9 @@ tools:
1414
network:
1515
allowed: [defaults, rust]
1616
safe-outputs:
17-
submit-pull-request-review:
18-
max: 1
19-
footer: "if-body"
2017
add-comment:
2118
max: 1
19+
hide-older-comments: true
2220
---
2321

2422
# Change Risk Assessor
@@ -92,7 +90,7 @@ Analyze the PR diff against these risk categories:
9290

9391
## Output
9492

95-
Submit a PR review with your verdict. Structure the review body as:
93+
Post a comment on the PR with your verdict using `add-comment`. Structure the comment as:
9694

9795
```
9896
## ⚡ Change Risk Assessment
@@ -111,6 +109,4 @@ Submit a PR review with your verdict. Structure the review body as:
111109
[List any breaking changes, or "None identified"]
112110
```
113111

114-
Keep the review body **concise** — focus on what matters. If approving a clean PR, a short summary is sufficient. Only expand into detailed findings when risks are found.
115-
116-
**Important**: Use `submit-pull-request-review` with event `APPROVE` or `REQUEST_CHANGES` — do not just post a comment. Your review should directly gate the PR.
112+
Keep the comment **concise** — focus on what matters. If approving a clean PR, a short summary is sufficient. Only expand into detailed findings when risks are found.

0 commit comments

Comments
 (0)