Skip to content

skills(review-reviewers): mark tend-triage no-op on own tracking-issue creation as a Non-issue - #802

Merged
max-sixty merged 1 commit into
mainfrom
hourly/review-30711849018
Aug 2, 2026
Merged

skills(review-reviewers): mark tend-triage no-op on own tracking-issue creation as a Non-issue#802
max-sixty merged 1 commit into
mainfrom
hourly/review-30711849018

Conversation

@tend-agent

Copy link
Copy Markdown
Collaborator

Problem

review-reviewers proposed a fix (a tend-triage label-skip for the bot's own monthly tracking issues) that a maintainer rejected as not worth the complication: "The wasted runs are real, but each one no-ops through the existing self-conversation guard, so the cost is a few CI minutes a month. Against that, the skip permanently puts tend-internal label names in .config/tend.yaml plus a regenerated workflow to keep in sync. The guard already covers this case."

The pattern the fix targeted — tend-triage firing on the issues: opened event when the bot's own review-*-tracking issue is created, reading it, and exiting via the self-conversation guard — is structural and recurring (it fires every month on the 1st). The review-reviewers Non-issues list already covers tend-mention no-ops on the bot's own tracking-issue comments, but not tend-triage on tracking-issue creation. So without codification a future run will re-derive the same fix, re-open it, and get re-rejected — pure maintainer churn, which is exactly what the Non-issues list exists to prevent.

Fix

Add one Non-issue bullet marking the tend-triage-on-own-tracking-issue-creation no-op as expected behavior, and stating the accepted disposition (record as carry observations; do not propose a label-skip/filter, because the standing config+workflow maintenance outweighs the CI minutes the guard already absorbs).

Per the repo's skill-authoring rules, the bullet states the structural rule without citing the specific PR/case that produced it.

Gate assessment

  • Evidence level: codifies explicit maintainer feedback (a single, unambiguous rejection with reasoning). The underlying no-op pattern is documented recurring (≥3 months of tracking-issue-creation triage runs).
  • Structural vs stochastic: structural — the trigger fires deterministically on the fixed label every month and always no-ops.
  • Change type: small addition to an existing list (prevents future work rather than adding behavior).
  • Invisible failure mode: a re-proposal would not surface as a CI failure — it surfaces as a re-rejected PR and maintainer churn — so codification is the only thing that catches it next time.
  • Passes both gates: yes.

Evidence log: https://gist.github.com/tend-agent/e08f6e62d6478163cb425a75648eb7e4

…e creation as a Non-issue

A maintainer rejected a label-skip fix for tend-triage firing on the
bot's own monthly tracking-issue creation, on the grounds that the guard
already absorbs it cleanly and the skip's standing maintenance (tend
label names in config + a regenerated workflow) outweighs the few CI
minutes a month saved. Codify the rejection in the Non-issues list so
future runs don't re-propose the same fix.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@tend-agent tend-agent added the claude-behavior Behavioral issues found by review-reviewers label Aug 1, 2026
@max-sixty
max-sixty merged commit b6b64f4 into main Aug 2, 2026
5 checks passed
@max-sixty
max-sixty deleted the hourly/review-30711849018 branch August 2, 2026 05:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claude-behavior Behavioral issues found by review-reviewers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants