Skip to content

[Smart Accounts Page] Improve concepts page#2788

Open
AyushBherwani1998 wants to merge 3 commits intomainfrom
feat/concepts-improvements
Open

[Smart Accounts Page] Improve concepts page#2788
AyushBherwani1998 wants to merge 3 commits intomainfrom
feat/concepts-improvements

Conversation

@AyushBherwani1998
Copy link
Member

@AyushBherwani1998 AyushBherwani1998 commented Mar 23, 2026

Description

  • Improves Concepts page for Delegation

Issue(s) fixed

Fixes #2772

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 restructure of the Delegation concepts section (new pages + link updates); main risk is broken internal links or sidebar ordering issues.

Overview
Reworks the Delegation Concepts docs into a clearer multi-page structure. The previous concepts/delegation/index.md is removed and replaced with concepts/delegation/overview.md, plus new concept pages for delegation-manager (execution modes/processing) and delegation-scopes (scope vs caveat framing).

Updates the Smart Accounts Kit sidebar and fixes/retargets inbound links across guides, reference, troubleshooting, and Advanced Permissions docs to point at the new overview and the new execution-mode anchor under delegation-manager. Also trims and reformats caveat-enforcers.md to focus on hook behavior (table + flow references) instead of embedding the full Solidity interface and other long sections.

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

@AyushBherwani1998 AyushBherwani1998 requested review from a team as code owners March 23, 2026 10:37
@vercel
Copy link

vercel bot commented Mar 23, 2026

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

Project Deployment Actions Updated (UTC)
metamask-docs Ready Ready Preview, Comment Mar 23, 2026 6:23pm

Request Review

Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a reason to prefer having a separate "Overview" page vs. an index page that displays when you click the "Delegation" label? This would save space on the sidebar and remove one click.

Copy link
Member Author

Choose a reason for hiding this comment

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

It's not ideal, we want to move away from have index page. It's not the natural UX of how sites work with drop down.

Comment on lines +158 to +161
| Component | Description |
|---|---|
| [Delegation Manager](delegation-manager.md) | Validates delegations and triggers executions on behalf of the delegator, ensuring tasks are executed accurately and securely. |
| [Caveat enforcers](caveat-enforcers.md) | Manage rules and restrictions for delegations, providing fine-tuned control over delegated executions. | No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

Since there are only two items, this might be better as a bullet point list.

Copy link
Member Author

Choose a reason for hiding this comment

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

since it's a lot of content on the page, it's not easy to traverse. tables are good for quick looks. Don't have a strong preference tho. It just feels more readable compared to bullet points.

Comment on lines +23 to +25
## Categories

The Smart Accounts Kit supports three categories of scopes:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
## Categories
The Smart Accounts Kit supports three categories of scopes:
## Scope types
The Smart Accounts Kit supports three types of scopes:

Copy link
Member Author

Choose a reason for hiding this comment

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

It's categories, instead of types. Scope types would be specific scope name.

Co-authored-by: Alexandra Carrillo <12214231+alexandratran@users.noreply.github.com>
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.

[Smart Accounts Kit] Improve Concepts page

2 participants