fix: disable upgrade notification for the manifest info command

[mwbrooks]

Changelog

Disable upgrade notification for the slack manifest and slack manifest info command. The command returns JSON that may be used for scripting and the upgrade notifications interfere with parsing the response output.

Summary

This pull request adds manifest info to the list of commands that suppress update notifications. The manifest info command outputs structured data consumed by other tools, so injecting an upgrade banner would corrupt the output.

Also extends isIgnoredCommand() to support multi-word subcommands (e.g., manifest info is ignored but manifest validate is not).

Preview

N/A

Testing

1. Build with an artificially old version to trigger the upgrade notification:

   go build -ldflags="-X 'github.com/slackapi/slack-cli/internal/version.Version=v1.0.0'" -o bin/slack

2. Delete the lastUpdateCheckedAt timestamp from ~/.slack/config.json so the check isn't skipped due to recency.
3. Create a project:

   ./bin/slack create my-app/
   cd my-app/

4. Verify slack manifest info does NOT show an upgrade notification:

   ./bin/slack manifest info --source local

5. Verify another command DOES show the upgrade notification.

   ./bin/slack manifest validate

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]

Codecov Report

❌ Patch coverage is 84.61538% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.63%. Comparing base (0160824) to head (4a12f91).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
internal/update/update.go	84.61%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #548      +/-   ##
==========================================
- Coverage   71.66%   71.63%   -0.03%     
==========================================
  Files         225      225              
  Lines       19066    19074       +8     
==========================================
+ Hits        13663    13664       +1     
- Misses       4199     4205       +6     
- Partials     1204     1205       +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.

[zimeg]

@mwbrooks LGTM! And thanks for saving confusion or error 🙏

I'm leaving a question that's alright to ignore and a praise for keeping code health improving constant! Please let's get this merged when time is right.

[zimeg]

Suggested change

	ignoredCommands := []string{"_fingerprint", "api", "manifest info", "version"}
	ignoredCommands := []string{"_fingerprint", "api", "manifest", "manifest info", "version"}

🪬 question: Is the shortened command automatic? I prefer extended commands often in scripts but am unsure if this should be covered too.

[mwbrooks]

brutal, I forgot we offer slack manifest as a short-form version of slack manifest info. We should probably deprecate that now that we have more manifest commands - normally running the top-level parent command would show the help info about the sub-commands.

answer: No, this matches against os.Args so it's the raw commands typed by the user. So aliases are not captured, either.

Let's add manifest here. It's safer because it captures manifest info but will also capture all other manifest commands. We'd require a re-write after Cobra parsing to capture aliases properly, but today the update goroutine is started before parallel to the Cobra parsing (not inside Cobra).

[mwbrooks]

@zimeg Commit 33d7e0c adds your suggested, some tests, and comments. It also adds a DEPRECATE(semver:major) to the slack manifest command to no longer print the manifest info. You should be credited as a co-author ✏️ 😄

[zimeg]

👁️‍🗨️ praise: Thanks! This makes me think much about tests as spec more than unit.