Skip to content

improvement(byok): remove web search block exa #2579

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
icecrasher321 merged 4 commits into from
Dec 25, 2025
Merged

improvement(byok): remove web search block exa #2579

icecrasher321 merged 4 commits into from
Dec 25, 2025

Conversation

@icecrasher321
Copy link
Collaborator

@icecrasher321 icecrasher321 commented Dec 25, 2025 •
edited
Loading

Summary

Can always use Exa block directly. Minor UX changes along with it.
Fix autolayout error

Type of Change

  • Other: UI/UX

Testing

Tested manually

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel
Copy link

vercel bot commented Dec 25, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Review Updated (UTC)
docs Skipped Skipped Dec 25, 2025 3:22am

@icecrasher321 icecrasher321 changed the title improvement(byok): remove web search block exa improvement(byok): remove web search block exa + fix autolayout error Dec 25, 2025
@icecrasher321 icecrasher321 changed the title improvement(byok): remove web search block exa + fix autolayout error improvement(byok): remove web search block exa Dec 25, 2025
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Dec 25, 2025 •
edited
Loading

Greptile Summary

Removed Exa from BYOK (Bring Your Own Key) system - users can now only use platform Exa API key with standard pricing. Fixed autolayout error by wrapping Panel component in ReactFlowProvider on error and empty workflow pages.

Major Changes:

  • Removed Exa from BYOK providers list across type definitions, API validation, and UI
  • Search API route no longer accepts workspaceId parameter or checks for BYOK keys
  • All Exa searches now use platform API key with fixed SEARCH_TOOL_COST (removed BYOK cost bypass)
  • Added ReactFlowProvider wrapper for Panel component in error page and empty workflow page to fix hook dependency error
  • Updated use-auto-layout hook documentation to clarify ReactFlowProvider requirement
  • Minor UI improvements to BYOK settings (adjusted skeleton styling and icon container)

Confidence Score: 5/5

  • This PR is safe to merge with no issues found
  • Changes are straightforward and well-executed: cleanly removes Exa BYOK feature across all layers (types, API, UI) and properly fixes ReactFlow hook dependency issue by adding required provider wrapper
  • No files require special attention

Important Files Changed

Filename Overview
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/error/index.tsx Wrapped Panel component in ReactFlowProvider to fix autolayout hook error
apps/sim/app/workspace/[workspaceId]/w/page.tsx Wrapped Panel component in ReactFlowProvider to fix autolayout hook error
apps/sim/app/api/tools/search/route.ts Removed Exa BYOK key support, always use platform Exa API key
apps/sim/app/workspace/[workspaceId]/w/components/sidebar/components/settings-modal/components/byok/byok.tsx Removed Exa from BYOK UI, updated skeleton styling for consistency

Sequence Diagram

sequenceDiagram
    participant User
    participant BYOK_UI as BYOK Settings UI
    participant Search_Tool as Search Tool
    participant Search_API as Search API Route
    participant Exa as Exa API

    Note over User,Exa: Before: BYOK for Exa
    User->>BYOK_UI: Configure Exa API key
    BYOK_UI->>Search_API: workspaceId in request
    Search_API->>Search_API: Check BYOK key
    Search_API->>Exa: Use workspace key
    
    Note over User,Exa: After: Platform Exa Only
    User->>Search_Tool: Use search block
    Search_Tool->>Search_API: query only (no workspaceId)
    Search_API->>Exa: Use platform env.EXA_API_KEY
    Exa-->>Search_API: Results
    Search_API-->>User: Results + SEARCH_TOOL_COST

    Note over User,Exa: ReactFlowProvider Fix
    User->>User: Load error/empty page
    User->>User: Panel renders
    User->>User: useAutoLayout hook called
    Note right of User: Panel needs ReactFlowProvider<br/>for useReactFlow hook
Loading

greptile-apps[bot]
greptile-apps bot reviewed Dec 25, 2025
View reviewed changes
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Additional Comments (1)

  1. apps/sim/lib/api-key/byok.ts, line 9 (link)

    logic: 'exa' still included in type definition but removed elsewhere

8 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@icecrasher321
Copy link
Collaborator Author

icecrasher321 commented Dec 25, 2025

@greptile

@icecrasher321 icecrasher321 merged commit 66766a9 into staging Dec 25, 2025
11 checks passed
@waleedlatif1 waleedlatif1 mentioned this pull request Dec 26, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@icecrasher321