Skip to content

ci: 精简 PR 检查并修复依赖审计漏洞 - #112

Merged
Dailin521 merged 3 commits into
mainfrom
codex/simplify-contribution-ci
Oct 28, 2026
Merged

Dailin521 merged 3 commits into
mainfrom
codex/simplify-contribution-ci

Conversation

@Dailin521

@Dailin521 Dailin521 commented Oct 28, 2026 •

Copy link
Copy Markdown
Owner

改了什么,为什么

普通贡献此前需要等待完整跨平台构建及发布取证。本 PR 按整个 PR 差异选择检查:普通文档只检查链接;Web、桌面 Renderer、共享 UI、Electron 运行时/打包与 Legacy .NET 使用对应检查。Core/存储、共享合同与依赖、CI、未知路径及跨领域变更保留完整代码检查。

四目标候选包、候选集合、历史正式 Release 恢复取证和 C10 仅在 main push 执行,最终 SHA 的发布验收保留。ci-gate 严格校验各范围的成功和预期跳过,Fork PR 不再因发布取证被主动拒绝。贡献指南和 PR 模板同步简化。

依赖审计发现的安全问题通过同大版本升级修复:Electron 44.9.3、桌面 Vite 7.3.7、Web Vite 8.2.4、Zod 4.6.7、Vitest 4.1.12 及受影响的间接依赖。Electron 原生探测、SBOM、候选清单和边界版本要求同步更新;保留原审计阈值。

维护者已另行关闭 main-protection 的 required_review_thread_resolution 并回读确认;PR、ci-gate、禁止强推和删除规则保持启用。

验证

  • npm test:638 passed、57 skipped、0 failed(Windows 本地;既有平台/可选环境跳过)。
  • npm run architecture:check:通过;Provider I/O 阶段 69 passed、15 skipped,跳过 Windows 不适用项。
  • npm run web:build、npm run desktop:pack:dir:通过;Electron 44.9.3 新生产包,含 production bundle 审计。
  • npm run desktop:test:e2e:packaged:7 passed、1 skipped;打包模式不执行 native browser handoff interception 用例。隔离 fixture Sync、Restore、分页和大首行流程通过。
  • npm audit --omit=dev --audit-level=moderate 与 npm audit --audit-level=high:均 0 vulnerabilities。
  • 46 项 CI/发布合同相关测试、YAML、任务依赖引用、文档链接与 git diff --check 通过。
  • 未在本机执行真实安装/卸载或其他操作系统验证;跨平台 PR CI 及合入后的 main 发布容器/C10 验证仍必需。没有运行公开发布操作。

无用户数据写入,存储算法与对外行为未修改。

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-28T02:53:32.426581Z 6d1d899 New commits
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@Dailin521 Dailin521 changed the title ci: 按改动范围精简 PR 检查并分离发布取证 ci: 精简 PR 检查并修复依赖审计漏洞 Oct 28, 2026
@Dailin521
Dailin521 merged commit 895df41 into main Oct 28, 2026
25 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