-
Notifications
You must be signed in to change notification settings - Fork 43
Fix: Use workflow ID instead of workflow name for comment search #13072
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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.