@hiver/skills 1.0.4 → 1.0.5

package/dist/cli.js

@@ -646,8 +646,8 @@ function List() {
 // src/commands/Update.jsx
 import React8, { useState as useState4, useEffect as useEffect2 } from "react";
 import { Box as Box7, Text as Text8, useApp as useApp3 } from "ink";
-import path3 from "path";
-import fs3 from "fs-extra";
+import path4 from "path";
+import fs4 from "fs-extra";
 
 // src/components/Confirm.jsx
 import React6 from "react";
@@ -692,6 +692,8 @@ function DiffView({ patch }) {
 }
 
 // src/utils/diffUtil.js
+import path3 from "path";
+import fs3 from "fs-extra";
 import { createTwoFilesPatch } from "diff";
 function computeDiff(filePath, localContent, latestContent) {
   return createTwoFilesPatch(
@@ -706,6 +708,63 @@ function computeDiff(filePath, localContent, latestContent) {
 function hasChanges(localContent, latestContent) {
   return localContent.trim() !== latestContent.trim();
 }
+function walkFiles(dir) {
+  const out = [];
+  if (!fs3.existsSync(dir)) return out;
+  const entries = fs3.readdirSync(dir, { withFileTypes: true });
+  for (const entry of entries) {
+    const full = path3.join(dir, entry.name);
+    if (entry.isDirectory()) {
+      for (const sub of walkFiles(full)) out.push(path3.join(entry.name, sub));
+    } else if (entry.isFile()) {
+      out.push(entry.name);
+    }
+  }
+  return out;
+}
+function isBinary(buf) {
+  const sample = buf.subarray(0, Math.min(buf.length, 8e3));
+  for (let i = 0; i < sample.length; i++) {
+    if (sample[i] === 0) return true;
+  }
+  return false;
+}
+function hasDirChanges(localDir, latestDir) {
+  const localFiles = new Set(walkFiles(localDir));
+  const latestFiles = new Set(walkFiles(latestDir));
+  if (localFiles.size !== latestFiles.size) return true;
+  for (const rel of latestFiles) {
+    if (!localFiles.has(rel)) return true;
+    const a = fs3.readFileSync(path3.join(localDir, rel));
+    const b = fs3.readFileSync(path3.join(latestDir, rel));
+    if (!a.equals(b)) return true;
+  }
+  for (const rel of localFiles) {
+    if (!latestFiles.has(rel)) return true;
+  }
+  return false;
+}
+function computeDirDiff(localDir, latestDir) {
+  const localFiles = new Set(walkFiles(localDir));
+  const latestFiles = new Set(walkFiles(latestDir));
+  const allFiles = /* @__PURE__ */ new Set([...localFiles, ...latestFiles]);
+  const parts = [];
+  for (const rel of [...allFiles].sort()) {
+    const localBuf = localFiles.has(rel) ? fs3.readFileSync(path3.join(localDir, rel)) : Buffer.alloc(0);
+    const latestBuf = latestFiles.has(rel) ? fs3.readFileSync(path3.join(latestDir, rel)) : Buffer.alloc(0);
+    if (localBuf.equals(latestBuf)) continue;
+    if (isBinary(localBuf) || isBinary(latestBuf)) {
+      const verb = !localFiles.has(rel) ? "added" : !latestFiles.has(rel) ? "removed" : "changed";
+      parts.push(`Binary file ${rel} ${verb} (${localBuf.length} \u2192 ${latestBuf.length} bytes)`);
+      continue;
+    }
+    const localContent = localBuf.toString("utf-8");
+    const latestContent = latestBuf.toString("utf-8");
+    if (localContent.trim() === latestContent.trim()) continue;
+    parts.push(computeDiff(rel, localContent, latestContent));
+  }
+  return parts.join("\n");
+}
 
 // src/commands/Update.jsx
 import { jsx as jsx7, jsxs as jsxs7 } from "react/jsx-runtime";
@@ -718,10 +777,10 @@ var STEPS2 = {
   DONE: "done"
 };
 function detectType(name, targetConfig, destRoot) {
-  const skillPath = path3.join(destRoot, targetConfig.skillsDir, name, "SKILL.md");
-  if (fs3.existsSync(skillPath)) return "skill";
-  const agentPath = path3.join(destRoot, targetConfig.agentsDir, `${name}.md`);
-  if (fs3.existsSync(agentPath)) return "agent";
+  const skillPath = path4.join(destRoot, targetConfig.skillsDir, name, "SKILL.md");
+  if (fs4.existsSync(skillPath)) return "skill";
+  const agentPath = path4.join(destRoot, targetConfig.agentsDir, `${name}.md`);
+  if (fs4.existsSync(agentPath)) return "agent";
   return null;
 }
 function Update({ directSkills }) {
@@ -737,6 +796,7 @@ function Update({ directSkills }) {
   const [installedCount, setInstalledCount] = useState4(0);
   const [missingNames, setMissingNames] = useState4([]);
   const [directInfo, setDirectInfo] = useState4([]);
+  const [skipReasons, setSkipReasons] = useState4([]);
   const destRoot = process.cwd();
   function handleTargetSelect(key) {
     setTargetKey(key);
@@ -832,46 +892,60 @@ function Update({ directSkills }) {
       advance();
       return;
     }
-    let localFile;
-    let latestFile;
     if (item.type === "skill") {
-      localFile = path3.join(destRoot, targetConfig.skillsDir, item.name, "SKILL.md");
-      latestFile = path3.join(getCollectionsDir(), collection, "skills", item.name, "SKILL.md");
-    } else {
-      localFile = path3.join(destRoot, targetConfig.agentsDir, `${item.name}.md`);
-      latestFile = path3.join(getCollectionsDir(), collection, "agents", `${item.name}.md`);
+      const localDir = path4.join(destRoot, targetConfig.skillsDir, item.name);
+      const latestDir = path4.join(getCollectionsDir(), collection, "skills", item.name);
+      if (!fs4.existsSync(localDir) || !fs4.existsSync(latestDir)) {
+        recordSkip(item.name, "source or local directory missing");
+        advance();
+        return;
+      }
+      if (!hasDirChanges(localDir, latestDir)) {
+        recordSkip(item.name, `already up-to-date with ${collection}`);
+        advance();
+        return;
+      }
+      setCurrentPatch(computeDirDiff(localDir, latestDir));
+      setStep(STEPS2.CONFIRM);
+      return;
     }
-    if (!fs3.existsSync(localFile) || !fs3.existsSync(latestFile)) {
+    const localFile = path4.join(destRoot, targetConfig.agentsDir, `${item.name}.md`);
+    const latestFile = path4.join(getCollectionsDir(), collection, "agents", `${item.name}.md`);
+    if (!fs4.existsSync(localFile) || !fs4.existsSync(latestFile)) {
+      recordSkip(item.name, "source or local file missing");
       advance();
       return;
     }
-    const localContent = fs3.readFileSync(localFile, "utf-8");
-    const latestContent = fs3.readFileSync(latestFile, "utf-8");
+    const localContent = fs4.readFileSync(localFile, "utf-8");
+    const latestContent = fs4.readFileSync(latestFile, "utf-8");
     if (!hasChanges(localContent, latestContent)) {
-      setSkippedCount((c) => c + 1);
+      recordSkip(item.name, `already up-to-date with ${collection}`);
       advance();
       return;
     }
-    const filename = item.type === "skill" ? "SKILL.md" : `${item.name}.md`;
-    setCurrentPatch(computeDiff(filename, localContent, latestContent));
+    setCurrentPatch(computeDiff(`${item.name}.md`, localContent, latestContent));
     setStep(STEPS2.CONFIRM);
   }
+  function recordSkip(name, reason) {
+    setSkippedCount((c) => c + 1);
+    setSkipReasons((prev) => [...prev, { name, reason }]);
+  }
   function handleConfirm(yes) {
     const item = queue[currentIdx];
     const targetConfig = targets[targetKey];
     if (yes) {
       if (item.type === "skill") {
-        const src = path3.join(getCollectionsDir(), selectedCollection, "skills", item.name);
-        const dest = path3.join(destRoot, targetConfig.skillsDir, item.name);
-        fs3.copySync(src, dest, { overwrite: true });
+        const src = path4.join(getCollectionsDir(), selectedCollection, "skills", item.name);
+        const dest = path4.join(destRoot, targetConfig.skillsDir, item.name);
+        fs4.copySync(src, dest, { overwrite: true });
       } else {
-        const src = path3.join(getCollectionsDir(), selectedCollection, "agents", `${item.name}.md`);
-        const dest = path3.join(destRoot, targetConfig.agentsDir, `${item.name}.md`);
-        fs3.copySync(src, dest, { overwrite: true });
+        const src = path4.join(getCollectionsDir(), selectedCollection, "agents", `${item.name}.md`);
+        const dest = path4.join(destRoot, targetConfig.agentsDir, `${item.name}.md`);
+        fs4.copySync(src, dest, { overwrite: true });
       }
       setUpdatedCount((c) => c + 1);
     } else {
-      setSkippedCount((c) => c + 1);
+      recordSkip(item.name, "declined");
     }
     advance();
   }
@@ -975,17 +1049,26 @@ function Update({ directSkills }) {
     }
     return /* @__PURE__ */ jsxs7(Box7, { flexDirection: "column", children: [
       missingNames.length > 0 && /* @__PURE__ */ jsxs7(ErrorMsg, { children: [
-        "Skipped: ",
+        "Not found: ",
         missingNames.join(", ")
       ] }),
-      /* @__PURE__ */ jsxs7(Success, { children: [
+      skipReasons.length > 0 && /* @__PURE__ */ jsxs7(Box7, { flexDirection: "column", marginTop: 1, children: [
+        /* @__PURE__ */ jsx7(Info, { children: "Skipped:" }),
+        skipReasons.map((s, i) => /* @__PURE__ */ jsxs7(Text8, { color: "gray", children: [
+          "  \u2022 ",
+          s.name,
+          " \u2014 ",
+          s.reason
+        ] }, `${s.name}-${i}`))
+      ] }),
+      /* @__PURE__ */ jsx7(Box7, { marginTop: 1, children: /* @__PURE__ */ jsxs7(Success, { children: [
         updatedCount,
         " updated, ",
         installedCount,
         " installed, ",
         skippedCount,
         " skipped"
-      ] })
+      ] }) })
     ] });
   }
   return null;
@@ -994,8 +1077,8 @@ function Update({ directSkills }) {
 // src/commands/Remove.jsx
 import React9, { useState as useState5 } from "react";
 import { Box as Box8, Text as Text9, useApp as useApp4 } from "ink";
-import path4 from "path";
-import fs4 from "fs-extra";
+import path5 from "path";
+import fs5 from "fs-extra";
 import { jsx as jsx8, jsxs as jsxs8 } from "react/jsx-runtime";
 var STEPS3 = {
   TARGET: "target",
@@ -1015,9 +1098,9 @@ function Remove({ skillNames }) {
     const targetConfig = targets[key];
     const missing = [];
     for (const name of skillNames) {
-      const skillPath = path4.join(destRoot, targetConfig.skillsDir, name);
-      const agentPath = path4.join(destRoot, targetConfig.agentsDir, `${name}.md`);
-      if (!fs4.existsSync(skillPath) && !fs4.existsSync(agentPath)) {
+      const skillPath = path5.join(destRoot, targetConfig.skillsDir, name);
+      const agentPath = path5.join(destRoot, targetConfig.agentsDir, `${name}.md`);
+      if (!fs5.existsSync(skillPath) && !fs5.existsSync(agentPath)) {
         missing.push(name);
       }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hiver/skills",
-  "version": "1.0.4",
+  "version": "1.0.5",
   "description": "Shared Claude Code skills & agents for Hiver teams",
   "type": "module",
   "bin": {