Skip to content

fix(contracts): regenerate stale contract interface artifacts - #1579

Open
AlphaMaleBaDI wants to merge 1 commit into
Split-Naira:mainfrom
AlphaMaleBaDI:fix/contract-interface-drift
Open

AlphaMaleBaDI wants to merge 1 commit into
Split-Naira:mainfrom
AlphaMaleBaDI:fix/contract-interface-drift

Conversation

@AlphaMaleBaDI

Copy link
Copy Markdown

Summary

npm run verify:data-integrity currently fails on every branch, including main at 044976e:

[verify:data-integrity] Generated artifacts are out of date.

The committed artifacts have drifted from contracts/lib.rs and contracts/errors.rs. This PR is the output of running the generator — no hand edits.

npm run generate:contract-types

What drifted

  • AlreadyClaimed (error code 22) was never generated into the interface
  • ClaimEpoch and LastClaimEpoch storage keys were missing
  • Several error doc strings were stale

Changes

File Change
contracts/interface/splitnaira.contract-interface.json regenerated: +37/−17
backend/src/generated/contract-types.ts adds AlreadyClaimed: 22
frontend/src/generated/contract-types.ts adds AlreadyClaimed: 22

Validation

$ npm run verify:data-integrity
✓ contracts/README.md error table matches errors.rs
Generated contracts/interface/splitnaira.contract-interface.json
✓ contract interface artifact is current
Generated backend types: backend/src/generated/contract-types.ts
Generated frontend types: frontend/src/generated/contract-types.ts
✓ generated contract types match interface artifact

All data integrity checks passed.

Why this is worth merging on its own

This failure is red on main and blocks the build and data-integrity jobs for every pull request in the repository, including PRs that do not touch contracts. Fixing it here unblocks everyone.

Related: #362 (the CI gate that detects this drift).

No linked issue — this is artifact drift, not a feature.

The committed artifacts drifted from contracts/lib.rs and contracts/errors.rs: the AlreadyClaimed error (code 22), the ClaimEpoch/LastClaimEpoch storage keys, and several error doc strings were missing.

Because CI runs verify:data-integrity, this drift fails the build and data-integrity jobs on every branch, including main.

This branch has not been deployed

No deployments
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.

1 participant