Skip to content

V1.12.1 - #66

Merged
hexian2001 merged 3 commits into
mainfrom
v1.12.1
Jul 3, 2026
Merged

V1.12.1#66
hexian2001 merged 3 commits into
mainfrom
v1.12.1

Conversation

@hexian2001

Copy link
Copy Markdown
Owner

No description provided.

@hexian2001
hexian2001 merged commit 7f0883f into main Jul 3, 2026
2 checks passed
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

GLM 审查信息

  • PR: V1.12.1 #66
  • 模型: glm-5.2
  • 审查文件: 10/10
  • 裁剪说明: 单文件 patch 已裁剪:src/llm/models-snapshot.json

结论

存在几个需要关注的问题,其中 lockfile 的 registry 不一致和 models-snapshot 中的可疑数据最值得在合并前确认。

发现

  1. package-lock.json@lingxiao-office/sdkresolved 指向 npmmirror 镜像源

    • 文件:package-lock.jsonnode_modules/@lingxiao-office/sdk 条目
    • "resolved": "https://registry.npmmirror.com/@lingxiao-office/sdk/-/sdk-1.0.1.tgz"
    • package.jsonpublishConfig.registryhttps://registry.npmjs.org/,且 workflow 中 setup-noderegistry-url 也是 npmjs.org。
    • npm ci 会按 lockfile 中的 resolved URL 拉取,如果 npmmirror 未同步该包或不可达,CI 构建会直接失败。建议用官方 registry 重新生成 lockfile。
  2. models-snapshot.json 中 TrustedRouter 模型日期为未来时间

    • 文件:src/llm/models-snapshot.jsontrustedrouter provider 下所有模型
    • release_date: "2026-06-01" / "2026-06-20"last_updated: "2026-06-27"
    • 如果代码中有按 release_date 过滤未发布模型的逻辑,这些模型会被隐藏;如果没有过滤逻辑,则用户会看到尚未发布的模型。需确认是否为占位数据。
  3. models-snapshot.json 中某模型 cache_read 价格从 0.4 骤降至 0.0043

    • 文件:src/llm/models-snapshot.json,约第 21295 行附近
    • cache_read: 0.4 → 0.0043,降幅约 100 倍,同时 input: 2 → 0.522output: 6 → 1.044
    • 0.0043 这个值看起来像百分比而非美元单价,疑似数据录入错误。建议与定价源核对。
  4. __APP_VERSION__ 的 Vite define 配置未出现在 diff 中

    • 文件:web/src/components/sidebar/SidebarVersionBadge.tsx
    • 代码依赖构建期注入的 __APP_VERSION__ 全局常量,但本次 diff 未包含 vite.config.ts 的修改。如果 Vite 配置中未添加 define: { __APP_VERSION__: JSON.stringify(version) },运行时 typeof __APP_VERSION__ 返回 "undefined",版本徽章会始终回退到 '—',等于此改动无效。
  5. publish-npm 依赖全部构建任务,存在不必要的阻塞风险

    • 文件:.github/workflows/release.ymlpublish-npm job
    • needs: [check-release, build-portable, build-binary, build-source] — 如果 build-binary 因某平台原生模块编译失败,npm 发布也会被跳过。如果 npm 包不依赖二进制产物,考虑仅依赖 check-release

建议测试

  • 在干净的 CI 环境(无 npmmirror 缓存)中执行 npm ci,验证 @lingxiao-office/sdk 能否正常拉取。
  • 本地执行 npm run build:package 后检查产物中 @lingxiao-office/sdk 是否被正确打包或标记为 external,确认发布包体积合理。
  • 启动 web dev server,检查 SidebarVersionBadge 是否显示了正确版本号而非 '—'
  • 如果存在按 release_date 过滤模型的逻辑,编写测试覆盖 TrustedRouter 模型的可见性行为。

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