Skip to content

chore(config): raise default max_steps_per_turn from 500 to 1000#2088

Merged
RealKai42 merged 2 commits into
mainfrom
core/raise-max-steps-per-turn-1000
Apr 28, 2026
Merged

chore(config): raise default max_steps_per_turn from 500 to 1000#2088
RealKai42 merged 2 commits into
mainfrom
core/raise-max-steps-per-turn-1000

Conversation

@liruifengv

@liruifengv liruifengv commented Apr 27, 2026

Copy link
Copy Markdown
Collaborator

Related Issue

N/A — internal request to give long agent runs more headroom before the per-turn cap.

Description

Bumps the default LoopControl.max_steps_per_turn from 500 to 1000 so multi-step agent turns are less likely to abort with MaxStepsReached. Users who want a tighter or looser limit can still override via:

  • the loop_control.max_steps_per_turn field in ~/.kimi/config.toml, or
  • the --max-steps-per-turn N CLI flag.

Files touched:

  • src/kimi_cli/config.pyLoopControl.max_steps_per_turn Field(default=500) → Field(default=1000)
  • tests/core/test_config.pytest_default_config_dump snapshot synced to 1000
  • docs/en/configuration/config-files.md and docs/zh/configuration/config-files.md — TOML example and field-reference table updated
  • CHANGELOG.md, docs/en/release-notes/changelog.md, docs/zh/release-notes/changelog.md — one Unreleased Core entry each

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked the related issue, if any. (N/A)
  • I have added tests that prove my fix is effective or that my feature works. (tests/core/test_config.py::test_default_config_dump snapshot covers the new default)
  • I have run make gen-changelog to update the changelog. (Unreleased entries added by hand in CHANGELOG.md and docs/zh/release-notes/changelog.md; docs/en/release-notes/changelog.md synced)
  • I have run make gen-docs to update the user documentation. N/A — config-files reference docs are authored directly; updated alongside the code change in docs/{en,zh}/configuration/config-files.md.

Long-running agent turns were occasionally hitting the 500-step cap. Bump
the default to 1000 so they have more headroom; users who need a tighter
or looser limit can still override via config or `--max-steps-per-turn`.
Copilot AI review requested due to automatic review settings April 27, 2026 11:11

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@liruifengv liruifengv changed the title core: raise default max_steps_per_turn from 500 to 1000 chore(config): raise default max_steps_per_turn from 500 to 1000 Apr 27, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Raises the default agent loop cap so long-running turns are less likely to stop due to the per-turn step limit, and updates the repository’s tests and documentation to reflect the new default.

Changes:

  • Increase LoopControl.max_steps_per_turn default from 500 → 1000.
  • Update default-config snapshot test expectations.
  • Update config documentation examples/tables and add release-note entries (root + docs changelogs).

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/kimi_cli/config.py Bumps LoopControl.max_steps_per_turn default to 1000.
tests/core/test_config.py Updates default config dump snapshot value for max_steps_per_turn.
docs/en/configuration/config-files.md Updates example config and default-value table to 1000.
docs/zh/configuration/config-files.md Updates example config and default-value table to 1000.
CHANGELOG.md Adds Unreleased entry documenting the default increase.
docs/en/release-notes/changelog.md Includes the synced Unreleased entry in the English docs changelog.
docs/zh/release-notes/changelog.md Adds the corresponding Chinese Unreleased entry.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants