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

+ },
+	}))
+	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

+	}
+
+	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!

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 🚢 💨

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

+// 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!

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

+	}
+
+	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

+	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.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

+	tests := []struct {
+ name string


Suggested change
tests:= []struct {
namestring
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.

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

+	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)

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

+ },
+	}))
+	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!

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

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

+	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

…geris-sandbox-integration-create-delete

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

+	}
+	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' ofhttps://github.com/slackapi/slack-cliinto 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 37c62fbCompare March 11, 2026 18:38

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

updates
cea2bcd

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

Merge branch 'main' ofhttps://github.com/slackapi/slack-cliinto 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 a35fbc3Compare March 12, 2026 19:09

vegeris commented Mar 12, 2026
View reviewed changes

Comment threadcmd/sandbox/create.go

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 sandboxesMar 16, 2026

mwbrooks approved these changes Mar 16, 2026
View reviewed changes

vegerisand 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

Comment threadcmd/sandbox/create.go Outdated

vegeris added 6 commits March 19, 2026 13:04

update archive-ttl support
ca46e13

fix tests, alphabetize
43e9bac

Merge branch 'main' ofhttps://github.com/slackapi/slack-cliinto 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 f6b4d21Compare 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 1b65aa8Compare 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 mainMar 19, 2026
8 checks passed

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

zimeg mentioned this pull request Apr 20, 2026
WIP: Sandbox management commands #375
Closed
2 tasks

Uh oh!

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!

codecovBot 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!

vegerisMar 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!

vegeris commented Mar 11, 2026•
edited
Loading

codecovBot commented Mar 11, 2026•
edited
Loading

vegerisMar 19, 2026•
edited
Loading