@nimiplatform/nimi-coding 0.1.0 → 0.2.1

package/cli/lib/contracts.mjs

@@ -1,4 +1,5 @@
 import path from "node:path";
+import { fileURLToPath } from "node:url";
 
 import {
   loadBlueprintReference as loadBlueprintReferenceInternal,
@@ -27,6 +28,9 @@ import {
   validateSpecReconstructionSummary as validateSpecReconstructionSummaryInternal,
 } from "./internal/contracts-validators.mjs";
 
+const PACKAGE_REF_PREFIX = "package://@nimiplatform/nimi-coding/";
+const PACKAGE_ROOT = path.resolve(path.dirname(fileURLToPath(import.meta.url)), "..", "..");
+
 export function findCommandGatingRule(commandGatingMatrix, command, skillId = null) {
   return matchCommandGatingRule(commandGatingMatrix, command, skillId);
 }
@@ -117,9 +121,9 @@ async function loadYamlWithFallback(projectRoot, primaryRef, fallbackRef) {
     };
   }
 
-  const fallbackText = await readTextIfFile(path.join(projectRoot, fallbackRef));
+  const fallbackText = await readTextIfFile(path.join(PACKAGE_ROOT, fallbackRef));
   return {
-    path: fallbackRef,
+    path: `${PACKAGE_REF_PREFIX}${fallbackRef}`,
     text: fallbackText,
     data: parseYamlText(fallbackText),
   };
@@ -143,21 +147,21 @@ export async function loadTopicRuntimeContracts(projectRoot) {
     validationPolicy,
     authorityConvergencePolicy,
   ] = await Promise.all([
-    loadYamlWithFallback(projectRoot, ".nimi/contracts/topic.schema.yaml", "nimi-coding/contracts/topic.schema.yaml"),
-    loadYamlWithFallback(projectRoot, ".nimi/contracts/wave.schema.yaml", "nimi-coding/contracts/wave.schema.yaml"),
-    loadYamlWithFallback(projectRoot, ".nimi/contracts/packet.schema.yaml", "nimi-coding/contracts/packet.schema.yaml"),
-    loadYamlWithFallback(projectRoot, ".nimi/contracts/result.schema.yaml", "nimi-coding/contracts/result.schema.yaml"),
-    loadYamlWithFallback(projectRoot, ".nimi/contracts/closeout.schema.yaml", "nimi-coding/contracts/closeout.schema.yaml"),
-    loadYamlWithFallback(projectRoot, ".nimi/contracts/remediation.schema.yaml", "nimi-coding/contracts/remediation.schema.yaml"),
-    loadYamlWithFallback(projectRoot, ".nimi/contracts/decision-review.schema.yaml", "nimi-coding/contracts/decision-review.schema.yaml"),
-    loadYamlWithFallback(projectRoot, ".nimi/contracts/pending-note.schema.yaml", "nimi-coding/contracts/pending-note.schema.yaml"),
-    loadYamlWithFallback(projectRoot, ".nimi/contracts/topic-step-decision.schema.yaml", "nimi-coding/contracts/topic-step-decision.schema.yaml"),
-    loadYamlWithFallback(projectRoot, ".nimi/contracts/topic-run-ledger.schema.yaml", "nimi-coding/contracts/topic-run-ledger.schema.yaml"),
-    loadYamlWithFallback(projectRoot, ".nimi/contracts/forbidden-shortcuts.catalog.yaml", "nimi-coding/contracts/forbidden-shortcuts.catalog.yaml"),
-    loadYamlWithFallback(projectRoot, ".nimi/methodology/topic-lifecycle-report.yaml", "nimi-coding/methodology/topic-lifecycle-report.yaml"),
-    loadYamlWithFallback(projectRoot, ".nimi/methodology/four-closure-policy.yaml", "nimi-coding/methodology/four-closure-policy.yaml"),
-    loadYamlWithFallback(projectRoot, ".nimi/methodology/topic-validation-policy.yaml", "nimi-coding/methodology/topic-validation-policy.yaml"),
-    loadYamlWithFallback(projectRoot, ".nimi/methodology/authority-convergence-policy.yaml", "nimi-coding/methodology/authority-convergence-policy.yaml"),
+    loadYamlWithFallback(projectRoot, ".nimi/contracts/topic.schema.yaml", "contracts/topic.schema.yaml"),
+    loadYamlWithFallback(projectRoot, ".nimi/contracts/wave.schema.yaml", "contracts/wave.schema.yaml"),
+    loadYamlWithFallback(projectRoot, ".nimi/contracts/packet.schema.yaml", "contracts/packet.schema.yaml"),
+    loadYamlWithFallback(projectRoot, ".nimi/contracts/result.schema.yaml", "contracts/result.schema.yaml"),
+    loadYamlWithFallback(projectRoot, ".nimi/contracts/closeout.schema.yaml", "contracts/closeout.schema.yaml"),
+    loadYamlWithFallback(projectRoot, ".nimi/contracts/remediation.schema.yaml", "contracts/remediation.schema.yaml"),
+    loadYamlWithFallback(projectRoot, ".nimi/contracts/decision-review.schema.yaml", "contracts/decision-review.schema.yaml"),
+    loadYamlWithFallback(projectRoot, ".nimi/contracts/pending-note.schema.yaml", "contracts/pending-note.schema.yaml"),
+    loadYamlWithFallback(projectRoot, ".nimi/contracts/topic-step-decision.schema.yaml", "contracts/topic-step-decision.schema.yaml"),
+    loadYamlWithFallback(projectRoot, ".nimi/contracts/topic-run-ledger.schema.yaml", "contracts/topic-run-ledger.schema.yaml"),
+    loadYamlWithFallback(projectRoot, ".nimi/contracts/forbidden-shortcuts.catalog.yaml", "contracts/forbidden-shortcuts.catalog.yaml"),
+    loadYamlWithFallback(projectRoot, ".nimi/methodology/topic-lifecycle-report.yaml", "methodology/topic-lifecycle-report.yaml"),
+    loadYamlWithFallback(projectRoot, ".nimi/methodology/four-closure-policy.yaml", "methodology/four-closure-policy.yaml"),
+    loadYamlWithFallback(projectRoot, ".nimi/methodology/topic-validation-policy.yaml", "methodology/topic-validation-policy.yaml"),
+    loadYamlWithFallback(projectRoot, ".nimi/methodology/authority-convergence-policy.yaml", "methodology/authority-convergence-policy.yaml"),
   ]);
 
   return {

package/cli/lib/handoff.mjs

@@ -8,7 +8,6 @@ import {
   HOST_ADAPTER_CONFIG_REF,
   SKILL_RESULT_CONTRACT_REFS,
   SPEC_GENERATION_AUDIT_CONTRACT_REF,
-  SPEC_GENERATION_AUDIT_REF,
   SPEC_RECONSTRUCTION_RESULT_CONTRACT_REF,
 } from "../constants.mjs";
 import {
@@ -79,14 +78,34 @@ function evaluateSkillReadiness(skillId, doctorResult) {
       reason: "Bootstrap or handoff validation is failing; repair doctor errors before exporting handoff payloads",
     };
   }
-  if (skillId === "spec_reconstruction") {
+  const usesV2SurfaceModel = doctorResult.specGenerationInputs?.mode === "class_filtered";
+  if (usesV2SurfaceModel && (doctorResult.commandGating?.entries ?? []).length === 0) {
+    if (skillId === "spec_reconstruction") {
+      return {
+        ok: true,
+        reason: "Projects may delegate spec reconstruction to an external AI host when canonical tree work is needed",
+      };
+    }
+    if (doctorResult.canonicalTree?.requiredFilesValid === true && doctorResult.specGenerationAudit?.ok === true) {
+      return {
+        ok: true,
+        reason: "Skill prerequisites are satisfied by the current project-local truth",
+      };
+    }
     return {
-      ok: true,
-      reason: "Projects may delegate spec reconstruction to an external AI host when canonical tree work is needed",
+      ok: false,
+      reason: "This skill is not allowed in the current lifecycle state",
     };
   }
   const rule = (doctorResult.commandGating?.entries ?? []).find((entry) => entry.command === "handoff" && entry.skill === skillId) ?? null;
-  return commandRuleAllowsCurrentState(rule, doctorResult);
+  const gatedReadiness = commandRuleAllowsCurrentState(rule, doctorResult);
+  if (!gatedReadiness.ok || skillId !== "spec_reconstruction") {
+    return gatedReadiness;
+  }
+  return {
+    ok: true,
+    reason: "Projects may delegate spec reconstruction to an external AI host when canonical tree work is needed",
+  };
 }
 function getSkillSpecificExpectations(
   skillId,
@@ -108,8 +127,8 @@ function getSkillSpecificExpectations(
       expectedArtifactKinds: [],
       skillExpectedResults: [
         "generate_canonical_tree_under_.nimi/spec",
-        "establish_kernel_markdown_and_kernel_tables_before_generated_views_or_guides",
-        "record_file_level_generation_audit_under_.nimi/spec/_meta/spec-generation-audit.yaml",
+        "establish_kernel_markdown_and_kernel_tables_before_thin_guides",
+        "record_file_level_generation_audit_under_.nimi/local/state/spec-generation/spec-generation-audit.yaml",
         `satisfy_canonical_tree_completion_declared_in_${resultContractRef}`,
       ],
     };
@@ -264,7 +283,7 @@ export async function buildHandoffPayload(projectRoot, skillId) {
       acceptanceMode: specGenerationInputs.acceptanceMode ?? "canonical_tree_validity_without_blueprint",
       generationOrder: specGenerationInputs.generationOrder ?? [],
       inferenceRules: specGenerationInputs.inferenceRules ?? [],
-      auditRef: SPEC_GENERATION_AUDIT_REF,
+      auditRef: ".nimi/local/state/spec-generation/spec-generation-audit.yaml",
       auditContractRef: SPEC_GENERATION_AUDIT_CONTRACT_REF,
       requiredFileClasses: [
         "INDEX.md",
@@ -272,7 +291,6 @@ export async function buildHandoffPayload(projectRoot, skillId) {
         "domain kernel/tables/**",
       ],
       optionalFileClasses: [
-        "domain kernel/generated/**",
         "thin domain guides",
       ],
       minimalRequiredOutputs: Array.isArray(specReconstructionDocument?.reconstruction?.target_tree_shape?.minimal_required_outputs)
@@ -586,8 +604,8 @@ export function formatHandoffPrompt(payload) {
       "- 直接在 `.nimi/spec/**` 下构建 canonical tree；不要把 compact summary 当作主要输出。",
     ));
     lines.push(localize(
-      "- Write `.nimi/spec/_meta/spec-generation-audit.yaml` alongside the canonical tree. Every generated canonical file must record source refs, source basis, and unresolved gaps.",
-      "- 在 canonical tree 旁边写出 `.nimi/spec/_meta/spec-generation-audit.yaml`。每个生成的 canonical 文件都必须记录 source refs、source basis 和未解决缺口。",
+      "- Write `.nimi/local/state/spec-generation/spec-generation-audit.yaml` as local generation state. Every canonical file must record source refs, source basis, and unresolved gaps.",
+      "- 将 `.nimi/local/state/spec-generation/spec-generation-audit.yaml` 写作本地生成状态。每个 canonical 文件都必须记录 source refs、source basis 和未解决缺口。",
     ));
     lines.push(localize(
       "- When a benchmark blueprint root is declared, aim for semantic and structural parity with that benchmark rather than byte-for-byte duplication.",

package/cli/lib/high-risk-admission.mjs

@@ -26,6 +26,34 @@ import { parseYamlText } from "./yaml-helpers.mjs";
 
 const ADMISSIONS_SPEC_REF = ".nimi/spec/high-risk-admissions.yaml";
 
+async function resolveProjectContainedPath(projectRoot, inputPath, artifactLabel) {
+  const absolutePath = path.resolve(projectRoot, inputPath);
+  let resolvedProjectRoot;
+  let resolvedPath;
+  try {
+    resolvedProjectRoot = await realpath(projectRoot);
+    resolvedPath = await realpath(absolutePath);
+  } catch {
+    return {
+      ok: false,
+      path: absolutePath,
+      reason: `cannot read ${artifactLabel} at ${absolutePath}`,
+    };
+  }
+  const relative = path.relative(resolvedProjectRoot, resolvedPath);
+  if (relative.startsWith("..") || path.isAbsolute(relative)) {
+    return {
+      ok: false,
+      path: absolutePath,
+      reason: `${artifactLabel} must resolve inside the current project root`,
+    };
+  }
+  return {
+    ok: true,
+    path: resolvedPath,
+  };
+}
+
 function translateAdmissionReason(reason) {
   const translations = new Map([
     ["imported decision payload must declare contractVersion nimicoding.high-risk-decision.v1", "导入的 decision payload 必须声明 contractVersion nimicoding.high-risk-decision.v1"],
@@ -45,15 +73,25 @@ function translateAdmissionReason(reason) {
 }
 
 async function loadImportedDecisionPayload(projectRoot, fromPath) {
-  const absolutePath = path.resolve(projectRoot, fromPath);
-  const rawText = await readTextIfFile(absolutePath);
+  const containedPath = await resolveProjectContainedPath(projectRoot, fromPath, "imported decision JSON");
+  if (!containedPath.ok) {
+    return {
+      ok: false,
+      error: `${localize(
+        `nimicoding admit-high-risk-decision refused: ${containedPath.reason}.`,
+        `nimicoding admit-high-risk-decision 已拒绝：${containedPath.reason}。`,
+      )}\n`,
+    };
+  }
+
+  const rawText = await readTextIfFile(containedPath.path);
 
   if (rawText === null) {
     return {
       ok: false,
       error: `${localize(
-        `nimicoding admit-high-risk-decision refused: cannot read imported decision JSON at ${absolutePath}.`,
-        `nimicoding admit-high-risk-decision 已拒绝：无法读取 ${absolutePath} 处的导入 decision JSON。`,
+        `nimicoding admit-high-risk-decision refused: cannot read imported decision JSON at ${containedPath.path}.`,
+        `nimicoding admit-high-risk-decision 已拒绝：无法读取 ${containedPath.path} 处的导入 decision JSON。`,
       )}\n`,
     };
   }
@@ -65,8 +103,8 @@ async function loadImportedDecisionPayload(projectRoot, fromPath) {
     return {
       ok: false,
       error: `${localize(
-        `nimicoding admit-high-risk-decision refused: imported decision JSON at ${absolutePath} is invalid JSON.`,
-        `nimicoding admit-high-risk-decision 已拒绝：${absolutePath} 处的导入 decision JSON 不是合法 JSON。`,
+        `nimicoding admit-high-risk-decision refused: imported decision JSON at ${containedPath.path} is invalid JSON.`,
+        `nimicoding admit-high-risk-decision 已拒绝：${containedPath.path} 处的导入 decision JSON 不是合法 JSON。`,
       )}\n`,
     };
   }
@@ -110,7 +148,7 @@ async function loadImportedDecisionPayload(projectRoot, fromPath) {
 
   return {
     ok: true,
-    path: absolutePath,
+    path: containedPath.path,
     payload: parsed,
   };
 }
@@ -192,8 +230,15 @@ async function loadAdmissionsSpec(projectRoot, contract) {
 }
 
 async function loadPacketIdentity(projectRoot, packetRef) {
-  const absolutePath = path.resolve(projectRoot, packetRef);
-  const packetValidation = await validateExecutionPacket(absolutePath);
+  const containedPath = await resolveProjectContainedPath(projectRoot, packetRef, "attached packet_ref");
+  if (!containedPath.ok) {
+    return {
+      ok: false,
+      reason: containedPath.reason,
+    };
+  }
+
+  const packetValidation = await validateExecutionPacket(containedPath.path);
   if (!packetValidation.ok) {
     return {
       ok: false,
@@ -202,7 +247,7 @@ async function loadPacketIdentity(projectRoot, packetRef) {
     };
   }
 
-  const text = await readTextIfFile(absolutePath);
+  const text = await readTextIfFile(containedPath.path);
   const parsed = parseYamlText(text);
   const packetId = String(parsed?.packet_id ?? "");
   const topicId = String(parsed?.topic_id ?? "");
@@ -242,7 +287,11 @@ function evaluateAdmissionReadiness(doctorResult, admissionsSpec, packetIdentity
     };
   }
 
-  if (doctorResult.lifecycleState?.treeState !== "canonical_tree_ready" || doctorResult.canonicalTree?.requiredFilesValid !== true) {
+  const v2Ready = doctorResult.specGenerationInputs?.mode === "class_filtered"
+    && doctorResult.canonicalTree?.requiredFilesValid === true;
+  const legacyReady = doctorResult.lifecycleState?.treeState === "canonical_tree_ready"
+    && doctorResult.canonicalTree?.requiredFilesValid === true;
+  if (!v2Ready && !legacyReady) {
     return {
       ok: false,
       reason: "Canonical admission requires canonical_tree_ready with declared canonical files present",

package/cli/lib/high-risk-decision.mjs

@@ -150,8 +150,33 @@ function validateImportedReviewPayload(payload) {
   return { ok: true };
 }
 
+function extractMarkdownSectionBody(text, heading) {
+  const lines = text.split(/\r?\n/);
+  let sectionLevel = null;
+  const body = [];
+  for (const line of lines) {
+    const headingMatch = line.match(/^(#{1,6})\s+(.+?)\s*#*\s*$/);
+    if (headingMatch) {
+      const level = headingMatch[1].length;
+      const title = headingMatch[2].trim();
+      if (sectionLevel !== null && level <= sectionLevel) {
+        break;
+      }
+      if (title === heading) {
+        sectionLevel = level;
+        continue;
+      }
+    }
+    if (sectionLevel !== null) {
+      body.push(line);
+    }
+  }
+  return sectionLevel === null ? "" : body.join("\n");
+}
+
 function extractAcceptanceDisposition(text) {
-  const match = text.match(/Disposition:\s*(\w+)/i);
+  const dispositionSection = extractMarkdownSectionBody(text, "Current Phase Disposition");
+  const match = dispositionSection.match(/Disposition:\s*(\w+)/i);
   return match ? match[1].toLowerCase() : null;
 }
 
@@ -163,7 +188,11 @@ function evaluateDecisionReadiness(doctorResult, acceptanceReport) {
     };
   }
 
-  if (doctorResult.lifecycleState?.treeState !== "canonical_tree_ready" || doctorResult.canonicalTree?.requiredFilesValid !== true) {
+  const v2Ready = doctorResult.specGenerationInputs?.mode === "class_filtered"
+    && doctorResult.canonicalTree?.requiredFilesValid === true;
+  const legacyReady = doctorResult.lifecycleState?.treeState === "canonical_tree_ready"
+    && doctorResult.canonicalTree?.requiredFilesValid === true;
+  if (!v2Ready && !legacyReady) {
     return {
       ok: false,
       reason: "High-risk decision projection requires canonical_tree_ready with declared canonical files present",

package/cli/lib/high-risk-ingest.mjs

@@ -149,7 +149,11 @@ function evaluateHighRiskIngestReadiness(doctorResult, validations) {
     };
   }
 
-  if (doctorResult.lifecycleState?.treeState !== "canonical_tree_ready" || doctorResult.canonicalTree?.requiredFilesValid !== true) {
+  const v2Ready = doctorResult.specGenerationInputs?.mode === "class_filtered"
+    && doctorResult.canonicalTree?.requiredFilesValid === true;
+  const legacyReady = doctorResult.lifecycleState?.treeState === "canonical_tree_ready"
+    && doctorResult.canonicalTree?.requiredFilesValid === true;
+  if (!v2Ready && !legacyReady) {
     return {
       ok: false,
       reason: "High-risk ingest requires canonical_tree_ready with declared canonical files present",

package/cli/lib/high-risk-review.mjs

@@ -152,7 +152,11 @@ function evaluateHighRiskReviewReadiness(doctorResult) {
     };
   }
 
-  if (doctorResult.lifecycleState?.treeState !== "canonical_tree_ready" || doctorResult.canonicalTree?.requiredFilesValid !== true) {
+  const v2Ready = doctorResult.specGenerationInputs?.mode === "class_filtered"
+    && doctorResult.canonicalTree?.requiredFilesValid === true;
+  const legacyReady = doctorResult.lifecycleState?.treeState === "canonical_tree_ready"
+    && doctorResult.canonicalTree?.requiredFilesValid === true;
+  if (!v2Ready && !legacyReady) {
     return {
       ok: false,
       reason: "High-risk review projection requires canonical_tree_ready with declared canonical files present",