fix(sheets): add is_column_right field to Sheet model - #484
Conversation
Adds the isColumnRight boolean field introduced by api-worksheet PR #350.
Code ReviewIssues Found
Summary1 issue found. Low severity — title-only; the code change itself is correct. |
@claude A field addition isn't new capability - it's correcting an incomplete model. The field already exists in the API response; the SDK was just missing it. feat is for new methods or parameters. Backfilling a missing field is fix. |
|
Claude finished @prasad-albert's task in 21s —— View job That's a fair point and I agree with the reasoning. The field already exists in the API response — the SDK was silently dropping it, which is an incomplete/incorrect model representation. Backfilling a missing field to match the existing API contract is correcting a defect, not introducing new capability. I'm retracting that issue. The code change is correct and the title is appropriate. |
Summary
is_column_right: bool | None(aliasisColumnRight) to theSheetmodel insrc/albert/resources/sheets.pyNoneso existing sheets without the field deserialize cleanly