Skip to content

add searchTerm to teams tool#65

Merged
derek-etherton-opslevel merged 3 commits intomainfrom
add-teams-search
Jul 16, 2025
Merged

add searchTerm to teams tool#65
derek-etherton-opslevel merged 3 commits intomainfrom
add-teams-search

Conversation

@derek-etherton-opslevel
Copy link
Copy Markdown
Contributor

Resolves #

Problem

When querying for teams, the server always fetches all teams across the account, which can take a lot of time to paginate and uses a disproportionate number of tokens.

Solution

Add a searchTerm parameter so that specific teams can be looked up, using far fewer resources.

Positive use case example (search works):

Screenshot 2025-07-16 at 8 17 57 AM

Negative use case examples (team doesn't exist, fall back to list):

Screenshot 2025-07-16 at 8 20 01 AM

Checklist

  • I have run this code, and it appears to resolve the stated issue.
  • This PR has no user interface changes or has already received approval from product management to change the interface.
  • Make a changie entry that explains the customer facing outcome of this change.

Comment thread src/cmd/root.go Outdated
Copy link
Copy Markdown
Contributor

@rocktavious rocktavious left a comment

Choose a reason for hiding this comment

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

LGTM - but i think there is a possible simplification

@derek-etherton-opslevel derek-etherton-opslevel merged commit 1b89e39 into main Jul 16, 2025
2 checks passed
@derek-etherton-opslevel derek-etherton-opslevel deleted the add-teams-search branch July 16, 2025 15:57
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.

2 participants