Skip to content

Conversation

@RichardHightower
Copy link
Contributor

This PR fixes the skilz visit command to generate correct marketplace URLs.

Problem

The visit command was generating incorrect marketplace URLs:

  • Wrong base path: skillzwave.ai/skill instead of skillzwave.ai/agent-skill
  • Wrong slug format: skill__SKILL suffix instead of clean owner__repo__skill format

Example

Before:
URL: https://skillzwave.ai/agent-skill/spillwavesolutions__sdd__sdd/

After:
URL: https://skillzwave.ai/agent-skill/spillwavesolutions__sdd__sdd/

Changes Made

URL Format Updates

  • Base URL: https://skillzwave.ai/skillhttps://skillzwave.ai/agent-skill
  • Slug format: owner__repo__skill__SKILLowner__repo__skill
  • Single names: Assume spillwavesolutions organization for single skill names

Code Changes

  • src/skilz/commands/visit_cmd.py: Updated resolve_marketplace_url() function
  • tests/test_visit_cmd.py: Updated test expectations to match new URL format

Supported Formats

The command now correctly handles:

  • skilz visit owner/repo/skillowner__repo__skill
  • skilz visit owner/repoowner__repo__repo (skill = repo name)
  • skilz visit skillspillwavesolutions__skill__skill

Testing

  • All 40 visit command tests pass
  • Manual verification of URL generation
  • Backward compatibility maintained for existing functionality

Impact

Users can now successfully visit correct marketplace pages using skilz visit command.

- Change base URL from skillzwave.ai/skill to skillzwave.ai/agent-skill
- Update slug format from owner__repo__skill__SKILL to owner__repo__skill
- Handle single skill names by assuming spillwavesolutions organization
- Update tests to match new URL format

Example:
- Before: skilz visit sdd -> https://skillzwave.ai/skill/sdd__SKILL/
- After: skilz visit sdd -> https://skillzwave.ai/agent-skill/spillwavesolutions__sdd__sdd

Fixes marketplace URL generation for visit command.
@RichardHightower RichardHightower merged commit d4f6397 into main Jan 9, 2026
1 check passed
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