You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Skills search route cache turns a KV GET 500 into a request 500 #249
The skills-search route still runs on Nitro's defineCachedEventHandler, whose unshielded KV cache read let a KV GET failed: 500 escape and 500 the request (Sentry SKILLD-28, 2026-09-16T18:30:16Z), the same family already fixed for the detail route (#146) and skill-files (#201).
Verify with:pnpm vitest run test/unit/skill-search-cache.test.ts test/unit/cache-read-through.test.ts
Estimated to change 2 files.
The daily-checkin routine opened this issue automatically on its 2026-09-16T21: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.
Summary: skills index route still exports via defineCachedEventHandler (layers/registry/server/api/skills/index.get.ts:43) with skillSearchCacheOptions, whose unshielded KV read let a KV GET 500 (Sentry SKILLD-28) surface as a request 500. The same
Next action: Implement: replace defineCachedEventHandler at layers/registry/server/api/skills/index.get.ts:43 with the shielded cached() pattern from skills/[...slug].get.ts (reusing skillSearchCacheKey/shouldBypassCache semantics), then run pnpm vitest run test/unit/skill-search-cache.test.ts test/unit/cache-read-through.test.ts.
Activity
harlan-github-agent commentedon Sep 16, 2026
🤖 ISSUE TRIAGE