feat(v5): decode calls for LA and MAv2

[adamegyed]

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 focuses on enhancing the encoding and decoding functionality for transactions in the smart-accounts package, particularly for light accounts and modular accounts. It introduces new utility functions and modifies existing ones to support better handling of calls.

Detailed summary

• Added LightAccountAbi type to define account ABI based on type and version.
• Introduced encodeCallsLA and decodeCallsLA functions for handling light account calls.
• Refactored encodeCalls and decodeCalls methods in base.ts to utilize new utilities.
• Enhanced error handling for self-calls in encodeCalls for modular accounts.
• Updated tests to validate encoding and decoding for both single and batch transactions.

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

[vercel]

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

1 Skipped Deployment

Project	Deployment	Preview	Comments	Updated (UTC)
aa-sdk-ui-demo	Ignored	Preview		Nov 18, 2025 10:21pm

[github-actions]

🌿 Documentation Preview

Name	Status	Preview	Updated (UTC)
Alchemy Docs	✅ Ready	🔗 Visit Preview	Nov 18, 2025, 10:22 PM

[Copilot]

Pull Request Overview

This PR adds encoding and decoding functionality for transaction calls in both Light Account (LA) and Modular Account v2 (MAv2) implementations. The changes enable proper serialization and deserialization of single and batch transaction data.

Key Changes

• Introduced decodeCalls method for both LA and MAv2 accounts to reverse the encoding process
• Refactored LA encoding logic into separate utility functions (encodeCallsLA and decodeCallsLA)
• Added special handling for MAv2 self-calls to avoid double-wrapping
• Added comprehensive tests for encoding/decoding single and batch transactions

Reviewed Changes

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

Show a summary per file

File	Description
packages/smart-accounts/src/ma-v2/accounts/base.ts	Added decodeCalls method and self-call handling in encodeCalls
packages/smart-accounts/src/ma-v2/accounts/account.test.ts	Added tests for encoding/decoding single and batch transactions
packages/smart-accounts/src/light-account/registry.ts	Added LightAccountAbi type helper for ABI type inference
packages/smart-accounts/src/light-account/accounts/calldataCodec.ts	New file with LA-specific encoding/decoding utilities
packages/smart-accounts/src/light-account/accounts/base.ts	Refactored to use new encoding/decoding utility functions
packages/smart-accounts/src/light-account/accounts/account.test.ts	Added tests for encoding/decoding single and batch transactions
docs-site	Updated subproject commit reference

[graphite-app]

Incomplete test assertion. The line expect(decoded[0].value); is checking for truthiness but lacks a matcher (like .toBe(0n) or .toBeDefined()). This test will always pass regardless of the actual value. Based on the encoding logic in calldataCodec.ts line 17 which always provides a value (calls[0].value ?? 0n), and the decoding logic on line 50 which includes the value when defined, this should be:

expect(decoded[0].value).toBe(0n);

Suggested change

	expect(decoded[0].value);
	expect(decoded[0].value).toBe(0n);

Spotted by Graphite Agent



Is this helpful? React 👍 or 👎 to let us know.