@diagrammo/dgmo 0.8.2 → 0.8.4

package/src/infra/types.ts

@@ -62,7 +62,6 @@ export interface InfraNode {
   groupId: string | null;
   tags: Record<string, string>; // tagGroup -> tagValue
   isEdge: boolean; // true for the `edge` entry-point component
-  nodeType?: string; // database, cache, queue, service, gateway, storage, function, network
   description?: string;
   lineNumber: number;
 }

package/src/initiative-status/parser.ts

@@ -13,9 +13,17 @@ import type {
 } from './types';
 import { VALID_STATUSES, STATUS_ALIASES } from './types';
 import { inferParticipantType } from '../sequence/participant-inference';
-import { matchTagBlockHeading, injectDefaultTagMetadata, validateTagValues } from '../utils/tag-groups';
+import {
+  matchTagBlockHeading,
+  injectDefaultTagMetadata,
+  validateTagValues,
+} from '../utils/tag-groups';
 import type { TagGroup } from '../utils/tag-groups';
-import { extractColor, parseFirstLine, ALL_CHART_TYPES, OPTION_NOCOLON_RE } from '../utils/parsing';
+import {
+  extractColor,
+  parseFirstLine,
+  OPTION_NOCOLON_RE,
+} from '../utils/parsing';
 
 // ============================================================
 // Heuristic — does this content look like an initiative-status diagram?
@@ -38,10 +46,14 @@ export function looksLikeInitiativeStatus(content: string): boolean {
     // Skip new-style first line (bare chart type name)
     if (parseFirstLine(trimmed)) continue;
     if (trimmed.includes('->')) hasArrow = true;
-    if (/\|\s*(done|doing|wip|blocked|paused|waiting|todo|na)\s*$/i.test(trimmed)) hasStatus = true;
+    if (
+      /\|\s*(done|doing|wip|blocked|paused|waiting|todo|na)\s*$/i.test(trimmed)
+    )
+      hasStatus = true;
     // Indented arrow is a strong signal — only initiative-status uses this
     const isIndented = line.length > 0 && line !== trimmed && /^\s/.test(line);
-    if (isIndented && (trimmed.startsWith('->') || /^-[^>].*->/.test(trimmed))) hasIndentedArrow = true;
+    if (isIndented && (trimmed.startsWith('->') || /^-[^>].*->/.test(trimmed)))
+      hasIndentedArrow = true;
     if (hasArrow && hasStatus) return true;
   }
   return hasIndentedArrow;
@@ -67,7 +79,11 @@ export function parseNodeMetadata(
   aliasMap: Map<string, string>,
   lineNum?: number,
   diagnostics?: DgmoError[]
-): { status: InitiativeStatus; metadata: Record<string, string>; hadStatusWord: boolean } {
+): {
+  status: InitiativeStatus;
+  metadata: Record<string, string>;
+  hadStatusWord: boolean;
+} {
   const metadata: Record<string, string> = {};
   let status: InitiativeStatus = null;
   let hadStatusWord = false;
@@ -125,7 +141,12 @@ export function parseNodeMetadata(
 // Parser
 // ============================================================
 
-function parseStatus(raw: string, line: number, diagnostics: DgmoError[]): InitiativeStatus {
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
+function _parseStatus(
+  raw: string,
+  line: number,
+  diagnostics: DgmoError[]
+): InitiativeStatus {
   const trimmed = raw.trim().toLowerCase();
   if (!trimmed) return 'na';
   const canonical = STATUS_ALIASES[trimmed] ?? trimmed;
@@ -191,7 +212,10 @@ export function parseInitiativeStatus(content: string): ParsedInitiativeStatus {
     const firstLineResult = parseFirstLine(trimmed);
     if (firstLineResult && !contentStarted) {
       if (firstLineResult.chartType !== 'initiative-status') {
-        const diag = makeDgmoError(lineNum, `Expected chart type "initiative-status", got "${firstLineResult.chartType}"`);
+        const diag = makeDgmoError(
+          lineNum,
+          `Expected chart type "initiative-status", got "${firstLineResult.chartType}"`
+        );
         result.diagnostics.push(diag);
         result.error = formatDgmoError(diag);
         return result;
@@ -203,16 +227,19 @@ export function parseInitiativeStatus(content: string): ParsedInitiativeStatus {
       continue;
     }
 
-    // hide directive (no colon): `hide phase Planning, phase Review`
+    // hide directive (colon syntax): `hide phase:Planning, phase:Review`
     const hideMatch = trimmed.match(/^hide\s+(.+)/i);
     if (hideMatch && !trimmed.match(/^hide\s*\|/)) {
-      // Parse comma-separated tag-value pairs: `phase Planning, phase Review`
+      // Parse comma-separated tag:value pairs: `phase:Planning, phase:Review`
       const pairs = hideMatch[1].split(',');
       for (const pair of pairs) {
-        const tokens = pair.trim().split(/\s+/);
-        if (tokens.length >= 2) {
-          const groupKey = tokens[0].toLowerCase();
-          const value = tokens.slice(1).join(' ').toLowerCase();
+        const colonIdx = pair.indexOf(':');
+        if (colonIdx > 0) {
+          const groupKey = pair.substring(0, colonIdx).trim().toLowerCase();
+          const value = pair
+            .substring(colonIdx + 1)
+            .trim()
+            .toLowerCase();
           if (groupKey && value) {
             if (!result.initialHiddenTagValues.has(groupKey)) {
               result.initialHiddenTagValues.set(groupKey, new Set());
@@ -231,7 +258,7 @@ export function parseInitiativeStatus(content: string): ParsedInitiativeStatus {
         const key = optMatch[1].toLowerCase();
         const value = optMatch[2].trim();
         // Only recognize known option keys (not node content)
-        if (key === 'active-tag' || key === 'sort') {
+        if (key === 'active-tag') {
           result.options[key] = value;
           continue;
         }
@@ -243,13 +270,11 @@ export function parseInitiativeStatus(content: string): ParsedInitiativeStatus {
     if (tagBlockMatch) {
       if (contentStarted) {
         result.diagnostics.push(
-          makeDgmoError(lineNum, 'Tag groups must appear before diagram content', 'error')
-        );
-        continue;
-      }
-      if (tagBlockMatch.deprecated) {
-        result.diagnostics.push(
-          makeDgmoError(lineNum, `'## ${tagBlockMatch.name}' is no longer supported — use 'tag ${tagBlockMatch.name}' instead`)
+          makeDgmoError(
+            lineNum,
+            'Tag groups must appear before diagram content',
+            'error'
+          )
         );
         continue;
       }
@@ -260,7 +285,10 @@ export function parseInitiativeStatus(content: string): ParsedInitiativeStatus {
         lineNumber: lineNum,
       };
       if (tagBlockMatch.alias) {
-        aliasMap.set(tagBlockMatch.alias.toLowerCase(), tagBlockMatch.name.toLowerCase());
+        aliasMap.set(
+          tagBlockMatch.alias.toLowerCase(),
+          tagBlockMatch.name.toLowerCase()
+        );
       }
       // Handle inline values from single-line tag declaration
       if (tagBlockMatch.inlineValues) {
@@ -298,7 +326,7 @@ export function parseInitiativeStatus(content: string): ParsedInitiativeStatus {
         continue;
       }
       // Non-indented line after tag group — close and fall through
-      currentTagGroup = null;
+      currentTagGroup = null; // eslint-disable-line no-useless-assignment
     }
 
     // Group header: [Group Name] or [Group Name] | metadata
@@ -349,13 +377,22 @@ export function parseInitiativeStatus(content: string): ParsedInitiativeStatus {
       if (trimmed.startsWith('->') || /^-[^>].*->/.test(trimmed)) {
         if (!lastNodeLabel) {
           result.diagnostics.push(
-            makeDgmoError(lineNum, 'Indented edge has no preceding node to use as source', 'warning')
+            makeDgmoError(
+              lineNum,
+              'Indented edge has no preceding node to use as source',
+              'warning'
+            )
           );
           continue;
         }
         edgeText = `${lastNodeLabel} ${trimmed}`;
       }
-      const edge = parseEdgeLine(edgeText, lineNum, aliasMap, result.diagnostics);
+      const edge = parseEdgeLine(
+        edgeText,
+        lineNum,
+        aliasMap,
+        result.diagnostics
+      );
       if (edge) result.edges.push(edge);
       continue;
     }
@@ -364,28 +401,32 @@ export function parseInitiativeStatus(content: string): ParsedInitiativeStatus {
     contentStarted = true;
     currentTagGroup = null;
     const node = parseNodeLine(trimmed, lineNum, aliasMap, result.diagnostics);
-    if (node) {
-      lastNodeLabel = node.label;
-      if (nodeLabels.has(node.label)) {
-        result.diagnostics.push(
-          makeDgmoError(lineNum, `Duplicate node "${node.label}"`, 'warning')
-        );
-      } else {
-        nodeLabels.add(node.label);
-      }
-      // Cascade group metadata into node (group provides defaults, node overrides)
-      if (currentGroup && isIndented && currentGroup.metadata) {
-        for (const [key, val] of Object.entries(currentGroup.metadata)) {
-          if (!(key in node.metadata)) {
-            node.metadata[key] = val;
-          }
+    if (!node) {
+      result.diagnostics.push(
+        makeDgmoError(lineNum, `Unexpected line: '${trimmed}'.`, 'warning')
+      );
+      continue;
+    }
+    lastNodeLabel = node.label;
+    if (nodeLabels.has(node.label)) {
+      result.diagnostics.push(
+        makeDgmoError(lineNum, `Duplicate node "${node.label}"`, 'warning')
+      );
+    } else {
+      nodeLabels.add(node.label);
+    }
+    // Cascade group metadata into node (group provides defaults, node overrides)
+    if (currentGroup && isIndented && currentGroup.metadata) {
+      for (const [key, val] of Object.entries(currentGroup.metadata)) {
+        if (!(key in node.metadata)) {
+          node.metadata[key] = val;
         }
       }
-      result.nodes.push(node);
-      // Add to current group if indented
-      if (currentGroup && isIndented) {
-        currentGroup.nodeLabels.push(node.label);
-      }
+    }
+    result.nodes.push(node);
+    // Add to current group if indented
+    if (currentGroup && isIndented) {
+      currentGroup.nodeLabels.push(node.label);
     }
   }
 
@@ -398,20 +439,40 @@ export function parseInitiativeStatus(content: string): ParsedInitiativeStatus {
   for (const edge of result.edges) {
     if (!nodeLabels.has(edge.source)) {
       result.diagnostics.push(
-        makeDgmoError(edge.lineNumber, `Edge source "${edge.source}" is not a declared node`, 'warning')
+        makeDgmoError(
+          edge.lineNumber,
+          `Edge source "${edge.source}" is not a declared node`,
+          'warning'
+        )
       );
       // Auto-create an implicit node
       if (!result.nodes.some((n) => n.label === edge.source)) {
-        result.nodes.push({ label: edge.source, status: 'na', shape: inferParticipantType(edge.source), lineNumber: edge.lineNumber, metadata: {} });
+        result.nodes.push({
+          label: edge.source,
+          status: 'na',
+          shape: inferParticipantType(edge.source),
+          lineNumber: edge.lineNumber,
+          metadata: {},
+        });
         nodeLabels.add(edge.source);
       }
     }
     if (!nodeLabels.has(edge.target)) {
       result.diagnostics.push(
-        makeDgmoError(edge.lineNumber, `Edge target "${edge.target}" is not a declared node`, 'warning')
+        makeDgmoError(
+          edge.lineNumber,
+          `Edge target "${edge.target}" is not a declared node`,
+          'warning'
+        )
       );
       if (!result.nodes.some((n) => n.label === edge.target)) {
-        result.nodes.push({ label: edge.target, status: 'na', shape: inferParticipantType(edge.target), lineNumber: edge.lineNumber, metadata: {} });
+        result.nodes.push({
+          label: edge.target,
+          status: 'na',
+          shape: inferParticipantType(edge.target),
+          lineNumber: edge.lineNumber,
+          metadata: {},
+        });
         nodeLabels.add(edge.target);
       }
     }
@@ -443,7 +504,12 @@ function parseNodeLine(
     const label = trimmed.slice(0, pipeIdx).trim();
     const metaSegment = trimmed.slice(pipeIdx + 1).trim();
     if (!label) return null;
-    const { status, metadata, hadStatusWord } = parseNodeMetadata(metaSegment, aliasMap, lineNum, diagnostics);
+    const { status, metadata, hadStatusWord } = parseNodeMetadata(
+      metaSegment,
+      aliasMap,
+      lineNum,
+      diagnostics
+    );
     return {
       label,
       // Unknown status bare word → keep null; no bare word at all → default 'na'
@@ -453,7 +519,13 @@ function parseNodeLine(
       metadata,
     };
   }
-  return { label: trimmed, status: 'na', shape: inferParticipantType(trimmed), lineNumber: lineNum, metadata: {} };
+  return {
+    label: trimmed,
+    status: 'na',
+    shape: inferParticipantType(trimmed),
+    lineNumber: lineNum,
+    metadata: {},
+  };
 }
 
 function parseEdgeLine(
@@ -481,8 +553,15 @@ function parseEdgeLine(
       const pipeIdx = targetRest.indexOf('|');
       if (pipeIdx >= 0) {
         const metaSegment = targetRest.slice(pipeIdx + 1).trim();
-        const parsed = parseNodeMetadata(metaSegment, aliasMap, lineNum, diagnostics);
-        status = parsed.hadStatusWord ? (parsed.status ?? null) : (parsed.status ?? 'na');
+        const parsed = parseNodeMetadata(
+          metaSegment,
+          aliasMap,
+          lineNum,
+          diagnostics
+        );
+        status = parsed.hadStatusWord
+          ? (parsed.status ?? null)
+          : (parsed.status ?? 'na');
         metadata = parsed.metadata;
         targetRest = targetRest.slice(0, pipeIdx).trim();
       }
@@ -505,7 +584,9 @@ function parseEdgeLine(
   let rest = trimmed.slice(arrowIdx + 2).trim();
 
   if (!source || !rest) {
-    diagnostics.push(makeDgmoError(lineNum, 'Edge is missing source or target'));
+    diagnostics.push(
+      makeDgmoError(lineNum, 'Edge is missing source or target')
+    );
     return null;
   }
 
@@ -515,8 +596,15 @@ function parseEdgeLine(
   const pipeIdx = rest.indexOf('|');
   if (pipeIdx >= 0) {
     const metaSegment = rest.slice(pipeIdx + 1).trim();
-    const parsed = parseNodeMetadata(metaSegment, aliasMap, lineNum, diagnostics);
-    status = parsed.hadStatusWord ? (parsed.status ?? null) : (parsed.status ?? 'na');
+    const parsed = parseNodeMetadata(
+      metaSegment,
+      aliasMap,
+      lineNum,
+      diagnostics
+    );
+    status = parsed.hadStatusWord
+      ? (parsed.status ?? null)
+      : (parsed.status ?? 'na');
     metadata = parsed.metadata;
     rest = rest.slice(0, pipeIdx).trim();
   }

package/src/kanban/parser.ts

@@ -26,12 +26,9 @@ const COLUMN_RE = /^\[(.+?)\](?:\s*\(([^)]+)\))?\s*(?:\|\s*(.+))?$/;
 const LEGACY_COLUMN_RE = /^==\s+(.+?)\s*(?:\[wip:\s*(\d+)\])?\s*==$/;
 
 /** Known kanban options (key-value). */
-const KNOWN_OPTIONS = new Set([
-  'color-off', 'hide',
-]);
+const KNOWN_OPTIONS = new Set(['hide']);
 /** Known kanban boolean options (bare keyword = on). */
-const KNOWN_BOOLEANS = new Set<string>([
-]);
+const KNOWN_BOOLEANS = new Set<string>(['no-auto-color']);
 
 // ============================================================
 // Parser
@@ -125,15 +122,11 @@ export function parseKanban(
       }
     }
 
-    // Tag group heading — `tag: Name` (new) or `## Name` (deprecated)
-    // Must be checked BEFORE OPTION_RE to prevent `tag: Rank` being swallowed as option
+    // Tag group heading — `tag Name`
+    // Must be checked BEFORE OPTION_RE to prevent `tag Rank` being swallowed as option
     if (!contentStarted) {
       const tagBlockMatch = matchTagBlockHeading(trimmed);
       if (tagBlockMatch) {
-        if (tagBlockMatch.deprecated) {
-          result.diagnostics.push(makeDgmoError(lineNumber, `'## ${tagBlockMatch.name}' is no longer supported — use 'tag: ${tagBlockMatch.name}' instead`));
-          continue;
-        }
         currentTagGroup = {
           name: tagBlockMatch.name,
           alias: tagBlockMatch.alias,
@@ -141,7 +134,10 @@ export function parseKanban(
           lineNumber,
         };
         if (tagBlockMatch.alias) {
-          aliasMap.set(tagBlockMatch.alias.toLowerCase(), tagBlockMatch.name.toLowerCase());
+          aliasMap.set(
+            tagBlockMatch.alias.toLowerCase(),
+            tagBlockMatch.name.toLowerCase()
+          );
         }
         result.tagGroups.push(currentTagGroup);
         continue;
@@ -160,7 +156,10 @@ export function parseKanban(
         }
       }
       // Bare boolean option (single keyword, no value)
-      if (KNOWN_BOOLEANS.has(trimmed.toLowerCase()) && !COLUMN_RE.test(trimmed)) {
+      if (
+        KNOWN_BOOLEANS.has(trimmed.toLowerCase()) &&
+        !COLUMN_RE.test(trimmed)
+      ) {
         result.options[trimmed.toLowerCase()] = 'on';
         continue;
       }
@@ -202,7 +201,12 @@ export function parseKanban(
     if (LEGACY_COLUMN_RE.test(trimmed)) {
       const legacyMatch = trimmed.match(LEGACY_COLUMN_RE)!;
       const name = legacyMatch[1].replace(/\s*\(.*\)\s*$/, '').trim();
-      result.diagnostics.push(makeDgmoError(lineNumber, `'== ${name} ==' is no longer supported. Use '[${name}]' instead`));
+      result.diagnostics.push(
+        makeDgmoError(
+          lineNumber,
+          `'== ${name} ==' is no longer supported. Use '[${name}]' instead`
+        )
+      );
       continue;
     }
 
@@ -227,7 +231,7 @@ export function parseKanban(
       columnCounter++;
       const colName = columnMatch[1].trim();
       const colColor = columnMatch[2]
-        ? resolveColor(columnMatch[2].trim(), palette) ?? undefined
+        ? (resolveColor(columnMatch[2].trim(), palette) ?? undefined)
         : undefined;
 
       // Parse pipe metadata (e.g., "| wip: 3, t: Sprint1")
@@ -236,7 +240,10 @@ export function parseKanban(
       const pipeStr = columnMatch[3];
       if (pipeStr) {
         const pipeSegments = ['', pipeStr];
-        Object.assign(columnMetadata, parsePipeMetadata(pipeSegments, aliasMap));
+        Object.assign(
+          columnMetadata,
+          parsePipeMetadata(pipeSegments, aliasMap)
+        );
         // Extract wip from metadata
         if (columnMetadata.wip) {
           const wipVal = parseInt(columnMetadata.wip, 10);
@@ -303,8 +310,8 @@ export function parseKanban(
       continue;
     }
 
-    // Un-indented non-column line in content phase — could be stray text
-    // For permissiveness, skip silently
+    // Un-indented non-column line in content phase — stray text
+    warn(lineNumber, `Unexpected line: '${trimmed}'.`);
   }
 
   // Finalize last card's endLineNumber
@@ -332,7 +339,8 @@ export function parseKanban(
   for (const col of result.columns) {
     for (const card of col.cards) {
       for (const [tagKey, tagValue] of Object.entries(card.tags)) {
-        const groupKey = aliasMap.get(tagKey.toLowerCase()) ?? tagKey.toLowerCase();
+        const groupKey =
+          aliasMap.get(tagKey.toLowerCase()) ?? tagKey.toLowerCase();
         const validValues = tagValueSets.get(groupKey);
         if (validValues && !validValues.has(tagValue.toLowerCase())) {
           const entries = result.tagGroups