@diagrammo/dgmo 0.7.3 → 0.8.0

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 } from '../utils/parsing';
+import { measureIndent, extractColor, parsePipeMetadata, MULTIPLE_PIPE_WARNING, parseFirstLine, prescanOptions, GROUP_HASH_RE } from '../utils/parsing';
 import { parseOffset } from '../utils/duration';
 import type { PaletteColors } from '../palettes';
 import { resolveColor } from '../colors';
@@ -30,44 +30,38 @@ import type {
 
 // ── Regexes ─────────────────────────────────────────────────
 
-/** Duration task: `30d: Label`, `1.5w: Label`, `10bd?: Label` */
-const DURATION_RE = /^(\d+(?:\.\d+)?)(d|bd|w|m|q|y)(\?)?:\s*(.+)$/;
+/** Duration task: `30d Label`, `1.5w Label`, `10bd? Label`, `2h Label`, `90min Label` */
+const DURATION_RE = /^(\d+(?:\.\d+)?)(min|bd|d|w|m|q|y|h)(\?)?\s+(.+)$/;
 
-/** Explicit date task: `2024-01-15: Label` */
-const EXPLICIT_DATE_RE = /^(\d{4}-\d{2}-\d{2}):\s*(.+)$/;
+/** Explicit date task: `2024-01-15 Label` or `2024-01-15 14:30 Label` */
+const EXPLICIT_DATE_RE = /^(\d{4}-\d{2}-\d{2}(?: \d{2}:\d{2})?)\s+(.+)$/;
 
-/** Timeline migration syntax: `2024-01-15 -> 30d: Label` */
-const TIMELINE_DURATION_RE = /^(\d{4}-\d{2}-\d{2})\s*->\s*(\d+(?:\.\d+)?)(d|bd|w|m|q|y)(\?)?:\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+(.+)$/;
 
 /** Group container: `[GroupName]` with optional pipe metadata */
 const GROUP_RE = /^\[(.+?)\]\s*(.*)$/;
 
-/** Dependency: `-> TargetName` with optional pipe metadata */
-const DEPENDENCY_RE = /^->\s*(.+)$/;
+/** Dependency: `-> TargetName` or `-label-> TargetName` with optional pipe metadata */
+const DEPENDENCY_RE = /^(?:-(.+?))?->\s*(.+)$/;
 
 /** Comment line */
 const COMMENT_RE = /^\/\//;
 
-/** Era: `era YYYY[-MM[-DD]] -> YYYY[-MM[-DD]]: Label (color?)` */
-const ERA_RE = /^era\s+(\d{4}(?:-\d{2}(?:-\d{2})?)?)\s*->\s*(\d{4}(?:-\d{2}(?:-\d{2})?)?)\s*:\s*(.+)$/i;
+/** 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;
 
-/** Marker: `marker: YYYY[-MM[-DD]] Label (color?)` */
-const MARKER_RE = /^marker:\s+(\d{4}(?:-\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;
 
-/** Holiday date: `2024-01-15: Label` */
-const HOLIDAY_DATE_RE = /^(\d{4}-\d{2}-\d{2}):\s*(.+)$/;
+/** Holiday date: `2024-01-15 Label` */
+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*(.+)$/;
+/** 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+(.+)$/;
 
-/** Workweek override: `workweek: sun-thu` */
-const WORKWEEK_RE = /^workweek:\s*(.+)$/i;
-
-/** chart: gantt */
-const CHART_TYPE_RE = /^chart\s*:\s*(.+)/i;
-
-/** Option lines */
-const OPTION_RE = /^([a-z][a-z0-9-]*)\s*:\s*(.+)$/i;
+/** Workweek override: `workweek sun-thu` */
+const WORKWEEK_RE = /^workweek\s+(.+)$/i;
 
 // Valid weekday names
 const WEEKDAY_MAP: Record<string, Weekday> = {
@@ -101,10 +95,9 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
       start: null,
       title: null,
       titleLineNumber: null,
-      orientation: 'horizontal',
       todayMarker: 'off',
       criticalPath: false,
-      dependencies: false,
+      dependencies: true,
       sort: 'default',
       defaultSwimlaneGroup: null,
       optionLineNumbers: {},
@@ -193,14 +186,19 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
 
     // ── Chart type ────────────────────────────────────────
 
-    const chartTypeMatch = line.match(CHART_TYPE_RE);
-    if (chartTypeMatch) {
-      const type = chartTypeMatch[1].trim().toLowerCase();
-      if (type !== 'gantt') {
-        return fail(lineNumber, `Expected chart type "gantt", got "${type}"`);
+    if (!seenChartType) {
+      const firstLineResult = parseFirstLine(line);
+      if (firstLineResult) {
+        if (firstLineResult.chartType !== 'gantt') {
+          return fail(lineNumber, `Expected chart type "gantt", got "${firstLineResult.chartType}"`);
+        }
+        seenChartType = true;
+        if (firstLineResult.title) {
+          result.options.title = firstLineResult.title;
+          result.options.titleLineNumber = lineNumber;
+        }
+        continue;
       }
-      seenChartType = true;
-      continue;
     }
 
     // ── Holidays block ────────────────────────────────────
@@ -261,22 +259,18 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
         currentTagGroup = null;
         // fall through to process this line normally
       } else {
-        // Parse tag entry: `Value(color)` or `Value` with optional `default` suffix
+        // Parse tag entry: `Value(color)` or `Value`
+        // First entry is the default (no `default` keyword needed)
         if (COMMENT_RE.test(line)) continue;
-        let entryLine = line;
-        let isDefault = false;
-        if (entryLine.endsWith(' default') || entryLine.endsWith('\tdefault')) {
-          isDefault = true;
-          entryLine = entryLine.replace(/\s+default$/, '').trim();
-        }
-        const extracted = extractColor(entryLine, palette);
+        const extracted = extractColor(line, palette);
         const color = extracted.color || seriesColors[currentTagGroup.entries.length % seriesColors.length] || '#888888';
+        const isFirstEntry = currentTagGroup.entries.length === 0;
         currentTagGroup.entries.push({
           value: extracted.label,
           color,
           lineNumber,
         });
-        if (isDefault) {
+        if (isFirstEntry) {
           currentTagGroup.defaultValue = extracted.label;
         }
         continue;
@@ -302,7 +296,8 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
       // Dependency under a task
       const depMatch = line.match(DEPENDENCY_RE);
       if (depMatch) {
-        const depParts = depMatch[1].split('|');
+        const label = depMatch[1]?.trim() || undefined;
+        const depParts = depMatch[2].split('|');
         const targetName = depParts[0].trim();
         let offset: Offset | undefined;
 
@@ -310,7 +305,7 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
           const meta = parsePipeMetadata(['', ...depParts.slice(1)], aliasMap, () => warn(lineNumber, MULTIPLE_PIPE_WARNING));
           if (meta.lag || meta.lead) {
             const key = meta.lag ? 'lag' : 'lead';
-            softError(lineNumber, `Unknown keyword "${key}". Use "offset: ${meta[key]}" instead.`);
+            softError(lineNumber, `"${key}" is no longer supported — use "offset: ${meta[key]}" instead.${key === 'lead' ? ' Negate the value for lead behavior: "offset: -...".' : ''}`);
           }
           if (meta.offset) {
             const raw = meta.offset;
@@ -327,6 +322,7 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
 
         lastTaskNode.dependencies.push({
           targetName,
+          label,
           offset,
           lineNumber,
         });
@@ -349,7 +345,7 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
 
     // ── Header options ────────────────────────────────────
 
-    if (line.toLowerCase() === 'holidays') {
+    if (line.toLowerCase() === 'holiday' || line.toLowerCase() === 'holidays') {
       inHolidaysBlock = true;
       holidaysBlockIndent = indent;
       inHeaderBlock = false;
@@ -357,9 +353,26 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
       continue;
     }
 
+    // Single-line holiday: `holiday 2024-12-25 Christmas`
+    const holidayInlineMatch = line.match(/^holiday\s+(\d{4}-\d{2}-\d{2})\s+(.+)$/i);
+    if (holidayInlineMatch) {
+      result.holidays.dates.push({
+        date: holidayInlineMatch[1],
+        label: holidayInlineMatch[2].trim(),
+        lineNumber,
+      });
+      result.options.holidaysLineNumber ??= lineNumber;
+      inHeaderBlock = false;
+      continue;
+    }
+
     // 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;
@@ -406,11 +419,49 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
       continue;
     }
 
-    // Options (start, title, orientation, etc.)
-    const optMatch = line.match(OPTION_RE);
-    if (optMatch && isKnownOption(optMatch[1].toLowerCase())) {
-      const key = optMatch[1].toLowerCase();
-      const value = optMatch[2].trim();
+    // Options — space-separated: `start 2024-04-01`, `title My Plan`
+    // Boolean options: bare keyword = on, `no-X` = off
+    const optNoColonMatch = line.match(/^([a-z][a-z0-9-]*)\s+(.+)$/i);
+    const bareKeyword = line.match(/^([a-z][a-z0-9-]*)$/i);
+
+    // Bare boolean keywords
+    if (bareKeyword && KNOWN_BOOLEANS.has(bareKeyword[1].toLowerCase())) {
+      const key = bareKeyword[1].toLowerCase();
+      result.options.optionLineNumbers[key] = lineNumber;
+      switch (key) {
+        case 'critical-path':
+          result.options.criticalPath = true;
+          break;
+        case 'today-marker':
+          result.options.todayMarker = 'on';
+          break;
+      }
+      continue;
+    }
+
+    // Negated booleans: `no-dependencies`, `no-critical-path`
+    if (bareKeyword && bareKeyword[1].toLowerCase().startsWith('no-')) {
+      const base = bareKeyword[1].toLowerCase().substring(3);
+      if (KNOWN_BOOLEANS.has(base)) {
+        result.options.optionLineNumbers[base] = lineNumber;
+        switch (base) {
+          case 'dependencies':
+            result.options.dependencies = false;
+            break;
+          case 'critical-path':
+            result.options.criticalPath = false;
+            break;
+          case 'today-marker':
+            result.options.todayMarker = 'off';
+            break;
+        }
+        continue;
+      }
+    }
+
+    if (optNoColonMatch && isKnownOption(optNoColonMatch[1].toLowerCase())) {
+      const key = optNoColonMatch[1].toLowerCase();
+      const value = optNoColonMatch[2].trim();
       result.options.optionLineNumbers[key] = lineNumber;
 
       switch (key) {
@@ -422,26 +473,21 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
           result.options.titleLineNumber = lineNumber;
           break;
         case 'orientation':
-          if (value === 'horizontal' || value === 'vertical') {
-            result.options.orientation = value;
-          } else {
-            warn(lineNumber, `Invalid orientation: "${value}". Expected "horizontal" or "vertical".`);
-          }
+          warn(lineNumber, `'orientation' is not supported for gantt charts`);
           break;
         case 'today-marker':
-          if (value === 'on' || value === 'off') {
-            result.options.todayMarker = value;
-          } else if (/^\d{4}-\d{2}-\d{2}$/.test(value)) {
+          if (/^\d{4}-\d{2}-\d{2}(?: \d{2}:\d{2})?$/.test(value)) {
             result.options.todayMarker = value;
           } else {
-            warn(lineNumber, `Invalid today-marker value: "${value}". Expected "on", "off", or YYYY-MM-DD.`);
+            warn(lineNumber, `Invalid today-marker value: "${value}". Expected YYYY-MM-DD.`);
           }
           break;
         case 'critical-path':
-          result.options.criticalPath = value === 'on';
+          result.options.criticalPath = true;
           break;
         case 'dependencies':
-          result.options.dependencies = value === 'on';
+          // Boolean with value — but `dependencies` is now default ON, so only `no-dependencies` turns it off
+          result.options.dependencies = true;
           break;
         case 'sort':
           if (value === 'tag' || value.startsWith('tag:')) {
@@ -460,6 +506,29 @@ 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 ────────────────────────────────────
 
     if (line === 'parallel') {
@@ -583,11 +652,12 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
     if (depMatch) {
       // Dependency without a task context is an error
       if (!lastTaskNode) {
-        softError(lineNumber, `Dependency "-> ${depMatch[1]}" must be indented under a task.`);
+        softError(lineNumber, `Dependency "-> ${depMatch[2]}" must be indented under a task.`);
         continue;
       }
       // This happens when the dep is at the same indent as the task
-      const depParts = depMatch[1].split('|');
+      const label = depMatch[1]?.trim() || undefined;
+      const depParts = depMatch[2].split('|');
       const targetName = depParts[0].trim();
       let offset: Offset | undefined;
 
@@ -595,7 +665,7 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
         const meta = parsePipeMetadata(['', ...depParts.slice(1)], aliasMap, () => warn(lineNumber, MULTIPLE_PIPE_WARNING));
         if (meta.lag || meta.lead) {
           const key = meta.lag ? 'lag' : 'lead';
-          warn(lineNumber, `"${key}" is deprecated — use "offset: ${meta[key]}" instead.${key === 'lead' ? ' Negate the value for lead behavior: "offset: -...".' : ''}`);
+          softError(lineNumber, `"${key}" is no longer supported — use "offset: ${meta[key]}" instead.${key === 'lead' ? ' Negate the value for lead behavior: "offset: -...".' : ''}`);
         }
         if (meta.offset) {
           const raw = meta.offset;
@@ -610,13 +680,13 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
         }
       }
 
-      lastTaskNode.dependencies.push({ targetName, offset, lineNumber });
+      lastTaskNode.dependencies.push({ targetName, label, offset, lineNumber });
       continue;
     }
 
     // ── Bare label = parse error ──────────────────────────
 
-    softError(lineNumber, `Expected duration (e.g., "10d: Task"), group brackets (e.g., "[Group]"), or keyword. Got: "${line}"`);
+    softError(lineNumber, `Expected duration (e.g., "10d Task"), group brackets (e.g., "[Group]"), or keyword. Got: "${line}"`);
     continue;
   }
 
@@ -631,7 +701,7 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
 
   // Validate sort: tag requires tag groups
   if (result.options.sort === 'tag' && result.tagGroups.length === 0) {
-    warn(0, 'sort: tag has no effect — no tag groups defined.');
+    warn(0, 'sort tag has no effect — no tag groups defined.');
     result.options.sort = 'default';
   }
 
@@ -677,7 +747,7 @@ export function parseGantt(content: string, palette?: PaletteColors): ParsedGant
     // Reject lag/lead — use offset instead
     if (metadata.lag || metadata.lead) {
       const key = metadata.lag ? 'lag' : 'lead';
-      softError(ln, `Unknown keyword "${key}". Use "offset: ${metadata[key]}" instead.`);
+      softError(ln, `"${key}" is no longer supported — use "offset: ${metadata[key]}" instead.${key === 'lead' ? ' Negate the value for lead behavior: "offset: -...".' : ''}`);
     }
 
     // Extract task-level offset from metadata
@@ -767,6 +837,11 @@ const KNOWN_OPTIONS = new Set([
   'critical-path', 'dependencies', 'chart', 'sort',
 ]);
 
+/** Boolean options that can appear as bare keywords or with `no-` prefix. */
+const KNOWN_BOOLEANS = new Set([
+  'critical-path', 'today-marker', 'dependencies',
+]);
+
 function isKnownOption(key: string): boolean {
   return KNOWN_OPTIONS.has(key);
 }

package/src/gantt/renderer.ts

@@ -14,16 +14,16 @@ import {
   LEGEND_HEIGHT,
   LEGEND_PILL_PAD,
   LEGEND_PILL_FONT_SIZE,
-  LEGEND_PILL_FONT_W,
   LEGEND_CAPSULE_PAD,
   LEGEND_DOT_R,
   LEGEND_ENTRY_FONT_SIZE,
-  LEGEND_ENTRY_FONT_W,
   LEGEND_ENTRY_DOT_GAP,
   LEGEND_ENTRY_TRAIL,
   LEGEND_GROUP_GAP,
   LEGEND_ICON_W,
+  measureLegendText,
 } from '../utils/legend-constants';
+import { TITLE_FONT_SIZE, TITLE_FONT_WEIGHT, TITLE_Y } from '../utils/title-constants';
 import type { PaletteColors } from '../palettes';
 import type { D3ExportDimensions } from '../d3';
 import type { ResolvedSchedule, ResolvedTask, ResolvedGroup, Weekday } from './types';
@@ -258,8 +258,8 @@ export function renderGantt(
     .select(container)
     .append('svg')
     .attr('viewBox', `0 0 ${containerWidth} ${outerHeight}`)
-    .attr('width', containerWidth)
-    .attr('height', outerHeight)
+    .attr('width', exportDims ? containerWidth : '100%')
+    .attr('preserveAspectRatio', 'xMidYMin meet')
     .attr('font-family', FONT_FAMILY)
     .style('overflow', 'visible');
 
@@ -273,10 +273,10 @@ export function renderGantt(
     svg
       .append('text')
       .attr('x', containerWidth / 2)
-      .attr('y', 30)
+      .attr('y', TITLE_Y)
       .attr('text-anchor', 'middle')
-      .attr('font-size', '20px')
-      .attr('font-weight', '700')
+      .attr('font-size', TITLE_FONT_SIZE)
+      .attr('font-weight', TITLE_FONT_WEIGHT)
       .attr('fill', palette.text)
       .text(title);
   }
@@ -1251,6 +1251,38 @@ function renderDependencyArrows(
         .attr('points', arrowheadPoints(tx, ty, headSize, angle))
         .attr('fill', arrowColor)
         .attr('opacity', 0.5);
+
+      // Label at midpoint of the dependency path
+      if (dep.label) {
+        const midX = (sx + tx) / 2;
+        const midY = (sy + ty) / 2;
+        // Background rect for readability
+        const labelEl = g.append('text')
+          .attr('class', 'gantt-dep-label')
+          .attr('data-dep-from', rt.task.id)
+          .attr('data-dep-to', targetTask.task.id)
+          .attr('x', midX)
+          .attr('y', midY - 4)
+          .attr('text-anchor', 'middle')
+          .attr('font-size', 10)
+          .attr('font-family', FONT_FAMILY)
+          .attr('fill', palette.text)
+          .attr('opacity', 0.7)
+          .text(dep.label);
+        // Insert a background rect behind the text for contrast
+        const bbox = (labelEl.node() as SVGTextElement)?.getBBox?.();
+        if (bbox && bbox.width > 0) {
+          g.insert('rect', '.gantt-dep-label:last-of-type')
+            .attr('class', 'gantt-dep-label-bg')
+            .attr('x', bbox.x - 2)
+            .attr('y', bbox.y - 1)
+            .attr('width', bbox.width + 4)
+            .attr('height', bbox.height + 2)
+            .attr('fill', palette.bg)
+            .attr('opacity', 0.8)
+            .attr('rx', 2);
+        }
+      }
     }
   }
 }
@@ -1407,13 +1439,13 @@ function renderTagLegend(
     const isSwimlane = currentSwimlaneGroup?.toLowerCase() === group.name.toLowerCase();
     const showIcon = !legendViewMode && tagGroups.length > 0;
     const iconReserve = showIcon ? LEGEND_ICON_W : 0;
-    const pillW = group.name.length * LEGEND_PILL_FONT_W + LEGEND_PILL_PAD + iconReserve;
+    const pillW = measureLegendText(group.name, LEGEND_PILL_FONT_SIZE) + LEGEND_PILL_PAD + iconReserve;
     let groupW = pillW;
     if (isActive) {
       const entries = filteredEntries.get(group.name.toLowerCase()) ?? group.entries;
       let entriesW = 0;
       for (const entry of entries) {
-        entriesW += LEGEND_DOT_R * 2 + LEGEND_ENTRY_DOT_GAP + entry.value.length * LEGEND_ENTRY_FONT_W + LEGEND_ENTRY_TRAIL;
+        entriesW += LEGEND_DOT_R * 2 + LEGEND_ENTRY_DOT_GAP + measureLegendText(entry.value, LEGEND_ENTRY_FONT_SIZE) + LEGEND_ENTRY_TRAIL;
       }
       groupW = LEGEND_CAPSULE_PAD * 2 + pillW + 4 + entriesW;
     } else if (isSwimlane && !isActive) {
@@ -1427,7 +1459,7 @@ function renderTagLegend(
 
   // Critical Path pill width
   const cpLabel = 'Critical Path';
-  const cpPillW = cpLabel.length * LEGEND_PILL_FONT_W + LEGEND_PILL_PAD;
+  const cpPillW = measureLegendText(cpLabel, LEGEND_PILL_FONT_SIZE) + LEGEND_PILL_PAD;
   if (hasCriticalPath) {
     if (visibleGroups.length > 0) totalW += LEGEND_GROUP_GAP;
     totalW += cpPillW;
@@ -1449,7 +1481,7 @@ function renderTagLegend(
     const isSwimlane = currentSwimlaneGroup?.toLowerCase() === group.name.toLowerCase();
     const showIcon = !legendViewMode && tagGroups.length > 0;
     const iconReserve = showIcon ? LEGEND_ICON_W : 0;
-    const pillW = group.name.length * LEGEND_PILL_FONT_W + LEGEND_PILL_PAD + iconReserve;
+    const pillW = measureLegendText(group.name, LEGEND_PILL_FONT_SIZE) + LEGEND_PILL_PAD + iconReserve;
     const pillH = isActive ? LEGEND_HEIGHT - LEGEND_CAPSULE_PAD * 2 : LEGEND_HEIGHT;
     const groupW = groupWidths[i];
 
@@ -1496,7 +1528,7 @@ function renderTagLegend(
     }
 
     // Pill text (offset to leave room for icon on right)
-    const textW = group.name.length * LEGEND_PILL_FONT_W + LEGEND_PILL_PAD;
+    const textW = measureLegendText(group.name, LEGEND_PILL_FONT_SIZE) + LEGEND_PILL_PAD;
     gEl.append('text')
       .attr('x', pillXOff + textW / 2)
       .attr('y', LEGEND_HEIGHT / 2 + LEGEND_PILL_FONT_SIZE / 2 - 2)
@@ -1589,7 +1621,7 @@ function renderTagLegend(
             }
           });
 
-        ex += LEGEND_DOT_R * 2 + LEGEND_ENTRY_DOT_GAP + entry.value.length * LEGEND_ENTRY_FONT_W + LEGEND_ENTRY_TRAIL;
+        ex += LEGEND_DOT_R * 2 + LEGEND_ENTRY_DOT_GAP + measureLegendText(entry.value, LEGEND_ENTRY_FONT_SIZE) + LEGEND_ENTRY_TRAIL;
       }
     }
 
@@ -2272,7 +2304,11 @@ function diamondPoints(cx: number, cy: number, size: number): string {
 const MONTH_ABBR = ['Jan','Feb','Mar','Apr','May','Jun','Jul','Aug','Sep','Oct','Nov','Dec'];
 
 function formatGanttDate(d: Date): string {
-  return `${MONTH_ABBR[d.getMonth()]} ${d.getDate()}, ${d.getFullYear()}`;
+  const base = `${MONTH_ABBR[d.getMonth()]} ${d.getDate()}, ${d.getFullYear()}`;
+  if (d.getHours() === 0 && d.getMinutes() === 0) return base;
+  const hh = String(d.getHours()).padStart(2, '0');
+  const mm = String(d.getMinutes()).padStart(2, '0');
+  return `${base} ${hh}:${mm}`;
 }
 
 function showGanttDateIndicators(

package/src/gantt/types.ts

@@ -7,8 +7,8 @@ import type { TagGroup } from '../utils/tag-groups';
 
 // ── Duration ────────────────────────────────────────────────
 
-/** Calendar units: d (days), w (weeks), m (months), q (quarters), y (years). bd = business days. */
-export type DurationUnit = 'd' | 'bd' | 'w' | 'm' | 'q' | 'y';
+/** Calendar units: d (days), w (weeks), m (months), q (quarters), y (years), h (hours), min (minutes). bd = business days. */
+export type DurationUnit = 'd' | 'bd' | 'w' | 'm' | 'q' | 'y' | 'h' | 'min';
 
 export interface Duration {
   amount: number;
@@ -24,6 +24,7 @@ export interface Offset {
 
 export interface GanttDependency {
   targetName: string; // raw string from `-> X` or `-> Group.X`
+  label?: string; // optional label from `-label-> X` syntax
   offset?: Offset;
   lineNumber: number;
 }
@@ -109,7 +110,6 @@ export interface GanttOptions {
   start: string | null; // YYYY[-MM[-DD]] or null for relative timeline
   title: string | null;
   titleLineNumber: number | null;
-  orientation: 'horizontal' | 'vertical';
   todayMarker: 'off' | 'on' | string; // 'on' = current date, string = YYYY-MM-DD
   criticalPath: boolean;
   dependencies: boolean;