Conversation
Add an optional parent_uid parameter to the search_projects tool so callers can retrieve child projects scoped to a specific parent foundation or umbrella project UID. When parent_uid is provided, the query payload Parent field is set to "project:<parent_uid>", mirroring the existing pattern used by the search_committees and search_committee_members tools. Also makes the name field optional (omitempty) so callers can filter by parent alone without supplying a name. 🤖 Generated with [GitHub Copilot](https://github.com/features/copilot) (via Zed) Signed-off-by: Eric Searcy <eric@linuxfoundation.org>
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.
Tip: disable this comment in your organization's Code Review settings.
There was a problem hiding this comment.
Pull request overview
Adds support for scoping search_projects results to a parent project, enabling callers to retrieve child projects under a specific foundation/umbrella UID.
Changes:
- Extends
SearchProjectsArgswith optionalparent_uidand makesnameoptional (omitempty). - Sets the query payload
Parentfield toproject:<parent_uid>when provided. - Updates the tool description and logging to include
parent_uid.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| Name string `json:"name,omitempty" jsonschema:"Name or partial name of the project to search for"` | ||
| ParentUID string `json:"parent_uid,omitempty" jsonschema:"Optional UID of a foundation or umbrella project to filter child projects by"` | ||
| PageSize int `json:"page_size,omitempty" jsonschema:"Number of results per page (default 10, max 100)"` |
There was a problem hiding this comment.
With name now omitempty and parent_uid optional, callers can omit both fields and this handler will issue an effectively unfiltered QueryResources request (Type=project, Sort=name_asc). If the tool is intended to require at least one filter (name or parent), add an explicit validation error when both are empty, or update the tool description/jsonschema text to clearly document that an unfiltered listing is allowed.
There was a problem hiding this comment.
not worried about this at the moment
Summary
Adds an optional
parent_uidparameter to thesearch_projectstool so callers can retrieve child projects scoped to a specific parent foundation or umbrella project UID.Changes
SearchProjectsArgsgains a new optionalparent_uidfield with a descriptivejsonschematag.parent_uidis provided, the query payloadParentfield is set to"project:<parent_uid>"— mirroring the existing pattern used bysearch_committeesandsearch_committee_members.namefield is nowomitemptyso callers can filter by parent alone without supplying a name.parent_uidalongsidenameandpage_size.Jira
ARCH-364