fix: selector reference warnings

[iamacook]

What it solves

Resolves console warning/unnecessary re-renders

How this PR fixes it

If a selector returns a new reference, it should be memoised either manually or inside createSelector, etc. The following changes solved the console warning:

• selectAllBatches no longer returns an empty array but the initialState.
• selectHiddenTokensPerChain now leverages createSelector as it can return an empty array.

How to test it

• Run the project locally and observe no console warnings.
• Batching should work as expected.
• Hiding/showing spam tokens should work as expected.

Screenshots

Console warning in question:

Checklist

• I've tested the branch on mobile 📱
• I've documented how it affects the analytics (if at all) 📊
• I've written a unit/e2e test for it (if applicable) 🧑‍💻

[github-actions]

Branch preview

✅ Deploy successful!

https://fix_selectors--walletweb.review-wallet-web.5afe.dev

[github-actions]

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type	Occurrences	Fixable
Errors	0	0
Warnings	0	0
Ignored	0	N/A

• Result: ✅ success
• Annotations: 0 total

---

^{Report generated by eslint-plus-action}

[katspaugh]

👍

[francovenica]

Looks good to me

I don't see the warning shown in the ticket description

Batch and hidding tokens works fine as well