Skip to content

fix: add batchUpdates option to WalletStatusTracker and verify transport cleanup - #820

Merged
Just-Bamford merged 1 commit into
Sorokit:mainfrom
opascal221-design:fix/issues-761-748-760-749-wallet-status-batching
Oct 17, 2026
Merged

Just-Bamford merged 1 commit into
Sorokit:mainfrom
opascal221-design:fix/issues-761-748-760-749-wallet-status-batching

Conversation

@opascal221-design

Copy link
Copy Markdown
Contributor

Closes #761
Closes #748
Closes #760
Closes #749

Summary

Changes

  • src/wallet/walletStatusTracker.ts — added batchUpdates config option, _flushPending flag, microtask-based coalescing in _emit(), and _flushPending reset in destroy()
  • src/tests/walletStatusTracker.test.ts — three new tests: coalescing, synchronous fallback, and post-destroy safety
  • src/tests/logger.test.ts — one new test confirming transport removal on unregister

@drips-wave

drips-wave Bot commented Oct 17, 2026

Copy link
Copy Markdown

@opascal221-design Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Just-Bamford
Just-Bamford merged commit 266173d into Sorokit:main Oct 17, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment