@open-mercato/cli 0.6.7-develop.6726.1.983ae8a07e → 0.6.7-develop.6744.1.0483b18565

package/dist/agentic/shared/scripts/run-agent-harness-release.mjs

@@ -63,9 +63,9 @@ tree. Externally supplied targets must use the same
 node_modules link to that protected tree; nested dependency copies are rejected.
 The complete release requires trusted Bubblewrap plus working user/network namespaces on Linux;
 native macOS, Windows, untrusted runtimes, and unavailable isolation fail closed. The command preflights complete matrix,
-fixture, review, and target coverage before
+fixture, judge, and target coverage before
 running deterministic validation, selected primary routing, optional portability routing, writable trusted-oracle
-gates, explicit om-code-review, and the release-matrix validation commands. It stores
+gates, explicit om-judge-agent-session with om-code-review, and the release-matrix validation commands. It stores
 only a sanitized aggregate report under .ai/harness/results/.`
 }
 
@@ -541,8 +541,8 @@ export function buildReleasePlan({ cases, registry, releaseMatrix, fixtures, see
     if (!entry || typeof entry !== 'object' || Array.isArray(entry) || Object.keys(entry).length !== 1 || typeof entry.caseId !== 'string') violations.push(`writable release assignment must be runner-neutral: ${String(entry?.caseId)}`)
   }
 
-  const configuredReviewIds = Array.isArray(releaseMatrix?.generatedCodeReview?.caseIds)
-    ? releaseMatrix.generatedCodeReview.caseIds
+  const configuredReviewIds = Array.isArray(releaseMatrix?.generativeJudge?.caseIds)
+    ? releaseMatrix.generativeJudge.caseIds
     : []
   const missingReviewMatrixCaseIds = difference(reviewEligibleCaseIds, configuredReviewIds)
   const unexpectedReviewMatrixCaseIds = difference(configuredReviewIds, reviewEligibleCaseIds)
@@ -553,10 +553,17 @@ export function buildReleasePlan({ cases, registry, releaseMatrix, fixtures, see
   if (JSON.stringify(configuredReviewIds) !== JSON.stringify(reviewEligibleCaseIds)) {
     violations.push('generated-code review matrix must list every writable case exactly once in catalog order')
   }
-  if (release.requireGeneratedCodeReview !== true) violations.push('release suite must require generated-code review')
-  if (releaseMatrix?.generatedCodeReview?.skill !== 'om-code-review') violations.push('generated-code review must explicitly use om-code-review')
-  if (releaseMatrix?.generatedCodeReview?.required !== true) violations.push('generated-code review must be mandatory for every writable case')
-  const reviewRunners = releaseMatrix?.generatedCodeReview?.runners ?? {}
+  if (release.requireGenerativeJudge !== true) violations.push('release suite must require the generative judge')
+  if (release.requireGeneratedCodeReview !== true) violations.push('release suite must retain generated-code review compatibility')
+  if (releaseMatrix?.generativeJudge?.skill !== 'om-judge-agent-session') violations.push('generative judge must explicitly use om-judge-agent-session')
+  if (releaseMatrix?.generativeJudge?.reviewSkill !== 'om-code-review') violations.push('generative judge must explicitly compose om-code-review')
+  if (releaseMatrix?.generativeJudge?.required !== true) violations.push('generative judge must be mandatory for every writable case')
+  if (releaseMatrix?.generatedCodeReview?.skill !== 'om-code-review'
+    || JSON.stringify(releaseMatrix.generatedCodeReview?.caseIds) !== JSON.stringify(configuredReviewIds)
+    || JSON.stringify(releaseMatrix.generatedCodeReview?.runners) !== JSON.stringify(releaseMatrix.generativeJudge?.runners)) {
+    violations.push('generated-code review compatibility matrix must match the generative judge')
+  }
+  const reviewRunners = releaseMatrix?.generativeJudge?.runners ?? {}
   for (const runner of RELEASE_SUPPORTED_RUNNERS) {
     if (!validModelSelector(reviewRunners?.[runner]?.modelSelector)) violations.push(`generated-code review lacks a valid bounded ${runner} model selector`)
   }
@@ -564,6 +571,12 @@ export function buildReleasePlan({ cases, registry, releaseMatrix, fixtures, see
     if (!RUNNERS.has(runner)) violations.push(`generated-code review declares an invalid runner: ${runner}`)
   }
   const reviewSkillFiles = [
+    '.ai/skills/om-judge-agent-session/SKILL.md',
+    '.ai/skills/om-judge-agent-session/references/agentic-setup.md',
+    '.ai/skills/om-judge-agent-session/references/input-normalization.md',
+    '.ai/skills/om-judge-agent-session/references/judge-workflow.md',
+    '.ai/skills/om-judge-agent-session/references/report-template.md',
+    '.ai/skills/om-judge-agent-session/references/rules.md',
     '.agents/skills/om-code-review/SKILL.md',
     '.agents/skills/om-code-review/references/agentic-setup.md',
     '.agents/skills/om-code-review/references/output-format.md',
@@ -572,7 +585,7 @@ export function buildReleasePlan({ cases, registry, releaseMatrix, fixtures, see
     '.agents/skills/.om-external-ownership.json',
   ]
   const reviewSkillReady = !root || reviewSkillFiles.every((relative) => fs.existsSync(path.join(root, relative)))
-  if (!reviewSkillReady) violations.push('generated-code review skill is not installed with ownership evidence')
+  if (!reviewSkillReady) violations.push('generative judge or code-review skill is not installed with ownership evidence')
 
   const registryCaseCountMatches = registry?.catalog?.expectedCaseCount === cases.length
   if (!registryCaseCountMatches) violations.push(`validator registry expects ${String(registry?.catalog?.expectedCaseCount)} cases but catalog contains ${cases.length}`)
@@ -720,8 +733,8 @@ export function buildReleasePlan({ cases, registry, releaseMatrix, fixtures, see
       })
     }
     if (configuredReviewIds.includes(caseId)) {
-      const modelSelector = releaseMatrix.generatedCodeReview?.runners?.[primaryRunner]?.modelSelector
-      if (!validModelSelector(modelSelector)) violations.push(`generated-code review lacks a valid bounded ${primaryRunner} model selector for ${caseId}`)
+      const modelSelector = releaseMatrix.generativeJudge?.runners?.[primaryRunner]?.modelSelector
+      if (!validModelSelector(modelSelector)) violations.push(`generative judge lacks a valid bounded ${primaryRunner} model selector for ${caseId}`)
       steps.push({ id: `review:${caseId}`, kind: 'review', caseId, runner: primaryRunner, modelSelector })
     }
   }
@@ -1696,8 +1709,8 @@ export function main(argv = process.argv.slice(2)) {
       const reviewExecution = execute(process.execPath, [
         evaluator, '--root', root, '--runner', reviewStep.runner,
         '--model', reviewStep.modelSelector, '--timeout', String(caseTimeout),
-        '--review-writable-result', path.join(root, sourceArtifact.path), '--writable-root', target,
-        '--review-validation-result', targetValidationArtifact.absolute,
+        '--judge-writable-result', path.join(root, sourceArtifact.path), '--writable-root', target,
+        '--judge-validation-result', targetValidationArtifact.absolute,
       ], root, caseTimeout + 60_000)
       const reviewArtifacts = readNewResults(root, beforeReview)
       resultArtifacts.push(...reviewArtifacts)

package/dist/lib/generators/module-facts.js

@@ -330,9 +330,9 @@ function parseApiRouteEntry(entryLiteral) {
       }
     }
   }
-  return { path: routePath, methods, auth };
+  return { path: routePath, methods, auth, sourcePath: null };
 }
-function extractApiRoutes(moduleId, registrySource, registryDescription, warnings) {
+function extractApiRoutes(moduleId, registrySource, registryDescription, routeSourcePaths, warnings) {
   if (registrySource == null) {
     warnings.push(`[module-facts] module registry unavailable (${registryDescription}); API route auth omitted for ${moduleId}`);
     return [];
@@ -356,7 +356,7 @@ function extractApiRoutes(moduleId, registrySource, registryDescription, warning
           const route = parseApiRouteEntry(element);
           if (route && !seenPaths.has(route.path)) {
             seenPaths.add(route.path);
-            routes.push(route);
+            routes.push({ ...route, sourcePath: routeSourcePaths.get(route.path) ?? null });
           }
         }
       }
@@ -514,6 +514,153 @@ function listSourceFilesRecursive(directory) {
   }
   return files;
 }
+function toPortableSourceRoot(moduleId, sourcePackage) {
+  return path.posix.join("node_modules", sourcePackage ?? "@open-mercato/core", "src", "modules", moduleId);
+}
+function toPortableSourcePath(moduleRoot, sourceRoot, filePath) {
+  const relativePath = path.relative(moduleRoot, filePath).split(path.sep).join("/");
+  return path.posix.join(sourceRoot, relativePath);
+}
+function sourceHasDefaultExport(sourceFile) {
+  return sourceFile.statements.some((statement) => {
+    if (ts.isExportAssignment(statement) && !statement.isExportEquals) return true;
+    if (!ts.isFunctionDeclaration(statement) && !ts.isClassDeclaration(statement)) return false;
+    return statement.modifiers?.some((modifier) => modifier.kind === ts.SyntaxKind.DefaultKeyword) ?? false;
+  });
+}
+function readRouteMetadataPath(sourceFile) {
+  const metadata = findObjectLiteralDeclaration(sourceFile, "metadata");
+  return metadata ? readStringPropertyInitializer(metadata, "path") ?? null : null;
+}
+function extractApiRouteSourcePaths(moduleId, moduleRoot, sourceRoot) {
+  const apiRoot = path.join(moduleRoot, "api");
+  const sources = /* @__PURE__ */ new Map();
+  if (!fs.existsSync(apiRoot)) return sources;
+  const methodDirectories = /* @__PURE__ */ new Set(["get", "post", "put", "patch", "delete"]);
+  for (const filePath of listSourceFilesRecursive(apiRoot)) {
+    const relativePath = path.relative(apiRoot, filePath).split(path.sep).join("/");
+    const segments = relativePath.split("/");
+    const fileName = segments.pop();
+    const fileStem = fileName.replace(/\.tsx?$/, "");
+    let routeSegments;
+    if (fileStem === "route") {
+      routeSegments = segments;
+    } else if (segments[0] && methodDirectories.has(segments[0].toLowerCase())) {
+      routeSegments = [...segments.slice(1), fileStem];
+    } else {
+      routeSegments = [...segments, fileStem];
+    }
+    const defaultPath = `/${[moduleId, ...routeSegments].filter(Boolean).join("/")}`;
+    const sourceFile = readSourceFile(filePath);
+    const routePath = sourceFile ? readRouteMetadataPath(sourceFile) ?? defaultPath : defaultPath;
+    sources.set(routePath, toPortableSourcePath(moduleRoot, sourceRoot, filePath));
+  }
+  return sources;
+}
+function extractModulePages(moduleId, moduleRoot, sourceRoot, surface) {
+  const pageRoot = path.join(moduleRoot, surface);
+  if (!fs.existsSync(pageRoot)) return [];
+  const pages = [];
+  const seen = /* @__PURE__ */ new Set();
+  for (const filePath of listSourceFilesRecursive(pageRoot)) {
+    if (!filePath.endsWith(".tsx")) continue;
+    const relativePath = path.relative(pageRoot, filePath).split(path.sep).join("/");
+    const segments = relativePath.split("/");
+    const fileName = segments.pop();
+    const fileStem = fileName.replace(/\.tsx$/, "");
+    const isModernPage = fileStem === "page";
+    if (!isModernPage && (fileStem.endsWith(".meta") || /^[A-Z]/.test(fileStem))) continue;
+    const sourceFile = readSourceFile(filePath);
+    if (!sourceFile || !sourceHasDefaultExport(sourceFile)) continue;
+    const routeSegments = isModernPage ? segments : [...segments, fileStem];
+    const routePath = surface === "frontend" ? `/${routeSegments.filter(Boolean).join("/")}` : isModernPage ? `/backend/${routeSegments.join("/") || moduleId}` : `/backend/${routeSegments[0] === moduleId ? routeSegments.filter(Boolean).join("/") : [moduleId, ...routeSegments].filter(Boolean).join("/")}`;
+    if (seen.has(routePath)) continue;
+    seen.add(routePath);
+    pages.push({
+      path: routePath,
+      sourcePath: toPortableSourcePath(moduleRoot, sourceRoot, filePath)
+    });
+  }
+  return pages.sort((left, right) => left.path.localeCompare(right.path));
+}
+function findAncestorVariableDeclaration(node) {
+  let current = node.parent;
+  while (current) {
+    if (ts.isVariableDeclaration(current)) return current;
+    current = current.parent;
+  }
+  return null;
+}
+function findAncestorCallExpression(node) {
+  let current = node.parent;
+  while (current) {
+    if (ts.isCallExpression(current)) return current;
+    if (ts.isVariableDeclaration(current) || ts.isStatement(current)) return null;
+    current = current.parent;
+  }
+  return null;
+}
+function extractNamedAiFacts(filePaths, moduleRoot, sourceRoot, propertyName) {
+  const facts = [];
+  const seen = /* @__PURE__ */ new Set();
+  for (const filePath of filePaths) {
+    const sourceFile = readSourceFile(filePath);
+    if (!sourceFile) continue;
+    const initializers = buildVariableInitializerMap(sourceFile);
+    const readDefinitionValue = (objectLiteral) => {
+      const initializer = getObjectPropertyInitializer(objectLiteral, propertyName);
+      if (!initializer) return void 0;
+      const resolved = unwrapExpression(initializer);
+      if (ts.isStringLiteralLike(resolved)) return resolved.text;
+      if (ts.isIdentifier(resolved)) {
+        const declarationInitializer = initializers.get(resolved.text);
+        if (declarationInitializer) {
+          const declarationValue = unwrapExpression(declarationInitializer);
+          if (ts.isStringLiteralLike(declarationValue)) return declarationValue.text;
+        }
+      }
+      return void 0;
+    };
+    const visit = (node) => {
+      if (ts.isObjectLiteralExpression(node)) {
+        const value = readDefinitionValue(node);
+        if (value && !seen.has(value)) {
+          const declaration = findAncestorVariableDeclaration(node);
+          const declarationName = declaration && ts.isIdentifier(declaration.name) ? declaration.name.text : "";
+          const declarationType = declaration?.type?.getText(sourceFile) ?? "";
+          const call = findAncestorCallExpression(node);
+          const callee = call?.expression.getText(sourceFile) ?? "";
+          const isDefinition = propertyName === "name" ? /AiTool/.test(declarationType) || /tool$/i.test(declarationName) || /AiTool/.test(callee) : /AiAgentDefinition/.test(declarationType) || /agent$/i.test(declarationName) || /defineAiAgent/.test(callee);
+          if (isDefinition) {
+            seen.add(value);
+            facts.push({
+              value,
+              sourcePath: toPortableSourcePath(moduleRoot, sourceRoot, filePath)
+            });
+          }
+        }
+      }
+      node.forEachChild(visit);
+    };
+    sourceFile.forEachChild(visit);
+  }
+  return facts.sort((left, right) => left.value.localeCompare(right.value));
+}
+function extractAiTools(moduleRoot, sourceRoot) {
+  const files = /* @__PURE__ */ new Set();
+  const rootFile = resolveConventionFile(moduleRoot, "ai-tools");
+  if (rootFile) files.add(rootFile);
+  const toolsDirectory = path.join(moduleRoot, "ai-tools");
+  if (fs.existsSync(toolsDirectory)) {
+    for (const filePath of listSourceFilesRecursive(toolsDirectory)) files.add(filePath);
+  }
+  return extractNamedAiFacts([...files], moduleRoot, sourceRoot, "name").map((fact) => ({ name: fact.value, sourcePath: fact.sourcePath }));
+}
+function extractAiAgents(moduleRoot, sourceRoot) {
+  const agentsFile = resolveConventionFile(moduleRoot, "ai-agents");
+  if (!agentsFile) return [];
+  return extractNamedAiFacts([agentsFile], moduleRoot, sourceRoot, "id").map((fact) => ({ id: fact.value, sourcePath: fact.sourcePath }));
+}
 function extractTableIds(moduleRoot) {
   if (!fs.existsSync(moduleRoot)) return [];
   const tableIds = [];
@@ -571,6 +718,7 @@ function extractModuleFacts(options) {
   if (!moduleRoot) {
     throw new Error(`[internal] extractModuleFacts requires moduleRoot or coreSrcRoot for module "${moduleId}"`);
   }
+  const sourceRoot = toPortableSourceRoot(moduleId, sourcePackage);
   const entitiesFilePath = resolveConventionFile(path.join(moduleRoot, "data"), "entities") ?? resolveConventionFile(path.join(moduleRoot, "db"), "entities") ?? resolveConventionFile(path.join(moduleRoot, "data"), "schema");
   const ceFilePath = resolveConventionFile(moduleRoot, "ce");
   const eventsFilePath = resolveConventionFile(moduleRoot, "events");
@@ -587,11 +735,20 @@ function extractModuleFacts(options) {
   const events = extractEvents(eventsFilePath);
   const aclFeatures = extractAclFeatures(aclFilePath);
   const { source: registrySource, description: registryDescription } = resolveRegistrySource(options);
-  const apiRoutes = extractApiRoutes(moduleId, registrySource, registryDescription, warnings);
+  const apiRouteSourcePaths = extractApiRouteSourcePaths(moduleId, moduleRoot, sourceRoot);
+  const apiRoutes = extractApiRoutes(moduleId, registrySource, registryDescription, apiRouteSourcePaths, warnings);
   const diTokens = extractDiTokens(diFilePath);
   const searchEntities = extractSearchEntities(searchFilePath, warnings);
   const notifications = extractNotifications(notificationsFilePath, warnings);
   const cli = extractCli(cliFilePath, warnings);
+  const cliCommands = cliFilePath ? cli.map((command) => ({
+    command,
+    sourcePath: toPortableSourcePath(moduleRoot, sourceRoot, cliFilePath)
+  })) : [];
+  const backendPages = extractModulePages(moduleId, moduleRoot, sourceRoot, "backend");
+  const frontendPages = extractModulePages(moduleId, moduleRoot, sourceRoot, "frontend");
+  const aiTools = extractAiTools(moduleRoot, sourceRoot);
+  const aiAgents = extractAiAgents(moduleRoot, sourceRoot);
   const hostTokens = {
     entityIds: extractHostEntityIds(entities),
     tableIds: extractTableIds(moduleRoot)
@@ -603,6 +760,7 @@ function extractModuleFacts(options) {
     coreVersion,
     sourcePackage,
     sourceVersion,
+    sourceRoot,
     entities,
     events,
     aclFeatures,
@@ -612,6 +770,11 @@ function extractModuleFacts(options) {
     hostTokens,
     notifications,
     cli,
+    backendPages,
+    frontendPages,
+    cliCommands,
+    aiTools,
+    aiAgents,
     warnings
   };
 }
@@ -652,6 +815,9 @@ ${EMPTY_SECTION_MARKER}`;
 
 ${values.join(" \xB7 ")}`;
 }
+function renderSourceLink(sourcePath) {
+  return `[${sourcePath}](../../../${sourcePath})`;
+}
 function describeAuthRule(rule) {
   if (!rule) return "public";
   if (rule.requireFeatures && rule.requireFeatures.length > 0) return rule.requireFeatures.join(", ");
@@ -674,13 +840,20 @@ function renderApiRoutesSection(routes) {
   if (routes.length === 0) return `## API routes
 
 ${EMPTY_SECTION_MARKER}`;
-  const header = "| Path | Methods | Auth (per-method requireFeatures) |";
-  const divider = "|---|---|---|";
+  const header = "| Path | Methods | Auth (per-method requireFeatures) | Source |";
+  const divider = "|---|---|---|---|";
   const rows = routes.map(
-    (route) => `| ${route.path} | ${route.methods.join(" ")} | ${renderApiRouteAuthCell(route)} |`
+    (route) => `| ${route.path} | ${route.methods.join(" ")} | ${renderApiRouteAuthCell(route)} | ${route.sourcePath ? renderSourceLink(route.sourcePath) : "\u2014"} |`
   );
   return ["## API routes", "", header, divider, ...rows].join("\n");
 }
+function renderLinkedFactsSection(heading, facts) {
+  if (facts.length === 0) return `${heading}
+
+${EMPTY_SECTION_MARKER}`;
+  const rows = facts.map((fact) => `| ${fact.label} | ${renderSourceLink(fact.sourcePath)} |`);
+  return [heading, "", "| ID / path | Source |", "|---|---|", ...rows].join("\n");
+}
 function renderHostTokensSection(hostTokens) {
   const entityIdsLine = hostTokens.entityIds.length > 0 ? hostTokens.entityIds.join(" \xB7 ") : EMPTY_SECTION_MARKER;
   const tableIdsLine = hostTokens.tableIds.length > 0 ? hostTokens.tableIds.join(" \xB7 ") : EMPTY_SECTION_MARKER;
@@ -690,6 +863,7 @@ function renderModuleFactsMarkdown(facts) {
   const sections = [
     `# ${facts.module} \u2014 module facts (generated, do not edit)`,
     renderVersionStamp(facts.coreVersion, facts.sourcePackage, facts.sourceVersion),
+    `Source root: ${renderSourceLink(facts.sourceRoot)}`,
     "",
     renderEntitiesSection(facts.entities),
     "",
@@ -699,6 +873,10 @@ function renderModuleFactsMarkdown(facts) {
     "",
     renderApiRoutesSection(facts.apiRoutes),
     "",
+    renderLinkedFactsSection("## Backend pages", facts.backendPages.map((page) => ({ label: page.path, sourcePath: page.sourcePath }))),
+    "",
+    renderLinkedFactsSection("## Frontend pages", facts.frontendPages.map((page) => ({ label: page.path, sourcePath: page.sourcePath }))),
+    "",
     renderInlineListSection("## DI service tokens", facts.diTokens),
     "",
     renderInlineListSection("## Search entities", facts.searchEntities),
@@ -707,7 +885,11 @@ function renderModuleFactsMarkdown(facts) {
     "",
     renderInlineListSection("## Notifications", facts.notifications),
     "",
-    renderInlineListSection("## CLI", facts.cli),
+    renderLinkedFactsSection("## CLI commands", facts.cliCommands.map((command) => ({ label: command.command, sourcePath: command.sourcePath }))),
+    "",
+    renderLinkedFactsSection("## AI tools / MCP capabilities", facts.aiTools.map((tool) => ({ label: tool.name, sourcePath: tool.sourcePath }))),
+    "",
+    renderLinkedFactsSection("## AI agents", facts.aiAgents.map((agent) => ({ label: agent.id, sourcePath: agent.sourcePath }))),
     ""
   ];
   return sections.join("\n");
@@ -719,6 +901,7 @@ function toModuleFactsJsonEntry(facts) {
     coreVersion: facts.coreVersion,
     sourcePackage: facts.sourcePackage,
     sourceVersion: facts.sourceVersion,
+    sourceRoot: facts.sourceRoot,
     entities: facts.entities,
     events: facts.events.map((event) => ({ id: event.id, category: event.category, entity: event.entity })),
     aclFeatures: facts.aclFeatures,
@@ -727,7 +910,12 @@ function toModuleFactsJsonEntry(facts) {
     searchEntities: facts.searchEntities,
     hostTokens: facts.hostTokens,
     notifications: facts.notifications,
-    cli: facts.cli
+    cli: facts.cli,
+    backendPages: facts.backendPages,
+    frontendPages: facts.frontendPages,
+    cliCommands: facts.cliCommands,
+    aiTools: facts.aiTools,
+    aiAgents: facts.aiAgents
   };
 }
 function buildModuleFactsJsonObject(factsByModule) {