Skip to content

ci: 精简纯文档检查并降低打包压缩耗时 - #100

Merged
Dailin521 merged 5 commits into
mainfrom
codex/ci-docs-and-compression
Sep 11, 2026
Merged

Dailin521 merged 5 commits into
mainfrom
codex/ci-docs-and-compression

Conversation

@Dailin521

@Dailin521 Dailin521 commented Sep 11, 2026

Copy link
Copy Markdown
Owner

普通文档 PR 目前会执行全部构建,Windows ZIP 的最大压缩也显著延长打包时间。本次为整个 PR 都属于 README、CHANGELOG 或中文版本公告的情况提供轻量检查,并将压缩改为 normal。同一 PR 的新提交取消旧运行;main 仍执行完整跨平台 CI,发布门禁保持不变。 分类按整个 PR 的差异检查,重命名同时检查新旧路径;未知路径、代码、依赖、工作流和行为文档不走轻量检查。ci-gate 始终执行,严格拒绝非预期跳过,C10 保持原有业务任务结果校验。 本机同内容 ZIP 测量为 310.14 秒降至 42.48 秒,增加 264,243 字节(0.20%);两份归档解压内容逐文件 SHA-256 一致。该数据是本机单次测量,不承诺固定 CI 耗时。 验证:本地根测试 543 通过、56 跳过;桌面测试 178 通过、1 跳过;最终 CI 分流及发布契约测试 10/10 通过;normal ZIP 的生产测试 4/4 通过,含公开更新检查和隔离 Sync → Restore。normal NSIS 已构建且包大小符合原限制。跨平台与真实安装/卸载验收由本 PR 的完整 CI 执行。 详细测量与边界见 docs/CI_PERFORMANCE_ZH.md。本次不修改版本、不发布新 Release。 范围收口:按维护者要求,暂不继续扩展自定义 Markdown 校验器。嵌套列表中的缩进链接、引用块内引用定义仍可能漏检,作为已知限制留待后续;本次链接检查不承诺完整 CommonMark/GFM 解析。main 的完整 CI、版本校验和正式发布验收不受此限制影响。

补充范围与验收:仅对跨运行时锁测试使用禁并行测试集合,减少与其他进程密集测试类的争用;保留测试内真实进程竞争、30 秒上限及所有断言,并增加固定阶段诊断。本地 Release Core.Tests 247 通过、1 个真实 WSL 跳过;本次远端 desktop-test 也已通过,资源争用仍属缓解假设而非已证明根因。

按既定收口决定,文档解析器的超大文本性能、站点根路径语义及完整命名字符实体支持同样暂缓,本 PR 不承诺完整 Markdown 链接验证器。

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 11, 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-09-11T17:06:57.605563Z 2a55ab4 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.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: bd86d96c94

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread scripts/ci-docs.mjs Outdated
Comment thread scripts/ci-docs.mjs Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d1791b970f

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/ci.yml
Comment thread scripts/ci-docs.mjs Outdated
Comment thread scripts/ci-docs.mjs Outdated
Comment thread scripts/ci-docs.mjs Outdated
Comment thread scripts/ci-docs.mjs

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9be5cea24e

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread scripts/ci-docs.mjs Outdated
Comment thread scripts/ci-docs.mjs Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e76589998d

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread scripts/ci-docs.mjs
Comment thread scripts/ci-docs.mjs

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2a55ab4608

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread scripts/ci-docs.mjs
Comment thread scripts/ci-docs.mjs
Comment thread scripts/ci-docs.mjs
@Dailin521
Dailin521 merged commit 9cd011c into main Sep 11, 2026
28 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