forked from anomalyco/opencode
-
Notifications
You must be signed in to change notification settings - Fork 0
Closed
Description
Task Description
Increase the default subagent/task execution timeout from 10 minutes to 30 minutes in both independent timeout locations.
Background
Research revealed two independent hardcoded timeout constants (not shared):
packages/opencode/src/session/async-tasks.tsline 52:DEFAULT_TASK_TIMEOUT = 10 * 60 * 1000— controls background/async task executionpackages/opencode/src/tool/task.tsline 381:taskTimeoutMs = 10 * 60 * 1000— controls sync task tool execution
The current 10-minute default (already increased from upstream's 5 minutes) is still too short for complex agentic tasks such as multi-file implementations, upstream syncs, and thorough research sessions.
The existing task-timeout entry in .fork-features/manifest.json also needs correcting — it only lists async-tasks.ts in modifiedFiles but is missing tool/task.ts, which is the second independent location.
Changes Required
packages/opencode/src/session/async-tasks.ts— changeDEFAULT_TASK_TIMEOUTfrom10 * 60 * 1000to30 * 60 * 1000packages/opencode/src/tool/task.ts— changetaskTimeoutMsfrom10 * 60 * 1000to30 * 60 * 1000.fork-features/manifest.json— updatetask-timeoutentry:- Add
tool/task.tstomodifiedFiles - Update
descriptionto reflect 30-minute default and correct both locations - Update
criticalCodemarkers to30 * 60 * 1000
- Add
Quality Gates (Non-Negotiable)
- Both timeout locations updated
- Manifest
task-timeoutentry updated with correctmodifiedFiles,description, andcriticalCode -
bun run typecheckpasses (0 errors) -
bun testpasses (0 failures) -
bun run .fork-features/verify.tspasses
Acceptance Criteria
- Both
async-tasks.tsandtask.tsuse30 * 60 * 1000as timeout - Manifest accurately reflects both modified files and new timeout value
- All local checks pass before push
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels