7 Commits

This Branch

This Branch

All Branches

Author	SHA1	Message	Date
Claude Code	5223cf136a	fix: address review feedback ... - Removed duplicate blank line in event.go - Use original event timestamp instead of time.Now() for EventStored - Fixed MockEventBroadcaster.Subscribe to return nil instead of closed channel - Added integration tests for EventStored with JetStreamEventStore Co-Authored-By: Claude Code <noreply@anthropic.com>	2026-01-13 22:25:10 +01:00
Claude Code	0f89b07c0b	feat(event sourcing): Publish EventStored event after successful SaveEvent ... Implement EventStored infrastructure event that notifies subscribers when an event is successfully persisted. This enables observability and triggers downstream workflows (caching, metrics, projections) without coupling to application events. Changes: - Add EventStored type to event.go containing EventID, ActorID, Version, Timestamp - Update InMemoryEventStore with optional EventBus and metrics support via builder methods - Update JetStreamEventStore with optional EventBus and metrics support via builder methods - Publish EventStored to __internal__ namespace after successful SaveEvent - EventStored not published if SaveEvent fails (e.g., version conflict) - EventStored publishing is optional - stores work without EventBus configured - Metrics are recorded for each EventStored publication - Add comprehensive test suite covering all acceptance criteria Meets acceptance criteria: - EventStored published after SaveEvent succeeds - EventStored contains EventID, ActorID, Version, Timestamp - No EventStored published if SaveEvent fails - EventBus receives EventStored in same operation - Metrics increment for each EventStored published Closes #61 Co-Authored-By: Claude Code <noreply@anthropic.com>	2026-01-13 21:25:51 +01:00
Claude Code	de30e1ef1b	fix: address critical TOCTOU race condition and error handling inconsistencies ... - Fix TOCTOU race condition in SaveEvent by holding the lock throughout entire version validation and publish operation - Add getLatestVersionLocked helper method to prevent race window where multiple concurrent threads read the same currentVersion - Fix GetLatestSnapshot to return error when no snapshot exists (not nil), distinguishing "not created" from "error occurred" - The concurrent version conflict test now passes with exactly 1 success and 49 conflicts instead of 50 successes These changes ensure thread-safe optimistic concurrency control and consistent error handling semantics. Co-Authored-By: Claude Code <noreply@anthropic.com>	2026-01-13 19:49:37 +01:00
Claude Code	b9e641c2aa	fix: Address thread safety and resource management issues ... - Fix thread safety issue in SaveEvent: Lock now only protects cache access. NATS I/O operations (GetLatestVersion calls) happen without holding the mutex, preventing lock contention when multiple concurrent SaveEvent calls occur. - Improve cache handling: Check cache first with minimal lock hold time. For cache misses, unlock before calling GetLatestVersion, then re-lock only to update cache. - Remove getLatestVersionLocked: No longer needed now that SaveEvent doesn't hold lock during GetLatestVersion calls. - Fix error handling consistency: GetLatestSnapshot now returns (nil, nil) when no snapshot exists, consistent with GetLatestVersion returning 0 for no events. Both methods now treat empty results as normal cases rather than errors. - Fix benchmark test: BenchmarkGetLatestVersion_NoCache now creates uncachedStore outside the timing loop. Previously, creating a new store on each iteration was too expensive and didn't properly measure GetLatestVersion performance. Co-Authored-By: Claude Code <noreply@anthropic.com>	2026-01-13 19:49:37 +01:00
Claude Code	ec3db5668f	perf: Optimize GetLatestVersion to O(1) using JetStream DeliverLast ... Closes #127 The GetLatestVersion method previously fetched all events for an actor to find the maximum version, resulting in O(n) performance. This implementation replaces the full scan with JetStream's DeliverLast() consumer option, which efficiently retrieves only the last message without scanning all events. Performance improvements: - Uncached lookups: ~1.4ms regardless of event count (constant time) - Cached lookups: ~630ns (very fast in-memory access) - Memory usage: Same 557KB allocated regardless of event count - Works correctly with cache invalidation The change is backward compatible: - Cache in getLatestVersionLocked continues to provide O(1) performance - SaveEvent remains correct with version conflict detection - All existing tests pass without modification - Benchmark tests verify O(1) behavior Co-Authored-By: Claude Code <noreply@anthropic.com>	2026-01-13 19:49:37 +01:00
Claude Code	fd1938672e	fix: address review feedback on cache invalidation ... - Fix cache not repopulated after invalidation: Always update cache with fresh data instead of just deleting on mismatch - Fix race condition: Hold mutex lock during entire fetch operation to prevent SaveEvent from running between fetch and cache update - Improve test: Add second GetLatestVersion call to verify cache was properly repopulated after invalidation Co-Authored-By: Claude Code <noreply@anthropic.com>	2026-01-13 01:31:03 +01:00
Claude Code	6de897ef60	fix(store): Implement version cache invalidation strategy for JetStreamEventStore ... Implements cache invalidation on GetLatestVersion when external writers modify the JetStream stream. The strategy ensures consistency in multi-store scenarios while maintaining performance for the single-writer case. Changes: - Add cache invalidation logic to GetLatestVersion() that detects stale cache - Document version cache behavior in JetStreamEventStore struct comment - Add detailed documentation in CLAUDE.md about cache invalidation strategy - Add TestJetStreamEventStore_CacheInvalidationOnExternalWrite integration test - Cache is invalidated by deleting entry, forcing fresh fetch on next check The implementation follows the acceptance criteria by: 1. Documenting the single-writer assumption in code comments 2. Implementing cache invalidation on GetLatestVersion miss 3. Adding comprehensive test for external write scenarios Closes #126 Co-Authored-By: Claude Code <noreply@anthropic.com>	2026-01-13 00:24:27 +01:00