[ENG-3120] Inject build fingerprint in mark-build for manifest parity#30
Merged
Merged
Conversation
|
Injects build fingerprint and production flag cleanly with modular code. 🎯 Quality: 100% Elite · 📦 Size: Large — consider splitting if possible 📈 This month: Your 53rd PR — above team average · Averaging Excellent |
mark-build now computes the manifest checksum from the lockfile and injects it (window.__PATCHSTACK_BUILD__) beside the production flag, so the disclosure widget can report the running build for manifest-parity checks. The fingerprint matches the server's byte-for-byte. Replaces the flag-only injector.
01ef539 to
6e9b869
Compare
Contributor
Author
|
/review |
daniloradovic
approved these changes
Jul 2, 2026
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 & why
connect mark-buildnow stamps built HTML with the build fingerprint — the manifest checksum computed from the lockfile — injected aswindow.__PATCHSTACK_BUILD__beside the existing production flag. The embedded disclosure widget reads it and reports it, so Patchstack can compare the running build against the last scanned manifest ("manifest parity").computeManifestChecksum(src/checksum.ts) — byte-for-byte identical to the server'sPulseController::storeManifestchecksum (sorted-package SHA-256, first 12 hex). Verified against the live SaaS endpoint (c14377037ec1on both sides).src/mark-build.ts— resolves the build dir, injects__PATCHSTACK_PROD__+__PATCHSTACK_BUILD__, idempotent re-runs.mark-buildCLI switched to the fingerprint injector; removes the superseded flag-onlybuildFlag.ts.Testing
vitest run— 107/107 (incl. newchecksum+mark-buildsuites)tsc --noEmitcleanconnect scan→mark-build→ widget heartbeat → SaaS parity.Cross-repo (3-repo feature)
Ref ENG-3120
🤖 Generated with Claude Code