variable: call the new /v1/variables routes - #86
Merged
Conversation
The API moved variables out of the sandboxes namespace (ellipsis#6002, live in prod), matching where the CLI verb and the dashboard page already put them. The old paths now 404, so this is the client half of the same move. No fallback to the old paths: variables are pre-customer and this CLI is their only client.
Picks up missing_sandbox_variables in AgentSessionExitStatus. Nothing here narrows to that union yet (exit_status is still a plain string), but the CLI should track the published SDK rather than drift a major behind.
3 tasks
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
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
/v1/variablesinstead of/v1/sandboxes/variables.ellipsis-dev/ellipsis#6002and is already live in prod: the new path serves, the old one 404s. This is the client half of that move.variableverb, its aliases, and its output are unchanged. Only the URLs it calls and theAPI:lines in--helpmove.@ellipsis-dev/sdkto 0.5.0 (published fromellipsis#6003), which addsmissing_sandbox_variablestoAgentSessionExitStatus. Nothing here narrows to that union yet, sinceexit_statusis still typed as a plain string, but the CLI should track the published SDK rather than drift behind.Test plan
bun run typecheckbun run test— 375 passedbun run buildbun build src/cli.tsx --compilethen--version(matches the CI compile-smoke step)agent variable list --helpshowsAPI: GET /v1/variablesagent variable listreturns the account's stored variable, confirming the new route end-to-end