Skip to content

variable: promote variables out of sandbox into a top-level command - #85

Merged
hbrooks merged 1 commit into
mainfrom
hunter/variable-top-level
Jul 27, 2026
Merged

variable: promote variables out of sandbox into a top-level command#85
hbrooks merged 1 commit into
mainfrom
hunter/variable-top-level

Conversation

@hbrooks

@hbrooks hbrooks commented Jul 27, 2026

Copy link
Copy Markdown
Member

Summary

  • agent sandbox variable ... becomes agent variable .... The sandbox parent command is gone; it held nothing else.
  • Hidden aliases unchanged: variables, var, env, plus ls/rm on the leaves.
  • variable replaces sandbox in the Platform group of TOP_LEVEL_GROUPS, so it does not fall through to "Other".
  • README and the ellipsis skill updated to the new spelling.
  • src/commands/sandbox.ts and its test renamed to variable.ts. No logic changes.

Variables are an account-wide store, not a property of a sandbox, so the extra level was pure typing. This matches the dashboard, where variables is now its own page rather than a tab under Sandboxes.

No redirect from the old path, matching the frontend call: variables are pre-customer.

The three REST routes are untouched (GET and PUT on /v1/sandboxes/variables, DELETE /v1/sandboxes/variables/{name}). The CLI now says variable while the API still says sandboxes/variables. Worth a follow-up in the mono repo if we want them to agree.

Test plan

  • npm run typecheck
  • npm test (375 pass)
  • agent --help shows variable under Platform
  • agent variable list returns the real table
  • Hidden aliases resolve: agent variables ls, agent var set
  • Leaf --help still prints the API route lines
  • Note for the reviewer: a removed top-level command silently becomes a session prompt via the session start --connect shorthand. So agent sandbox ... now starts a session instead of erroring.

Variables are an account-wide store, not a property of a sandbox, and
`sandbox` held nothing else, so the extra level was pure typing. Matches
the dashboard, where variables is now its own page rather than a tab
under Sandboxes.

No redirect from the old path: variables are pre-customer.
@hbrooks
hbrooks merged commit 1b8e0ff into main Jul 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant