Skip to content

fix(fetch_custom_activity): ignore null entries#271

Open
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/custom-activity-null-entries
Open

fix(fetch_custom_activity): ignore null entries#271
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/custom-activity-null-entries

Conversation

@Gujiassh
Copy link

Summary

  • filter null entries out of fetch_custom_activity results before formatting them
  • treat all-null custom activity arrays as an empty result instead of returning a malformed success payload
  • add focused regression coverage for both the success path and the all-null path

Testing

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