chore: regenerate against the v1.7.8 API surface - #178
Merged
Conversation
One behavioural change and two documentation ones. StorageSummary.total_gb_hours is removed. The server retired it in robosystems#1107: it was a storage-billing unit, and there is no storage billing — the overage column is never written, the billing config has no storage entries, and storage is included in the tier. `avg_storage_gb` is now documented as time-weighted, which is what it has become. Note on versioning: this is a removal from a response model, which the 1.0 contract would normally make a major. It rides as a patch because the server change has already shipped — the field is gone from the API, so a client still declaring it would be describing a response it can never receive. There is nothing to deprecate-then-remove; the client is catching up to the wire, not initiating the break. The other two are docstring-only, from robosystems#1105: restore is allowed on entity *subgraphs*, which are written directly and have no materialization path, so a subgraph has no other recovery route. Gate: 519 passed, ruff + format + basedpyright clean.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
One behavioural change, two documentation ones.
StorageSummary.total_gb_hoursremovedThe server retired it in robosystems#1107. It was a storage-billing unit, and there is no storage billing — the overage column is never written, the billing config has no storage entries, and storage is included in the tier. The field described a mechanism that had been removed.
avg_storage_gbis now documented as time-weighted, which is what it became: each reading weighted by the span since the previous one, so an irregular snapshot cadence doesn't skew it.Docstrings only
From robosystems#1105: restore is allowed on entity subgraphs — they're written directly and have no materialization path, so a subgraph has no other recovery route. Touches
restore_backupandbackup_list_response.On the version
This removes a field from a response model, which the 1.0 contract would normally make a major. It rides as a patch deliberately:
The server change already shipped. The field is gone from the API, so a client still declaring it would be describing a response it can never receive. There's nothing to deprecate-then-remove — the client is catching up to the wire, not initiating the break. A deprecation cycle only makes sense when you control removal timing on both sides.
Worth knowing rather than assuming: if you'd want strictness here, the lever is server-side sequencing — deprecate the field in the API first, then remove it a cycle later, and the client follows in step.
No known consumer reads it (
robosystems-appgreps clean).Gate
519 passed, 17 skipped.
ruff check+ruff format --check+basedpyrightall clean.