full-implementation #3

Mrixs · 2026-08-19T12:07:47Z

Mrixs commented

2026-08-19 12:07:47 +00:00

No description provided.

Mrixs added 47 commits 2026-08-19 12:07:48 +00:00

add plan: full-implementation c92baeca02

feat: implement synthetic edge fallback in FindRoute 26c4be2d3a

- Add addSyntheticEdgesForNode function that connects nodes to city hubs
- Modify FindRoute to add synthetic edges as fallback when no route found
- Write TestFindRouteWithSyntheticFallback test

Co-Authored-By: Claude <noreply@anthropic.com>

feat: Add ResetCircuitBreaker helper 286cb8653a

feat: Implement on-demand /search integration in lazy graph expansion 4de50f4948

- Integrate on-demand Yandex /search calls in FindRoute when lazy expansion + synthetic fallback fails
- Add real route segments from API response as edges, then retry BFS search
- Integrate existing cache key generation and TTL policies (SearchNearTermTTL: 3h, SearchFarTermTTL: 7d)
- Write tests: TestSearchRoutes_onDemand, TestSearchRoutes_onDemandVerifyIntegration, TestFindRouteWithSyntheticFallback
- All tests pass before task 5 (transfer depth limiting)

feat: Implement Pareto-front ranking with multi-criteria sorting and TestRouteParetoRanking test 7adb2ebbb3

- Add Cost field to Edge and RouteLeg structs
- Propagate Cost in FindRoute itinerary creation
- Write TestRouteParetoRanking test verifying non-dominated route selection

feat: Implement basic caching layer with cache-aside pattern for /search results 6ae491ef1c

- 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)

feat: Implement Station Status endpoint (Task 8) 3da7f5282c

- Implement GET /v1/stations/{id}/status endpoint
- Returns station status (active/closed) based on real scheduled edges
- Add stationStatusResponse, cityResponse, routeSearchResponse, routeGeoJSONResponse types
- Add stub implementations for other API handlers to enable compilation
- Add TestHandlerStationStatus test passing

feat: Add bus transport type to routing with Pareto ranking support d93445ad55

feat: Implement synthetic edges city↔airport (Task 10) 06730fe05c

- Add TransferTime constants (AirportToCity, CityToStation, StationToStation)
- Update Edge struct with Synthetic field
- Enhance addSyntheticEdgesForNode to use constants and mark synthetic edges
- Update BuildGraphFromStations to set Synthetic field
- Mark synthetic edges in GeoJSON output as dashed lines
- Write TestSyntheticAirportCityEdges and TestRouteWithSyntheticAirportCityEdges tests

feat: Implement lazy hub expansion depth limiting with transfer depth limit of 5 (Task 14) 7c6fe4a99c

feat: Implement lazy hub expansion depth limiting with MaxTransfers support 026b779cb3

- Add transfer depth limiting in BFS/Dijkstra (MaxTransfers field in SearchOptions)
- Track transfer count at each step; stop when depth > 5
- Synthetic edge fallback on expansion failure
- Add TestLazyExpansionDepthLimit and TestFindRouteMaxTransfers tests

Co-Authored-By: Claude <noreply@anthropic.com>

feat: Implement Pareto-front ranking integration with multi-criteria sorting d67bc5aca7

- Add RankingMode field to SearchOptions (fastest/fewest_transfers/cheapest)
- Update FindRoutesPareto to respect ranking mode when sorting
- Add ranking_mode query parameter to RouteSearch endpoint
- Add TestParetoFrontGeneration with subtests for all three modes

Co-Authored-By: Claude <noreply@anthropic.com>

feat: Implement flight change notifications with route re-search on cancellation/major delay 2907ed3e0d

- Add CheckAndRescheduleRoute, checkRouteForChanges, rescheduleRoute methods to Graph
- Add Route tracking fields (LastChecked, NeedsReSearch, ReSearchReason) to Itinerary
- Implement change detection: cancellation (duration > 86400s), major delay (duration > cost*2)
- Write TestRouteReSearchOnChange test covering both cancellation and major delay scenarios
- Fix graph.go syntax errors and getStationNeighbors function

feat: Implement user preferences (saved cities, search history) with Redis storage and API handlers 5842667fff

feat: implement observability and metrics (Task 22) c58fbb50b1

- 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

feat: Full test suite and linter - fixed test failures and increased coverage to 61.1% 81686e9adc

feat: Full test suite and linter - increased coverage to 61.7%, fixed test failures 0bba23692c

fix: address code review findings 82757665e9

fix: address code review findings 9c402c0086

fix: address code review findings 777fda95a3

fix: address code review findings 78f662c985

fix: address code review findings 6dcec6a7a5

- 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

fix: address code review findings 9b89b7b9ab

fix: address code review findings 8aecaf1468

fix: update CLAUDE.md with missing documentation for new API endpoints and architectural features 4e55b52a86

fix: address code review findings 2cf9e20608

fix: address code review findings 48650d96cf

fix: address code review findings 0d603ad15f

move completed plan: 2026-08-15-full-implementation.md a509b71614

feat: create static directory and frontend files bd83cca99d

feat: update Go server to serve static files b000d7d44f

feat: mark manual testing tasks as completed 6e65e6161c

fix: address code review findings 502979d43b

fix: address code review findings b0dcc2dd3a

fix: address code review findings 1513a9fb67

- Add SearchID field to routeSearchRoute struct and generate search_id in RouteSearch handler
- Populate Itinerary.ID field with unique IDs using generateItineraryID function
- Fix information disclosure in error handling - log errors internally instead of leaking to clients
- Fix circuit breaker trip metric recording - only record when state transitions to open
- Fix context not passed in route expansion - use context.WithTimeout instead of context.TODO
- Fix admin auth fails open on missing API key - return 401 Unauthorized without revealing configuration

fix: address code review findings ef77b2c0a0

move completed plan: 2026-08-18-implement-leaflet-frontend.md 3a9dd24e33

feat: implement multi-stage Dockerfile for deployment infrastructure 69312022ea

Co-Authored-By: Claude <noreply@anthropic.com>

feat: update docker-compose.yml with cron and watchtower services cca176270c

feat: create Gitea Actions CI/CD workflow adfffa0f4b

Co-Authored-By: Claude <noreply@anthropic.com>

feat: verify acceptance criteria for deployment infrastructure da67d0eae7

Co-Authored-By: Claude <noreply@anthropic.com>

fix: address code review findings b14682f424

- Create README.md with project overview, quick start, Docker deployment, CI/CD info
- Update CLAUDE.md with Deployment section
- Fix cmd/api/main.go to use REDIS_ADDR env var with fallback
- Fix Dockerfile to use golang:1.22-alpine instead of golang:1.26-alpine
- Fix .gitea/workflows/deploy.yml to include Docker registry prefix in image tags

fix: address code review findings 4a7be531ed

fix: address code review findings c2bdffb0ab

fix: address code review findings c3bc719878

move completed plan: 2026-08-18-implement-deployment-infrastructure.md f48ec66166

fix: remove build artifacts and coverage files from tracking ae48045055

- Remove compiled binary 'api' from git tracking
- Remove 'cover.out' coverage file from git tracking
- Update .gitignore to include 'api' and 'cover.out'
- Clean up temporary files: coverage.out, dump.rdb

Mrixs self-assigned this 2026-08-19 12:08:32 +00:00

Mrixs closed this pull request

2026-08-19 12:15:25 +00:00

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Mrixs/trip-planner#3