@newlandnpt/pep-cli 0.4.0 → 0.5.0

package/README.md

@@ -137,6 +137,24 @@ pep skills add group/sub/project -p    # move it into this project
 `pep skills list` prints each repository with the directory it lives in, which is what `-p` matches
 against.
 
+### Deleting a skill
+
+Delete a skill folder by hand and it **stays deleted**. `update` refreshes what is still on disk
+and leaves the rest alone, naming them so the run is not silent:
+
+```
+group/sub/project: updated b
+  deleted locally, left alone: a
+  (`pep skills add group/sub/project` puts them back)
+```
+
+`add` is the command that installs, so that is what brings a deleted skill back. This split is
+deliberate: `update` means "bring what I have up to date", not "refill everything the repository
+offers".
+
+Skills that are **new upstream** are still installed by `update` — the test is "was mine and is now
+gone", not "is absent".
+
 `pep auth token` writes only the access token to stdout, so agents can use it without parsing status
 text. Refresh occurs automatically shortly before expiry and the rotated refresh token replaces the old
 credential atomically.

package/dist/pep.cjs

@@ -1195,6 +1195,18 @@ function hashSkill(files) {
   }
   return hash.digest("hex").slice(0, 32);
 }
+function pickSkipped(previous, skipped) {
+  return Object.fromEntries(
+    skipped.map((skill) => [skill, previous?.skills[skill] ?? ""])
+  );
+}
+async function missingSkills(directory, skills) {
+  const gone = /* @__PURE__ */ new Set();
+  for (const skill of skills) {
+    if (!await (0, import_promises4.stat)((0, import_node_path4.join)(directory, skill)).catch(() => null)) gone.add(skill);
+  }
+  return gone;
+}
 async function updateSkills(dependencies) {
   const doFetch = dependencies.fetch ?? globalThis.fetch;
   const name = dependencies.source;
@@ -1211,7 +1223,9 @@ async function updateSkills(dependencies) {
   const state = await dependencies.stateStore.read();
   const previous = state?.packages[name];
   const sameTarget = previous?.directory === dependencies.directory && previous?.linkedInto === dependencies.linkInto;
-  if (commit !== void 0 && commit === previous?.commit && sameTarget) {
+  const gone = previous !== void 0 && sameTarget ? await missingSkills(previous.directory, Object.keys(previous.skills)) : /* @__PURE__ */ new Set();
+  const nothingToRestore = !dependencies.restoreMissing || gone.size === 0;
+  if (commit !== void 0 && commit === previous?.commit && sameTarget && nothingToRestore) {
     await response.body?.cancel();
     return { status: "unchanged", name, commit };
   }
@@ -1223,7 +1237,9 @@ async function updateSkills(dependencies) {
     if (group) group.push(file);
     else bySkill.set(file.skill, [file]);
   }
-  const skills = [...bySkill.keys()].sort();
+  const inArchive = [...bySkill.keys()].sort();
+  const skipped = dependencies.restoreMissing ? [] : inArchive.filter((skill) => gone.has(skill));
+  const skills = inArchive.filter((skill) => !skipped.includes(skill));
   const hashes = {};
   const updated = [];
   const unchanged = [];
@@ -1231,10 +1247,11 @@ async function updateSkills(dependencies) {
     const hash = hashSkill(bySkill.get(skill));
     hashes[skill] = hash;
     const before = previous?.skills[skill];
-    if (before !== void 0 && before !== "" && before === hash) unchanged.push(skill);
+    if (gone.has(skill)) updated.push(skill);
+    else if (before !== void 0 && before !== "" && before === hash) unchanged.push(skill);
     else updated.push(skill);
   }
-  const removed = Object.keys(previous?.skills ?? {}).filter((skill) => !skills.includes(skill));
+  const removed = Object.keys(previous?.skills ?? {}).filter((skill) => !inArchive.includes(skill));
   if (previous !== void 0) {
     const stale = sameTarget ? removed : Object.keys(previous.skills);
     for (const skill of stale) {
@@ -1248,6 +1265,7 @@ async function updateSkills(dependencies) {
     await (0, import_promises4.rm)((0, import_node_path4.join)(dependencies.directory, skill), { recursive: true, force: true });
   }
   for (const file of files) {
+    if (skipped.includes(file.skill)) continue;
     const target = (0, import_node_path4.join)(dependencies.directory, file.skill, ...file.path.split("/"));
     await (0, import_promises4.mkdir)((0, import_node_path4.dirname)(target), { recursive: true });
     await (0, import_promises4.writeFile)(target, file.data);
@@ -1272,7 +1290,9 @@ async function updateSkills(dependencies) {
         ...commit ? { commit } : {},
         directory: dependencies.directory,
         ...dependencies.linkInto !== void 0 ? { linkedInto: dependencies.linkInto } : {},
-        skills: hashes
+        // 跳过的那些**原样留着上次的哈希** —— 账上仍然认它是这个包的（将来 add 能补回来，
+        // 上游删掉时也还清得动），只是这次没去碰它。
+        skills: { ...pickSkipped(previous, skipped), ...hashes }
       }
     }
   });
@@ -1282,8 +1302,9 @@ async function updateSkills(dependencies) {
     updated,
     unchanged,
     skills,
-    fileCount: files.length,
+    fileCount: files.filter((one) => !skipped.includes(one.skill)).length,
     removed,
+    skipped,
     directory: dependencies.directory,
     ...dependencies.linkInto !== void 0 ? { linkedInto: dependencies.linkInto } : {},
     ...copiedCount > 0 ? { copiedCount } : {},
@@ -1292,7 +1313,7 @@ async function updateSkills(dependencies) {
 }
 
 // src/cli.ts
-var VERSION = false ? "0.0.0-dev" : "0.4.0";
+var VERSION = false ? "0.0.0-dev" : "0.5.0";
 function usage() {
   return `PEP CLI ${VERSION}
 
@@ -1348,7 +1369,10 @@ it already lives and never moves anything. On update, -p and --dir instead NARRO
 repositories installed there: \`pep skills update -p\` refreshes only this project's, and reports
 nothing to do when the project has none. To move a repository, add it again with the new flag \u2014
 add is where the location is decided, and the copy in the old location is then removed.
-Either way update only touches skills it wrote itself; anything you put there by hand is left alone.
+Delete a skill folder by hand and update LEAVES IT DELETED \u2014 it refreshes what is still there and
+reports the ones it left alone. Run \`pep skills add <repo>\` to put them back; add is the command
+that installs. Either way update only touches skills it wrote itself; anything you put there by
+hand is left alone.
 
 \`pep docs list\` prints the documents this account can read (path + description); feed a path
 straight to \`pep docs get\` to print that document as markdown on stdout. The documentation
@@ -1375,6 +1399,10 @@ function reportUpdate(result) {
   if (result.unchanged.length > 0) {
     console.log(`  unchanged: ${result.unchanged.join(" ")}`);
   }
+  if (result.skipped.length > 0) {
+    console.log(`  deleted locally, left alone: ${result.skipped.join(" ")}`);
+    console.log(`  (\`pep skills add ${result.name}\` puts them back)`);
+  }
   console.log(`  ${result.fileCount} file(s) in ${result.directory}`);
   if (result.linkedInto !== void 0) {
     console.log(
@@ -1565,17 +1593,22 @@ ${usage()}`);
     const stateStore = fileSkillsStateStore();
     const state = await stateStore.read();
     const installed = Object.keys(state?.packages ?? {}).sort();
-    const run2 = async (source, target) => reportUpdate(
+    const run2 = async (source, target, restoreMissing = false) => reportUpdate(
       await updateSkills({
         ...remote,
         source,
         directory: target.directory,
         ...target.linkInto !== void 0 ? { linkInto: target.linkInto } : {},
+        ...restoreMissing ? { restoreMissing: true } : {},
         stateStore
       })
     );
     if (command === "add") {
-      await run2(requested, targetForSource(chosenTarget, state?.packages[requested]));
+      await run2(
+        requested,
+        targetForSource(chosenTarget, state?.packages[requested]),
+        true
+      );
       return;
     }
     if (installed.length === 0) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@newlandnpt/pep-cli",
-  "version": "0.4.0",
+  "version": "0.5.0",
   "description": "PEP CLI —— 凭 PEP 身份读取 Newland 开发者文档与 agent skills",
   "license": "UNLICENSED",
   "type": "module",