Skip to content

Commit 9e5bdfb

Browse files
BenBtgCopilot
andcommitted
fix(bug-fix): accept workflow-authored assessment comments from bot/service accounts
Address the open Copilot finding on assessment-author matching. The workflow previously required the prior assessment comment to be authored by `github-actions[bot]`. That is too strict for portable repos where bug-assess may post through a different bot/service account token. Updated Step 1 to select the most recent assessment comment that appears workflow-authored by combining: - bot/service-account authorship, and - expected bug-assess structure (assessment header plus remediation/files/tests sections). This keeps the spoof-resistance intent while removing dependence on one fixed login. Recompiled with gh-aw v0.79.8 and kept checkout v7.0.0 pin fixups. Assisted-by: GitHub Copilot (model: Claude Opus 4.8, autonomous) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
1 parent 15473d5 commit 9e5bdfb

2 files changed

Lines changed: 7 additions & 2 deletions

File tree

.github/workflows/bug-fix.lock.yml

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/bug-fix.md

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,12 @@ whose first line has the shape:
7676
**Bug assessment — <slug>:** <Valid | Likely valid, needs reproduction | Invalid> · severity **<critical | high | medium | low>**
7777
```
7878

79-
Find the **most recent** such assessment comment **authored by `github-actions[bot]`** (the `bug-assess` workflow). If there is more than one, use the latest workflow-authored one. If no workflow-authored assessment exists, follow the "no assessment" path below.
79+
Find the **most recent** such assessment comment that appears
80+
**workflow-authored**: the author is a **bot/service account** and the comment
81+
matches the expected `bug-assess` structure (assessment header plus sections
82+
like **Proposed Remediation**, **Files likely to change**, and **Tests to add or
83+
update**). If there is more than one, use the latest matching one. If no
84+
workflow-authored assessment exists, follow the "no assessment" path below.
8085
If **no** assessment comment exists on the issue:
8186

8287
1. Add **one** comment explaining that a fix cannot be proposed because no

0 commit comments

Comments
 (0)