@diagrammo/dgmo 0.8.1 → 0.8.3

package/src/echarts.ts

@@ -82,6 +82,7 @@ export interface ParsedExtendedChart {
   ylabelLineNumber?: number;
   sizelabel?: string;
   showLabels?: boolean;
+  shade?: boolean;
   categoryColors?: Record<string, string>;
   categoryLineNumbers?: Record<string, number>;
   nodeColors?: Record<string, string>;
@@ -126,8 +127,8 @@ const VALID_EXTENDED_TYPES = new Set<ExtendedChartType>([
 
 /** Known option keywords for the extended chart parser. */
 const KNOWN_EXTENDED_OPTIONS = new Set([
-  'chart', 'title', 'series', 'xlabel', 'ylabel', 'sizelabel', 'labels',
-  'columns', 'rows', 'x',
+  'chart', 'title', 'series', 'xlabel', 'ylabel', 'sizelabel',
+  'no-labels', 'columns', 'rows', 'x',
 ]);
 
 /**
@@ -140,7 +141,7 @@ function parseScatterRow(
   currentCategory: string,
   lineNumber: number,
 ): ParsedScatterPoint | null {
-  const dataRow = parseDataRowValues(line);
+  const dataRow = parseDataRowValues(line, { multiValue: true });
   if (!dataRow || dataRow.values.length < 2) return null;
   const { label: rawLabel, color: pointColor } = extractColor(dataRow.label, palette);
   return {
@@ -376,14 +377,11 @@ export function parseExtendedChart(
       if (firstToken === 'ylabel') { result.ylabel = value; result.ylabelLineNumber = lineNumber; continue; }
       if (firstToken === 'sizelabel') { result.sizelabel = value; continue; }
 
-      if (firstToken === 'labels') {
-        result.showLabels = value.toLowerCase() === 'on' || value.toLowerCase() === 'true';
-        continue;
-      }
-
       if (firstToken === 'columns') {
         if (value) {
-          result.columns = value.split(',').map((s) => s.trim());
+          result.columns = value.includes(',')
+            ? value.split(',').map((s) => s.trim())
+            : value.split(/\s+/);
         } else {
           const collected = collectIndentedValues(lines, i);
           i = collected.newIndex;
@@ -394,7 +392,9 @@ export function parseExtendedChart(
 
       if (firstToken === 'rows') {
         if (value) {
-          result.rows = value.split(',').map((s) => s.trim());
+          result.rows = value.includes(',')
+            ? value.split(',').map((s) => s.trim())
+            : value.split(/\s+/);
         } else {
           const collected = collectIndentedValues(lines, i);
           i = collected.newIndex;
@@ -415,6 +415,10 @@ export function parseExtendedChart(
       }
     }
 
+    // Bare boolean options
+    if (firstToken === 'no-labels') { result.showLabels = false; continue; }
+    if (firstToken === 'shade') { result.shade = true; continue; }
+
     // Bare keyword options (no value)
     if (firstToken === 'series' && spaceIdx === -1) {
       const parsed = parseSeriesNames('', lines, i, palette);
@@ -472,9 +476,9 @@ export function parseExtendedChart(
       }
     }
 
-    // Heatmap data row: "RowLabel val1, val2, val3, ..."
+    // Heatmap data row: "RowLabel val1, val2, val3, ..." or "RowLabel val1 val2 val3"
     if (result.type === 'heatmap') {
-      const dataRow = parseDataRowValues(trimmed);
+      const dataRow = parseDataRowValues(trimmed, { multiValue: true });
       if (dataRow && dataRow.values.length > 0) {
         if (!result.heatmapRows) result.heatmapRows = [];
         result.heatmapRows.push({ label: dataRow.label, values: dataRow.values, lineNumber });
@@ -918,6 +922,12 @@ function buildFunctionOption(
       itemStyle: {
         color: fnColor,
       },
+      ...(parsed.shade && {
+        areaStyle: {
+          color: fnColor,
+          opacity: 0.15,
+        },
+      }),
       emphasis: EMPHASIS_SELF,
     };
   });
@@ -1287,7 +1297,7 @@ function buildScatterOption(
   const hasCategories = points.some((p) => p.category !== undefined);
   const hasSize = points.some((p) => p.size !== undefined);
 
-  const showLabels = parsed.showLabels ?? false;
+  const showLabels = parsed.showLabels ?? true;
   const labelFontSize = 11;
 
   // When showLabels is on, we render labels ourselves via graphic — disable ECharts labels
@@ -2165,13 +2175,23 @@ function buildAreaOption(
 
 // ── Segment label formatter ──────────────────────────────────
 
-function segmentLabelFormatter(mode: ParsedChart['labels']): string {
-  switch (mode) {
-    case 'name':    return '{b}';
-    case 'value':   return '{b} — {c}';
-    case 'percent': return '{b} — {d}%';
-    default:        return '{b} — {c} ({d}%)';
-  }
+function segmentLabelFormatter(parsed: ParsedChart): string {
+  const showName = !parsed.noLabelName;
+  const showValue = !parsed.noLabelValue;
+  const showPercent = !parsed.noLabelPercent;
+
+  const parts: string[] = [];
+  if (showName) parts.push('{b}');
+  if (showValue) parts.push('{c}');
+  if (showPercent) parts.push('{d}%');
+
+  if (parts.length === 0) return '{b}'; // fallback: always show name
+  if (parts.length === 1) return parts[0];
+
+  // Name is joined with " — ", value+percent are grouped with parens when all three
+  if (showName && showValue && showPercent) return '{b} — {c} ({d}%)';
+  if (showName) return '{b} — ' + parts.slice(1).join(' ');
+  return parts.join(' ');
 }
 
 // ── Pie / Doughnut ───────────────────────────────────────────
@@ -2210,7 +2230,7 @@ function buildPieOption(
         data,
         label: {
           position: 'outside',
-          formatter: segmentLabelFormatter(parsed.labels),
+          formatter: segmentLabelFormatter(parsed),
           color: textColor,
           fontFamily: FONT_FAMILY,
         },
@@ -2329,7 +2349,7 @@ function buildPolarAreaOption(
         data,
         label: {
           position: 'outside',
-          formatter: segmentLabelFormatter(parsed.labels),
+          formatter: segmentLabelFormatter(parsed),
           color: textColor,
           fontFamily: FONT_FAMILY,
         },

package/src/er/parser.ts

@@ -243,15 +243,10 @@ export function parseERDiagram(
       firstLineParsed = true;
     }
 
-    // Tag group heading — `tag Name` or deprecated `## Name`
+    // Tag group heading — `tag Name`
     if (!contentStarted && indent === 0) {
       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,
@@ -337,19 +332,16 @@ export function parseERDiagram(
     currentTable = null;
     contentStarted = true;
 
-    // Try relationship
+    // Reject top-level relationships — must be indented under source table
     const rel = parseRelationship(trimmed, lineNumber, pushError);
     if (rel) {
-      getOrCreateTable(rel.source, lineNumber);
-      getOrCreateTable(rel.target, lineNumber);
-
-      result.relationships.push({
-        source: tableId(rel.source),
-        target: tableId(rel.target),
-        cardinality: { from: rel.from, to: rel.to },
-        ...(rel.label && { label: rel.label }),
-        lineNumber,
-      });
+      result.diagnostics.push(
+        makeDgmoError(
+          lineNumber,
+          `Relationship "${rel.source} → ${rel.target}" must be indented under the source table "${rel.source}"`,
+          'warning',
+        ),
+      );
       continue;
     }
 

package/src/gantt/parser.ts

@@ -6,7 +6,7 @@ import { makeDgmoError, formatDgmoError, suggest } from '../diagnostics';
 import type { DgmoError } from '../diagnostics';
 import type { TagGroup, TagEntry } from '../utils/tag-groups';
 import { matchTagBlockHeading } from '../utils/tag-groups';
-import { measureIndent, extractColor, parsePipeMetadata, MULTIPLE_PIPE_WARNING, parseFirstLine, prescanOptions, GROUP_HASH_RE } from '../utils/parsing';
+import { measureIndent, extractColor, parsePipeMetadata, MULTIPLE_PIPE_ERROR, parseFirstLine, prescanOptions } from '../utils/parsing';
 import { parseOffset } from '../utils/duration';
 import type { PaletteColors } from '../palettes';
 import { resolveColor } from '../colors';
@@ -37,7 +37,7 @@ const DURATION_RE = /^(\d+(?:\.\d+)?)(min|bd|d|w|m|q|y|h)(\?)?\s+(.+)$/;
 const EXPLICIT_DATE_RE = /^(\d{4}-\d{2}-\d{2}(?: \d{2}:\d{2})?)\s+(.+)$/;
 
 /** Timeline migration syntax: `2024-01-15 -> 30d Label` or `2024-01-15 14:30 -> 2h Label` */
-const TIMELINE_DURATION_RE = /^(\d{4}-\d{2}-\d{2}(?: \d{2}:\d{2})?)\s*->\s*(\d+(?:\.\d+)?)(min|bd|d|w|m|q|y|h)(\?)?\s+(.+)$/;
+const TIMELINE_DURATION_RE = /^(\d{4}-\d{2}-\d{2}(?: \d{2}:\d{2})?)\s*(?:->|\u2013>)\s*(\d+(?:\.\d+)?)(min|bd|d|w|m|q|y|h)(\?)?\s+(.+)$/;
 
 /** Group container: `[GroupName]` with optional pipe metadata */
 const GROUP_RE = /^\[(.+?)\]\s*(.*)$/;
@@ -49,7 +49,7 @@ const DEPENDENCY_RE = /^(?:-(.+?))?->\s*(.+)$/;
 const COMMENT_RE = /^\/\//;
 
 /** Era: `era YYYY[-MM[-DD[ HH:MM]]] -> YYYY[-MM[-DD[ HH:MM]]] Label (color?)` */
-const ERA_RE = /^era\s+(\d{4}(?:-\d{2}(?:-\d{2}(?: \d{2}:\d{2})?)?)?)\s*->\s*(\d{4}(?:-\d{2}(?:-\d{2}(?: \d{2}:\d{2})?)?)?)\s+(.+)$/i;
+const ERA_RE = /^era\s+(\d{4}(?:-\d{2}(?:-\d{2}(?: \d{2}:\d{2})?)?)?)\s*(?:->|\u2013>)\s*(\d{4}(?:-\d{2}(?:-\d{2}(?: \d{2}:\d{2})?)?)?)\s+(.+)$/i;
 
 /** Marker: `marker YYYY[-MM[-DD[ HH:MM]]] Label (color?)` */
 const MARKER_RE = /^marker\s+(\d{4}(?:-\d{2}(?:-\d{2}(?: \d{2}:\d{2})?)?)?)\s+(.+)$/i;
@@ -58,11 +58,17 @@ const MARKER_RE = /^marker\s+(\d{4}(?:-\d{2}(?:-\d{2}(?: \d{2}:\d{2})?)?)?)\s+(.
 const HOLIDAY_DATE_RE = /^(\d{4}-\d{2}-\d{2})\s+(.+)$/;
 
 /** Holiday range: `2024-12-24 -> 2024-12-31 Label` */
-const HOLIDAY_RANGE_RE = /^(\d{4}-\d{2}-\d{2})\s*->\s*(\d{4}-\d{2}-\d{2})\s+(.+)$/;
+const HOLIDAY_RANGE_RE = /^(\d{4}-\d{2}-\d{2})\s*(?:->|\u2013>)\s*(\d{4}-\d{2}-\d{2})\s+(.+)$/;
 
 /** Workweek override: `workweek sun-thu` */
 const WORKWEEK_RE = /^workweek\s+(.+)$/i;
 
+/** Era entry (inside era block, no `era` prefix): `YYYY[-MM[-DD[ HH:MM]]] -> YYYY[-MM[-DD[ HH:MM]]] Label` */
+const ERA_ENTRY_RE = /^(\d{4}(?:-\d{2}(?:-\d{2}(?: \d{2}:\d{2})?)?)?)\s*(?:->|\u2013>)\s*(\d{4}(?:-\d{2}(?:-\d{2}(?: \d{2}:\d{2})?)?)?)\s+(.+)$/;
+
+/** Marker entry (inside marker block, no `marker` prefix): `YYYY[-MM[-DD[ HH:MM]]] Label` */
+const MARKER_ENTRY_RE = /^(\d{4}(?:-\d{2}(?:-\d{2}(?: \d{2}:\d{2})?)?)?)\s+(.+)$/;
+
 // Valid weekday names
 const WEEKDAY_MAP: Record<string, Weekday> = {
   mon: 'mon', tue: 'tue', wed: 'wed', thu: 'thu', fri: 'fri', sat: 'sat', sun: 'sun',
@@ -156,6 +162,10 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
   let inTagBlock = false;
   let currentTagGroup: TagGroup | null = null;
   let tagBlockIndent = 0;
+  let inEraBlock = false;
+  let eraBlockIndent = 0;
+  let inMarkerBlock = false;
+  let markerBlockIndent = 0;
   let lastTaskNode: (GanttNode & { kind: 'task' }) | null = null;
   let taskIdCounter = 0;
   const seriesColors = palette ? getSeriesColors(palette) : [];
@@ -170,10 +180,16 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
 
     // Skip empty lines
     if (!line) {
-      // Empty line ends holidays/tag blocks only if at root indent
+      // Empty line ends holidays/tag/era/marker blocks only if at root indent
       if (inHolidaysBlock && indent <= holidaysBlockIndent) {
         inHolidaysBlock = false;
       }
+      if (inEraBlock && indent <= eraBlockIndent) {
+        inEraBlock = false;
+      }
+      if (inMarkerBlock && indent <= markerBlockIndent) {
+        inMarkerBlock = false;
+      }
       if (inTagBlock && indent <= tagBlockIndent) {
         inTagBlock = false;
         if (currentTagGroup) {
@@ -249,6 +265,57 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
       }
     }
 
+    // ── Era block entries ─────────────────────────────────
+
+    if (inEraBlock) {
+      if (indent <= eraBlockIndent) {
+        inEraBlock = false;
+        // fall through to process this line normally
+      } else {
+        if (COMMENT_RE.test(line)) continue;
+        const eraEntryMatch = line.match(ERA_ENTRY_RE);
+        if (eraEntryMatch) {
+          const eraLabelRaw = eraEntryMatch[3].trim();
+          const eraExtracted = extractColor(eraLabelRaw, palette);
+          result.eras.push({
+            startDate: eraEntryMatch[1],
+            endDate: eraEntryMatch[2],
+            label: eraExtracted.label,
+            color: eraExtracted.color || null,
+            lineNumber,
+          });
+        } else {
+          warn(lineNumber, `Unrecognized era entry: "${line}"`);
+        }
+        continue;
+      }
+    }
+
+    // ── Marker block entries ─────────────────────────────
+
+    if (inMarkerBlock) {
+      if (indent <= markerBlockIndent) {
+        inMarkerBlock = false;
+        // fall through to process this line normally
+      } else {
+        if (COMMENT_RE.test(line)) continue;
+        const markerEntryMatch = line.match(MARKER_ENTRY_RE);
+        if (markerEntryMatch) {
+          const markerLabelRaw = markerEntryMatch[2].trim();
+          const markerExtracted = extractColor(markerLabelRaw, palette);
+          result.markers.push({
+            date: markerEntryMatch[1],
+            label: markerExtracted.label,
+            color: markerExtracted.color || null,
+            lineNumber,
+          });
+        } else {
+          warn(lineNumber, `Unrecognized marker entry: "${line}"`);
+        }
+        continue;
+      }
+    }
+
     // ── Tag block entries ─────────────────────────────────
 
     if (inTagBlock && currentTagGroup) {
@@ -302,7 +369,7 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
         let offset: Offset | undefined;
 
         if (depParts.length > 1) {
-          const meta = parsePipeMetadata(['', ...depParts.slice(1)], aliasMap, () => warn(lineNumber, MULTIPLE_PIPE_WARNING));
+          const meta = parsePipeMetadata(['', ...depParts.slice(1)], aliasMap, () => warn(lineNumber, MULTIPLE_PIPE_ERROR));
           if (meta.lag || meta.lead) {
             const key = meta.lag ? 'lag' : 'lead';
             softError(lineNumber, `"${key}" is no longer supported — use "offset: ${meta[key]}" instead.${key === 'lead' ? ' Negate the value for lead behavior: "offset: -...".' : ''}`);
@@ -369,10 +436,6 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
     // Tag block heading
     const tagMatch = matchTagBlockHeading(line);
     if (tagMatch) {
-      if (tagMatch.deprecated) {
-        softError(lineNumber, `'## ${tagMatch.name}' is no longer supported — use 'tag ${tagMatch.name}' instead`);
-        continue;
-      }
       inTagBlock = true;
       tagBlockIndent = indent;
       inHeaderBlock = false;
@@ -388,7 +451,28 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
       continue;
     }
 
-    // Era
+    // Top-level workweek (outside holiday block)
+    const topWorkweekMatch = line.match(WORKWEEK_RE);
+    if (topWorkweekMatch) {
+      const days = parseWorkweek(topWorkweekMatch[1].trim());
+      if (days) {
+        result.holidays.workweek = days;
+      } else {
+        warn(lineNumber, `Invalid workweek format: "${topWorkweekMatch[1]}". Use day range like "sun-thu" or comma-separated days.`);
+      }
+      inHeaderBlock = false;
+      continue;
+    }
+
+    // Era block: bare `era` keyword starts a block
+    if (line.toLowerCase() === 'era') {
+      inEraBlock = true;
+      eraBlockIndent = indent;
+      inHeaderBlock = false;
+      continue;
+    }
+
+    // Era (inline)
     const eraMatch = line.match(ERA_RE);
     if (eraMatch) {
       const eraLabelRaw = eraMatch[3].trim();
@@ -404,7 +488,15 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
       continue;
     }
 
-    // Marker
+    // Marker block: bare `marker` keyword starts a block
+    if (line.toLowerCase() === 'marker') {
+      inMarkerBlock = true;
+      markerBlockIndent = indent;
+      inHeaderBlock = false;
+      continue;
+    }
+
+    // Marker (inline)
     const markerMatch = line.match(MARKER_RE);
     if (markerMatch) {
       const markerLabelRaw = markerMatch[2].trim();
@@ -472,9 +564,6 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
           result.options.title = value;
           result.options.titleLineNumber = lineNumber;
           break;
-        case 'orientation':
-          warn(lineNumber, `'orientation' is not supported for gantt charts`);
-          break;
         case 'today-marker':
           if (/^\d{4}-\d{2}-\d{2}(?: \d{2}:\d{2})?$/.test(value)) {
             result.options.todayMarker = value;
@@ -506,28 +595,7 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
 
     inHeaderBlock = false;
 
-    // ── `# Group` alternate syntax ──────────────────────────
 
-    const hashGroupMatch = line.match(GROUP_HASH_RE);
-    if (hashGroupMatch) {
-      const nameExtracted = extractColor(hashGroupMatch[1], palette);
-      const group: GanttGroup = {
-        name: nameExtracted.label,
-        color: nameExtracted.color || null,
-        metadata: {},
-        lineNumber,
-        children: [],
-      };
-      const groupNode: GanttNode = { kind: 'group', ...group };
-      currentContainer().push(groupNode);
-      blockStack.push({
-        node: groupNode as GanttGroup,
-        indent,
-        containerType: 'group',
-      });
-      lastTaskNode = null;
-      continue;
-    }
 
     // ── Parallel block ────────────────────────────────────
 
@@ -560,7 +628,7 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
       let metadata: Record<string, string> = {};
       let color: string | null = null;
 
-      const pipeWarn = () => warn(lineNumber, MULTIPLE_PIPE_WARNING);
+      const pipeWarn = () => warn(lineNumber, MULTIPLE_PIPE_ERROR);
       if (segments.length > 0 && segments[0].trim()) {
         // Check if first segment after brackets is pipe metadata
         metadata = parsePipeMetadata(['', ...segments], aliasMap, pipeWarn);
@@ -662,7 +730,7 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
       let offset: Offset | undefined;
 
       if (depParts.length > 1) {
-        const meta = parsePipeMetadata(['', ...depParts.slice(1)], aliasMap, () => warn(lineNumber, MULTIPLE_PIPE_WARNING));
+        const meta = parsePipeMetadata(['', ...depParts.slice(1)], aliasMap, () => warn(lineNumber, MULTIPLE_PIPE_ERROR));
         if (meta.lag || meta.lead) {
           const key = meta.lag ? 'lag' : 'lead';
           softError(lineNumber, `"${key}" is no longer supported — use "offset: ${meta[key]}" instead.${key === 'lead' ? ' Negate the value for lead behavior: "offset: -...".' : ''}`);
@@ -726,7 +794,7 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
 
     // Parse pipe metadata
     const metadata = segments.length > 1
-      ? parsePipeMetadata(segments, aliasMap, () => warn(ln, MULTIPLE_PIPE_WARNING))
+      ? parsePipeMetadata(segments, aliasMap, () => warn(ln, MULTIPLE_PIPE_ERROR))
       : {};
 
     // Extract progress from metadata or shorthand
@@ -833,7 +901,7 @@ function parseWorkweek(s: string): Weekday[] | null {
 // ── Known option keys ─────────────────────────────────────
 
 const KNOWN_OPTIONS = new Set([
-  'start', 'title', 'orientation', 'today-marker',
+  'start', 'title', 'today-marker',
   'critical-path', 'dependencies', 'chart', 'sort',
 ]);
 

package/src/graph/flowchart-parser.ts

@@ -4,19 +4,15 @@ import { makeDgmoError, formatDgmoError, suggest } from '../diagnostics';
 import {
   measureIndent,
   extractColor,
-  normalizeDirection,
   inferArrowColor,
   parseFirstLine,
   OPTION_NOCOLON_RE,
-  GROUP_HASH_RE,
-  DOUBLE_HASH_RE,
   ALL_CHART_TYPES,
 } from '../utils/parsing';
 import type {
   ParsedGraph,
   GraphNode,
   GraphEdge,
-  GraphGroup,
   GraphShape,
   GraphDirection,
 } from './types';
@@ -218,7 +214,7 @@ export function parseFlowchart(
   const lines = content.split('\n');
   const result: ParsedGraph = {
     type: 'flowchart',
-    direction: 'TB',
+    direction: 'LR',
     nodes: [],
     edges: [],
     options: {},
@@ -238,11 +234,6 @@ export function parseFlowchart(
   let contentStarted = false;
   let firstLineParsed = false;
 
-  // Group support
-  let currentGroup: GraphGroup | null = null;
-  let groupIndent = -1;
-  const groups: GraphGroup[] = [];
-
   function getOrCreateNode(ref: NodeRef, lineNumber: number): GraphNode {
     const existing = nodeMap.get(ref.id);
     if (existing) return existing;
@@ -253,15 +244,10 @@ export function parseFlowchart(
       shape: ref.shape,
       lineNumber,
       ...(ref.color && { color: ref.color }),
-      ...(currentGroup && { group: currentGroup.id }),
     };
     nodeMap.set(ref.id, node);
     result.nodes.push(node);
 
-    if (currentGroup && !currentGroup.nodeIds.includes(ref.id)) {
-      currentGroup.nodeIds.push(ref.id);
-    }
-
     return node;
   }
 
@@ -418,45 +404,19 @@ export function parseFlowchart(
       }
     }
 
-    // ## group headings — emit helpful error
-    if (DOUBLE_HASH_RE.test(trimmed)) {
-      result.diagnostics.push(
-        makeDgmoError(lineNumber, 'Use `#` for groups \u2014 nesting is done with indentation.', 'error')
-      );
-      continue;
-    }
-
-    // # GroupName — alternate group notation
-    const hashGroupMatch = trimmed.match(GROUP_HASH_RE);
-    if (hashGroupMatch) {
-      const { label, color } = extractColor(hashGroupMatch[1].trim(), palette);
-      currentGroup = {
-        id: `group:${label.toLowerCase()}`,
-        label,
-        nodeIds: [],
-        lineNumber,
-        ...(color && { color }),
-      };
-      groupIndent = indent;
-      groups.push(currentGroup);
-      continue;
-    }
-
     // Options (space-separated, before content)
     if (!contentStarted) {
+      // Bare boolean: direction-tb
+      if (/^direction-tb$/i.test(trimmed)) {
+        result.direction = 'TB';
+        continue;
+      }
+
       const optMatch = trimmed.match(OPTION_NOCOLON_RE);
       if (optMatch && !trimmed.includes('->')) {
         const key = optMatch[1].toLowerCase();
         const value = optMatch[2].trim();
 
-        if (key === 'direction' || key === 'orientation') {
-          const dir = normalizeDirection(value);
-          if (dir) {
-            result.direction = dir;
-          }
-          continue;
-        }
-
         // Boolean: no-color = color off
         if (key === 'no-color') {
           result.options['color'] = 'off';
@@ -469,18 +429,10 @@ export function parseFlowchart(
       }
     }
 
-    // Close current group when indent returns to or below the group level
-    if (currentGroup && indent <= groupIndent) {
-      currentGroup = null;
-      groupIndent = -1;
-    }
-
     // Content line (nodes and edges)
     processContentLine(trimmed, lineNumber, indent);
   }
 
-  if (groups.length > 0) result.groups = groups;
-
   // 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).');

package/src/graph/state-parser.ts

@@ -4,7 +4,6 @@ import { makeDgmoError, formatDgmoError, suggest } from '../diagnostics';
 import {
   measureIndent,
   extractColor,
-  normalizeDirection,
   parseFirstLine,
   OPTION_NOCOLON_RE,
   ALL_CHART_TYPES,
@@ -154,7 +153,7 @@ export function parseState(
   const lines = content.split('\n');
   const result: ParsedGraph = {
     type: 'state',
-    direction: 'TB',
+    direction: 'LR',
     nodes: [],
     edges: [],
     options: {},
@@ -268,19 +267,17 @@ export function parseState(
 
     // Options (space-separated, before content)
     if (!contentStarted) {
+      // Bare boolean: direction-tb
+      if (/^direction-tb$/i.test(trimmed)) {
+        result.direction = 'TB';
+        continue;
+      }
+
       const optMatch = trimmed.match(OPTION_NOCOLON_RE);
       if (optMatch && !trimmed.includes('->')) {
         const key = optMatch[1].toLowerCase();
         const value = optMatch[2].trim();
 
-        if (key === 'direction' || key === 'orientation') {
-          const dir = normalizeDirection(value);
-          if (dir) {
-            result.direction = dir;
-          }
-          continue;
-        }
-
         // Boolean: no-color = color off
         if (key === 'no-color') {
           result.options['color'] = 'off';