skillfish 1.0.29 → 1.0.31

package/LICENSE

@@ -633,8 +633,8 @@ the "copyright" line and a pointer to where the full notice is found.
     Copyright (C) <year>  <name of author>
 
     This program is free software: you can redistribute it and/or modify
-    it under the terms of the GNU Affero General Public License as published by
-    the Free Software Foundation, either version 3 of the License, or
+    it under the terms of the GNU Affero General Public License as published
+    by the Free Software Foundation, either version 3 of the License, or
     (at your option) any later version.
 
     This program is distributed in the hope that it will be useful,
@@ -658,4 +658,4 @@ specific requirements.
   You should also get your employer (if you work as a programmer) or school,
 if any, to sign a "copyright disclaimer" for the program, if necessary.
 For more information on this, and how to apply and follow the GNU AGPL, see
-<https://www.gnu.org/licenses/>.
+<https://www.gnu.org/licenses/>.

package/dist/lib/registry.js

@@ -42,7 +42,16 @@ export async function submitSkillsToRegistry(skills) {
             // Validate Content-Type before parsing JSON
             const contentType = res.headers.get('content-type');
             if (!contentType?.includes('application/json')) {
-                throw new Error('Unexpected response type from registry');
+                let body = '';
+                try {
+                    body = (await res.text()).trim();
+                }
+                catch {
+                    // ignore read errors
+                }
+                const statusInfo = !res.ok ? ` (HTTP ${res.status})` : '';
+                const bodyHint = body ? `: ${body.slice(0, 200)}` : '';
+                throw new Error(`Registry returned non-JSON response${statusInfo}${bodyHint}`);
             }
             const data = (await res.json());
             if (res.ok && data.success) {
@@ -118,12 +127,23 @@ export async function searchSkillsInRegistry(query, limit = 5) {
         }, MAX_RETRIES);
         // Validate Content-Type before parsing JSON
         const contentType = res.headers.get('content-type');
-        if (!contentType?.includes('application/json')) {
+        const isJson = contentType?.includes('application/json');
+        // Handle non-JSON responses (e.g. 403 plain text from CDN/WAF)
+        if (!isJson) {
+            let body = '';
+            try {
+                body = (await res.text()).trim();
+            }
+            catch {
+                // ignore read errors
+            }
+            const statusInfo = !res.ok ? ` (HTTP ${res.status})` : '';
+            const bodyHint = body ? `: ${body.slice(0, 200)}` : '';
             return {
                 success: false,
                 results: [],
                 totalCount: 0,
-                error: 'Unexpected response type from registry',
+                error: `Registry returned non-JSON response${statusInfo}${bodyHint}`,
             };
         }
         const data = (await res.json());
@@ -145,16 +165,14 @@ export async function searchSkillsInRegistry(query, limit = 5) {
         // Validate and transform API response to our SearchResult format
         // Filter out malformed items that are missing required string fields
         const results = data.skills
-            .filter((item) => typeof item.name === 'string' &&
-            typeof item.slug === 'string' &&
-            typeof item.github === 'string')
+            .filter((item) => typeof item.name === 'string' && typeof item.slug === 'string')
             .map((item) => ({
             name: item.name,
             slug: item.slug,
             type: 'skill',
             owner: item.owner?.name ?? '',
             ownerUrl: item.owner?.url ?? '',
-            github: item.github,
+            github: item.github ?? '',
             description: item.description ?? '',
             stars: item.github_stars ?? 0,
             relevanceScore: item.relevance_score ?? 0,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skillfish",
-  "version": "1.0.29",
+  "version": "1.0.31",
   "description": "All in one Skill manager for AI coding agents. Install, update, and sync Skills across Claude Code, Cursor, Copilot + more.",
   "type": "module",
   "bin": {