Skip to content

[dashboards] Fix MCP proxy auth + setup docs#66

Closed
matthallett1 wants to merge 3 commits intomainfrom
contrib/matthallett/dashboard-review-fixes
Closed

[dashboards] Fix MCP proxy auth + setup docs#66
matthallett1 wants to merge 3 commits intomainfrom
contrib/matthallett/dashboard-review-fixes

Conversation

@matthallett1
Copy link
Copy Markdown
Collaborator

@matthallett1 matthallett1 commented Mar 17, 2026

Summary

  • Fix proxy auth method — The MCP proxy was sending the access key as Authorization: Bearer header, but open-brain-mcp expects ?key= query parameter. Search and capture silently failed through the dashboard.
  • Improve setup docs — Rewrote Quick Start with a credential source table, OAuth sign-in gotcha, and search troubleshooting (OpenRouter key, embeddings, similarity threshold).
  • Harden gitignore — Added .env.local, .env.*.local, and deno.lock to prevent credential/artifact leaks.

Found during live testing

Issue Root cause Fix
Search always returned "No thoughts found" Proxy sent key as Bearer header instead of query param Changed to ?key= in fetch URL
"Database error querying schema" on sign-in OAuth user has no password Documented: send recovery email or create email/password user
Search still empty after proxy fix Stale OpenRouter API key in Supabase secrets Updated secret (not in code) + documented in troubleshooting

Test plan

  • Verified locally: stats load (2059 thoughts), search returns results, capture works
  • Tested with real Supabase project + deployed Edge Function
  • Verify CI passes (markdownlint, metadata check)

🤖 Generated with Claude Code

headcrest and others added 2 commits March 16, 2026 18:37
Addresses 6 items from code review:
1. Package name "web" → "open-brain-dashboard"
2. MCP key moved from query string to Authorization header
3. robots.txt disallows crawling (personal dashboard)
4. package-lock.json removed from git + gitignored
5. app.d.ts: MCP env vars optional, comment explains preferred vs fallback
6. Indentation fix in MCP proxy error return

svelte-check: 0 errors, 0 warnings
vite build: clean production build

Original dashboard by @headcrest.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot added the dashboard Contribution: frontend template label Mar 17, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 17, 2026

OB1 Automated Review

Folder structure — All files are in allowed directories
Required files — README.md and metadata.json found in all contribution folders
Metadata valid — All metadata.json files have required fields
No credentials — No API keys, tokens, or secrets detected
SQL safety — No destructive SQL or core table modifications
Category artifacts — Required file types present for each category
PR format — Title follows [category] Description format
No binary blobs — No oversized or binary files
README completeness — All READMEs include Prerequisites, Steps, and Expected Outcome
Primitive dependencies — All declared primitive dependencies exist and are linked in README
LLM clarity review — Covered by Claude PR Review workflow
Remote MCP pattern — No local MCP server patterns detected — uses remote MCP correctly
Scope check — Files changed outside contribution folder(s):

  • .gitignore
    Contribution PRs should only modify files in their own folder.
    Internal links — All relative links in READMEs resolve to existing files

Result: 13/14 checks passed. Please fix the issues above and push again.


Post-Merge Tasks

These don't block merge — they're reminders for admins after this PR lands.

The proxy was sending the MCP key as an Authorization Bearer header,
but the open-brain-mcp Edge Function expects it as a ?key= query
parameter. Also rewrites the README with a credential source table,
OAuth sign-in troubleshooting, and search debugging guidance based
on real setup experience.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@matthallett1 matthallett1 changed the title [dashboards] Add Open Brain dashboard with review fixes [dashboards] Fix MCP proxy auth + setup docs Mar 17, 2026
@justfinethanku
Copy link
Copy Markdown
Collaborator

Combined into PR #83 with the original dashboard from #57.

Matt — next time, submit this as a PR review on the original PR with requested changes, or push fixes directly to the contributor's branch. Opening a separate PR to patch an open PR creates a dependency chain that's a pain to merge. The fixes were good, the process just needs to be tighter.

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

Labels

dashboard Contribution: frontend template

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants