-
Notifications
You must be signed in to change notification settings - Fork 5
Closed
Labels
good first issueGood for newcomersGood for newcomers
Description
Problem
pumpfun tokens trending --limit -1 uses Python negative slicing, returning ~294 items instead of erroring. --limit 0 returns an empty list silently.
Fix
Add min=1 to all five typer.Option definitions for --limit in src/pumpfun_cli/commands/tokens.py:
limit: int = typer.Option(20, "--limit", "-n", min=1, help="...")There are 5 commands that need this: trending, new, graduating, top, search.
Files to change
src/pumpfun_cli/commands/tokens.py— addmin=1to all 5--limitoptionstests/test_commands/— add test that--limit 0or--limit -1errors
How to test
uv run pytest tests/test_commands/ -vReactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomers