Skip to content

subagents: native followUp verb, parity status, markdown threads - #102

Merged
GaganSD merged 4 commits into
mainfrom
work/pi-subagents
Oct 30, 2026
Merged

GaganSD merged 4 commits into
mainfrom
work/pi-subagents

Conversation

@GaganSD

@GaganSD GaganSD commented Oct 30, 2026 •

Copy link
Copy Markdown
Owner
  • Add native followUp tool and command control, preserving steering priority and rejecting post-settlement input
  • Expose cached context usage and current tool in exact-ID status
  • Render thread Markdown with Pi’s native theme and code highlighting; invalidate cached colors on theme changes
  • Remove unused inspector wiring and duplicate parsing without adding abstractions
  • Verify native follow-up execution, report persistence, and tool-output expansion; 173 tests, typecheck, lint, package build, and local Pi smoke pass
  • 19 TypeScript source files / 2,740 physical source lines

- Add subagent followUp and /subagents followUp, mapping to the child
  AgentSession's native follow-up verb (queues after pending work);
  steer still enters after the current turn
- Exact-ID status presents cached context usage and current tool
  inside the closed output schema; unknown values stay unknown
- Worker threads render finalized conversation with Pi's native
  markdown theme and show context usage in the header; tool output
  stays clamped behind Tools
- wrapSession rejects steer/followUp after agent_settled; the native
  session is being disposed and would silently drop acknowledged input
- InspectView.invalidate clears markdown and body caches so theme
  changes re-render conversation colors
@greptile-apps

greptile-apps Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 4/5

[Medium impact] The changes affect sub-agent control and inspection within interactive Pi sessions.

The PR appears safe to merge, with a non-blocking gap in tests for native follow-up execution.

Findings

  1. P2 Follow-up execution remains untested ▶

Summary

Adds native followUp control, expands exact-ID status, and renders finalized thread messages with Markdown.

  • Running sub-agents can queue a follow-up after pending work.
  • Exact-ID status shows a sub-agent’s current tool and context usage.
  • Worker threads render finalized conversation with Pi’s Markdown theme.

Reviews (1) · Last reviewed commit: "subagents: close post-settlement input a..." · Reviewed by Greptile

Comment thread pi-subagents/test/runs.test.ts Resolved
@GaganSD
GaganSD merged commit f259a88 into main Oct 30, 2026
6 checks passed
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.

1 participant