Skip to content

Preserve native subagent model omissions - #155

Merged
badcuban merged 2 commits into
mainfrom
agent/fix-native-subagent-model-attribution
Aug 14, 2026
Merged

Preserve native subagent model omissions#155
badcuban merged 2 commits into
mainfrom
agent/fix-native-subagent-model-attribution

Conversation

@badcuban

Copy link
Copy Markdown
Collaborator

Summary

  • preserve native Codex subagent model and reasoning omissions across later coordination events
  • prevent a wait item from retroactively applying the parent turn's settings to an already-known child
  • align activity reconstruction with the durable server subagent projection

Root cause

Native subAgentActivity spawn rows intentionally leave model and effort unknown. The first spawn record respected that, but a later wait row reused the parent turn's model selection as a fallback because null was treated as missing rather than as the established child value.

Validation

  • vp exec vp run '@threadlines/web#test' session-logic (141 passed)
  • vp fmt
  • vp lint
  • vp run typecheck
  • git diff --check

This repairs the two failures currently shared by main and feature PR CI.

@vercel

vercel Bot commented Aug 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
threadlines Skipped Skipped Aug 14, 2026 6:37am

Request Review

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S labels Aug 14, 2026
@badcuban
badcuban merged commit b4d8699 into main Aug 14, 2026
15 checks passed
@badcuban
badcuban deleted the agent/fix-native-subagent-model-attribution branch August 14, 2026 06:46
badcuban added a commit that referenced this pull request Aug 14, 2026
Two fixes for the inline subagent activity bar in the conversation:

Flicker: a running agent's streamed commentary is a timeline entry that
renders nothing, but its timestamp moves with every update. It could
split and re-merge the turn's work groups as the agent streamed, and the
live group never rendered the activity receipt, so the tracker blinked
in and out mid-turn. Live commentary entries now stay out of grouping
entirely, and a live work group that owns the turn's tracker renders the
receipt above its spine so the bar survives group churn.

Missing model/effort: the durable roster seeds a record for every agent
before lifecycle rows fold in, so the "only inherit for a brand-new
record" guard from #155 never fired and a collab spawn that stated no
model lost the parent turn's dispatched selection. Native-spawn omission
is now a sticky per-record fact instead: native subAgentActivity spawns
still never inherit, while collab and Task spawns inherit the turn's
model and effort even when the roster seeded their record first.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant