Skip to content

refactor(tool)!: rename date-picker type to date-range - #795

Merged
hjlarry merged 1 commit into
langgenius:mainfrom
hjlarry:agent/rename-date-picker-to-date-range
Aug 7, 2026
Merged

refactor(tool)!: rename date-picker type to date-range#795
hjlarry merged 1 commit into
langgenius:mainfrom
hjlarry:agent/rename-date-picker-to-date-range

Conversation

@hjlarry

@hjlarry hjlarry commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • rename the tool parameter constant from DATE_PICKER to DATE_RANGE
  • change the accepted manifest value from date-picker to date-range
  • cover acceptance of date-range and rejection of the legacy value in declaration tests

Why

The parameter carries a { start, end } date range. Naming the wire type date-range aligns it with the existing date type and avoids coupling the protocol to a picker UI.

Breaking change

This intentionally removes support for date-picker without a compatibility alias. Plugin manifests using the earlier value must migrate to date-range.

Related

Validation

  • go test ./pkg/entities/plugin_entities
  • go test ./pkg/entities/...
  • gofmt
  • git diff --check

@hjlarry
hjlarry marked this pull request as ready for review August 7, 2026 02:36
@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. go Pull requests that update go code labels Aug 7, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Aug 7, 2026
@hjlarry
hjlarry merged commit 1310a18 into langgenius:main Aug 7, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go Pull requests that update go code lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants