diff options
| author | Adam Malczewski <[email protected]> | 2026-06-26 19:03:09 +0900 |
|---|---|---|
| committer | Adam Malczewski <[email protected]> | 2026-06-26 19:03:09 +0900 |
| commit | e8b4bf1fe4fedc48bd0dc56b5745467542946474 (patch) | |
| tree | 1d7c31f23bc7fccdf3052dc59ff8da63700eade8 /packages/kernel/src/runtime/run-turn.ts | |
| parent | 9b91d1bca83e7599fb7d7de6038cedf186e61764 (diff) | |
| download | dispatch-e8b4bf1fe4fedc48bd0dc56b5745467542946474.tar.gz dispatch-e8b4bf1fe4fedc48bd0dc56b5745467542946474.zip | |
fix(kernel): disable MAX_STEPS limit (0 = unlimited)
Agents were being cut off mid-task at 50 steps. The MAX_STEPS=50
hardcoded limit was silently terminating turns while the model was
actively making tool calls, leaving conversations idle with a
dangling tool-result as the last chunk.
Setting MAX_STEPS to 0 disables the limit — the loop runs until the
model stops making tool calls naturally or the abort signal fires.
The max-steps code path is preserved for when MAX_STEPS > 0.
Diffstat (limited to 'packages/kernel/src/runtime/run-turn.ts')
| -rw-r--r-- | packages/kernel/src/runtime/run-turn.ts | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/packages/kernel/src/runtime/run-turn.ts b/packages/kernel/src/runtime/run-turn.ts index ac87a1f..273482f 100644 --- a/packages/kernel/src/runtime/run-turn.ts +++ b/packages/kernel/src/runtime/run-turn.ts @@ -27,7 +27,9 @@ import { usageEvent, } from "./events.js"; -export const MAX_STEPS = 50; +/** Max steps per turn. 0 = unlimited (the loop runs until the model stops + * making tool calls or the abort signal fires). */ +export const MAX_STEPS = 0; function zeroUsage(): Usage { return { inputTokens: 0, outputTokens: 0 }; @@ -615,7 +617,7 @@ export async function runTurn(input: RunTurnInput): Promise<RunTurnResult> { input.emit(turnStartEvent(conversationId, turnId)); try { - for (let step = 0; step < MAX_STEPS; step++) { + for (let step = 0; MAX_STEPS === 0 || step < MAX_STEPS; step++) { if (signal.aborted) { finishReason = "aborted"; break; @@ -708,7 +710,7 @@ export async function runTurn(input: RunTurnInput): Promise<RunTurnResult> { break; } - if (step === MAX_STEPS - 1) { + if (MAX_STEPS > 0 && step === MAX_STEPS - 1) { finishReason = "max-steps"; // No next step → no tool-result boundary. Leave any pending // steering messages for the caller (it owns the queue). |
