Skip to content

fix(scripts): add missing ArithmeticOverflow and TooManyDeposits error codes (#497) - #629

Open
urielit1313-creator wants to merge 1 commit into
MergeFi:mainfrom
urielit1313-creator:fix/error-codes-missing-variants-497
Open

urielit1313-creator wants to merge 1 commit into
MergeFi:mainfrom
urielit1313-creator:fix/error-codes-missing-variants-497

Conversation

@urielit1313-creator

Copy link
Copy Markdown

Summary of Changes

Closes #497

  1. Synchronize Error Code Mappings:
    • Added 21: "ArithmeticOverflow" to errorMappings.escrow.
    • Added 21: "ArithmeticOverflow" to errorMappings.milestones.
    • Added 15: "TooManyDeposits" and 16: "ArithmeticOverflow" to errorMappings.maintenancePool.
  2. Proper Error Decoding:
    • Resolves issues where invoking methods resulting in ArithmeticOverflow or TooManyDeposits printed "No known mapping for error code".

Testing & Verification

  • Tested with Node.js asserting that decodeErrorCode(21), decodeErrorCode(15), and decodeErrorCode(16) resolve to the correct contract variant strings.

Bounty Payout Address (Base USDC): 0xfE777c43e4B0A24fFA1527eD520d1E7f9ECED483

@vercel

vercel Bot commented Oct 14, 2026

Copy link
Copy Markdown

@urielit1313-creator is attempting to deploy a commit to the chonilius' projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

scripts/error-codes.mjs is missing ArithmeticOverflow and maintenance-pool's TooManyDeposits

1 participant