chore: release v0.9.0

Author: yeasy

CHANGELOG.md

@@ -5,6 +5,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.9.0] - 2026-01-21
+
+### Changed
+- **Config Tests**: Fixed and updated `internal/config` tests to align with default repository configuration.
+- **Code Quality**: Resolved linter warnings including `io/ioutil` deprecation and unhandled errors.
+- **Stability**: general improvements and test coverage updates.
+
 ## [0.8.0] - 2026-01-19
 
 ### Added

cmd/install.go

@@ -131,10 +131,6 @@ If no agent is specified, skills are installed to .agent/skills/ by default.`,
 
 				// If git discovery failed or wasn't applicable, fall back to API
 				if err != nil || targetRepo.Type != "dir" {
-					if err != nil && targetRepo.Type == "dir" {
-						// Optional: log or print why git failed if verbose
-						// fmt.Printf("Git discovery failed: %v. Falling back to API...\n", err)
-					}
 					repos, err = repository.FetchSkills(*targetRepo)
 					if err != nil {
 						fmt.Printf("Failed to fetch skills from repo '%s': %v\n", input, err)
@@ -269,13 +265,6 @@ func installSingleSkill(input string, global bool, agents []string) error {
 		} else {
 			// It's a URL (e.g., https://github.com/xxx.git)
 			repoURL = input
-			if !strings.HasSuffix(repoURL, ".git") && !strings.HasPrefix(input, "http") {
-				// It might be a SkillHub slug (no standard user/repo format easily distinguishable from just a name,
-				// but usually slugs are kebab-case).
-				// We can try to resolve it via SkillHub if it doesn't look like a URL.
-				// However, `input` here in this block is usually "http..." because of `isURL` check?
-				// Wait, `isURL` is true for http/git prefix.
-			}
 			urlParts := strings.Split(strings.TrimSuffix(repoURL, ".git"), "/")
 			skillName = urlParts[len(urlParts)-1]
 		}

cmd/root.go

@@ -45,7 +45,7 @@ the Agent ecosystem.`,
 	// Uncomment the following line if your bare application
 	// has an action associated with it:
 	// Run: func(cmd *cobra.Command, args []string) { },
-	Version: "0.8.0",
+	Version: "0.9.0",
 }
 
 // Execute adds all child commands to the root command and sets flags appropriately.

internal/config/config_test.go

@@ -14,9 +14,9 @@ func TestDefaultConfig(t *testing.T) {
 	if len(config.Skills) != 0 {
 		t.Errorf("Expected empty skills list, got %d", len(config.Skills))
 	}
-	// We now have 8 default repos: community, anthropics, scientific, superpowers, openai, matlab, composio, vercel
-	if len(config.Repos) != 8 {
-		t.Errorf("Expected 8 default repos, got %d", len(config.Repos))
+	// We now have 9 default repos: community, anthropics, scientific, superpowers, openai, matlab, composio, vercel, skillhub
+	if len(config.Repos) != 9 {
+		t.Errorf("Expected 9 default repos, got %d", len(config.Repos))
 	}
 }
 
@@ -35,6 +35,7 @@ func TestDefaultReposConfiguration(t *testing.T) {
 		"matlab":      {repoType: "dir", url: "matlab/skills/skills"},
 		"composio":    {repoType: "dir", url: "ComposioHQ/awesome-claude-skills"},
 		"vercel":      {repoType: "dir", url: "vercel-labs/agent-skills"},
+		"skillhub":   {repoType: "skillhub", url: "skills"},
 	}
 
 	for _, repo := range config.Repos {

internal/skillhub/client.go

@@ -3,7 +3,7 @@ package skillhub
 import (
 	"encoding/json"
 	"fmt"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"net/url"
 	"regexp"
@@ -67,7 +67,7 @@ func (c *Client) Search(query string) ([]Skill, error) {
 	if err != nil {
 		return nil, err
 	}
-	defer resp.Body.Close()
+	defer func() { _ = resp.Body.Close() }()
 
 	if resp.StatusCode != http.StatusOK {
 		return nil, fmt.Errorf("SkillHub API returned status: %d", resp.StatusCode)
@@ -88,13 +88,13 @@ func (c *Client) Resolve(slug string) (string, error) {
 	if err != nil {
 		return "", err
 	}
-	defer resp.Body.Close()
+	defer func() { _ = resp.Body.Close() }()
 
 	if resp.StatusCode != http.StatusOK {
 		return "", fmt.Errorf("failed to fetch skill page: %d", resp.StatusCode)
 	}
 
-	body, err := ioutil.ReadAll(resp.Body)
+	body, err := io.ReadAll(resp.Body)
 	if err != nil {
 		return "", err
 	}
@@ -115,5 +115,29 @@ func (c *Client) Resolve(slug string) (string, error) {
 		return rawURL, nil
 	}
 
+	// Fallback: try to find repoUrl in Next.js hydration data or JSON
+	// Matches: "repoUrl":"https://github.com/..."
+	reJson := regexp.MustCompile(`"repoUrl":"(https://github\.com/[^"]+)"`)
+	matchesJson := reJson.FindStringSubmatch(string(body))
+	if len(matchesJson) > 1 {
+		rawURL := matchesJson[1]
+		// unescape backward slashes if any (though usually forward slashes are fine in JSON)
+		// But in the hydration data we saw, it was like \"repoUrl\":\"https...\"
+		// The string(body) should have the raw bytes.
+		// If it's inside a JS string, it might be escaped.
+		// The curl output showed: \"repoUrl\":\"https://github.com/MadAppGang/claude-code\"
+		// So the regex needs to handle the escaped quotes?
+		// Actually, if we use a broader regex, we can capture it.
+		// Let's rely on finding https://github.com inside the quote.
+		return rawURL, nil
+	}
+
+	// Try one more pattern for escaped JSON
+	reEscaped := regexp.MustCompile(`\\?"repoUrl\\?":\\?"(https://github\.com/[^"\\]+)\\?"`)
+	matchesEscaped := reEscaped.FindStringSubmatch(string(body))
+	if len(matchesEscaped) > 1 {
+		return matchesEscaped[1], nil
+	}
+
 	return "", fmt.Errorf("GitHub URL not found for skill: %s", slug)
 }