@diagrammo/dgmo 0.11.2 → 0.12.0

package/docs/language-reference.md

@@ -1967,25 +1967,6 @@ Phase headers accept pipe metadata for per-phase styling:
 
 The phase bar tints to a soft mix of the color over the background. Phases without metadata fall back to the neutral gray bar.
 
-### Diagnostic codes
-
-| Code | Severity | Fires when |
-|------|----------|------------|
-| `E_RACI_MULTI_ACCOUNTABLE` | error | RACI/RASCI task has more than one A |
-| `E_DACI_MULTI_DRIVER` | error | DACI task has more than one D |
-| `E_DACI_MULTI_ACCOUNTABLE` | error | DACI task has more than one A |
-| `E_RACI_INVALID_MARKER` | error | Marker isn't in the resolved variant's alphabet |
-| `E_RACI_UNEXPECTED_LINE` | error | Free-text line appears after the first role assignment |
-| `E_RACI_MIXED_VARIANTS` | error | Both `D` and `S` markers appear without a `variant-*` lock |
-| `E_RACI_DUPLICATE_VARIANT` | error | More than one `variant-*` directive in the chart |
-| `W_RACI_MISSING_ACCOUNTABLE` | warning | RACI/RASCI task has no A |
-| `W_RACI_MISSING_RESPONSIBLE` | warning | RACI/RASCI task has no R |
-| `W_DACI_MISSING_DRIVER` | warning | DACI task has no D |
-| `W_DACI_MISSING_ACCOUNTABLE` | warning | DACI task has no A |
-| `W_RACI_UNKNOWN_ROLE` | warning | Role used without being declared in `roles` |
-
-Warnings are non-blocking; there is no source-level suppression mechanism — by design, source should not silently mute the linter.
-
 ### Display vs source ordering
 
 Markers in cells are always **rendered in canonical alphabet order** (`R A C I`, `R A S C I`, `D A C I`) regardless of source order. Source casing and order are preserved in the file; mutations operate on source order to keep round-trips byte-stable except for the cell that changed.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diagrammo/dgmo",
-  "version": "0.11.2",
+  "version": "0.12.0",
   "description": "DGMO diagram markup language — parser, renderer, and color system",
   "license": "MIT",
   "repository": {

package/src/completion.ts

@@ -164,6 +164,15 @@ export const COMPLETION_REGISTRY = new Map<string, DirectiveSpec>([
       'no-value': { description: 'Hide value labels at each point' },
     }),
   ],
+  [
+    'multi-line',
+    withGlobals({
+      series: { description: 'Series name(s)' },
+      'x-label': { description: 'X-axis label' },
+      'y-label': { description: 'Y-axis label' },
+      'no-value': { description: 'Hide value labels at each point' },
+    }),
+  ],
   [
     'polar-area',
     withGlobals({
@@ -270,8 +279,15 @@ export const COMPLETION_REGISTRY = new Map<string, DirectiveSpec>([
   ],
   [
     'flowchart',
+    // Spec §5 §4.6: direction-lr, orientation-vertical, no-color, solid-fill
     withGlobals({
       'direction-lr': { description: 'Switch to left-to-right layout' },
+      'orientation-vertical': {
+        description: 'Use vertical orientation for ranks',
+      },
+      'no-color': {
+        description: 'Resolve all nodes to muted neutral fill',
+      },
     }),
   ],
   [
@@ -284,88 +300,76 @@ export const COMPLETION_REGISTRY = new Map<string, DirectiveSpec>([
   ],
   [
     'er',
+    // Spec §9 §8.5: notation (chen/crow), active-tag.
     withGlobals({
+      notation: {
+        description: 'ER notation style',
+        values: ['chen', 'crow'],
+      },
       'active-tag': { description: 'Active tag group name' },
     }),
   ],
   [
     'org',
+    // Spec §7 §6.5: direction-tb, sub-node-label, show-sub-node-count,
+    // hide, active-tag. solid-fill via SOLID_FILL_CAPABLE.
     withGlobals({
+      'direction-tb': { description: 'Switch to top-to-bottom layout' },
       'sub-node-label': { description: 'Label for sub-nodes' },
       'show-sub-node-count': { description: 'Show sub-node counts' },
+      hide: { description: 'Hide tag:value pairs' },
       'active-tag': { description: 'Active tag group name' },
     }),
   ],
   [
     'kanban',
+    // Spec §11 §10.4: no-auto-color, hide, active-tag.
     withGlobals({
       'no-auto-color': { description: 'Disable automatic card coloring' },
+      hide: { description: 'Hide tag:value pairs' },
       'active-tag': { description: 'Active tag group name' },
     }),
   ],
-  // RACI / RASCI / DACI — three chart-type ids, one parser, same directives.
+  // RACI / RASCI / DACI — one chart type (`raci`), variant inferred from
+  // markers or locked via `variant-*` bare directive. `rasci`/`daci` are
+  // not first-line keywords so they get no separate registry entry.
   [
     'raci',
     withGlobals({
-      variant: {
-        description: 'Variant rule set',
-        values: ['raci', 'rasci', 'daci'],
+      'variant-raci': {
+        description: 'Lock chart to RACI variant (R / A / C / I markers)',
       },
-      roles: {
+      'variant-rasci': {
         description:
-          'Comma-separated role list (declares column order; enables unknown-role linting)',
-      },
-      draft: {
-        description: 'Suppress missing-A / missing-R warnings during authoring',
+          'Lock chart to RASCI variant (adds Support — R / A / S / C / I)',
       },
-      'active-tag': { description: 'Active tag group name' },
-    }),
-  ],
-  [
-    'rasci',
-    withGlobals({
-      variant: {
-        description: 'Variant rule set',
-        values: ['raci', 'rasci', 'daci'],
-      },
-      roles: {
+      'variant-daci': {
         description:
-          'Comma-separated role list (declares column order; enables unknown-role linting)',
-      },
-      draft: {
-        description: 'Suppress missing-A / missing-R warnings during authoring',
-      },
-      'active-tag': { description: 'Active tag group name' },
-    }),
-  ],
-  [
-    'daci',
-    withGlobals({
-      variant: {
-        description: 'Variant rule set',
-        values: ['raci', 'rasci', 'daci'],
+          'Lock chart to DACI variant (Driver / Approver / Contributor / Informed)',
       },
       roles: {
         description:
-          'Comma-separated role list (declares column order; enables unknown-role linting)',
-      },
-      draft: {
-        description: 'Suppress missing-A / missing-R warnings during authoring',
+          'Declare role column order (inline `roles A, B, C` or indented block with per-role pipe metadata)',
       },
       'active-tag': { description: 'Active tag group name' },
     }),
   ],
   [
     'c4',
+    // Spec §8 §7.7: direction-tb, active-tag.
     withGlobals({
+      'direction-tb': { description: 'Switch to top-to-bottom layout' },
       'active-tag': { description: 'Active tag group name' },
     }),
   ],
   [
     'state',
+    // Spec §6 §5.5: direction-tb, no-color, solid-fill.
     withGlobals({
       'direction-tb': { description: 'Switch to top-to-bottom layout' },
-      color: { description: 'Color mode', values: ['off'] },
+      'no-color': {
+        description: 'Resolve all states to muted neutral fill',
+      },
     }),
   ],
   [
@@ -392,6 +396,7 @@ export const COMPLETION_REGISTRY = new Map<string, DirectiveSpec>([
   ],
   [
     'gantt',
+    // Spec §13 §12.2 Options.
     withGlobals({
       start: { description: 'Project start date (YYYY-MM-DD)' },
       'today-marker': {
@@ -399,8 +404,14 @@ export const COMPLETION_REGISTRY = new Map<string, DirectiveSpec>([
       },
       sort: { description: 'Sort order', values: ['time', 'group', 'tag'] },
       'critical-path': { description: 'Show critical path' },
-      dependencies: { description: 'Show dependencies' },
+      'no-dependencies': { description: 'Hide dependency arrows' },
+      'sprint-length': { description: 'Sprint duration (e.g. 2w)' },
+      'sprint-number': { description: 'Starting sprint number' },
+      'sprint-start': { description: 'Sprint start date (YYYY-MM-DD)' },
       'active-tag': { description: 'Active tag group name' },
+      // Legacy positive form `dependencies` — kept for back-compat. Use
+      // `no-dependencies` to suppress dependency arrows in new code.
+      dependencies: { description: 'Show dependencies (legacy form)' },
     }),
   ],
   [
@@ -427,15 +438,13 @@ export const COMPLETION_REGISTRY = new Map<string, DirectiveSpec>([
   ],
   [
     'tech-radar',
+    // Spec §20 documents one directive: `show-blip-legend`. `rings` is a
+    // structural block keyword; quadrant/ring/trend/color are pipe metadata
+    // that live in PIPE_METADATA.
     withGlobals({
-      rings: { description: 'Ring names block (innermost to outermost)' },
-      quadrant: {
-        description:
-          'Quadrant position (top-left, top-right, bottom-left, bottom-right)',
+      'show-blip-legend': {
+        description: 'Render the four-column blip listing alongside the radar',
       },
-      ring: { description: 'Ring assignment for a blip' },
-      trend: { description: 'Blip trend (new, up, down, stable)' },
-      color: { description: 'Override quadrant color' },
     }),
   ],
   [
@@ -452,30 +461,29 @@ export const COMPLETION_REGISTRY = new Map<string, DirectiveSpec>([
   ],
   [
     'journey-map',
+    // Spec §22 directives: `no-legend`, `active-tag`. `persona` is a
+    // structural keyword (like `tag` / `roles`), not a directive.
+    // `solid-fill` is added via SOLID_FILL_CAPABLE below.
     withGlobals({
       'no-legend': { description: 'Hide the score legend' },
-      persona: { description: 'Define the journey persona' },
+      'active-tag': { description: 'Active tag group name' },
     }),
   ],
   [
     'pyramid',
+    // Spec §23.5 documents `inverted`; `solid-fill` is added via
+    // SOLID_FILL_CAPABLE below (working but not yet in spec §23.5).
+    // `color`/`description` are layer pipe-metadata, not directives.
     withGlobals({
       inverted: { description: 'Flip apex to the bottom (funnel orientation)' },
-      color: { description: 'Override layer color (pipe metadata)' },
-      description: { description: 'Layer description (pipe or indented body)' },
     }),
   ],
   [
     'ring',
-    withGlobals({
-      color: {
-        description:
-          'Override ring color (pipe metadata; closed set of 11 named colors)',
-      },
-      description: {
-        description: 'Ring description (pipe shorthand or indented body)',
-      },
-    }),
+    // Per spec §24.5 the only chart-specific directive is `solid-fill`,
+    // applied via SOLID_FILL_CAPABLE below. `color`/`description` are
+    // layer pipe-metadata, not directives — they live in PIPE_METADATA.
+    withGlobals({}),
   ],
 ]);
 
@@ -576,19 +584,28 @@ export interface PipeKeySpec {
 
 /**
  * Pipe metadata keys for inline `| key value` on data lines.
- * Keyed by chart type → { node: ..., edge: ... }.
+ * Keyed by chart type → { context-name: keys }.
+ *
+ * Contexts are open-ended. The two universal ones are:
+ *   - `node` — the default for any non-arrow line
+ *   - `edge` — lines containing an arrow (`->`, `--`)
+ *
+ * Charts with richer line types declare additional contexts:
+ *   - raci: `role`, `phase`, `assignment`
+ *   - ring / pyramid: `layer`
+ *   - tech-radar: `quadrant`, `blip`
+ *   - journey-map: `step`
+ *
+ * Consumers that don't classify lines beyond node/edge can fall back to
+ * the union of all contexts via `unionPipeKeys(spec)` below.
  *
  * IMPORTANT: NEVER add 'sequence' here. The `|` character in sequence
  * diagrams separates display names from identifiers and tag metadata.
  * Adding sequence would trigger false pipe-metadata completions on every `|`.
  */
-export const PIPE_METADATA = new Map<
-  string,
-  {
-    node: Record<string, PipeKeySpec>;
-    edge: Record<string, PipeKeySpec>;
-  }
->([
+export type PipeContextMap = Record<string, Record<string, PipeKeySpec>>;
+
+export const PIPE_METADATA = new Map<string, PipeContextMap>([
   [
     'infra',
     {
@@ -667,21 +684,25 @@ export const PIPE_METADATA = new Map<
     },
   ],
   [
+    // Tech-radar pipe metadata (spec §20). Two contexts:
+    //   - quadrant: top-level quadrant headers (`Tools | quadrant: top-left, color: blue`)
+    //   - blip: indented blip lines (`  Vite | ring: Adopt, trend: up`)
     'tech-radar',
     {
-      node: {
+      quadrant: {
         quadrant: {
           description: 'Quadrant position',
           values: ['top-left', 'top-right', 'bottom-left', 'bottom-right'],
         },
-        ring: { description: 'Ring assignment for blip' },
+        color: { description: 'Override quadrant color' },
+      },
+      blip: {
+        ring: { description: 'Ring assignment (must match a declared ring)' },
         trend: {
           description: 'Blip trend indicator',
           values: ['new', 'up', 'down', 'stable'],
         },
-        color: { description: 'Override quadrant color' },
       },
-      edge: {},
     },
   ],
   [
@@ -698,8 +719,69 @@ export const PIPE_METADATA = new Map<
       },
     },
   ],
+  [
+    // RACI pipe metadata (spec §24A). Two contexts:
+    //   - role: declarations inside the `roles` block (`Cap | color: red`)
+    //   - phase: bracketed phase headers (`[Departure] | color: teal`)
+    'raci',
+    {
+      role: {
+        color: { description: 'Role column tint (palette name)' },
+      },
+      phase: {
+        color: { description: 'Phase bar tint (palette name)' },
+      },
+    },
+  ],
+  [
+    // Ring layer pipe metadata (spec §24.4). One context: `layer`.
+    'ring',
+    {
+      layer: {
+        color: { description: 'Ring color (palette name)' },
+        description: { description: 'Layer description (one-liner shorthand)' },
+      },
+    },
+  ],
+  [
+    // Pyramid layer pipe metadata (spec §23.4). Identical surface to ring.
+    'pyramid',
+    {
+      layer: {
+        color: { description: 'Layer color (palette name)' },
+        description: { description: 'Layer description (one-liner shorthand)' },
+      },
+    },
+  ],
+  [
+    // Journey-map step pipe metadata (spec §22). One static key: `score`.
+    // Tag aliases (e.g. `ch: Web`) are user-defined via the `tag` block
+    // and resolved dynamically — not part of the static map.
+    'journey-map',
+    {
+      step: {
+        score: { description: 'Step score (1–5 integer; high = good)' },
+      },
+    },
+  ],
 ]);
 
+/**
+ * Union of all pipe-metadata keys across every context for a chart.
+ * Use this when a consumer can't precisely classify which line context
+ * the cursor is on (e.g. raci role vs phase) and wants to surface every
+ * potentially-valid key. Returns an empty record for unknown chart types.
+ */
+export function unionPipeKeys(chartType: string): Record<string, PipeKeySpec> {
+  const spec = PIPE_METADATA.get(chartType);
+  if (!spec) return {};
+  const merged: Record<string, PipeKeySpec> = {};
+  for (const ctx of Object.values(spec)) {
+    Object.assign(merged, ctx);
+  }
+  return merged;
+}
+
 // ============================================================
 // Derived metadata key set
 // ============================================================