skillmds 1.3.7 → 1.3.8

package/CHANGELOG.md

@@ -2,6 +2,10 @@
 
 All notable changes to `get-skill` (published as `skillmds` until 1.3.6) are documented here. Versions follow [semver](https://semver.org).
 
+## 1.3.8
+
+- **Skills install all their files.** `add`, `update` and the MCP install tool now fetch every registry skill's bundle, not only skills marked as packs. Some skills were recorded as single-file while carrying scripts or references, and installed only their SKILL.md. A single-file skill whose bundle can't be reached still installs from its SKILL.md, as before.
+
 ## 1.3.7
 
 - **The package is now `get-skill`.** Install and run it as `npx get-skill add owner/name`, `npx get-skill search "pdf"`, and so on; MCP clients start the server with `npx -y get-skill`. `npm i -g get-skill` installs the `get-skill` and `skillmd` commands. Every release is still published as `skillmds` too, with identical contents, so existing setups keep working.

package/dist/cli.js

@@ -920,19 +920,20 @@ async function registryCandidate(owner, name, source, client) {
     type: skill.type,
     securityFlags: skill.security_flags
   };
+  const missing = [];
+  const bundle = await fetchBundle(client.base, registrySlug, client.token, { missing });
+  if (bundle && bundle.length) {
+    const sk = bundle.find((file) => file.path === "SKILL.md");
+    const note2 = bundleNote(bundle, missing);
+    return {
+      ...meta,
+      type: bundle.length > 1 ? "pack" : meta.type,
+      raw: sk ? sk.contents.toString("utf8") : skillMdFor(skill),
+      files: bundle,
+      ...note2 ? { note: note2 } : {}
+    };
+  }
   if (skill.type === "pack") {
-    const missing = [];
-    const bundle = await fetchBundle(client.base, registrySlug, client.token, { missing });
-    if (bundle && bundle.length) {
-      const sk = bundle.find((file) => file.path === "SKILL.md");
-      const note2 = bundleNote(bundle, missing);
-      return {
-        ...meta,
-        raw: sk ? sk.contents.toString("utf8") : skillMdFor(skill),
-        files: bundle,
-        ...note2 ? { note: note2 } : {}
-      };
-    }
     const pack = await resolvePackFiles(skill);
     if (pack) return { ...meta, ...pack };
   }
@@ -1666,9 +1667,9 @@ function makeDefaultUpdateDeps(opts = {}) {
           if (e.status === 404) return null;
           throw e;
         }
+        const bundle = await fetchBundle(base, slug, token);
+        if (bundle?.length) return { files: bundle, commit_sha: skill.commit_sha ?? void 0 };
         if (skill.type === "pack") {
-          const bundle = await fetchBundle(base, slug, token);
-          if (bundle?.length) return { files: bundle, commit_sha: skill.commit_sha ?? void 0 };
           const pack = await resolvePackFiles(skill);
           if (pack) return { files: pack.files, commit_sha: skill.commit_sha ?? void 0 };
         }

package/dist/mcp.js

@@ -205,24 +205,23 @@ async function installFromRegistry(args, ctx, deps = {}) {
   if (bad) return bad;
   let files = [{ path: "SKILL.md", contents: raw }];
   let note = "";
-  if (skill.type === "pack") {
-    try {
-      const missing = [];
-      const bundle = await getBundle(ctx.base, `${owner}/${name}`, ctx.token, { missing });
-      if (bundle?.length) {
-        files = bundle.some((f) => f.path === "SKILL.md") ? bundle : [...bundle, { path: "SKILL.md", contents: raw }];
-        if (missing.length) note = missingFilesNote(missing);
-      } else {
-        const pack = await getPackFiles(skill);
-        if (pack) {
-          files = pack.files;
-          if (pack.pinMiss) note = "pinned commit unavailable upstream \u2014 fetched the source repo's default branch instead";
-        } else note = "pack assets could not be fetched \u2014 only SKILL.md was written";
-      }
-    } catch (e) {
-      if (e instanceof IntegrityError) return notice(`Refused: ${owner}/${name} failed integrity verification \u2014 ${e.message}. Not installed.`);
-      note = `pack assets could not be fetched (${e instanceof Error ? e.message : String(e)}) \u2014 only SKILL.md was written`;
+  const pack = skill.type === "pack";
+  try {
+    const missing = [];
+    const bundle = await getBundle(ctx.base, `${owner}/${name}`, ctx.token, { missing });
+    if (bundle?.length) {
+      files = bundle.some((f) => f.path === "SKILL.md") ? bundle : [...bundle, { path: "SKILL.md", contents: raw }];
+      if (missing.length) note = missingFilesNote(missing);
+    } else if (pack) {
+      const fromGitHub = await getPackFiles(skill);
+      if (fromGitHub) {
+        files = fromGitHub.files;
+        if (fromGitHub.pinMiss) note = "pinned commit unavailable upstream \u2014 fetched the source repo's default branch instead";
+      } else note = "pack assets could not be fetched \u2014 only SKILL.md was written";
     }
+  } catch (e) {
+    if (e instanceof IntegrityError) return notice(`Refused: ${owner}/${name} failed integrity verification \u2014 ${e.message}. Not installed.`);
+    if (pack) note = `pack assets could not be fetched (${e instanceof Error ? e.message : String(e)}) \u2014 only SKILL.md was written`;
   }
   const written = skillMdOf(files);
   if (written === null) files = [...files, { path: "SKILL.md", contents: raw }];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skillmds",
-  "version": "1.3.7",
+  "version": "1.3.8",
   "description": "Find, install, lint and publish Agent Skills (SKILL.md) from skillmd.com: the get-skill CLI and MCP server.",
   "license": "MIT",
   "author": "SkillMD",