@lmzhen/dsh-tool-skill-manage 0.12.0 → 0.13.1

package/lib/index.js

@@ -130,7 +130,7 @@ const GATES = [
 	{
 		id: "human-confirm",
 		appliesTo: ["admission"],
-		run: async ({ view, origin, confirm, warn, confirmMode, confirmTimeoutSeconds }) => {
+		run: async ({ view, origin, confirm, warn, confirmMode, confirmTimeoutSeconds, describeTarget }) => {
 			const action = view.action;
 			if (action === void 0 || origin !== "foreground" || view.name === "") return null;
 			if (!(action === "create" || action === "delete" && view.absorbedInto === void 0)) return null;
@@ -151,6 +151,9 @@ const GATES = [
 				}, seconds * 1e3);
 				timer.unref();
 			}) : void 0;
+			const context = await describeTarget?.();
+			const details = [context?.description === void 0 || context.description === "" ? null : "What it does: " + context.description, context?.versions === void 0 ? null : "Content versions recorded: " + String(context.versions)].filter((line) => line !== null);
+			const consequence = action === "create" ? `Create skill "${view.name}"? A new skill directory is written into the family tree.` : `Delete skill "${view.name}"? It is archived under .archive and leaves the catalog.`;
 			const asked = confirm({
 				action,
 				name: view.name,
@@ -158,7 +161,7 @@ const GATES = [
 				question: {
 					id: CONFIRM_QUESTION_ID,
 					header: "Confirm",
-					question: action === "create" ? `Create skill "${view.name}"? A new skill directory is written into the family tree.` : `Delete skill "${view.name}"? It is archived under .archive and leaves the catalog.`,
+					question: details.length === 0 ? consequence : consequence + " " + details.join(" · "),
 					options: [{ label: confirmLabel }, { label: CANCEL_LABEL }]
 				},
 				...mode === "timeout" ? { signal: cancel.signal } : {}
@@ -813,6 +816,17 @@ function apply(ctx, rawConfig = {}) {
 				confirm: async (request) => confirmSkillWrite(request, exec),
 				confirmMode: settings().skillWriteConfirm,
 				confirmTimeoutSeconds: settings().skillWriteConfirmTimeoutSeconds,
+				describeTarget: async () => {
+					const versions = typeof args.name === "string" && args.name !== "" ? (await library.listVersions(args.name).catch(() => [])).length : 0;
+					const body = typeof args.content === "string" && args.content !== "" ? args.content : typeof args.name === "string" && args.name !== "" ? await library.read(args.name).catch(() => null) : null;
+					const frontmatter = body === null ? null : parseFrontmatter(body)?.frontmatter ?? null;
+					const parsedDescription = frontmatter === null ? void 0 : frontmatter.description;
+					const description = typeof parsedDescription === "string" ? parsedDescription : "";
+					return {
+						...description === "" ? {} : { description },
+						...versions === 0 ? {} : { versions }
+					};
+				},
 				warn: warnWriteGateOnce
 			});
 			if (refusal !== null) return {
@@ -851,7 +865,7 @@ function apply(ctx, rawConfig = {}) {
 				});
 				if (decision.action === "staged") return {
 					ok: true,
-					message: decision.message,
+					message: decision.pendingId === void 0 ? decision.message : decision.message + " (staged id: " + decision.pendingId + " — /evolution pending lists it)",
 					skills: [],
 					...decision.pendingId !== void 0 ? { pending_id: decision.pendingId } : {}
 				};

package/lib/types/write-gates.d.ts

@@ -38,6 +38,12 @@ export type WriteConfirmMode = 'auto' | 'ask' | 'timeout';
 export declare const DEFAULT_WRITE_CONFIRM_MODE: WriteConfirmMode;
 /** Seconds a `timeout`-mode prompt waits before the write is cancelled. */
 export declare const DEFAULT_WRITE_CONFIRM_TIMEOUT_SECONDS = 120;
+/** What the operator is told about the target before answering. Every field is optional: a target
+ * with no frontmatter, or one the library cannot read, simply carries less. */
+export interface ConfirmContext {
+    readonly description?: string | undefined;
+    readonly versions?: number | undefined;
+}
 /** One confirm question, as the gate writes it and the seam asks it. */
 export interface WriteConfirmRequest {
     /** The action being confirmed (`create` or `delete`). */
@@ -88,6 +94,9 @@ export interface WriteGateContext {
     readonly confirmMode?: WriteConfirmMode;
     /** The registry's `skillWriteConfirmTimeoutSeconds`, read only in `timeout` mode. */
     readonly confirmTimeoutSeconds?: number;
+    /** Read the target's own words for the confirmation card. LAZY on purpose: a write that is not
+     * asked about must not pay a read for a question nobody sees. Absent means "say nothing extra". */
+    readonly describeTarget?: (() => Promise<ConfirmContext | undefined>) | undefined;
     /** Report a degraded gate; must not throw. The implementation decides how often it speaks — the
      * shipped seam latches once per PROCESS, because the conditions it reports (no question service,
      * an unreadable session log) belong to the deployment, not to one write. */

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-tool-skill-manage",
   "description": "Model-facing skill_manage tool (community build)",
-  "version": "0.12.0",
+  "version": "0.13.1",
   "publishConfig": {
     "access": "public"
   },
@@ -27,24 +27,24 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-approval": "^0.12.0",
-    "@lmzhen/dsh-evolution-core": "^0.12.0"
+    "@lmzhen/dsh-evolution-approval": "^0.13.1",
+    "@lmzhen/dsh-evolution-core": "^0.13.1"
   },
   "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.12.0",
-    "@lmzhen/dsh-skill-usage": "^0.12.0"
+    "@lmzhen/dsh-evolution-io": "^0.13.1",
+    "@lmzhen/dsh-skill-usage": "^0.13.1"
   },
   "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.12.0",
-    "@lmzhen/dsh-evolution-io": "^0.12.0",
-    "@lmzhen/dsh-skill-usage": "^0.12.0"
+    "@lmzhen/dsh-evolution-core": "^0.13.1",
+    "@lmzhen/dsh-evolution-io": "^0.13.1",
+    "@lmzhen/dsh-skill-usage": "^0.13.1"
   }
 }