bcnmy / biconomy-client-sdk

Biconomy SDK is a plug & play toolkit for dApps to build transaction legos that enable a highly customised one-click experience for their users
MIT License
73 stars 76 forks source link

chore: refactor codebase #574

Closed joepegler closed 2 hours ago

joepegler commented 5 days ago

PR-Codex overview

This PR focuses on restructuring the project directory, updating imports, and improving file organization for better readability and maintainability.

Detailed summary

The following files were skipped due to too many changes: src/bundler/utils/getAAError.ts, src/account/utils/getChain.ts, src/sdk/modules/validators/ValidationModule.ts, src/sdk/modules/validators/K1ValidatorModule.ts, package.json, tests/src/testSetup.ts, CHANGELOG.md, src/sdk/clients/createBicoBundlerClient.ts, src/sdk/clients/decorators/smartAccount/signMessage.ts, src/sdk/account/utils/toHolder.ts, src/sdk/clients/decorators/erc7579/getActiveHook.ts, src/modules/base/BaseModule.ts, scripts/send:userOp.ts, src/sdk/clients/createBicoPaymasterClient.ts, src/sdk/account/utils/utils.test.ts, scripts/fetch:deployment.ts, src/sdk/clients/decorators/erc7579/accountId.ts, src/sdk/account/utils/Types.ts, src/sdk/clients/decorators/erc7579/getFallbackBySelector.ts, src/modules/base/BaseValidationModule.ts, src/sdk/clients/decorators/erc7579/installModule.ts, src/sdk/clients/decorators/smartAccount/writeContract.ts, src/sdk/clients/decorators/erc7579/getInstalledExecutors.ts, src/sdk/clients/decorators/erc7579/getInstalledValidators.ts, src/sdk/clients/decorators/erc7579/uninstallModule.ts, src/sdk/clients/decorators/erc7579/uninstallFallback.ts, src/sdk/clients/decorators/erc7579/uninstallModules.ts, src/sdk/clients/decorators/smartAccount/sendTransaction.ts, src/sdk/clients/createBicoBundlerClient.test.ts, src/sdk/clients/decorators/erc7579/installModules.ts, src/sdk/clients/decorators/smartAccount/signTypedData.ts, src/account/utils/Constants.ts, src/sdk/clients/decorators/erc7579/isModuleInstalled.ts, src/sdk/modules/validators/k1Validator.test.ts, src/sdk/clients/decorators/erc7579/erc7579.decorators.test.ts, src/sdk/clients/decorators/erc7579/supportsModule.ts, src/account/utils/Utils.ts, scripts/viem:bundler.ts, tests/smart.sessions.test.ts, src/sdk/clients/decorators/smartAccount/account.decorators.test.ts, src/sdk/clients/decorators/erc7579/supportsExecutionMode.ts, src/sdk/clients/decorators/erc7579/index.ts, src/modules/executors/OwnableExecutor.ts, src/modules/utils/Types.ts, tests/playground.test.ts, src/sdk/clients/createNexusClient.ts, src/sdk/clients/createNexusClient.test.ts, src/sdk/account/toNexusAccount.test.ts, tests/src/testUtils.ts, src/sdk/clients/decorators/smartAccount/index.ts, src/sdk/account/toNexusAccount.ts

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

joepegler commented 1 day ago

Size limit is failing just because of the base branch. This PR is ready to be merged despite that failing test

github-actions[bot] commented 2 hours ago

size-limit report πŸ“¦

Path Size
core (esm) 11.11 KB (-33.82% πŸ”½)
core (cjs) 17.14 KB (-22.07% πŸ”½)
bundler (tree-shaking) 197 B (-95.93% πŸ”½)
paymaster (tree-shaking) 108 B (-95.64% πŸ”½)