build(zip): make the canonical archive reproducible and atomic - #119
Conversation
|
方向是对的,但我本地复现到一个需要在合并前处理的兼容性问题:
另外,当前 DSH distribution acceptance 连续两次失败都是安装 |
|
Thanks for continuing to improve this PR. We rechecked the current revision and found no blocking issue in the reproducible ZIP implementation, cross-platform package smoke tests, ZIP freshness check, or focused local tests. One CI reliability issue still needs to be addressed before merge. The receipt shows that Please add one bounded retry around the real DSH runtime installation, limited to clearly transient network failures such as Once that is updated, please rerun the full checks and we can do the final merge verification. |
|
Thanks again for the original implementation and for keeping this branch updated. Because this PR comes from a fork with maintainer edits disabled, we could not push the small CI reliability fix directly to your branch. We therefore preserved all commits and authorship from this PR in repository-local PR #129 and added the bounded transient-network retry there. We will use #129 for the final CI verification and merge. Your original contribution remains fully credited in the preserved commit history. |
|
Thank you again for the original reproducible and atomic ZIP implementation. This work has now been merged through #129 as merge commit The final GitHub checks were all green, including the complete DSH distribution acceptance, which now finishes in 28 seconds instead of timing out during runtime installation. Closing this PR as superseded by the merged #129. Thanks for the contribution. |
问题与价值
修复规范
archify.zip缺少确定性构建、输入完整性检查和原子发布保障的问题。Fixes #116
范围
zip实现的确定性 ZIP 写入器。node >=18。.equals(),失败时不再展开巨大 diff。稳定性影响
已运行测试
另外完成:
规范 ZIP 已在 Node 22 下重新构建,并使用
cmp验证字节一致。可视化证据
不适用;本 PR 不改变页面视觉效果。
生成物
archify.zip。Checklist
archify/运行完整npm test