feat(task): implement fire-and-forget TaskTool with background execution #6046
+1,013
−63
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ation PR1 of async subagent delegation stack (Issue sst#5887). This commit introduces the SessionRunner namespace which provides a unified interface for executing session loops: - runOnce(): Synchronous execution wrapping SessionPrompt.prompt (implemented) - runBackground(): Fire-and-forget execution for async delegation (scaffold only) - Options/RunResult types for future background execution - State tracking infrastructure for active background runs No behavior change - this is purely an abstraction layer. Next: PR2 will implement runBackground() with lifecycle events.
…le events - Add Job, JobKind, JobStatus, EnqueueOptions schemas - Add enqueue(), cancel(), get(), list() APIs - Add lifecycle events (Queued, Started, Completed, Failed, Canceled, TimedOut) - Add configurable timeout and concurrency via experimental.backgroundTasks - Fix: queue now restarts after idle (removed stale started flag) - Fix: dedupe now returns completion promise, not immediate job reference
- Fix circular import by using regular function for runOnce
- Add toolPartID to Tool.Context for post-return metadata updates - Pass toolPartID from prompt.ts when invoking tool.execute - TaskTool now returns immediately after enqueueing work to SessionRunner - Subscribe to child session part updates for live progress in parent metadata - Update parent metadata with final job status on completion/failure/cancel/timeout - Handle enqueue failures with logging and proper cleanup - Use ctx.callID for accurate tool call tracing
edlsh
force-pushed
the
feat/async-subagent-4-tasktool
branch
from
c138f7e to
32b6eb9
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Phase 4: Rework TaskTool to schedule background execution (fire-and-forget)
Changes
toolPartIDto Tool.Context for post-return metadata updatesSession.updatePart()instead ofctx.metadata()Stack