Skip to content

feat: support effort max for deepseek and opus-4-7 models#273

Open
LinruoT wants to merge 1 commit intoNanmiCoder:mainfrom
LinruoT:effort-max
Open

feat: support effort max for deepseek and opus-4-7 models#273
LinruoT wants to merge 1 commit intoNanmiCoder:mainfrom
LinruoT:effort-max

Conversation

@LinruoT
Copy link
Copy Markdown

@LinruoT LinruoT commented May 4, 2026

Summary

  • Add deepseek and opus-4-7 to the effort max allowlist
  • Remove USER_TYPE restriction, allowing all users to use max effort

@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label May 4, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026

PR quality triage

Changed areas: area:cli-core

CLI core policy: Blocked by policy until a maintainer applies allow-cli-core-change and approves the PR.

CLI core files:

  • src/utils/effort.ts
  • src/utils/settings/types.ts

Expected checks:

  • change-policy
  • server-checks

Test coverage signals:

  • Agent/model runtime path changed: use mock/request-shape tests in PR and maintainer live-model smoke before release.

Risk notes:

  • No special risk notes from changed paths.

Hard merge gates still come from GitHub Actions, not AI review.

Dosu handoff: Dosu can be used as the AI reviewer for risk explanation, missing-test prompts, and maintainer Q&A. If it does not comment automatically from the PR template, ask:

@dosubot review this PR for changed-area risk, missing tests, docs impact, desktop startup risk, and CLI core impact.

@dosubot dosubot Bot added the enhancement New feature or request label May 4, 2026
Comment thread src/utils/effort.ts
if (supported3P !== undefined) {
return supported3P
}
if (model.toLowerCase().includes('deepseek')) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Or just ['deepseek', ...].includes(model.toLowerCase())

Comment thread src/utils/effort.ts
return value
}
if (value === 'max' && process.env.USER_TYPE === 'ant') {
if (value === 'low' || value === 'medium' || value === 'high' || value === 'max') {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

alternatively use string array includes value

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:cli-core enhancement New feature or request needs-maintainer-approval size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants