Skip to content

fix(get_notetaker_meetings): ignore null meeting entries#265

Open
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/notetaker-null-meetings
Open

fix(get_notetaker_meetings): ignore null meeting entries#265
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/notetaker-null-meetings

Conversation

@Gujiassh
Copy link

Summary

  • treat all-null meeting arrays the same as an empty notetaker result instead of returning a success payload with null entries
  • count only non-null meetings in the returned pagination metadata
  • add regression coverage for the nullable meetings path

Testing

  • npx jest src/core/tools/platform-api-tools/get-notetaker-meetings-tool/get-notetaker-meetings-tool.test.ts
  • npx eslint src/core/tools/platform-api-tools/get-notetaker-meetings-tool/get-notetaker-meetings-tool.ts src/core/tools/platform-api-tools/get-notetaker-meetings-tool/get-notetaker-meetings-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