feat: add use send calls hook #1649

moldy530 · 2025-05-29T20:47:26Z

Pull Request Checklist

Did you add new tests and confirm existing tests pass? (yarn test)
Did you update relevant docs? (docs are found in the site folder, and guidelines for updating/adding docs can be found in the contribution guide)
Do your commits follow the Conventional Commits standard?
Does your PR title also follow the Conventional Commits standard?
If you have a breaking change, is it correctly reflected in your commit message? (e.g. feat!: breaking change)
Did you run lint (yarn lint:check) and fix any issues? (yarn lint:write)
Did you follow the contribution guidelines?

PR-Codex overview

This PR introduces a new mode property for configuration, updates the @account-kit/wallet-client version, and refines the useSmartWalletClient and useSendUserOperation hooks to support the new mode. It also enhances type definitions and error handling in user operations.

Detailed summary

Added mode property in createConfig.ts with a default value of "local".
Updated @account-kit/wallet-client version from 0.1.0-alpha.4 to 0.1.0-alpha.6.
Modified useSmartWalletClient to accept a generic type for accounts.
Enhanced getSmartWalletClient to utilize the new mode parameter.
Refined useSendUserOperation to handle new logic for user operations.
Introduced useSendCalls hook for handling calls with Smart Wallet.
Updated type definitions to include mode as either "local" or "remote".

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

vercel · 2025-05-29T20:47:27Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
aa-sdk-ui-demo	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 9, 2025 5:30pm

moldy530 · 2025-05-29T20:47:43Z

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

feat: add use send calls hook #1649 👈 (View in Graphite)
feat: add initial smart wallet client actions #1648 : 7 other dependent PRs (#1701 , #1706 , #1719 and 4 others)
main

How to use the Graphite Merge Queue

Add the label graphite-merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

_{An organization admin has enabled the Graphite Merge Queue in this repository.} _{Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.}

This stack of pull requests is managed by Graphite. Learn more about stacking.

account-kit/react/src/experimental/hooks/useSendCalls.ts

jakehobbs · 2025-06-02T17:30:52Z

account-kit/react/src/experimental/hooks/useSendCalls.ts

+        const signature = await client.signSignatureRequest(
+          preparedCalls.signatureRequest,
+        );
+
+        const { preparedCallIds } = await client.sendPreparedCalls({
+          ...preparedCalls,
+          signature,
+        });


i assume we are going to need to handle the 7702 stuff in here too, and include the eip7702Auth in the returned request below.

if we end up doing the "multi" signature type though, i think it would work w/ no changes here other than how we construct the returned request: https://github.com/alchemyplatform/wallet-server/pull/187/files#diff-f9efb2f119f513d936133e68e1f0faa125d4b2b5400ec33cbc170c5ba4fcc406R46

ah yea this is gonna have to be updated with the breaking changes we've made for 7702

* feat: add initial smart wallet client actions * feat: add use send calls hook (#1649) * fix: address PR comments * fix(ui-demo): update the demo app to handle correct proxy for wallet api methods * feat: add wallet client hooks (#1733) * feat(core): leverage the signer proxy for chain agnostic routes (#1742) - [ ] Did you add new tests and confirm existing tests pass? (`yarn test`) - [ ] Did you update relevant docs? (docs are found in the `site` folder, and guidelines for updating/adding docs can be found in the [contribution guide](https://github.com/alchemyplatform/aa-sdk/blob/main/CONTRIBUTING.md)) - [ ] Do your commits follow the [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) standard? - [ ] Does your PR title also follow the [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) standard? - [ ] If you have a breaking change, is it [correctly reflected in your commit message](https://www.conventionalcommits.org/en/v1.0.0/#examples)? (e.g. `feat!: breaking change`) - [ ] Did you run lint (`yarn lint:check`) and fix any issues? (`yarn lint:write`) - [ ] Did you follow the [contribution guidelines](https://github.com/alchemyplatform/aa-sdk/blob/main/CONTRIBUTING.md)?  --- This PR focuses on enhancing the configuration management for transport connections in the `createConfig.ts` file by adding a `chainAgnosticUrl` property to the transport configuration, allowing for better handling of RPC URLs. - Added `chainAgnosticUrl` to the transport configuration in `createConfig.ts`. - Updated logic to set `chainAgnosticUrl` based on `signerConnection.rpcUrl`. - Modified tests in `store.test.ts` to verify the presence of `chainAgnosticUrl` in various configurations. - Ensured backward compatibility by using existing transport configurations when necessary. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`  * fix: misc fixes from dogfooding wallet apis (#1748) * feat: port over wallet client (#1753) * chore: new package template * feat: port over client * chore: update to latest * docs: lint cleanup * test: attempt to exclude wallet client e2e from unit test runner * Revert "test: attempt to exclude wallet client e2e from unit test runner" This reverts commit a4a0fd4. * test: try to exclude bun tests again * docs: attempt at docsgen * fix: add index.mdx * fix: remove ai slop * docs: cleanup signSignatureRequest * fix: omit internal export * docs: fix session signer hoisting * fix: docs fixes * fix: fix default client retries (#1761) * fix: use workspace version of wallet client (#1762) * docs: cleanup todo * feat: support policyIds array on wallet client * feat: support multiple policyIds in wallet client (#1763) * feat: support multiple policyIds in wallet client * chore: bump wallet types --------- Co-authored-by: jakehobbs <[email protected]> Co-authored-by: Adam Egyed <[email protected]> Co-authored-by: adam <[email protected]>

* feat: add initial smart wallet client actions * feat: add use send calls hook (alchemyplatform#1649) * fix: address PR comments * fix(ui-demo): update the demo app to handle correct proxy for wallet api methods * feat: add wallet client hooks (alchemyplatform#1733) * feat(core): leverage the signer proxy for chain agnostic routes (alchemyplatform#1742) - [ ] Did you add new tests and confirm existing tests pass? (`yarn test`) - [ ] Did you update relevant docs? (docs are found in the `site` folder, and guidelines for updating/adding docs can be found in the [contribution guide](https://github.com/alchemyplatform/aa-sdk/blob/main/CONTRIBUTING.md)) - [ ] Do your commits follow the [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) standard? - [ ] Does your PR title also follow the [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) standard? - [ ] If you have a breaking change, is it [correctly reflected in your commit message](https://www.conventionalcommits.org/en/v1.0.0/#examples)? (e.g. `feat!: breaking change`) - [ ] Did you run lint (`yarn lint:check`) and fix any issues? (`yarn lint:write`) - [ ] Did you follow the [contribution guidelines](https://github.com/alchemyplatform/aa-sdk/blob/main/CONTRIBUTING.md)?  --- This PR focuses on enhancing the configuration management for transport connections in the `createConfig.ts` file by adding a `chainAgnosticUrl` property to the transport configuration, allowing for better handling of RPC URLs. - Added `chainAgnosticUrl` to the transport configuration in `createConfig.ts`. - Updated logic to set `chainAgnosticUrl` based on `signerConnection.rpcUrl`. - Modified tests in `store.test.ts` to verify the presence of `chainAgnosticUrl` in various configurations. - Ensured backward compatibility by using existing transport configurations when necessary. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`  * fix: misc fixes from dogfooding wallet apis (alchemyplatform#1748) * feat: port over wallet client (alchemyplatform#1753) * chore: new package template * feat: port over client * chore: update to latest * docs: lint cleanup * test: attempt to exclude wallet client e2e from unit test runner * Revert "test: attempt to exclude wallet client e2e from unit test runner" This reverts commit a4a0fd4. * test: try to exclude bun tests again * docs: attempt at docsgen * fix: add index.mdx * fix: remove ai slop * docs: cleanup signSignatureRequest * fix: omit internal export * docs: fix session signer hoisting * fix: docs fixes * fix: fix default client retries (alchemyplatform#1761) * fix: use workspace version of wallet client (alchemyplatform#1762) * docs: cleanup todo * feat: support policyIds array on wallet client * feat: support multiple policyIds in wallet client (alchemyplatform#1763) * feat: support multiple policyIds in wallet client * chore: bump wallet types --------- Co-authored-by: jakehobbs <[email protected]> Co-authored-by: Adam Egyed <[email protected]> Co-authored-by: adam <[email protected]>

moldy530 mentioned this pull request May 29, 2025

feat: add initial smart wallet client actions #1648

Merged

7 tasks

vercel bot deployed to Preview May 29, 2025 20:54 View deployment

moldy530 force-pushed the moldy/add-exp-use-send-calls-hook branch from 22d2016 to 9ad7468 Compare May 29, 2025 21:05

moldy530 force-pushed the moldy/wallet-api-base branch from 895fe14 to 85be9fa Compare May 29, 2025 21:06

moldy530 force-pushed the moldy/add-exp-use-send-calls-hook branch from 9ad7468 to b027dfe Compare May 29, 2025 21:06

vercel bot deployed to Preview May 29, 2025 21:18 View deployment

moldy530 commented Jun 2, 2025

View reviewed changes

account-kit/react/src/experimental/hooks/useSendCalls.ts Show resolved Hide resolved

jakehobbs reviewed Jun 2, 2025

View reviewed changes

moldy530 force-pushed the moldy/add-exp-use-send-calls-hook branch from b027dfe to 39dbd19 Compare June 4, 2025 15:43

moldy530 force-pushed the moldy/wallet-api-base branch from 85be9fa to 0b53828 Compare June 4, 2025 15:43

vercel bot deployed to Preview June 4, 2025 15:50 View deployment

moldy530 force-pushed the moldy/add-exp-use-send-calls-hook branch from 39dbd19 to 4f8c06c Compare June 9, 2025 15:12

moldy530 force-pushed the moldy/wallet-api-base branch from 0b53828 to 215f8a6 Compare June 9, 2025 15:12

vercel bot deployed to Preview June 9, 2025 15:19 View deployment

moldy530 force-pushed the moldy/add-exp-use-send-calls-hook branch from 4f8c06c to 462f1f1 Compare June 9, 2025 17:09

vercel bot had a problem deploying to Preview June 9, 2025 17:14 Failure

feat: add use send calls hook

e1ce20d

moldy530 force-pushed the moldy/add-exp-use-send-calls-hook branch from 462f1f1 to e1ce20d Compare June 9, 2025 17:24

vercel bot deployed to Preview June 9, 2025 17:30 View deployment

moldy530 marked this pull request as ready for review June 9, 2025 18:11

moldy530 requested review from dancoombs, dphilipson, linnall, adamegyed, howydev, Zer0dot, jaypaik and avarobinson as code owners June 9, 2025 18:11

moldy530 requested review from florrdv and blakecduncan as code owners June 9, 2025 18:11

moldy530 merged commit ed3fd7b into moldy/wallet-api-base Jun 9, 2025
6 checks passed

moldy530 deleted the moldy/add-exp-use-send-calls-hook branch June 9, 2025 18:11

moldy530 added a commit that referenced this pull request Jun 9, 2025

feat: add use send calls hook (#1649)

3850778

This was referenced Jun 17, 2025

feat: use smart wallet client in signMessage/signTypedData hooks #1701

Merged

feat: add useGetCallsStatus #1706

Merged

moldy530 added a commit that referenced this pull request Jun 19, 2025

feat: add use send calls hook (#1649)

9b2cd4f

jakehobbs mentioned this pull request Jun 20, 2025

feat: add usePrepareCalls & useSendPreparedCalls hooks #1719

Merged

7 tasks

moldy530 added a commit that referenced this pull request Jun 23, 2025

feat: add use send calls hook (#1649)

b072b4c

moldy530 mentioned this pull request Jun 23, 2025

feat: support all wallet server account types in account-kit/core #1723

Merged

7 tasks

moldy530 added a commit that referenced this pull request Jun 23, 2025

feat: add use send calls hook (#1649)

d64d543

jakehobbs mentioned this pull request Jun 23, 2025

feat: add useGrantPermissions hook #1725

Merged

7 tasks

moldy530 added a commit that referenced this pull request Jun 25, 2025

feat: add use send calls hook (#1649)

c980fe2

moldy530 mentioned this pull request Jun 25, 2025

refactor(ui-demo): refactor session key card to use wallet apis #1732

Closed

7 tasks

moldy530 added a commit that referenced this pull request Jun 30, 2025

feat: add use send calls hook (#1649)

feebe54

jakehobbs mentioned this pull request Jul 1, 2025

fix: misc fixes from dogfooding wallet apis #1748

Merged

adamegyed pushed a commit that referenced this pull request Jul 11, 2025

feat: add use send calls hook (#1649)

17ab966

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: add use send calls hook #1649

feat: add use send calls hook #1649

Uh oh!

moldy530 commented May 29, 2025 •

edited by pr-codex bot

Loading

Uh oh!

vercel bot commented May 29, 2025 •

edited

Loading

Uh oh!

moldy530 commented May 29, 2025 •

edited by jakehobbs

Loading

Uh oh!

Uh oh!

Uh oh!

jakehobbs Jun 2, 2025

Uh oh!

jakehobbs Jun 2, 2025 •

edited

Loading

Uh oh!

moldy530 Jun 6, 2025

Uh oh!

Uh oh!

Uh oh!

feat: add use send calls hook #1649

feat: add use send calls hook #1649

Uh oh!

Conversation

moldy530 commented May 29, 2025 • edited by pr-codex bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pull Request Checklist

PR-Codex overview

Detailed summary

Uh oh!

vercel bot commented May 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

moldy530 commented May 29, 2025 • edited by jakehobbs Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

How to use the Graphite Merge Queue

Uh oh!

Uh oh!

Uh oh!

jakehobbs Jun 2, 2025

Choose a reason for hiding this comment

Uh oh!

jakehobbs Jun 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

moldy530 Jun 6, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

moldy530 commented May 29, 2025 •

edited by pr-codex bot

Loading

vercel bot commented May 29, 2025 •

edited

Loading

moldy530 commented May 29, 2025 •

edited by jakehobbs

Loading

jakehobbs Jun 2, 2025 •

edited

Loading