Skip to content

feat: web search tool details#4208

Open
Yash-Singh1 wants to merge 1 commit into
mainfrom
yash/web-search-tool-details
Open

feat: web search tool details#4208
Yash-Singh1 wants to merge 1 commit into
mainfrom
yash/web-search-tool-details

Conversation

@Yash-Singh1

@Yash-Singh1 Yash-Singh1 commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

What Changed

Added web search tool details

Why

Gives user info on what the model is actually searching while it's searching

UI Changes

Screenshot 2026-07-20 at 4 05 32 PM Screenshot 2026-07-20 at 4 06 29 PM Screenshot 2026-07-20 at 4 06 35 PM

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Note

Low Risk
Chat UI and work-log derivation only; no auth, persistence, or API contract changes beyond displaying existing tool payloads.

Overview
Web search tool calls in the chat work log now show what the model searched instead of a generic label.

session-logic now keeps toolData from orchestration payloads for web_search entries (same pattern as MCP), so search items survive lifecycle collapse. Preview and expand rendering moved into workEntryDetails: the row shows the first query (or +N more), find-in-page style actions when relevant, and expanding shows pretty-printed JSON of the full payload (queries, results, URLs).

Reviewed by Cursor Bugbot for commit 2711d4e. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Add web search tool details to work timeline entries

  • Extracts workEntryPreview and buildToolCallExpandedBody into a new workEntryDetails.ts module, removing local implementations from MessagesTimeline.tsx.
  • Work entries with itemType: 'web_search' now show the first search query in compact preview, with a +N more suffix when multiple queries exist.
  • The expanded body for web_search entries now includes a pretty-printed JSON block of the tool data alongside existing command/detail/changed-files sections.
  • toDerivedWorkLogEntry in session-logic.ts now populates toolData for web_search items in addition to mcp_tool_call.

Macroscope summarized 2711d4e.

@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: d8f9d824-937e-4e74-8026-55ea6ae8b829

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch yash/web-search-tool-details

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.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels Jul 20, 2026
@macroscopeapp

macroscopeapp Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved

This PR adds UI display logic for web search tool details, extracting existing code to a new module and adding formatting for web search previews. Changes are limited to presentation layer with comprehensive test coverage and no security or infrastructure implications.

You can customize Macroscope's approvability policy. Learn more.

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

Labels

size:L 100-499 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant