@nimiplatform/nimi-coding 0.2.8 → 0.3.0

package/cli/lib/internal/validators-shared.mjs

@@ -1,18 +1,6 @@
 export const VALIDATOR_CLI_RESULT_CONTRACT = "validator-cli-result.v1";
 
 export const VALIDATOR_NATIVE_REFUSAL_CODES = {
-  EXECUTION_PACKET_MISSING: "EXECUTION_PACKET_MISSING",
-  EXECUTION_PACKET_INVALID: "EXECUTION_PACKET_INVALID",
-  ORCHESTRATION_STATE_MISSING: "ORCHESTRATION_STATE_MISSING",
-  ORCHESTRATION_STATE_INVALID: "ORCHESTRATION_STATE_INVALID",
-  PROMPT_MISSING: "PROMPT_MISSING",
-  PROMPT_INVALID: "PROMPT_INVALID",
-  WORKER_OUTPUT_MISSING: "WORKER_OUTPUT_MISSING",
-  WORKER_OUTPUT_INVALID: "WORKER_OUTPUT_INVALID",
-  RUNNER_SIGNAL_MISSING: "RUNNER_SIGNAL_MISSING",
-  RUNNER_SIGNAL_INVALID: "RUNNER_SIGNAL_INVALID",
-  ACCEPTANCE_MISSING: "ACCEPTANCE_MISSING",
-  ACCEPTANCE_INVALID: "ACCEPTANCE_INVALID",
   SPEC_TREE_MISSING: "SPEC_TREE_MISSING",
   SPEC_TREE_INVALID: "SPEC_TREE_INVALID",
   SPEC_AUDIT_MISSING: "SPEC_AUDIT_MISSING",
@@ -22,7 +10,3 @@ export const VALIDATOR_NATIVE_REFUSAL_CODES = {
 export function makeValidatorRefusal(code, message) {
   return { code, message };
 }
-
-export function normalizeArgv(args) {
-  return args[0] === "--" ? args.slice(1) : args;
-}

package/cli/lib/internal/validators-spec-helpers.mjs

@@ -1,21 +1,7 @@
-import path from "node:path";
 import { readdir } from "node:fs/promises";
-
-import { readTextIfFile } from "../fs-helpers.mjs";
-
-function posixRelative(targetRoot, absolutePath) {
-  return path.relative(targetRoot, absolutePath).split(path.sep).join(path.posix.sep);
-}
+import path from "node:path";
 
 export async function collectTreeFiles(rootPath) {
-  const text = await readTextIfFile(path.join(rootPath, "INDEX.md"));
-  if (text === null) {
-    const info = await readTextIfFile(rootPath);
-    if (info !== null) {
-      return [];
-    }
-  }
-
   async function walk(currentPath) {
     let entries;
     try {
@@ -30,7 +16,7 @@ export async function collectTreeFiles(rootPath) {
       if (entry.isDirectory()) {
         files.push(...await walk(childPath));
       } else if (entry.isFile()) {
-        files.push(posixRelative(rootPath, childPath));
+        files.push(path.relative(rootPath, childPath).split(path.sep).join(path.posix.sep));
       }
     }
     return files.sort();
@@ -38,117 +24,6 @@ export async function collectTreeFiles(rootPath) {
 
   return walk(rootPath);
 }
-
-function escapeRegexLiteral(value) {
-  return value.replace(/[|\\{}()[\]^$+?.]/g, "\\$&");
-}
-
-function globToRegex(pattern) {
-  const DOUBLE_STAR_SLASH = "__DOUBLE_STAR_SLASH__";
-  const DOUBLE_STAR = "__DOUBLE_STAR__";
-  const SINGLE_STAR = "__SINGLE_STAR__";
-
-  let source = pattern
-    .replaceAll("**/", DOUBLE_STAR_SLASH)
-    .replaceAll("**", DOUBLE_STAR)
-    .replaceAll("*", SINGLE_STAR);
-
-  source = escapeRegexLiteral(source)
-    .replaceAll(DOUBLE_STAR_SLASH, "(?:.*/)?")
-    .replaceAll(DOUBLE_STAR, ".*")
-    .replaceAll(SINGLE_STAR, "[^/]*");
-
-  return new RegExp(`^${source}$`);
-}
-
-function compilePathClassMatchers(specTreeModel) {
-  const classes = [
-    ...specTreeModel.normativeClasses.map((entry) => ({ ...entry, category: "normative" })),
-    ...specTreeModel.derivedClasses.map((entry) => ({ ...entry, category: "derived" })),
-    ...specTreeModel.guidanceClasses.map((entry) => ({ ...entry, category: "guidance" })),
-  ];
-
-  return classes.map((entry) => ({
-    ...entry,
-    includeMatchers: entry.pathPatterns.map(globToRegex),
-    excludeMatchers: (entry.excludedPathPatterns ?? []).map(globToRegex),
-  }));
-}
-
-function isAllowedTopLevelSupportFile(relativePath) {
-  return [
-    "INDEX.md",
-    "bootstrap-state.yaml",
-    "product-scope.yaml",
-  ].includes(relativePath);
-}
-
-export function classifySpecTreeFiles(canonicalRoot, files, specTreeModel) {
-  const matchers = compilePathClassMatchers(specTreeModel);
-  const classifications = [];
-  const unexpected = [];
-  const conflicts = [];
-
-  for (const relativePath of files) {
-    const canonicalRelativePath = path.posix.join(canonicalRoot, relativePath);
-
-    if (relativePath.startsWith("_meta/")) {
-      classifications.push({ path: relativePath, classId: "_meta", category: "meta" });
-      continue;
-    }
-
-    if (isAllowedTopLevelSupportFile(relativePath)) {
-      classifications.push({ path: relativePath, classId: "support", category: "support" });
-      continue;
-    }
-
-    const matched = matchers.filter((matcher) => (
-      matcher.includeMatchers.some((regex) => regex.test(canonicalRelativePath))
-      && !matcher.excludeMatchers.some((regex) => regex.test(canonicalRelativePath))
-    ));
-
-    if (matched.length === 0) {
-      unexpected.push(relativePath);
-      continue;
-    }
-
-    if (matched.length > 1) {
-      conflicts.push({
-        path: relativePath,
-        classes: matched.map((entry) => entry.id),
-      });
-      continue;
-    }
-
-    classifications.push({
-      path: relativePath,
-      classId: matched[0].id,
-      category: matched[0].category,
-    });
-  }
-
-  return {
-    classifications,
-    unexpected,
-    conflicts,
-  };
-}
-
-export function classifyAuditCoveredFiles(files, specTreeModel) {
-  const classifications = classifySpecTreeFiles(specTreeModel.canonicalRoot, files, specTreeModel);
-  const auditedFiles = classifications.classifications.filter((entry) => (
-    entry.category !== "meta"
-    && (
-      entry.category !== "support"
-      || entry.path === "INDEX.md"
-    )
-  ));
-  return {
-    classifications,
-    auditedFiles,
-  };
-}
-
 export function isSourceRefWithinDeclaredRoots(sourceRef, declaredInputs) {
   const roots = [
     ...declaredInputs.code_roots,
@@ -157,10 +32,6 @@ export function isSourceRefWithinDeclaredRoots(sourceRef, declaredInputs) {
     ...(declaredInputs.benchmark_blueprint_root ? [declaredInputs.benchmark_blueprint_root] : []),
   ];
 
-  if (declaredInputs.human_note_paths.includes(sourceRef)) {
-    return true;
-  }
-
   return roots.some((root) => (
     root === "."
       ? !path.posix.isAbsolute(sourceRef)
@@ -169,17 +40,13 @@ export function isSourceRefWithinDeclaredRoots(sourceRef, declaredInputs) {
 }
 
 export function isDeclaredInputsCompatibleWithConfig(declaredInputs, generationInputs, blueprintReference) {
-  const benchmarkRoot = generationInputs.benchmarkBlueprintRoot ?? blueprintReference.root ?? null;
-
-  const rootsAlign = (declaredRoots, configuredRoots) => declaredRoots.every((entry) => configuredRoots.includes(entry));
+  const blueprintRoot = blueprintReference.present ? blueprintReference.root : null;
+  const rootsAlign = (declaredRoots, configuredRoots) =>
+    declaredRoots.every((entry) => configuredRoots.includes(entry));
 
   return rootsAlign(declaredInputs.code_roots, generationInputs.codeRoots ?? [])
     && rootsAlign(declaredInputs.docs_roots, generationInputs.docsRoots ?? [])
     && rootsAlign(declaredInputs.structure_roots, generationInputs.structureRoots ?? [])
-    && rootsAlign(declaredInputs.human_note_paths, generationInputs.humanNotePaths ?? [])
-    && (
-      declaredInputs.benchmark_blueprint_root === null
-        ? benchmarkRoot === null
-        : declaredInputs.benchmark_blueprint_root === benchmarkRoot
-    );
+    && declaredInputs.human_note_paths.length === 0
+    && declaredInputs.benchmark_blueprint_root === blueprintRoot;
 }