Skip to content

fix(onboard): correct Brave Search API key URL#1591

Merged
cv merged 1 commit intoNVIDIA:mainfrom
kagura-agent:fix/brave-search-url
Apr 8, 2026
Merged

fix(onboard): correct Brave Search API key URL#1591
cv merged 1 commit intoNVIDIA:mainfrom
kagura-agent:fix/brave-search-url

Conversation

@kagura-agent
Copy link
Copy Markdown
Contributor

@kagura-agent kagura-agent commented Apr 8, 2026

What

The Brave Search API key help URL shown during onboarding points to https://api.search.brave.com/app/keys, which returns a 403 Forbidden error.

Fix

Updated the URL to https://api-dashboard.search.brave.com/app/keys, which is the correct dashboard URL (redirects to login page as expected).

Verification

# Old URL → 403
curl -sI 'https://api.search.brave.com/app/keys' | head -3
# HTTP/2 403

# New URL → 303 redirect to login
curl -sI 'https://api-dashboard.search.brave.com/app/keys' | head -3
# HTTP/2 303 → /login?redirect=%2Fapp%2Fkeys

Closes #1573

Summary by CodeRabbit

  • Bug Fixes
    • Updated the Brave Search API key retrieval URL in the onboarding process to direct users to the correct location for obtaining their API keys.

The help URL for obtaining a Brave Search API key pointed to
https://api.search.brave.com/app/keys which returns a 403 error.
Updated to https://api-dashboard.search.brave.com/app/keys which
is the correct dashboard URL.

Fixes NVIDIA#1573
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 8, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d85c11f5-25c4-4faa-9137-ff9b556828ac

📥 Commits

Reviewing files that changed from the base of the PR and between edc561a and fdd01ec.

📒 Files selected for processing (1)
  • bin/lib/onboard.js

📝 Walkthrough

Walkthrough

A URL constant in the onboarding process has been updated from an outdated Brave Search API endpoint to the correct dashboard URL, fixing a 403 error that prevented users from retrieving their API key during setup.

Changes

Cohort / File(s) Summary
Brave Search API URL Fix
bin/lib/onboard.js
Updated the Brave Search API key help URL from https://api.search.brave.com/app/keys to https://api-dashboard.search.brave.com/app/keys to resolve 403 errors and direct users to the correct API key retrieval page.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A link that was broken, now fixed with great care,
The dashboard awaits, it's the right place to share,
No more 403 errors to cause us dismay,
The API keys flow in the proper way! 🔑✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix(onboard): correct Brave Search API key URL' directly and clearly describes the main change: correcting the Brave Search API key URL in the onboarding flow.
Linked Issues check ✅ Passed The PR directly addresses issue #1573 by updating the incorrect Brave Search API key URL from https://api.search.brave.com/app/keys (which returns 403) to https://api-dashboard.search.brave.com/app/keys (the correct dashboard URL).
Out of Scope Changes check ✅ Passed The PR contains only a single line change updating the Brave Search API key URL in onboard.js, which is directly scoped to fixing issue #1573 with no extraneous modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@wscurran wscurran added documentation Improvements or additions to documentation Getting Started Use this label to identify setup, installation, or onboarding issues. priority: high Important issue that should be resolved in the next release fix Integration: Brave labels Apr 8, 2026
@wscurran
Copy link
Copy Markdown
Contributor

wscurran commented Apr 8, 2026

✨ Thanks for submitting this fix, which proposes a way to correct the Brave Search API key URL presented during onboarding. This ensures users are directed to the proper dashboard to retrieve their keys.


Possibly related open issues:

1 similar comment
@wscurran
Copy link
Copy Markdown
Contributor

wscurran commented Apr 8, 2026

✨ Thanks for submitting this fix, which proposes a way to correct the Brave Search API key URL presented during onboarding. This ensures users are directed to the proper dashboard to retrieve their keys.


Possibly related open issues:

@cv cv added the v0.0.10 Release target label Apr 8, 2026
@cv cv self-assigned this Apr 8, 2026
Copy link
Copy Markdown
Contributor

@cv cv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM — Brave moved their API key dashboard to api-dashboard.search.brave.com. Straightforward URL correction, no security concerns.

@cv cv merged commit 377e64f into NVIDIA:main Apr 8, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation fix Getting Started Use this label to identify setup, installation, or onboarding issues. Integration: Brave priority: high Important issue that should be resolved in the next release v0.0.10 Release target

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Nemoclaw][All Platform] Brave Search API key URL "https://api.search.brave.com/app/keys" hit 403 error

3 participants