Skip to content

[codex] Cache Doctor OpenRouter results#19

Merged
dankritz merged 1 commit into
mainfrom
codex/cache-doctor-results
May 11, 2026
Merged

[codex] Cache Doctor OpenRouter results#19
dankritz merged 1 commit into
mainfrom
codex/cache-doctor-results

Conversation

@dankritz
Copy link
Copy Markdown
Contributor

Summary

  • Add an append-only 004_doctor_cache.sql migration with a doctor_cache table keyed by deployment.
  • Cache OpenRouter-enhanced Doctor packets per related deployment, root-cause category, and OpenRouter model so switching between apps does not repeatedly call the LLM.
  • Keep deterministic Doctor state recomputation cheap/current, then reuse cached AI enrichment when applicable.
  • Fix healthy Doctor copy: healthy cards now say no repair is needed and the generated prompt no longer starts with a fix instruction.

Validation

  • npm run typecheck --workspace @vibestack/api && npm run typecheck --workspace @vibestack/web
  • npm run test --workspace @vibestack/api -- doctor.test.ts
  • npm test
  • npm run build
  • npm run lint

@dankritz dankritz marked this pull request as ready for review May 11, 2026 13:26
@dankritz dankritz merged commit 6e2cde6 into main May 11, 2026
3 checks passed
@dankritz dankritz deleted the codex/cache-doctor-results branch May 11, 2026 13:26
@kilo-code-bot
Copy link
Copy Markdown

kilo-code-bot Bot commented May 11, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (4 files)
  • apps/api/migrations/004_doctor_cache.sql - 0 issues
  • apps/api/src/doctor.test.ts - 0 issues
  • apps/api/src/doctor.ts - 0 issues
  • apps/web/src/main.tsx - 0 issues

Reviewed by grok-code-fast-1 · 128,359 tokens

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant