Skip to content

feat: iframe wallets sdk#21978

Merged
Thunkar merged 7 commits into
nextfrom
gj/iframe_wallets_sdk
Mar 26, 2026
Merged

feat: iframe wallets sdk#21978
Thunkar merged 7 commits into
nextfrom
gj/iframe_wallets_sdk

Conversation

@Thunkar

@Thunkar Thunkar commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

Adds support for iframe wallets. This has been tested on gregoswap and demo-wallet, landing it on aztec-packages so it's available in nightlies for further development.

Once stable, docs will follow!

Thunkar and others added 6 commits March 24, 2026 12:52
- Add schema parsing (parseWithOptionals) to iframe connection handler
  so serialized args are properly deserialized before calling wallet methods
- Fix race condition in iframe discovery by registering message listener
  before appending iframe to DOM
- Fix async iterator types in discovery session to satisfy TypeScript
- Deduplicate DisconnectCallback type into shared types.ts
- Use shared WalletInfo type instead of inline type literals
- Fix various lint issues (no-misused-promises, require-await, jsdoc)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Thunkar Thunkar requested a review from mverzilli March 24, 2026 22:28
@Thunkar Thunkar self-assigned this Mar 24, 2026
Comment thread yarn-project/wallet-sdk/src/iframe/handlers/iframe_connection_handler.ts Outdated
Comment thread yarn-project/wallet-sdk/src/iframe/provider/iframe_discovery.ts Outdated
Comment thread yarn-project/wallet-sdk/src/manager/wallet_manager.ts Outdated
@Thunkar Thunkar added the claudebox Owned by claudebox. it can push to this PR. label Mar 25, 2026
AztecBot added a commit that referenced this pull request Mar 25, 2026
@AztecBot AztecBot force-pushed the gj/iframe_wallets_sdk branch from 387017a to 0cc4d9b Compare March 25, 2026 09:52
Comment on lines +34 to +36
type IteratorState =
| { status: 'discovering'; resolve: ((result: IteratorResult<WalletProvider>) => void) | null }
| { status: 'done' };

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.

🤩

@AztecBot AztecBot force-pushed the gj/iframe_wallets_sdk branch from 0cc4d9b to 5d31ab1 Compare March 25, 2026 10:42
@Thunkar Thunkar enabled auto-merge (squash) March 25, 2026 11:12
Base automatically changed from merge-train/fairies to next March 26, 2026 04:08
@Thunkar Thunkar added this pull request to the merge queue Mar 26, 2026
Merged via the queue into next with commit 823c5c9 Mar 26, 2026
13 checks passed
@Thunkar Thunkar deleted the gj/iframe_wallets_sdk branch March 26, 2026 04:42
AztecBot pushed a commit that referenced this pull request Mar 26, 2026
Adds support for iframe wallets. This has been tested on gregoswap and
demo-wallet, landing it on aztec-packages so it's available in nightlies
for further development.

Once stable, docs will follow!
@AztecBot

Copy link
Copy Markdown
Collaborator

✅ Successfully backported to backport-to-v4-next-staging #22036.

AztecBot added a commit that referenced this pull request Mar 26, 2026
BEGIN_COMMIT_OVERRIDE
feat: iframe wallets sdk (#21978)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-to-v4-next claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants