fix(prompt-clustering): add explicit safeoutputs create_discussion call - #49760
Conversation
…t-clustering-analysis Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot suggest stdin syntax for safe outputs , more robust |
There was a problem hiding this comment.
🟢 Ready to approve
The targeted workflow fix is valid, compiled consistently, and introduces no unresolved issues.
This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.
Pull request overview
Fixes missing safe outputs in the prompt-clustering workflow.
Changes:
- Explicitly posts the generated report as a discussion, with
noopfallback. - Regenerates workflow metadata.
- Removes an obsolete Go loop-variable copy.
File summaries
| File | Description |
|---|---|
.github/workflows/prompt-clustering-analysis.md |
Adds explicit discussion publishing instructions. |
.github/workflows/prompt-clustering-analysis.lock.yml |
Updates the compiled body hash. |
cmd/gh-aw/format_list_test.go |
Removes an unnecessary Go loop-variable copy. |
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 0
- Review effort level: Balanced
We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot Please give this PR a finishing pass and run the pr-finisher skill. This PR currently looks unblocked from triage. Please verify readiness, re-run formatting/checks as needed, and push any final polish before handoff.
|
Finishing pass complete on c4ea16b. No additional code changes were needed. Local |
PR Triage
Automated triage — see full report issue for details. Structured data: {
"action": "auto_merge",
"category": "bug",
"pr_number": 49760,
"risk": "low"
}
|
|
🎉 This pull request is included in a new release. Release: |
The Prompt Clustering Analysis workflow was completing its NLP analysis successfully but never posting results — the agent consumed all 22 turns and exited with no safe output produced (
total_items: 0, postureread_only).Root cause: The workflow prompt described what to analyze and how to format the report, but gave no explicit instruction to call
safeoutputs create_discussion. Without it, the agent finished the analysis phase, ran out of turns, and returned garbled output.Changes
prompt-clustering-analysis.md: Added explicit posting instructions at the end of Phase 5 — write the rendered report to/tmp/gh-aw/agent/discussion_body.md, then:Also updated the closing directive to explicitly require
safeoutputs create_discussion(ornoopon failure), matching the pattern used in other daily audit workflows.cmd/gh-aw/format_list_test.go: Removed pre-existingtt := ttloop variable copy flagged by golint as unnecessary since Go 1.22.