Skip to content

Conversation

@o-az
Copy link
Member

@o-az o-az commented Oct 27, 2025

• SDK

  • Account upgrade support in React Native. In order to support account upgrades (wallet_prepareAccountUpgrade), access to webauthn.create must be present in the environment, regardless of mode (dialog or not). To cover this, I added a React Native passkey adapter that wires react-native-passkeys into the default relay fallback, handles RP host, and normalizes WebAuthn payload encoding
  • Updated SIWE handling (src/core/internal/siwe.ts), normalizing domains and preserving existing the React Native bridge and storage cache, preventing fallback to navigator.credentials and caching passkey verification for follow-up signatures.

• Dialog

  • Updated wallet_connect route to carry the original signInWithEthereum object, filling in domain/URI defaults without destructuring into {}. Also fixes schema error introduced in the prior PR

react-native-relay-mode example:

  • Session keys,
  • Working SIWE,
  • EOA to Porto account upgrade

Deliverables:

  • core SDK support integration,
  • examples updates,
  • docs updates

@changeset-bot
Copy link

changeset-bot bot commented Oct 27, 2025

⚠️ No Changeset found

Latest commit: 126565f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Oct 27, 2025

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

Project Deployment Preview Comments Updated (UTC)
dialog.porto Ready Ready Preview Comment Nov 6, 2025 1:17pm
docs.porto Ready Ready Preview Comment Nov 6, 2025 1:17pm
id.porto Ready Ready Preview Comment Nov 6, 2025 1:17pm
playground.porto Ready Ready Preview Comment Nov 6, 2025 1:17pm
wagmi.porto Ready Ready Preview Comment Nov 6, 2025 1:17pm
3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
interop.porto Ignored Ignored Preview Nov 6, 2025 1:17pm
porto-ui Ignored Ignored Preview Nov 6, 2025 1:17pm
proxy.porto Ignored Ignored Preview Nov 6, 2025 1:17pm

@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 27, 2025

Open in StackBlitz

npm i https://pkg.pr.new/porto@1000

commit: 126565f

@github-actions

This comment was marked as off-topic.

@github-actions

This comment was marked as resolved.

@o-az o-az force-pushed the o-az/rn-acct-upgrade-siwe branch 2 times, most recently from 4f5fc5d to a2ab290 Compare October 30, 2025 19:15
@vercel vercel bot temporarily deployed to Preview – interop.porto October 30, 2025 20:38 Inactive
@o-az o-az force-pushed the o-az/rn-acct-upgrade-siwe branch from 7993dd8 to 5e03969 Compare October 30, 2025 21:56
@o-az o-az marked this pull request as ready for review October 30, 2025 21:57
@o-az o-az force-pushed the o-az/rn-acct-upgrade-siwe branch from 5e03969 to 6b0830a Compare October 30, 2025 23:37
@o-az o-az changed the title feat: React Native account upgrade support feat: React Native SIWE & account upgrade support Oct 30, 2025
@o-az o-az requested a review from jxom November 1, 2025 04:35
})

return { message, signature: signature_erc8010 }
// If authUrl is provided, authenticate with the server
Copy link
Collaborator

Choose a reason for hiding this comment

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

Confirming that we don’t run into the double authenticate issue when using with dialog?

Copy link
Collaborator

@jxom jxom left a comment

Choose a reason for hiding this comment

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

Looks good as long as it still works with existing auth examples and no double authenticate.

@o-az o-az force-pushed the o-az/rn-acct-upgrade-siwe branch from 9fb5209 to c24ecf9 Compare November 4, 2025 04:14
@o-az o-az force-pushed the o-az/rn-acct-upgrade-siwe branch from 96c21a8 to 126565f Compare November 6, 2025 13:05
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