@diagrammo/dgmo 0.14.1 → 0.15.1

package/src/c4/parser.ts

@@ -421,7 +421,7 @@ export function parseC4(content: string, palette?: PaletteColors): ParsedC4 {
         );
         const shape = inferC4Shape(
           nodeName,
-          metadata.tech ?? metadata.technology
+          metadata['tech'] ?? metadata['technology']
         );
 
         const dNode: C4DeploymentNode = {
@@ -555,11 +555,11 @@ export function parseC4(content: string, palette?: PaletteColors): ParsedC4 {
           // parsePipeMetadata expects segments split by |; first segment is pre-pipe
           const meta = parsePipeMetadata(['', metaPart], metaAliasMap);
           // tech/technology on pipe overrides [tech] in label
-          if (meta.tech) {
-            technology = meta.tech;
+          if (meta['tech']) {
+            technology = meta['tech'];
           }
-          if (meta.technology) {
-            technology = meta.technology;
+          if (meta['technology']) {
+            technology = meta['technology'];
           }
         }
 
@@ -729,7 +729,7 @@ export function parseC4(content: string, palette?: PaletteColors): ParsedC4 {
 
       const shape =
         explicitShape ??
-        inferC4Shape(namePart, metadata.tech ?? metadata.technology);
+        inferC4Shape(namePart, metadata['tech'] ?? metadata['technology']);
 
       // Extract description from pipe metadata into dedicated field
       let isADescription: string[] | undefined;
@@ -806,7 +806,7 @@ export function parseC4(content: string, palette?: PaletteColors): ParsedC4 {
       // Determine shape: explicit > inference
       const shape =
         explicitShape ??
-        inferC4Shape(namePart, metadata.tech ?? metadata.technology);
+        inferC4Shape(namePart, metadata['tech'] ?? metadata['technology']);
 
       // Extract description from pipe metadata into dedicated field
       let prefixDescription: string[] | undefined;

package/src/chart-types.ts

@@ -490,8 +490,3 @@ export const chartTypes: readonly ChartTypeMeta[] = [
     fallback: true,
   },
 ] as const;
-
-// Chart-type ids currently flagged as beta. Renderers/UIs use this set to
-// surface a "β BETA" badge next to titles, nav entries, and template tiles.
-// Promote/demote by editing this set + the corresponding registry titles.
-export const BETA_CHART_IDS: ReadonlySet<string> = new Set(['c4', 'venn']);

package/src/class/parser.ts

@@ -184,14 +184,6 @@ export function parseClassDiagram(
     error: null,
   };
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  const _fail = (line: number, message: string): ParsedClassDiagram => {
-    const diag = makeDgmoError(line, message);
-    result.diagnostics.push(diag);
-    result.error = formatDgmoError(diag);
-    return result;
-  };
-
   const classMap = new Map<string, ClassNode>();
 
   // Per-parse alias literal → canonical class id (TD-18). Per C8.
@@ -548,7 +540,7 @@ export function looksLikeClassDiagram(content: string): boolean {
 // Symbol extraction (for completion API)
 // ============================================================
 
-import type { DiagramSymbols } from '../completion';
+import type { DiagramSymbols } from '../completion-types';
 
 /**
  * Extract class names (entities) from class diagram document text.

package/src/cli.ts

@@ -705,7 +705,9 @@ async function main(): Promise<void> {
 
   if (opts.cat) {
     const useColor =
-      !opts.noColor && !process.env.NO_COLOR && process.stdout.isTTY === true;
+      !opts.noColor &&
+      !process.env['NO_COLOR'] &&
+      process.stdout.isTTY === true;
 
     let catContent: string;
     if (opts.input && opts.input !== '-') {
@@ -822,9 +824,9 @@ async function main(): Promise<void> {
         }
       }
       const mcpServers =
-        (settings.mcpServers as Record<string, unknown> | undefined) ?? {};
+        (settings['mcpServers'] as Record<string, unknown> | undefined) ?? {};
       mcpServers['dgmo'] = mcpEntry;
-      settings.mcpServers = mcpServers;
+      settings['mcpServers'] = mcpServers;
       writeFileSync(
         settingsPath,
         JSON.stringify(settings, null, 2) + '\n',
@@ -842,9 +844,9 @@ async function main(): Promise<void> {
         }
       }
       const mcpServers =
-        (mcp.mcpServers as Record<string, unknown> | undefined) ?? {};
+        (mcp['mcpServers'] as Record<string, unknown> | undefined) ?? {};
       mcpServers['dgmo'] = mcpEntry;
-      mcp.mcpServers = mcpServers;
+      mcp['mcpServers'] = mcpServers;
       writeFileSync(mcpPath, JSON.stringify(mcp, null, 2) + '\n', 'utf-8');
       console.log(
         `✓ MCP server configured: ${join(process.cwd(), '.mcp.json')}`
@@ -1118,7 +1120,7 @@ async function main(): Promise<void> {
       );
     } else if (os === 'win32') {
       const appData =
-        process.env.APPDATA ?? join(homedir(), 'AppData', 'Roaming');
+        process.env['APPDATA'] ?? join(homedir(), 'AppData', 'Roaming');
       configPath = join(appData, 'Claude', 'claude_desktop_config.json');
     } else {
       configPath = join(
@@ -1154,7 +1156,7 @@ async function main(): Promise<void> {
       }
     }
 
-    const existingDgmo = config.mcpServers?.dgmo;
+    const existingDgmo = config.mcpServers?.['dgmo'];
     if (existingDgmo && existingDgmo.command === 'dgmo-mcp') {
       console.log(`✓ dgmo MCP server already configured in ${configPath}`);
     } else {

package/src/completion-types.ts

@@ -0,0 +1,28 @@
+// ============================================================
+// Completion / symbol-extraction shared types
+// ============================================================
+//
+// Lives in its own leaf module so chart-type parsers (Class, ER,
+// Flowchart, Infra, Pert) can `import type { DiagramSymbols }` here
+// without depending on completion.ts — which itself imports those
+// parsers' extractSymbols() functions, producing a cycle hub.
+//
+// Keep this file dependency-free.
+
+// ChartType is just a string — alias for documentation clarity.
+export type ChartType = string;
+
+export interface DiagramSymbols {
+  kind: ChartType;
+  entities: string[]; // table names, node IDs, class names, etc.
+  keywords: string[]; // diagram-specific reserved words
+  /**
+   * Map of alias-literal → canonical entity name, collected from
+   * `Name as <alias>` declarations in the document. Editor surfaces
+   * both forms in autocomplete; selecting an alias inserts the alias
+   * literal (the alias is input convenience, not a display name).
+   */
+  aliases?: Record<string, string>;
+}
+
+export type ExtractFn = (docText: string) => DiagramSymbols;

package/src/completion.ts

@@ -17,29 +17,22 @@ import { extractSymbols as extractInfraSymbols } from './infra/parser';
 import { extractSymbols as extractClassSymbols } from './class/parser';
 import { extractPertSymbols } from './pert/parser';
 import { parseFirstLine, ALL_CHART_TYPES } from './utils/parsing';
-import { CHART_TYPE_DESCRIPTIONS } from './dgmo-router';
+// Read chart-type descriptions directly from the source-of-truth data
+// module instead of via dgmo-router.ts. dgmo-router imports every
+// parser, and the parsers (Class/ER/Infra/Pert/Flowchart) type-only
+// import DiagramSymbols back from this file — creating a hub of cycles
+// through completion ↔ dgmo-router. Going through chart-types.ts (a
+// leaf module with zero imports) breaks 7 of the 10 known cycles.
+import { chartTypes } from './chart-types';
 
 // ============================================================
 // Symbol extraction
 // ============================================================
 
-// ChartType is just a string — alias here for documentation clarity.
-export type ChartType = string;
-
-export interface DiagramSymbols {
-  kind: ChartType;
-  entities: string[]; // table names, node IDs, class names, etc.
-  keywords: string[]; // diagram-specific reserved words
-  /**
-   * Map of alias-literal → canonical entity name, collected from
-   * `Name as <alias>` declarations in the document. Editor surfaces
-   * both forms in autocomplete; selecting an alias inserts the alias
-   * literal (the alias is input convenience, not a display name).
-   */
-  aliases?: Record<string, string>;
-}
-
-export type ExtractFn = (docText: string) => DiagramSymbols;
+// Types live in ./completion-types so the chart-type parsers can
+// import them without taking a cycle through this file.
+import type { ChartType, DiagramSymbols, ExtractFn } from './completion-types';
+export type { ChartType, DiagramSymbols, ExtractFn };
 
 const extractorRegistry = new Map<ChartType, ExtractFn>();
 
@@ -575,6 +568,10 @@ for (const [type, spec] of COMPLETION_REGISTRY) {
 // ============================================================
 
 /** All chart types with descriptions, for chart type autocomplete. Excludes `multi-line` alias. */
+const CHART_TYPE_DESCRIPTIONS: Record<string, string> = Object.fromEntries(
+  chartTypes.map((c) => [c.id, c.description])
+);
+
 export const CHART_TYPES: ReadonlyArray<{ name: string; description: string }> =
   [...ALL_CHART_TYPES]
     .filter((t) => t !== 'multi-line')

package/src/cycle/layout.ts

@@ -863,8 +863,8 @@ function fitToCanvas(
   nodes: CycleLayoutNode[],
   edges: CycleLayoutEdge[],
   parsed: ParsedCycle,
-  cx: number,
-  cy: number,
+  _cx: number,
+  _cy: number,
   radius: number,
   width: number,
   height: number,