Docs/migration 1x to 2x - #655
Conversation
31f5f5e to
162361b
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
162361b to
e4ae074
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Add docs/migration-1.x-to-2.x.md covering the 2.x breaking changes: typed per-operation error hierarchy, first-run serialize/deserialize round trip, removed config types and fields, construction-time config validation, and wait_for_condition changes; plus the optional should_complete feature. Also document the initial_state serdes requirement on WaitForConditionConfig.
e4ae074 to
2681198
Compare
Codex AI reviewFound one blocking documentation correctness issue. No runtime behavior changed; tests were not run under the read-only review constraint. Reviewed commit |
Claude AI reviewThis is a documentation PR: a v1→v2 migration guide plus a docstring expansion on Almost everything checks out: the error hierarchy and its package-root exports ( One confirmed finding (see inline): the "New in 2.x: Custom Completion Predicate" section documents an API that does not exist in this repository — Residual note: this is a docs-only change, so there are no code tests to add; correctness here means the documented API matches the shipped SDK. Reviewed commit |
Issue #, if available: N/A
Description of changes:
Migration guide for python v2
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.