fix(supervise): retain unassessed native child reports - #1492
Merged
Merged
Conversation
tangletools
previously approved these changes
Oct 1, 2026
tangletools
left a comment
Contributor
There was a problem hiding this comment.
✅ Auto-approved PR — 66999884
Blanket team auto-approval is intentional. The merge gates are CI and codex-p1.
No automated review runs on this PR. This approval rests on the rule above alone.
tangletools · auto-approval · reason: blanket_auto_approve · 2026-10-01T13:07:34Z
tangletools
approved these changes
Oct 1, 2026
tangletools
left a comment
Contributor
There was a problem hiding this comment.
✅ Auto-approved PR — 5f75a9e2
Blanket team auto-approval is intentional. The merge gates are CI and codex-p1.
No automated review runs on this PR. This approval rests on the rule above alone.
tangletools · auto-approval · reason: blanket_auto_approve · 2026-10-01T13:24:34Z
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
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.
Problem
A recursive external child with Runtime tools is a managed node. When its assignment has no completion check (
resolveDeliverable: () => null), the backend drops its terminal artifact and the finalizer has no checked result. The child finishes successfully but the parent's durable result is null.Change
Retain the newest successfully completed harness artifact through the existing driver metadata port. A nested child without a check may return that evidence only when the checked finalizer produced no result. Its settlement has no quality verdict, and the existing parent await_event receipt explicitly labels it unassessed and supplies the observe_agent read. Accepted submit_result and checked finalizer boundaries remain intact. A checked descendant cannot confer its verdict on the child's raw report.
No authored profile or method workaround can recover the omitted own-harness output through the existing checked-only finalizer. This fixes the shared Runtime owner. Frozen registrations and running pursuits are unchanged. No deployment or release-cohort mutation is part of this PR.
Proof
CI corrections
The first remote CI run identified two existing bridge assertions that still expected a successful child report to be null. They now assert the retained report and absent verdict. It also identified a deterministically reproducible Router test timing assumption inherited from current main. The separate test-only correction awaits the actual retry timer before aborting; Router production code and retry policy are unchanged. All 75 affected cases pass. All four final-head remote CI jobs passed: ci, official-optimizers, packed-cohort and agent-bench. Current-head approval is present and no inline findings remain.
Evidence is retained on GTR under /mnt/traces/frontier-qualification-20260930/runtime-unassessed-managed-child-report-20261001. This is an offline provider integration proof; activation and live fleet proof belong to the normal release and launch owners.