mason-context 0.16.3 → 0.17.0

package/dist/mason-review.js

@@ -688,7 +688,7 @@ async function findMissingPartners(resolvedRoot, changedFiles, existsOnDisk, all
 
 // src/review/evidence.ts
 import path10 from "path";
-import { z as z6 } from "zod";
+import { z as z7 } from "zod";
 
 // src/review/evidence/vitest.ts
 import { z as z4 } from "zod";
@@ -904,20 +904,76 @@ function parseSarif(raw, sourceRoot) {
   };
 }
 
+// src/review/evidence/check.ts
+import { z as z6 } from "zod";
+var schema3 = z6.object({
+  version: z6.literal(1),
+  commit: z6.string().regex(/^(?:[a-fA-F0-9]{40}|[a-fA-F0-9]{64})$/).optional(),
+  results: z6.array(z6.object({
+    name: z6.string().min(1).max(1e3),
+    status: z6.enum(["passed", "failed", "skipped"]),
+    message: z6.string().max(1e5).optional(),
+    locations: z6.array(z6.object({
+      file: z6.string().min(1).max(4e3),
+      line: z6.number().int().positive().optional(),
+      column: z6.number().int().positive().optional()
+    })).max(100).default([])
+  })).max(1e5)
+});
+function parseCheck(raw, sourceRoot) {
+  const report = schema3.parse(raw);
+  const diagnostics = [], findings = [];
+  const counts = { total: report.results.length, passed: 0, failed: 0, skipped: 0 };
+  for (const [index2, result] of report.results.entries()) {
+    counts[result.status]++;
+    const locations = result.locations.flatMap((location2) => {
+      const file = evidencePath(location2.file, sourceRoot);
+      if (!file) {
+        diagnostics.push(`Check location is outside the declared checkout or invalid: ${location2.file}`);
+        return [];
+      }
+      return [{ ...location2, file }];
+    });
+    if (result.status === "failed") findings.push({
+      id: String(index2),
+      ...messagePreview([result.name, result.message].filter(Boolean).join("\n")),
+      severity: "error",
+      state: "active",
+      locations
+    });
+  }
+  if (!counts.total) diagnostics.push("No check results were recorded; an empty report is not passing evidence.");
+  if (counts.skipped) diagnostics.push(`${counts.skipped} checks were skipped; verification is incomplete.`);
+  return {
+    outcome: counts.failed ? "failed" : !counts.total ? "unavailable" : !counts.passed ? "skipped" : "passed",
+    counts,
+    findings,
+    incomplete: diagnostics.length > 0,
+    diagnostics,
+    ...report.commit ? { reportedCommits: [report.commit] } : {}
+  };
+}
+
 // src/review/evidence.ts
-var checkSchema = z6.object({
-  id: z6.string().min(1).max(100),
-  kind: z6.enum(["tests", "static-analysis", "security", "complexity", "duplication"]),
-  tool: z6.string().min(1).max(200),
-  command: z6.string().min(1).max(2e3),
-  commit: z6.string().regex(/^(?:[a-fA-F0-9]{40}|[a-fA-F0-9]{64})$/).nullable().optional(),
-  workingTreeClean: z6.boolean().optional(),
-  source: z6.string().max(2e3).optional(),
-  sourceRoot: z6.string().min(1).max(4e3).optional(),
-  status: z6.enum(["completed", "skipped", "unavailable"]).default("completed"),
-  reason: z6.string().min(1).max(2e3).optional(),
-  exitCode: z6.number().int().nullable().optional(),
-  report: z6.object({ format: z6.enum(["vitest-json", "sarif"]), path: z6.string().min(1).max(4e3) }).optional()
+var checkSchema = z7.object({
+  id: z7.string().min(1).max(100),
+  kind: z7.enum(["tests", "static-analysis", "security", "complexity", "duplication", "documentation"]),
+  tool: z7.string().min(1).max(200),
+  command: z7.string().min(1).max(2e3),
+  commit: z7.string().regex(/^(?:[a-fA-F0-9]{40}|[a-fA-F0-9]{64})$/).nullable().optional(),
+  workingTreeClean: z7.boolean().optional(),
+  source: z7.string().max(2e3).optional(),
+  sourceRoot: z7.string().min(1).max(4e3).optional(),
+  status: z7.enum(["completed", "skipped", "unavailable"]).default("completed"),
+  reason: z7.string().min(1).max(2e3).optional(),
+  exitCode: z7.number().int().nullable().optional(),
+  environment: z7.object({
+    toolVersion: z7.string().min(1).max(200),
+    runtime: z7.string().min(1).max(200).optional(),
+    platform: z7.string().min(1).max(200).optional(),
+    configuration: z7.array(z7.string().min(1).max(4e3)).max(100).optional()
+  }).optional(),
+  report: z7.object({ format: z7.enum(["vitest-json", "sarif", "mason-check-json"]), path: z7.string().min(1).max(4e3) }).optional()
 });
 function relativeArtifact(root, file) {
   const relative = path10.isAbsolute(file) ? path10.relative(root, file) : file;
@@ -959,7 +1015,7 @@ async function collectReviewEvidence(root, manifests, changedFiles, decisions, d
     let manifest, raw;
     try {
       manifest = relativeArtifact(root, supplied);
-      raw = z6.object({ version: z6.literal(1), checks: z6.array(z6.unknown()) }).parse(await readStoreJson(root, manifest));
+      raw = z7.object({ version: z7.literal(1), checks: z7.array(z7.unknown()) }).parse(await readStoreJson(root, manifest));
     } catch (error) {
       output.diagnostics.push(`${supplied}: ${String(error)}`);
       continue;
@@ -998,7 +1054,8 @@ async function collectReviewEvidence(root, manifests, changedFiles, decisions, d
         counts: {},
         incomplete: false,
         diagnostics: [],
-        truncated: false
+        truncated: false,
+        ...check.environment ? { environment: check.environment } : {}
       };
       output.checks.push(result);
       if (check.workingTreeClean !== true) result.diagnostics.push("The check's working tree was dirty or not recorded; its results cannot be attributed to the claimed commit alone.");
@@ -1011,13 +1068,13 @@ async function collectReviewEvidence(root, manifests, changedFiles, decisions, d
       }
       try {
         if (!check.report) throw new Error("Completed check has no report artifact.");
-        if (check.kind === "tests" !== (check.report.format === "vitest-json")) throw new Error("Test checks require vitest-json; analysis checks require sarif.");
+        if (check.report.format !== "mason-check-json" && check.kind === "tests" !== (check.report.format === "vitest-json")) throw new Error("Test checks require vitest-json or mason-check-json; analysis checks require sarif or mason-check-json.");
         const file = relativeArtifact(root, check.report.path);
         const report = await readStoreJson(root, file);
         if (report === null) throw new Error(`Report artifact is missing: ${file}`);
         const sourceRoot = check.sourceRoot ?? root;
         if (!path10.posix.isAbsolute(sourceRoot) && !/^[A-Za-z]:[\\/]/.test(sourceRoot)) throw new Error("sourceRoot must identify the absolute checkout root on the check runner.");
-        const parsed = check.report.format === "vitest-json" ? parseVitest(report, sourceRoot) : parseSarif(report, sourceRoot);
+        const parsed = check.report.format === "vitest-json" ? parseVitest(report, sourceRoot) : check.report.format === "mason-check-json" ? parseCheck(report, sourceRoot) : parseSarif(report, sourceRoot);
         result.outcome = parsed.outcome;
         result.counts = parsed.counts;
         result.incomplete = parsed.incomplete;
@@ -1037,7 +1094,7 @@ async function collectReviewEvidence(root, manifests, changedFiles, decisions, d
           result.outcome = "failed";
           result.diagnostics.push(`Check command exited ${check.exitCode} despite a report with no active failures.`);
         }
-        if (check.report.format === "vitest-json" && pairs === void 0) {
+        if (check.kind === "tests" && pairs === void 0) {
           try {
             pairs = (await buildTestMap(root)).paired;
           } catch (error) {
@@ -1259,6 +1316,10 @@ function evidenceSummary(report) {
   for (const check of evidence.checks) {
     lines.push(`  [${check.kind}] ${display(check.id)}: ${check.outcome}, ${check.freshness} (${display(check.tool)})`);
     lines.push(`    command: ${display(check.command)}; tested commit: ${check.commit ?? "unknown"}`);
+    if (check.environment) {
+      lines.push(`    environment: ${display(check.tool)} ${display(check.environment.toolVersion)}` + (check.environment.runtime ? `; ${display(check.environment.runtime)}` : "") + (check.environment.platform ? `; ${display(check.environment.platform)}` : ""));
+      if (check.environment.configuration?.length) lines.push("    configuration: " + check.environment.configuration.map((value) => display(value)).join(", "));
+    }
     lines.push(`    source: ${display(check.source ?? check.manifest)}; report: ${display(check.report?.path ?? "none")}`);
     if (Object.keys(check.counts).length) lines.push(`    counts: ${JSON.stringify(check.counts)}`);
     for (const diagnostic of check.diagnostics.slice(0, 5)) lines.push(`    note: ${display(diagnostic)}`);