Skip to content

Comments

feat(market): add ability to list modules in a marketplace#32

Merged
mrbrandao merged 1 commit intoRedHatProductSecurity:mainfrom
SecKatie:feat/market-ls-modules
Feb 3, 2026
Merged

feat(market): add ability to list modules in a marketplace#32
mrbrandao merged 1 commit intoRedHatProductSecurity:mainfrom
SecKatie:feat/market-ls-modules

Conversation

@SecKatie
Copy link
Contributor

@SecKatie SecKatie commented Feb 3, 2026

Summary

  • Add lola market ls <name> command to list all modules in a specific marketplace
  • Display module name, version, description, and tags in a formatted table
  • Fix bug in to_cache_dict() where marketplace name was incorrectly set to description

Test plan

  • Run lola market ls (without name) - lists all marketplaces
  • Run lola market ls <name> - lists modules in that marketplace
  • Run lola market ls nonexistent - shows error message
  • All 512 tests pass

AI Disclosure

AI-assisted with Claude Code

🤖 Generated with Claude Code

Add `lola market ls <name>` to display all modules in a specific
marketplace with their version, description, and tags.

- Add show() method to MarketplaceRegistry
- Update market_ls CLI to accept optional marketplace name
- Fix bug in to_cache_dict() where name was set to description
- Add tests for new functionality

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@mrbrandao mrbrandao merged commit 8cbbfe4 into RedHatProductSecurity:main Feb 3, 2026
4 checks passed
@mrbrandao
Copy link
Collaborator

That's great, thanks @SecKatie , merged

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.

2 participants