Skip to content

Conversation

@srtaalej
Copy link
Contributor

@srtaalej srtaalej commented Feb 6, 2026

Changelog

The templates available to the create command can now be shown without requiring an immediate selection with the --list flag.

Summary

This PR adds --list flag to slack create to print available app templates non-interactively and to slack create agent to print only agent templates.

Screenshot 2026-02-06 at 5 01 33 PM

Requirements

@srtaalej srtaalej self-assigned this Feb 6, 2026
@srtaalej srtaalej requested a review from a team as a code owner February 6, 2026 22:02
@srtaalej srtaalej added enhancement M-T: A feature request for new functionality semver:minor Use on pull requests to describe the release version increment labels Feb 6, 2026
@codecov
Copy link

codecov bot commented Feb 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.72%. Comparing base (3eef3bd) to head (2724306).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #333      +/-   ##
==========================================
+ Coverage   64.68%   64.72%   +0.03%     
==========================================
  Files         212      212              
  Lines       17782    17809      +27     
==========================================
+ Hits        11502    11526      +24     
- Misses       5206     5208       +2     
- Partials     1074     1075       +1     

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

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.

@srtaalej LGTM! I'm hoping we can extend soon the "agent" argument to the samples command, but this might be blurring the line of what these commands surface... Nothing blocking for this PR!

}

for _, category := range categories {
templates := getSelectionOptions(clients, category.id)
Copy link
Member

Choose a reason for hiding this comment

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

⭐ praise: This is a nice filter setup!

Copy link
Member

@mwbrooks mwbrooks left a comment

Choose a reason for hiding this comment

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

✅ Looks great and works well @srtaalej! Thank you!

📝 I've left some minor feedback that we're duplicating code and that it can lead to errors down the road. Non-blocking but perhaps we can add a comment to help reduce errors and remind us to refactor this area?

🍿 Test worked well!

2026-02-09-create-list-flag.mov

Comment on lines +257 to +261
categories = []categoryInfo{
{id: "slack-cli#getting-started", name: "Getting started"},
{id: "slack-cli#ai-apps", name: "AI Agent apps"},
{id: "slack-cli#automation-apps", name: "Automation apps"},
}
Copy link
Member

Choose a reason for hiding this comment

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

suggestion(non-blocking): Not a deal breaker, but I don't like that we're duplicating the categories in 2 areas. In getSelectionOptionsForCategory(...) we already list the category titles and now we're duplicating the categories here. We're also slightly changing the titles ("AI Agent app" vs "AI Agent apps" / "Starter app" vs "Getting started"). This will become more error-prone if we add or change categories later on.

We don't need to refactor this if it's complicated, but we should at least add a comment in both sections noting that changes must occur in the other section. And we should try to keep the naming consistent.

@mwbrooks mwbrooks added this to the Next Release milestone Feb 9, 2026
@zimeg zimeg added the changelog Use on updates to be included in the release notes label Feb 9, 2026
srtaalej and others added 2 commits February 10, 2026 12:24
Co-authored-by: Eden Zimbelman <eden.zimbelman@salesforce.com>
@mwbrooks mwbrooks merged commit 2e9d76f into main Feb 13, 2026
8 checks passed
@mwbrooks mwbrooks deleted the ale-list-flag branch February 13, 2026 00:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants