Skip to content

fix(claude): align configuration editing with upstream - #536

Merged
SaladDay merged 1 commit into
mainfrom
fix/530-claude-config-save
Oct 25, 2026
Merged

SaladDay merged 1 commit into
mainfrom
fix/530-claude-config-save

Conversation

@SaladDay

Copy link
Copy Markdown
Owner

Claude 配置预览中的全局设置此前可能不会写入 settings.json。本次按 cc-switch 的配置编辑逻辑 迁移:编辑器显示完整配置,保存时将供应商字段写回记录、全局改动写入实际配置;外部修改发生冲突时由用户选择保留哪一边。涵盖新增、编辑、交互复制、供应商切换和代理接管。

Claude 旧通用片段保留供读取,停止运行时合并和新写入;旧供应商记录中未生效的字段可在编辑器中明确选择复制。TUI 复用现有表单、JSON 编辑器和确认框,文档同步指向可编辑完整配置的入口。Codex、Gemini 通用片段保持原有行为。

验证:

  • cargo fmt --check、git diff --check 通过。

  • 单元测试:4548 通过,2 忽略。

  • 相关集成测试:132 通过;provider_commands 的 3 个 Usage Query 测试失败已在干净基线 6c2f7f0 复现,与本次修改无关。

  • cargo clippy --locked --lib --tests 被未修改的 src/cli/tui/ui/home_chart.rs:806 空范围错误阻断。

  • 全新独立上下文盲审通过。

Related #530

@SaladDay
SaladDay merged commit b54a105 into main Oct 25, 2026
6 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.

1 participant