Skip to content

fix: route calibration CLI and serialize profile ids#1203

Open
rivet-ops wants to merge 1 commit into
garrytan:garrytan/fix-wavefrom
rivet-ops:rivet/v0361-staging-blockers
Open

fix: route calibration CLI and serialize profile ids#1203
rivet-ops wants to merge 1 commit into
garrytan:garrytan/fix-wavefrom
rivet-ops:rivet/v0361-staging-blockers

Conversation

@rivet-ops
Copy link
Copy Markdown

Summary

  • Route gbrain calibration through the CLI-only dispatcher so it reaches the existing calibration handler.
  • Cast calibration_profiles.id from BIGSERIAL to int in the latest-profile query so calibration --json can serialize profile rows.
  • Add regression coverage for both dispatch and JSON-safe profile id selection.

Verification

  • bun run typecheck
  • bun test test/cli.test.ts test/calibration-cli.test.ts test/cross-brain-calibration.test.ts test/recall-footer.test.ts test/doctor-calibration-checks.test.ts test/budget-meter.test.ts test/auto-think-phase.test.ts test/eval-takes-quality-rubric.test.ts — 111 pass / 0 fail
  • bun run verify
  • Hermetic PGLite smoke: init isolated GBRAIN_HOME, insert calibration profile, run gbrain calibration --json, verify id parses as a JSON number.

Notes

This is scoped to the v0.36.1 calibration adoption blockers. It does not run production migrations, enable embeddings, or change runtime config.

- Add calibration to CLI-only dispatch so the command reaches its handler
- Cast calibration profile BIGSERIAL ids to int for JSON output
- Pin regressions in CLI and calibration tests
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