Skip to content

fix(get_updates): ignore null update entries#266

Open
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/get-updates-null-guard
Open

fix(get_updates): ignore null update entries#266
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/get-updates-null-guard

Conversation

@Gujiassh
Copy link

Summary

  • treat all-null update arrays the same as an empty updates result instead of letting null entries fall through to formatting
  • filter update rows before deciding whether to return the standard no-updates message
  • add regression coverage for the nullable updates path

Testing

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