Skip to content

fix: stabilize model modal and prepare v2.0.0-beta.16 - #29

Merged
Alpaca233114514 merged 7 commits into
mainfrom
codex/fix-model-api-key-modal
Aug 5, 2026
Merged

fix: stabilize model modal and prepare v2.0.0-beta.16#29
Alpaca233114514 merged 7 commits into
mainfrom
codex/fix-model-api-key-modal

Conversation

@Alpaca233114514

@Alpaca233114514 Alpaca233114514 commented Aug 1, 2026

Copy link
Copy Markdown
Collaborator

概要

  • 修复模型弹窗首次打开和重复打开时供应商、协议与 API Key 状态不同步的问题
  • 在请求前校验并清理模型名称与模型 ID,保存失败时保留弹窗并展示明确错误
  • 在供应商切换时清理失效密钥,并为无密钥供应商保留可用流程
  • 新增端到端测试,覆盖必填校验、连续添加、无密钥供应商和编辑状态恢复
  • 将应用作者/Windows 发布者元数据从 PapyrusOR 更新为 LiYuanStudio
  • 将根、前端、后端 manifest、对应 lockfile、备用根配置和 README 统一到 2.0.0-beta.16

原因

Arco Modal、Form 实例与异步供应商数据共同复用时,initialValue 不会随每次打开更新,导致表单显示值与实际提交值可能不一致。改为在弹窗打开时从当前 props 构造完整表单快照,并通过 Form.useWatch 驱动 API Key 选项。

同时,发布元数据仍保留旧作者名,且 root-package.json 和其他版本文件未完整同步,导致 Windows 应用列表及仓库版本信息显示旧值。

用户与开发者影响

模型新增和编辑流程会保持供应商、协议、密钥与能力开关一致;无效表单不会发送网络请求。下一次构建的 Windows 安装包将显示发布者 LiYuanStudio,所有发布相关清单和锁文件统一为 BA16。

本次不修改后端数据结构。已安装旧版本需要使用新安装包重新安装后,系统应用列表才会刷新发布者和版本信息。

验证

  • npm --prefix frontend run typecheck
  • npm --prefix backend test -- --runInBand tests/integration/api.test.ts -t "prerelease-aware"
  • node --test scripts/__tests__/verify-packaged-app.test.js(5/5 通过)
  • manifest 与 lockfile 版本一致性检查:全部为 2.0.0-beta.16
  • 项目版本校验器接受 2.0.0-beta.16
  • Electron Builder 元数据解析:companyName=LiYuanStudio
  • 原 PR 已执行:npm --prefix frontend run build
  • 原 PR 已执行:npx playwright test model-management.spec.ts
  • git diff --check

@Alpaca233114514
Alpaca233114514 marked this pull request as ready for review August 1, 2026 16:42
@Alpaca233114514 Alpaca233114514 changed the title fix: stabilize model modal validation and provider state fix: stabilize model modal and prepare v2.0.0-beta.15 Aug 1, 2026
@Alpaca233114514 Alpaca233114514 changed the title fix: stabilize model modal and prepare v2.0.0-beta.15 fix: stabilize model modal and prepare v2.0.0-beta.16 Aug 1, 2026
@Alpaca233114514
Alpaca233114514 merged commit ad8e250 into main Aug 5, 2026
8 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