Skip to content

[Smart Accounts Kit][Troubleshoot] Invalid Delegator#2766

Merged
AyushBherwani1998 merged 4 commits into
mainfrom
feat/invalid-delegator
Mar 18, 2026
Merged

[Smart Accounts Kit][Troubleshoot] Invalid Delegator#2766
AyushBherwani1998 merged 4 commits into
mainfrom
feat/invalid-delegator

Conversation

@AyushBherwani1998

@AyushBherwani1998 AyushBherwani1998 commented Mar 17, 2026

Copy link
Copy Markdown
Member

Description

  • Adds Invalid Delegator troubleshooting page

Issue(s) fixed

Part of #2761

Preview

Checklist

  • If this PR updates or adds documentation content that changes or adds technical meaning, it has received an approval from an engineer or DevRel from the relevant team.
  • If this PR updates or adds documentation content, it has received an approval from a technical writer.

External contributor checklist

  • I've read the contribution guidelines.
  • I've created a new issue (or assigned myself to an existing issue) describing what this PR addresses.

Note

Low Risk
Documentation-only update that adds a new troubleshooting page and links it from the error code table; no runtime or contract logic changes.

Overview
Adds a new troubleshooting page for the InvalidDelegator() revert, explaining when it occurs and how to resolve it (including an example disableDelegation user operation for smart-account delegators).

Updates error-codes.md to refine the InvalidDelegator() description and link directly to the new invalid-delegator.md page.

Written by Cursor Bugbot for commit b91f883. This will update automatically on new commits. Configure here.

@AyushBherwani1998
AyushBherwani1998 requested review from a team as code owners March 17, 2026 09:02
@vercel

vercel Bot commented Mar 17, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
metamask-docs Ready Ready Preview, Comment Mar 18, 2026 10:16am

Request Review

cursor Bot pushed a commit that referenced this pull request Mar 17, 2026
- Review PR #2768 (Invalid signature) and PR #2766 (Invalid delegator)
- Check alignment with Consensys style guide
- Identify issues with conversational tone, contractions, and formatting
- Provide prioritized recommendations for improvements

Co-authored-by: Ayush Bherwani <ayush.bherwani1998@gmail.com>
cursor Bot pushed a commit that referenced this pull request Mar 17, 2026
- Separate review comments for PR #2768 and PR #2766
- Formatted for direct use in GitHub PR review interface
- Includes specific line numbers and diff suggestions

Co-authored-by: Ayush Bherwani <ayush.bherwani1998@gmail.com>

@bgravenorst bgravenorst left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor feedback.

Comment thread smart-accounts-kit/troubleshooting/invalid-delegator.md Outdated
Comment thread smart-accounts-kit/troubleshooting/error-codes.md Outdated

@shahbaz17 shahbaz17 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@AyushBherwani1998
AyushBherwani1998 merged commit bf266b8 into main Mar 18, 2026
18 checks passed
@AyushBherwani1998
AyushBherwani1998 deleted the feat/invalid-delegator branch March 18, 2026 15:50
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