mdkg 0.3.6 → 0.3.8

package/dist/commands/spec.js

@@ -3,6 +3,9 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.runSpecListCommand = runSpecListCommand;
 exports.runSpecShowCommand = runSpecShowCommand;
 exports.runSpecValidateCommand = runSpecValidateCommand;
+exports.runManifestListCommand = runManifestListCommand;
+exports.runManifestShowCommand = runManifestShowCommand;
+exports.runManifestValidateCommand = runManifestValidateCommand;
 const capability_1 = require("./capability");
 const validate_1 = require("./validate");
 const errors_1 = require("../util/errors");
@@ -24,6 +27,9 @@ function loadSpecRecords(options) {
     };
     return sortSpecRecords((0, capability_1.filterCapabilityRecords)((0, capability_1.loadCapabilityRecords)(listOptions), listOptions));
 }
+function commandSurface(options) {
+    return options.surface ?? "spec";
+}
 function matchesSpecRef(record, id) {
     const normalized = id.toLowerCase();
     return (record.id === id ||
@@ -34,15 +40,15 @@ function matchesSpecRef(record, id) {
         record.path.toLowerCase() === normalized ||
         record.aliases.some((alias) => alias.toLowerCase() === normalized));
 }
-function resolveSpecRecord(records, id) {
+function resolveSpecRecord(records, id, surface) {
     const matches = records.filter((record) => matchesSpecRef(record, id));
     if (matches.length === 1) {
         return matches[0];
     }
     if (matches.length > 1) {
-        throw new errors_1.UsageError(`SPEC reference is ambiguous: ${id}`);
+        throw new errors_1.UsageError(`${surface === "manifest" ? "manifest capability" : "SPEC"} reference is ambiguous: ${id}`);
     }
-    throw new errors_1.NotFoundError(`SPEC not found: ${id}`);
+    throw new errors_1.NotFoundError(`${surface === "manifest" ? "manifest capability" : "SPEC"} not found: ${id}`);
 }
 function stringValue(value) {
     return typeof value === "string" ? value : undefined;
@@ -50,33 +56,74 @@ function stringValue(value) {
 function stringArrayValue(value) {
     return Array.isArray(value) ? value.filter((item) => typeof item === "string") : [];
 }
-function printSpecList(records, json) {
+function compatibilityLabel(record) {
+    const mode = record.manifest?.compatibility_mode;
+    if (mode === "legacy") {
+        return "legacy SPEC.md";
+    }
+    if (mode === "transitional") {
+        return "transitional MANIFEST.md type: spec";
+    }
+    return "MANIFEST.md";
+}
+function legacySpecDeprecation() {
+    return "mdkg spec is a legacy alias for mdkg manifest during the one-compatibility-release bridge.";
+}
+function printSpecList(records, json, surface = "spec") {
     if (json) {
-        console.log(JSON.stringify({
-            kind: "spec",
-            count: records.length,
-            items: records,
-        }, null, 2));
+        const receipt = surface === "manifest"
+            ? {
+                kind: "manifest",
+                compatibility_kind: "spec",
+                count: records.length,
+                items: records,
+            }
+            : {
+                kind: "spec",
+                canonical_kind: "manifest",
+                legacy_alias: true,
+                deprecation: legacySpecDeprecation(),
+                count: records.length,
+                items: records,
+            };
+        console.log(JSON.stringify(receipt, null, 2));
         return;
     }
+    if (surface === "spec") {
+        console.log(legacySpecDeprecation());
+    }
     if (records.length === 0) {
-        console.log("no SPEC.md capabilities found");
+        console.log("no MANIFEST.md/SPEC.md capabilities found");
         return;
     }
-    console.log(`SPEC.md capabilities: ${records.length}`);
+    console.log(`${surface === "manifest" ? "MANIFEST.md" : "MANIFEST.md/SPEC.md"} capabilities: ${records.length}`);
     for (const record of records) {
         const kind = stringValue(record.spec?.spec_kind);
         const kindLabel = kind ? ` | ${kind}` : "";
-        console.log(`${record.qid} | ${record.visibility}${kindLabel} | ${record.title}`);
+        console.log(`${record.qid} | ${record.visibility}${kindLabel} | ${compatibilityLabel(record)} | ${record.title}`);
     }
 }
-function printSpec(record, json) {
+function printSpec(record, json, surface = "spec") {
     if (json) {
-        console.log(JSON.stringify({ kind: "spec", item: record }, null, 2));
+        const receipt = surface === "manifest"
+            ? { kind: "manifest", compatibility_kind: "spec", item: record }
+            : {
+                kind: "spec",
+                canonical_kind: "manifest",
+                legacy_alias: true,
+                deprecation: legacySpecDeprecation(),
+                item: record,
+            };
+        console.log(JSON.stringify(receipt, null, 2));
         return;
     }
+    if (surface === "spec") {
+        console.log(legacySpecDeprecation());
+    }
     console.log(`${record.qid} | ${record.visibility}`);
     console.log(`title: ${record.title}`);
+    console.log(`semantic_kind: ${record.manifest?.semantic_kind ?? "manifest"}`);
+    console.log(`compatibility: ${compatibilityLabel(record)}`);
     const specKind = stringValue(record.spec?.spec_kind);
     if (specKind) {
         console.log(`spec_kind: ${specKind}`);
@@ -88,14 +135,26 @@ function printSpec(record, json) {
     }
 }
 function runSpecListCommand(options) {
-    printSpecList(loadSpecRecords(options), options.json);
+    const surface = commandSurface(options);
+    printSpecList(loadSpecRecords(options), options.json, surface);
 }
 function runSpecShowCommand(options) {
-    printSpec(resolveSpecRecord(loadSpecRecords(options), options.id), options.json);
+    const surface = commandSurface(options);
+    printSpec(resolveSpecRecord(loadSpecRecords(options), options.id, surface), options.json, surface);
 }
 function runSpecValidateCommand(options) {
+    const surface = commandSurface(options);
     if (options.id) {
-        resolveSpecRecord(loadSpecRecords(options), options.id);
+        resolveSpecRecord(loadSpecRecords(options), options.id, surface);
     }
     (0, validate_1.runValidateCommand)({ root: options.root, json: options.json });
 }
+function runManifestListCommand(options) {
+    runSpecListCommand({ ...options, surface: "manifest" });
+}
+function runManifestShowCommand(options) {
+    runSpecShowCommand({ ...options, surface: "manifest" });
+}
+function runManifestValidateCommand(options) {
+    runSpecValidateCommand({ ...options, surface: "manifest" });
+}

package/dist/commands/status.js

@@ -229,6 +229,7 @@ function collectStatus(root) {
             selected_exists: selected.state === null ? null : !selectedMissing,
             selected_achieved: selected.state === null ? null : selectedAchieved,
             active_node: selectedNode?.attributes.active_node ?? null,
+            last_active_node: selectedNode?.attributes.last_active_node ?? null,
             goal_state: selectedNode?.attributes.goal_state ?? null,
             status: selectedNode?.status ?? null,
         },

package/dist/commands/subgraph.js

@@ -323,6 +323,9 @@ function runSubgraphVerifyCommand(options) {
         throw new errors_1.ValidationError("subgraph verify failed");
     }
 }
+function dirtySourceGuidance(paths) {
+    return `source_path has dirty tracked changes: ${paths.join(", ")}; commit or stash the child repo first, then refresh the root-owned subgraph bundle from the clean accepted child commit`;
+}
 function pushAuditCheck(receipt, check) {
     receipt.checks.push(check);
     if (!check.ok && check.severity === "error") {
@@ -349,7 +352,7 @@ function auditOneAlias(options) {
         source_repo: options.health.source_repo,
         capability_summary: {
             node_count: options.nodeTypes.length,
-            spec_count: options.nodeTypes.filter((type) => type === "spec").length,
+            spec_count: options.nodeTypes.filter((type) => type === "manifest" || type === "spec").length,
             work_count: options.nodeTypes.filter((type) => type === "work").length,
             skill_count: options.nodeTypes.filter((type) => type === "skill").length,
         },
@@ -392,7 +395,7 @@ function auditOneAlias(options) {
                 ok: !gitState.dirtyTracked,
                 severity: "warning",
                 message: gitState.dirtyTracked
-                    ? `source_path has dirty tracked changes: ${gitState.dirtyTrackedPaths.join(", ")}`
+                    ? dirtySourceGuidance(gitState.dirtyTrackedPaths)
                     : "source_path has no dirty tracked changes",
                 path: options.subgraph.source_path,
                 details: { dirty_tracked_paths: gitState.dirtyTrackedPaths },
@@ -539,7 +542,7 @@ function upgradePlanForAlias(options) {
         blockers.push(sourceGitError.message);
     }
     if (options.audit.dirty_tracked) {
-        blockers.push(`source_path has dirty tracked changes: ${options.audit.dirty_tracked_paths?.join(", ")}`);
+        blockers.push(dirtySourceGuidance(options.audit.dirty_tracked_paths ?? []));
     }
     const rootOwnedErrors = options.audit.checks.filter((check) => check.id === "subgraph.bundle.root_owned" && !check.ok);
     blockers.push(...rootOwnedErrors.map((check) => check.message));
@@ -705,7 +708,7 @@ function inspectSourcePath(root, alias, subgraph, allowDirty) {
         ? status.stdout.split(/\r?\n/).map((line) => line.slice(3)).filter(Boolean).sort()
         : [];
     if (dirtyTrackedPaths.length > 0 && !allowDirty) {
-        throw new errors_1.UsageError(`subgraph ${alias} source_path has dirty tracked changes: ${dirtyTrackedPaths.join(", ")}`);
+        throw new errors_1.UsageError(`subgraph ${alias} ${dirtySourceGuidance(dirtyTrackedPaths)}`);
     }
     return {
         sourceRoot,

package/dist/commands/task.js

@@ -297,6 +297,7 @@ function runTaskDoneCommandLocked(options) {
             ws: loaded.ws,
             relates: loaded.id,
             scope: loaded.id,
+            kind: options.checkpointKind,
             runId: options.runId,
             note: `checkpoint created from mdkg task done for ${loaded.id}`,
             now,
@@ -309,6 +310,7 @@ function runTaskDoneCommandLocked(options) {
             ws: loaded.ws,
             relates: loaded.id,
             scope: loaded.id,
+            kind: options.checkpointKind,
             runId: options.runId,
             note: `checkpoint created from mdkg task done for ${loaded.id}`,
             now,

package/dist/commands/upgrade.js

@@ -13,6 +13,10 @@ const paths_1 = require("../core/paths");
 const version_1 = require("../core/version");
 const project_db_1 = require("../core/project_db");
 const errors_1 = require("../util/errors");
+const frontmatter_1 = require("../graph/frontmatter");
+const workspace_files_1 = require("../graph/workspace_files");
+const agent_file_types_1 = require("../graph/agent_file_types");
+const atomic_1 = require("../util/atomic");
 const init_manifest_1 = require("./init_manifest");
 const skill_support_1 = require("./skill_support");
 const skill_mirror_1 = require("./skill_mirror");
@@ -63,6 +67,13 @@ function writeFile(filePath, content) {
     fs_1.default.mkdirSync(path_1.default.dirname(filePath), { recursive: true });
     fs_1.default.writeFileSync(filePath, content, "utf8");
 }
+function repoRelativePath(root, filePath) {
+    return path_1.default.relative(root, filePath).split(path_1.default.sep).join("/");
+}
+function renderFrontmatterDocument(frontmatter, body) {
+    const frontmatterBlock = ["---", ...(0, frontmatter_1.formatFrontmatter)(frontmatter, frontmatter_1.DEFAULT_FRONTMATTER_KEY_ORDER), "---"].join("\n");
+    return body.length > 0 ? `${frontmatterBlock}\n${body}` : frontmatterBlock;
+}
 function createSummary() {
     return {
         created: 0,
@@ -273,6 +284,114 @@ function migrateConfigIfNeeded(root, dryRun, summary, changes) {
         writeFile(cfgPath, `${JSON.stringify(nextConfig.config, null, 2)}\n`);
     }
 }
+function migrateClosedGoalActiveNodes(root, dryRun, summary, changes) {
+    const config = (0, config_1.validateConfigSchema)((0, migrate_1.migrateConfig)(JSON.parse(fs_1.default.readFileSync((0, paths_1.configPath)(root), "utf8"))).config);
+    const filesByAlias = (0, workspace_files_1.listWorkspaceDocFilesByAlias)(root, config);
+    let planned = 0;
+    for (const files of Object.values(filesByAlias)) {
+        for (const filePath of files) {
+            const content = fs_1.default.readFileSync(filePath, "utf8");
+            let parsed;
+            try {
+                parsed = (0, frontmatter_1.parseFrontmatter)(content, filePath);
+            }
+            catch {
+                continue;
+            }
+            const frontmatter = { ...parsed.frontmatter };
+            if (frontmatter.type !== "goal") {
+                continue;
+            }
+            const status = typeof frontmatter.status === "string" ? frontmatter.status : "";
+            const goalState = typeof frontmatter.goal_state === "string" ? frontmatter.goal_state : "";
+            const activeNode = typeof frontmatter.active_node === "string" ? frontmatter.active_node : undefined;
+            if (!activeNode || (status !== "done" && goalState !== "achieved")) {
+                continue;
+            }
+            const relativePath = path_1.default.relative(root, filePath).split(path_1.default.sep).join("/");
+            const lastActiveNode = typeof frontmatter.last_active_node === "string" ? frontmatter.last_active_node : undefined;
+            if (lastActiveNode && lastActiveNode !== activeNode) {
+                planned += 1;
+                record(summary, changes, {
+                    path: relativePath,
+                    category: "goal_lifecycle",
+                    action: "conflict",
+                    reason: `closed goal has active_node ${activeNode} but different last_active_node ${lastActiveNode}; local content preserved`,
+                });
+                continue;
+            }
+            planned += 1;
+            record(summary, changes, {
+                path: relativePath,
+                category: "goal_lifecycle",
+                action: "migrate",
+                reason: "move closed goal active_node to last_active_node",
+            });
+            if (!dryRun) {
+                if (!lastActiveNode) {
+                    frontmatter.last_active_node = activeNode;
+                }
+                delete frontmatter.active_node;
+                writeFile(filePath, renderFrontmatterDocument(frontmatter, parsed.body.length > 0 ? parsed.body : ""));
+            }
+        }
+    }
+    if (planned === 0) {
+        summary.unchanged += 1;
+    }
+}
+function migrateLegacySpecManifests(root, dryRun, summary, changes) {
+    const config = (0, config_1.validateConfigSchema)((0, migrate_1.migrateConfig)(JSON.parse(fs_1.default.readFileSync((0, paths_1.configPath)(root), "utf8"))).config);
+    const filesByAlias = (0, workspace_files_1.listWorkspaceDocFilesByAlias)(root, config);
+    let planned = 0;
+    for (const files of Object.values(filesByAlias)) {
+        for (const filePath of files) {
+            if (path_1.default.basename(filePath) !== agent_file_types_1.LEGACY_SPEC_BASENAME) {
+                continue;
+            }
+            const content = fs_1.default.readFileSync(filePath, "utf8");
+            let parsed;
+            try {
+                parsed = (0, frontmatter_1.parseFrontmatter)(content, filePath);
+            }
+            catch {
+                continue;
+            }
+            if (parsed.frontmatter.type !== "spec") {
+                continue;
+            }
+            planned += 1;
+            const targetPath = path_1.default.join(path_1.default.dirname(filePath), agent_file_types_1.CANONICAL_MANIFEST_BASENAME);
+            const relativePath = repoRelativePath(root, filePath);
+            const relativeTargetPath = repoRelativePath(root, targetPath);
+            if (fs_1.default.existsSync(targetPath)) {
+                record(summary, changes, {
+                    path: relativePath,
+                    target_path: relativeTargetPath,
+                    category: "manifest_migration",
+                    action: "conflict",
+                    reason: "sibling MANIFEST.md already exists; legacy SPEC.md content preserved",
+                });
+                continue;
+            }
+            record(summary, changes, {
+                path: relativePath,
+                target_path: relativeTargetPath,
+                category: "manifest_migration",
+                action: "migrate",
+                reason: "rename legacy SPEC.md to MANIFEST.md and normalize type: spec to type: manifest",
+            });
+            if (!dryRun) {
+                const frontmatter = { ...parsed.frontmatter, type: "manifest" };
+                (0, atomic_1.atomicWriteFile)(filePath, renderFrontmatterDocument(frontmatter, parsed.body));
+                fs_1.default.renameSync(filePath, targetPath);
+            }
+        }
+    }
+    if (planned === 0) {
+        summary.unchanged += 1;
+    }
+}
 function isIgnoredBySimpleGitignore(root, relativePath) {
     const ignorePath = path_1.default.join(root, ".gitignore");
     if (!fs_1.default.existsSync(ignorePath)) {
@@ -362,6 +481,9 @@ function ensureArchiveIgnorePolicy(root, dryRun, summary, changes) {
 function isWritableChange(change) {
     return change.action === "create" || change.action === "update" || change.action === "migrate" || change.action === "sync";
 }
+function writablePathForChange(change) {
+    return change.target_path ?? change.path;
+}
 function buildApplySideEffects(options) {
     const hasDirectWrites = options.changes.some(isWritableChange);
     if (!hasDirectWrites && options.existingManifest) {
@@ -408,7 +530,8 @@ function emitHumanReceipt(receipt) {
     }
     else {
         for (const change of receipt.changes) {
-            console.log(`  ${change.action}: ${change.path} (${change.reason})`);
+            const target = change.target_path ? ` -> ${change.target_path}` : "";
+            console.log(`  ${change.action}: ${change.path}${target} (${change.reason})`);
         }
     }
     if (receipt.apply_side_effects.length > 0) {
@@ -440,6 +563,8 @@ function runUpgradeCommand(options) {
     const agentWorkspace = isAgentWorkspace(root);
     const managedCurrentFiles = [];
     migrateConfigIfNeeded(root, dryRun, summary, changes);
+    migrateClosedGoalActiveNodes(root, dryRun, summary, changes);
+    migrateLegacySpecManifests(root, dryRun, summary, changes);
     for (const file of currentManifest.files) {
         if (!shouldIncludeFile(file, agentWorkspace)) {
             continue;
@@ -483,8 +608,8 @@ function runUpgradeCommand(options) {
         }
     }
     const preservedCustomizations = changes.filter((change) => change.action === "conflict");
-    const blockingConflicts = [];
-    const willWritePaths = changes.filter(isWritableChange).map((change) => change.path);
+    const blockingConflicts = changes.filter((change) => change.action === "conflict" && change.category === "manifest_migration");
+    const willWritePaths = changes.filter(isWritableChange).map(writablePathForChange);
     const receipt = {
         action: "upgrade",
         dry_run: dryRun,