chore(sdk): regenerate from API 1.10.x - #194
Merged
Merged
Conversation
Generated tier: the REST MCP tool surface (list_mcp_tools, call_mcp_tool and their four models) is removed, following its removal from the API in 1.10.2; the connected-apps operations (list/revoke OAuth grants) and the reconciling-item operations (preview/resolve) are added, with the reconciling-item fields on FiscalCalendarResponse and the allow_reconciling_items flag on close-period and backfill-plan-history. The GraphQL schema snapshot picks up reconcilingItemCount and reconcilingItemSample. Stable tier untouched: every symbol the integration template imports still resolves, and no facade referenced the removed operations. Curated release notes for the minor are in .github/release-notes/v1.12.0.md.
Member
Author
|
@claude please review this PR |
|
Claude encountered an error after 2s —— View job I'll analyze this and get back to you. |
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
1 participant
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.
Summary
Regenerates the SDK against the API's 1.10 line (1.10.0–1.10.4 plus the OAuth connected-apps endpoints landing in robosystems #1275). The client was last generated from 1.9.x. Net effect on the public surface: the REST MCP tool pair leaves
api/mcp, the connected-apps and reconciling-item operations arrive, two close operations gain a flag, and the GraphQL schema snapshot picks up two fields. Curated release notes for the minor are included.Changes
robosystems_client/apiandmodels— removed:api/mcp/(list_mcp_tools,call_mcp_tool) and the modelsMCPToolCall,MCPToolCallArguments,MCPToolsResponse,MCPToolsResponseToolsItem(the API droppedGET …/mcp/toolsandPOST …/mcp/call-toolin 1.10.2). Added:api/user/list_user_o_auth_grants,revoke_user_o_auth_grant(+OAuthGrantInfo,OAuthGrantsResponse);api/extensions_robo_ledger/preview_reconciling_item,resolve_reconciling_item(+ the reconciling-item plan/delta/catch-up/request/response models and their operation envelopes). Widened:FiscalCalendarResponsegainsreconciling_item_count/reconciling_item_sample;ClosePeriodOperationandBackfillPlanHistoryOperationgainallow_reconciling_items;DeleteSubgraphOp.backup_firstandcreate_event_blockcarry richer descriptions.models/__init__.pyfollows.graphql/schema.graphql(+10 lines:reconcilingItemCount,reconcilingItemSampleon the fiscal calendar) and regenerated the ariadne models from it, so the CI drift gate stays clean..github/release-notes/v1.12.0.md— the contract document for the minor: names the generated-tier removal and its replacement (the Streamable HTTP transport), and describes the additions.No hand-written code changed. Facades and root exports are untouched.
Compatibility
Under the two-tier rule (CLAUDE.local.md § Versioning): ADDITIVE on the stable tier — with one generated-tier removal (
api.mcp.*), which rides a minor and is named in the release notes. Checked by hand against the integration template's emit path:AuthenticatedClient,types.UNSET, all six emit operations (create_file_upload,ingest_file,materialize,create_event_block,assert_metrics,create_taxonomy_block) and their request models resolve in the regenerated tree; no facade referenced the removed operations. The removal meets the dead-surface test: the endpoints no longer exist upstream, and nothing in the stable tier or the template imported them. Intended version: 1.12.0 (the notes file is named for it; the bump itself stays with the release dispatch).Testing
just generate-sdk(REST + schema refresh) andjust generate-graphqlagainst the local stack running robosystemsmain+ #1275 (verified the two grants operations and nomcppaths in itsopenapi.jsonbefore generating).just test-all— passes: 526 tests (17 skipped), ruff check and format clean, basedpyright 0 errors. The pre-commit hook re-ran the same gate on commit.