artifact-graph 0.9.2 → 0.9.3

package/CHANGELOG.md

@@ -1,5 +1,24 @@
 # Changelog
 
+## 0.9.3
+
+### Fixed
+
+- Scenario titles and explicit relation fields inside Markdown fenced code blocks
+  (```` ``` ```` or `~~~`) are no longer scanned: examples such as
+  `**关联决策**: ADR-0001` shown inside a fence produce neither scenario nodes nor graph
+  edges. A fence closes only on the same marker with a length at least as long as the
+  opening fence.
+- Explicit relation fields indented by four or more spaces or a tab are now treated as
+  indented code and no longer produce edges. Outside code blocks, a real relation field
+  may be indented by at most 0—3 plain spaces. Behavior tightening: adopters who relied
+  on deeply indented relation fields must move them to line start (0—3 spaces).
+- The public `validateScenarioPrdLinks` function once again returns the complete
+  diagnostics (including `FORMAT_ERROR` for invalid feature references) when called
+  directly on scan results, restoring the public API behavior regressed in 0.9.2.
+- `validateGraph` still uses scan diagnostics as the single source for scanned invalid
+  relations, so the same invalid feature reference is reported exactly once.
+
 ## 0.9.2
 
 ### Fixed

package/dist/cli.js

@@ -4936,7 +4936,7 @@ function validateGraph(graph, schema = DEFAULT_SCHEMA) {
   }
   issues.push(...validateE2eTests(graph));
   issues.push(...validateE2eRegistry(graph));
-  issues.push(...validateScenarioPrdLinks(graph, schema));
+  issues.push(...validateScenarioPrdLinksInternal(graph, schema, false));
   issues.push(...validateCodeCommentTraceabilityFormat(graph));
   issues.push(...validateCodeCommentScenarioFeatureConsistency(graph));
   for (const edge2 of graph.edges) {
@@ -5038,7 +5038,7 @@ function validateGraph(graph, schema = DEFAULT_SCHEMA) {
   issues.sort((left, right) => left.code.localeCompare(right.code) || left.path.localeCompare(right.path) || left.line - right.line);
   return issues;
 }
-function validateScenarioPrdLinks(graph, schema = DEFAULT_SCHEMA) {
+function validateScenarioPrdLinksInternal(graph, schema, includeScanDiagnosedInvalid) {
   if (!schema.relationFields.scenario?.includes("\u5173\u8054\u529F\u80FD") || !schema.relationFields.feature?.includes("scenarios")) {
     return [];
   }
@@ -5072,7 +5072,7 @@ function validateScenarioPrdLinks(graph, schema = DEFAULT_SCHEMA) {
     }
     pushDuplicateIssues(issues, scenarioUid, validRefs, "feature");
     for (const ref of refs) {
-      if (ref.valid === false) continue;
+      if (ref.valid === false && !includeScanDiagnosedInvalid) continue;
       if (!featurePattern.test(ref.target)) {
         issues.push(issue("FORMAT_ERROR", `scenario ${scenarioUid} has invalid feature reference ${ref.target}`, ref.path, ref.line, { node: scenarioUid }));
       }
@@ -5576,8 +5576,12 @@ function parseFeature(path, raw) {
 }
 function parseScenarios(path, raw, schema = DEFAULT_SCHEMA) {
   const lines = raw.split(/\r?\n/);
+  const codeLines = markdownCodeLineMask(lines);
   const starts = [];
   lines.forEach((line, index) => {
+    if (codeLines[index]) {
+      return;
+    }
     const match = /^#{2,3}\s+(S-\d+[a-z]?)\s*[:：]\s*(.+?)\s*$/.exec(line);
     if (match) {
       starts.push({ code: match[1], title: match[2], line: index + 1, index });
@@ -5590,9 +5594,9 @@ function parseScenarios(path, raw, schema = DEFAULT_SCHEMA) {
     const start = starts[i];
     const end = starts[i + 1]?.index ?? lines.length;
     const block = lines.slice(start.index, end);
-    const featureRefs = markdownRelationOccurrences(path, start, block, "\u5173\u8054\u529F\u80FD", "feature", schema);
-    const decisionRefs = markdownRelationOccurrences(path, start, block, "\u5173\u8054\u51B3\u7B56", "decision", schema);
-    const entityRefs = markdownRelationOccurrences(path, start, block, "\u5173\u8054\u5B9E\u4F53", "entity", schema);
+    const featureRefs = markdownRelationOccurrences(path, start, block, codeLines, "\u5173\u8054\u529F\u80FD", "feature", schema);
+    const decisionRefs = markdownRelationOccurrences(path, start, block, codeLines, "\u5173\u8054\u51B3\u7B56", "decision", schema);
+    const entityRefs = markdownRelationOccurrences(path, start, block, codeLines, "\u5173\u8054\u5B9E\u4F53", "entity", schema);
     nodes.push({
       type: "scenario",
       code: start.code,
@@ -6829,12 +6833,39 @@ function decisionFrontmatterEdges(path, decisionCode, value, targetType, kind) {
 }
 function matchExplicitRelationFieldLine(line, label) {
   const escaped = escapeRegExp(label);
-  const match = new RegExp(`^\\s*(?:\\*\\*${escaped}\\*\\*|${escaped})[:\uFF1A]\\s*(.*?)\\s*$`).exec(line);
+  const match = new RegExp(`^ {0,3}(?:\\*\\*${escaped}\\*\\*|${escaped})[:\uFF1A]\\s*(.*?)\\s*$`).exec(line);
   return match?.[1] ?? null;
 }
-function markdownRelationOccurrences(path, scenario, block, label, targetType, schema = DEFAULT_SCHEMA) {
+function markdownCodeLineMask(lines) {
+  const result = new Array(lines.length).fill(false);
+  let fence;
+  lines.forEach((line, index) => {
+    if (fence) {
+      result[index] = true;
+      const closing = /^ {0,3}(`+|~+)[ \t]*$/.exec(line);
+      if (closing && closing[1][0] === fence.marker && closing[1].length >= fence.length) {
+        fence = void 0;
+      }
+      return;
+    }
+    const opening = /^ {0,3}(`{3,}|~{3,})(.*)$/.exec(line);
+    if (opening) {
+      result[index] = true;
+      fence = { marker: opening[1][0], length: opening[1].length };
+      return;
+    }
+    if (/^(?: {4,}| {0,3}\t)/.test(line)) {
+      result[index] = true;
+    }
+  });
+  return result;
+}
+function markdownRelationOccurrences(path, scenario, block, codeLines, label, targetType, schema = DEFAULT_SCHEMA) {
   const result = [];
   block.forEach((line, index) => {
+    if (codeLines[scenario.index + index]) {
+      return;
+    }
     const relationText = matchExplicitRelationFieldLine(line, label);
     if (!relationText) {
       return;

package/dist/index.cjs

@@ -4914,7 +4914,7 @@ function validateGraph(graph, schema = DEFAULT_SCHEMA) {
   }
   issues.push(...validateE2eTests(graph));
   issues.push(...validateE2eRegistry(graph));
-  issues.push(...validateScenarioPrdLinks(graph, schema));
+  issues.push(...validateScenarioPrdLinksInternal(graph, schema, false));
   issues.push(...validateCodeCommentTraceabilityFormat(graph));
   issues.push(...validateCodeCommentScenarioFeatureConsistency(graph));
   for (const edge2 of graph.edges) {
@@ -5017,6 +5017,9 @@ function validateGraph(graph, schema = DEFAULT_SCHEMA) {
   return issues;
 }
 function validateScenarioPrdLinks(graph, schema = DEFAULT_SCHEMA) {
+  return validateScenarioPrdLinksInternal(graph, schema, true);
+}
+function validateScenarioPrdLinksInternal(graph, schema, includeScanDiagnosedInvalid) {
   if (!schema.relationFields.scenario?.includes("\u5173\u8054\u529F\u80FD") || !schema.relationFields.feature?.includes("scenarios")) {
     return [];
   }
@@ -5050,7 +5053,7 @@ function validateScenarioPrdLinks(graph, schema = DEFAULT_SCHEMA) {
     }
     pushDuplicateIssues(issues, scenarioUid, validRefs, "feature");
     for (const ref of refs) {
-      if (ref.valid === false) continue;
+      if (ref.valid === false && !includeScanDiagnosedInvalid) continue;
       if (!featurePattern.test(ref.target)) {
         issues.push(issue("FORMAT_ERROR", `scenario ${scenarioUid} has invalid feature reference ${ref.target}`, ref.path, ref.line, { node: scenarioUid }));
       }
@@ -5554,8 +5557,12 @@ function parseFeature(path, raw) {
 }
 function parseScenarios(path, raw, schema = DEFAULT_SCHEMA) {
   const lines = raw.split(/\r?\n/);
+  const codeLines = markdownCodeLineMask(lines);
   const starts = [];
   lines.forEach((line, index) => {
+    if (codeLines[index]) {
+      return;
+    }
     const match = /^#{2,3}\s+(S-\d+[a-z]?)\s*[:：]\s*(.+?)\s*$/.exec(line);
     if (match) {
       starts.push({ code: match[1], title: match[2], line: index + 1, index });
@@ -5568,9 +5575,9 @@ function parseScenarios(path, raw, schema = DEFAULT_SCHEMA) {
     const start = starts[i];
     const end = starts[i + 1]?.index ?? lines.length;
     const block = lines.slice(start.index, end);
-    const featureRefs = markdownRelationOccurrences(path, start, block, "\u5173\u8054\u529F\u80FD", "feature", schema);
-    const decisionRefs = markdownRelationOccurrences(path, start, block, "\u5173\u8054\u51B3\u7B56", "decision", schema);
-    const entityRefs = markdownRelationOccurrences(path, start, block, "\u5173\u8054\u5B9E\u4F53", "entity", schema);
+    const featureRefs = markdownRelationOccurrences(path, start, block, codeLines, "\u5173\u8054\u529F\u80FD", "feature", schema);
+    const decisionRefs = markdownRelationOccurrences(path, start, block, codeLines, "\u5173\u8054\u51B3\u7B56", "decision", schema);
+    const entityRefs = markdownRelationOccurrences(path, start, block, codeLines, "\u5173\u8054\u5B9E\u4F53", "entity", schema);
     nodes.push({
       type: "scenario",
       code: start.code,
@@ -6807,12 +6814,39 @@ function decisionFrontmatterEdges(path, decisionCode, value, targetType, kind) {
 }
 function matchExplicitRelationFieldLine(line, label) {
   const escaped = escapeRegExp(label);
-  const match = new RegExp(`^\\s*(?:\\*\\*${escaped}\\*\\*|${escaped})[:\uFF1A]\\s*(.*?)\\s*$`).exec(line);
+  const match = new RegExp(`^ {0,3}(?:\\*\\*${escaped}\\*\\*|${escaped})[:\uFF1A]\\s*(.*?)\\s*$`).exec(line);
   return match?.[1] ?? null;
 }
-function markdownRelationOccurrences(path, scenario, block, label, targetType, schema = DEFAULT_SCHEMA) {
+function markdownCodeLineMask(lines) {
+  const result = new Array(lines.length).fill(false);
+  let fence;
+  lines.forEach((line, index) => {
+    if (fence) {
+      result[index] = true;
+      const closing = /^ {0,3}(`+|~+)[ \t]*$/.exec(line);
+      if (closing && closing[1][0] === fence.marker && closing[1].length >= fence.length) {
+        fence = void 0;
+      }
+      return;
+    }
+    const opening = /^ {0,3}(`{3,}|~{3,})(.*)$/.exec(line);
+    if (opening) {
+      result[index] = true;
+      fence = { marker: opening[1][0], length: opening[1].length };
+      return;
+    }
+    if (/^(?: {4,}| {0,3}\t)/.test(line)) {
+      result[index] = true;
+    }
+  });
+  return result;
+}
+function markdownRelationOccurrences(path, scenario, block, codeLines, label, targetType, schema = DEFAULT_SCHEMA) {
   const result = [];
   block.forEach((line, index) => {
+    if (codeLines[scenario.index + index]) {
+      return;
+    }
     const relationText = matchExplicitRelationFieldLine(line, label);
     if (!relationText) {
       return;

package/dist/index.js

@@ -4804,7 +4804,7 @@ function validateGraph(graph, schema = DEFAULT_SCHEMA) {
   }
   issues.push(...validateE2eTests(graph));
   issues.push(...validateE2eRegistry(graph));
-  issues.push(...validateScenarioPrdLinks(graph, schema));
+  issues.push(...validateScenarioPrdLinksInternal(graph, schema, false));
   issues.push(...validateCodeCommentTraceabilityFormat(graph));
   issues.push(...validateCodeCommentScenarioFeatureConsistency(graph));
   for (const edge2 of graph.edges) {
@@ -4907,6 +4907,9 @@ function validateGraph(graph, schema = DEFAULT_SCHEMA) {
   return issues;
 }
 function validateScenarioPrdLinks(graph, schema = DEFAULT_SCHEMA) {
+  return validateScenarioPrdLinksInternal(graph, schema, true);
+}
+function validateScenarioPrdLinksInternal(graph, schema, includeScanDiagnosedInvalid) {
   if (!schema.relationFields.scenario?.includes("\u5173\u8054\u529F\u80FD") || !schema.relationFields.feature?.includes("scenarios")) {
     return [];
   }
@@ -4940,7 +4943,7 @@ function validateScenarioPrdLinks(graph, schema = DEFAULT_SCHEMA) {
     }
     pushDuplicateIssues(issues, scenarioUid, validRefs, "feature");
     for (const ref of refs) {
-      if (ref.valid === false) continue;
+      if (ref.valid === false && !includeScanDiagnosedInvalid) continue;
       if (!featurePattern.test(ref.target)) {
         issues.push(issue("FORMAT_ERROR", `scenario ${scenarioUid} has invalid feature reference ${ref.target}`, ref.path, ref.line, { node: scenarioUid }));
       }
@@ -5444,8 +5447,12 @@ function parseFeature(path, raw) {
 }
 function parseScenarios(path, raw, schema = DEFAULT_SCHEMA) {
   const lines = raw.split(/\r?\n/);
+  const codeLines = markdownCodeLineMask(lines);
   const starts = [];
   lines.forEach((line, index) => {
+    if (codeLines[index]) {
+      return;
+    }
     const match = /^#{2,3}\s+(S-\d+[a-z]?)\s*[:：]\s*(.+?)\s*$/.exec(line);
     if (match) {
       starts.push({ code: match[1], title: match[2], line: index + 1, index });
@@ -5458,9 +5465,9 @@ function parseScenarios(path, raw, schema = DEFAULT_SCHEMA) {
     const start = starts[i];
     const end = starts[i + 1]?.index ?? lines.length;
     const block = lines.slice(start.index, end);
-    const featureRefs = markdownRelationOccurrences(path, start, block, "\u5173\u8054\u529F\u80FD", "feature", schema);
-    const decisionRefs = markdownRelationOccurrences(path, start, block, "\u5173\u8054\u51B3\u7B56", "decision", schema);
-    const entityRefs = markdownRelationOccurrences(path, start, block, "\u5173\u8054\u5B9E\u4F53", "entity", schema);
+    const featureRefs = markdownRelationOccurrences(path, start, block, codeLines, "\u5173\u8054\u529F\u80FD", "feature", schema);
+    const decisionRefs = markdownRelationOccurrences(path, start, block, codeLines, "\u5173\u8054\u51B3\u7B56", "decision", schema);
+    const entityRefs = markdownRelationOccurrences(path, start, block, codeLines, "\u5173\u8054\u5B9E\u4F53", "entity", schema);
     nodes.push({
       type: "scenario",
       code: start.code,
@@ -6697,12 +6704,39 @@ function decisionFrontmatterEdges(path, decisionCode, value, targetType, kind) {
 }
 function matchExplicitRelationFieldLine(line, label) {
   const escaped = escapeRegExp(label);
-  const match = new RegExp(`^\\s*(?:\\*\\*${escaped}\\*\\*|${escaped})[:\uFF1A]\\s*(.*?)\\s*$`).exec(line);
+  const match = new RegExp(`^ {0,3}(?:\\*\\*${escaped}\\*\\*|${escaped})[:\uFF1A]\\s*(.*?)\\s*$`).exec(line);
   return match?.[1] ?? null;
 }
-function markdownRelationOccurrences(path, scenario, block, label, targetType, schema = DEFAULT_SCHEMA) {
+function markdownCodeLineMask(lines) {
+  const result = new Array(lines.length).fill(false);
+  let fence;
+  lines.forEach((line, index) => {
+    if (fence) {
+      result[index] = true;
+      const closing = /^ {0,3}(`+|~+)[ \t]*$/.exec(line);
+      if (closing && closing[1][0] === fence.marker && closing[1].length >= fence.length) {
+        fence = void 0;
+      }
+      return;
+    }
+    const opening = /^ {0,3}(`{3,}|~{3,})(.*)$/.exec(line);
+    if (opening) {
+      result[index] = true;
+      fence = { marker: opening[1][0], length: opening[1].length };
+      return;
+    }
+    if (/^(?: {4,}| {0,3}\t)/.test(line)) {
+      result[index] = true;
+    }
+  });
+  return result;
+}
+function markdownRelationOccurrences(path, scenario, block, codeLines, label, targetType, schema = DEFAULT_SCHEMA) {
   const result = [];
   block.forEach((line, index) => {
+    if (codeLines[scenario.index + index]) {
+      return;
+    }
     const relationText = matchExplicitRelationFieldLine(line, label);
     if (!relationText) {
       return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "artifact-graph",
-  "version": "0.9.2",
+  "version": "0.9.3",
   "description": "Git-native Markdown artifact graph scanner and validator",
   "license": "Apache-2.0",
   "type": "module",