Skip to content

fix: use plain comments for interactive triage actions#1329

Open
ralphbean wants to merge 1 commit into
mainfrom
fix/602-sticky-comment-conversational-flow
Open

fix: use plain comments for interactive triage actions#1329
ralphbean wants to merge 1 commit into
mainfrom
fix/602-sticky-comment-conversational-flow

Conversation

@ralphbean
Copy link
Copy Markdown
Contributor

Summary

  • Only the sufficient action uses sticky post-comment now (triage summaries update in place)
  • Interactive actions (insufficient, blocked, duplicate) post as regular gh issue comment so the conversation reads chronologically
  • Added test case insufficient-uses-plain-comment to assert the new behavior

Closes #602

Test plan

  • post-triage-test.sh — all 35 tests pass
  • make lint clean

🤖 Generated with Claude Code

Only the "sufficient" action (triage summary) uses sticky comments now.
Interactive actions (insufficient/needs-info, blocked, duplicate) post
as regular comments so the conversation reads chronologically and the
user's replies remain in context.

Closes #602

Assisted-by: Claude claude-opus-4-6 <noreply@anthropic.com>
Signed-off-by: Ralph Bean <rbean@redhat.com>
@github-actions
Copy link
Copy Markdown

Site preview

Preview: https://ca647349-site.fullsend-ai.workers.dev

Commit: 4f0cdf2d34f5719326d9f36d8592308b543899bd

@fullsend-ai-review
Copy link
Copy Markdown

Review

Findings

No findings.

@fullsend-ai-review fullsend-ai-review Bot added the ready-for-merge All reviewers approved — ready to merge label May 21, 2026
Copy link
Copy Markdown
Contributor

@waynesun09 waynesun09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logic inversion is correct and well-motivated. The new insufficient-uses-plain-comment test covers the key behavioral change. Clean, minimal PR — no bugs or security concerns found across 10 review agents.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-merge All reviewers approved — ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sticky comments break conversational flow for needs-info interactions

2 participants