feat: add gas benchmarks to CI

[iap]

Summary

Adds gas snapshot tracking to catch gas regressions early.

Changes

• Gas snapshot baseline: 158 test gas measurements
• Makefile targets:
  • make gas-snapshot: generate new snapshot
  • make gas-check: verify against baseline
• CI integration: gas check runs after tests

Benefits

• Catch gas regressions before merge
• Track gas usage over time
• Prevent accidental gas increases

Verification

cd contracts
make gas-snapshot  # Generate baseline
make gas-check     # Verify (should pass)

Risk

Low - adds monitoring only, no code changes.

Related

• Closes Add gas benchmarks to CI #164

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

[greptile-apps]

Greptile Summary

This PR adds a forge snapshot-based gas regression guard to the Contracts CI, committing a 148-test baseline and two new Makefile targets (gas-snapshot / gas-check). It also removes .nvmrc and pnpm-workspace.yaml (both were single-entry/root-only files) and updates CONTRIBUTING.md to reflect mise as the sole Node.js version manager.

• Gas snapshot baseline (contracts/.gas-snapshot): 148 unit/e2e measurements captured by forge snapshot, with invariant and integration tests excluded via --nmp \"test/{invariant,integration}/**\".
• Makefile targets: gas-snapshot generates the baseline; gas-check re-runs the same filtered test set against the stored baseline with a 2% tolerance, failing if any test regresses beyond that threshold.
• CI step: make gas-check is appended to the contracts-unit-invariant job, gating downstream jobs on no gas regressions.

Confidence Score: 5/5

Safe to merge — the change is purely additive monitoring with no contract or production logic touched.

The PR only adds a committed gas baseline, two Makefile targets, and a CI step that reads that baseline. No contract code, deployment scripts, or critical configuration is modified. The unrelated file removals (.nvmrc, pnpm-workspace.yaml) are safe: the lockfile has a single root importer with no workspace-protocol dependencies, and Node version management is already handled by .mise.toml.

No files require special attention. The known non-determinism concern in contracts/.gas-snapshot (fuzz test mean varying across seeds) and the missing trailing newline were both raised in prior review threads.

Important Files Changed

Filename	Overview
.github/workflows/contracts-ci.yml	Appends a single make gas-check step to the contracts-unit-invariant job; straightforward addition with no structural changes to the workflow.
contracts/Makefile	Adds gas-snapshot and gas-check targets using forge snapshot; invariant and integration tests correctly excluded via --nmp; fuzz seed non-determinism and tolerance edge-case noted in prior threads.
contracts/.gas-snapshot	New 148-test baseline snapshot; missing trailing newline flagged in prior review thread.
CONTRIBUTING.md	Updates Node.js tooling references: removes .nvmrc mention, makes mise the primary version manager — consistent with the .nvmrc deletion.
.nvmrc	Deleted; Node 22 version pin is now carried solely by .mise.toml, consistent with CONTRIBUTING.md update.
pnpm-workspace.yaml	Deleted; file only declared packages: [.] (root only), matching the lockfile's single root importer — safe to remove with no workspace-protocol dependencies present.

Sequence Diagram

sequenceDiagram
    participant CI as GitHub Actions
    participant FT as forge test -vv
    participant PL as make test-production-lock
    participant GC as make gas-check
    participant SN as .gas-snapshot

    CI->>FT: Run deterministic test suite (all tests)
    FT-->>CI: Pass / Fail

    CI->>PL: Run production lock invariants
    PL-->>CI: Pass / Fail

    CI->>GC: "forge snapshot --check --snap .gas-snapshot --tolerance 2 --nmp test/{invariant,integration}/**"
    GC->>SN: Read baseline measurements
    SN-->>GC: 148 gas entries
    GC->>GC: Re-run unit/e2e tests, measure gas
    GC->>GC: Compare each test within 2% tolerance
    GC-->>CI: Pass (within tolerance) / Fail (regression detected)

Loading

_{Reviews (4): Last reviewed commit: "docs: update CONTRIBUTING.md for mise-on..." | Re-trigger Greptile}