fix(discover): remove redundant positional argument from trending command (Vibe Kanban)#4
Merged
MinaraAgent merged 1 commit intomainfrom Mar 31, 2026
Merged
Conversation
…mand Remove [category] positional argument from 'discover trending' to make CLI consistent with documented behavior. Use only -t/--type flag for specifying tokens or stocks. Before: discover trending [category] -t <type> After: discover trending -t <type> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
MinaraAgent
changed the title
discover command (vibe-kanban)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
[category]positional argument fromdiscover trendingcommand-t, --typeflag is now the only way to specify tokens/stocks typeWhy
There was an inconsistency between CLI help output and documented behavior:
-tor--typeflag-tflag--helpshowed both a positional argument AND the flag, creating confusionThis fix makes the CLI consistent with the documented Skill behavior.
Changes
Before:
After:
Usage Examples
discover trending -t tokens- View trending tokensdiscover trending -t stocks- View trending stocksdiscover trending- Interactive prompt (unchanged)All 263 tests pass.
This PR was written using Vibe Kanban