Skip to content

[codex] Make Doctor current-state aware#18

Merged
dankritz merged 1 commit into
mainfrom
codex/doctor-current-state
May 11, 2026
Merged

[codex] Make Doctor current-state aware#18
dankritz merged 1 commit into
mainfrom
codex/doctor-current-state

Conversation

@dankritz
Copy link
Copy Markdown
Contributor

Summary

  • Make Doctor prefer the latest deployment state instead of diagnosing the newest historical failure.
  • Return rootCauseCategory: healthy with a passed health-check result when the latest deployment succeeded and the app is running.
  • Keep older failures as informational evidence so agents can see history without mistaking it for the current app state.
  • Add regression coverage for a newer successful deployment superseding an older failed deployment.

Validation

  • npm run typecheck --workspace @vibestack/api
  • 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:11
@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 (2 files)
  • apps/api/src/doctor.test.ts - 0 issues
  • apps/api/src/doctor.ts - 0 issues

Reviewed by grok-code-fast-1 · 65,301 tokens

@dankritz dankritz merged commit c72ac5c into main May 11, 2026
3 checks passed
@dankritz dankritz deleted the codex/doctor-current-state branch May 11, 2026 13:13
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