Repository navigation
daily-checkin: Nitro's route cache writes KV outside writeCache on the skill detail route, so a KV 429 still escapes as a 500 (Sentry SKILLD-1V). #137
Description
Nitro's route cache writes KV outside writeCache on the skill detail route, so a KV 429 still escapes as a 500 (Sentry SKILLD-1V).
Target: layers/registry/server/api/skills/[...slug].get.ts
Verify with: pnpm vitest run test/unit/cache-write-best-effort.test.ts
Estimated to change 2 files.
The daily-checkin routine opened this issue automatically on its 2026-09-04T21:40:00.000Z run. It is not Harlan's own report. Close it to reject the proposal, and the reason you give stops it being offered again.
Activity
harlan-github-agent commentedon Sep 4, 2026
🤖 ISSUE TRIAGE
layers/registry/server/api/skills/[...slug].get.ts:456wraps its handler in Nitro'sdefineCachedEventHandler, whose internal KVsetItembypasses the repo's best-effortwriteCachehelper (shared/server/cache.ts). A `KV PUT failed: 42defineCachedEventHandlerwrapper inlayers/registry/server/api/skills/[...slug].get.tsto a plain handler that reads withreadCache(useStorage('cache'), key)on miss computes the response then persists withwriteCache(tfix: resolve issue #137 (#142)