feat: implement wa-sqlite for client-side offline storage #173
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add wa-sqlite with IDBBatchAtomicVFS for IndexedDB-backed persistence - Create clientDatabase.js with serialized operation queue pattern - Create walletsStore.js for wallet data management - Migrate all localStorage usage to SQLite: - NetworkContext settings - PasskeyStorage credentials - WalletDataCache - DeviceManager - SlippageSelector/DeadlineSelector preferences - Fix token icon rendering (handle all string formats) - Reduce swap gas reserve from 0.01 to 0.001 ETH - Add vite-plugin-wasm and vite-plugin-top-level-await for WASM support Closes #171
hadv
force-pushed
the
feature/wa-sqlite-offline-storage
branch
from
5137500 to
17eddc6
Compare
The test was making real network calls which timed out in CI environment. Now uses vi.spyOn to mock the method and avoid actual RPC calls.
- Update PasskeySettings.jsx to use useModularAccountManager hook - Update DeviceManagement.jsx to use modular account for passkey data - Update AddCurrentDevice.jsx and AddMobileDevice.jsx for modular accounts - Replace twoFactorEnabled with mfaEnabled terminology - Adapt passkey data format from P256MFAValidatorModule structure - Remove dependency on legacy useP256SDK in device management components Note: On-chain passkey addition/removal and MFA toggle via UserOperation are not yet implemented for modular accounts. Coming in a future update.
- Create new ModularAccountSDK class for modular account UserOp building - Add executeOwnerOnly, addPasskey, removePasskey, enableMFA, disableMFA methods - Add ERC-7579 mode encoding helpers (encodeSimpleSingleMode, encodeSingleExecution) - Add useModularAccountSDK React hook Update components to use ModularAccountSDK: - AddCurrentDevice: on-chain passkey addition via modularSDK.addPasskey() - AddMobileDevice: on-chain passkey addition for mobile devices - DeviceManagement: on-chain passkey removal via modularSDK.removePasskey() - PasskeySettings: MFA toggle via modularSDK.enableMFA/disableMFA() The P256AccountSDK remains unchanged for legacy P256Account operations.
hadv
force-pushed
the
feature/wa-sqlite-offline-storage
branch
from
9a373aa to
ab81755
Compare
- Fix ModularAccountManager returning factory address by using raw eth_call - Remove isModular flags and conditional rendering from components - Fix ReferenceError in SwapScreen (sdk vs modularSDK) - Fix ReferenceError in RecoveryManager (missing component) - Update WalletDetailScreen to use modularSDK
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR implements wa-sqlite for client-side offline storage, replacing all localStorage usage with IndexedDB-backed SQLite database.
Changes
Core Database Implementation
IDBBatchAtomicVFSfor IndexedDB-backed persistencelocalStorage → SQLite Migration
Bug Fixes
/)Build Configuration
vite-plugin-wasmandvite-plugin-top-level-awaitfor WASM supportTesting
Closes #171
Pull Request opened by Augment Code with guidance from the PR author