Description
Raised in review of #194 (#151).
Problem
The legacy @aictrl/sdk client in packages/sdk/src/gen is frozen and is not regenerated: script/build.ts only regenerates src/v2/gen, and #166 notes that the headless OpenAPI has no paths. It still exports surface for subsystems that the CLI no longer has:
-
packages/sdk/src/gen/sdk.gen.ts: thePtyclient class (list/create/get/update/remove/connect, for/ptyand/pty/{id}). No CLI server route has served these, even before chore: remove dead pty, ide, upgrade and opentui leftovers (#151) #194. -
packages/sdk/src/gen/types.gen.ts:EventInstallationUpdatedandEventInstallationUpdateAvailable(lines ~10-24)Pty, plusEventPtyCreated/Updated/Exited/Deleted(~627-663)- their entries in the
Eventunion (~673-703)
chore: remove dead pty, ide, upgrade and opentui leftovers (#151) #194 removes the only code that published these events (
src/cli/upgrade.ts, which had no importers, andsrc/pty), so nothing ever emitted them.
The regenerated src/v2/gen already has none of this.
Why it was not done in #194
createAictrlClient from the root @aictrl/sdk export uses the legacy client, which is also the client handed to plugins. Removing exported types and classes is a public type change for @aictrl/sdk and needs its own version bump. It fits with the regeneration and release in #166.
Acceptance
- The
Ptyclass and thePty*request/response types are gone fromsrc/gen/sdk.gen.tsandsrc/gen/types.gen.ts. Alternatively, the legacysrc/genclient is removed in favour of v2. EventInstallation*,EventPty*andPtyare gone from theEventunion.bun turbo typecheckpasses, and the@aictrl/sdkversion bump matches the impact.
Activity
byapparov commentedon Oct 17, 2026
Tracked together with the SDK regeneration in #166, so the published SDK changes once.