skillsforest 0.1.1 → 0.2.0

package/bin/skillsforest.js

@@ -13,8 +13,8 @@ program
   .version('0.1.0');
 
 program
-  .command('add <slug>')
-  .description('Install a skill by slug from the registry')
+  .command('add <owner/slug>')
+  .description('Install a skill by owner/slug from the registry (e.g. owner/my-skill)')
   .option('--agent <type>', 'Agent type: cursor, claude, codex, generic')
   .option('--path <dir>', 'Custom install path (relative to cwd)')
   .action(addCommand);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skillsforest",
-  "version": "0.1.1",
+  "version": "0.2.0",
   "description": "Install agent skills from SkillsForest",
   "main": "src/index.js",
   "bin": {

package/src/commands/add.js

@@ -1,5 +1,5 @@
 const readline = require('readline');
-const { download } = require('../lib/registry');
+const { download, parseRegistryIdentifier } = require('../lib/registry');
 const { detectAgent, getSkillsDir } = require('../lib/detector');
 const { install } = require('../lib/installer');
 
@@ -14,7 +14,12 @@ function promptAgent() {
   });
 }
 
-async function addCommand(slug, opts) {
+async function addCommand(registrySlug, opts) {
+  if (!parseRegistryIdentifier(registrySlug)) {
+    console.error('Use owner/slug format (e.g. npx skillsforest add owner/my-skill).');
+    process.exit(1);
+  }
+
   let agent = opts.agent;
   const cwd = process.cwd();
 
@@ -34,15 +39,17 @@ async function addCommand(slug, opts) {
   const targetDir = opts.path ? require('path').resolve(cwd, opts.path) : null;
 
   try {
-    const data = await download(slug);
+    const data = await download(registrySlug);
     if (!data.files || !Array.isArray(data.files)) {
       throw new Error('Invalid response: missing files');
     }
-    const skillDir = install(slug, data.files, agent, targetDir);
+    const skillDir = install(data.skill, data.files, agent, targetDir);
     console.log(`Installed "${data.skill}" (v${data.version || '1.0.0'}) to ${skillDir}`);
   } catch (err) {
-    if (err.status === 404) {
-      console.error(`Skill "${slug}" not found. It may be private (use "skillsforest login") or the slug may be wrong.`);
+    if (err.status === 400) {
+      console.error(err.message);
+    } else if (err.status === 404) {
+      console.error(`Skill "${registrySlug}" not found. It may be private (use "skillsforest login") or the identifier may be wrong.`);
     } else if (err.status === 403) {
       console.error('Access denied. Token may be missing or invalid (try "skillsforest login").');
     } else {

package/src/commands/search.js

@@ -13,10 +13,10 @@ async function searchCommand(query) {
       console.log('No skills found.');
       return;
     }
-    console.log(formatRow('NAME', 'DESCRIPTION', 'AUTHOR'));
+    console.log(formatRow('REGISTRY SLUG', 'DESCRIPTION', 'AUTHOR'));
     console.log('-'.repeat(24 + 62 + 17));
     for (const s of items) {
-      console.log(formatRow(s.slug || s.name, s.description, s.author));
+      console.log(formatRow(s.registry_slug || s.slug || s.name, s.description, s.author));
     }
     const total = data.meta?.total ?? items.length;
     if (total > items.length) {

package/src/lib/registry.js

@@ -10,9 +10,8 @@ async function fetchRegistry(pathname, options = {}) {
     headers.Authorization = `Bearer ${token}`;
   }
   const res = await fetch(url, { ...options, headers });
-  const body = await res.text();
-
   if (!res.ok) {
+    const body = await res.text();
     let msg = `Registry request failed: ${res.status} ${res.statusText}`;
     try {
       const j = JSON.parse(body);
@@ -24,23 +23,7 @@ async function fetchRegistry(pathname, options = {}) {
     err.status = res.status;
     throw err;
   }
-
-  const contentType = res.headers.get('content-type') || '';
-  if (!contentType.includes('application/json')) {
-    const preview = body.trimStart().slice(0, 80);
-    throw new Error(
-      `Registry returned non-JSON (${contentType || 'unknown type'}). ` +
-        (body.trimStart().startsWith('<')
-          ? 'Got HTML — the API URL may be wrong or the service may be returning an error page.'
-          : `Body preview: ${preview}`)
-    );
-  }
-  try {
-    return JSON.parse(body);
-  } catch (e) {
-    const preview = body.trimStart().slice(0, 80);
-    throw new Error(`Invalid JSON from registry. Body preview: ${preview}`);
-  }
+  return res.json();
 }
 
 async function search(query, params = {}) {
@@ -48,16 +31,39 @@ async function search(query, params = {}) {
   return fetchRegistry(`/?${sp}`);
 }
 
-async function show(slug) {
-  return fetchRegistry(`/${encodeURIComponent(slug)}`);
+function parseRegistryIdentifier(registrySlug) {
+  const parts = registrySlug.split('/');
+  if (parts.length !== 2 || !parts[0] || !parts[1]) {
+    return null;
+  }
+  return { owner: parts[0], slug: parts[1] };
 }
 
-async function download(slug) {
-  return fetchRegistry(`/${encodeURIComponent(slug)}/download`);
+async function show(registrySlug) {
+  const id = parseRegistryIdentifier(registrySlug);
+  if (!id) {
+    const err = new Error('Use owner/slug format (e.g. owner/my-skill)');
+    err.status = 400;
+    throw err;
+  }
+  const pathname = `/${encodeURIComponent(id.owner)}/${encodeURIComponent(id.slug)}`;
+  return fetchRegistry(pathname);
+}
+
+async function download(registrySlug) {
+  const id = parseRegistryIdentifier(registrySlug);
+  if (!id) {
+    const err = new Error('Use owner/slug format (e.g. owner/my-skill)');
+    err.status = 400;
+    throw err;
+  }
+  const pathname = `/${encodeURIComponent(id.owner)}/${encodeURIComponent(id.slug)}/download`;
+  return fetchRegistry(pathname);
 }
 
 module.exports = {
   search,
   show,
   download,
+  parseRegistryIdentifier,
 };