You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Improve the smoke-safeoutputs-discussions workflow prompt so the agent correctly targets the discussion it just created (using a temporary ID) rather than a hardcoded pre-existing discussion.
update-discussion agent hardcoded discussion_number: 45 because it had no guidance on how to reference the created discussion. This should instead use the temporary ID from the create-discussion result.
add-comment was never tested — the agent didn't produce add_comment messages, likely because the handler's target: "triggering" had no context and wasn't surfaced to the agent.
Approach
Open the smoke test workflow: gh-aw-mcpg:.github/workflows/smoke-safeoutputs-discussions.md
Note: this file is in the gh-aw-mcpg repository, not gh-aw. Coordinate with gh-aw-mcpg maintainers or submit a PR there.
Update the agent prompt to:
Instruct the agent to save the temporary ID from the create_discussion result (e.g. #aw_disc1)
Use that temporary ID in update_discussion.discussion_number and close_discussion.discussion_number
Include an add_comment message targeting the created discussion
Update the workflow config section if needed to enable target: "created" once that is implemented (see companion issue)
Files to Modify
gh-aw-mcpg:.github/workflows/smoke-safeoutputs-discussions.md — update agent prompt and config
Acceptance Criteria
Agent prompt explicitly instructs use of temporary IDs for cross-step references
close-discussion targets the created discussion (not a pre-existing one)
update-discussion targets the created discussion (not a pre-existing one)
add-comment is exercised and passes in the smoke run
Smoke test run produces 0 failures (all 4 enabled tool types succeed)
Objective
Improve the
smoke-safeoutputs-discussionsworkflow prompt so the agent correctly targets the discussion it just created (using a temporary ID) rather than a hardcoded pre-existing discussion.Context
From issue #23283 smoke test findings:
close-discussioncorrectly rejected the agent's attempt to close discussion Fix #664 - track added actions and only emit ai-reaction action if feature is used #45 (wrong labels) — but the prompt should have instructed the agent to close the discussion it just created, which is auto-labeled withsmoke-testbycreate-discussion.update-discussionagent hardcodeddiscussion_number: 45because it had no guidance on how to reference the created discussion. This should instead use the temporary ID from thecreate-discussionresult.add-commentwas never tested — the agent didn't produceadd_commentmessages, likely because the handler'starget: "triggering"had no context and wasn't surfaced to the agent.Approach
gh-aw-mcpg:.github/workflows/smoke-safeoutputs-discussions.mdgh-aw-mcpgrepository, notgh-aw. Coordinate with gh-aw-mcpg maintainers or submit a PR there.create_discussionresult (e.g.#aw_disc1)update_discussion.discussion_numberandclose_discussion.discussion_numberadd_commentmessage targeting the created discussiontarget: "created"once that is implemented (see companion issue)Files to Modify
gh-aw-mcpg:.github/workflows/smoke-safeoutputs-discussions.md— update agent prompt and configAcceptance Criteria
close-discussiontargets the created discussion (not a pre-existing one)update-discussiontargets the created discussion (not a pre-existing one)add-commentis exercised and passes in the smoke runReferences
Related to Safe-outputs discussions smoke test: update-discussion API error and close-discussion label enforcement #23283