Skip to content

fix(get_updates): treat date-only toDate as end of day#258

Open
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/get-updates-date-only-todate
Open

fix(get_updates): treat date-only toDate as end of day#258
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/get-updates-date-only-todate

Conversation

@Gujiassh
Copy link

Summary

  • normalize date-only toDate values in get_updates to the end of the requested day
  • keep date-only fromDate normalization at the start of the day
  • update the co-located Jest coverage to lock the inclusive end-date behavior

Testing

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