build(reproducibility): restack deterministic npm toolchain - #91
build(reproducibility): restack deterministic npm toolchain#91seonghobae wants to merge 17 commits into
Conversation
|
Warning Review limit reached
Next review available in: 13 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (17)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Purpose
Clean protected-main successor to superseded #89 after #76 integrated. This branch starts from protected
mainc85d710804139c0697d7ef8fa47d02b1389e6d84and replays only #89's unique package-manager/reproducibility delta; predecessor checks/reviews do not transfer.Exact preservation proof
Fresh comparison of #76's exact predecessor head
e0106ce16b7b8b493f46bf075ec5baf58762bd95to #89 headadcc42d2b0fec3106faabb59bbf7e79124714f7aidentified exactly 17 unique files. Fresh comparison of protectedmainto this replacement branch identifies the same 17 paths with the same per-file additions/deletions. Each replacement file was populated from #89's exact current-head content; #76's already-integrated lockfile, reviewer CI, nanoid doctoring, exact-head CI contract and changelog remediation are inherited from protected main rather than replayed.Unique paths are:
.github/workflows/ci.yml.npmrcCHANGELOG.mddocs/doctoring/package-manager-reproducibility.mdpackage.jsonscripts/lockfile-change-control.mjstest/workflow-readiness.test.tsvitest.config.tsImplemented boundary
24.19.0/ npm11.17.0development and lockfile toolchain;strict-allow-scripts=truewith exact reviewed install-script identities and an explicit deny forfsevents@2.3.3;scripts/lockfile-change-control.mjs.Fresh exact-head evidence
Current exact head:
ebde3720ee27292be0ce3315986e0438dd66af9f.cirun31367707468: terminal success;reviewer-cirun31367707469: terminal success;Security Scanrun31367707433: terminal success under its own scanner/revision semantics;The fresh technical evidence is sufficient to leave Draft. It is not merge authority: live enforceable governance under #27 and qualifying independent formal review remain separate fail-closed gates.
Supersession
#89 was closed only after this clean protected-main successor received fresh terminal-success application/reviewer/security evidence proving the restack is technically intact. Old #89 evidence does not transfer.
Merge boundary
Do not merge unless the unchanged exact head still satisfies every actual current CI/security/coverage/package/provenance gate, current review/thread evidence remains clean, live ruleset/governance requirements are proven, and qualifying independent non-author formal approval is satisfied where required. No repair/self-modifying workflow, protection weakening, synthetic approval, version bump, release or deployment authority is introduced.
Related: #27, #29, #77, #79, #80, #90