fix(agents): triage already-engaged + ship-more (missed in #259)#262
Merged
fix(agents): triage already-engaged + ship-more (missed in #259)#262
Conversation
Mirror of adcontextprotocol/adcp#2957: 1. Already-engaged check before expert consultation. Silent-defer when the issue is assigned to a repo member, has an open PR referencing it, or has a repo-member comment in the last 7 days — a human is probably already working on it in a Conductor workspace / drafts / Slack and the bot's synthesis would pre-frame their thinking. 2. Tighten "never create labels" rule: must run gh label list first, apply only names returned, never POST to /labels, default not-apply on uncertainty. Missing bucket labels go in the comment body plus the run summary. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Mirror of adcontextprotocol/adcp#2958. Flip the default from "flag unless obviously a small bug" to "execute unless breaking or ambiguous." - Drop <150 line scope cap - Drop classification-only-Bug-or-Doc gate - Replace with single non-breaking-vs-breaking binary - "When in doubt: Execute" mantra (inverted on adcp-go — TEE posture means when in doubt, Flag) CODEOWNERS + human review still gate every merge. Generated files, pinned deps, TEE-bound paths (adcp-go) still always Flag. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Two commits intended for #259 that landed on the branch after the merge button was clicked — carrying them forward here as a follow-up.
1. Already-engaged check (mirror of
adcontextprotocol/adcp#2957)Silent-defer when the issue is assigned to a repo member, has an open PR referencing it, or has a repo-member comment in the last 7 days. Prevents competing analysis on work a human is already engaged on (the bot can't see Conductor workspaces / local drafts / Slack).
2. Ship-more default (mirror of
adcontextprotocol/adcp#2958)Default flipped: Execute when outcome is clear; Flag only for genuine ambiguity or breaking changes. Drops <150 line scope cap + classification-only-Bug-or-Doc gate. Adds non-breaking-vs-breaking as the primary binary.
For this repo specifically:
a2a-sdk,httpcore,datamodel-code-generator)Scope and author association are NOT gates. CODEOWNERS + human review still gate merge.
Test plan
🤖 Generated with Claude Code