less store 13

[chrisnojima-zoom]

No description provided.

[Copilot]

Pull request overview

This PR continues the “zustand-store-pruning” effort by removing the wallets zustand store and migrating a couple of users-store RPC helpers into the feature/components that invoke them, while updating the pruning checklist documentation accordingly.

Changes:

• Removed the wallets zustand store and its unit tests; wallets screens now load/delete accounts via direct RPC calls and pass required data via route params.
• Removed reportUser / setUserBlocks dispatch helpers from the users store and updated call sites to invoke the underlying RPCs directly.
• Updated the store-pruning checklist statuses/notes for team-building, tracker, users, and wallets.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
skill/zustand-store-pruning/references/store-checklist.md	Updates pruning progress notes/statuses for several stores.
shared/wallets/index.tsx	Loads wallet accounts via direct RPC into component state; passes needed params to modals.
shared/wallets/remove-account.tsx	Removes wallets-store dependency; uses route params for name/balance display.
shared/wallets/really-remove-account.tsx	Removes wallets-store dependency; deletes account via direct RPC and uses route params.
shared/stores/wallets.tsx	Deletes the wallets zustand store implementation.
shared/stores/tests/wallets.test.ts	Deletes wallets-store unit tests.
shared/stores/users.tsx	Removes reportUser and setUserBlocks dispatch helpers from the users store.
shared/chat/blocking/block-modal.tsx	Calls report/block RPCs directly instead of via users-store dispatch helpers.
shared/teams/team/rows/member-row.tsx	Uses direct userSetUserBlocks RPC instead of users-store dispatch.
shared/teams/channel/rows.tsx	Uses direct userSetUserBlocks RPC instead of users-store dispatch.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Account duplicates fields that already exist on the RPC type (T.RPCStellar.WalletAccountLocal). To avoid drift when the protocol type changes, consider defining this as a Pick (or alias) of the generated RPC type instead of re-declaring the shape here.

Suggested change

	type Account = {
	accountID: string
	balanceDescription: string
	deviceReadOnly: boolean
	isDefault: boolean
	name: string
	}
	type Account = Pick<
	T.RPCStellar.WalletAccountLocal,
	'accountID' | 'balanceDescription' | 'deviceReadOnly' | 'isDefault' | 'name'
	>

[Copilot]

The delete-account RPC error callback is currently a no-op, so failures will be silent and the user won’t know why the account wasn’t removed. Consider handling the error (e.g., show a banner/toast and/or log) so this flow is diagnosable.

Suggested change

	}, () => {})
	}, err => {
	console.error('Error deleting wallet account', err)
	})