fix: isFlushingEvents flag can get stuck true if publishEvents throws

[jonathannorris]

Wraps the publish loop in a try/finally so isFlushingEvents is always reset to false, even if publishEvents() throws InterruptedException.

Without this, any exception escaping the loop leaves the flag permanently true and all subsequent flushEvents() calls return early, causing events to accumulate and never be sent.

Fixes #192

[Copilot]

Pull request overview

Ensures the local event flushing logic can’t get permanently “locked” by resetting isFlushingEvents via try/finally even when publishEvents() throws, preventing future flushEvents() calls from returning early and leaving events unsent.

Changes:

• Wraps the event publish loop in flushEvents() in a try/finally.
• Guarantees isFlushingEvents is always reset to false after a flush attempt, even on exception.

---

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