Skip to content

fix: include standard-contracts historical artifacts in release image - #25155

Merged
spalladino merged 1 commit into
merge-train/spartan-v5from
spl/fix-release-image-historical-artifacts
Aug 10, 2026
Merged

fix: include standard-contracts historical artifacts in release image#25155
spalladino merged 1 commit into
merge-train/spartan-v5from
spl/fix-release-image-historical-artifacts

Conversation

@spalladino

Copy link
Copy Markdown
Contributor

Since #25032, @aztec/standard-contracts has a top-level import of historical artifact JSONs from standard-contracts/artifacts-historical/. The release image dockerignore ignores everything and then whitelists specific paths, and it only whitelisted standard-contracts/artifacts/ — so the historical artifacts never made it into the image.

  • Every node built from a v5-next release image crash-loops on startup at ESM link time with ERR_MODULE_NOT_FOUND for HandshakeRegistry-5.0.1.json.
  • This has broken all spartan deploys from this line since 2026-07-29.
  • The fix is a single whitelist entry: !/yarn-project/standard-contracts/artifacts-historical/.

Split out from #25148 so the deploy fix can land independently of that PR's review. Once this merges, #25148 will be rebased and its duplicate commit will drop out.

PR #25032 added a top-level import of
standard-contracts/artifacts-historical/HandshakeRegistry-5.0.1.json, but
the release image dockerignore only whitelists artifacts/, so every node
started from a v5-line release image crash-loops at ESM link time with
ERR_MODULE_NOT_FOUND. This broke all spartan network deploys from
v5-next since 2026-07-29.
@spalladino
spalladino requested a review from charlielye as a code owner August 10, 2026 14:42
@AztecBot AztecBot added the port-to-next Forward-port this merged PR into next label Aug 10, 2026
@spalladino
spalladino requested a review from nchamo August 10, 2026 14:52
@spalladino

Copy link
Copy Markdown
Contributor Author

@nchamo does this need porting to v6?

@nchamo

nchamo commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

No, we don't need to port it to v6. In v6 we'll start "from scratch", without legacy versions. So no need to do this

@spalladino spalladino removed the port-to-next Forward-port this merged PR into next label Aug 10, 2026
@spalladino
spalladino merged commit ec75942 into merge-train/spartan-v5 Aug 10, 2026
23 of 27 checks passed
@spalladino
spalladino deleted the spl/fix-release-image-historical-artifacts branch August 10, 2026 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants