Skip to content

feat(review): tools for labeling using Claude Managed Agents - #5407

Draft
purp wants to merge 1 commit into
NVIDIA:mainfrom
purp:feat/5276-labeling-agent-runtime/purp
Draft

purp wants to merge 1 commit into
NVIDIA:mainfrom
purp:feat/5276-labeling-agent-runtime/purp

Conversation

@purp

@purp purp commented Nov 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

Add the runtime foundation for automatic labeling of accepted issues and PRs: a bounded managed classifier adapter and deterministic invocation selection that reuse the standalone /label-issue contract. This draft installs no automatic labeling workflow and performs no label writes.

Related Issue

Closes #5406. Starts Part 3 of #5276; this PR does not close the overall tracker.

Integration order: #5260 supplies accepted-work policy and opening-time PR acceptance labeling; #5355 supplies the standalone skill and migration helpers; #5151 supplies label/routing preparation before observed manual migration; automatic acceptance-triggered labeling follows those observed runs.

Verified target-repository ADMIN access; #5406 has no explicit state:accepted label.

Changes

  • Add a version-pinned CMA session adapter with request preview, configurable budget/deadline, completion validation, bounded event pagination, and failure recovery. The classifier receives prepared evidence and trusted instructions with tools/MCP disabled and no supplied GitHub credentials.
  • Select explicit issue/PR acceptance events and authorized human PR-comment/manual invocations with force support. Opening/reopening alone does not dispatch classification.
  • Leave acceptance, evidence expansion, semantic validation, coordinated writes, checkpoints, and local ambiguity reporting in the shared skill/helper boundary.
  • Register 30 offline tests in mise/branch CI and protect the runtime files with CODEOWNERS.

Before activation: connect #5355's actual interfaces, provision and verify the CMA configuration, coordinate execution per item across entry points, and observe end-to-end runs. Hook #5260's successful acceptance-label write explicitly, including drafts and independently of subsequent reviewer-routing failure: a GITHUB_TOKEN label write does not trigger another workflow. Recheck current acceptance rather than trusting a historical label.

Release Notes

Testing

  • 30 offline Python tests pass using uv run --no-project --python /usr/bin/python3 --offline python -m unittest discover -s tasks/scripts -p '*label*test.py'.
  • Python syntax, TOML task registration, and complete staged git diff --check pass.
  • Scope and documentation review; remove the unused serialization wrapper and stale ambiguity-comment guidance.
  • Ruff/formatting and the mise wrapper: tools were unavailable and downloads were blocked. Markdownlint and Actionlint were not run.
  • Live CMA smoke test and end-to-end labeling: pending configuration and shared-helper integration. No live sessions, label mutations, or automatic dispatch were performed.
  • Unit tests added/updated.
  • E2E tests added/updated (not applicable to the offline foundation).

Checklist

  • Follows Conventional Commits.
  • Commits are signed off (DCO).
  • Internal integration documentation updated.

Signed-off-by: Jim Meyer <jimeyer@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Nov 9, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@purp purp changed the title feat(review): add managed labeling runtime foundation feat(review): automatically label issues and PRs using Claude Managed Agents Nov 9, 2026
@purp purp changed the title feat(review): automatically label issues and PRs using Claude Managed Agents feat(review): tools for labeling using Claude Managed Agents Nov 9, 2026

This branch has not been deployed

No deployments
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.

feat(review): add managed labeling runtime foundation

1 participant