Skip to content

Notify parent agents when Guardian stops a subagent - #49312

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/c9bb90827393068d9c92edaf0443a52e6ff57f9f
Sep 29, 2026
Merged

copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/c9bb90827393068d9c92edaf0443a52e6ff57f9f

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

Notify parent agents when Guardian stops a subagent

Why

A subagent stopped by repeated Guardian denials remained interrupted without notifying its parent. The parent needs guidance to report the stop and wait for explicit user confirmation before continuing the blocked work.

What changed

Forward TooManyDenials interruptions from MultiAgentV2 children to their parents with Guardian-specific guidance to inform the user and avoid resuming or retrying until the user explicitly confirms. Preserve the child's Interrupted status and keep ordinary interruptions silent.

Testing

Add integration coverage for parent notifications under CircuitBreakAction::Strict, silence under CircuitBreakAction::Default, and preservation of the child's interrupted status. Add a unit test ensuring long denial messages stay within the notification token limit while retaining the guidance.

## Why

A subagent stopped by repeated Guardian denials remained interrupted without notifying its parent. The parent needs guidance to report the stop and wait for explicit user confirmation before continuing the blocked work.

## What changed

Forward `TooManyDenials` interruptions from `MultiAgentV2` children to their parents with Guardian-specific guidance to inform the user and avoid resuming or retrying until the user explicitly confirms. Preserve the child's `Interrupted` status and keep ordinary interruptions silent.

## Testing

Add integration coverage for parent notifications under `CircuitBreakAction::Strict`, silence under `CircuitBreakAction::Default`, and preservation of the child's interrupted status. Add a unit test ensuring long denial messages stay within the notification token limit while retaining the guidance.

GitOrigin-RevId: c9bb90827393068d9c92edaf0443a52e6ff57f9f
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/c9bb90827393068d9c92edaf0443a52e6ff57f9f branch from 88e6b53 to 6347513 Compare September 29, 2026 16:53
@copyberry
copyberry Bot merged commit 6347513 into main Sep 29, 2026
1 check passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/c9bb90827393068d9c92edaf0443a52e6ff57f9f branch September 29, 2026 16:53
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 29, 2026
@resure
resure deployed to issue-triage September 29, 2026 16:55 — with GitHub Actions Active
@resure
resure deployed to issue-triage September 29, 2026 16:55 — with GitHub Actions Active
@resure
resure deployed to issue-triage September 29, 2026 16:55 — with GitHub Actions Active

This branch was successfully deployed

1 active deployment
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants