Skip to content

fix(list_users_and_teams): ignore null users in name results#261

Open
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/list-users-and-teams-null-count
Open

fix(list_users_and_teams): ignore null users in name results#261
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/list-users-and-teams-null-count

Conversation

@Gujiassh
Copy link

Summary

  • count only non-null users in name-search results for list_users_and_teams
  • treat all-null name-search responses as empty rather than returning an inflated count with an empty list
  • add regression coverage for the execute path with a nullable user response

Testing

  • npx jest src/core/tools/platform-api-tools/list-users-and-teams-tool/list-users-and-teams-tool.test.ts
  • npx eslint src/core/tools/platform-api-tools/list-users-and-teams-tool/list-users-and-teams-tool.ts src/core/tools/platform-api-tools/list-users-and-teams-tool/list-users-and-teams-tool.test.ts
  • npm run build

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
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