@diagrammo/dgmo 0.6.2 → 0.6.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diagrammo/dgmo",
-  "version": "0.6.2",
+  "version": "0.6.3",
   "description": "DGMO diagram markup language — parser, renderer, and color system",
   "license": "MIT",
   "type": "module",
@@ -29,7 +29,8 @@
     ".claude/commands",
     ".github/copilot-instructions.md",
     ".cursorrules",
-    ".windsurfrules"
+    ".windsurfrules",
+    "AGENTS.md"
   ],
   "sideEffects": false,
   "scripts": {
@@ -40,6 +41,7 @@
     "test:watch": "vitest",
     "gallery": "pnpm build && node scripts/generate-gallery.mjs",
     "check:duplication": "jscpd ./src",
+    "check:deadcode": "knip",
     "postinstall": "node -e \"console.log('\\n💡 Claude Code user? Run: dgmo --install-claude-skill\\n')\""
   },
   "dependencies": {
@@ -64,9 +66,9 @@
     "@types/d3-scale": "^4.0.8",
     "@types/d3-selection": "^3.0.11",
     "@types/d3-shape": "^3.1.7",
-    "@types/dagre": "^0.7.54",
     "@types/jsdom": "^28.0.0",
     "jscpd": "^4.0.8",
+    "knip": "^6.0.1",
     "tsup": "^8.5.1",
     "typescript": "^5.7.3",
     "vitest": "^4.0.18"

package/src/c4/layout.ts

@@ -86,7 +86,6 @@ const DESC_LINE_HEIGHT = 16;
 const DESC_CHAR_WIDTH = 6.5;
 const CARD_V_PAD = 14;
 const CARD_H_PAD = 20;
-const TECH_LINE_HEIGHT = 16;
 const META_LINE_HEIGHT = 16;
 const META_CHAR_WIDTH = 6.5;
 const MARGIN = 40;
@@ -478,10 +477,6 @@ export function rollUpContextRelationships(parsed: ParsedC4): ContextRelationshi
   const allRels = collectAllRelationships(parsed.elements, ownerMap);
 
   // Also include orphan relationships
-  for (const rel of parsed.relationships) {
-    // Orphan rels have no source element name — skip them for context roll-up
-  }
-
   // Separate system-level (explicit) from nested (rolled-up)
   const topLevelNames = new Set(parsed.elements.map((e) => e.name));
   const explicitKeys = new Set<string>();

package/src/c4/parser.ts

@@ -3,7 +3,6 @@
 // ============================================================
 
 import type { PaletteColors } from '../palettes';
-import type { DgmoError } from '../diagnostics';
 import { makeDgmoError, formatDgmoError, suggest } from '../diagnostics';
 import type { TagGroup } from '../utils/tag-groups';
 import { matchTagBlockHeading } from '../utils/tag-groups';
@@ -751,21 +750,6 @@ function attachElement(
 // Post-parse validation
 // ============================================================
 
-function collectAllNames(result: ParsedC4): Map<string, number> {
-  const names = new Map<string, number>();
-  function walk(elements: C4Element[]) {
-    for (const el of elements) {
-      names.set(el.name.toLowerCase(), el.lineNumber);
-      walk(el.children);
-      for (const g of el.groups) {
-        walk(g.children);
-      }
-    }
-  }
-  walk(result.elements);
-  return names;
-}
-
 function validateRelationshipTargets(
   result: ParsedC4,
   knownNames: Map<string, number>,

package/src/c4/renderer.ts

@@ -9,8 +9,7 @@ import type { PaletteColors } from '../palettes';
 import { mix } from '../palettes/color-utils';
 import { renderInlineText } from '../utils/inline-markdown';
 import type { ParsedC4 } from './types';
-import type { C4Shape } from './types';
-import type { C4LayoutResult, C4LayoutNode, C4LayoutEdge, C4LayoutBoundary, C4LegendGroup } from './layout';
+import type { C4LayoutResult, C4LayoutEdge, C4LegendGroup } from './layout';
 import { parseC4 } from './parser';
 import { layoutC4Context, layoutC4Containers, layoutC4Components, layoutC4Deployment, collectCardMetadata } from './layout';
 import {
@@ -50,7 +49,6 @@ const CARD_V_PAD = 14;
 const TYPE_LABEL_HEIGHT = 18;
 const DIVIDER_GAP = 6;
 const NAME_HEIGHT = 20;
-const TECH_LINE_HEIGHT = 16;
 const META_FONT_SIZE = 11;
 const META_CHAR_WIDTH = 6.5;
 const META_LINE_HEIGHT = 16;
@@ -257,7 +255,6 @@ export function renderC4Context(
   const scale = Math.min(MAX_SCALE, scaleX, scaleY);
 
   const scaledW = diagramW * scale;
-  const scaledH = diagramH * scale;
   const offsetX = (width - scaledW) / 2;
   const offsetY = titleHeight + DIAGRAM_PADDING;
 
@@ -1300,7 +1297,6 @@ export function renderC4Containers(
   const scale = Math.min(MAX_SCALE, scaleX, scaleY);
 
   const scaledW = diagramW * scale;
-  const scaledH = diagramH * scale;
   const offsetX = (width - scaledW) / 2;
   const offsetY = titleHeight + DIAGRAM_PADDING;
 

package/src/class/layout.ts

@@ -2,7 +2,6 @@ import dagre from '@dagrejs/dagre';
 import type {
   ParsedClassDiagram,
   ClassNode,
-  ClassRelationship,
   RelationshipType,
 } from './types';
 

package/src/class/parser.ts

@@ -383,3 +383,31 @@ export function looksLikeClassDiagram(content: string): boolean {
 
   return false;
 }
+
+// ============================================================
+// Symbol extraction (for completion API)
+// ============================================================
+
+import type { DiagramSymbols } from '../completion';
+
+/**
+ * Extract class names (entities) from class diagram document text.
+ * Used by the dgmo completion API for ghost hints and popup completions.
+ */
+export function extractSymbols(docText: string): DiagramSymbols {
+  const entities: string[] = [];
+  let inMetadata = true;
+  for (const rawLine of docText.split('\n')) {
+    const line = rawLine.trim();
+    if (inMetadata && /^[a-z-]+\s*:/i.test(line)) continue;
+    inMetadata = false;
+    if (line.length === 0 || /^\s/.test(rawLine)) continue;
+    const m = CLASS_DECL_RE.exec(line);
+    if (m && !entities.includes(m[1]!)) entities.push(m[1]!);
+  }
+  return {
+    kind: 'class',
+    entities,
+    keywords: ['extends', 'implements', 'abstract', 'interface', 'enum'],
+  };
+}

package/src/class/renderer.ts

@@ -5,10 +5,11 @@
 import * as d3Selection from 'd3-selection';
 import * as d3Shape from 'd3-shape';
 import { FONT_FAMILY } from '../fonts';
+import { runInExportContainer, extractExportSvg } from '../utils/export-container';
 import type { PaletteColors } from '../palettes';
 import { mix } from '../palettes/color-utils';
 import type { ParsedClassDiagram, ClassModifier, RelationshipType } from './types';
-import type { ClassLayoutResult, ClassLayoutNode, ClassLayoutEdge } from './layout';
+import type { ClassLayoutResult } from './layout';
 import { parseClassDiagram } from './parser';
 import { layoutClassDiagram } from './layout';
 
@@ -99,8 +100,6 @@ function isSourceMarker(type: RelationshipType): boolean {
 // Main renderer
 // ============================================================
 
-type GSelection = d3Selection.Selection<SVGGElement, unknown, null, undefined>;
-
 export function renderClassDiagram(
   container: HTMLDivElement,
   parsed: ParsedClassDiagram,
@@ -125,7 +124,6 @@ export function renderClassDiagram(
   const scale = Math.min(MAX_SCALE, scaleX, scaleY);
 
   const scaledW = diagramW * scale;
-  const scaledH = diagramH * scale;
   const offsetX = (width - scaledW) / 2;
   const offsetY = titleHeight + DIAGRAM_PADDING;
 
@@ -506,16 +504,10 @@ export function renderClassDiagramForExport(
   const layout = layoutClassDiagram(parsed);
   const isDark = theme === 'dark';
 
-  const container = document.createElement('div');
   const exportWidth = layout.width + DIAGRAM_PADDING * 2;
   const exportHeight = layout.height + DIAGRAM_PADDING * 2 + (parsed.title ? 40 : 0);
-  container.style.width = `${exportWidth}px`;
-  container.style.height = `${exportHeight}px`;
-  container.style.position = 'absolute';
-  container.style.left = '-9999px';
-  document.body.appendChild(container);
 
-  try {
+  return runInExportContainer(exportWidth, exportHeight, (container) => {
     renderClassDiagram(
       container,
       parsed,
@@ -525,19 +517,6 @@ export function renderClassDiagramForExport(
       undefined,
       { width: exportWidth, height: exportHeight }
     );
-
-    const svgEl = container.querySelector('svg');
-    if (!svgEl) return '';
-
-    if (theme === 'transparent') {
-      svgEl.style.background = 'none';
-    }
-
-    svgEl.setAttribute('xmlns', 'http://www.w3.org/2000/svg');
-    svgEl.style.fontFamily = FONT_FAMILY;
-
-    return svgEl.outerHTML;
-  } finally {
-    document.body.removeChild(container);
-  }
+    return extractExportSvg(container, theme);
+  });
 }