feat: add optional filepath argument to run command by zimeg · Pull Request #417 · slackapi/slack-cli

zimeg · 2026-03-19T06:46:22Z

Changelog

The run command now accepts an optional argument for the app entry point:
$ slack run ./src/app.py

Summary

This PR adds an optional filepath argument to the run command.

Preview

Current and continued behavior:

With a path argument provided:

When the file doesn't exist:

Reviewers

The following commands shows the optional argument of a filepath for the run command:

$ slack create asdf -t slack-samples/bolt-js-starter-template
$ cd asdf
$ slack run         # No argument defaults to the start hook
$ slack run app.js  # The common path appears to be alright

$ mkdir src
$ mv app-oauth.js src

$ slack run src/app-oauth.js  # Find a SDK runtime error!
$ slack run src/example.js    # Find the file not found

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.

…nto zimeg-fix-env-quote

codecov · 2026-03-19T06:49:35Z

Codecov Report

❌ Patch coverage is 57.89474% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.29%. Comparing base (a841285) to head (ef1b0af).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
internal/pkg/platform/localserver.go	0.00%	7 Missing ⚠️
internal/pkg/platform/run.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #417      +/-   ##
==========================================
+ Coverage   70.28%   70.29%   +0.01%     
==========================================
  Files         220      220              
  Lines       18484    18497      +13     
==========================================
+ Hits        12992    13003      +11     
- Misses       4318     4320       +2     
  Partials     1174     1174

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

zimeg

🌚 A few thoughts of these changes shared for kind reviewers!

zimeg · 2026-03-19T07:05:19Z

cmd/platform/run.go

+	var appPath string
+	if len(args) > 0 {
+		appPath = args[0]
+		if _, err := clients.Fs.Stat(appPath); err != nil {
+			return slackerror.New(slackerror.ErrNotFound).
+				WithMessage("The app path %q could not be found", appPath).
+				WithRemediation("Check that the file exists and the path is correct")
+		}
+	}


Suggested change

var appPath string

if len(args) > 0 {

appPath = args[0]

if _, err := clients.Fs.Stat(appPath); err != nil {

return slackerror.New(slackerror.ErrNotFound).

WithMessage("The app path %q could not be found", appPath).

WithRemediation("Check that the file exists and the path is correct")

}

}

🪓 note: I'm curious of removing this in favor of leaving errors to the hook implementations. Removing this might let for strange argument parsing as perhaps:

$ slack run localhost

👾 ramble: For now it seems to make a better experience for the intended filepath arguments I think-

thought: Interesting idea to leave this to the framework. I think for now, we should leave this in the CLI. We can loosen it later if there is a feature request, but this starts with ensuring an error-free experience. If developers are running a build (e.g. TypeScript: dist/index.js) then they'll need to start their watch server or build a dist before starting the Slack CLI.

zimeg · 2026-03-19T07:06:25Z

cmd/platform/run.go

+		Args:    cobra.MaximumNArgs(1),
 		Example: style.ExampleCommandsf([]style.ExampleCommand{
 			{Command: "platform run", Meaning: "Start a local development server"},
-			{Command: "platform run --activity-level debug", Meaning: "Run a local development server with debug activity"},


🗣️ note: IIRC this activity level flag isn't used often so we might encourage examples that outline another common use case?

👾 ramble: I'm now curious if this flag updates a LOG_LEVEL environment variable for run command?

I agree, we can probably remove it as an example since it's Deno/ROSI specific as well.

thought: In the future, we will probably introduce a --log-level flag that the CLI will use and pass into Bolt Frameworks. The same log level should be passed into the Deno SDK and could be inherited by --activity-level for Deno apps.

The end result would be a universal log-level flag that works for all frameworks and ROSI.

cmd/platform/run.go

mwbrooks

✅ Woohoo! This is an exciting and cool addition, thanks for adding it @zimeg!

🧪 Works beautifully with local tests. 🎉

📝 Left a minor nit about renaming appPath to appFilePath for clarity. Pedantic, but the clarity can avoid confusion for developers reading the code with refresh eyes.

cmd/platform/run.go

mwbrooks · 2026-03-20T21:50:26Z

cmd/platform/run.go

+		Args:    cobra.MaximumNArgs(1),
 		Example: style.ExampleCommandsf([]style.ExampleCommand{
 			{Command: "platform run", Meaning: "Start a local development server"},
-			{Command: "platform run --activity-level debug", Meaning: "Run a local development server with debug activity"},


I agree, we can probably remove it as an example since it's Deno/ROSI specific as well.

thought: In the future, we will probably introduce a --log-level flag that the CLI will use and pass into Bolt Frameworks. The same log level should be passed into the Deno SDK and could be inherited by --activity-level for Deno apps.

The end result would be a universal log-level flag that works for all frameworks and ROSI.

cmd/platform/run.go

mwbrooks · 2026-03-20T21:55:25Z

cmd/platform/run.go

+	var appPath string
+	if len(args) > 0 {
+		appPath = args[0]
+		if _, err := clients.Fs.Stat(appPath); err != nil {
+			return slackerror.New(slackerror.ErrNotFound).
+				WithMessage("The app path %q could not be found", appPath).
+				WithRemediation("Check that the file exists and the path is correct")
+		}
+	}


thought: Interesting idea to leave this to the framework. I think for now, we should leave this in the CLI. We can loosen it later if there is a feature request, but this starts with ensuring an error-free experience. If developers are running a build (e.g. TypeScript: dist/index.js) then they'll need to start their watch server or build a dist before starting the Slack CLI.

cmd/platform/run.go

mwbrooks · 2026-03-20T21:56:35Z

docs/reference/hooks/index.md

 All Bolt SDKs leverage this `start` hook operating mode.

-A custom start path can be set with the `SLACK_CLI_CUSTOM_FILE_PATH` variable.
+A custom start path can be provided as a positional argument to the `run` command (e.g., `slack run ./src/app.py`), which sets both the `SLACK_APP_PATH` and `SLACK_CLI_CUSTOM_FILE_PATH` environment variables for the hook process.


praise: Thank you for updating this and choosing to support both at first. #backwards-compatible

@mwbrooks If it helps upgrades go smooth I think it's meaningful - I appreciate your help in thinking through this 🧠 💡 ✨

Co-authored-by: Michael Brooks <michael@michaelbrooks.ca>

zimeg · 2026-03-23T19:33:33Z

@mwbrooks I'm quite excited for these changes too! Let's get this merged next - I'd like to first appreciate the comments shared ahead:

Log Levels: This might a CLI option to share with hooks and SDKs
Required File: We make assertion that the argument provided is to an existing file at this time

Both seem great to revisit later but I feel confident in this change so will merge 🚢

zimeg added 7 commits March 17, 2026 22:47

fix: unquote environment variable passed to hook commands

5bd98a8

Merge branch 'main' into zimeg-fix-env-quote

5b2cf70

Merge branch 'main' into zimeg-fix-env-quote

cc1ce45

Merge branch 'main' into zimeg-fix-env-quote

228907d

Merge branch 'zimeg-fix-env-quote' of github.com:slackapi/slack-cli i…

d942d79

…nto zimeg-fix-env-quote

Merge branch 'main' into zimeg-fix-env-quote

5122e45

feat: add optional filepath argument to run command

bc64865

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

zimeg self-assigned this Mar 19, 2026

zimeg requested review from a team as code owners March 19, 2026 06:46

zimeg added enhancement M-T: A feature request for new functionality changelog Use on updates to be included in the release notes semver:minor Use on pull requests to describe the release version increment labels Mar 19, 2026

zimeg commented Mar 19, 2026

View reviewed changes

mwbrooks approved these changes Mar 20, 2026

View reviewed changes

zimeg and others added 2 commits March 23, 2026 10:13

style: make more clear the app file path argument

4416d88

Co-authored-by: Michael Brooks <michael@michaelbrooks.ca>

refactor: continue preference for app file path

2428bd8

Co-authored-by: Michael Brooks <michael@michaelbrooks.ca>

Base automatically changed from zimeg-fix-env-quote to main March 23, 2026 19:24

chore: merge w main

ef1b0af

zimeg enabled auto-merge (squash) March 23, 2026 19:33

zimeg merged commit fae6821 into main Mar 23, 2026
8 checks passed

zimeg deleted the zimeg-feat-run-path branch March 23, 2026 19:34

Conversation

zimeg commented Mar 19, 2026

Changelog

Summary

Preview

Reviewers

Requirements

Uh oh!

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

Uh oh!

Codecov Report

Uh oh!

zimeg left a comment

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!

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!

Uh oh!

Uh oh!

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!

zimeg commented Mar 23, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

codecov bot commented Mar 19, 2026 •

edited

Loading