@diagrammo/dgmo 0.8.23 → 0.8.26

package/src/index.ts

@@ -23,6 +23,28 @@ export type { ParseInArrowLabelResult } from './utils/arrows';
 
 export { render } from './render';
 
+// ============================================================
+// Chart-type registry (single source of truth)
+// ============================================================
+
+export { chartTypes } from './chart-types';
+export type { ChartTypeMeta } from './chart-types';
+
+export {
+  normalize as normalizeChartTypePrompt,
+  matchesContiguously,
+  scoreChartType,
+  confidence as chartTypeConfidence,
+  suggestChartTypes,
+  MIN_PRIMARY_SCORE,
+  AMBIGUITY_THRESHOLD,
+} from './chart-type-scoring';
+export type {
+  ChartTypeScore,
+  Confidence as ChartTypeConfidence,
+  SuggestionResult as ChartTypeSuggestionResult,
+} from './chart-type-scoring';
+
 // ============================================================
 // Router
 // ============================================================
@@ -34,6 +56,8 @@ export {
   isExtendedChartType,
   getAllChartTypes,
   CHART_TYPE_DESCRIPTIONS,
+  chartTypeParsers,
+  knownChartTypeIds,
 } from './dgmo-router';
 export type { RenderCategory } from './dgmo-router';
 
@@ -496,7 +520,6 @@ export {
   applyGroupOrdering,
   groupMessagesBySection,
   buildNoteMessageMap,
-  collectNoteLineNumbers,
 } from './sequence/renderer';
 export type {
   RenderStep,

package/src/infra/parser.ts

@@ -765,7 +765,7 @@ export function parseInfra(content: string): ParsedInfra {
     }
   }
 
-  validateTagGroupNames(result.tagGroups, warn);
+  validateTagGroupNames(result.tagGroups, warn, setError);
 
   return result;
 }

package/src/internal.ts

@@ -10,7 +10,11 @@ export {
   isArchiveColumn,
 } from './kanban/mutations';
 export {
-  groupMessagesBySection,
+  applyGroupOrdering,
+  applyPositionOverrides,
   buildNoteMessageMap,
-  collectNoteLineNumbers,
+  buildRenderSequence,
+  computeActivations,
+  groupMessagesBySection,
 } from './sequence/renderer';
+export { orderArcNodes } from './d3';

package/src/journey-map/layout.ts

@@ -27,7 +27,7 @@ export interface StepLayout {
   color: string;
 }
 
-export interface PhaseLayout {
+interface PhaseLayout {
   x: number;
   y: number;
   width: number;
@@ -88,8 +88,6 @@ export function scoreToColor(score: number, palette: PaletteColors): string {
 // Layout Engine
 // ============================================================
 
-export const COLLAPSED_PHASE_WIDTH = 60;
-
 export function layoutJourneyMap(
   parsed: ParsedJourneyMap,
   palette: PaletteColors,
@@ -121,14 +119,18 @@ export function layoutJourneyMap(
     ? parsed.phases.flatMap((p) => p.steps)
     : parsed.steps;
 
-  // Compute step card heights based on content (matches kanban card sizing)
+  // Compute step card heights based on content (matches kanban card sizing).
+  // Char-width constants MUST match the renderer's wrapText() in renderer.ts
+  // (`fontSize * 0.6`) and the title wrap (`TITLE_CHAR_WIDTH`) — otherwise the
+  // layout reserves too little vertical space and rendered text overflows.
   const annoIconIndent = ANNO_ICON_SIZE + ANNO_ICON_GAP;
   const annoTextW = STEP_CARD_WIDTH - CARD_PADDING_X * 2 - annoIconIndent;
   const descTextWidth = STEP_CARD_WIDTH - CARD_PADDING_X * 2;
-  const charWidth = 4.8; // average char width at FONT_SIZE_META (10px)
+  const FONT_SIZE_META = 10;
+  const charWidth = FONT_SIZE_META * 0.6; // matches renderer wrapText()
 
   const titleTextWidth = STEP_CARD_WIDTH - CARD_PADDING_X * 2;
-  const titleCharWidth = 6.5; // average char width at FONT_SIZE_STEP (12px)
+  const titleCharWidth = 6.5; // matches renderer TITLE_CHAR_WIDTH (FONT_SIZE_STEP 12px)
   const TITLE_LINE_HEIGHT = 16;
 
   const stepHeights = allSteps.map((step) => {

package/src/journey-map/parser.ts

@@ -408,7 +408,11 @@ export function parseJourneyMap(
     return fail(1, 'No phases or steps found');
   }
 
-  validateTagGroupNames(result.tagGroups, warn);
+  validateTagGroupNames(result.tagGroups, warn, (line, msg) => {
+    const diag = makeDgmoError(line, msg);
+    result.diagnostics.push(diag);
+    if (!result.error) result.error = formatDgmoError(diag);
+  });
 
   return result;
 }

package/src/kanban/parser.ts

@@ -373,7 +373,11 @@ export function parseKanban(
     return fail(1, 'No columns found. Use [Column Name] to define columns');
   }
 
-  validateTagGroupNames(result.tagGroups, warn);
+  validateTagGroupNames(result.tagGroups, warn, (line, msg) => {
+    const diag = makeDgmoError(line, msg);
+    result.diagnostics.push(diag);
+    if (!result.error) result.error = formatDgmoError(diag);
+  });
 
   return result;
 }

package/src/org/collapse.ts

@@ -72,10 +72,7 @@ function computeHiddenCounts(
 }
 
 /** Remove children of collapsed nodes on the cloned tree. */
-function pruneCollapsed(
-  node: OrgNode,
-  collapsedIds: Set<string>
-): void {
+function pruneCollapsed(node: OrgNode, collapsedIds: Set<string>): void {
   for (const child of node.children) {
     pruneCollapsed(child, collapsedIds);
   }

package/src/org/parser.ts

@@ -344,7 +344,7 @@ export function parseOrg(content: string, palette?: PaletteColors): ParsedOrg {
     collectAll(result.roots);
 
     validateTagValues(allNodes, result.tagGroups, pushWarning, suggest);
-    validateTagGroupNames(result.tagGroups, pushWarning);
+    validateTagGroupNames(result.tagGroups, pushWarning, pushError);
   }
 
   if (

package/src/org/renderer.ts

@@ -138,14 +138,16 @@ export function renderOrg(
   const titleReserve = fixedTitle ? TITLE_HEIGHT : 0;
 
   // Ancestor breadcrumb trail (focus mode) — rendered inside the scaled group
-  const hasAncestorTrail = !exportDims && ancestorPath && ancestorPath.length > 0;
+  const hasAncestorTrail =
+    !exportDims && ancestorPath && ancestorPath.length > 0;
   const ancestorTrailHeight = hasAncestorTrail
     ? ancestorPath.length * ANCESTOR_ROW_HEIGHT + ANCESTOR_TRAIL_BOTTOM_GAP
     : 0;
 
   // Compute scale to fit diagram in viewport
   const diagramW = layout.width;
-  let diagramH = layout.height + (fixedTitle ? 0 : titleOffset) + ancestorTrailHeight;
+  let diagramH =
+    layout.height + (fixedTitle ? 0 : titleOffset) + ancestorTrailHeight;
   if (fixedLegend) {
     // Remove the legend space from diagram height — legend is rendered separately
     diagramH -= layoutLegendShift;
@@ -594,15 +596,17 @@ export function renderOrg(
       : rootContainer
         ? rootContainer.x + rootContainer.width / 2
         : null;
-    const rootTopY = rootNode ? rootNode.y : rootContainer ? rootContainer.y : null;
+    const rootTopY = rootNode
+      ? rootNode.y
+      : rootContainer
+        ? rootContainer.y
+        : null;
     if (rootCenterX !== null && rootTopY !== null) {
       // Trail connects directly to the top edge of the root node.
       // The last ancestor dot sits ANCESTOR_TRAIL_BOTTOM_GAP above the root.
       const trailBottomY = rootTopY - ANCESTOR_TRAIL_BOTTOM_GAP;
 
-      const trailG = contentG
-        .append('g')
-        .attr('class', 'org-ancestor-trail');
+      const trailG = contentG.append('g').attr('class', 'org-ancestor-trail');
 
       const count = ancestorPath!.length;
 
@@ -631,13 +635,14 @@ export function renderOrg(
         const dotY = dotPositions[i];
 
         // Resolve color from tag groups (same logic as node cards)
-        const resolvedColor = ancestor.color
-          ?? resolveTagColor(
-               ancestor.metadata,
-               parsed.tagGroups,
-               activeTagGroup ?? null,
-               ancestor.isContainer
-             );
+        const resolvedColor =
+          ancestor.color ??
+          resolveTagColor(
+            ancestor.metadata,
+            parsed.tagGroups,
+            activeTagGroup ?? null,
+            ancestor.isContainer
+          );
         const dotColor = resolvedColor ?? palette.textMuted;
 
         const rowG = trailG
@@ -676,14 +681,18 @@ export function renderOrg(
         // Hover effect
         rowG
           .on('mouseenter', function () {
-            d3Selection.select(this).select('circle')
+            d3Selection
+              .select(this)
+              .select('circle')
               .attr('r', ANCESTOR_DOT_R + 1);
             d3Selection.select(this).select('text').attr('fill', palette.text);
           })
           .on('mouseleave', function () {
-            d3Selection.select(this).select('circle')
-              .attr('r', ANCESTOR_DOT_R);
-            d3Selection.select(this).select('text').attr('fill', palette.textMuted);
+            d3Selection.select(this).select('circle').attr('r', ANCESTOR_DOT_R);
+            d3Selection
+              .select(this)
+              .select('text')
+              .attr('fill', palette.textMuted);
           });
       }
     }

package/src/sequence/parser.ts

@@ -26,7 +26,7 @@ import {
 const KNOWN_SEQ_OPTIONS = new Set(['active-tag']);
 
 /** Known sequence-diagram boolean options (bare keyword or `no-` prefix). */
-const KNOWN_SEQ_BOOLEANS = new Set(['activations', 'collapse-notes']);
+const KNOWN_SEQ_BOOLEANS = new Set(['activations']);
 
 /**
  * Participant types that can be declared via "Name is a type" syntax.
@@ -1345,7 +1345,7 @@ export function parseSequenceDgmo(content: string): ParsedSequenceDgmo {
         entities.push({ metadata: g.metadata, lineNumber: g.lineNumber });
     }
     validateTagValues(entities, result.tagGroups, pushWarning, suggest);
-    validateTagGroupNames(result.tagGroups, pushWarning);
+    validateTagGroupNames(result.tagGroups, pushWarning, pushError);
   }
 
   return result;

package/src/sequence/participant-inference.ts

@@ -185,7 +185,6 @@ const PARTICIPANT_RULES: readonly InferenceRule[] = [
   { pattern: /^Admin$/i, type: 'actor' },
   { pattern: /^User$/i, type: 'actor' },
   { pattern: /^Customer$/i, type: 'actor' },
-  { pattern: /^Client$/i, type: 'actor' },
   { pattern: /^Agent$/i, type: 'actor' },
   { pattern: /^Person$/i, type: 'actor' },
   { pattern: /^Buyer$/i, type: 'actor' },