Skip to content

fix: accept metadata without retired capacity fields - #47

Merged
thestephenromano merged 1 commit into
mainfrom
fix/rate-card-metadata
Nov 5, 2026
Merged

thestephenromano merged 1 commit into
mainfrom
fix/rate-card-metadata

Conversation

@thestephenromano

@thestephenromano thestephenromano commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

The API's rate-card correction removes units_charged and compute_schedule_version from public responses. Requiring them makes otherwise valid pricing responses fail SDK validation.

Accept both fields as optional, deprecated compatibility data for older deployments. Commercial evidence remains exclusively fcu_quoted / fcu_charged; there is no fallback conversion. Sync and async tests exercise the new response shape. The pinned deployed contract and historical fixtures remain unchanged for this compatibility preparation.

Validation: 539 tests passed, 18 opt-in integration tests skipped; Ruff lint/format and mypy passed. GitHub CI passed on Python 3.11 and 3.14, including the Docker notebook check.

Companion to API #155 and API MR !210. Independent of solver PR #46. Draft: coordinate with the API rollout; do not merge/release as part of solver work.

@thestephenromano
thestephenromano marked this pull request as ready for review November 5, 2026 14:28
@thestephenromano
thestephenromano merged commit 5547614 into main Nov 5, 2026
8 checks passed
@thestephenromano
thestephenromano deleted the fix/rate-card-metadata branch November 5, 2026 14:28
thestephenromano added a commit that referenced this pull request Nov 5, 2026
Release Please generated this release from merged changes. Its unsigned
commit was rejected by the signed-commit rule; the identical tree was
signed locally and pushed without changing any generated version or
changelog content.

##
[0.2.0](v0.1.1...v0.2.0)
(2026-11-05)


### ⚠ BREAKING CHANGES

* **client:** align Python SDK with API v0.5.0
([#41](#41))

### Features

* **client:** align Python SDK with API v0.5.0
([#41](#41))
([043287a](043287a))


### Bug Fixes

* accept metadata without retired capacity fields
([#47](#47))
([5547614](5547614))


### Documentation

* adopt strategic pragmatism guidance
([#42](#42))
([7484eb2](7484eb2))


Companion to API !210. Solver PR #46 remains excluded.
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