Skip to content

fix: support universal multi-call deploys - #25275

Open
loglapa wants to merge 1 commit into
AztecProtocol:merge-train/spartanfrom
loglapa:vl/fix-universal-multicall-deploy
Open

fix: support universal multi-call deploys#25275
loglapa wants to merge 1 commit into
AztecProtocol:merge-train/spartanfrom
loglapa:vl/fix-universal-multicall-deploy

Conversation

@loglapa

@loglapa loglapa commented Aug 20, 2026

Copy link
Copy Markdown

Fixes #23629

Summary

  • let wallet implementations select the entrypoint used for transactions without a signing account
  • use DefaultMultiCallEntrypoint for CLI wallet NO_FROM simulation and submission paths
  • cover universal deployment without -f using a sponsored FPC in the CLI wallet flow

Testing

  • yarn build
  • yarn workspace @aztec/wallet-sdk test base-wallet/base_wallet.test.ts
  • LOG_LEVEL=error yarn-project/end-to-end/scripts/run_test.sh compose ../cli-wallet/test/flows/universal_deploy.sh
  • yarn format cli-wallet
  • yarn format wallet-sdk
  • yarn lint cli-wallet
  • yarn lint wallet-sdk

The build and package checks ran in aztecprotocol/devbox:3.0. The compose flow exercised the local Aztec network and sponsored FPC end to end.

@loglapa
loglapa marked this pull request as ready for review August 20, 2026 09:55
@loglapa
loglapa force-pushed the vl/fix-universal-multicall-deploy branch from 78279d9 to cf463f3 Compare August 21, 2026 10:47
@loglapa

loglapa commented Aug 21, 2026

Copy link
Copy Markdown
Author

Rebased onto the current merge-train/spartan tip. Validation in aztecprotocol/devbox:3.0: immutable install, full TypeScript build, wallet-sdk tests (30/30), cli-wallet test command (no Jest tests in that package), format check, lint, and git diff --check all pass. The repository-wide bootstrap reaches the unchanged ipc-runtime prerequisite and stops because this devbox warns that Rust 1.89.0 is not installed; the yarn-project build and changed tests are green. The failed external check only reports the missing ci-external/ci-external-once label; a maintainer needs to add it because contributors cannot add repository labels.

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