qfai 0.7.1 → 0.8.0

package/dist/cli/index.mjs

@@ -870,8 +870,8 @@ import { readFile as readFile3 } from "fs/promises";
 import path6 from "path";
 import { fileURLToPath } from "url";
 async function resolveToolVersion() {
-  if ("0.7.1".length > 0) {
-    return "0.7.1";
+  if ("0.8.0".length > 0) {
+    return "0.8.0";
   }
   try {
     const packagePath = resolvePackageJsonPath();
@@ -2055,7 +2055,7 @@ async function validateDeltas(root, config) {
       issues.push(
         issue2(
           "QFAI-DELTA-002",
-          "delta.md \u306E\u5909\u66F4\u533A\u5206\u304C\u4E0D\u8DB3\u3057\u3066\u3044\u307E\u3059\u3002",
+          "delta.md \u306E\u5909\u66F4\u533A\u5206\u304C\u4E0D\u8DB3\u3057\u3066\u3044\u307E\u3059\u3002`## \u5909\u66F4\u533A\u5206` \u3068\u30C1\u30A7\u30C3\u30AF\u30DC\u30C3\u30AF\u30B9\uFF08Compatibility / Change/Improvement\uFF09\u3092\u8FFD\u52A0\u3057\u3066\u304F\u3060\u3055\u3044\u3002",
           "error",
           deltaPath,
           "delta.section"
@@ -2598,7 +2598,7 @@ async function validateTraceability(root, config) {
       issues.push(
         issue6(
           "QFAI-TRACE-020",
-          "Spec \u306B QFAI-CONTRACT-REF \u304C\u3042\u308A\u307E\u305B\u3093\u3002",
+          "Spec \u306B QFAI-CONTRACT-REF \u304C\u3042\u308A\u307E\u305B\u3093\u3002\u4F8B: `QFAI-CONTRACT-REF: none` \u307E\u305F\u306F `QFAI-CONTRACT-REF: UI-0001`",
           "error",
           file,
           "traceability.specContractRefRequired"
@@ -2609,7 +2609,7 @@ async function validateTraceability(root, config) {
         issues.push(
           issue6(
             "QFAI-TRACE-023",
-            "Spec \u306E QFAI-CONTRACT-REF \u306B none \u3068\u5951\u7D04 ID \u304C\u6DF7\u5728\u3057\u3066\u3044\u307E\u3059\u3002",
+            "Spec \u306E QFAI-CONTRACT-REF \u306B none \u3068\u5951\u7D04 ID \u304C\u6DF7\u5728\u3057\u3066\u3044\u307E\u3059\u3002none \u304B \u5951\u7D04ID \u306E\u3069\u3061\u3089\u304B\u4E00\u65B9\u3060\u3051\u306B\u3057\u3066\u304F\u3060\u3055\u3044\u3002",
             "error",
             file,
             "traceability.specContractRefFormat"
@@ -2622,7 +2622,7 @@ async function validateTraceability(root, config) {
             "QFAI-TRACE-021",
             `Spec \u306E\u5951\u7D04 ID \u304C\u4E0D\u6B63\u3067\u3059: ${contractRefs.invalidTokens.join(
               ", "
-            )}`,
+            )} (\u4F8B: UI-0001 / API-0001 / DB-0001)`,
             "error",
             file,
             "traceability.specContractRefFormat",
@@ -2662,7 +2662,7 @@ async function validateTraceability(root, config) {
       issues.push(
         issue6(
           "QFAI-TRACE-031",
-          "Scenario \u306B QFAI-CONTRACT-REF \u304C\u3042\u308A\u307E\u305B\u3093\u3002",
+          "Scenario \u306B QFAI-CONTRACT-REF \u304C\u3042\u308A\u307E\u305B\u3093\u3002\u4F8B: `# QFAI-CONTRACT-REF: none` \u307E\u305F\u306F `# QFAI-CONTRACT-REF: UI-0001`",
           "error",
           file,
           "traceability.scenarioContractRefRequired"
@@ -2673,7 +2673,7 @@ async function validateTraceability(root, config) {
         issues.push(
           issue6(
             "QFAI-TRACE-033",
-            "Scenario \u306E QFAI-CONTRACT-REF \u306B none \u3068\u5951\u7D04 ID \u304C\u6DF7\u5728\u3057\u3066\u3044\u307E\u3059\u3002",
+            "Scenario \u306E QFAI-CONTRACT-REF \u306B none \u3068\u5951\u7D04 ID \u304C\u6DF7\u5728\u3057\u3066\u3044\u307E\u3059\u3002none \u304B \u5951\u7D04ID \u306E\u3069\u3061\u3089\u304B\u4E00\u65B9\u3060\u3051\u306B\u3057\u3066\u304F\u3060\u3055\u3044\u3002",
             "error",
             file,
             "traceability.scenarioContractRefFormat"
@@ -2686,7 +2686,7 @@ async function validateTraceability(root, config) {
             "QFAI-TRACE-032",
             `Scenario \u306E\u5951\u7D04 ID \u304C\u4E0D\u6B63\u3067\u3059: ${scenarioContractRefs.invalidTokens.join(
               ", "
-            )}`,
+            )} (\u4F8B: UI-0001 / API-0001 / DB-0001)`,
             "error",
             file,
             "traceability.scenarioContractRefFormat",
@@ -3184,7 +3184,7 @@ function formatReportMarkdown(data) {
   lines.push(`- \u8A2D\u5B9A: ${data.configPath}`);
   lines.push(`- \u7248: ${data.version}`);
   lines.push("");
-  lines.push("## \u6982\u8981");
+  lines.push("## Summary");
   lines.push("");
   lines.push(`- specs: ${data.summary.specs}`);
   lines.push(`- scenarios: ${data.summary.scenarios}`);
@@ -3194,8 +3194,79 @@ function formatReportMarkdown(data) {
   lines.push(
     `- issues: info ${data.summary.counts.info} / warning ${data.summary.counts.warning} / error ${data.summary.counts.error}`
   );
+  lines.push(
+    `- fail-on=error: ${data.summary.counts.error > 0 ? "FAIL" : "PASS"}`
+  );
+  lines.push(
+    `- fail-on=warning: ${data.summary.counts.error + data.summary.counts.warning > 0 ? "FAIL" : "PASS"}`
+  );
+  lines.push("");
+  lines.push("## Findings");
+  lines.push("");
+  lines.push("### Issues (by code)");
+  lines.push("");
+  const severityOrder = {
+    error: 0,
+    warning: 1,
+    info: 2
+  };
+  const issueKeyToCount = /* @__PURE__ */ new Map();
+  for (const issue7 of data.issues) {
+    const key = `${issue7.severity}|${issue7.code}`;
+    const current = issueKeyToCount.get(key);
+    if (current) {
+      current.count += 1;
+      continue;
+    }
+    issueKeyToCount.set(key, {
+      severity: issue7.severity,
+      code: issue7.code,
+      count: 1
+    });
+  }
+  const issueSummaryRows = Array.from(issueKeyToCount.values()).sort((a, b) => {
+    const sa = severityOrder[a.severity] ?? 999;
+    const sb = severityOrder[b.severity] ?? 999;
+    if (sa !== sb) return sa - sb;
+    return a.code.localeCompare(b.code);
+  }).map((x) => [x.severity, x.code, String(x.count)]);
+  if (issueSummaryRows.length === 0) {
+    lines.push("- (none)");
+  } else {
+    lines.push(
+      ...formatMarkdownTable(["Severity", "Code", "Count"], issueSummaryRows)
+    );
+  }
+  lines.push("");
+  lines.push("### Issues (list)");
+  lines.push("");
+  if (data.issues.length === 0) {
+    lines.push("- (none)");
+  } else {
+    const sortedIssues = [...data.issues].sort((a, b) => {
+      const sa = severityOrder[a.severity] ?? 999;
+      const sb = severityOrder[b.severity] ?? 999;
+      if (sa !== sb) return sa - sb;
+      const code = a.code.localeCompare(b.code);
+      if (code !== 0) return code;
+      const fileA = a.file ?? "";
+      const fileB = b.file ?? "";
+      const file = fileA.localeCompare(fileB);
+      if (file !== 0) return file;
+      const lineA = a.loc?.line ?? 0;
+      const lineB = b.loc?.line ?? 0;
+      return lineA - lineB;
+    });
+    for (const item of sortedIssues) {
+      const location = item.file ? ` (${item.file})` : "";
+      const refs = item.refs && item.refs.length > 0 ? ` refs=${item.refs.join(",")}` : "";
+      lines.push(
+        `- ${item.severity.toUpperCase()} [${item.code}] ${item.message}${location}${refs}`
+      );
+    }
+  }
   lines.push("");
-  lines.push("## ID\u96C6\u8A08");
+  lines.push("### IDs");
   lines.push("");
   lines.push(formatIdLine("SPEC", data.ids.spec));
   lines.push(formatIdLine("BR", data.ids.br));
@@ -3204,14 +3275,14 @@ function formatReportMarkdown(data) {
   lines.push(formatIdLine("API", data.ids.api));
   lines.push(formatIdLine("DB", data.ids.db));
   lines.push("");
-  lines.push("## \u30C8\u30EC\u30FC\u30B5\u30D3\u30EA\u30C6\u30A3");
+  lines.push("### Traceability");
   lines.push("");
   lines.push(`- \u4E0A\u6D41ID\u691C\u51FA\u6570: ${data.traceability.upstreamIdsFound}`);
   lines.push(
     `- \u30B3\u30FC\u30C9/\u30C6\u30B9\u30C8\u53C2\u7167: ${data.traceability.referencedInCodeOrTests ? "\u3042\u308A" : "\u306A\u3057"}`
   );
   lines.push("");
-  lines.push("## \u5951\u7D04\u30AB\u30D0\u30EC\u30C3\u30B8");
+  lines.push("### Contract Coverage");
   lines.push("");
   lines.push(`- total: ${data.traceability.contracts.total}`);
   lines.push(`- referenced: ${data.traceability.contracts.referenced}`);
@@ -3220,7 +3291,7 @@ function formatReportMarkdown(data) {
     `- specContractRefMissing: ${data.traceability.specs.contractRefMissing}`
   );
   lines.push("");
-  lines.push("## \u5951\u7D04\u2192Spec");
+  lines.push("### Contract \u2192 Spec");
   lines.push("");
   const contractToSpecs = data.traceability.contracts.idToSpecs;
   const contractIds = Object.keys(contractToSpecs).sort(
@@ -3239,7 +3310,7 @@ function formatReportMarkdown(data) {
     }
   }
   lines.push("");
-  lines.push("## Spec\u2192\u5951\u7D04");
+  lines.push("### Spec \u2192 Contracts");
   lines.push("");
   const specToContracts = data.traceability.specs.specToContracts;
   const specIds = Object.keys(specToContracts).sort(
@@ -3257,7 +3328,7 @@ function formatReportMarkdown(data) {
     lines.push(...formatMarkdownTable(["Spec", "Status", "Contracts"], rows));
   }
   lines.push("");
-  lines.push("## Spec\u3067 contract-ref \u672A\u5BA3\u8A00");
+  lines.push("### Specs missing contract-ref");
   lines.push("");
   const missingRefSpecs = data.traceability.specs.missingRefSpecs;
   if (missingRefSpecs.length === 0) {
@@ -3268,7 +3339,7 @@ function formatReportMarkdown(data) {
     }
   }
   lines.push("");
-  lines.push("## SC\u30AB\u30D0\u30EC\u30C3\u30B8");
+  lines.push("### SC coverage");
   lines.push("");
   lines.push(`- total: ${data.traceability.sc.total}`);
   lines.push(`- covered: ${data.traceability.sc.covered}`);
@@ -3298,7 +3369,7 @@ function formatReportMarkdown(data) {
     lines.push(`- missingIds: ${missingWithSources.join(", ")}`);
   }
   lines.push("");
-  lines.push("## SC\u2192\u53C2\u7167\u30C6\u30B9\u30C8");
+  lines.push("### SC \u2192 referenced tests");
   lines.push("");
   const scRefs = data.traceability.sc.refs;
   const scIds = Object.keys(scRefs).sort((a, b) => a.localeCompare(b));
@@ -3315,7 +3386,7 @@ function formatReportMarkdown(data) {
     }
   }
   lines.push("");
-  lines.push("## Spec:SC=1:1 \u9055\u53CD");
+  lines.push("### Spec:SC=1:1 violations");
   lines.push("");
   const specScIssues = data.issues.filter(
     (item) => item.code === "QFAI-TRACE-012"
@@ -3330,7 +3401,7 @@ function formatReportMarkdown(data) {
     }
   }
   lines.push("");
-  lines.push("## Hotspots");
+  lines.push("### Hotspots");
   lines.push("");
   const hotspots = buildHotspots(data.issues);
   if (hotspots.length === 0) {
@@ -3343,35 +3414,28 @@ function formatReportMarkdown(data) {
     }
   }
   lines.push("");
-  lines.push("## \u30C8\u30EC\u30FC\u30B5\u30D3\u30EA\u30C6\u30A3\uFF08\u691C\u8A3C\uFF09");
+  lines.push("## Guidance");
   lines.push("");
-  const traceIssues = data.issues.filter(
-    (item) => item.rule?.startsWith("traceability.") || item.code.startsWith("QFAI_TRACE") || item.code.startsWith("QFAI-TRACE-")
+  lines.push(
+    "- \u6B21\u306E\u624B\u9806: `qfai doctor --fail-on error` \u2192 `qfai validate --fail-on error` \u2192 `qfai report`"
   );
-  if (traceIssues.length === 0) {
-    lines.push("- (none)");
-  } else {
-    for (const item of traceIssues) {
-      const location = item.file ? ` (${item.file})` : "";
-      lines.push(
-        `- ${item.severity.toUpperCase()} [${item.code}] ${item.message}${location}`
-      );
-    }
-  }
-  lines.push("");
-  lines.push("## \u691C\u8A3C\u7D50\u679C");
-  lines.push("");
-  if (data.issues.length === 0) {
-    lines.push("- (none)");
+  if (data.summary.counts.error > 0) {
+    lines.push("- error \u304C\u3042\u308B\u305F\u3081\u3001\u307E\u305A error \u304B\u3089\u4FEE\u6B63\u3057\u3066\u304F\u3060\u3055\u3044\u3002");
+  } else if (data.summary.counts.warning > 0) {
+    lines.push(
+      "- warning \u306E\u6271\u3044\uFF08Hard Gate \u306B\u3059\u308B\u304B\uFF09\u306F\u904B\u7528\u3067\u6C7A\u3081\u3066\u304F\u3060\u3055\u3044\u3002"
+    );
   } else {
-    for (const item of data.issues) {
-      const location = item.file ? ` (${item.file})` : "";
-      const refs = item.refs && item.refs.length > 0 ? ` refs=${item.refs.join(",")}` : "";
-      lines.push(
-        `- ${item.severity.toUpperCase()} [${item.code}] ${item.message}${location}${refs}`
-      );
-    }
+    lines.push(
+      "- issue \u306F\u691C\u51FA\u3055\u308C\u307E\u305B\u3093\u3067\u3057\u305F\u3002\u904B\u7528\u30C6\u30F3\u30D7\u30EC\u306B\u6CBF\u3063\u3066\u7D99\u7D9A\u3057\u3066\u304F\u3060\u3055\u3044\u3002"
+    );
   }
+  lines.push(
+    "- \u5909\u66F4\u533A\u5206\uFF08Compatibility / Change/Improvement\uFF09\u306F `.qfai/specs/*/delta.md` \u306B\u8A18\u9332\u3057\u307E\u3059\u3002"
+  );
+  lines.push(
+    "- \u53C2\u7167\u30EB\u30FC\u30EB\u306E\u6B63\u672C: `.qfai/promptpack/steering/traceability.md` / `.qfai/promptpack/steering/compatibility-vs-change.md`"
+  );
   return lines.join("\n");
 }
 function formatReportJson(data) {
@@ -3695,6 +3759,8 @@ async function runValidate(options) {
   const configResult = await loadConfig(root);
   const result = await validateProject(root, configResult);
   const normalized = normalizeValidationResult(root, result);
+  const failOn = resolveFailOn(options, configResult.config.validation.failOn);
+  const willFail = shouldFail(normalized, failOn);
   const format = options.format ?? "text";
   if (format === "text") {
     emitText(normalized);
@@ -3704,11 +3770,10 @@ async function runValidate(options) {
       root,
       configResult.config.output.validateJsonPath
     );
-    emitGitHubOutput(normalized, root, jsonPath);
+    emitGitHubOutput(normalized, root, jsonPath, { failOn, willFail });
   }
   await emitJson(normalized, root, configResult.config.output.validateJsonPath);
-  const failOn = resolveFailOn(options, configResult.config.validation.failOn);
-  return shouldFail(normalized, failOn) ? 1 : 0;
+  return willFail ? 1 : 0;
 }
 function resolveFailOn(options, fallback) {
   if (options.failOn) {
@@ -3733,7 +3798,7 @@ function emitText(result) {
 `
   );
 }
-function emitGitHubOutput(result, root, jsonPath) {
+function emitGitHubOutput(result, root, jsonPath, status) {
   const deduped = dedupeIssues(result.issues);
   const omitted = Math.max(deduped.length - GITHUB_ANNOTATION_LIMIT, 0);
   const dropped = Math.max(result.issues.length - deduped.length, 0);
@@ -3742,7 +3807,8 @@ function emitGitHubOutput(result, root, jsonPath) {
     omitted,
     dropped,
     jsonPath,
-    root
+    root,
+    ...status
   });
   const issues = deduped.slice(0, GITHUB_ANNOTATION_LIMIT);
   for (const issue7 of issues) {
@@ -3766,7 +3832,9 @@ function emitGitHubSummary(result, options) {
     `error=${result.counts.error}`,
     `warning=${result.counts.warning}`,
     `info=${result.counts.info}`,
-    `annotations=${Math.min(options.total, GITHUB_ANNOTATION_LIMIT)}/${options.total}`
+    `annotations=${Math.min(options.total, GITHUB_ANNOTATION_LIMIT)}/${options.total}`,
+    `failOn=${options.failOn}`,
+    `result=${options.willFail ? "FAIL" : "PASS"}`
   ].join(" ");
   process.stdout.write(`${summary}
 `);
@@ -3784,6 +3852,9 @@ function emitGitHubSummary(result, options) {
     `qfai validate note: \u8A73\u7D30\u306F ${relative} \u307E\u305F\u306F --format text \u3092\u53C2\u7167\u3057\u3066\u304F\u3060\u3055\u3044\u3002
 `
   );
+  process.stdout.write(
+    "qfai validate note: \u6B21\u306F qfai report \u3067 report.md \u3092\u751F\u6210\u3067\u304D\u307E\u3059\uFF08\u4F8B: qfai report\uFF09\u3002\n"
+  );
 }
 function dedupeIssues(issues) {
   const seen = /* @__PURE__ */ new Set();