@diagrammo/dgmo 0.8.23 → 0.8.25

package/src/journey-map/layout.ts

@@ -121,14 +121,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' },

package/src/sequence/renderer.ts

@@ -62,8 +62,6 @@ const NOTE_CHAR_W = 6;
 const NOTE_CHARS_PER_LINE = Math.floor(
   (NOTE_MAX_W - NOTE_PAD_H * 2 - NOTE_FOLD) / NOTE_CHAR_W
 );
-const COLLAPSED_NOTE_H = 20;
-const COLLAPSED_NOTE_W = 40;
 const ACTIVATION_WIDTH = 10;
 const SELF_CALL_HEIGHT = 25;
 const SELF_CALL_WIDTH = 30;
@@ -557,13 +555,8 @@ export interface SectionMessageGroup {
 export interface SequenceRenderOptions {
   collapsedSections?: Set<number>; // keyed by section lineNumber
   collapsedGroups?: Set<number>; // keyed by group lineNumber
-  expandedNoteLines?: Set<number>; // keyed by note lineNumber; undefined = all expanded (CLI default)
   exportWidth?: number; // Explicit width for CLI/export rendering (bypasses getBoundingClientRect)
   activeTagGroup?: string | null; // Active tag group name for tag-driven recoloring; null = explicitly none
-  expandAllNotes?: boolean; // Whether the "Expand Notes" toggle is active
-  onExpandAllNotes?: (expand: boolean) => void; // Toggle all notes expanded/collapsed
-  controlsExpanded?: boolean; // Controls group expanded state (managed by React)
-  onToggleControlsExpand?: () => void; // Callback to toggle controls group
 }
 
 /**
@@ -960,15 +953,6 @@ export function renderSequenceDiagram(
   const collapsedGroupIds = collapsed?.collapsedGroupIds ?? new Map();
 
   const collapsedSections = options?.collapsedSections;
-  const expandedNoteLines = options?.expandedNoteLines;
-  const collapseNotesDisabled =
-    parsedOptions['collapse-notes']?.toLowerCase() === 'no';
-  // A note is expanded if: expandedNoteLines is undefined (CLI/export),
-  // collapse-notes: no is set, or the note's lineNumber is in the set.
-  const isNoteExpanded = (note: SequenceNote): boolean =>
-    expandedNoteLines === undefined ||
-    collapseNotesDisabled ||
-    expandedNoteLines.has(note.lineNumber);
 
   const sourceParticipants = collapsed
     ? collapsed.participants
@@ -1232,9 +1216,7 @@ export function renderSequenceDiagram(
           const note = els[j] as SequenceNote;
           const sc = isNoteAfterSelfCall(note);
           const maxW = noteEffectiveMaxW(note.participantId, note.position, sc);
-          const noteH = isNoteExpanded(note)
-            ? computeNoteHeight(note.text, charsForWidth(maxW))
-            : COLLAPSED_NOTE_H;
+          const noteH = computeNoteHeight(note.text, charsForWidth(maxW));
           totalExtent += noteH + NOTE_OFFSET_BELOW;
           j++;
         }
@@ -1500,9 +1482,10 @@ export function renderSequenceDiagram(
               prevNote.position,
               isNoteAfterSelfCall(prevNote)
             );
-            const prevNoteH = isNoteExpanded(prevNote)
-              ? computeNoteHeight(prevNote.text, charsForWidth(prevMaxW))
-              : COLLAPSED_NOTE_H;
+            const prevNoteH = computeNoteHeight(
+              prevNote.text,
+              charsForWidth(prevMaxW)
+            );
             noteTopY = prevNoteY + prevNoteH + NOTE_OFFSET_BELOW;
           } else {
             // First note after a message — use larger offset after self-calls
@@ -1539,9 +1522,7 @@ export function renderSequenceDiagram(
       note.position,
       isNoteAfterSelfCall(note)
     );
-    const noteH = isNoteExpanded(note)
-      ? computeNoteHeight(note.text, charsForWidth(maxW))
-      : COLLAPSED_NOTE_H;
+    const noteH = computeNoteHeight(note.text, charsForWidth(maxW));
     contentBottomY = Math.max(
       contentBottomY,
       noteTopY + noteH + NOTE_TRAILING_GAP
@@ -1736,32 +1717,6 @@ export function renderSequenceDiagram(
     }
   }
 
-  // Collect all note line numbers (for controls group visibility + "all expanded" check)
-  const allNoteLineNumbers: number[] = [];
-  const collectNoteLines = (els: SequenceElement[]): void => {
-    for (const el of els) {
-      if (isSequenceNote(el)) {
-        allNoteLineNumbers.push(el.lineNumber);
-      } else if (isSequenceBlock(el)) {
-        collectNoteLines(el.children);
-        if ('elseChildren' in el) collectNoteLines(el.elseChildren);
-        if ('branches' in el && Array.isArray(el.branches)) {
-          for (const branch of el.branches) {
-            collectNoteLines(branch.children);
-          }
-        }
-      }
-    }
-  };
-  collectNoteLines(elements);
-
-  // Show controls group only in interactive mode (expandedNoteLines defined)
-  // when notes exist and collapse-notes is not disabled
-  const showNotesControl =
-    allNoteLineNumbers.length > 0 &&
-    !collapseNotesDisabled &&
-    expandedNoteLines !== undefined;
-
   const hasTagGroups = parsed.tagGroups.length > 0;
 
   // Build set of collapsed group names for drill-bar rendering
@@ -2632,8 +2587,6 @@ export function renderSequenceDiagram(
     ? mix(palette.surface, palette.bg, 50)
     : mix(palette.bg, palette.surface, 15);
 
-  const collapsedNoteFill = mix(palette.textMuted, palette.bg, 15);
-
   const renderNoteElements = (els: SequenceElement[]): void => {
     for (const el of els) {
       if (isSequenceNote(el)) {
@@ -2642,168 +2595,98 @@ export function renderSequenceDiagram(
         const noteTopY = noteYMap.get(el);
         if (noteTopY === undefined) continue;
 
-        const expanded = isNoteExpanded(el);
         const isRight = el.position === 'right';
+        const afterSelfCall = isNoteAfterSelfCall(el);
+        const maxW = noteEffectiveMaxW(
+          el.participantId,
+          el.position,
+          afterSelfCall
+        );
+        const maxChars = charsForWidth(maxW);
+        const wrappedLines = wrapTextLines(el.text, maxChars);
+        const noteH = wrappedLines.length * NOTE_LINE_H + NOTE_PAD_V * 2;
+        const maxLineLen = Math.max(...wrappedLines.map((l) => l.length));
+        const noteW = Math.min(
+          maxW,
+          Math.max(80, maxLineLen * NOTE_CHAR_W + NOTE_PAD_H * 2 + NOTE_FOLD)
+        );
+        // Shift notes past self-call loopback when applicable
+        const rightOffset =
+          afterSelfCall && isRight
+            ? ACTIVATION_WIDTH / 2 + SELF_CALL_WIDTH + NOTE_GAP
+            : ACTIVATION_WIDTH + NOTE_GAP;
+        const noteX = isRight
+          ? px + rightOffset
+          : px - ACTIVATION_WIDTH - NOTE_GAP - noteW;
+
+        const noteG = svg
+          .append('g')
+          .attr('class', 'note')
+          .attr('data-note-toggle', '')
+          .attr('data-line-number', String(el.lineNumber))
+          .attr('data-line-end', String(el.endLineNumber));
+
+        // Folded-corner path
+        noteG
+          .append('path')
+          .attr(
+            'd',
+            [
+              `M ${noteX} ${noteTopY}`,
+              `L ${noteX + noteW - NOTE_FOLD} ${noteTopY}`,
+              `L ${noteX + noteW} ${noteTopY + NOTE_FOLD}`,
+              `L ${noteX + noteW} ${noteTopY + noteH}`,
+              `L ${noteX} ${noteTopY + noteH}`,
+              'Z',
+            ].join(' ')
+          )
+          .attr('fill', noteFill)
+          .attr('stroke', palette.textMuted)
+          .attr('stroke-width', 0.75)
+          .attr('class', 'note-box');
 
-        if (expanded) {
-          // --- Expanded note: full folded-corner box with wrapped text ---
-          const afterSelfCall = isNoteAfterSelfCall(el);
-          const maxW = noteEffectiveMaxW(
-            el.participantId,
-            el.position,
-            afterSelfCall
-          );
-          const maxChars = charsForWidth(maxW);
-          const wrappedLines = wrapTextLines(el.text, maxChars);
-          const noteH = wrappedLines.length * NOTE_LINE_H + NOTE_PAD_V * 2;
-          const maxLineLen = Math.max(...wrappedLines.map((l) => l.length));
-          const noteW = Math.min(
-            maxW,
-            Math.max(80, maxLineLen * NOTE_CHAR_W + NOTE_PAD_H * 2 + NOTE_FOLD)
-          );
-          // Shift notes past self-call loopback when applicable
-          const rightOffset =
-            afterSelfCall && isRight
-              ? ACTIVATION_WIDTH / 2 + SELF_CALL_WIDTH + NOTE_GAP
-              : ACTIVATION_WIDTH + NOTE_GAP;
-          const noteX = isRight
-            ? px + rightOffset
-            : px - ACTIVATION_WIDTH - NOTE_GAP - noteW;
-
-          const noteG = svg
-            .append('g')
-            .attr('class', 'note')
-            .attr('data-note-toggle', '')
-            .attr('data-line-number', String(el.lineNumber))
-            .attr('data-line-end', String(el.endLineNumber));
-
-          // Folded-corner path
-          noteG
-            .append('path')
-            .attr(
-              'd',
-              [
-                `M ${noteX} ${noteTopY}`,
-                `L ${noteX + noteW - NOTE_FOLD} ${noteTopY}`,
-                `L ${noteX + noteW} ${noteTopY + NOTE_FOLD}`,
-                `L ${noteX + noteW} ${noteTopY + noteH}`,
-                `L ${noteX} ${noteTopY + noteH}`,
-                'Z',
-              ].join(' ')
-            )
-            .attr('fill', noteFill)
-            .attr('stroke', palette.textMuted)
-            .attr('stroke-width', 0.75)
-            .attr('class', 'note-box');
-
-          // Fold triangle
-          noteG
-            .append('path')
-            .attr(
-              'd',
-              [
-                `M ${noteX + noteW - NOTE_FOLD} ${noteTopY}`,
-                `L ${noteX + noteW - NOTE_FOLD} ${noteTopY + NOTE_FOLD}`,
-                `L ${noteX + noteW} ${noteTopY + NOTE_FOLD}`,
-              ].join(' ')
-            )
-            .attr('fill', 'none')
-            .attr('stroke', palette.textMuted)
-            .attr('stroke-width', 0.75)
-            .attr('class', 'note-fold');
-
-          // Render text with inline markdown
-          wrappedLines.forEach((line, li) => {
-            const textY = noteTopY + NOTE_PAD_V + (li + 1) * NOTE_LINE_H - 3;
-            const isBullet = line.startsWith('- ');
-            const bulletIndent = isBullet ? 10 : 0;
-            const displayLine = isBullet ? line.slice(2) : line;
-            const textEl = noteG
+        // Fold triangle
+        noteG
+          .append('path')
+          .attr(
+            'd',
+            [
+              `M ${noteX + noteW - NOTE_FOLD} ${noteTopY}`,
+              `L ${noteX + noteW - NOTE_FOLD} ${noteTopY + NOTE_FOLD}`,
+              `L ${noteX + noteW} ${noteTopY + NOTE_FOLD}`,
+            ].join(' ')
+          )
+          .attr('fill', 'none')
+          .attr('stroke', palette.textMuted)
+          .attr('stroke-width', 0.75)
+          .attr('class', 'note-fold');
+
+        // Render text with inline markdown
+        wrappedLines.forEach((line, li) => {
+          const textY = noteTopY + NOTE_PAD_V + (li + 1) * NOTE_LINE_H - 3;
+          const isBullet = line.startsWith('- ');
+          const bulletIndent = isBullet ? 10 : 0;
+          const displayLine = isBullet ? line.slice(2) : line;
+          const textEl = noteG
+            .append('text')
+            .attr('x', noteX + NOTE_PAD_H + bulletIndent)
+            .attr('y', textY)
+            .attr('fill', palette.text)
+            .attr('font-size', NOTE_FONT_SIZE)
+            .attr('class', 'note-text');
+
+          if (isBullet) {
+            noteG
               .append('text')
-              .attr('x', noteX + NOTE_PAD_H + bulletIndent)
+              .attr('x', noteX + NOTE_PAD_H)
               .attr('y', textY)
               .attr('fill', palette.text)
               .attr('font-size', NOTE_FONT_SIZE)
-              .attr('class', 'note-text');
-
-            if (isBullet) {
-              noteG
-                .append('text')
-                .attr('x', noteX + NOTE_PAD_H)
-                .attr('y', textY)
-                .attr('fill', palette.text)
-                .attr('font-size', NOTE_FONT_SIZE)
-                .text('\u2022');
-            }
-
-            renderInlineText(textEl, displayLine, palette, NOTE_FONT_SIZE);
-          });
-        } else {
-          // --- Collapsed note: compact indicator ---
-          const cFold = 6;
-          const afterSelfCallC = isNoteAfterSelfCall(el);
-          const rightOffsetC =
-            afterSelfCallC && isRight
-              ? ACTIVATION_WIDTH / 2 + SELF_CALL_WIDTH + NOTE_GAP
-              : ACTIVATION_WIDTH + NOTE_GAP;
-          const noteX = isRight
-            ? px + rightOffsetC
-            : px - ACTIVATION_WIDTH - NOTE_GAP - COLLAPSED_NOTE_W;
-
-          const noteG = svg
-            .append('g')
-            .attr('class', 'note note-collapsed')
-            .attr('data-note-toggle', '')
-            .attr('data-line-number', String(el.lineNumber))
-            .attr('data-line-end', String(el.endLineNumber))
-            .style('cursor', 'pointer');
-
-          // Small folded-corner rectangle
-          noteG
-            .append('path')
-            .attr(
-              'd',
-              [
-                `M ${noteX} ${noteTopY}`,
-                `L ${noteX + COLLAPSED_NOTE_W - cFold} ${noteTopY}`,
-                `L ${noteX + COLLAPSED_NOTE_W} ${noteTopY + cFold}`,
-                `L ${noteX + COLLAPSED_NOTE_W} ${noteTopY + COLLAPSED_NOTE_H}`,
-                `L ${noteX} ${noteTopY + COLLAPSED_NOTE_H}`,
-                'Z',
-              ].join(' ')
-            )
-            .attr('fill', collapsedNoteFill)
-            .attr('stroke', palette.border)
-            .attr('stroke-width', 0.75)
-            .attr('class', 'note-box');
-
-          // Fold triangle
-          noteG
-            .append('path')
-            .attr(
-              'd',
-              [
-                `M ${noteX + COLLAPSED_NOTE_W - cFold} ${noteTopY}`,
-                `L ${noteX + COLLAPSED_NOTE_W - cFold} ${noteTopY + cFold}`,
-                `L ${noteX + COLLAPSED_NOTE_W} ${noteTopY + cFold}`,
-              ].join(' ')
-            )
-            .attr('fill', 'none')
-            .attr('stroke', palette.border)
-            .attr('stroke-width', 0.75)
-            .attr('class', 'note-fold');
+              .text('\u2022');
+          }
 
-          // "..." text
-          noteG
-            .append('text')
-            .attr('x', noteX + COLLAPSED_NOTE_W / 2)
-            .attr('y', noteTopY + COLLAPSED_NOTE_H / 2 + 3)
-            .attr('text-anchor', 'middle')
-            .attr('fill', palette.textMuted)
-            .attr('font-size', 9)
-            .attr('class', 'note-text')
-            .text('\u2026');
-        }
+          renderInlineText(textEl, displayLine, palette, NOTE_FONT_SIZE);
+        });
       } else if (isSequenceBlock(el)) {
         renderNoteElements(el.children);
         if (el.elseIfBranches) {
@@ -2822,9 +2705,7 @@ export function renderSequenceDiagram(
 
   // Render legend LAST so it sits on top of all other SVG elements
   // (group boxes, lifelines, participants, etc.) and can receive clicks.
-  if (hasTagGroups || showNotesControl) {
-    const controlsExpanded = options?.controlsExpanded ?? false;
-
+  if (hasTagGroups) {
     const legendY = TOP_MARGIN + titleOffset;
     const resolvedGroups = parsed.tagGroups
       .filter((tg) => tg.entries.length > 0)
@@ -2836,41 +2717,17 @@ export function renderSequenceDiagram(
         })),
       }));
 
-    const allExpanded = showNotesControl && (options?.expandAllNotes ?? false);
-
-    const controlsGroup = showNotesControl
-      ? {
-          toggles: [
-            {
-              id: 'expand-all-notes',
-              type: 'toggle' as const,
-              label: 'Expand Notes',
-              active: allExpanded,
-              onToggle: () => {},
-            },
-          ],
-        }
-      : undefined;
-
     const legendConfig: LegendConfig = {
       groups: resolvedGroups,
       position: { placement: 'top-center', titleRelation: 'below-title' },
       mode: 'fixed',
-      controlsGroup,
     };
     const legendState: LegendState = {
       activeGroup: activeTagGroup ?? null,
-      controlsExpanded,
+      controlsExpanded: false,
     };
 
-    const legendCallbacks: LegendCallbacks = {
-      onControlsExpand: () => {
-        options?.onToggleControlsExpand?.();
-      },
-      onControlsToggle: (_toggleId: string, active: boolean) => {
-        options?.onExpandAllNotes?.(active);
-      },
-    };
+    const legendCallbacks: LegendCallbacks = {};
 
     const legendG = svg
       .append('g')
@@ -2891,7 +2748,7 @@ export function renderSequenceDiagram(
 /**
  * Build a mapping from each note's lineNumber to the lineNumber of its
  * associated message (the last message before the note in document order).
- * Used by the app to expand notes when cursor is on the associated message.
+ * Used by the app to highlight the associated message when cursor is on a note.
  */
 export function buildNoteMessageMap(
   elements: SequenceElement[]
@@ -2924,31 +2781,6 @@ export function buildNoteMessageMap(
   return map;
 }
 
-/**
- * Collect all note line numbers from a sequence diagram's elements.
- * Used by the app to compute the "expand all" set.
- */
-export function collectNoteLineNumbers(elements: SequenceElement[]): number[] {
-  const result: number[] = [];
-  const walk = (els: SequenceElement[]): void => {
-    for (const el of els) {
-      if (isSequenceNote(el)) {
-        result.push(el.lineNumber);
-      } else if (isSequenceBlock(el)) {
-        walk(el.children);
-        if (el.elseIfBranches) {
-          for (const branch of el.elseIfBranches) {
-            walk(branch.children);
-          }
-        }
-        walk(el.elseChildren);
-      }
-    }
-  };
-  walk(elements);
-  return result;
-}
-
 function renderParticipant(
   svg: d3Selection.Selection<SVGSVGElement, unknown, null, undefined>,
   participant: SequenceParticipant,

package/src/sharing.ts

@@ -21,7 +21,6 @@ export interface CompactViewState {
   rm?: string; // render mode override
   htv?: Record<string, string[]>; // hidden tag values
   ha?: string[]; // hidden attributes
-  enl?: number[]; // expanded note lines (sequence)
   sem?: boolean; // semantic colors (ER)
   cm?: boolean; // compact meta (kanban)
   c4l?: string; // C4 level

package/src/sitemap/parser.ts

@@ -509,7 +509,7 @@ export function parseSitemap(
     };
     collectAll(result.roots);
     validateTagValues(allNodes, result.tagGroups, pushWarning, suggest);
-    validateTagGroupNames(result.tagGroups, pushWarning);
+    validateTagGroupNames(result.tagGroups, pushWarning, pushError);
   }
 
   if (