Skip to content

improvement(search): guide independent live searches and reads into one step - #8282

Merged
waleedlatif1 merged 2 commits into
stagingfrom
improvement/live-search-parallel-guidance
Sep 25, 2026
Merged

waleedlatif1 merged 2 commits into
stagingfrom
improvement/live-search-parallel-guidance

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Shared live-search guidance now tells MCP/tool clients that one search across providers returns one ranked list for the same question, and that independent searches and reads of different documents should be issued together in the same step
  • Follows Anthropic's parallel tool use guidance: the model rarely batches independent calls unless prompted, and each extra step costs a full model round trip
  • Companion worker change adds the same instruction to the Assistant prompt

Type of Change

  • Improvement

Testing

Live-search tests (298) pass; lint, audits, docs manifest and type-check clean

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

vercel Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Sep 25, 2026 12:54am UTC

Request Review

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge.

Summary

The PR clarifies shared live-search guidance: one cross-provider search returns a ranked list, and independent searches and document reads should be issued in the same step. It also adds assertions for both instructions.

Reviews (2) · Last reviewed commit: "test(search): assert the batching guidan..."

Comment thread apps/sim/lib/sim-search/live/providers.ts
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

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

Copy link
Copy Markdown
Contributor

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

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@waleedlatif1
waleedlatif1 merged commit 55b1a5f into staging Sep 25, 2026
25 checks passed
@waleedlatif1
waleedlatif1 deleted the improvement/live-search-parallel-guidance branch September 25, 2026 00:58

This branch was previously deployed

1 inactive deployment
Preview — 098e5d75 Deployed Sep 25, 2026 by vercel[bot]
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