Repository navigation
Conversation
Codex Review Summary: ❌ FailedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
4 of 7 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
摘要
修复近期 Cache maintenance 与 Sync MicYou DSP 的失败,按当前维护范围暂停 Linux egui 专属 CI/自动发行,并参考 #1412、#1417,在不继续扩张 VM 数量的前提下减少重复构建。
修复维护任务
Cache maintenance
--prune-budget:先清理失效 PR 与旧代,再按最后访问时间淘汰分支缓存至预算内。pull-requests: read;PR 关闭事件只执行默认分支脚本,不以actions: write执行 PR 代码。Sync MicYou DSP
692ecbb9cee1ac45da590cadb59b645f23c1ea65;导入的 Rust、模型和许可证字节保持不变。工作流整理与覆盖变化
ci.yml,复用统一变更检测,保留test/device-build检查名和全部原有步骤。两项没有产物依赖,改为并行;VM 总数不增加,但峰值并发增加一个槽位,测试失败时可能已有设备构建开销。check-linux-egui.yml、release-linux-egui.yml、CI 调用及桌面 tag 派发。保留 Linux 源码、本地打包/校验脚本与本地契约。Windows frontend and contractscheck 不再产生;其断言改由Windows stable checks执行并纳入Windows checks汇总。若分支保护单独要求旧名称,需同步调整。进一步减少重复编译(参考 #1412)
windows_build_jobs=2/4对照入口,push/PR 和 macOS 默认仍为 2;concurrency group 包含该参数,不让对照互相取消。未新增 job 或大规格 VM。build.target-dir污染的问题,所有 fixture 命令显式使用同一临时 target。后续发行不再生成重复下载别名
OpenLess_harmony_arm64.hap,SHA256SUMS只列这一份;保留商店签名流程。.app.tar.gz更新包及签名、CDN manifest 生成/传递;旧 alias helper 仅留作历史/手动用途,自动发行不再调用。单一签名 JSON、阿里云优先与 OSS 迁移
详细设计、限制和切换步骤见
docs/release-distribution.md。updater-signatures.json,记录 schema、tag、精确文件名、原始签名、长度及摘要。临时.sig仍用于生成 CDN 清单;新 Release 不再附加多个独立.sig。.sig;最终仍用内置公钥验签安装包,不把 JSON 摘要代替来源认证。.sig的现有客户端升级到首个索引版本时,必须使用仍内嵌签名的 CDN 清单;它们不能在 CDN 全故障时独立解析新索引。这无法仅修改未来附件解决。旧 Release 不动。mirror-release.yml/mirror-release.py:published/edited 事件、手动/dispatch、每 6 小时补偿。只用默认分支工具执行带凭据的 job;校验摘要和 Minisign 后,先成功上传包,再推进七目标滚动清单;保留历史独立.sig兼容(包括旧版 Tauri 无 version comment 的 Stable)。OSS_BUCKET、OSS_ACCESS_KEY_ID、OSS_ACCESS_KEY_SECRET,已通过标准输入安全配置到主仓库,并核对 Secrets 名称与更新时间;未回显或落盘密钥。本机只读 OSS 探测网络超时,尚不能宣称凭据在线验收通过。仍需合并并成功跑一次主仓库同步,再合并网站配套 Draft PR 建议增加新功能 #1 停用旧任务。目前线上旧任务未禁用。合并 Release 入口,平台同时构建
release-tauri.yml统一显示为Release (all platforms),tag 只进入这一条 workflow。prepare-release预检创建/验证共享草稿;桌面三目标、Android 四 ABI、Harmony 随后平行启动,不再等待其他平台完成。workflow_call;Harmony 不再在桌面发布后才被 dispatch,也不再轮询等桌面创建 Release。自动 HAP checkout 固定到触发提交,手动维修保留 tag 输入。gh release view:RESTGET /releases/tags/<tag>对真实草稿返404,旧写法会误判为“尚未创建”并重复建 Release,签名汇总也会当作无旧索引而覆盖另一平台记录。现只把release not found/HTTP 404当缺失,其他错误直接失败;索引附件改按assets[].apiUrl下载(id是 GraphQL node ID,不是 REST 数字 ID)。已用现存 Beta.5 草稿实测只读:正确识别为已有草稿,并保持其历史.sig格式不变。性能依据与限制
完整记录见
docs/ci-runner-balance.md。修改前样本:Windows 独立前端 VM 95s,其中契约 42s;Linux 两台 VM 合计约 17 VM-min;iOS 两项串行 475s + 879s。这些只支持优化方向,不是修改后的远端加速结果。不把 VM-min 当成墙钟时间,也不承诺未经测量的提速百分比。桌面发行 Intel Mac/Windows 约 29 分钟的关键路径仍待后续受控测量。
验证
npm test:生产前端构建及全部 170 个 JS/TS 测试文件通过。git diff --check通过。b45711a7的完整远端 CI:15 项检查全部通过(run42818661012)。--check,纳入完整前端测试防止再漏;本地 170 个测试文件通过。42837557955);Harmony 在快照检查处失败,当前提交修复该问题。42840396456。