Synced from GitHub
fix(client-runtime): Pi subagents show their thinking level#33346
Pi delegates can save a different thinking level from their parent, but subagent metadata omitted it. The existing web hover and mobile Agents model lines now show the child's stated level.
Both clients share child-selection precedence and effort formatting. A saved app-owned child selection takes precedence over its own reported selection; unknown effort stays absent, boolean thinking stays excluded, and known values survive a missing or stale catalog. Existing layouts and provider-fetch behavior are unchanged.
Validation: deterministic red/green tests cover Pi thinking, child selection and provider-instance fallback, boolean thinking, absent OpenCode variant, and stale catalog values. pnpm exec vp run check:changed passes. Local cubic found no issues.
Standard change: shared selection behavior and web/mobile consumers.
UI evidence: before, raw fallback after, catalog label and saved-child precedence, 390px web.
The before capture removes thinking from the shared list. Catalog/precedence captures use sanitized browser projection fixtures feeding the real mounted consumer: saved child High, reported Max, parent Max. They prove client rendering given projected data, not transport or provider discovery. Twelve additional public-helper runtime assertions cover instance/model rejection, catalog transition, ordering and mobile workflow catalog labels. Final proof audit found no remaining code gaps. Native mobile and packaged Electron rendering were not run.
Council review
Rounds: 2
| Delegate | Round 1 | Round 2 |
|---|---|---|
| Claude Opus 5.5 | 9/10 | 9/10 |
| GPT 6.1 Sol | 8/10 | 9/10 |
| Grok 4.7 | 9/10 | 9/10 |
Created with GPT 6.1 Sol in the Codex harness, Claude Opus 5.5 in the Claude harness, and Grok 4.7 in the Grok harness through T3 Code. Sol completed evidence recovery after Grok's client fixture failed.
Closes #30442
Checks
1 failing · 24 successful · 16 skipped
Failing1
- CodeRabbitGitHubCancelled
Added 4 commits
- fix(client-runtime): keep only stated subagent effort across catalog changes6a813e3
- test(client-runtime): assert subagent selection precedence directly6cbd285
- fix(client-runtime): Pi subagents show the thinking level they run at7b225a2
- fix(web,mobile): subagent hover and Agents rows show the child's Pi thinking levelc75efc4
Synced from GitHub
Approvability
Verdict: Approved at 6a813e3
Macroscope's review found this PR approvable — Subagent metadata on web and mobile now shows Pi thinking levels using the appropriate child selection. The changes are confined to metadata presentation and preserve provider-instance matching without changing execution defaults.
You can add or adjust custom eligibility rules. Learn more.
Thanks, @StiensWout, for the thorough fix and evidence! #30489 by @juliusmarminge already addresses the same Pi subagent thinking-level issue, so we’re keeping that PR. Please share any remaining mobile or selection-precedence gaps there.
Reopening this as the keeper: it merges cleanly with main and includes the Pi thinking-level fix plus child-selection and mobile coverage. Thank you, @StiensWout!