Skip to content

fix: preserve nested fanout subagent tool history#270

Closed
jamesjwood wants to merge 10 commits into
nicobailon:mainfrom
Cambridge-Vision-Technology:cvt/fix-nested-fanout
Closed

fix: preserve nested fanout subagent tool history#270
jamesjwood wants to merge 10 commits into
nicobailon:mainfrom
Cambridge-Vision-Technology:cvt/fix-nested-fanout

Conversation

@jamesjwood

Copy link
Copy Markdown
Contributor

Summary

  • avoid registering the main subagent extension in child mode; fanout children get the child-safe tool from the explicit fanout-child extension
  • preserve live nested subagent tool calls/results inside authorized fanout children so they can observe list/delegate outcomes
  • add regression coverage for duplicate registration and fanout history preservation

Tests

  • npm test
  • npm run test:integration

nicobailon and others added 10 commits May 30, 2026 14:47
Explicit acceptance now owns completion through a bounded same-session self-review loop. The runtime derives acceptance provenance internally, uses an explicit CompletionPolicy to avoid competing completion guards, and keeps raw acceptance reports out of normal output while preserving ledger/status details.
Prepare the 0.27.0 release with acceptance finalization notes, goal-style guidance, and version metadata.
Guide agents to use acceptance criteria, evidence, verification, stop rules, and loop caps when delegating implementation from plans or specs.
Remove provider-unfriendly required-only acceptance subschemas while preserving runtime validation for empty acceptance contracts.
Make acceptance prompts map required evidence to acceptance-report JSON fields so visible prose does not satisfy structured evidence by itself.
Prepare the 0.28.0 release with foreground timeouts, per-agent resource limits, and acceptance compatibility fixes.
@nicobailon

Copy link
Copy Markdown
Owner

Thanks for the PR. I verified the nested fanout history issue is real, but this branch also pulls in the broader 0.27/0.28 acceptance-finalization work plus unrelated nested activity timestamp UI, so it is too wide to merge as this bugfix.

I split the narrow history preservation fix into #291 and left the separate fanout child registration conflict to #281.

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.

2 participants