Skip to content

Add default models for provider quick options#17

Merged
kaichen merged 2 commits into
mainfrom
feat/provider-quick-options
Sep 12, 2025
Merged

Add default models for provider quick options#17
kaichen merged 2 commits into
mainfrom
feat/provider-quick-options

Conversation

@kaichen

@kaichen kaichen commented Sep 12, 2025

Copy link
Copy Markdown
Owner

Summary

• Added default model configuration for provider quick options (kimi, deepseek, glm, qwen)
• Enhanced provider selection to automatically set appropriate model when creating profiles
• Improved user experience by eliminating need to manually specify common models

Test plan

  • Test provider quick options with default models
  • Verify profile creation includes default models
  • Test manual model override still works
  • Test existing profiles remain unaffected

🤖 Generated with Claude Code

kaichen and others added 2 commits September 12, 2025 14:17
- Add provider quick flags (--kimi, --deepseek, --glm, --qwen) for faster profile setup
- Support base_url=URL and --base-url URL overrides in both main command and profile add
- Implement auto-profile creation when using provider flags without existing profile
- Add providers.js module with predefined provider configurations
- Refactor CLAUDE.md to reference separate Agents.md documentation
- Update CLI help text to include new provider syntax options
- Improve base URL resolution priority: explicit override > profile > provider default

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Add defaultModel field to provider configurations (kimi, deepseek, glm, qwen)
- Automatically set model when creating profiles via provider flags
- Improve provider flag handling to respect user-provided model preferences

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@claude

claude Bot commented Sep 12, 2025

Copy link
Copy Markdown

Claude encountered an error —— View job


I'll analyze this and get back to you.

@kaichen

kaichen commented Sep 12, 2025

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Codex Review: Here are some suggestions.

Reply with @codex fix comments to fix any unresolved comments.

About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".

Comment thread bin/cli.js
Comment on lines +197 to +198
// Default model if not provided
if (!model && p && p.defaultModel) model = p.defaultModel;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[P2] Preserve existing model when overwriting provider profile

When a provider flag is used with kimicc profile add, this block injects the provider’s defaultModel whenever the caller didn’t pass --model. The overwrite path later passes model to addProfile, so running kimicc profile add --kimi <token> to rotate a key will reset any custom model that was already configured on that profile. Users who previously selected a different model lose that setting unexpectedly. Consider only applying the default model when creating a new profile or when --model is explicitly supplied.

Useful? React with 👍 / 👎.

@kaichen
kaichen merged commit 63a3049 into main Sep 12, 2025
2 of 3 checks passed
@kaichen
kaichen deleted the feat/provider-quick-options branch September 12, 2025 08:02
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.

1 participant