Fix: Use workflow ID instead of workflow name for comment search#13072
Fix: Use workflow ID instead of workflow name for comment search#13072
Conversation
…tead of GITHUB_WORKFLOW Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
Agent Container Tool Check
Result: 10/12 tools available Note: Core shell tools, version control, JSON/YAML processing, HTTP tools, GitHub CLI, and Node/Python/Go runtimes are all available.
|
|
GitHub MCP merged PRs: Add MCP tool usage statistics to audit command; Ignore safe-outputs.jobs messages in handler managers ✅
|
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
Smoke Test: Claude ✅Last 2 Merged PRs:
Test Results:
Overall Status: PASS ✅
|
The
hide-older-commentsfeature was searching by workflow name ("Smoke Claude") instead of workflow ID ("smoke-claude"), causing it to miss previous comments with matching<!-- gh-aw-workflow-id: {id} -->markers.Changes
process.env.GITHUB_WORKFLOWtoprocess.env.GH_AW_WORKFLOW_ID(line 305)GH_AW_WORKFLOW_IDenvironment variableThe workflow compiler already sets
GH_AW_WORKFLOW_IDcorrectly viaGetWorkflowIDFromPath()in the safe-outputs job. This change aligns the comment handler with that existing behavior.Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.