Skip to content

feat: update .env.sample for clarity#100

Merged
mwbrooks merged 1 commit intofeat-ai-apps-thinking-stepsfrom
mwbrooks-env-defaults
Jan 30, 2026
Merged

feat: update .env.sample for clarity#100
mwbrooks merged 1 commit intofeat-ai-apps-thinking-stepsfrom
mwbrooks-env-defaults

Conversation

@mwbrooks
Copy link
Member

@mwbrooks mwbrooks commented Jan 30, 2026

Type of change

  • New feature
  • Bug fix
  • Documentation

Summary

Related to PR slack-samples/bolt-python-assistant-template#44

This pull request updates the .env.sample file to clarify what needs to be set when using the CLI or manually running the app:

  • Adds comments above SLACK_APP_TOKEN, SLACK_BOT_TOKEN to clarify that these only need to be set when manually running the app.
  • Comments out SLACK_APP_TOKEN, SLACK_BOT_TOKEN by default, since our preference is to use the CLI which auto-sets these values.
  • Adds comment above SLACK_API_URL to explain when this should be set.
  • Updates OPENAI_API_KEY to clarify that this is required for this template.

Reviewers

# Create the app
$ lack create test-app -t slack-samples/bolt-js-assistant-template -b mwbrooks-env-defaults
$ cd test-app/

# Set environment variables
$ mv .env.sample .env
$ vim .env
# → Set OPENAI_API_KEY

# Test the app
$ lack run
# → Confirm you can roll the dice to hit the OpenAI API

# Clean up
$ lack delete -f
$ cd ..
$ rm -rf test-app

Requirements

  • I have ensured the changes I am contributing align with existing patterns and have tested and linted my code
  • I've read and agree to the Code of Conduct

@mwbrooks mwbrooks self-assigned this Jan 30, 2026
@mwbrooks mwbrooks added the enhancement New feature or request label Jan 30, 2026
@mwbrooks mwbrooks requested a review from a team as a code owner January 30, 2026 00:24
Copy link
Member

@zimeg zimeg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mwbrooks LGTM! No comments to make - this is nice and focused to what this sample shows 👾 ✨

@mwbrooks
Copy link
Member Author

Thanks for the quick review and confidence check @zimeg! 🚀

@mwbrooks mwbrooks merged commit da0089c into feat-ai-apps-thinking-steps Jan 30, 2026
1 of 4 checks passed
@mwbrooks mwbrooks deleted the mwbrooks-env-defaults branch January 30, 2026 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants