Skip to content

Commit a2d3ece

Browse files
Copilotpelikhan
andauthored
Clarify Test Quality Sentinel safe-output comment invocation to prevent Copilot engine permission failures (#39867)
* Initial plan * chore: plan fix for tqs safe-output usage Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Clarify TQS add-comment safe-output usage Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> Co-authored-by: Peli de Halleux <pelikhan@users.noreply.github.com>
1 parent d253194 commit a2d3ece

2 files changed

Lines changed: 10 additions & 2 deletions

File tree

.github/workflows/test-quality-sentinel.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/test-quality-sentinel.md

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -272,7 +272,15 @@ Guideline violations always trigger `REQUEST_CHANGES` regardless of the quality
272272

273273
## Step 7: Post PR Comment with Results
274274

275-
Post a comment to the pull request with the full analysis using `add-comment`. Use the `tqs-report-template` skill for the exact comment format.
275+
Post a comment to the pull request with the full analysis using the `add-comment` safe-output tool (tool call, not shell). Use the `tqs-report-template` skill for the exact comment format.
276+
277+
Use this shape:
278+
279+
```json
280+
{"add_comment":{"body":"<full markdown report>"}}
281+
```
282+
283+
Do **not** invoke `safeoutputs` from bash, and do **not** set `item_number` for this step. Let `add_comment` auto-target the triggering pull request.
276284

277285
## Step 8: Submit PR Review Based on Result
278286

0 commit comments

Comments
 (0)