Skip to content

Code review with Gemini ; unify some cli interfaces ; fix album listing#48

Merged
crossjam merged 11 commits intomainfrom
gemini/code-review-2026-02-15
Mar 2, 2026
Merged

Code review with Gemini ; unify some cli interfaces ; fix album listing#48
crossjam merged 11 commits intomainfrom
gemini/code-review-2026-02-15

Conversation

@crossjam
Copy link
Copy Markdown
Owner

With Gemini CLI

  • Created a refactoring review and plan
  • Unified the CLI interfaces for albums, tracks, plays, and artists
  • Fixed the albums list subcommand to group by album
  • Default ordering for all list subcommands that were unified should be "last played"

- Collapse duplicate albums by title in 'albums list' for cleaner display
- Remove 'artist' and 'tracks' columns from default album list view
- Fix 'min-plays' filtering bug in album query logic
- Change default sort order to 'recent' for all 'list' commands
- Update documentation and tests to reflect these changes
@crossjam crossjam linked an issue Feb 16, 2026 that may be closed by this pull request
- Refactor 'stats' command to use shared options from 'command_utils'.
- Add support for '-f' short option for '--format'.
- Update default limit for 'plays list' from 20 to 50.
- Add 'tests/test_plays_unify.py' to verify 'plays list' changes.
@crossjam crossjam self-assigned this Feb 16, 2026
@crossjam
Copy link
Copy Markdown
Owner Author

crossjam commented Mar 2, 2026

LGTM! Fire in the disco!

@crossjam crossjam merged commit 7e6e8a4 into main Mar 2, 2026
6 checks passed
@crossjam crossjam deleted the gemini/code-review-2026-02-15 branch March 2, 2026 01:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Group album listing by album

1 participant