variable: call the new /v1/variables routes (#86)

* variable: call the new /v1/variables routes

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.

* deps: @ellipsis-dev/sdk 0.5.0

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.

Author: hbrooks

bun.lock

@@ -20,7 +20,7 @@
     "test:watch": "vitest"
   },
   "dependencies": {
-    "@ellipsis-dev/sdk": "^0.4.0",
+    "@ellipsis-dev/sdk": "^0.5.0",
     "chalk": "^5.6.2",
     "cli-table3": "^0.6.5",
     "commander": "^12.1.0",

package.json

@@ -20,7 +20,7 @@ export function registerVariable(program: Command): void {
       variable.command('list').description('List the variable names (never their values)'),
       'ls',
     ),
-    'GET /v1/sandboxes/variables',
+    'GET /v1/variables',
   )
     .option('--json', 'output raw JSON')
     .action(async (opts: { json?: boolean }) => {
@@ -34,7 +34,7 @@ export function registerVariable(program: Command): void {
     variable
       .command('set [assignments...]')
       .description('Create or update variables, e.g. `set A=1 B=2`'),
-    'PUT /v1/sandboxes/variables',
+    'PUT /v1/variables',
   )
     .option('-f, --from-file <path>', 'load variables from a .env or .json file')
     .option('--json', 'output raw JSON')
@@ -52,7 +52,7 @@ export function registerVariable(program: Command): void {
 
   apiRoutes(
     alsoKnownAs(variable.command('delete <name>').description('Delete a variable'), 'rm'),
-    'DELETE /v1/sandboxes/variables/{name}',
+    'DELETE /v1/variables/{name}',
   )
     .option('--json', 'output raw JSON')
     .action(async (name: string, opts: { json?: boolean }) => {

src/commands/variable.ts

@@ -400,33 +400,28 @@ export class ApiClient {
     return this.request('DELETE', '/v1/defaults', undefined, { repository })
   }
 
-  // -------------------------- sandbox variables ---------------------------
+  // ------------------------------- variables --------------------------------
   // All three return the full current list (the backend echoes it after every
   // mutation), so callers can render the resulting state.
 
   async listSandboxVariables(): Promise<SandboxVariableSummary[]> {
-    const res = await this.request<GetSandboxVariablesResponse>(
-      'GET',
-      '/v1/sandboxes/variables',
-    )
+    const res = await this.request<GetSandboxVariablesResponse>('GET', '/v1/variables')
     return res.variables
   }
 
   async putSandboxVariables(
     variables: SandboxVariableInput[],
   ): Promise<SandboxVariableSummary[]> {
-    const res = await this.request<GetSandboxVariablesResponse>(
-      'PUT',
-      '/v1/sandboxes/variables',
-      { variables },
-    )
+    const res = await this.request<GetSandboxVariablesResponse>('PUT', '/v1/variables', {
+      variables,
+    })
     return res.variables
   }
 
   async deleteSandboxVariable(name: string): Promise<SandboxVariableSummary[]> {
     const res = await this.request<GetSandboxVariablesResponse>(
       'DELETE',
-      `/v1/sandboxes/variables/${encodeURIComponent(name)}`,
+      `/v1/variables/${encodeURIComponent(name)}`,
     )
     return res.variables
   }

src/lib/api.ts

@@ -160,7 +160,7 @@ describe('ApiClient sandbox variables', () => {
 
     const out = await new ApiClient('http://api.test', 't').listSandboxVariables()
     expect(out).toEqual([{ name: 'A', created_at: '', updated_at: '' }])
-    expect(fetchMock.mock.calls[0][0]).toBe('http://api.test/v1/sandboxes/variables')
+    expect(fetchMock.mock.calls[0][0]).toBe('http://api.test/v1/variables')
     expect((fetchMock.mock.calls[0][1] as RequestInit).method).toBe('GET')
   })
 
@@ -172,7 +172,7 @@ describe('ApiClient sandbox variables', () => {
 
     await new ApiClient('http://api.test', 't').putSandboxVariables([{ name: 'TOKEN', value: 'x' }])
     const [url, init] = fetchMock.mock.calls[0]
-    expect(url).toBe('http://api.test/v1/sandboxes/variables')
+    expect(url).toBe('http://api.test/v1/variables')
     expect((init as RequestInit).method).toBe('PUT')
     expect(JSON.parse((init as RequestInit).body as string)).toEqual({
       variables: [{ name: 'TOKEN', value: 'x' }],
@@ -187,7 +187,7 @@ describe('ApiClient sandbox variables', () => {
 
     await new ApiClient('http://api.test', 't').deleteSandboxVariable('MY/VAR')
     const [url, init] = fetchMock.mock.calls[0]
-    expect(url).toBe('http://api.test/v1/sandboxes/variables/MY%2FVAR')
+    expect(url).toBe('http://api.test/v1/variables/MY%2FVAR')
     expect((init as RequestInit).method).toBe('DELETE')
   })
 })