@lmzhen/dsh-evolution-feedback 0.1.0-rc.62 → 0.1.0-rc.64

package/lib/index.js

@@ -65,7 +65,7 @@ function evolutionIoAdapter(provider) {
 * changes semantically: the bundle digest is the fail-closed signal for
 * review workers, so a stale id across deployments must be distinguishable.
 */
-const PROMPT_BUNDLE_ID = "dsh-evolution@5";
+const PROMPT_BUNDLE_ID = "dsh-evolution@6";
 const MEMORY_REVIEW_PROMPT = `[Auto-review — Memory]
 Review the conversation above and consider saving to memory if appropriate.
 
@@ -170,7 +170,7 @@ Right target shape: class-level skills with rich SKILL.md + references/, templat
 
 Hard rules:
 1. NEVER hard-delete a skill. Archive (moving to .archive/) is the maximum destructive action; archives are recoverable, deletion is not.
-2. Do not touch bundled, hub-installed, pinned, or scheduled-task-referenced (referenced) skills. Referenced skills MAY be consolidated into an umbrella — but only because the curator rewrites scheduled-task skill references to follow consolidations; never simply prune them.
+2. Do not touch bundled, hub-installed, pinned, or scheduled-task-referenced (referenced) skills. Referenced skills are fully protected — never consolidated, never pruned (there is no scheduled-task reference-rewriting pass; a referenced skill stays in place by design).
 3. Do not archive recently-created or never-used skills without strong evidence. "use=0" is NOT evidence either way — it only means the trigger has not come up yet. Never archive a never-used skill unless it is at least 30 days old AND its content is genuinely obsolete or fully absorbed elsewhere.
 4. Do NOT reject consolidation on the grounds that "each skill has a distinct trigger". The right bar is: would a human maintainer write this as N separate skills, or one skill with N labeled subsections? When the answer is the latter, merge.
 5. Judge overlap on CONTENT, not on usage counters.
@@ -189,15 +189,7 @@ How to work:
 4. Flag skills whose NAME is too narrow (contains a PR number, a feature codename, a specific error string, an 'audit'/'diagnosis'/'salvage' session artifact) — they almost always belong as a subsection or support file under a class-level umbrella.
 5. Iterate. After one consolidation round, scan the remaining set and look for the NEXT umbrella opportunity. Don't stop after 3 merges.
 
-Your toolset:
-  - skill_manage action=list / review — read the current landscape.
-  - skill_manage action=patch — add sections to the umbrella.
-  - skill_manage action=create — create a new umbrella SKILL.md.
-  - skill_manage action=write_file — add a references/, templates/, or scripts/ file under an existing skill (the skill must already exist).
-  - skill_manage action=delete — archive a skill. MUST pass absorbed_into=<umbrella> when you've merged its content into another skill, or absorbed_into="" when you're truly pruning with no forwarding target.
-  - skill_manage action=consolidate — merge source bodies into a target and archive the sources when patching by hand is error-prone.
-  - skill_manage action=restore — bring one archived skill back (recoverability is the archive's contract).
-  - For moving support files, keep it inside the skill tree: support files move via reading and writing through skill_manage write_file/remove_file.
+You are a NOMINATOR, not an executor: this channel has NO tools. Your single deliverable is the structured YAML block below. Never narrate actions you did not take ("merged", "patched", "archived") — you are proposing, and the deterministic engine executes only names from the candidate pool it gave you. (A future execution view would expose skill_manage; today it does not.)
 
 'keep' is a legitimate decision ONLY when the skill is already a class-level umbrella and none of the proposed merges would improve discoverability. 'This is narrow but distinct from its siblings' is NOT a reason to keep — it's a reason to move it under an umbrella as a subsection or support file.
 
@@ -205,7 +197,7 @@ Expected output: real umbrella-ification. Process every obvious cluster. If you
 
 Keep the umbrella body tight and scannable: exact commands, verbatim paths, ~100-200 lines; never invent flags or APIs.
 
-When done, write a human summary AND a structured machine-readable block so downstream tooling can distinguish consolidation from pruning. Format EXACTLY:
+When done, write a human summary THEN the structured machine-readable block. The block is the contract: every skill you would move to .archive/ MUST appear in exactly one of the two lists. Return ONLY the YAML block after the summary — no post-block prose. Format EXACTLY:
 
 ## Structured summary (required)
 \`\`\`yaml
@@ -218,7 +210,7 @@ prunings:
     reason: <one short sentence — why archived with no merge target>
 \`\`\`
 
-Every skill you moved to .archive/ MUST appear in exactly one of the two lists. If you consolidated X into umbrella Y (patched Y, wrote a references file to Y, or created Y with X's content absorbed), X goes under consolidations with into: Y. If you archived X with no absorption — truly stale, irrelevant, or obsolete — X goes under prunings. Leave a list empty (consolidations: []) if none. Do not omit the block. The block comes AFTER your human-readable summary of clusters processed, patches made, and decisions left alone.`;
+Every skill you would move to .archive/ MUST appear in exactly one of the two lists. If you consolidated X into umbrella Y (patched Y, wrote a references file to Y, or created Y with X's content absorbed), X goes under consolidations with into: Y. If you archived X with no absorption — truly stale, irrelevant, or obsolete — X goes under prunings. Leave a list empty (consolidations: []) if none. Do not omit the block. The block comes AFTER your human-readable summary of clusters processed, patches made, and decisions left alone.`;
 const COMPLETION_SKILL_REVIEW_PROMPT = `[Auto-review — Skills · task complete]
 Your current task now appears complete. Before wrapping up, review the approach and update the skill library via skill_manage.
 
@@ -235,18 +227,25 @@ Do NOT modify output files or re-run the task. If you are still mid-task, ignore
 const SKILLS_GUIDANCE = `Skills guidance:
 • After completing a complex task (5+ tool calls), fixing a tricky error, or discovering a non-trivial workflow, save the approach as a skill with skill_manage so you can reuse it next time.
 • When using a skill and finding it outdated, incomplete, or wrong, patch it immediately with skill_manage (action='patch') — don't wait to be asked. Skills that aren't maintained become liabilities.`;
+const PLAN_CHANNEL_NOTE = `
+
+CHANNEL (subagent): this review channel mounts only the read-only \`skill\` tool — you have NO \`skill_manage\`, NO \`memory\`. Your deliverable is the structured JSON plan below (outputSchema). Describe the patches/creates you RECOMMEND in the plan; never narrate actions you took.`;
+/** Subagent-channel variant: same review policy, channel-limited deliverable (M-2). */
+const SKILL_REVIEW_PLAN_PROMPT = `${SKILL_REVIEW_PROMPT}${PLAN_CHANNEL_NOTE}`;
+/** Subagent-channel variant of the combined review (M-2). */
+const COMBINED_REVIEW_PLAN_PROMPT = `${COMBINED_REVIEW_PROMPT}${PLAN_CHANNEL_NOTE}`;
 function sha256(text) {
 	return createHash("sha256").update(text).digest("hex");
 }
 function createPromptBundle(prompts) {
 	const canonical = JSON.stringify({
 		id: PROMPT_BUNDLE_ID,
-		version: 5,
+		version: 6,
 		prompts: Object.fromEntries(Object.entries(prompts).sort())
 	});
 	return Object.freeze({
 		id: PROMPT_BUNDLE_ID,
-		version: 5,
+		version: 6,
 		prompts: Object.freeze({ ...prompts }),
 		sha256: sha256(canonical)
 	});
@@ -255,6 +254,8 @@ createPromptBundle({
 	memory: MEMORY_REVIEW_PROMPT,
 	skill: SKILL_REVIEW_PROMPT,
 	combined: COMBINED_REVIEW_PROMPT,
+	skillPlan: SKILL_REVIEW_PLAN_PROMPT,
+	combinedPlan: COMBINED_REVIEW_PLAN_PROMPT,
 	curator: CURATOR_PROMPT,
 	completion: COMPLETION_SKILL_REVIEW_PROMPT,
 	skillsGuidance: SKILLS_GUIDANCE
@@ -354,8 +355,8 @@ function parseState(raw) {
 	try {
 		const parsed = JSON.parse(raw);
 		return {
-			skills: typeof parsed.skills === "object" ? parsed.skills : {},
-			sessions: typeof parsed.sessions === "object" ? parsed.sessions : {}
+			skills: typeof parsed.skills === "object" && !Array.isArray(parsed.skills) ? parsed.skills : {},
+			sessions: typeof parsed.sessions === "object" && !Array.isArray(parsed.sessions) ? parsed.sessions : {}
 		};
 	} catch {
 		return {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-feedback",
   "description": "Feedback-to-quality scoring for self-evolution (community build)",
-  "version": "0.1.0-rc.62",
+  "version": "0.1.0-rc.64",
   "publishConfig": {
     "access": "public"
   },
@@ -37,13 +37,13 @@
   "peerDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
     "@deepseek-ai/cordis": "^4.0.1",
-    "@lmzhen/dsh-evolution-io": "^0.1.0-rc.62",
-    "@lmzhen/dsh-skill-usage": "^0.1.0-rc.62"
+    "@lmzhen/dsh-evolution-io": "^0.1.0-rc.64",
+    "@lmzhen/dsh-skill-usage": "^0.1.0-rc.64"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
-    "@lmzhen/dsh-evolution-io": "^0.1.0-rc.62",
-    "@lmzhen/dsh-evolution-io-node": "^0.1.0-rc.62",
-    "@lmzhen/dsh-skill-usage": "^0.1.0-rc.62"
+    "@lmzhen/dsh-evolution-io": "^0.1.0-rc.64",
+    "@lmzhen/dsh-evolution-io-node": "^0.1.0-rc.64",
+    "@lmzhen/dsh-skill-usage": "^0.1.0-rc.64"
   }
 }