Skip to content

chore: silent triage on already-final-state comments#438

Merged
bokelley merged 1 commit into
mainfrom
bokelley/triage-silent-final-state
May 3, 2026
Merged

chore: silent triage on already-final-state comments#438
bokelley merged 1 commit into
mainfrom
bokelley/triage-silent-final-state

Conversation

@bokelley

@bokelley bokelley commented May 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Mirrors adcontextprotocol/adcp#3938 — cuts read-receipt noise from the triage routine.

The Comment-engagement branch for already-final-state issues (PR drafted, deferred with linkage, flagged for human) was posting brief acknowledgments ("acknowledged — noted", "standing by for CI green") that announced the routine had read the thread without adding signal. The routine now stays silent on those threads by default; engagement requires the new info to actually change the disposition, in which case it re-runs experts (rule 3 path) rather than posting a bare ack.

Same fix applied across all four AdCP repos with active triage routines.

Test plan

  • Next comment.created run on a final-state issue with an ack-shaped comment: routine silent.
  • Substantive challenges (rule 3 path) and Clarify-unlocks (rule 4 path) unchanged.
  • First-pass triage (auto.opened) unchanged.

🤖 Generated with Claude Code

Cuts read-receipt noise. The "comment engagement → already-final-state"
branch was producing "acknowledged — noted" / "standing by for CI green"
comments that announced the routine had read the thread without adding
signal. Engagement on a final-state issue now requires the new info to
actually change the disposition; in that case the routine re-runs experts
rather than posting a bare ack.

Mirrors adcontextprotocol/adcp#3938.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@bokelley bokelley force-pushed the bokelley/triage-silent-final-state branch from 120b632 to fe9a408 Compare May 3, 2026 03:06
@bokelley bokelley changed the title triage: silent on already-final-state comments chore: silent triage on already-final-state comments May 3, 2026
@bokelley bokelley merged commit 2aa63f1 into main May 3, 2026
16 of 22 checks passed
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.

1 participant