Tracking: subagent lifecycle PRs (#579 → #576 → #580 → #581) — merge order, conflict map, rebase plan
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 4k
- Forks
- 453
- Avg merge
- 12h 30m
- Merged PRs (30d)
- 46
Description
Summary
Four currently-open PRs from me form one coordinated subagent lifecycle roadmap. They are semantically orthogonal features, but they all extend the same agent-core plumbing (PilotDeckSubagentForkApi, SubAgentSession, AgentLoop), so they overlap textually and need an explicit merge sequence. This issue tracks the proposed order, the measured conflict map, and the rebase/validation plan, so maintainers can sequence reviews instead of discovering conflicts one merge at a time.
| PR | Feature | Size |
|---|---|---|
| #579 | Expose subagent execution timeout in settings | +88 / 6 files (UI only) |
| #576 | Configure subagent roles and model bindings | +4283 / 49 files |
| #580 | Continue completed subagent tasks (task_id) |
+2152 / 17 files |
| #581 | Run subagents in the background (run_in_background) |
+2109 / 21 files |
Measured conflict map
Actual test merges (pairwise, each PR merged on top of the other; base 97633a0):
| #576 | #579 | #580 | #581 | |
|---|---|---|---|---|
| #576 | — | 1 file (UI types) | 4 core files | 3 core files |
| #579 | — | clean | clean | |
| #580 | — | 1 core file (agent.ts) |
||
| #581 | — |
Core files involved: AgentLoop.ts, SubAgentSession.ts, agent.ts, AgentRuntimeDependencies.ts, tool/protocol/types.ts; the only UI overlap is modelPool/types/index.ts between #576 and #579.
Against current main (e5bfb3fe): #576, #579, #580 merge cleanly. #581 has a single-file conflict in AgentLoop.ts caused by recent upstream movement in that region — I will rebase that branch (see checklist).
All conflicts are in additive extensions of the same types/fork-API entry points; each resolution is small. #580 + #581 are additionally validated merged together on an integration branch in my fork (work/subagent-lifecycle-integration) with both test suites re-run.
Proposed merge order
#579 → #576 → #580 → #581
Rationale:
- #579 is the smallest, UI-only, and clean against everything except one shared UI file with #576 — a fast first merge.
- #576 is the foundation the later two conceptually build on (role catalog, model binding, fork-API plumbing) and is clean against current
main. - #580 next; its rebase after #576 is the largest (4 files) but the changes are orthogonal in intent.
- #581 last; it needs a small rebase against current
mainregardless, and the #580+#581 combination is already validated together.
Any other order works too — I will rebase the remaining branches onto main after each merge and re-run the targeted test suites (agent/subagent specs + settings UI specs) before updating the PRs.
Checklist
- #579 merged
- #576 merged (rebase after #579: 1 UI file)
- #580 merged (rebase after #576: 4 core files)
- #581 merged (rebase after #576 + #580, plus current-
mainrebase inAgentLoop.ts) - Re-run combined lifecycle validation (continuation + background) on the result
Happy to re-sequence if maintainers prefer a different order or want some of these consolidated.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the conflict map and checklist, then inspect AgentLoop.ts, SubAgentSession.ts, agent.ts, AgentRuntimeDependencies.ts, and tool/protocol/types.ts. Follow the proposed PR order and review the targeted agent/subagent and settings UI test suites. Done means the four PRs are sequenced, rebased as needed, merged, and combined lifecycle validation has passed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- ai-infra-agents
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Active
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100