Skip to content

fix(client): align date filter inputs with api#77

Merged
cjumel merged 1 commit into
mainfrom
cursor/platform-sdk-sync-python-date-filters-20260531
Jun 1, 2026
Merged

fix(client): align date filter inputs with api#77
cjumel merged 1 commit into
mainfrom
cursor/platform-sdk-sync-python-date-filters-20260531

Conversation

@Shedna
Copy link
Copy Markdown
Contributor

@Shedna Shedna commented May 31, 2026

Description

The public gateway accepts fromDate and toDate as ISO date strings and also normalizes full ISO datetime strings on search and research, but the Python client surface still typed those parameters as date only.

This PR synchronizes the Python SDK by:

  • updating search / async_search / research / async_research to accept date | str | None,
  • documenting the accepted date-filter formats in client docstrings and the README,
  • adding focused tests that cover ISO datetime string payloads for search and research.

Checklist

  • I have installed prek on this project (for instance with the make install-dev command)
    before creating any commit, or I have run successfully the make lint command on my changes.
  • I have run successfully the make typecheck test command on my changes.
  • I have updated the README.md if my changes affected it.

@cjumel cjumel force-pushed the cursor/platform-sdk-sync-python-date-filters-20260531 branch from 132ac18 to ea4b5f0 Compare June 1, 2026 10:50
@cjumel cjumel self-requested a review June 1, 2026 10:51
Copy link
Copy Markdown
Contributor

@cjumel cjumel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the tracking purposes: the PR just updated the README whereas it was not relevant, so I fixed that and updated the AGENTS.md to try and prevent such updates

@cjumel cjumel merged commit 7b64243 into main Jun 1, 2026
3 checks passed
@cjumel cjumel deleted the cursor/platform-sdk-sync-python-date-filters-20260531 branch June 1, 2026 10:53
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.

2 participants