Remove client-side migration and reorganize wallets #2572
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.
Description
supersedes #2546 #2552 #2553 #2554
see description in individual PRs + commit messages
Additional context
For me personally, it is definitely better to just have one big PR because I know what I did and then I can resolve conflicts between commits immediately. If I want to use the web interface to review and document my changes, I can also push to my own repository and do it there.
The small PRs were mostly meant to make review easier, but since that is apparently not really the case, I now squashed every individual PR into one commit that I added here.
I hope that is better.
Checklist
Are your changes backward compatible? Please answer below:
yes
On a scale of 1-10 how well and how have you QA'd this change and any features it might affect? Please answer below:
tested PRs individually before
For frontend changes: Tested on mobile, light and dark mode? Please answer below:
n/a
Did you introduce any new environment variables? If so, call them out explicitly here:
no
Did you use AI for this? If so, how much did it assist you?
no
Note
Refactors wallet architecture to a hook-based provider, extracts crypto utilities, removes v1 migration and old storage, and updates imports/components accordingly.
hooks/global
withWalletsProvider
, state/actions, and always-on hooks (useServerWallets
, automated retries, key init, old storage cleanup).wallets/client/context/*
and v1→v2 migration code.wallets/lib/crypto
(deriveKey
,encrypt
,decrypt
,generateRandomPassphrase
).hooks/passphrase
.localStorage
wallets.hooks/query
to use new global/crypto/logger hooks; drop migration mutation; minor import/logic cleanup.indicator
,wallet
,payment
,diagnostics
) to new paths.components/draggable
tocomponents/dnd
and re-export viacomponents/index
.components/dnd.js
to useuseDndHandlers
from new hooks.pages/_app.js
andpages/wallets/index.js
to usewallets/client/hooks
and new APIs.Written by Cursor Bugbot for commit 983a638. This will update automatically on new commits. Configure here.