Skip to content

perf(search): search up to eight live accounts at once - #8891

Merged
waleedlatif1 merged 1 commit into
stagingfrom
perf/live-search-account-concurrency
Oct 10, 2026
Merged

waleedlatif1 merged 1 commit into
stagingfrom
perf/live-search-account-concurrency

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • searchLiveKnowledge waits for every selected account before fusing results, and it searched accounts four at a time (ACCOUNT_CONCURRENCY). An organization with 5–8 connected accounts paid a second full provider round trip on every unscoped search.
  • Production over the last 7 days: 7-account searches ran p50 2.05 s / p90 2.68 s, against 0.62 s / 1.16 s for single-account searches. No provider call came near the 12 s per-account timeout.
  • Raised to eight. Each account is a single provider call, and MAX_ACCOUNTS (20) still bounds the total.

Test plan

  • CI
  • After deploy: compare p50 of live searches with 7+ accounts in the "Knowledge search completed" diagnostics.

🤖 Generated with Claude Code

@vercel

vercel Bot commented Oct 10, 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 Oct 10, 2026 8:43pm 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.

Heads up: you’ve reached your flex budget. Increase your flex budget or wait for usage to reset.

Turn on auto-fix | Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Medium impact] The PR appears safe to merge; no actionable issue was found in the higher account limit.

Summary

Raises ACCOUNT_CONCURRENCY from four to eight so searches with five to eight eligible accounts can start them together.

  • Live searches run up to eight connected accounts at once.

Reviews (2) · Last reviewed commit: "perf(search): search up to eight live ac..." · Reviewed by Greptile

A live search waits for every account before fusing results, and accounts ran
four at a time, so an organization with five to eight connected accounts paid
a second full provider round trip on every unscoped search. Production
seven-account searches took about 2.1 s at p50 against about 0.6 s for one
account. Each account is a single provider call, so eight at once stays well
inside provider and token-refresh limits.
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1
waleedlatif1 force-pushed the perf/live-search-account-concurrency branch from 327e6b7 to 0ad0911 Compare October 10, 2026 20:43
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 10, 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 1 file

Confidence score: 5/5

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

Heads up: you’ve reached your flex budget. Increase your flex budget or wait for usage to reset.

Turn on auto-fix | Re-trigger cubic

@waleedlatif1
waleedlatif1 merged commit 71f05ec into staging Oct 10, 2026
48 checks passed
@waleedlatif1
waleedlatif1 deleted the perf/live-search-account-concurrency branch October 10, 2026 22:04

This branch was previously deployed

1 inactive deployment
Preview — 0ad09111 Deployed Oct 10, 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