Skip to content

feat(cli): migrate platform commands to thin IPC wrappers #30247

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
noanflaherty merged 1 commit into from
May 11, 2026
Merged

feat(cli): migrate platform commands to thin IPC wrappers #30247

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
50 changes: 50 additions & 0 deletions assistant/openapi.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10490,6 +10490,56 @@ paths:
schema:
type: string
description: Conversation ID (optional)
/v1/platform/callback-routes:
get:
operationId: platform_callbackroutes_get
summary: List registered platform callback routes
description: Lists all callback routes registered with the platform for this assistant.
tags:
- platform
responses:
"200":
description: Successful response
/v1/platform/callback-routes/register:
post:
operationId: platform_callbackroutes_register_post
summary: Register a platform callback route
description: Registers a callback route with the platform gateway for inbound provider webhooks.
tags:
- platform
responses:
"200":
description: Successful response
/v1/platform/connect:
post:
operationId: platform_connect_post
summary: Connect to the Vellum Platform
description: Checks existing credentials and emits the show_platform_login signal for connected clients to show a login UI.
tags:
- platform
responses:
"200":
description: Successful response
/v1/platform/disconnect:
post:
operationId: platform_disconnect_post
summary: Disconnect from the Vellum Platform
description: Deletes stored platform credentials and emits platform_disconnected signal to connected clients.
tags:
- platform
responses:
"200":
description: Successful response
/v1/platform/status:
get:
operationId: platform_status_get
summary: Get platform deployment context and connection status
description: Aggregates platform context, credentials, assistant ID, webhook secret, and Velay tunnel status.
tags:
- platform
responses:
"200":
description: Successful response
/v1/playground/seed-conversation:
post:
operationId: playground_seedconversation_post
Expand Down
6 changes: 3 additions & 3 deletions assistant/src/cli/COMMAND_INVENTORY.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,6 @@ Run `bun run lint:inventory` to validate.
| `oauth/shared` | _(shared utilities, no direct subcommands)_ | `ipc` | _(none — utility module)_ | `LEGACY` | Utility helpers for oauth commands |
| `oauth/status` | _(pending migration)_ | `ipc` | _(pending migration)_ | `LEGACY` | |
| `oauth/token` | _(pending migration)_ | `ipc` | _(pending migration)_ | `LEGACY` | |
| `platform/index` | _(namespace registrar)_ | `ipc` | _(pending migration)_ | `LEGACY` | Registers platform/\* subcommands |
| `platform/connect` | _(pending migration)_ | `ipc` | _(pending migration)_ | `LEGACY` | |
| `platform/disconnect` | _(pending migration)_ | `ipc` | _(pending migration)_ | `LEGACY` | |
| `platform/index` | `status`, `callback-routes register`, `callback-routes list` | `ipc` | `platform_status`, `platform_callback_routes_register`, `platform_callback_routes_list` | `THIN` | Registers platform/\* subcommands |
| `platform/connect` | _(inline in platform/index)_ | `ipc` | `platform_connect` | `THIN` | |
| `platform/disconnect` | _(inline in platform/index)_ | `ipc` | `platform_disconnect` | `THIN` | |
Comment on lines +75 to +76
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot May 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚩 COMMAND_INVENTORY.md status values are non-standard for connect/disconnect

The COMMAND_INVENTORY.md defines three status values: THIN, LOCAL, and LEGACY. The updated entries for platform/connect and platform/disconnect use _(inline in platform/index)_ which doesn't match any defined status. These commands ARE now thin IPC wrappers (they import only from ipc/cli-client.js and cli/output.js), so they should arguably be marked THIN. Additionally, they're not truly "inline" — they live in separate files (connect.ts, disconnect.ts) imported by index.ts. The table is truncated so the full status column isn't visible, but the description column value is misleading.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Loading
Loading