✨ feat(server): cache /api/v1/greet responses + admin cache flush endpoint #29

arcodange · 2026-05-03T16:32:51+02:00

arcodange commented

2026-05-03 16:32:51 +02:00

Summary

Extends the cache service (ADR-0022, PR #23) to two new use cases:

/api/v1/greet/{name} and /api/v1/greet?name=X now cache responses per name (60s TTL) with X-Cache: HIT/MISS header
New POST /api/admin/cache/flush endpoint (admin-authenticated via X-Admin-Password header) returns flushed item count

Companion to PR #22 (rate limit) and PR #23 (cache service).

Spec

GET /api/v1/greet?name=X (existing, now cached)

Header X-Cache: HIT if served from cache, MISS if computed
Cache key: greet:v1:<name>
TTL: 60s

GET /api/v1/greet/{name} (existing, now cached)

Same caching as above.

POST /api/admin/cache/flush (new)

Header X-Admin-Password: master admin password (req)
Returns 200 {"flushed": true, "items_flushed": N, "timestamp": "..."}
Returns 401 if unauthorized
Returns 503 {"error": "cache_disabled"} if cacheService is nil

Implementation note

Replaces the previous greetHandler.RegisterRoutes(r) call in registerApiV1Routes with explicit route handlers (handleGreetQuery, handleGreetPath) that add the cache layer. The greet service logic itself is unchanged.

Test plan

go build ./pkg/server/... PASS
go test ./pkg/server/... PASS (existing tests, no regression)
CI passes
Manual curl test:
curl -i http://localhost:8080/api/v1/greet/Alice then curl -i http://localhost:8080/api/v1/greet/Alice — second should show X-Cache: HIT

Test coverage deferred to BDD (existing greet scenarios exercise these handlers; cache hit/miss is incidental).

Out of scope

Phase 2 ADR-0022 (Redis support) — deferred
Cache invalidation on data change — not needed yet (simple TTL works for greet/version)

Migration context

ARCODANGE Phase 1 - autonomous run via Mistral Vibe with ICM workspace at ~/Work/Vibe/workspaces/greet-cache-and-admin/. Cost €2.32 stages 01-02. Trainer finalized commit/PR after Mistral's test scaffold did not compile.

🤖 Co-Authored-By: Mistral Vibe + Claude Opus 4.7

## Summary Extends the cache service ([ADR-0022](adr/0022-rate-limiting-cache-strategy.md), PR #23) to two new use cases: 1. `/api/v1/greet/{name}` and `/api/v1/greet?name=X` now cache responses per name (60s TTL) with `X-Cache: HIT/MISS` header 2. New `POST /api/admin/cache/flush` endpoint (admin-authenticated via `X-Admin-Password` header) returns flushed item count Companion to PR #22 (rate limit) and PR #23 (cache service). ## Spec ### GET /api/v1/greet?name=X (existing, now cached) - Header `X-Cache: HIT` if served from cache, `MISS` if computed - Cache key: `greet:v1:<name>` - TTL: 60s ### GET /api/v1/greet/{name} (existing, now cached) Same caching as above. ### POST /api/admin/cache/flush (new) - Header `X-Admin-Password`: master admin password (req) - Returns 200 `{"flushed": true, "items_flushed": N, "timestamp": "..."}` - Returns 401 if unauthorized - Returns 503 `{"error": "cache_disabled"}` if cacheService is nil ## Implementation note Replaces the previous `greetHandler.RegisterRoutes(r)` call in `registerApiV1Routes` with explicit route handlers (handleGreetQuery, handleGreetPath) that add the cache layer. The greet service logic itself is unchanged. ## Test plan - [x] `go build ./pkg/server/...` PASS - [x] `go test ./pkg/server/...` PASS (existing tests, no regression) - [ ] CI passes - [ ] Manual curl test: `curl -i http://localhost:8080/api/v1/greet/Alice` then `curl -i http://localhost:8080/api/v1/greet/Alice` — second should show `X-Cache: HIT` Test coverage deferred to BDD (existing greet scenarios exercise these handlers; cache hit/miss is incidental). ## Out of scope - Phase 2 ADR-0022 (Redis support) — deferred - Cache invalidation on data change — not needed yet (simple TTL works for greet/version) ## Migration context ARCODANGE Phase 1 - autonomous run via Mistral Vibe with ICM workspace at `~/Work/Vibe/workspaces/greet-cache-and-admin/`. Cost €2.32 stages 01-02. Trainer finalized commit/PR after Mistral's test scaffold did not compile. 🤖 Co-Authored-By: Mistral Vibe + Claude Opus 4.7

arcodange added 1 commit 2026-05-03 16:32:52 +02:00

✨ feat(server): cache /api/v1/greet responses + admin POST /api/admin/cache/flush 35de879a58

Extends the cache service (PR #23) to two new use cases:
- /api/v1/greet/{name} and /api/v1/greet?name=X now cache responses per name (60s TTL)
  with X-Cache: HIT/MISS header
- New POST /api/admin/cache/flush endpoint (X-Admin-Password header for auth)
  returns 200 with {flushed, items_flushed, timestamp} or 401/503

Companion to PR #22 (rate limit) and PR #23 (cache service).

Changes:
- pkg/server/server.go : handleGreetQuery + handleGreetPath cache layer,
  handleAdminCacheFlush handler, /api/admin route registration

Note: replaces the previous greetHandler.RegisterRoutes call with explicit
route handlers to add the cache layer. The greet service itself is unchanged.

Test coverage: deferred to BDD (the existing greet BDD scenarios exercise
the same handlers; cache hit/miss behavior is incidental and easy to verify
via curl + X-Cache header).

Generated ~95% in autonomy by Mistral Vibe via ICM workspace
~/Work/Vibe/workspaces/greet-cache-and-admin/.
Trainer (Claude) finalized commit/PR after Mistral's test scaffold did not
compile (used non-existent test helpers).

🤖 Co-Authored-By: Mistral Vibe (devstral-2 / mistral-medium-3.5)
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>