@lmzhen/dsh-evolution-merge 0.30.3 → 0.30.4

package/lib/index.js

@@ -588,6 +588,7 @@ function recoveryFor(journal) {
 		case "committed":
 		case "rolled-back":
 		case "failed": return "none";
+		default: return "refuse";
 	}
 }
 //#endregion
@@ -648,7 +649,7 @@ function isSafeSkillName(name) {
 	return /^[A-Za-z0-9][A-Za-z0-9._-]*$/.test(name) && name !== "." && name !== "..";
 }
 function skillFilePath(root, name, path) {
-	if (!isSafeSkillName(name) || path.startsWith("/") || path.includes("..")) throw new Error("unsafe skill file: " + name + "/" + path);
+	if (!isSafeSkillName(name) || path.startsWith("/") || path.split("/").includes("..")) throw new Error("unsafe skill file: " + name + "/" + path);
 	const base = resolve(root, name);
 	const file = resolve(base, path);
 	if (!file.startsWith(base + sep)) throw new Error("skill file escapes skill: " + name + "/" + path);
@@ -662,68 +663,6 @@ function skillBodyPath(root, name) {
 	return path;
 }
 //#endregion
-//#region lib/types/commit-recovery.js
-/** Startup replay for non-terminal commit journals. */
-async function recoverCommitJournals(io, home, now, skillRoot, restoreSource) {
-	const dir = join(home, "merge", "journals");
-	if (!await io.exists(dir)) return [];
-	const files = (await io.list(dir)).filter((name) => name.endsWith(".json"));
-	const reports = [];
-	for (const file of files) {
-		let journal;
-		try {
-			journal = JSON.parse(await io.readText(join(dir, file)) ?? "");
-		} catch {
-			continue;
-		}
-		const recovery = recoveryFor(journal);
-		if (recovery === "none") continue;
-		const snapshot = await readCommitSnapshot(io, home, journal.runId);
-		let action = "report-only";
-		let ok = true;
-		if (recovery === "retry-commit") action = "no-host-write";
-		else if (snapshot === null) {
-			action = "snapshot-missing";
-			ok = false;
-		} else try {
-			if (recovery === "rollback-sources") {
-				let restoredArchived = restoreSource !== void 0;
-				if (restoreSource !== void 0) {
-					for (const source of snapshot.sourceBodies) if (!await restoreSource(source.name)) restoredArchived = false;
-				}
-				if (!restoredArchived) for (const source of snapshot.sourceBodies) await io.writeText(skillBodyPath(skillRoot, source.name), source.body);
-			}
-			if (snapshot.create && snapshot.targetBody === null) try {
-				await io.remove(skillBodyPath(skillRoot, snapshot.target));
-			} catch {}
-			else if (snapshot.targetBody !== null) await io.writeText(skillBodyPath(skillRoot, snapshot.target), snapshot.targetBody);
-			const oldPaths = new Set((snapshot.targetSupport ?? []).map((file) => file.path));
-			for (const file of snapshot.targetSupport ?? []) await io.writeText(skillFilePath(skillRoot, snapshot.target, file.path), file.content);
-			for (const file of snapshot.targetSupportAfter ?? []) if (!oldPaths.has(file.path)) try {
-				await io.remove(skillFilePath(skillRoot, snapshot.target, file.path));
-			} catch {}
-			for (const file of snapshot.sourceSupport ?? []) await io.writeText(skillFilePath(skillRoot, file.name, file.path), file.content);
-			action = recovery === "rollback-sources" ? "restored-sources-and-target" : "restored-target";
-		} catch (error) {
-			action = "restore-failed: " + (error instanceof Error ? error.message : String(error));
-			ok = false;
-		}
-		await writeJournal(io, home, {
-			...journal,
-			phase: ok ? "rolled-back" : "failed",
-			updatedAt: new Date(now).toISOString()
-		});
-		reports.push({
-			runId: journal.runId,
-			phase: journal.phase,
-			action,
-			ok
-		});
-	}
-	await io.writeText(join(home, "merge", "recovery.json"), JSON.stringify(reports, null, 2));
-	return reports;
-}
-//#endregion
 //#region lib/types/commit-receipt.js
 /**
 * Commit receipt: <run>/commit.json, written immediately after a successful commit.
@@ -799,6 +738,113 @@ async function readCommitReceipt(io, home, runId, target) {
 	return null;
 }
 //#endregion
+//#region lib/types/commit-recovery.js
+/** Startup replay for non-terminal commit journals. */
+async function recoverCommitJournals(io, home, now, skillRoot, restoreSource, isActive) {
+	const dir = join(home, "merge", "journals");
+	if (!await io.exists(dir)) return [];
+	const files = (await io.list(dir)).filter((name) => name.endsWith(".json"));
+	const reports = [];
+	for (const file of files) {
+		let journal;
+		try {
+			journal = JSON.parse(await io.readText(join(dir, file)) ?? "");
+		} catch {
+			continue;
+		}
+		if (isActive?.(journal.runId) === true) {
+			reports.push({
+				runId: journal.runId,
+				phase: journal.phase,
+				action: "skipped-active-run",
+				ok: true
+			});
+			continue;
+		}
+		const recovery = recoveryFor(journal);
+		if (recovery === "none") continue;
+		if (recovery === "refuse") {
+			reports.push({
+				runId: journal.runId,
+				phase: journal.phase,
+				action: "unknown-phase-refused",
+				ok: false
+			});
+			continue;
+		}
+		let receipt = null;
+		try {
+			receipt = await readCommitReceipt(io, home, journal.runId, journal.target);
+		} catch {
+			reports.push({
+				runId: journal.runId,
+				phase: journal.phase,
+				action: "receipt-read-failed",
+				ok: false
+			});
+			continue;
+		}
+		if (receipt !== null) {
+			await writeJournal(io, home, {
+				...journal,
+				phase: "committed",
+				unitId: receipt.unitId,
+				updatedAt: new Date(now).toISOString()
+			});
+			reports.push({
+				runId: journal.runId,
+				phase: journal.phase,
+				action: "receipt-committed",
+				ok: true
+			});
+			continue;
+		}
+		const snapshot = await readCommitSnapshot(io, home, journal.runId);
+		let action = "report-only";
+		let ok = true;
+		if (recovery === "retry-commit") action = "no-host-write";
+		else if (snapshot === null) {
+			action = "snapshot-missing";
+			ok = false;
+		} else try {
+			if (recovery === "rollback-sources") {
+				let restoredArchived = restoreSource !== void 0;
+				if (restoreSource !== void 0) {
+					for (const source of snapshot.sourceBodies) if (!await restoreSource(source.name)) restoredArchived = false;
+				}
+				if (!restoredArchived) for (const source of snapshot.sourceBodies) await io.writeText(skillBodyPath(skillRoot, source.name), source.body);
+			}
+			if (snapshot.create && snapshot.targetBody === null) try {
+				await io.remove(skillBodyPath(skillRoot, snapshot.target));
+			} catch {}
+			else if (snapshot.targetBody !== null) await io.writeText(skillBodyPath(skillRoot, snapshot.target), snapshot.targetBody);
+			const oldPaths = new Set((snapshot.targetSupport ?? []).map((file) => file.path));
+			for (const file of snapshot.targetSupport ?? []) await io.writeText(skillFilePath(skillRoot, snapshot.target, file.path), file.content);
+			for (const file of snapshot.targetSupportAfter ?? []) if (!oldPaths.has(file.path)) try {
+				await io.remove(skillFilePath(skillRoot, snapshot.target, file.path));
+			} catch {}
+			for (const file of snapshot.sourceSupport ?? []) await io.writeText(skillFilePath(skillRoot, file.name, file.path), file.content);
+			action = recovery === "rollback-sources" ? "restored-sources-and-target" : "restored-target";
+		} catch (error) {
+			action = "restore-failed: " + (error instanceof Error ? error.message : String(error));
+			ok = false;
+		}
+		if (ok) await writeJournal(io, home, {
+			...journal,
+			phase: "rolled-back",
+			updatedAt: new Date(now).toISOString()
+		});
+		reports.push({
+			runId: journal.runId,
+			phase: journal.phase,
+			action,
+			ok
+		});
+	}
+	await io.writeText(join(home, "merge", "recovery.json"), JSON.stringify(reports, null, 2));
+	return reports;
+}
+//#endregion
 //#region lib/types/coordinator.js
 /**
 * Pure merge transition table. No io, no run shell, no subagents.
@@ -3552,11 +3598,16 @@ function skillLibraryPort(skills) {
 		async readSupportFiles(name) {
 			if (typeof skills.listSupportFiles !== "function") return [];
 			const probe = await skills.listSupportFiles(name);
-			if (probe.kind !== "present") return [];
-			return await Promise.all(probe.value.map(async (path) => ({
-				path,
-				content: await skills.readSupportFile(name, path) ?? ""
-			})));
+			if (probe.kind === "unknown") throw new Error("support file list unknown: " + name);
+			if (probe.kind === "absent") return [];
+			return await Promise.all(probe.value.map(async (path) => {
+				const content = await skills.readSupportFile(name, path);
+				if (content === null) throw new Error("support file disappeared: " + name + "/" + path);
+				return {
+					path,
+					content
+				};
+			}));
 		}
 	};
 }
@@ -3614,6 +3665,7 @@ function createMergeRuntime(deps) {
 				b: command.b
 			}];
 			const existing = (await loadMergeState(deps.io, deps.home)).state.rows.find((row) => row.pairKey === pairKey(command.a, command.b));
+			if (existing?.state === "skipped") await dropRow(deps.io, deps.home, deps.now, existing.id);
 			if (existing?.state === "pending") return {
 				ok: false,
 				text: "该 pair 已在待批准队列，先 approve/reject/discard: " + existing.id
@@ -4482,8 +4534,12 @@ function apply(ctx, rawConfig = {}) {
 				...policyStageLimits(policySnapshot?.get?.())
 			}
 		});
-		recoverCommitJournals(io, home, Date.now(), skillsRoot(), async (name) => {
+		const mergeSkillsRoot = skills.root ?? skillsRoot();
+		recoverCommitJournals(io, home, Date.now(), mergeSkillsRoot, async (name) => {
 			return (await skills.restoreFromArchive(name)).ok;
+		}, (runId) => {
+			const run = runs.find(runId);
+			return run !== void 0 && run.state === "running";
 		}).then((reports) => {
 			if (reports.length > 0) ctx.logger.info("merge commit recovery: " + reports.map((report) => report.runId + "=" + report.action).join(", "));
 		}).catch((error) => {

package/lib/types/adapters/library.js

@@ -13,9 +13,16 @@ export function skillLibraryPort(skills) {
             if (typeof skills.listSupportFiles !== 'function')
                 return [];
             const probe = await skills.listSupportFiles(name);
-            if (probe.kind !== 'present')
+            if (probe.kind === 'unknown')
+                throw new Error('support file list unknown: ' + name);
+            if (probe.kind === 'absent')
                 return [];
-            return await Promise.all(probe.value.map(async (path) => ({ path, content: await skills.readSupportFile(name, path) ?? '' })));
+            return await Promise.all(probe.value.map(async (path) => {
+                const content = await skills.readSupportFile(name, path);
+                if (content === null)
+                    throw new Error('support file disappeared: ' + name + '/' + path);
+                return { path, content };
+            }));
         },
     };
 }

package/lib/types/commit-recovery.d.ts

@@ -6,5 +6,5 @@ export interface CommitRecoveryReport {
     readonly action: string;
     readonly ok: boolean;
 }
-export declare function recoverCommitJournals(io: EvolutionIoLike, home: string, now: number, skillRoot: string, restoreSource?: (name: string) => Promise<boolean>): Promise<readonly CommitRecoveryReport[]>;
+export declare function recoverCommitJournals(io: EvolutionIoLike, home: string, now: number, skillRoot: string, restoreSource?: (name: string) => Promise<boolean>, isActive?: (runId: string) => boolean): Promise<readonly CommitRecoveryReport[]>;
 //# sourceMappingURL=commit-recovery.d.ts.map

package/lib/types/commit-recovery.js

@@ -3,7 +3,8 @@ import { join } from 'node:path';
 import { recoveryFor } from "./domain/commit-journal.js";
 import { writeJournal } from "./domain/commit-journal-store.js";
 import { readCommitSnapshot, skillBodyPath, skillFilePath } from "./commit-snapshot.js";
-export async function recoverCommitJournals(io, home, now, skillRoot, restoreSource) {
+import { readCommitReceipt } from "./commit-receipt.js";
+export async function recoverCommitJournals(io, home, now, skillRoot, restoreSource, isActive) {
     const dir = join(home, 'merge', 'journals');
     if (!(await io.exists(dir)))
         return [];
@@ -17,9 +18,30 @@ export async function recoverCommitJournals(io, home, now, skillRoot, restoreSou
         catch {
             continue;
         }
+        if (isActive?.(journal.runId) === true) {
+            reports.push({ runId: journal.runId, phase: journal.phase, action: 'skipped-active-run', ok: true });
+            continue;
+        }
         const recovery = recoveryFor(journal);
         if (recovery === 'none')
             continue;
+        if (recovery === 'refuse') {
+            reports.push({ runId: journal.runId, phase: journal.phase, action: 'unknown-phase-refused', ok: false });
+            continue;
+        }
+        let receipt = null;
+        try {
+            receipt = await readCommitReceipt(io, home, journal.runId, journal.target);
+        }
+        catch {
+            reports.push({ runId: journal.runId, phase: journal.phase, action: 'receipt-read-failed', ok: false });
+            continue;
+        }
+        if (receipt !== null) {
+            await writeJournal(io, home, { ...journal, phase: 'committed', unitId: receipt.unitId, updatedAt: new Date(now).toISOString() });
+            reports.push({ runId: journal.runId, phase: journal.phase, action: 'receipt-committed', ok: true });
+            continue;
+        }
         const snapshot = await readCommitSnapshot(io, home, journal.runId);
         let action = 'report-only';
         let ok = true;
@@ -75,7 +97,8 @@ export async function recoverCommitJournals(io, home, now, skillRoot, restoreSou
                 ok = false;
             }
         }
-        await writeJournal(io, home, { ...journal, phase: ok ? 'rolled-back' : 'failed', updatedAt: new Date(now).toISOString() });
+        if (ok)
+            await writeJournal(io, home, { ...journal, phase: 'rolled-back', updatedAt: new Date(now).toISOString() });
         reports.push({ runId: journal.runId, phase: journal.phase, action, ok });
     }
     await io.writeText(join(home, 'merge', 'recovery.json'), JSON.stringify(reports, null, 2));

package/lib/types/commit-snapshot.js

@@ -45,7 +45,7 @@ export function isSafeSkillName(name) {
     return /^[A-Za-z0-9][A-Za-z0-9._-]*$/.test(name) && name !== '.' && name !== '..';
 }
 export function skillFilePath(root, name, path) {
-    if (!isSafeSkillName(name) || path.startsWith('/') || path.includes('..'))
+    if (!isSafeSkillName(name) || path.startsWith('/') || path.split('/').includes('..'))
         throw new Error('unsafe skill file: ' + name + '/' + path);
     const base = resolve(root, name);
     const file = resolve(base, path);

package/lib/types/domain/commit-journal.d.ts

@@ -10,7 +10,7 @@ export interface CommitJournal {
     readonly updatedAt: string;
 }
 export declare function advanceJournal(journal: CommitJournal, phase: JournalPhase, at: string): CommitJournal;
-export type JournalRecovery = 'none' | 'retry-commit' | 'rollback-target' | 'rollback-sources';
+export type JournalRecovery = 'none' | 'retry-commit' | 'rollback-target' | 'rollback-sources' | 'refuse';
 /** What a restart may do with the last durable phase. */
 export declare function recoveryFor(journal: CommitJournal): JournalRecovery;
 //# sourceMappingURL=commit-journal.d.ts.map

package/lib/types/domain/commit-journal.js

@@ -20,6 +20,7 @@ export function recoveryFor(journal) {
         case 'committed':
         case 'rolled-back':
         case 'failed': return 'none';
+        default: return 'refuse';
     }
 }
 //# sourceMappingURL=commit-journal.js.map

package/lib/types/index.js

@@ -138,9 +138,13 @@ export function apply(ctx, rawConfig = {}) {
             io, ctx, authority: authorityFrom({ origin: 'subagent' }),
             limits: { ...DEFAULT_SKILL_LIMITS, ...policyStageLimits(policySnapshot?.get?.()) },
         });
-        void recoverCommitJournals(io, home, Date.now(), skillsRoot(), async (name) => {
+        const mergeSkillsRoot = skills.root ?? skillsRoot();
+        void recoverCommitJournals(io, home, Date.now(), mergeSkillsRoot, async (name) => {
             const result = await skills.restoreFromArchive(name);
             return result.ok;
+        }, (runId) => {
+            const run = runs.find(runId);
+            return run !== undefined && run.state === 'running';
         }).then((reports) => { if (reports.length > 0)
             ctx.logger.info('merge commit recovery: ' + reports.map(report => report.runId + '=' + report.action).join(', ')); }).catch((error) => { ctx.logger.warn('merge commit recovery failed: ' + (error instanceof Error ? error.message : String(error))); });
         const channel = subagentChannelAdapter(createSubagentChannel(ctx));

package/lib/types/runtime.js

@@ -55,6 +55,9 @@ export function createMergeRuntime(deps) {
                 }
                 const pair = [{ a: command.a, b: command.b }];
                 const existing = (await loadMergeState(deps.io, deps.home)).state.rows.find(row => row.pairKey === pairKey(command.a, command.b));
+                if (existing?.state === 'skipped') {
+                    await dropRow(deps.io, deps.home, deps.now, existing.id);
+                }
                 if (existing?.state === 'pending') {
                     return { ok: false, text: '该 pair 已在待批准队列，先 approve/reject/discard: ' + existing.id };
                 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-merge",
   "description": "Skill auto-merge domain: screening, two-round authoring, queues and commit orchestration (community build)",
-  "version": "0.30.3",
+  "version": "0.30.4",
   "publishConfig": {
     "access": "public"
   },
@@ -27,8 +27,8 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.5",
-    "@lmzhen/dsh-evolution-core": "^0.30.3",
-    "@lmzhen/dsh-evolution-merge-engine": "^0.30.3"
+    "@lmzhen/dsh-evolution-core": "^0.30.4",
+    "@lmzhen/dsh-evolution-merge-engine": "^0.30.4"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.5"