Skip to content

chore(sdk): regenerate — three operation summaries lost a parenthetical - #214

Merged
jfrench9 merged 1 commit into
mainfrom
chore/api-summary-wrapping
Sep 18, 2026
Merged

jfrench9 merged 1 commit into
mainfrom
chore/api-summary-wrapping

Conversation

@jfrench9

Copy link
Copy Markdown
Member

Summary

Regenerated against the API for robosystems#1428. Three RoboLedger operations carried a parenthetical in their OpenAPI summary that merely repeated the opening clause of their own description, so it was dropped upstream:

Before After
Execute Event Block (publish to source-of-truth system) Execute Event Block
Terminate Schedule Early (No Entry) Terminate Schedule Early
Auto-Map Elements via AI (async) Auto-Map Elements via AI

A summary is the API reference page's <h1> and its nav label; the first was 55 characters and wrapped to two lines. Nothing is lost, because each description already opened with the same statement — and in the first case names QuickBooks, which the summary did not.

Changes

Three files under robosystems_client/api/extensions_robo_ledger/. The summary is the first line of each generated function docstring, and each operation has four (sync, sync_detailed, asyncio, asyncio_detailed) — so 12 docstring lines, and nothing else.

Breaking Changes

None. summary is descriptive metadata: no signature, parameter, model, return type or error mapping moves, and no symbol resolution changes. Generated tier, and cosmetic within it — this is not even a surface removal, so it needs no deprecation note.

Deploy Notes

Merge after robosystems#1428, so the clients never describe a spec that is not on main. Nothing here needs a release of its own — it can ride whenever the next one is cut. (Note the two SDKs still carry the merged-but-unreleased Plaid regeneration from earlier; this simply joins it.)

Testing

just test-all — ruff, ruff format, basedpyright clean; 591 tests pass, 17 skipped.

The diff was read line by line before committing to confirm it is confined to the three summaries and contains no other regeneration drift.

Certification

  • I have the right to submit this work under the Apache 2.0 license, and do so. Where any part of it is owned by my employer, I have their permission.

🤖 Generated with Claude Code

https://claude.ai/code/session_011KBrhQHzwrB39C4c4Vw9Pq

Docstrings only. Three RoboLedger operations carried a parenthetical in
their OpenAPI summary that merely repeated the opening clause of their own
description, so it was dropped upstream (robosystems#1428):

  Execute Event Block (publish to source-of-truth system) -> Execute Event Block
  Terminate Schedule Early (No Entry)                     -> Terminate Schedule Early
  Auto-Map Elements via AI (async)                         -> Auto-Map Elements via AI

The generated summary line is the first line of each function's docstring,
so that is the whole diff — no signature, model, parameter or return type
moves. Generated tier, and cosmetic within it.

Claude-Session: https://claude.ai/code/session_011KBrhQHzwrB39C4c4Vw9Pq
@jfrench9
jfrench9 merged commit afd42e2 into main Sep 18, 2026
4 checks passed
@jfrench9
jfrench9 deleted the chore/api-summary-wrapping branch September 18, 2026 01:34
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