Skip to content

fix(agent-core): correct v1 Agent tool description to match 2-hour default timeout - #2450

Closed
C0d3N1nja97342 wants to merge 1 commit into
MoonshotAI:mainfrom
C0d3N1nja97342:fix/subagent-timeout-doc
Closed

fix(agent-core): correct v1 Agent tool description to match 2-hour default timeout#2450
C0d3N1nja97342 wants to merge 1 commit into
MoonshotAI:mainfrom
C0d3N1nja97342:fix/subagent-timeout-doc

Conversation

@C0d3N1nja97342

Copy link
Copy Markdown

Related Issue

Resolves #2420

Problem

The v1 Agent tool description (agent.md) told the model subagents use a 'fixed 30-minute timeout', but DEFAULT_SUBAGENT_TIMEOUT_MS is 2 hours. The word 'fixed' also hid that the timeout is configurable. v2 was already aligned.

What changed

  • Updated agent.md to say 'configurable timeout that defaults to 2 hours' and mention KIMI_SUBAGENT_TIMEOUT_MS / [subagent] timeout_ms
  • Updated the test assertion to match the new text

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked a related issue.
  • I have added tests that prove my feature works.
  • This PR needs no changeset (documentation/description text only).
  • This PR needs no doc update.

…fault timeout

The v1 Agent tool description said subagents use a 'fixed 30-minute timeout', but DEFAULT_SUBAGENT_TIMEOUT_MS is 2 hours (subagent-host.ts). v2 was already aligned. Update the description to say 'configurable timeout that defaults to 2 hours' and mention the KIMI_SUBAGENT_TIMEOUT_MS / [subagent] timeout_ms config knobs.

Update the test assertion to match the new text and add checks for the configurability note.

Fixes MoonshotAI#2420
@changeset-bot

changeset-bot Bot commented Jul 31, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 5050989

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@C0d3N1nja97342

Copy link
Copy Markdown
Author

Closing in favor of #2423, which also addresses #2420 (the agent.md line + test assertion) alongside its primary MCP fix.

@C0d3N1nja97342
C0d3N1nja97342 deleted the fix/subagent-timeout-doc branch July 31, 2026 06:17
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.

[Bug] agent-core v1 Agent tool description says "fixed 30-minute timeout", actual default is 2 hours

1 participant