artifact-graph 0.9.1 → 0.9.3

package/CHANGELOG.md

@@ -1,5 +1,38 @@
 # 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
+
+- Scenario explicit relation field recognition is tightened to whole-line field syntax matching:
+  plain prose mentions of `关联功能` / `关联决策` / `关联实体` no longer produce edges or
+  spurious `FORMAT_ERROR` diagnostics.
+- An invalid explicit relation reference is no longer reported twice across the scan and
+  validation stages: `validateGraph` now returns exactly one `FORMAT_ERROR` carrying the file,
+  line number, node, invalid value, and the expected rule.
+- Behavior tightening: Markdown list-item forms such as `- **关联决策**: ADR-0001` are no longer
+  recognized as explicit relation fields (0.9.1 misidentified them via substring matching);
+  standard line-start field syntax is unaffected.
+
 ## 0.9.1
 
 ### 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,6 +5072,7 @@ function validateScenarioPrdLinks(graph, schema = DEFAULT_SCHEMA) {
     }
     pushDuplicateIssues(issues, scenarioUid, validRefs, "feature");
     for (const ref of refs) {
+      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 }));
       }
@@ -5575,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 });
@@ -5589,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,
@@ -6826,13 +6831,42 @@ function decisionFrontmatterEdges(path, decisionCode, value, targetType, kind) {
     return [edge(toUid("decision", decisionCode), toUid(targetType, code), kind, "frontmatter", path, 1)];
   });
 }
-function markdownRelationOccurrences(path, scenario, block, label, targetType, schema = DEFAULT_SCHEMA) {
+function matchExplicitRelationFieldLine(line, label) {
+  const escaped = escapeRegExp(label);
+  const match = new RegExp(`^ {0,3}(?:\\*\\*${escaped}\\*\\*|${escaped})[:\uFF1A]\\s*(.*?)\\s*$`).exec(line);
+  return match?.[1] ?? null;
+}
+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 (!line.includes(label)) {
+    if (codeLines[scenario.index + index]) {
       return;
     }
-    const relationText = line.split(/[:：]/).slice(1).join(":").trim();
+    const relationText = matchExplicitRelationFieldLine(line, label);
     if (!relationText) {
       return;
     }
@@ -6865,7 +6899,8 @@ function relationOccurrences(node, field, targetType) {
       target,
       path: typeof record.path === "string" ? record.path : node.path,
       line: typeof record.line === "number" ? record.line : node.line,
-      raw: typeof record.raw === "string" ? record.raw : void 0
+      raw: typeof record.raw === "string" ? record.raw : void 0,
+      valid: typeof record.valid === "boolean" ? record.valid : void 0
     }];
   });
 }

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,6 +5053,7 @@ function validateScenarioPrdLinks(graph, schema = DEFAULT_SCHEMA) {
     }
     pushDuplicateIssues(issues, scenarioUid, validRefs, "feature");
     for (const ref of refs) {
+      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 }));
       }
@@ -5553,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 });
@@ -5567,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,
@@ -6804,13 +6812,42 @@ function decisionFrontmatterEdges(path, decisionCode, value, targetType, kind) {
     return [edge(toUid("decision", decisionCode), toUid(targetType, code), kind, "frontmatter", path, 1)];
   });
 }
-function markdownRelationOccurrences(path, scenario, block, label, targetType, schema = DEFAULT_SCHEMA) {
+function matchExplicitRelationFieldLine(line, label) {
+  const escaped = escapeRegExp(label);
+  const match = new RegExp(`^ {0,3}(?:\\*\\*${escaped}\\*\\*|${escaped})[:\uFF1A]\\s*(.*?)\\s*$`).exec(line);
+  return match?.[1] ?? null;
+}
+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 (!line.includes(label)) {
+    if (codeLines[scenario.index + index]) {
       return;
     }
-    const relationText = line.split(/[:：]/).slice(1).join(":").trim();
+    const relationText = matchExplicitRelationFieldLine(line, label);
     if (!relationText) {
       return;
     }
@@ -6843,7 +6880,8 @@ function relationOccurrences(node, field, targetType) {
       target,
       path: typeof record.path === "string" ? record.path : node.path,
       line: typeof record.line === "number" ? record.line : node.line,
-      raw: typeof record.raw === "string" ? record.raw : void 0
+      raw: typeof record.raw === "string" ? record.raw : void 0,
+      valid: typeof record.valid === "boolean" ? record.valid : void 0
     }];
   });
 }

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,6 +4943,7 @@ function validateScenarioPrdLinks(graph, schema = DEFAULT_SCHEMA) {
     }
     pushDuplicateIssues(issues, scenarioUid, validRefs, "feature");
     for (const ref of refs) {
+      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 }));
       }
@@ -5443,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 });
@@ -5457,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,
@@ -6694,13 +6702,42 @@ function decisionFrontmatterEdges(path, decisionCode, value, targetType, kind) {
     return [edge(toUid("decision", decisionCode), toUid(targetType, code), kind, "frontmatter", path, 1)];
   });
 }
-function markdownRelationOccurrences(path, scenario, block, label, targetType, schema = DEFAULT_SCHEMA) {
+function matchExplicitRelationFieldLine(line, label) {
+  const escaped = escapeRegExp(label);
+  const match = new RegExp(`^ {0,3}(?:\\*\\*${escaped}\\*\\*|${escaped})[:\uFF1A]\\s*(.*?)\\s*$`).exec(line);
+  return match?.[1] ?? null;
+}
+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 (!line.includes(label)) {
+    if (codeLines[scenario.index + index]) {
       return;
     }
-    const relationText = line.split(/[:：]/).slice(1).join(":").trim();
+    const relationText = matchExplicitRelationFieldLine(line, label);
     if (!relationText) {
       return;
     }
@@ -6733,7 +6770,8 @@ function relationOccurrences(node, field, targetType) {
       target,
       path: typeof record.path === "string" ? record.path : node.path,
       line: typeof record.line === "number" ? record.line : node.line,
-      raw: typeof record.raw === "string" ? record.raw : void 0
+      raw: typeof record.raw === "string" ? record.raw : void 0,
+      valid: typeof record.valid === "boolean" ? record.valid : void 0
     }];
   });
 }

package/package.json

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