Skip to content

fix(skills): explain unfederated skills.sh imports - #4196

Open
tsumon wants to merge 2 commits into
Tencent:mainfrom
tsumon:fix/4167-skills-sh-not-federated
Open

tsumon wants to merge 2 commits into
Tencent:mainfrom
tsumon:fix/4167-skills-sh-not-federated

Conversation

@tsumon

@tsumon tsumon commented Oct 12, 2026 •

Copy link
Copy Markdown

Closes #4167

When ClawHub returns HTTP 404 while resolving a skills.sh listing, return a clear message that the listing is unavailable through ClawHub federation and direct users to import the skill using the GitHub folder URL shown by skills.sh. Preserve the HTTP 404 details and ErrSkillSourceInvalid classification. Other sources and HTTP statuses keep their existing error behavior; this does not guess a GitHub subdirectory from the listing slug.

Tests cover the skills.sh 404 guidance and the unchanged generic 404 for other source URLs.

Validation:

  • gofmt and git diff --check pass.
  • App workflow (format, vet, test, and build) passes.
  • Go Lint and Tencent CodeCC pass.

This branch has not been deployed

No deployments
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.

[Bug]: skills.sh listing URL fails when ClawHub has no federated copy

1 participant