Skip to content

Explain a missing console API key instead of a raw 401 - #87

Merged
om986 merged 1 commit into
mainfrom
fix/console-api-key-error
Oct 13, 2026
Merged

om986 merged 1 commit into
mainfrom
fix/console-api-key-error

Conversation

@om986

@om986 om986 commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Show a banner in the graph panel when the console gets a 401, telling you to paste OM_API_SECRET and click Save API key.
  • Distinguish a missing key from a saved key the server rejected, and reload the console after the key is saved.

Test plan

  • Open the console with OM_API_SECRET set and no saved key. The graph panel should say no API key is saved, and the API key field should be outlined.
  • Save a wrong key. The banner should say the saved key was refused.
  • Save the real OM_API_SECRET value. The banner should clear and the graph should load.
  • Refresh after choosing Full graph snapshot and confirm that choice stays selected.

Made with Cursor

The graph request fails when OM_API_SECRET is set and no key is saved. Show that in the panel and reload after the key is saved.

Co-authored-by: Cursor <cursoragent@cursor.com>
@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: ad3296c6-2477-4f15-81ac-ccce15ade7a3
  • 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.

@om986
om986 merged commit c52de90 into main Oct 13, 2026
7 checks passed
@om986
om986 deleted the fix/console-api-key-error branch October 13, 2026 01:48
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