docs: audit package reachability before removing shipped assets - #848
Merged
Merged
Conversation
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
devswha
marked this pull request as ready for review
September 15, 2026 13:54
devswha
marked this pull request as draft
September 15, 2026 14:06
Re-measure the package surface at a1116bf (origin/dev 3622d3b merged: PR #847, #849, #850, #851; PR-06 retired by the owner). Root tarball 1,521,037 B / 5,703,327 B unpacked / sha256 8164d622; alias unchanged. Category estimates, policy-docs subset (94,452 B), and the validation record re-run at this tree; the artifacts/rebaseline-2025 shipped-file count is corrected from 7 to 6 (the packed list always had 6 files at the recorded 439,339 B). Conclusion unchanged: KEEP / NO PR-08.
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.
What
Read-only audit of the npm package surface (PR-07, the final item of the
2026-09-15 maintenance series). Adds one document:
docs/operations/package-surface-audit-20260915.md. Nothing else changes.Status: final converged-chain refresh (2026-09-16) — evidence is current
The parking condition recorded on this PR has been met: the active
maintenance chain has fully converged into
dev(PR #846, #847, #849,#850, #851 all merged; PR-06 was retired by the owner without being
defined or started). The promised one final refresh and re-measurement
was performed on the converged tree.
Identity
origin/dev3622d3bb8667287cc089ce37b12d7bc8c385e72b(PR ci: execute the existing Redis quota regression in isolation #850 merged)467ca1b(refresh commit on top of merge commita1116bfc74d8d75aa2d4ebbc7e4a9c1d3a0f4178)a1116bf…+ the measurement-refresh commit; integration methodgit merge --no-edit origin/dev(no conflicts — the chain PRs touchedsrc/backends/*,docs/*,.github/workflows/test.yml, none of which this PR modifies)origin/dev: the single audit document (+297 lines net; the refresh itself is +46/−42 over the parked revision)67e0cd8…→ first refresh3c078b1…/f6d5a8d(reviewed against baseb0a3934)Re-measurement (all at the converged tree, linux x64 / Node v24.18.0 / npm 11.16.0)
npm pack --dry-run --json(root)npm pack --dry-run --json(alias)node scripts/release-artifacts.mjs --output-dir /tmp/maint07-release-refresh --source-sha a1116bf…8164d622…, alias63be3af3…), metadata-verified, clean-install smoked; no publishnpm run lintnpm run release:checknpm run check:no-private-assetsnpm run benchmark:perf -- --costmetricsOne factual correction found during re-measurement and fixed in the doc:
the
artifacts/rebaseline-2025/shipped-file count read "7 files" but thepacked list has always contained 6 files at the recorded 439,339 B total.
Review status (recorded honestly)
The independent read-only review PASS recorded on this PR covered head
f6d5a8dagainst baseb0a3934— that verdict applies to the audit'squalitative content (reachability analysis, candidate table, dispositions),
which this refresh does not change. The refresh itself is measurement-only
(SHAs, byte counts, hashes, and the validation record). No fresh
model-backed review was run for the refresh; merge is at the owner's
instruction to complete the maintenance series.
Why (Issue omission)
Documentation-only audit with no behavior, contract, or packaging change —
low-risk exemption per docs/WORKFLOW.md P02a; the work item is PR-07 of the
owner-issued maintenance plan.
Conclusion
Unchanged by the refreshed evidence:
KEEP / NO PR-08 FOLLOW-UP.Rollback
Revert the single document (two commits + merge commit on this branch).