Skip to content

Fix PaymasterStakeTooLowError message to match ERC-7769 spec#4391

Merged
jxom merged 1 commit intowevm:mainfrom
mohamedhesham840:fix/paymaster-stake-too-low-error-message
Mar 17, 2026
Merged

Fix PaymasterStakeTooLowError message to match ERC-7769 spec#4391
jxom merged 1 commit intowevm:mainfrom
mohamedhesham840:fix/paymaster-stake-too-low-error-message

Conversation

@mohamedhesham840
Copy link
Copy Markdown
Contributor

@mohamedhesham840 mohamedhesham840 commented Mar 11, 2026

Summary

PaymasterStakeTooLowError (code -32505) had the same error message as
PaymasterRateLimitError (code -32504). Both said
"paymaster (or signature aggregator) is throttled/banned."

Per ERC-7769, code -32505 means
the paymaster's stake or unstake-delay is too low, not that it's throttled/banned.

Before: "UserOperation rejected because paymaster (or signature aggregator) is throttled/banned."
After: "UserOperation rejected because paymaster (or signature aggregator) stake or unstake-delay is too low."

Changes

  • Fixed error message in PaymasterStakeTooLowError class
  • Updated corresponding inline snapshot in getBundlerError.test.ts

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 11, 2026

@mohamedhesham840 is attempting to deploy a commit to the Wevm Team on Vercel.

A member of the Team first needs to authorize it.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 11, 2026

🦋 Changeset detected

Latest commit: 872d168

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
viem Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@jxom jxom force-pushed the main branch 2 times, most recently from bdf8494 to 69e0d6e Compare March 13, 2026 22:42
@mohamedhesham840
Copy link
Copy Markdown
Contributor Author

Hi @jxom
Updated PaymasterStakeTooLowError to match the ERC-7769 spec and updated test.
Happy to adjust if needed

@jxom jxom merged commit a8c8e6e into wevm:main Mar 17, 2026
2 of 3 checks passed
@github-actions github-actions bot mentioned this pull request Mar 17, 2026
@tmm tmm mentioned this pull request Mar 30, 2026
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.

2 participants