Skip to content

Fix MetaMask success error handling#47

Open
2bf wants to merge 1 commit into
flashbots:mainfrom
2bf:codex/metamask-success-error-36
Open

Fix MetaMask success error handling#47
2bf wants to merge 1 commit into
flashbots:mainfrom
2bf:codex/metamask-success-error-36

Conversation

@2bf

@2bf 2bf commented May 20, 2026

Copy link
Copy Markdown

Summary

  • Treat known MetaMask success-after-add errors as successful RPC additions instead of showing an alert
  • Normalize wallet error code/message handling so missing fields do not throw inside the catch handler
  • Preserve user-rejection behavior and existing fallback errors

Fixes #36.

Verification

  • npm run lint
  • npm run build

Note: pnpm is not installed in this environment, so dependencies were installed with npm install --no-package-lock --ignore-scripts for local verification without changing lockfiles.

@2bf 2bf requested a review from sketsdever as a code owner May 20, 2026 08:12
@vercel

vercel Bot commented May 20, 2026

Copy link
Copy Markdown

@2bf is attempting to deploy a commit to the Flashbots Team on Vercel.

A member of the Team first needs to authorize it.

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.

Fix/remove metamask success error

1 participant