Add TACo Account Abstraction signing demo #693

theref · 2025-08-19T15:48:05Z

Summary

Add comprehensive TACo Account Abstraction signing demo for adopters
Single-file implementation showing TACo threshold signatures with ERC-4337
Complete documentation with example output and setup instructions

Demo Features

Real TACo testnet integration (2-of-3 threshold)
MetaMask Delegation Toolkit smart account creation
Balance tracking and fund management
Pimlico bundler integration for UserOperation execution
Clean developer experience with minimal boilerplate

Test Plan

Demo builds successfully
All tests pass
Linting passes
Documentation is clear and complete

Copilot

Pull Request Overview

This PR adds a comprehensive TACo Account Abstraction signing demo that demonstrates how to use TACo's distributed threshold signatures with ERC-4337 Account Abstraction. The demo creates a smart account using MetaMask Delegation Toolkit with TACo signers and executes real transactions on Sepolia testnet.

Key Changes:

Added complete working demo with TACo testnet integration (2-of-3 threshold)
Implemented MetaMask Delegation Toolkit smart account creation with balance tracking
Added Pimlico bundler integration for UserOperation execution with comprehensive documentation

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
`demos/taco-aa-signing/src/index.ts`	Main demo implementation showing TACo threshold signatures with ERC-4337
`demos/taco-aa-signing/package.json`	Project dependencies including TACo, viem, and MetaMask delegation toolkit
`demos/taco-aa-signing/README.md`	Complete documentation with setup instructions and example output
`demos/taco-aa-signing/tsconfig.json`	TypeScript configuration extending the root config
`demos/taco-aa-signing/.env.example`	Environment configuration template for RPC, private key, and bundler
`demos/README.md`	Updated main demos index to include the new TACo AA signing demo

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

demos/taco-aa-signing/src/index.ts

codecov-commenter · 2025-08-20T16:34:31Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (signing-epic@7cc3d47). Learn more about missing BASE report.

Additional details and impacted files

@@               Coverage Diff               @@
##             signing-epic     #693   +/-   ##
===============================================
  Coverage                ?   90.14%           
===============================================
  Files                   ?       97           
  Lines                   ?     8463           
  Branches                ?      528           
===============================================
  Hits                    ?     7629           
  Misses                  ?      791           
  Partials                ?       43

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

demos/taco-aa-signing/src/index.ts

demos/taco-aa-signing/package.json

demos/taco-aa-signing/README.md

demos/taco-aa-signing/src/taco-account.ts

demos/README.md

Muhammad-Altabba

Great to see AA in this nice demo 🚀

I had some questions mostly about adding some helper functions to taco-web.

demos/taco-aa-signing/src/taco-account.ts

demos/taco-aa-signing/src/index.ts

demos/taco-aa-signing/package.json

demos/taco-aa-signing/src/index.ts

demos/taco-aa-signing/src/taco-account.ts

- Rename demo folder to be MetaMask Delegation Toolkit specific - Update package name to taco-mdt-aa-signing-demo - Update README title to include "MetaMask Delegation Toolkit" - Add note explaining MDT-specific implementation requirements - Update main demos README link reference Addresses Derek's feedback in PR nucypher#693 about making the demo more specific to MetaMask's Delegation Toolkit implementation.

…gner - Add getCohortMultisigAddress() method to SigningCoordinatorAgent - Calls getSigningCoordinatorChild(chainId) to get child contract - Uses cohortMultisigs(cohortId) mapping to get actual multisig address - Update taco-mdt-aa-signing demo to use cohort multisig address - Replace signers[0] placeholder with proper deployed multisig contract - Add console log showing cohort multisig address being used - Update documentation to reflect multisig vs individual signer usage Addresses Derek's feedback in PR nucypher#693 to use the deployed cohort multisig contract (e.g., 0x857949079dB532301157Eb7Fb5AEC3398043A186) instead of individual signer addresses for ERC-1271 compatibility. Successfully tested - demo retrieves correct multisig and executes transactions.

Muhammad-Altabba

LGTM.
Also it is good to have a follow-up task to check and add possible abstracted code to taco-web, if it was generic enough to be so.

derekpierre · 2025-11-11T15:53:47Z

You will need to rebase this over signing-epic now that #737 is merged.

- Rename demo folder to be MetaMask Delegation Toolkit specific - Update package name to taco-mdt-aa-signing-demo - Update README title to include "MetaMask Delegation Toolkit" - Add note explaining MDT-specific implementation requirements - Update main demos README link reference Addresses Derek's feedback in PR nucypher#693 about making the demo more specific to MetaMask's Delegation Toolkit implementation.

…gner - Add getCohortMultisigAddress() method to SigningCoordinatorAgent - Calls getSigningCoordinatorChild(chainId) to get child contract - Uses cohortMultisigs(cohortId) mapping to get actual multisig address - Update taco-mdt-aa-signing demo to use cohort multisig address - Replace signers[0] placeholder with proper deployed multisig contract - Add console log showing cohort multisig address being used - Update documentation to reflect multisig vs individual signer usage Addresses Derek's feedback in PR nucypher#693 to use the deployed cohort multisig contract (e.g., 0x857949079dB532301157Eb7Fb5AEC3398043A186) instead of individual signer addresses for ERC-1271 compatibility. Successfully tested - demo retrieves correct multisig and executes transactions.

- Rename demo folder to be MetaMask Delegation Toolkit specific - Update package name to taco-mdt-aa-signing-demo - Update README title to include "MetaMask Delegation Toolkit" - Add note explaining MDT-specific implementation requirements - Update main demos README link reference Addresses Derek's feedback in PR nucypher#693 about making the demo more specific to MetaMask's Delegation Toolkit implementation.

…gner - Add getCohortMultisigAddress() method to SigningCoordinatorAgent - Calls getSigningCoordinatorChild(chainId) to get child contract - Uses cohortMultisigs(cohortId) mapping to get actual multisig address - Update taco-mdt-aa-signing demo to use cohort multisig address - Replace signers[0] placeholder with proper deployed multisig contract - Add console log showing cohort multisig address being used - Update documentation to reflect multisig vs individual signer usage Addresses Derek's feedback in PR nucypher#693 to use the deployed cohort multisig contract (e.g., 0x857949079dB532301157Eb7Fb5AEC3398043A186) instead of individual signer addresses for ERC-1271 compatibility. Successfully tested - demo retrieves correct multisig and executes transactions.

demos/taco-mdt-aa-signing/package.json

demos/taco-mdt-aa-signing/src/index.ts

derekpierre · 2025-11-18T20:49:00Z

packages/shared/src/contracts/agents/signing-coordinator.ts

+        // ABI for the cohortMultisigs function
+        'function cohortMultisigs(uint32) view returns (address)',
+      ],
+      provider,


This only works if the child contract is on the same chain as the signing coordinator i.e. only the ETH chain. If the child chain is different the same provider cannot be used, because the provider is used for calling the signing coordinator, and therefore won't be on the same chain.

The best you can do here is return the address of the signing coordinator child contract and not actually make a call on the child contract if it is on a different chain.

Unless a separate provider is passed for the relevant child chain.

Removed the function.

- Rename demo folder to be MetaMask Delegation Toolkit specific - Update package name to taco-mdt-aa-signing-demo - Update README title to include "MetaMask Delegation Toolkit" - Add note explaining MDT-specific implementation requirements - Update main demos README link reference Addresses Derek's feedback in PR nucypher#693 about making the demo more specific to MetaMask's Delegation Toolkit implementation.

…gner - Add getCohortMultisigAddress() method to SigningCoordinatorAgent - Calls getSigningCoordinatorChild(chainId) to get child contract - Uses cohortMultisigs(cohortId) mapping to get actual multisig address - Update taco-mdt-aa-signing demo to use cohort multisig address - Replace signers[0] placeholder with proper deployed multisig contract - Add console log showing cohort multisig address being used - Update documentation to reflect multisig vs individual signer usage Addresses Derek's feedback in PR nucypher#693 to use the deployed cohort multisig contract (e.g., 0x857949079dB532301157Eb7Fb5AEC3398043A186) instead of individual signer addresses for ERC-1271 compatibility. Successfully tested - demo retrieves correct multisig and executes transactions.

- Rename demo folder to be MetaMask Delegation Toolkit specific - Update package name to taco-mdt-aa-signing-demo - Update README title to include "MetaMask Delegation Toolkit" - Add note explaining MDT-specific implementation requirements - Update main demos README link reference Addresses Derek's feedback in PR nucypher#693 about making the demo more specific to MetaMask's Delegation Toolkit implementation.

…gner - Add getCohortMultisigAddress() method to SigningCoordinatorAgent - Calls getSigningCoordinatorChild(chainId) to get child contract - Uses cohortMultisigs(cohortId) mapping to get actual multisig address - Update taco-mdt-aa-signing demo to use cohort multisig address - Replace signers[0] placeholder with proper deployed multisig contract - Add console log showing cohort multisig address being used - Update documentation to reflect multisig vs individual signer usage Addresses Derek's feedback in PR nucypher#693 to use the deployed cohort multisig contract (e.g., 0x857949079dB532301157Eb7Fb5AEC3398043A186) instead of individual signer addresses for ERC-1271 compatibility. Successfully tested - demo retrieves correct multisig and executes transactions.

Reabsed and added a commit

derekpierre · 2025-11-20T17:36:56Z

@theref I rebased this PR over signing-epic and used the latest alpha.5 releases. Also made some tweaks to the logic.

Add comprehensive demo showing how to integrate TACo's distributed threshold signatures with Account Abstraction wallets using MetaMask Delegation Toolkit. Features: - Single-file implementation - Real TACo testnet integration with 2-of-3 threshold - Balance tracking and fund management - ERC-4337 UserOperation signing with Pimlico bundler - Clean documentation with example output

…uality - Replace ts-node with tsx for better performance and compatibility - Add --dry-run flag support for CI syntax checking - Relax TypeScript strict mode for third-party type incompatibilities - Add explicit TypeScript type annotations for better type safety - Apply Prettier formatting to improve code consistency - Fix type casting for viem/account-abstraction compatibility - Update dependencies in pnpm-lock.yaml

- Add createViemTacoAccount utility to wrap TACo signers as Viem accounts - Replace localAccount parameter with TACo-wrapped account in smart account creation - Update function signature to eliminate private key requirement for signatory - Improve type safety by replacing explicit 'any' types where possible - Add ESLint exemptions for necessary type casts due to library incompatibilities Resolves nucypher#699: AA demo now works without local wallet, making it AI agent ready

Co-authored-by: Derek Pierre <derek.pierre@gmail.com> Co-authored-by: Muhammad Altabba <24407834+Muhammad-Altabba@users.noreply.github.com>

- Rename demo folder to be MetaMask Delegation Toolkit specific - Update package name to taco-mdt-aa-signing-demo - Update README title to include "MetaMask Delegation Toolkit" - Add note explaining MDT-specific implementation requirements - Update main demos README link reference Addresses Derek's feedback in PR nucypher#693 about making the demo more specific to MetaMask's Delegation Toolkit implementation.

…gner - Add getCohortMultisigAddress() method to SigningCoordinatorAgent - Calls getSigningCoordinatorChild(chainId) to get child contract - Uses cohortMultisigs(cohortId) mapping to get actual multisig address - Update taco-mdt-aa-signing demo to use cohort multisig address - Replace signers[0] placeholder with proper deployed multisig contract - Add console log showing cohort multisig address being used - Update documentation to reflect multisig vs individual signer usage Addresses Derek's feedback in PR nucypher#693 to use the deployed cohort multisig contract (e.g., 0x857949079dB532301157Eb7Fb5AEC3398043A186) instead of individual signer addresses for ERC-1271 compatibility. Successfully tested - demo retrieves correct multisig and executes transactions.

Update to use the new SigningCohortParticipant.signerAddress property instead of the deprecated operator property.

…libraries.

theref requested review from Muhammad-Altabba, Copilot and derekpierre August 19, 2025 15:49

Copilot AI reviewed Aug 19, 2025

View reviewed changes

demos/taco-aa-signing/src/index.ts Outdated Show resolved Hide resolved

demos/taco-aa-signing/src/index.ts Outdated Show resolved Hide resolved

demos/taco-aa-signing/src/index.ts Outdated Show resolved Hide resolved

demos/taco-aa-signing/src/index.ts Outdated Show resolved Hide resolved

theref force-pushed the sign-example branch 2 times, most recently from ab18a58 to a7a58f7 Compare August 19, 2025 16:53

theref force-pushed the sign-example branch 2 times, most recently from 3f6b92a to c874f8d Compare August 22, 2025 13:21

cygnusv reviewed Aug 25, 2025

View reviewed changes

demos/taco-aa-signing/src/index.ts Show resolved Hide resolved

theref mentioned this pull request Sep 5, 2025

Remove local wallet dependency for AA demo using Viem wrapping #699

Open

4 tasks

derekpierre reviewed Sep 5, 2025

View reviewed changes

Muhammad-Altabba reviewed Sep 8, 2025

View reviewed changes

theref force-pushed the sign-example branch from a0519f0 to 0004216 Compare September 10, 2025 13:49

Muhammad-Altabba approved these changes Sep 15, 2025

View reviewed changes

derekpierre mentioned this pull request Oct 14, 2025

[EPIC][WIP] Signing Epic #674

Draft

27 tasks

derekpierre assigned theref Oct 14, 2025

theref force-pushed the sign-example branch from 0004216 to 2c31ae6 Compare November 12, 2025 11:20

theref force-pushed the sign-example branch from f952a9e to 1e1aabd Compare November 12, 2025 14:17

derekpierre reviewed Nov 18, 2025

View reviewed changes

demos/taco-mdt-aa-signing/package.json Outdated Show resolved Hide resolved

demos/taco-mdt-aa-signing/src/index.ts Outdated Show resolved Hide resolved

derekpierre previously requested changes Nov 18, 2025

View reviewed changes

derekpierre force-pushed the sign-example branch from f93d58a to 66b7ab0 Compare November 20, 2025 17:35

derekpierre requested review from KPrasch, Muhammad-Altabba, cygnusv, manumonti and vzotova November 26, 2025 15:11

theref and others added 9 commits November 26, 2025 10:12

Apply suggestions from code review

5363470

Co-authored-by: Derek Pierre <derek.pierre@gmail.com> Co-authored-by: Muhammad Altabba <24407834+Muhammad-Altabba@users.noreply.github.com>

fix(demos): use signerAddress instead of operator in taco-mdt-aa-signing

beb2bae

Update to use the new SigningCohortParticipant.signerAddress property instead of the deprecated operator property.

Make updates based on latest npm dev releases of taco and shared …

19f2d03

…libraries.

Linting clean up.

1efc8d1

derekpierre force-pushed the sign-example branch from 66b7ab0 to 1efc8d1 Compare November 26, 2025 15:15

Add TACo Account Abstraction signing demo #693

Are you sure you want to change the base?

Add TACo Account Abstraction signing demo #693

Conversation

theref commented Aug 19, 2025

Summary

Demo Features

Test Plan

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

codecov-commenter commented Aug 20, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Muhammad-Altabba left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Muhammad-Altabba left a comment

Choose a reason for hiding this comment

Uh oh!

derekpierre commented Nov 11, 2025

Uh oh!

Uh oh!

Uh oh!

derekpierre Nov 18, 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!

derekpierre Nov 20, 2025

Choose a reason for hiding this comment

Uh oh!

derekpierre commented Nov 20, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

codecov-commenter commented Aug 20, 2025 •

edited

Loading

derekpierre Nov 18, 2025 •

edited

Loading