Skip to content

CI hardening: required ABI surface + local anvil integration gates#26

Merged
snissn merged 2 commits into
masterfrom
feat/ci-abi-e2e
Feb 5, 2026
Merged

CI hardening: required ABI surface + local anvil integration gates#26
snissn merged 2 commits into
masterfrom
feat/ci-abi-e2e

Conversation

@snissn
Copy link
Copy Markdown
Contributor

@snissn snissn commented Feb 5, 2026

Implements the testing/CI hardening plan focused on preventing ABI/UI drift and enforcing local-chain integration on every PR.

What changed

  • Added ABI surface regression test:
    • test/testUiAbiSurface.js
    • Asserts generated ABI contains all UI-required functions per collection (listIds*, get*(uint256), create*, optional update*/transfer*, and delete*).
  • Added local E2E integration test:
    • test/integration/testCliLocalIntegration.js
    • Covers th preview auto-deploy on anvil, manifest publication, and faucet endpoint behavior.
  • Split scripts for CI gating:
    • pnpm test for static/unit/template checks.
    • pnpm test:integration for local-chain E2E checks.
  • Improved UI ABI mismatch diagnostics:
    • Added ABI function guards in UI contract call paths with explicit collection/function error messaging.
  • CI workflow split into two required jobs:
    • static
    • integration-local (installs Foundry, checks anvil, runs integration tests).
  • README updated with testing/CI usage.

Validation

  • pnpm test
  • pnpm test:integration
  • pnpm typecheck

Add ABI surface regression tests, local anvil preview/faucet integration tests, split CI into static and integration-local jobs, and improve UI ABI mismatch diagnostics.
@snissn snissn merged commit 0d7b964 into master Feb 5, 2026
2 checks passed
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