Skip to content

feat(mcp): search_attributes full tool and subagent evals - #2244

Open
nikkikapadia wants to merge 3 commits into
nikki/search-attributes/find-specific-attrfrom
nikki/search-attributes/evals
Open

nikkikapadia wants to merge 3 commits into
nikki/search-attributes/find-specific-attrfrom
nikki/search-attributes/evals

Conversation

@nikkikapadia

Copy link
Copy Markdown
Member

Closes EXP-1352

This PR adds a bunch of evals for the new search_attributes tool. There's a file for entire tool evals and then another file to test the subagent behaviour.

In the full tool eval we test multiple cases like what the tool will do if there's no query, query that results in no results, choosing between two close attributes, choosing between attributes with no descriptions, testing if it hits specific tools, etc.

In the subagent evals we test the outputs of the hint and the attribute shortlist agents. So whether it gets the hint key words we expect and whether it ranks attributes properly (based on factors such as deprecation).

The rest of the file changes are adjustments to existing tests/logic, adding mocks or exporting all of the tools so we can test them.

@nikkikapadia
nikkikapadia added this pull request to stack #2245 November 13, 2026 17:02
@vercel

vercel Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
sentry-local Skipped Skipped Nov 13, 2026 6:51pm UTC

Request Review

@linear-code

linear-code Bot commented Nov 13, 2026

Copy link
Copy Markdown

EXP-1352

@cursor cursor 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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 6ce2ce6. Configure here.

@nikkikapadia
nikkikapadia marked this pull request as ready for review November 13, 2026 19:17
@github-actions github-actions Bot added the risk: medium PR risk score: medium label Nov 13, 2026

This branch was successfully deployed

1 active and 1 inactive deployments
Actions — 999d2871 Deployed Nov 13, 2026 by nikkikapadia via eval #1872
Preview — 999d2871 Deployed Nov 13, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: medium PR risk score: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant