feat(experiment): create and delete developer sandboxes by vegeris · Pull Request #389 · slackapi/slack-cli

vegeris · 2026-03-11T17:29:43Z

Changelog

Experimental Feature: --experiment sandboxes

We're adding support for managing Slack developer sandboxes from within the CLI. We now allow one to create or delete a sandbox from the CLI.

sandbox create allows you to create a new developer sandbox.

sandbox delete allows you to delete an existing developer sandbox.

Summary

This PR adds sandbox create and sandbox delete commands, which allow one to create or delete a sandbox from the CLI.

Follow up to #379

Testing

Try out the commands:

% hermes sandbox create --experiment=sandboxes --name "my-test-box" --password "secretPassword"

% hermes sandbox delete --experiment=sandboxes --sandbox-id E012345

Requirements

I've read and understood the Contributing Guidelines and have done my best effort to follow them.
I've read and agree to the Code of Conduct.

…geris-sandbox-integration

codecov · 2026-03-11T18:41:34Z

Codecov Report

❌ Patch coverage is 83.33333% with 48 lines in your changes missing coverage. Please review.
✅ Project coverage is 68.93%. Comparing base (8b31c6a) to head (b5de063).
⚠️ Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
cmd/sandbox/create.go	84.66%	20 Missing and 5 partials ⚠️
cmd/sandbox/delete.go	73.52%	14 Missing and 4 partials ⚠️
internal/api/sandbox.go	95.83%	1 Missing and 1 partial ⚠️
internal/style/style.go	0.00%	2 Missing ⚠️
cmd/sandbox/sandbox.go	66.66%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #389      +/-   ##
==========================================
+ Coverage   68.62%   68.93%   +0.30%     
==========================================
  Files         218      220       +2     
  Lines       18162    18446     +284     
==========================================
+ Hits        12464    12716     +252     
- Misses       4538     4558      +20     
- Partials     1160     1172      +12

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

…geris-sandbox-integration-create-delete

vegeris · 2026-03-12T19:05:15Z

cmd/sandbox/create.go

+		},
+	}))
+	clients.IO.PrintInfo(ctx, false, "Manage this sandbox from the CLI or visit\n%s", style.Secondary("https://api.slack.com/developer-program/sandboxes"))
+}


Preview:

% hermes sandbox create --experiment=sandboxes --name "EV test box 7245" --password "jhsdjdkhfkdfgkgd" 🏖️ Sandbox Created Team ID: E0197Q6CTV1 URL: https://ev-test-box-7245.enterprise.dev.slack.com/ Manage this sandbox from the CLI or visit https://api.slack.com/developer-program/sandboxes

praise: Excellent formatting choice!

vegeris · 2026-03-12T19:06:53Z

cmd/sandbox/delete.go

+	}
+
+	return nil
+}


Preview:

% hermes sandbox delete --experiment=sandboxes --sandbox-id E0196ATAVGT Choose a Slack team where your email address matches your Slack developer account ? Select a team for authentication slackforceorg E014LMDF01H ⚠️ Danger zone Sandbox (E0196ATAVGT) and all of its data will be permanently deleted This cannot be undone ? Are you sure you want to delete the sandbox? Yes ✅ Sandbox deleted Sandbox E0196ATAVGT has been permanently deleted 🏖️ Developer Sandboxes Owned by Slack developer account ev@mail.com EV test box 8274368 (E0123456) URL: https://ev-test-box-29378.slack.com Status: ACTIVE Created: 2026-03-12 Expires: 2026-09-12 Learn more at https://docs.slack.dev/tools/developer-sandboxes

🫧 thought: This is solid! I'm optimistic that a conclusion of the charm experiment might let us hide selections and warnings earlier for outputs that are focused on the result, but nothing to think about in this PR!

praise: Excellent formatting choice!

cmd/sandbox/create.go

zimeg

@vegeris LGTM! Thanks for bringing a complete sandbox experiment to scripting. I'm excited for these options 🌈 ✨

A few comments that follow are quibbles to implementation but nothing that ought block this from merging. It might be nice to add more tests to API methods, but we can explore some of this in future changes of course 🚢 💨

internal/api/sandbox.go

cmd/sandbox/sandbox.go

zimeg · 2026-03-16T17:06:53Z

internal/shared/types/sandbox.go

+// Sandbox represents a Slack Developer Sandbox
 type Sandbox struct {
-	DateArchived  int64  `json:"date_archived"`   // When the developer sandbox is or will be archived, as epoch seconds
-	DateCreated   int64  `json:"date_created"`    // When the developer sandbox was created, as epoch seconds


🌟 praise: To kind epoch!

cmd/sandbox/create.go

cmd/sandbox/delete.go

zimeg · 2026-03-16T18:33:36Z

cmd/sandbox/delete.go

+	}
+
+	return nil
+}


🫧 thought: This is solid! I'm optimistic that a conclusion of the charm experiment might let us hide selections and warnings earlier for outputs that are focused on the result, but nothing to think about in this PR!

zimeg · 2026-03-16T18:40:07Z

cmd/sandbox/create.go

+	if err := cmd.MarkFlagRequired("name"); err != nil {
+		panic(err)
+	}
+	if err := cmd.MarkFlagRequired("password"); err != nil {
+		panic(err)
+	}


🏁 issue(non-blocking): We might want prompt alternatives for required flags but no blocker for this PR!

I went ahead and added those prompts 👀

zimeg · 2026-03-16T18:41:31Z

cmd/sandbox/create.go

+	}
+
+	cmd.Flags().StringVar(&createCmdFlags.name, "name", "", "Organization name for the new sandbox")
+	cmd.Flags().StringVar(&createCmdFlags.domain, "domain", "", "Team domain (e.g., pizzaknifefight). If not provided, derived from org name")


🍕 praise: Toward common workplace shenanigan IIRC!

I changed this line to just Team domain. If not provided, will be derived from org name, thinking we don't need to give an example of a domain. If it makes the messaging more fun / Slack-y I can bring it back!

zimeg · 2026-03-16T18:45:21Z

cmd/sandbox/create_test.go

+	tests := []struct {
+		name    string


Suggested change

tests := []struct {

name string

tests := map[string]struct {

🧪 quibble: Let's use table tests here! I was confused in thinking "24h" was both the expected case and actual result at a glance...

👾 ramble: And perhaps a bounds of expected archive date might be most confident in testing? Like, some "24h" is greater than now but less than "48h" from now?

Agreed, we have a standard table test format now and all of our other tests have been updated to use it. Let's use it here too.

mwbrooks

✅ Looking good @vegeris! Thank you for adding the sandbox create and sandbox delete commands.

🧪 Local testing works well!

✏️ I've left some suggestions that I'd appreciate you handle before merging this PR. The 🔡 alphabetical ones feel like nits but go a long way for future maintainers who need to read and find content.

cmd/sandbox/create.go

mwbrooks · 2026-03-16T21:54:28Z

cmd/sandbox/create.go

+	cmd.Flags().StringVar(&createCmdFlags.locale, "locale", "", "Locale (eg. en-us, languageCode-countryCode)")
+	cmd.Flags().StringVar(&createCmdFlags.template, "template", "", "Template ID for pre-defined data to preload")
+	cmd.Flags().StringVar(&createCmdFlags.eventCode, "event-code", "", "Event code for the sandbox")
+	cmd.Flags().StringVar(&createCmdFlags.archiveTTL, "archive-ttl", "", "Time-to-live duration; sandbox will be archived at end of day after this period (e.g., 2h, 1d, 7d)")


suggestion: In the Long Description can be list all of the available formats for TTL? 1h, 1d, 1w?, 1m? 1y?, etc

Good point; I also updated the parsing of archiveTTL to only recognize day/week/month; no 'hour' option (though we'll likely look at adding support for these shorter archive times in the future)

cmd/sandbox/create.go

mwbrooks · 2026-03-16T22:09:39Z

cmd/sandbox/create.go

+		},
+	}))
+	clients.IO.PrintInfo(ctx, false, "Manage this sandbox from the CLI or visit\n%s", style.Secondary("https://api.slack.com/developer-program/sandboxes"))
+}


praise: Excellent formatting choice!

internal/api/api_mock.go

internal/api/sandbox.go

Co-authored-by: Eden Zimbelman <eden.zimbelman@salesforce.com>

vegeris · 2026-03-19T15:51:55Z

cmd/sandbox/create.go

+	eventCode   string
+	archiveTTL  string // TTL duration, e.g. 1d, 2h
+	archiveDate string // explicit date yyyy-mm-dd
+	partner     bool


Decided to include support for the partner flag, since we have validation on the backend to make sure only partner developers can create partner sandboxes

cmd/sandbox/create.go

…geris-sandbox-integration-create-delete

vegeris · 2026-03-19T18:46:48Z

cmd/sandbox/create.go

+	}
+	if target.After(maxAllowed) {
+		return 0, slackerror.New(slackerror.ErrInvalidArchiveTTL)
+	}


🏁 I added some basic validation around acceptable archive dates but I'm not sure if we want to keep this in the CI; the backend will error out if an invalid date is provided but the error message isn't particularly user friendly (it's the generic 'invalid_args' response). This at least highlights which arg is invalid

Actually, I think I'll just include validation for 'date cannot be in the past' for the --archive-date flag and that's it. It's not possible to provide a date in the past for --archive-ttl anyway. This just leaves out any extra validation for max archive date (backend will return invalid_args)

vegeris · 2026-03-19T22:01:04Z

I'm going to go ahead and merge but happy to do a follow up if anything else comes up 🚀

vegeris added 2 commits March 9, 2026 13:58

Sandbox management commands

c0b11c5

Merge branch 'main' of https://github.com/slackapi/slack-cli into eve…

eabc9ba

…geris-sandbox-integration

vegeris added the semver:patch Use on pull requests to describe the release version increment label Mar 11, 2026

vegeris force-pushed the evegeris-sandbox-integration-create-delete branch 5 times, most recently from 6a724f1 to 37c62fb Compare March 11, 2026 18:38

vegeris force-pushed the evegeris-sandbox-integration-create-delete branch 2 times, most recently from 98ecc15 to f200970 Compare March 11, 2026 23:43

updates

cea2bcd

vegeris force-pushed the evegeris-sandbox-integration-create-delete branch from f200970 to cea2bcd Compare March 12, 2026 18:56

Merge branch 'main' of https://github.com/slackapi/slack-cli into eve…

b1699bb

…geris-sandbox-integration-create-delete

vegeris marked this pull request as ready for review March 12, 2026 19:00

vegeris requested a review from a team as a code owner March 12, 2026 19:00

vegeris commented Mar 12, 2026

View reviewed changes

update

a35fbc3

vegeris force-pushed the evegeris-sandbox-integration-create-delete branch from b23fbe4 to a35fbc3 Compare March 12, 2026 19:09

vegeris commented Mar 12, 2026

View reviewed changes

cmd/sandbox/create.go Show resolved Hide resolved

zimeg approved these changes Mar 16, 2026

View reviewed changes

zimeg added experiment Experimental feature accessed behind the --experiment flag or toggle enhancement M-T: A feature request for new functionality labels Mar 16, 2026

zimeg added this to the Next Release milestone Mar 16, 2026

zimeg changed the title ~~feat: create and delete developer sandboxes~~ feat(experiment): create and delete developer sandboxes Mar 16, 2026

mwbrooks approved these changes Mar 16, 2026

View reviewed changes

vegeris and others added 3 commits March 18, 2026 17:23

Update cmd/sandbox/sandbox.go

a50fb40

Co-authored-by: Eden Zimbelman <eden.zimbelman@salesforce.com>

update name to domain handling, add unit tests, alphabetize

0b525a5

support partner flag, handle template ID

2e50c5b

vegeris commented Mar 19, 2026

View reviewed changes

cmd/sandbox/create.go Outdated Show resolved Hide resolved

vegeris added 6 commits March 19, 2026 13:04

update archive-ttl support

ca46e13

fix tests, alphabetize

43e9bac

Merge branch 'main' of https://github.com/slackapi/slack-cli into eve…

9851ac7

…geris-sandbox-integration-create-delete

update spacing for emoji

bf34f27

update tests

b1b57a7

lint

6bd1ac5

vegeris commented Mar 19, 2026

View reviewed changes

vegeris added 2 commits March 19, 2026 14:53

update archive date validation

521833b

prompt for required flags

f6b4d21

vegeris force-pushed the evegeris-sandbox-integration-create-delete branch from edcda2b to f6b4d21 Compare March 19, 2026 19:26

vegeris added 2 commits March 19, 2026 16:36

prompt when id missing for delete cmd

ee75a8e

rename err

1b65aa8

vegeris force-pushed the evegeris-sandbox-integration-create-delete branch from cf4078f to 1b65aa8 Compare March 19, 2026 20:39

vegeris added 5 commits March 19, 2026 16:42

rename Sandbox struct fields

9b6634f

reword

74248b8

lint

78376a6

clean up tests

4d7894c

update test

b5de063

vegeris merged commit 55a4069 into main Mar 19, 2026
8 checks passed

vegeris deleted the evegeris-sandbox-integration-create-delete branch March 19, 2026 22:02

Conversation

vegeris commented Mar 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changelog

Summary

Testing

Requirements

Uh oh!

codecov bot commented Mar 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

zimeg left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mwbrooks left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

vegeris Mar 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

vegeris commented Mar 19, 2026

Uh oh!

Uh oh!

vegeris commented Mar 11, 2026 •

edited

Loading

codecov bot commented Mar 11, 2026 •

edited

Loading

vegeris Mar 19, 2026 •

edited

Loading