Skip to content

fix(get_board_activity): ignore null activity logs#273

Open
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/board-activity-null-logs
Open

fix(get_board_activity): ignore null activity logs#273
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/board-activity-null-logs

Conversation

@Gujiassh
Copy link

Summary

  • filter null activity log entries before deciding whether get_board_activity has results
  • keep the existing no-activity message for all-null responses instead of returning an empty success payload
  • add focused regression coverage for both a normal activity response and an all-null activity array

Testing

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