Skip to content

2483d embedded wallets pass#2603

Open
m4sterbunny wants to merge 16 commits intomainfrom
2483d-embedded-wallets-pass
Open

2483d embedded wallets pass#2603
m4sterbunny wants to merge 16 commits intomainfrom
2483d-embedded-wallets-pass

Conversation

@m4sterbunny
Copy link
Contributor

@m4sterbunny m4sterbunny commented Dec 29, 2025

Description

Simplification how the cryptography is presented
Proofs
Applies docs style

DOES NOT - touch images

Issue(s) fixed

Partial fix for 2483 #

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
Primarily documentation edits plus a KaTeX stylesheet/dependency update; low functional risk, with only minor chance of build/regression if KaTeX version/CSS integrity mismatches.

Overview
Improves Embedded Wallets documentation clarity by reframing cryptography/key-management explanations (SSS vs TSS/MPC), tightening terminology, and adding more explicit step-by-step flows and trust/verification details (including Torus network context and front‑running protection).

Adds a shared _sdk-key-model-table.mdx partial and embeds it across infrastructure and SDK overview pages to help readers choose the correct signing/key model, while also enabling math rendering site-wide via KaTeX CSS inclusion and adding the katex dependency (updating lockfile accordingly).

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

@vercel
Copy link

vercel bot commented Dec 29, 2025

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

Project Deployment Actions Updated (UTC)
metamask-docs Ready Ready Preview, Comment Feb 5, 2026 7:16am

Request Review

@socket-security
Copy link

socket-security bot commented Dec 29, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedkatex@​0.16.22 ⏵ 0.16.2793100100 +191 -2100

View full report

@socket-security
Copy link

socket-security bot commented Dec 29, 2025

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

View full report

bgravenorst
bgravenorst previously approved these changes Feb 2, 2026
Copy link
Contributor

@bgravenorst bgravenorst left a comment

Choose a reason for hiding this comment

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

lgtm

@m4sterbunny
Copy link
Contributor Author

PTAL @bgravenorst had to fix conflict making review stale

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Copy link
Contributor

@bgravenorst bgravenorst left a comment

Choose a reason for hiding this comment

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

lgtm

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