@lmzhen/dsh-tool-skill-manage 0.9.0 → 0.11.1

package/README.md

@@ -72,6 +72,28 @@ only the second one refuses a write:
 The third row is a view, not a rule: a description the family accepts can still be cut in a catalog
 viewer, which is why the authoring feedback names the bar rather than treating the cut as a limit.
 
+### The create-time duplicate hint
+
+A `create` compares its candidate against the library listing before it writes, over the name +
+description projection (core `nearDuplicateSummaries`, at `SUMMARY_DUPLICATE_HINT_THRESHOLD`), and
+names the closest existing skills in the same "Authoring check" block. Three properties are
+deliberate: it adds no read of its own (`library.list()` already reads every SKILL.md to parse
+its frontmatter — that listing is what publishes `description` — and the hint never calls
+`library.read`, never holds a body and never compares content); it is a HINT (the write proceeds
+— only the model's next decision changes); and a listing failure degrades to a named line instead of
+silence, because "nothing similar was checked" and "nothing similar exists" must not look alike. The
+library-wide BODY scan stays where it was, in the `review` action.
+
+### The retention line
+
+A whole-body replacement that keeps less than half of the previous SKILL.md says so in the same result
+message (`Content kept 26% of the previous body (2104 of 8123 characters); the replaced version is
+preserved in this skill's history.`). It is computed where the write path still holds both bodies — the
+library's own in-lock read, core `contentRetentionFeedback` — because a caller re-reading the file
+would race the writers the library serializes. Like the duplicate hint it is FEEDBACK, not a gate: no ratio
+refuses a write, small skills stay quiet (a 40-character skill cut in half is not news), and patches,
+support-file writes, creates and archives have no ratio to report.
+
 ## Known limitations
 
 - No known durable consumer gaps at this time. Runtime contracts are covered by package and boundary tests.

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_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, 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, 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";
 //#region lib/types/write-gates.js
 /**
 * The write-admission sequence: ONE ordered table of the reasons a skill write is refused.
@@ -213,7 +213,8 @@ const Config = z.object({
 	maxSkillFileBytes: z.number().min(1).default(DEFAULT_SKILL_LIMITS.maxSkillFileBytes),
 	descriptionStrict: z.boolean().default(false),
 	threatExemptLabels: z.array(z.string()).default([]),
-	strictCrossSource: z.boolean().default(false)
+	strictCrossSource: z.boolean().default(false),
+	skillVersionKeep: z.number().min(1).default(DEFAULT_SKILL_LIMITS.versionKeep ?? DEFAULT_SKILL_VERSION_KEEP)
 });
 /** Schema the platform validates the user layer against; defaults mirror the core
 * constants and the row schema, so an empty document resolves to today's behaviour. */
@@ -376,7 +377,8 @@ function apply(ctx, rawConfig = {}) {
 		maxNameLength: limit("maxSkillNameLength", rawConfig.maxSkillNameLength, DEFAULT_SKILL_LIMITS.maxNameLength),
 		maxDescriptionLength: limit("maxDescriptionLength", rawConfig.maxDescriptionLength, DEFAULT_SKILL_LIMITS.maxDescriptionLength),
 		maxSkillContentChars: limit("maxSkillContentChars", readNumberParam(rawConfig, "skillContentChars"), DEFAULT_SKILL_LIMITS.maxSkillContentChars),
-		maxSkillFileBytes: limit("maxSkillFileBytes", rawConfig.maxSkillFileBytes, DEFAULT_SKILL_LIMITS.maxSkillFileBytes)
+		maxSkillFileBytes: limit("maxSkillFileBytes", rawConfig.maxSkillFileBytes, DEFAULT_SKILL_LIMITS.maxSkillFileBytes),
+		versionKeep: limit("skillVersionKeep", rawConfig.skillVersionKeep, DEFAULT_SKILL_LIMITS.versionKeep ?? DEFAULT_SKILL_VERSION_KEEP)
 	};
 	const library = newSkillLibrary({
 		config: rawConfig,
@@ -523,10 +525,12 @@ function apply(ctx, rawConfig = {}) {
 		};
 		let feedbackLines = [];
 		const stagedAnchor = typeof args.staged_from_sha256 === "string" && args.staged_from_sha256 !== "" ? args.staged_from_sha256 === "absent" ? { absent: true } : { sha256: args.staged_from_sha256 } : void 0;
+		let candidateDescription = "";
 		if ((action === "create" || action === "edit" || action === "update") && args.content) {
 			const parsed = parseFrontmatter(args.content);
 			if (parsed) {
 				const feedback = authoringFeedback(parsed.frontmatter);
+				candidateDescription = parsed.frontmatter.description ?? "";
 				feedbackLines = feedback.lines;
 				if (settings().descriptionStrict && feedback.over60) return {
 					ok: false,
@@ -536,8 +540,10 @@ function apply(ctx, rawConfig = {}) {
 			}
 		}
 		let result;
-		if (action === "create") result = await library.create(name, args.content ?? "", origin);
-		else if (action === "edit" || action === "update") result = await library.update(name, args.content ?? "", origin, stagedAnchor);
+		if (action === "create") {
+			feedbackLines.push(...await duplicateHintLines(name, candidateDescription));
+			result = await library.create(name, args.content ?? "", origin);
+		} else if (action === "edit" || action === "update") result = await library.update(name, args.content ?? "", origin, stagedAnchor);
 		else if (action === "patch") result = await library.patch(name, args.old_string ?? "", args.new_string ?? "", args.file_path ?? "", args.replace_all === true, origin);
 		else if (action === "delete") result = await library.archive(name, args.absorbed_into ? { absorbedInto: args.absorbed_into } : {});
 		else if (action === "write_file") result = await library.writeSupportFile(name, args.file_path ?? "", args.file_content ?? "", origin, stagedAnchor);
@@ -579,6 +585,39 @@ function apply(ctx, rawConfig = {}) {
 			skills: []
 		};
 	}
+	/** Batch C (2026-09-27, design §3C-2): the create-time near-duplicate hint — one
+	* line naming the existing skill(s) this candidate is about to near-copy, over
+	* core's `summary` projection (name + description). Three properties are the
+	* design's, not stylistic choices:
+	*
+	*   - It adds NO read of its own. `library.list()` reads each SKILL.md to parse its
+	*     frontmatter — that listing is what publishes `description` — and the hint never
+	*     calls `library.read`, never holds a body and never compares content, so a create
+	*     pays no SECOND whole-tree pass for it.
+	*   - It is a HINT: the write proceeds either way (G4 — create's foreground
+	*     behavior is unchanged). The model gets the news, not a refusal.
+	*   - A failed listing degrades to a NAMED line, never to silence: "nothing
+	*     similar was checked" and "nothing similar exists" must not look alike
+	*     (the PLAN-R2 P2-5 posture the review path already takes).
+	*
+	* The level and the math live in core (`nearDuplicateSummaries`,
+	* `SUMMARY_DUPLICATE_HINT_THRESHOLD`) — this function owns the wording only. */
+	async function duplicateHintLines(name, description) {
+		let matches;
+		try {
+			matches = nearDuplicateSummaries({
+				candidate: {
+					name,
+					description
+				},
+				existing: await library.list()
+			});
+		} catch (error) {
+			return [`Duplicate check skipped: the skill listing failed (${error instanceof Error ? error.message : String(error)}).`];
+		}
+		if (matches.length === 0) return [];
+		return [`Near-duplicate of existing skill(s): ${matches.map((match) => `${match.name} (${match.score.toFixed(2)})`).join(", ")} — prefer patch/update on that skill over adding a near-copy.`];
+	}
 	async function buildSkillReviewText() {
 		const list = await library.list();
 		const report = await ctx.skillUsage.report();

package/lib/types/index.d.ts

@@ -46,6 +46,10 @@ export interface Config {
      * the family tree) is REFUSED instead of warned about. Default false —
      * warn only, keeping the family tree an autonomous evolution zone. */
     strictCrossSource?: boolean;
+    /** Content versions retained per skill (skill-history.ts). E2: a deployment value on this row,
+     * like the four caps above — the settings layer deliberately has no card for it (retention is
+     * storage policy, not an authoring knob). */
+    skillVersionKeep?: number;
 }
 export declare const Config: z<Config>;
 /** Write behaviour a user may change (G3/S3.4). Field names are the CANONICAL

package/package.json

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