Skip to content

Defer schedule catchup window default to server#1646

Draft
chaptersix wants to merge 1 commit into
mainfrom
codex/schedule-catchup-window-default
Draft

Defer schedule catchup window default to server#1646
chaptersix wants to merge 1 commit into
mainfrom
codex/schedule-catchup-window-default

Conversation

@chaptersix

Copy link
Copy Markdown

What changed

  • Change the default SchedulePolicy.catchup_window from an SDK-defined year to None.
  • Omit the protobuf field when it is unset.
  • Preserve absent fields as None when decoding.
  • Add serialization and server-backed default coverage.
  • Update the changelog.

Why

The server owns the default. Materializing one year in the SDK prevents server-side default changes from applying consistently.

Validation

  • Ruff lint and format checks
  • Pyright: zero errors
  • Runtime pytest was not run locally because Rust is unavailable for building the Python bridge.

Breaking changes

An unconfigured policy now exposes None before it is sent to the server. Described schedules continue to expose the server-resolved duration.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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