@diagrammo/dgmo 0.8.3 → 0.8.4

package/src/graph/flowchart-parser.ts

@@ -9,13 +9,7 @@ import {
   OPTION_NOCOLON_RE,
   ALL_CHART_TYPES,
 } from '../utils/parsing';
-import type {
-  ParsedGraph,
-  GraphNode,
-  GraphEdge,
-  GraphShape,
-  GraphDirection,
-} from './types';
+import type { ParsedGraph, GraphNode, GraphEdge, GraphShape } from './types';
 
 // ============================================================
 // Helpers
@@ -36,10 +30,7 @@ interface NodeRef {
  * Try to parse a node reference from a text fragment.
  * Order matters: subroutine & document before process.
  */
-function parseNodeRef(
-  text: string,
-  palette?: PaletteColors
-): NodeRef | null {
+function parseNodeRef(text: string, palette?: PaletteColors): NodeRef | null {
   const t = text.trim();
   if (!t) return null;
 
@@ -47,7 +38,12 @@ function parseNodeRef(
   let m = t.match(/^\[\[([^\]]+)\]\]$/);
   if (m) {
     const { label, color } = extractColor(m[1].trim(), palette);
-    return { id: nodeId('subroutine', label), label, shape: 'subroutine', color };
+    return {
+      id: nodeId('subroutine', label),
+      label,
+      shape: 'subroutine',
+      color,
+    };
   }
 
   // Document: [Label~]
@@ -99,7 +95,12 @@ function splitArrows(line: string): string[] {
   const segments: string[] = [];
   let lastIndex = 0;
   // Simpler approach: find all `->` positions, then determine if there's a label prefix
-  const arrowPositions: { start: number; end: number; label?: string; color?: string }[] = [];
+  const arrowPositions: {
+    start: number;
+    end: number;
+    label?: string;
+    color?: string;
+  }[] = [];
 
   // Find all -> occurrences
   let searchFrom = 0;
@@ -124,10 +125,14 @@ function splitArrows(line: string): string[] {
         scanBack--;
       }
       // Check if this `-` could be the start of the arrow
-      if (line[scanBack] === '-' && (scanBack === 0 || /\s/.test(line[scanBack - 1]))) {
+      if (
+        line[scanBack] === '-' &&
+        (scanBack === 0 || /\s/.test(line[scanBack - 1]))
+      ) {
         // Content between opening `-` and `->` (strip trailing `-` that is part of `->`)
         let arrowContent = line.substring(scanBack + 1, idx);
-        if (arrowContent.endsWith('-')) arrowContent = arrowContent.slice(0, -1);
+        if (arrowContent.endsWith('-'))
+          arrowContent = arrowContent.slice(0, -1);
         // Parse label and color from arrow content
         const colorMatch = arrowContent.match(/\(([^)]+)\)\s*$/);
         if (colorMatch) {
@@ -159,7 +164,8 @@ function splitArrows(line: string): string[] {
     }
     // Arrow marker
     let arrowToken = '->';
-    if (arrow.label && arrow.color) arrowToken = `-${arrow.label}(${arrow.color})->`;
+    if (arrow.label && arrow.color)
+      arrowToken = `-${arrow.label}(${arrow.color})->`;
     else if (arrow.label) arrowToken = `-${arrow.label}->`;
     else if (arrow.color) arrowToken = `-(${arrow.color})->`;
     segments.push(arrowToken);
@@ -190,7 +196,9 @@ function parseArrowToken(token: string, palette?: PaletteColors): ArrowInfo {
   const m = token.match(/^-(.+?)(?:\(([^)]+)\))?->$/);
   if (m) {
     const label = m[1]?.trim() || undefined;
-    let color = m[2] ? resolveColor(m[2].trim(), palette) ?? undefined : undefined;
+    let color = m[2]
+      ? (resolveColor(m[2].trim(), palette) ?? undefined)
+      : undefined;
     if (label && !color) {
       color = inferArrowColor(label);
     }
@@ -214,7 +222,7 @@ export function parseFlowchart(
   const lines = content.split('\n');
   const result: ParsedGraph = {
     type: 'flowchart',
-    direction: 'LR',
+    direction: 'TB',
     nodes: [],
     edges: [],
     options: {},
@@ -406,9 +414,9 @@ export function parseFlowchart(
 
     // Options (space-separated, before content)
     if (!contentStarted) {
-      // Bare boolean: direction-tb
-      if (/^direction-tb$/i.test(trimmed)) {
-        result.direction = 'TB';
+      // Bare boolean: direction-lr
+      if (/^direction-lr$/i.test(trimmed)) {
+        result.direction = 'LR';
         continue;
       }
 
@@ -435,7 +443,10 @@ export function parseFlowchart(
 
   // Validation: no nodes found
   if (result.nodes.length === 0 && !result.error) {
-    const diag = makeDgmoError(1, 'No nodes found. Add flowchart content with shape syntax like [Process] or (Start).');
+    const diag = makeDgmoError(
+      1,
+      'No nodes found. Add flowchart content with shape syntax like [Process] or (Start).'
+    );
     result.diagnostics.push(diag);
     result.error = formatDgmoError(diag);
   }
@@ -449,7 +460,13 @@ export function parseFlowchart(
     }
     for (const node of result.nodes) {
       if (!connectedIds.has(node.id)) {
-        result.diagnostics.push(makeDgmoError(node.lineNumber, `Node "${node.label}" is not connected to any other node`, 'warning'));
+        result.diagnostics.push(
+          makeDgmoError(
+            node.lineNumber,
+            `Node "${node.label}" is not connected to any other node`,
+            'warning'
+          )
+        );
       }
     }
   }
@@ -485,7 +502,7 @@ export function looksLikeFlowchart(content: string): boolean {
   // Look for patterns like `[X] ->` or `-> [X]` or `(X) ->` etc.
   const shapeNearArrow =
     /[\])][ \t]*-.*->/.test(content) || // shape ] or ) followed by arrow
-    /->[ \t]*[\[(<\/]/.test(content); // arrow followed by shape opener
+    /->[ \t]*[[(</]/.test(content); // arrow followed by shape opener
 
   return shapeNearArrow;
 }
@@ -509,7 +526,11 @@ export function extractSymbols(docText: string): DiagramSymbols {
   for (const rawLine of docText.split('\n')) {
     const line = rawLine.trim();
     // Skip old-style colon metadata and new-style space-separated options
-    if (inMetadata && (/^[a-z-]+\s*:/i.test(line) || /^[a-z-]+\s+\S/i.test(line))) continue;
+    if (
+      inMetadata &&
+      (/^[a-z-]+\s*:/i.test(line) || /^[a-z-]+\s+\S/i.test(line))
+    )
+      continue;
     inMetadata = false;
     if (line.length === 0 || /^\s/.test(rawLine)) continue;
     const m = NODE_ID_RE.exec(line);

package/src/graph/state-parser.ts

@@ -8,12 +8,7 @@ import {
   OPTION_NOCOLON_RE,
   ALL_CHART_TYPES,
 } from '../utils/parsing';
-import type {
-  ParsedGraph,
-  GraphNode,
-  GraphGroup,
-  GraphDirection,
-} from './types';
+import type { ParsedGraph, GraphNode, GraphGroup } from './types';
 
 // ============================================================
 // Constants
@@ -36,7 +31,12 @@ const GROUP_BRACKET_RE = /^\[([^\]]+)\](?:\(([^)]+)\))?\s*$/;
  */
 function splitArrows(line: string): string[] {
   const segments: string[] = [];
-  const arrowPositions: { start: number; end: number; label?: string; color?: string }[] = [];
+  const arrowPositions: {
+    start: number;
+    end: number;
+    label?: string;
+    color?: string;
+  }[] = [];
 
   let searchFrom = 0;
   while (searchFrom < line.length) {
@@ -52,9 +52,13 @@ function splitArrows(line: string): string[] {
       while (scanBack > 0 && line[scanBack] !== '-') {
         scanBack--;
       }
-      if (line[scanBack] === '-' && (scanBack === 0 || /\s/.test(line[scanBack - 1]))) {
+      if (
+        line[scanBack] === '-' &&
+        (scanBack === 0 || /\s/.test(line[scanBack - 1]))
+      ) {
         let arrowContent = line.substring(scanBack + 1, idx);
-        if (arrowContent.endsWith('-')) arrowContent = arrowContent.slice(0, -1);
+        if (arrowContent.endsWith('-'))
+          arrowContent = arrowContent.slice(0, -1);
         const colorMatch = arrowContent.match(/\(([^)]+)\)\s*$/);
         if (colorMatch) {
           color = colorMatch[1].trim();
@@ -81,7 +85,8 @@ function splitArrows(line: string): string[] {
     if (beforeText || i === 0) segments.push(beforeText);
 
     let arrowToken = '->';
-    if (arrow.label && arrow.color) arrowToken = `-${arrow.label}(${arrow.color})->`;
+    if (arrow.label && arrow.color)
+      arrowToken = `-${arrow.label}(${arrow.color})->`;
     else if (arrow.label) arrowToken = `-${arrow.label}->`;
     else if (arrow.color) arrowToken = `-(${arrow.color})->`;
     segments.push(arrowToken);
@@ -101,11 +106,14 @@ interface ArrowInfo {
 function parseArrowToken(token: string, palette?: PaletteColors): ArrowInfo {
   if (token === '->') return {};
   const colorOnly = token.match(/^-\(([^)]+)\)->$/);
-  if (colorOnly) return { color: resolveColor(colorOnly[1].trim(), palette) ?? undefined };
+  if (colorOnly)
+    return { color: resolveColor(colorOnly[1].trim(), palette) ?? undefined };
   const m = token.match(/^-(.+?)(?:\(([^)]+)\))?->$/);
   if (m) {
     const label = m[1]?.trim() || undefined;
-    const color = m[2] ? resolveColor(m[2].trim(), palette) ?? undefined : undefined;
+    const color = m[2]
+      ? (resolveColor(m[2].trim(), palette) ?? undefined)
+      : undefined;
     return { label, color };
   }
   return {};
@@ -122,13 +130,20 @@ interface NodeRef {
   color?: string;
 }
 
-function parseStateNodeRef(text: string, palette?: PaletteColors): NodeRef | null {
+function parseStateNodeRef(
+  text: string,
+  palette?: PaletteColors
+): NodeRef | null {
   const t = text.trim();
   if (!t) return null;
 
   // Pseudostate: [*]
   if (t === '[*]') {
-    return { id: PSEUDOSTATE_ID, label: PSEUDOSTATE_LABEL, shape: 'pseudostate' };
+    return {
+      id: PSEUDOSTATE_ID,
+      label: PSEUDOSTATE_LABEL,
+      shape: 'pseudostate',
+    };
   }
 
   // State: bare text with optional (color) suffix
@@ -350,7 +365,13 @@ export function parseState(
         // Use explicit source if available, else implicit from indent
         const sourceId = lastNodeId ?? implicitSourceId;
         if (sourceId) {
-          addEdge(sourceId, node.id, lineNumber, pendingArrow.label, pendingArrow.color);
+          addEdge(
+            sourceId,
+            node.id,
+            lineNumber,
+            pendingArrow.label,
+            pendingArrow.color
+          );
         }
         pendingArrow = null;
       }
@@ -367,7 +388,10 @@ export function parseState(
 
   // Validation: no nodes found
   if (result.nodes.length === 0 && !result.error) {
-    const diag = makeDgmoError(1, 'No states found. Add state transitions like: Idle -> Active');
+    const diag = makeDgmoError(
+      1,
+      'No states found. Add state transitions like: Idle -> Active'
+    );
     result.diagnostics.push(diag);
     result.error = formatDgmoError(diag);
   }
@@ -381,7 +405,13 @@ export function parseState(
     }
     for (const node of result.nodes) {
       if (!connectedIds.has(node.id)) {
-        result.diagnostics.push(makeDgmoError(node.lineNumber, `State "${node.label}" is not connected to any other state`, 'warning'));
+        result.diagnostics.push(
+          makeDgmoError(
+            node.lineNumber,
+            `State "${node.label}" is not connected to any other state`,
+            'warning'
+          )
+        );
       }
     }
   }

package/src/infra/parser.ts

@@ -7,7 +7,11 @@
 // and connections, [Group] containers, tag groups, pipe metadata.
 
 import { makeDgmoError, formatDgmoError, suggest } from '../diagnostics';
-import { measureIndent, parseFirstLine, OPTION_NOCOLON_RE } from '../utils/parsing';
+import {
+  measureIndent,
+  parseFirstLine,
+  OPTION_NOCOLON_RE,
+} from '../utils/parsing';
 import { matchTagBlockHeading } from '../utils/tag-groups';
 import type {
   ParsedInfra,
@@ -21,21 +25,17 @@ import { INFRA_BEHAVIOR_KEYS, EDGE_ONLY_KEYS } from './types';
 // Regex patterns
 // ============================================================
 
-// Connection: -label-> Target  or  -> Target  (with optional | split: N%  or pipe metadata)
-const CONNECTION_RE =
-  /^-(?:([^-].*?))?->\s*(.+?)(?:(?:\s*\|\s*|\s+)split\s*:?\s*(\d+)%)?\s*$/;
+// Connection: -label-> Target  or  -> Target  (pipe metadata handled by extractPipeMetadata)
+const CONNECTION_RE = /^-(?:([^-].*?))?->\s*(.+?)\s*$/;
 
 // Simple connection shorthand: -> Target (no label, no dash prefix needed for edge)
-const SIMPLE_CONNECTION_RE =
-  /^->\s*(.+?)(?:(?:\s*\|\s*|\s+)split\s*:?\s*(\d+)%)?\s*$/;
+const SIMPLE_CONNECTION_RE = /^->\s*(.+?)\s*$/;
 
-// Async connection: ~label~> Target  or  ~> Target  (with optional | split: N%  or pipe metadata)
-const ASYNC_CONNECTION_RE =
-  /^~(?:([^~].*?))?~>\s*(.+?)(?:(?:\s*\|\s*|\s+)split\s*:?\s*(\d+)%)?\s*$/;
+// Async connection: ~label~> Target  or  ~> Target
+const ASYNC_CONNECTION_RE = /^~(?:([^~].*?))?~>\s*(.+?)\s*$/;
 
 // Async simple connection shorthand: ~> Target
-const ASYNC_SIMPLE_CONNECTION_RE =
-  /^~>\s*(.+?)(?:(?:\s*\|\s*|\s+)split\s*:?\s*(\d+)%)?\s*$/;
+const ASYNC_SIMPLE_CONNECTION_RE = /^~>\s*(.+?)\s*$/;
 
 // Deprecated xN fanout suffix (e.g. "x5" at end of line)
 const DEPRECATED_FANOUT_RE = /\bx(\d+)\s*$/;
@@ -68,8 +68,12 @@ const EDGE_NODE_NAMES = new Set(['edge', 'internet']);
 
 // Known top-level option keys (space-separated, no colon)
 const TOP_LEVEL_OPTIONS = new Set([
-  'slo-availability', 'slo-p90-latency-ms', 'slo-warning-margin',
-  'default-latency-ms', 'default-uptime', 'default-rps',
+  'slo-availability',
+  'slo-p90-latency-ms',
+  'slo-warning-margin',
+  'default-latency-ms',
+  'default-uptime',
+  'default-rps',
 ]);
 
 // ============================================================
@@ -94,9 +98,10 @@ function parsePropertyValue(raw: string): string | number {
   return raw.trim();
 }
 
-function extractPipeMetadata(
-  rest: string,
-): { tags: Record<string, string>; clean: string } {
+function extractPipeMetadata(rest: string): {
+  tags: Record<string, string>;
+  clean: string;
+} {
   const tags: Record<string, string> = {};
   let clean = rest;
   let match: RegExpExecArray | null;
@@ -108,6 +113,30 @@ function extractPipeMetadata(
   return { tags, clean: clean.trim() };
 }
 
+// Detect unparsed pipe metadata left in a target name after extractPipeMetadata.
+// Common case: `split 100%` without a colon isn't picked up by PIPE_META_RE.
+const UNPARSED_SPLIT_RE = /\bsplit\s+(\d+)%/;
+
+function warnUnparsedPipeMeta(
+  targetName: string,
+  lineNumber: number,
+  warnFn: (line: number, message: string) => void
+): void {
+  if (!targetName.includes('|')) return;
+  const splitMatch = targetName.match(UNPARSED_SPLIT_RE);
+  if (splitMatch) {
+    warnFn(
+      lineNumber,
+      `'split ${splitMatch[1]}%' needs a colon — use 'split: ${splitMatch[1]}%'`
+    );
+  } else {
+    warnFn(
+      lineNumber,
+      `Unparsed pipe metadata in target — pipe values use 'key: value' syntax`
+    );
+  }
+}
+
 // ============================================================
 // Parser
 // ============================================================
@@ -150,20 +179,26 @@ export function parseInfra(content: string): ParsedInfra {
     if (currentNode && !nodeMap.has(currentNode.id)) {
       // Validate mutual exclusion: concurrency vs instances/max-rps
       const keys = new Set(currentNode.properties.map((p) => p.key));
-      if (keys.has('concurrency') && (keys.has('instances') || keys.has('max-rps'))) {
-        const conflicting = [keys.has('instances') ? 'instances' : '', keys.has('max-rps') ? 'max-rps' : '']
+      if (
+        keys.has('concurrency') &&
+        (keys.has('instances') || keys.has('max-rps'))
+      ) {
+        const conflicting = [
+          keys.has('instances') ? 'instances' : '',
+          keys.has('max-rps') ? 'max-rps' : '',
+        ]
           .filter(Boolean)
           .join(', ');
         warn(
           currentNode.lineNumber,
-          `'concurrency' (serverless) is mutually exclusive with ${conflicting}. Serverless nodes scale via concurrency, not instances.`,
+          `'concurrency' (serverless) is mutually exclusive with ${conflicting}. Serverless nodes scale via concurrency, not instances.`
         );
       }
       // Validate mutual exclusion: buffer (queue) vs max-rps (service)
       if (keys.has('buffer') && keys.has('max-rps')) {
         warn(
           currentNode.lineNumber,
-          `'buffer' (queue) and 'max-rps' (service) represent different capacity models. A queue buffers messages; a service processes them.`,
+          `'buffer' (queue) and 'max-rps' (service) represent different capacity models. A queue buffers messages; a service processes them.`
         );
       }
       nodeMap.set(currentNode.id, currentNode);
@@ -202,7 +237,10 @@ export function parseInfra(content: string): ParsedInfra {
       const firstLineResult = parseFirstLine(trimmed);
       if (firstLineResult) {
         if (firstLineResult.chartType !== 'infra') {
-          setError(lineNumber, `Expected chart type 'infra', got '${firstLineResult.chartType}'`);
+          setError(
+            lineNumber,
+            `Expected chart type 'infra', got '${firstLineResult.chartType}'`
+          );
         }
         if (firstLineResult.title) {
           result.title = firstLineResult.title;
@@ -255,11 +293,16 @@ export function parseInfra(content: string): ParsedInfra {
         finishCurrentTagGroup();
         const gLabel = groupMatch[1].trim();
         const gId = groupId(gLabel);
-        const groupMeta = groupMatch[2] ? extractPipeMetadata('|' + groupMatch[2]).tags : undefined;
+        const groupMeta = groupMatch[2]
+          ? extractPipeMetadata('|' + groupMatch[2]).tags
+          : undefined;
         currentGroup = {
           id: gId,
           label: gLabel,
-          metadata: groupMeta && Object.keys(groupMeta).length > 0 ? groupMeta : undefined,
+          metadata:
+            groupMeta && Object.keys(groupMeta).length > 0
+              ? groupMeta
+              : undefined,
           lineNumber,
         };
         result.groups.push(currentGroup);
@@ -310,6 +353,11 @@ export function parseInfra(content: string): ParsedInfra {
         }
         continue;
       }
+      warn(
+        lineNumber,
+        `Invalid tag value '${trimmed}' in tag group '${currentTagGroup.name}'.`
+      );
+      continue;
     }
 
     // Inside a [Group] but no current node — group properties or component declaration
@@ -333,6 +381,8 @@ export function parseInfra(content: string): ParsedInfra {
           currentGroup.collapsed = val.toLowerCase() === 'true';
           continue;
         }
+        // Fall through to component matching — could be a component name
+        // that happens to match PROPERTY_RE (e.g., "MyService v2")
       }
 
       const compMatch = trimmed.match(COMPONENT_RE);
@@ -365,9 +415,17 @@ export function parseInfra(content: string): ParsedInfra {
     if (currentNode && indent > baseIndent) {
       // Detect deprecated xN fanout syntax
       const deprecatedFanout = trimmed.match(DEPRECATED_FANOUT_RE);
-      if (deprecatedFanout && (trimmed.startsWith('->') || trimmed.startsWith('-') || trimmed.startsWith('~'))) {
+      if (
+        deprecatedFanout &&
+        (trimmed.startsWith('->') ||
+          trimmed.startsWith('-') ||
+          trimmed.startsWith('~'))
+      ) {
         const n = deprecatedFanout[1];
-        setError(lineNumber, `'x${n}' fanout syntax is no longer supported — use '| fanout: ${n}' instead`);
+        setError(
+          lineNumber,
+          `'x${n}' fanout syntax is no longer supported — use '| fanout: ${n}' instead`
+        );
         continue;
       }
 
@@ -375,14 +433,20 @@ export function parseInfra(content: string): ParsedInfra {
       const asyncSimpleConn = trimmed.match(ASYNC_SIMPLE_CONNECTION_RE);
       if (asyncSimpleConn) {
         const targetRaw = asyncSimpleConn[1].trim();
-        const splitStr = asyncSimpleConn[2];
         const pipeMeta = extractPipeMetadata(targetRaw);
         const targetName = pipeMeta.clean || targetRaw;
-        const split = splitStr ? parseFloat(splitStr)
-          : pipeMeta.tags.split ? parseFloat(pipeMeta.tags.split) : null;
-        const fanoutRaw = pipeMeta.tags.fanout ? parseInt(pipeMeta.tags.fanout, 10) : null;
+        warnUnparsedPipeMeta(targetName, lineNumber, warn);
+        const split = pipeMeta.tags.split
+          ? parseFloat(pipeMeta.tags.split)
+          : null;
+        const fanoutRaw = pipeMeta.tags.fanout
+          ? parseInt(pipeMeta.tags.fanout, 10)
+          : null;
         if (fanoutRaw !== null && fanoutRaw < 1) {
-          warn(lineNumber, `Fan-out multiplier must be at least 1 (got fanout: ${fanoutRaw}). Ignoring.`);
+          warn(
+            lineNumber,
+            `Fan-out multiplier must be at least 1 (got fanout: ${fanoutRaw}). Ignoring.`
+          );
         }
         const fanout = fanoutRaw !== null && fanoutRaw >= 1 ? fanoutRaw : null;
         result.edges.push({
@@ -402,14 +466,20 @@ export function parseInfra(content: string): ParsedInfra {
       if (asyncConnMatch) {
         const label = asyncConnMatch[1]?.trim() || '';
         const targetRaw = asyncConnMatch[2].trim();
-        const splitStr = asyncConnMatch[3];
         const pipeMeta = extractPipeMetadata(targetRaw);
         const targetName = pipeMeta.clean || targetRaw;
-        const split = splitStr ? parseFloat(splitStr)
-          : pipeMeta.tags.split ? parseFloat(pipeMeta.tags.split) : null;
-        const fanoutRaw = pipeMeta.tags.fanout ? parseInt(pipeMeta.tags.fanout, 10) : null;
+        warnUnparsedPipeMeta(targetName, lineNumber, warn);
+        const split = pipeMeta.tags.split
+          ? parseFloat(pipeMeta.tags.split)
+          : null;
+        const fanoutRaw = pipeMeta.tags.fanout
+          ? parseInt(pipeMeta.tags.fanout, 10)
+          : null;
         if (fanoutRaw !== null && fanoutRaw < 1) {
-          warn(lineNumber, `Fan-out multiplier must be at least 1 (got fanout: ${fanoutRaw}). Ignoring.`);
+          warn(
+            lineNumber,
+            `Fan-out multiplier must be at least 1 (got fanout: ${fanoutRaw}). Ignoring.`
+          );
         }
         const fanout = fanoutRaw !== null && fanoutRaw >= 1 ? fanoutRaw : null;
 
@@ -437,15 +507,20 @@ export function parseInfra(content: string): ParsedInfra {
       const simpleConn = trimmed.match(SIMPLE_CONNECTION_RE);
       if (simpleConn) {
         const targetRaw = simpleConn[1].trim();
-        const splitStr = simpleConn[2];
-        // Parse pipe metadata for fanout/split (and clean target name)
         const pipeMeta = extractPipeMetadata(targetRaw);
         const targetName = pipeMeta.clean || targetRaw;
-        const split = splitStr ? parseFloat(splitStr)
-          : pipeMeta.tags.split ? parseFloat(pipeMeta.tags.split) : null;
-        const fanoutRaw = pipeMeta.tags.fanout ? parseInt(pipeMeta.tags.fanout, 10) : null;
+        warnUnparsedPipeMeta(targetName, lineNumber, warn);
+        const split = pipeMeta.tags.split
+          ? parseFloat(pipeMeta.tags.split)
+          : null;
+        const fanoutRaw = pipeMeta.tags.fanout
+          ? parseInt(pipeMeta.tags.fanout, 10)
+          : null;
         if (fanoutRaw !== null && fanoutRaw < 1) {
-          warn(lineNumber, `Fan-out multiplier must be at least 1 (got fanout: ${fanoutRaw}). Ignoring.`);
+          warn(
+            lineNumber,
+            `Fan-out multiplier must be at least 1 (got fanout: ${fanoutRaw}). Ignoring.`
+          );
         }
         const fanout = fanoutRaw !== null && fanoutRaw >= 1 ? fanoutRaw : null;
         result.edges.push({
@@ -465,15 +540,20 @@ export function parseInfra(content: string): ParsedInfra {
       if (connMatch) {
         const label = connMatch[1]?.trim() || '';
         const targetRaw = connMatch[2].trim();
-        const splitStr = connMatch[3];
-        // Parse pipe metadata for fanout/split (and clean target name)
         const pipeMeta = extractPipeMetadata(targetRaw);
         const targetName = pipeMeta.clean || targetRaw;
-        const split = splitStr ? parseFloat(splitStr)
-          : pipeMeta.tags.split ? parseFloat(pipeMeta.tags.split) : null;
-        const fanoutRaw = pipeMeta.tags.fanout ? parseInt(pipeMeta.tags.fanout, 10) : null;
+        warnUnparsedPipeMeta(targetName, lineNumber, warn);
+        const split = pipeMeta.tags.split
+          ? parseFloat(pipeMeta.tags.split)
+          : null;
+        const fanoutRaw = pipeMeta.tags.fanout
+          ? parseInt(pipeMeta.tags.fanout, 10)
+          : null;
         if (fanoutRaw !== null && fanoutRaw < 1) {
-          warn(lineNumber, `Fan-out multiplier must be at least 1 (got fanout: ${fanoutRaw}). Ignoring.`);
+          warn(
+            lineNumber,
+            `Fan-out multiplier must be at least 1 (got fanout: ${fanoutRaw}). Ignoring.`
+          );
         }
         const fanout = fanoutRaw !== null && fanoutRaw >= 1 ? fanoutRaw : null;
 
@@ -525,7 +605,10 @@ export function parseInfra(content: string): ParsedInfra {
 
         // Validate edge-only keys
         if (EDGE_ONLY_KEYS.has(key) && !currentNode.isEdge) {
-          warn(lineNumber, `Property '${key}' is only valid on the entry point (Edge/Internet).`);
+          warn(
+            lineNumber,
+            `Property '${key}' is only valid on the entry point (Edge/Internet).`
+          );
         }
 
         const value = parsePropertyValue(rawVal);
@@ -534,7 +617,10 @@ export function parseInfra(content: string): ParsedInfra {
       }
 
       // Unknown indented line
-      warn(lineNumber, `Unexpected line inside component '${currentNode.label}'.`);
+      warn(
+        lineNumber,
+        `Unexpected line inside component '${currentNode.label}'.`
+      );
       continue;
     }
 
@@ -592,6 +678,9 @@ export function parseInfra(content: string): ParsedInfra {
         continue;
       }
     }
+
+    // Catch-all: nothing matched this line
+    warn(lineNumber, `Unexpected line: '${trimmed}'.`);
   }
 
   // Flush last open blocks
@@ -661,7 +750,8 @@ export function extractSymbols(docText: string): DiagramSymbols {
         // Recognize new-style bare options (`key value`) and old-style (`key: value`)
         const firstLine = parseFirstLine(line);
         if (firstLine) continue; // chart type line
-        if (/^(?:direction-tb|animate|no-animate|slo-|default-)/i.test(line)) continue;
+        if (/^(?:direction-tb|animate|no-animate|slo-|default-)/i.test(line))
+          continue;
         if (/^[a-z-]+\s*:/i.test(line)) continue; // legacy colon options
         inMetadata = false;
       } else {
@@ -671,8 +761,14 @@ export function extractSymbols(docText: string): DiagramSymbols {
 
     if (!indented) {
       // Root-level: tag group declaration, group header, or component
-      if (/^tag\s/i.test(line)) { inTagGroup = true; continue; }
-      if (/^tag\s*:/i.test(line)) { inTagGroup = true; continue; } // legacy
+      if (/^tag\s/i.test(line)) {
+        inTagGroup = true;
+        continue;
+      }
+      if (/^tag\s*:/i.test(line)) {
+        inTagGroup = true;
+        continue;
+      } // legacy
       inTagGroup = false;
       if (/^\[/.test(line)) continue; // [Group] header
       const m = COMPONENT_RE.exec(line);
@@ -687,7 +783,15 @@ export function extractSymbols(docText: string): DiagramSymbols {
       if (/^\w[\w-]*\s*:/.test(line)) continue; // property (key: value) legacy
       // New-style property: first token is a known behavior/property key
       const firstToken = line.split(/\s/)[0].toLowerCase();
-      if ((INFRA_BEHAVIOR_KEYS.has(firstToken) || EDGE_ONLY_KEYS.has(firstToken) || firstToken === 'description' || firstToken === 'instances' || firstToken === 'collapsed') && /\s/.test(line)) continue;
+      if (
+        (INFRA_BEHAVIOR_KEYS.has(firstToken) ||
+          EDGE_ONLY_KEYS.has(firstToken) ||
+          firstToken === 'description' ||
+          firstToken === 'instances' ||
+          firstToken === 'collapsed') &&
+        /\s/.test(line)
+      )
+        continue;
       const m = COMPONENT_RE.exec(line);
       if (m && !entities.includes(m[1]!)) entities.push(m[1]!);
     }