@lmzhen/dsh-tool-skill-manage 0.15.2 → 0.16.0

package/lib/index.js

@@ -1,7 +1,7 @@
 import { effectiveSessionPolicy } from "@lmzhen/dsh-evolution-approval";
 import z from "@deepseek-ai/schemastery";
 import { defineTool } from "@deepseek-ai/dsh-tools";
-import { DEFAULT_ARCHIVE_RETENTION_POLICY, DEFAULT_CITATION_POLICY, DEFAULT_REFERENCE_REWRITE_POLICY, DEFAULT_SKILL_LIMITS, DEFAULT_SKILL_VERSION_KEEP, DEFAULT_SUPPORT_FILE_CHAR_POLICY, DSH_AUTHORING_STANDARDS, SKILLS_GUIDANCE, SKILLS_GUIDANCE_SECTION_ORDER, SKILL_ACTION_REQUIRED_FIELDS, authoringFeedback, callingScope, clampedNumber, computeDedupGroups, contentHash, errorText, evolutionIoAdapter, installParamSection, isPresent, isUnknown, isUnreadWrite, nearDuplicateSummaries, newSkillLibrary, paramNamespace, parseFrontmatter, policyStageLimits, probePresent, probeUnknown, readNumberParam, resolveExecOrigins, sessionReadSkillNames } from "@lmzhen/dsh-evolution-core";
+import { DEFAULT_ARCHIVE_RETENTION_POLICY, DEFAULT_CITATION_POLICY, DEFAULT_REFERENCE_REWRITE_POLICY, DEFAULT_SKILL_LIMITS, DEFAULT_SKILL_VERSION_KEEP, DEFAULT_SUPPORT_FILE_CHAR_POLICY, DSH_AUTHORING_STANDARDS, SKILLS_GUIDANCE, SKILLS_GUIDANCE_SECTION_ORDER, SKILL_ACTION_REQUIRED_FIELDS, anchorVerdict, authoringFeedback, callingScope, clampedNumber, computeDedupGroups, contentHash, errorText, evolutionIoAdapter, fuzzyPatch, installParamSection, isPresent, isUnknown, isUnreadWrite, nearDuplicateSummaries, newSkillLibrary, normalizeFrontmatter, paramNamespace, parseFrontmatter, policyStageLimits, probePresent, probeUnknown, readNumberParam, resolveExecOrigins, sessionReadSkillNames } from "@lmzhen/dsh-evolution-core";
 //#region lib/types/write-gates.js
 /**
 * The write-admission sequence: ONE ordered table of the reasons a skill write is refused.
@@ -295,6 +295,89 @@ const SKILL_SETTINGS_CAPS = [
 function validateSkillSettings(value, ceilings) {
 	for (const key of SKILL_SETTINGS_CAPS) if (value[key] > ceilings[key]) throw new Error(`${key} may only be tightened: ${value[key]} exceeds the deployment value ${ceilings[key]}`);
 }
+/**
+* What the staged skill write WOULD store, resolved WITHOUT writing.
+*
+* It lives here because this module owns what a skill operation MEANS: the bytes come from the same
+* helpers the write path uses (`normalizeFrontmatter`, `fuzzyPatch`, the `trimEnd() + '\n'` trailing
+* rule the library commits with), and the stage-time anchor is judged by core's own `anchorVerdict`, so
+* the preview cannot advertise bytes the approve path then refuses. Everything it cannot resolve is an
+* answer with a reason — never a guess and never a partial diff.
+* @param library - the family skill library (reads only).
+* @param args - the staged record's args: `{ operation }`.
+* @returns the before/after pair, or why there is none to show.
+*/
+async function previewSkillWrite(library, args) {
+	const operation = (args ?? {}).operation ?? {};
+	const name = typeof operation.name === "string" ? operation.name.trim() : "";
+	if (name === "") return {
+		available: false,
+		reason: "the staged write names no skill"
+	};
+	const filePath = typeof operation.file_path === "string" && operation.file_path !== "" ? operation.file_path : null;
+	const path = filePath ?? "SKILL.md";
+	const before = await (filePath === null ? library.read(name) : library.readSupportFile(name, filePath)).catch(() => null);
+	const staged = operation.staged_from_sha256;
+	if (anchorVerdict(typeof staged === "string" && staged !== "" ? staged === "absent" ? { absent: true } : { sha256: staged } : void 0, before) !== "match") return {
+		available: false,
+		reason: "the target changed after this write was staged — approving it would be refused"
+	};
+	const action = typeof operation.action === "string" ? operation.action : "";
+	/** The bytes the write path commits for a body: frontmatter normalized, one trailing newline. */
+	const bodyOnDisk = (content) => {
+		const norm = normalizeFrontmatter(content);
+		if (norm.issues.length > 0) return {
+			available: false,
+			reason: "frontmatter cannot be auto-fixed: " + norm.issues.join("; ")
+		};
+		return {
+			available: true,
+			path,
+			before,
+			after: (norm.changed ? norm.content : content).trimEnd() + "\n"
+		};
+	};
+	if (action === "create" || action === "update") return typeof operation.content === "string" ? bodyOnDisk(operation.content) : {
+		available: false,
+		reason: "\"" + action + "\" carries no content"
+	};
+	if (action === "patch") {
+		if (before === null) return {
+			available: false,
+			reason: "the skill has no body to patch"
+		};
+		const patched = fuzzyPatch(before, operation.old_string ?? "", operation.new_string ?? "", operation.replace_all === true);
+		return patched === null ? {
+			available: false,
+			reason: "the staged old text is not in the current body — approving it would be refused"
+		} : bodyOnDisk(patched);
+	}
+	if (action === "write_file") {
+		if (typeof operation.file_content !== "string") return {
+			available: false,
+			reason: "\"write_file\" carries no content"
+		};
+		return {
+			available: true,
+			path,
+			before,
+			after: operation.file_content
+		};
+	}
+	if (action === "remove_file") return before === null ? {
+		available: false,
+		reason: "that support file is not there to remove"
+	} : {
+		available: true,
+		path,
+		before,
+		after: null
+	};
+	return {
+		available: false,
+		reason: "\"" + action + "\" has no byte-level preview"
+	};
+}
 /** One abort signal that carries BOTH cancellations: the call's own (the operator cancelled the
 * turn) and the gate's deadline (`skillWriteConfirm: 'timeout'`). `undefined` when neither exists —
 * the platform's `ask()` then waits without a signal, which is the `ask` mode.
@@ -874,7 +957,9 @@ function apply(ctx, rawConfig = {}) {
 		}
 	}));
 	ctx.inject(["evolutionApproval"], (approvalCtx) => {
-		const dispose = approvalCtx.evolutionApproval.registerRunner("skill", (args) => {
+		const approval = approvalCtx.evolutionApproval;
+		const preview = async (args) => await previewSkillWrite(library, args);
+		const dispose = approval.registerRunner("skill", (args) => {
 			const wrapped = args ?? {};
 			const operation = wrapped.operation ?? {};
 			const replayAction = operation.action;
@@ -887,7 +972,7 @@ function apply(ctx, rawConfig = {}) {
 				return executeCore(operation, wrapped.libraryOrigin ?? wrapped.origin ?? "background_review");
 			});
 			return executeCore(operation, wrapped.libraryOrigin ?? wrapped.origin ?? "background_review");
-		});
+		}, preview);
 		approvalCtx.effect(() => dispose, "tool-skill-manage.approval-runner");
 	});
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-tool-skill-manage",
   "description": "Model-facing skill_manage tool (community build)",
-  "version": "0.15.2",
+  "version": "0.16.0",
   "publishConfig": {
     "access": "public"
   },
@@ -27,24 +27,24 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-approval": "^0.15.2",
-    "@lmzhen/dsh-evolution-core": "^0.15.2"
+    "@lmzhen/dsh-evolution-approval": "^0.16.0",
+    "@lmzhen/dsh-evolution-core": "^0.16.0"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-skill": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-system-prompt": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-tools": "^0.1.5-rc.2",
-    "@lmzhen/dsh-evolution-io": "^0.15.2",
-    "@lmzhen/dsh-skill-usage": "^0.15.2"
+    "@lmzhen/dsh-evolution-io": "^0.16.0",
+    "@lmzhen/dsh-skill-usage": "^0.16.0"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-agent-loop-testkit": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-skill": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-system-prompt": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-tools": "^0.1.5-rc.2",
-    "@lmzhen/dsh-evolution-core": "^0.15.2",
-    "@lmzhen/dsh-evolution-io": "^0.15.2",
-    "@lmzhen/dsh-skill-usage": "^0.15.2"
+    "@lmzhen/dsh-evolution-core": "^0.16.0",
+    "@lmzhen/dsh-evolution-io": "^0.16.0",
+    "@lmzhen/dsh-skill-usage": "^0.16.0"
   }
 }