Skip to content

Add multi-provider Claude presets and preserve external model IDs#199

Merged
SDSLeon merged 1 commit into
masterfrom
lightcode/polite-ember-5332f7e2
Jun 17, 2026
Merged

Add multi-provider Claude presets and preserve external model IDs#199
SDSLeon merged 1 commit into
masterfrom
lightcode/polite-ember-5332f7e2

Conversation

@SDSLeon

@SDSLeon SDSLeon commented Jun 17, 2026

Copy link
Copy Markdown
Owner
  • Add z.ai, DeepSeek, and MiniMax provider presets to Claude profile settings so users can seed external-provider env and model configuration from a single menu
  • Preserve external-provider model IDs (e.g. glm-5.2[1m]) when launching Claude so provider-specific suffixes are not stripped by context-size handling
  • Remove agent status entries across native, WSL, and discovery lists when a Claude profile is deleted, keeping detection state in sync with saved profiles
  • Extend preset and argv behavior with tests covering preset env rows, profile settings UI, and context-suffix rules for built-in vs external models

- Replace GLM-only preset with z.ai, DeepSeek, and MiniMax presets and dropdown UI
- Skip context-suffix rewriting for external-provider model ids (e.g. glm-5.2[1m])
- Add removeAgentStatus and call it when deleting a Claude profile
@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
lightcode-landing Skipped Skipped Jun 17, 2026 5:40am

@SDSLeon SDSLeon merged commit e7bc38c into master Jun 17, 2026
6 checks passed
@SDSLeon SDSLeon deleted the lightcode/polite-ember-5332f7e2 branch June 17, 2026 05:40

@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 automated review suggestions for this pull request.

Reviewed commit: 2bb8570555

ℹ️ 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
  • 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 address that feedback".

setEnvRows((rows) => applyPresetEnvRows(preset.envRows, rows, nextRowId));
setSelectedEfforts(nextEfforts);
setModelRows(nextModelRows);
setAgentInstance({ ...props.instance, config });

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 Badge Persist preset env with the model config

When a user applies a provider preset and then navigates away before pressing Save, this immediately writes the preset's models/efforts to shared settings, but the env rows added above remain only in local component state because the environment is written only by save() via setClaudeProfileEnvironment. That leaves the saved profile advertising external model IDs while missing required ANTHROPIC_BASE_URL/token/compact-window env, so launching the profile can fail or route to the wrong provider. Either keep all preset changes local until Save or persist the env portion together with this config update.

Useful? React with 👍 / 👎.

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