@lmzhen/dsh-tool-skill-manage 0.3.62 → 0.3.64

package/lib/index.js

@@ -69,11 +69,6 @@ function apply(ctx, rawConfig = {}) {
 			message: await buildSkillReviewText(),
 			skills: []
 		};
-		if (action === "skip") return {
-			ok: true,
-			message: "Skipped; no skill changes this pass.",
-			skills: []
-		};
 		if (action === "list") {
 			const list = await library.list();
 			return {
@@ -116,7 +111,7 @@ function apply(ctx, rawConfig = {}) {
 		};
 		if (result.ok) {
 			if (result.normalizedFrontmatterFields && result.normalizedFrontmatterFields.length > 0) feedbackLines.push(`frontmatter auto-quoted (YAML compatibility): ${result.normalizedFrontmatterFields.join(", ")}`);
-			const mutating = action !== "list" && action !== "review" && action !== "skip" && action !== "pin" && action !== "unpin";
+			const mutating = action !== "list" && action !== "review" && action !== "pin" && action !== "unpin";
 			if (name && action === "create") await ctx.skillUsage.ensureRecordCreated(name, origin !== "foreground");
 			if (name && action === "delete") await ctx.skillUsage.markArchived(name);
 			else if (name && mutating && action !== "create" && result.noop !== true) await ctx.skillUsage.record(name, "patch");
@@ -130,14 +125,20 @@ function apply(ctx, rawConfig = {}) {
 	async function buildSkillReviewText() {
 		const list = await library.list();
 		const report = await ctx.skillUsage.report();
+		const warnedFlag = (name) => {
+			const record = report.get(name);
+			return record?.quality_warn === true || record?.feedback_warn === true;
+		};
 		const lines = list.map((summary) => {
 			const record = report.get(summary.name);
-			const quality = record?.quality_score !== void 0 ? ` quality:${record.quality_score.toFixed(2)}${record.quality_warn ? "⚠" : ""}` : "";
-			return `- ${summary.name} | ${record?.state ?? "active"} | use:${record?.use_count ?? 0} view:${record?.view_count ?? 0} patch:${record?.patch_count ?? 0}${quality}${summary.protectedBy ? ` [${summary.protectedBy}]` : ""}`;
+			const warned = warnedFlag(summary.name);
+			const quality = record?.quality_score !== void 0 ? ` quality:${record.quality_score.toFixed(2)}${warned ? "⚠" : ""}` : warned ? " quality:⚠" : "";
+			const protection = summary.protectionUnknown ? " [protection:unknown]" : summary.protectedBy ? ` [${summary.protectedBy}]` : "";
+			return `- ${summary.name} | ${record?.state ?? "active"} | use:${record?.use_count ?? 0} view:${record?.view_count ?? 0} patch:${record?.patch_count ?? 0}${quality}${protection}`;
 		});
 		const groups = computeDedupGroups({ contents: new Map(await Promise.all(list.map(async (summary) => [summary.name, await library.read(summary.name) ?? ""]))) });
 		const dedupLines = groups.slice(0, MAX_DEDUP_GROUPS_IN_REVIEW).map((group) => `- ${group.join(" ~ ")}`);
-		const warned = list.filter((summary) => report.get(summary.name)?.quality_warn === true).map((summary) => summary.name);
+		const warned = list.filter((summary) => warnedFlag(summary.name)).map((summary) => summary.name);
 		const warningLine = warned.length === 0 ? "" : `\nWarning skills (${warned.length}): ${warned.join(", ")} — low quality; consider consolidating them before authoring new skills.`;
 		return [
 			list.length === 0 ? "No skills yet. Create one with action=create, or it is safe to author a new class-level umbrella." : `Skills: ${list.length} total. Below each name: state, use/view/patch counts, quality (0-1, ⚠ = low) and protection.${groups.length > 0 ? `\n\nNear-duplicate groups (${groups.length}):` : ""}`,
@@ -149,7 +150,7 @@ function apply(ctx, rawConfig = {}) {
 	}
 	ctx.tools.register(defineTool({
 		name: "skill_manage",
-		description: "Manage reusable skills. review returns the library review text (state/usage/quality per skill); list returns names only; create/edit take full SKILL.md content; patch applies old_string -> new_string; delete archives to .archive; pin protects a skill from deletion, background review, and the lifecycle (pin/unpin are never allowed from a background review — foreground and delegated subagents may). Protected bundled/hub skills reject any mutation; pinned skills reject deletion and are read-only to the background review.Prefer patching an umbrella over creating narrow skills. restructure moves entire body sections (by their exact \"## heading\" line, via restructure: [{heading, to_file: \"references/<topic>.md\"}]) into a references/ file and replaces each with a pointer line — the skill name and directory never change.Created/edited SKILL.md MUST start with YAML frontmatter (a name/description block), or creation is rejected. " + DSH_AUTHORING_STANDARDS,
+		description: "Manage reusable skills. review returns the library review text (state/usage/quality per skill); list returns names only; create/edit/update take full SKILL.md content (edit is an alias of update); patch applies old_string -> new_string; delete archives to .archive (absorbed_into names an umbrella skill); write_file/remove_file add or remove one support file under references/ or scripts/; pin protects a skill from deletion, background review, and the lifecycle (pin/unpin are never allowed from a background review — foreground and delegated subagents may). Protected bundled/hub skills reject any mutation; pinned skills reject deletion and are read-only to the background review.Prefer patching an umbrella over creating narrow skills. restructure moves entire body sections (by their exact \"## heading\" line, via restructure: [{heading, to_file: \"references/<topic>.md\"}]) into a references/ file and replaces each with a pointer line — the skill name and directory never change.Created/edited SKILL.md MUST start with YAML frontmatter (a name/description block), or creation is rejected. " + DSH_AUTHORING_STANDARDS,
 		parameters: {
 			action: {
 				type: "string",
@@ -165,7 +166,6 @@ function apply(ctx, rawConfig = {}) {
 					"write_file",
 					"remove_file",
 					"restructure",
-					"skip",
 					"pin",
 					"unpin"
 				]
@@ -223,7 +223,7 @@ function apply(ctx, rawConfig = {}) {
 			const libraryOrigin = origins.library;
 			const sessionPolicy = effectiveSessionPolicy(ctx, exec.agent?.session);
 			const approval = ctx.get("evolutionApproval");
-			if (approval && args.action !== "list" && args.action !== "review" && args.action !== "skip" && args.action !== "pin" && args.action !== "unpin") {
+			if (approval && args.action !== "list" && args.action !== "review" && args.action !== "pin" && args.action !== "unpin") {
 				const decision = await approval.request({
 					kind: "skill",
 					summary: `skill ${args.action ?? "?"} ${args.name ?? ""}`.trim(),

package/package.json

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