Skip to content

Enable append-only comments for campaign generator workflow#11295

Closed
Copilot wants to merge 6 commits intomainfrom
copilot/analyze-campaign-message-issue
Closed

Enable append-only comments for campaign generator workflow#11295
Copilot wants to merge 6 commits intomainfrom
copilot/analyze-campaign-message-issue

Conversation

Copy link
Contributor

Copilot AI commented Jan 22, 2026

The agentic-campaign-generator workflow's message text implied new comments would be created ("post the link + checklist back on this issue when ready"), but the workflow was updating the activation comment instead.

Changes

  • Added append-only-comments: true to .github/workflows/agentic-campaign-generator.md messages configuration
  • Recompiled workflow to regenerate lock file

Behavior

With append-only-comments: true:

  • run-started message creates the initial comment
  • run-success/run-failure messages create separate comments
  • Each status update is preserved as a distinct comment on the issue
messages:
  append-only-comments: true
  run-started: "### :rocket: Campaign setup started..."
  run-success: "### :white_check_mark: Campaign setup complete..."
  run-failure: "### :x: Campaign setup {status}..."

The setting is handled by notify_comment_error.cjs and update_activation_comment.cjs, which check messagesConfig?.appendOnlyComments to determine whether to create new comments or update existing ones.

Original prompt

Analyze why during campaign generation, message status with append only true still updates the old comment instead of creating a new one, see example here: https://github.com/githubnext/gh-aw/issues/11294


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Investigate message status update in campaign generation Enable append-only comments for campaign generator workflow Jan 22, 2026
Copilot AI requested a review from mnkiefer January 22, 2026 13:57
@mnkiefer
Copy link
Contributor

@copilot This is the derivative. Should be part of generator.go?

@mnkiefer mnkiefer marked this pull request as ready for review January 22, 2026 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants