Skip to content

Nitro route-cache KV write 500s skill detail on a 429 #146

Closed

Description

@harlan-github-agent

The defineCachedEventHandler route cache writes KV directly, so a KV 429 escapes writeCache and 500s the skill detail endpoint.

Target: shared/server/cache.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-05T21: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

  1. harlan-github-agent commented on Sep 6, 2026

    @harlan-github-agent
    ContributorAuthor

    🤖 ISSUE TRIAGE

    Harlan Agent Kit posted this automated triage. It is not Harlan's personal assessment or commitment. AI open source policy.

    • Route: Needs info
    • Difficulty: 2/5
    • Impact: 2/5
    • Reproduction: No
    • Codebase review: Not needed
    • Summary: Code review contradicts the issue's mechanism. The route cache on skill detail (layers/registry/server/api/skills/[...slug].get.ts:456) runs on nitro 2.13.4 (the only locked version, deployed since well before 2026-09-05). In that version t
    • Next action: Read the SKILLD-1V Sentry event's request status for the skill detail culprit. If it is 500, share the stack so the unguarded write path can be located, since none exists in the current code. If it is 200 (nitro captureError of an already h
  2. added
    harlan-agent-needs-infoIssue triage needs more information before work can continue.
    and removed
    harlan-agent-runningAn Agent holds a Task on this issue or pull request right now.
    on Sep 6, 2026
  3. harlan-zw commented on Sep 7, 2026

    @harlan-zw
    Collaborator

    Fixed by #142 (merged 2026-09-07): skill detail cache writes route through writeCache, so a KV 429 no longer 500s the endpoint.

    🤖 Harlan Agent Kit closed this during an issue triage. It is not Harlan's personal reply.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions