Skip to content

feat(plan): support a dedicated planning model#895

Merged
XingYu-Zhong merged 2 commits into
KunAgent:developfrom
luoye520ww:codex/fix-878-plan-execution-model
Jul 18, 2026
Merged

feat(plan): support a dedicated planning model#895
XingYu-Zhong merged 2 commits into
KunAgent:developfrom
luoye520ww:codex/fix-878-plan-execution-model

Conversation

@luoye520ww

@luoye520ww luoye520ww commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add an optional Plan-mode model/provider/account slot to Kun settings.
  • Route Plan turns to the configured slot while preserving explicit per-turn overrides.
  • Expose the setting in the existing Subagent settings editor with English and Chinese labels.

This is the model-routing portion of #878. Project import/export is intentionally separate so it can be reviewed and tested independently.

Compatibility

  • An empty Plan slot falls back to the normal conversation model.
  • Explicit per-turn model/provider/account values always take precedence.
  • Existing agent and chat turns are unchanged.

Tests

npm.cmd exec vitest run src/renderer/src/agent/kun-runtime.test.ts src/renderer/src/components/subagents/SubagentSettingsEditor.test.ts
npm.cmd run typecheck
npm.cmd run lint -- --quiet
npm.cmd run build
git diff --check

Results: 2 test files / 36 tests passed; Typecheck, lint, build, and diff check passed locally. Linux, macOS, and Windows package checks are green in CI.

Issue

Part of #878

@XingYu-Zhong XingYu-Zhong left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reviewed against current develop after branch update. Scope is focused, the change remains needed, and no blocking issue was found. Required CI must pass before merge.

@XingYu-Zhong
XingYu-Zhong merged commit 73f6a0d into KunAgent:develop Jul 18, 2026
9 of 10 checks passed
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.

2 participants