Skip to content

feat(sdk): add fhevm-npm check deployments - #140

Merged
0xalexbel merged 1 commit into
release/0.13.xfrom
devex/alexb/v13/check-deployments
Oct 8, 2026
Merged

0xalexbel merged 1 commit into
release/0.13.xfrom
devex/alexb/v13/check-deployments

Conversation

@0xalexbel

@0xalexbel 0xalexbel commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Adds fhevm-npm check deployments, the guard for a new deployments/ archive of on-chain deployment
records. This is the first of three PRs:

  1. This PR: the command, the folder rules, and the consistency check in make check-pre.
  2. CI: a workflow that runs the frozen-record check on every PR (next).
  3. The first record: the cleartext v13 Sepolia deployment of 2026-10-05 (after CI).

The model

A record is deployments/<chain>/<deploymentId>/, copied from the operator's deployment folder once the
on-chain event is final: the config the deploy tool ran with, the seal (manifest.json), the generated
addresses.sol and the transaction journal.

A record is evidence of transactions that cannot be undone, so it is frozen: once on the base branch,
nothing in it may change. A later event, such as an upgrade to v14 or an ownership change, gets a new
folder, which points at the previous seal through previousManifest. deployments/README.md states the
rules.

The command

./fhevm-npm-cli check deployments                                # consistency
./fhevm-npm-cli check deployments --base origin/release/0.13.x   # plus: no existing record changed

Deployment records live in deployments/<chain>/<deploymentId>/, one folder per on-chain event, and are
frozen once they land: a later event gets a new folder.

`check deployments` checks every record for completeness and consistency: required files, the same
deploymentId and admin in the folder name, the config and the seal, a chain folder that is a host chain of
fhevm-chains.config.json with the sealed chain id, and no forge scratch or nested repository. With
`--base <ref>` it also fails if a record folder that exists at the merge base changed in any way.

The consistency part runs in make check-pre. deployments/README.md states the rules, and
deployments/.gitignore keeps forge scratch out of commits.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@0xalexbel
0xalexbel requested a review from a team October 8, 2026 15:12
@cla-bot cla-bot Bot added the cla-signed label Oct 8, 2026
@0xalexbel
0xalexbel merged commit 07e8587 into release/0.13.x Oct 8, 2026
5 checks passed
@0xalexbel
0xalexbel deleted the devex/alexb/v13/check-deployments branch October 8, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants