Skip to content

[Beta] EIP7702 Unmanaged/Managed Execution for In-App & Ecosystem Wallets, turn your EOAs into Smart EOAs with a simple flag. #138

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 18 commits into from
May 7, 2025

Conversation

0xFirekeeper
Copy link
Member

@0xFirekeeper 0xFirekeeper commented Mar 14, 2025

Attempt to create an ultimate 7702 account, with the best dx possible.

Closes TOOL-3195


PR-Codex overview

This PR focuses on enhancing the Thirdweb library by introducing new features and improvements for transaction handling, particularly around EIP-7702, session key management, and wallet execution modes.

Detailed summary

  • Updated YParity logic in ThirdwebTransactionInput.cs.
  • Replaced SessionKeyParams_7702 with SessionSpec in EIP712.cs.
  • Added TwExecute and TwGetTransactionHash methods in BundlerClient.cs.
  • Modified signature generation in PrivateKeyWallet.cs.
  • Enhanced transaction encoding in EIP712Encoder.cs.
  • Introduced ExecutionMode enum in EcosystemWallet.cs.
  • Updated Create method in InAppWallet to include executionMode.
  • Added WaitForTransactionReceipt method in Utils.cs.
  • Implemented new classes for TwExecuteResponse and TwGetTransactionHashResponse.
  • Improved error handling and transaction management in SendTransaction and ExecuteTransaction methods.

The following files were skipped due to too many changes: Thirdweb/Thirdweb.Utils/Constants.cs

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

Test Only
Attempt to create an ultimate 7702 account, with the best dx possible.
Eventually will have an implied executor, improving creation DX.

Closes TOOL-3195
Copy link

linear bot commented Mar 14, 2025

@codecov-commenter
Copy link

codecov-commenter commented Mar 14, 2025

Codecov Report

Attention: Patch coverage is 38.22630% with 202 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@108020e). Learn more about missing BASE report.
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
Thirdweb/Thirdweb.Wallets/EIP712Encoder.cs 60.20% 75 Missing and 3 partials ⚠️
Thirdweb/Thirdweb.Utils/Utils.cs 0.00% 47 Missing ⚠️
...SmartWallet/Thirdweb.AccountAbstraction/AATypes.cs 0.00% 39 Missing ⚠️
...allet/Thirdweb.AccountAbstraction/BundlerClient.cs 0.00% 28 Missing ⚠️
Thirdweb/Thirdweb.Wallets/EIP712.cs 0.00% 9 Missing ⚠️
...rdweb.Wallets/PrivateKeyWallet/PrivateKeyWallet.cs 85.71% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #138   +/-   ##
=======================================
  Coverage        ?   62.23%           
=======================================
  Files           ?       41           
  Lines           ?     6427           
  Branches        ?      822           
=======================================
  Hits            ?     4000           
  Misses          ?     2240           
  Partials        ?      187           

☔ 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.

@0xFirekeeper 0xFirekeeper changed the title [Beta] ThirdwebWallet - An All Encompassing Wallet [Beta] SmarterWallet - Improved SmartWallet powered purely by 7702 and thirdweb managed execution May 7, 2025
@0xFirekeeper 0xFirekeeper changed the title [Beta] SmarterWallet - Improved SmartWallet powered purely by 7702 and thirdweb managed execution [Beta] EIP7702 Unmanaged/Managed Execution for In-App & Ecosystem Wallets, turn your EOAs into Smart EOAs with a simple flag. May 7, 2025
@0xFirekeeper 0xFirekeeper merged commit 4ffdab4 into main May 7, 2025
3 checks passed
@0xFirekeeper 0xFirekeeper deleted the firekeeper/7702-wallet branch May 7, 2025 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants