feat: remove the experiment flag for the `sandbox` commands and add new partner template IDs by vegeris · Pull Request #512 · slackapi/slack-cli

vegeris · 2026-04-28T19:22:03Z

Changelog

This is effectively the release of the sandbox commands and should be highlighted in the changelog:

slack sandbox create
slack sandbox delete
slack sandbox list

Summary

This PR removes the experiment flag currently required to run the sandbox commands

Testing

Instead of:

lack sandbox list --experiment=sandboxes

We can run:

lack sandbox list

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.

codecov · 2026-04-28T19:25:57Z

Codecov Report

❌ Patch coverage is 20.00000% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.27%. Comparing base (d2ab928) to head (508df39).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
internal/prompts/team_select.go	0.00%	7 Missing ⚠️
cmd/sandbox/sandbox.go	50.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #512   +/-   ##
=======================================
  Coverage   71.26%   71.27%           
=======================================
  Files         222      222           
  Lines       18682    18665   -17     
=======================================
- Hits        13314    13303   -11     
+ Misses       4187     4184    -3     
+ Partials     1181     1178    -3

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

vegeris · 2026-04-29T14:29:28Z

At line 80 there was a recommendation to use charm for the 'Choose a Slack team where your email address matches...' info message; is charm stable yet, such that we can make that change?

Huh (Charm) is now the default and only prompt library. So, if you're using a prompt then you're now using Charm's Huh :)

yea perhaps we get rid of that comment?

Ah, gotcha :) Updated to use the 'Help' config for the prompt instead of a separate log statement

vegeris · 2026-04-29T14:31:07Z

+	"it-incident-management": 4,
+	"customer-support":       5,
+	"sales":                  6,
+	"marketing":              7,


This PR also adds mappings for the new partner templates

Thanks @vegeris. In the future, it would be better for this to be a separate PR so that it can stand out in the GitHub Release Notes and Changelog.

suggestion: Can you please update the Changelog section of this PR Description so that the documentation team knows to document it on docs.slack.dev.

suggestion: @vegeris Can you please add these to the Long Description so that developers can discover the partner template Ids that are available to use?

I wasn't sure if we would want to highlight it in the Long Description as the grand majority of users would be regular non-partner developers; for the list command for example, we only add the 'sandbox type' label to partner sandboxes to reduce visual noise for regular sandboxes

document it on docs.slack.dev

Speaking of, I'd like to double check the process for getting new commands documented. My current assumption was that we would have a brief announcement in the Changelog through this PR and that once these changes are included in a release, that I would post in #api-docs to get the commands documented on docs.slack.dev?

Also, if there's a strong preference I can apply the changes unrelated to removing the experiment flag first and then come back to this PR

mwbrooks

🙏🏻 Thanks for removing the experiment @vegeris

✏️ I've left a few change request suggestions. In the future, I'd encourage the PR to only remove the experiment and add other additions in separate PRs. It allows both the documentation team to spot changes easier and our GitHub Release Notes (based on PR titles) to communicate the changes to our developers.

mwbrooks · 2026-04-29T16:08:45Z

+	"it-incident-management": 4,
+	"customer-support":       5,
+	"sales":                  6,
+	"marketing":              7,


Thanks @vegeris. In the future, it would be better for this to be a separate PR so that it can stand out in the GitHub Release Notes and Changelog.

suggestion: Can you please update the Changelog section of this PR Description so that the documentation team knows to document it on docs.slack.dev.

mwbrooks · 2026-04-29T16:10:37Z

+	"it-incident-management": 4,
+	"customer-support":       5,
+	"sales":                  6,
+	"marketing":              7,


suggestion: @vegeris Can you please add these to the Long Description so that developers can discover the partner template Ids that are available to use?

mwbrooks · 2026-04-29T16:12:05Z

Huh (Charm) is now the default and only prompt library. So, if you're using a prompt then you're now using Charm's Huh :)

mwbrooks · 2026-04-29T16:13:30Z

 	ErrDotEnvFileRead                                = "dotenv_file_read_error"
 	ErrDotEnvFileWrite                               = "dotenv_file_write_error"
 	ErrDotEnvVarMarshal                              = "dotenv_var_marshal_error"
+	ErrDomainLong                                    = "domain_long"


suggestion: Can this please be prefixed with ErrSandboxDomainLong = "sandbox_domain_long" so that we know it's part of the Sandbox error codes. You'll need to reorganize it alphabetically.

💯 Updated this error code and alphabetized the sandbox error codes added previously!

rm experiment flag

b9cf57f

vegeris added 2 commits April 29, 2026 10:20

incl partner template names, new err code msg

ce63d49

Merge branch 'main' into evegeris-rm-sandbox-experiment

2342ef9

vegeris commented Apr 29, 2026

View reviewed changes

vegeris marked this pull request as ready for review April 29, 2026 14:30

vegeris requested a review from a team as a code owner April 29, 2026 14:30

vegeris commented Apr 29, 2026

View reviewed changes

vegeris added the semver:minor Use on pull requests to describe the release version increment label Apr 29, 2026

srtaalej added the experiment Experimental feature accessed behind the --experiment flag or toggle label Apr 29, 2026

srtaalej assigned vegeris Apr 29, 2026

mwbrooks added this to the Next Release milestone Apr 29, 2026

mwbrooks changed the title ~~feat: Remove the experiment flag for the sandbox commands~~ feat: remove the experiment flag for the sandbox commands Apr 29, 2026

mwbrooks requested changes Apr 29, 2026

View reviewed changes

mwbrooks changed the title ~~feat: remove the experiment flag for the sandbox commands~~ feat: remove the experiment flag for the sandbox commands and add new partner template IDs Apr 29, 2026

mwbrooks added enhancement M-T: A feature request for new functionality changelog Use on updates to be included in the release notes labels Apr 29, 2026

vegeris added 2 commits April 30, 2026 09:25

alphabetize sandbox errors

9c99351

update info log

508df39

vegeris requested a review from mwbrooks April 30, 2026 19:14

Conversation

vegeris commented Apr 28, 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 Apr 28, 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!

mwbrooks Apr 29, 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 Apr 30, 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!

mwbrooks left a comment

Choose a reason for hiding this comment

Uh oh!

mwbrooks Apr 29, 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!

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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

vegeris commented Apr 28, 2026 •

edited

Loading

codecov Bot commented Apr 28, 2026 •

edited

Loading

mwbrooks Apr 29, 2026 •

edited

Loading

vegeris Apr 30, 2026 •

edited

Loading

mwbrooks Apr 29, 2026 •

edited

Loading