Skip to content

Explain a missing graph schema instead of a raw 500 - #88

Merged
om986 merged 1 commit into
mainfrom
fix/console-setup-errors
Oct 13, 2026
Merged

om986 merged 1 commit into
mainfrom
fix/console-setup-errors

Conversation

@om986

@om986 om986 commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • When om migrate has not created the graph tables, graph API calls return 503 with schema_not_ready instead of a 500 and the Postgres error. The console banner says to run om migrate and refresh.

Test plan

  • Start Postgres without running om migrate, open the console with a valid API key, and confirm the banner says to run om migrate (no 500 and no SQL error).
  • Run om migrate and refresh. The console loads.
  • go test ./internal/api/

@coderabbitai

coderabbitai Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: OpenSourceOM/core/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 684b6244-9a60-4308-a8b7-de7c5e341e33
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

The console showed a status code when om migrate had not created the graph tables.

Co-authored-by: Cursor <cursoragent@cursor.com>
@om986
om986 force-pushed the fix/console-setup-errors branch from e26c999 to 953f7f8 Compare October 13, 2026 02:11
@om986 om986 changed the title Explain console failures when the API key or graph schema is missing Explain a missing graph schema instead of a raw 500 Oct 13, 2026
@om986
om986 merged commit e09df8d into main Oct 13, 2026
7 checks passed
@om986
om986 deleted the fix/console-setup-errors branch October 13, 2026 02:14
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