variable: call the new /v1/variables routes

package.json

@@ -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",

src/commands/variable.ts

@@ -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/lib/api.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
   }

test/api.test.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')
   })
 })