Skip to content

feat: CI-enforced transfer limit tests#31

Open
anirudhvenu wants to merge 4 commits intomasterfrom
feat/transfer-limit-tests
Open

feat: CI-enforced transfer limit tests#31
anirudhvenu wants to merge 4 commits intomasterfrom
feat/transfer-limit-tests

Conversation

@anirudhvenu
Copy link
Copy Markdown
Contributor

@anirudhvenu anirudhvenu commented Mar 20, 2026

Changes

Adds a vitest test suite and CI workflow to enforce chain-specific transfer limits.

What's tested

  • Each chain's default maxBridgeAmount is 550
  • MegaETH has exactly one destination override (MegaETH: 5000)
  • Monad and Citrea have no destination overrides

Dependency update

Switched @avail-project/nexus-core from git-hosted references to the published npm package (^1.2.0). The git-hosted version requires building from source during pnpm install, which fails in CI due to a TypeScript type inference issue in the nexus-core build step.

Files added/modified

  • tests/chain-limits.test.ts — 7 test assertions
  • vitest.config.ts — test runner config
  • .github/workflows/ci-limits.yml — CI workflow on PRs and pushes to master, path-scoped to runtime and test files
  • package.json — added vitest devDependency and test script
  • turbo.json — added test task
  • apps/*/package.json, packages/fast-bridge-app/package.json — nexus-core switched from git to npm

How to update

When adding a new chain or destination override, update the corresponding test to assert the exact shape and entry count. See the MegaETH tests as a template.

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nexus-fast-bridge Ready Ready Preview, Comment Mar 20, 2026 11:17am

Request Review

@anirudhvenu anirudhvenu force-pushed the feat/transfer-limit-tests branch from 1f4287b to 434259e Compare March 20, 2026 10:11
@anirudhvenu anirudhvenu marked this pull request as ready for review March 20, 2026 10:19
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