Skip to content

fix(metamask): connect the granted subset on partial multichain session grants - #135

Merged
towanTG merged 1 commit into
developfrom
task/metamask-multichain-connector-e48103ed-a1
Jul 30, 2026
Merged

fix(metamask): connect the granted subset on partial multichain session grants#135
towanTG merged 1 commit into
developfrom
task/metamask-multichain-connector-e48103ed-a1

Conversation

@towanTG

@towanTG towanTG commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

No description provided.

…on grants

MetaMask can approve fewer scopes than requested (no Solana account, an
EVM network not added). The connector failed the entire connect on the
first ungranted scope; now it connects the granted chains (still atomic
for that set) and throws wallet_chain_not_supported only when nothing was
granted. Found by UI nightly validation (widget requests 14 chains).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@towanTG
towanTG marked this pull request as ready for review July 30, 2026 11:23
@towanTG
towanTG merged commit e3f47a4 into develop Jul 30, 2026
1 check passed
@towanTG
towanTG deleted the task/metamask-multichain-connector-e48103ed-a1 branch July 30, 2026 11:23
towanTG pushed a commit that referenced this pull request Jul 30, 2026
Rebased onto develop: the MetaMask partial-grant half of this PR landed
separately as #135 — this keeps only the WC stale-pairing fallback
(client.connect with a reused pairingTopic throws on a stale relay topic;
fall back to a fresh pairing so the QR flow still appears) and its tests.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
towanTG added a commit that referenced this pull request Jul 30, 2026
Rebased onto develop: the MetaMask partial-grant half of this PR landed
separately as #135 — this keeps only the WC stale-pairing fallback
(client.connect with a reused pairingTopic throws on a stale relay topic;
fall back to a fresh pairing so the QR flow still appears) and its tests.

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.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.

1 participant