Skip to content

Conversation

@yokowu
Copy link
Collaborator

@yokowu yokowu commented Sep 4, 2025

变更描述

变更类型

  • Bug修复 (不兼容的变更,修复某个问题)
  • 新功能 (不兼容的变更,添加新功能)
  • 破坏性变更 (修复或功能会导致现有功能无法按预期工作)
  • 文档更新
  • 代码重构
  • 其他 (请说明)

影响范围

测试验证

相关问题

关闭 #

@yokowu yokowu merged commit cb70bd3 into chaitin:main Sep 4, 2025
2 checks passed
@yokowu yokowu deleted the feat-config-name branch September 4, 2025 10:57
@safe1ine
Copy link
Collaborator

@MonkeyCode-AI review

Copy link

@MonkeyCode-AI MonkeyCode-AI left a comment

Choose a reason for hiding this comment

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

Pull Request 概述

这个 PR 主要对 OpenAI 配置逻辑进行了重构,优化了 API 配置名称的处理方式。主要变更包括:

  1. 移除了 CurrentApiConfigName 字段的默认值赋值
  2. 当模型状态为默认时,使用 m.ShowName 替代硬编码的 "default" 字符串作为当前 API 配置名称
  3. 在存储 API 配置时,使用 m.ShowName 作为键替代之前通过格式化生成的名称

这些变更使代码更加清晰和易于维护,避免了硬编码字符串的使用。

Pull Request 变更详情

文件路径 变更类型 变更内容
backend/internal/openai/usecase/openai.go 修改 移除了 CurrentApiConfigName 的默认值,使用 m.ShowName 替代硬编码字符串

@MonkeyCode-AI
Copy link

⏳ MonkeyCode-AI 正在分析,请稍等片刻...

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.

3 participants