Skip to content

feat(agent-toolkit): scope DOCUMENTS search to specific docs via docIds - #485

Open
miloszpi-monday wants to merge 1 commit into
mondaycom:masterfrom
miloszpi-monday:feat/docs-search-ids-filter
Open

feat(agent-toolkit): scope DOCUMENTS search to specific docs via docIds#485
miloszpi-monday wants to merge 1 commit into
mondaycom:masterfrom
miloszpi-monday:feat/docs-search-ids-filter

Conversation

@miloszpi-monday

Copy link
Copy Markdown
Contributor

Summary

  • Adds an optional docIds filter on the search tool for DOCUMENTS search, forwarded as search.docs(ids:).
  • Callers can scope a docs search to a known set of documents (IDs as they appear in document URLs, at most 512) instead of the whole account. Combines with workspaceIds; empty/null means no filter.
  • Hand-applies only the new ids argument to the local schema snapshot (verified against a live fetch) to avoid pulling unrelated schema drift. Toolkit version bumped to 5.70.0.

Test plan

  • Run yarn workspace @mondaydotcomorg/agent-toolkit test (or the search-tool suite) and confirm the new docIds cases pass
  • Confirm an unfiltered DOCUMENTS search still sends docIds: undefined and returns the same results as before
  • Confirm docIds: [id] plus a matching searchTerm returns only those docs
  • Confirm docIds combines with workspaceIds, and that a scalar/[]/null is accepted the same way as other id filters
  • Smoke a live search.docs(ids:) call against API version 2026-10

Made with Cursor

search gains an optional docIds argument, forwarded to search.docs(ids:),
so a DOCUMENTS search can be scoped to a known set of documents (ids as
they appear in document URLs, at most 512) instead of the whole account.
It combines with workspaceIds, and an empty array or null means no filter.

search.docs(ids:) exists from API version 2026-10, the version the toolkit
pins, and was verified against the live API.

The schema snapshot carries only the hand-applied ids argument, verified
against a live fetch, to keep unrelated server-side drift out of the diff.

Co-authored-by: Cursor <cursoragent@cursor.com>
@miloszpi-monday
miloszpi-monday marked this pull request as ready for review September 11, 2026 09:20
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