Skip to content

Comments

feat(web): FE Changes for Brave Web Search 3/3 (#8597)#8641

Merged
justin-tahara merged 1 commit intorelease/v2.12from
hotfix/web-fe-2.12
Feb 20, 2026
Merged

feat(web): FE Changes for Brave Web Search 3/3 (#8597)#8641
justin-tahara merged 1 commit intorelease/v2.12from
hotfix/web-fe-2.12

Conversation

@justin-tahara
Copy link
Contributor

@justin-tahara justin-tahara commented Feb 20, 2026

Description

How Has This Been Tested?

Additional Options

  • [Required] I have considered whether this PR needs to be cherry-picked to the latest beta branch.
  • [Optional] Override Linear Check

Summary by cubic

Adds Brave Search as a new web search provider in Admin. Users can now configure Brave Search via API and see the Brave logo in the UI.

  • New Features
    • Added “brave” provider with label, helper text, logo, and API key docs link.
    • Provider marked as requires API key; no extra config keys.

Written for commit 45b8b5f. Summary will update on new commits.

@justin-tahara justin-tahara requested a review from a team as a code owner February 20, 2026 19:28
@justin-tahara justin-tahara merged commit 6120d01 into release/v2.12 Feb 20, 2026
14 of 16 checks passed
@justin-tahara justin-tahara deleted the hotfix/web-fe-2.12 branch February 20, 2026 19:29
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 20, 2026

Greptile Summary

This PR adds frontend support for Brave Web Search as a new search provider option in the web search configuration interface. The changes include adding the Brave logo asset and updating the search provider utilities to recognize and configure Brave alongside existing providers like Google PSE, Serper, Exa, and SearXNG.

Changes:

  • Added Brave.svg logo file to web/public/ directory
  • Extended WebSearchProviderType union type to include "brave"
  • Added Brave configuration to SEARCH_PROVIDER_DETAILS with label, subtitle, helper text, logo path, and API key documentation URL
  • Added Brave capabilities to SEARCH_PROVIDER_CAPABILITIES (requires API key, no additional config fields)

The implementation follows the established pattern for other search providers and integrates seamlessly with the existing UI in page.tsx.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The changes are straightforward frontend additions that follow existing patterns for other search providers. The code is consistent with the codebase structure, properly typed, and integrates well with the backend implementation (verified that BRAVE exists in backend enums). No logic bugs, security issues, or breaking changes detected.
  • No files require special attention

Important Files Changed

Filename Overview
web/public/Brave.svg Added Brave Search logo SVG asset for UI display
web/src/app/admin/configuration/web-search/searchProviderUtils.ts Added brave to search provider types and configured its metadata and capabilities

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User Opens Web Search Config] --> B{Select Search Provider}
    B --> C[Google PSE]
    B --> D[Serper]
    B --> E[Exa]
    B --> F[SearXNG]
    B --> G[Brave - NEW]
    
    G --> H{Provider Configured?}
    H -->|No| I[Show Connect Button]
    I --> J[Open Setup Modal]
    J --> K[Enter Brave API Key]
    K --> L[Validate API Key]
    L -->|Success| M[Save Configuration]
    L -->|Error| N[Show Error Message]
    
    H -->|Yes| O[Show Set as Default Button]
    O --> P[Activate Brave Provider]
    P --> Q[Update Active Search Provider]
    
    M --> R[SEARCH_PROVIDER_DETAILS<br/>brave config]
    Q --> R
    
    R --> S[Backend: BRAVE enum<br/>in shared_configs/enums.py]
Loading

Last reviewed commit: 45b8b5f

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