Mrixs/trip-planner
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
48 Commits 4 Branches 0 Tags
6e65e6161c9feabcc7850e8deae66d33b9c32161
Commit Graph

11 Commits

Author SHA1 Message Date
Vladimir Zagainov
0d603ad15f fix: address code review findings 2026-08-18 14:25:34 +03:00
Vladimir Zagainov
8aecaf1468 fix: address code review findings 2026-08-17 23:40:52 +03:00
Vladimir Zagainov
6dcec6a7a5 fix: address code review findings 
- Fix type mismatch in search_cache.go: SearchWithCache now returns *yandex.Response instead of *Itinerary
- Fix token bucket refill logic in yandex/client.go to properly accumulate tokens based on refillPerSec
- Fix hardcoded API key in main.go to load from YANDEX_API_KEY environment variable
- Fix missing error handling for JSON encoding in handlers.go RouteGeoJSON function
- Fix incorrect redis.Nil handling in cache/store.go CacheAside.Exists method
- Fix incorrect error return type in station_status.go updateStationStatus to return newStatus instead of empty string
 2026-08-17 22:42:08 +03:00
Vladimir Zagainov
78f662c985 fix: address code review findings 2026-08-17 22:14:48 +03:00
Vladimir Zagainov
777fda95a3 fix: address code review findings 2026-08-17 21:50:37 +03:00
Vladimir Zagainov
9c402c0086 fix: address code review findings 2026-08-17 21:19:36 +03:00
Vladimir Zagainov
82757665e9 fix: address code review findings 2026-08-17 20:34:17 +03:00
Vladimir Zagainov
c58fbb50b1 feat: implement observability and metrics (Task 22) 
- Add metrics package tracking cache hit-rate, API quota, circuit breaker trips, search time
- Integrate metrics with cache layer, yandex client, and API handlers
- Add /metrics HTTP endpoint for Prometheus-compatible metrics exposure
- Write tests for metrics functionality across cache, yandex, and API handlers
- Update test files to support new metrics infrastructure
 2026-08-17 15:27:44 +03:00
Vladimir Zagainov
6ae491ef1c feat: Implement basic caching layer with cache-aside pattern for /search results 
- Implement cache-aside pattern via SearchCacheService in RouteSearch handler
- Add TTL policies: 3 hours near-term, 7 days far-term
- Write TestCacheAsideSearch and variants for TTL verification
- Extend CacheAside to fully implement Cache interface (Get, Set, Exists, Delete, Increment, Decrement)
 2026-08-16 14:53:33 +03:00
Vladimir Zagainov
e063d26d4c fix: code correctness, security, and simplicity improvements 2026-08-13 21:23:15 +03:00
Vladimir Zagainov
571d11d376 feat: implement cache-aside layer with Redis cache interface, cache keys, TTL policies, and tests 
- Created cache/store.go with Cache interface, redisClient wrapper, cacheStore, TTL constants (CityTTL 30d, SearchNearTermTTL 3h, SearchFarTermTTL 7d)
- Implemented cache keys: cities:{code}, stations:{id}, search:{from}:{to}:{date}
- Implemented cache-aside pattern via CacheAside struct with GetOrSetFuncPattern, GetCity, GetStation, GetSearch
- Added TTL-aware search result caching with near-term (3h) and far-term (7d) policies
- Wrote 6 unit tests: CacheGetSet, CacheKeyString, CacheAsideGetOrSet, CacheAsideGetCity, CacheAsideGetSearch, CacheInvalidate
- All tests pass with Redis integration
 2026-08-13 19:55:02 +03:00