@opendata-ai/openchart-engine 6.18.0 → 6.19.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@opendata-ai/openchart-engine",
-  "version": "6.18.0",
+  "version": "6.19.1",
   "description": "Headless compiler for openchart: spec validation, data compilation, scales, and layout",
   "license": "Apache-2.0",
   "author": "Riley Hilliard",
@@ -45,7 +45,7 @@
     "typecheck": "tsc --noEmit"
   },
   "dependencies": {
-    "@opendata-ai/openchart-core": "6.18.0",
+    "@opendata-ai/openchart-core": "6.19.1",
     "d3-array": "^3.2.0",
     "d3-format": "^3.1.2",
     "d3-interpolate": "^3.0.0",

package/src/annotations/__tests__/compute.test.ts

@@ -866,8 +866,8 @@ describe('computeAnnotations', () => {
 
       // Curve should start from right edge of text, not top edge
       // Right edge x ≈ label.x + textWidth
-      // "Curve test" = 10 chars * 12 * 0.55 = 66
-      expect(connector.from.x).toBeCloseTo(label.x + 66, 1);
+      // "Curve test" = 10 chars * 12 * 0.57 = 68.4
+      expect(connector.from.x).toBeCloseTo(label.x + 68.4, 1);
     });
   });
 

package/src/compile.ts

@@ -240,7 +240,7 @@ export function compileChart(spec: unknown, options: CompileOptions): ChartLayou
   // Responsive strategy
   const breakpoint = getBreakpoint(options.width);
   const heightClass = getHeightClass(options.height);
-  const strategy = getLayoutStrategy(breakpoint, heightClass);
+  let strategy = getLayoutStrategy(breakpoint, heightClass);
 
   // Apply breakpoint-conditional overrides from the expanded spec
   const rawSpec = expandedSpec as Record<string, unknown>;
@@ -286,6 +286,9 @@ export function compileChart(spec: unknown, options: CompileOptions): ChartLayou
         ...chartSpec,
         annotations: bp.annotations as NormalizedChartSpec['annotations'],
       };
+      // User explicitly provided annotations at this breakpoint — override the
+      // responsive strategy so they render inline instead of being stripped.
+      strategy = { ...strategy, annotationPosition: 'inline' };
     }
   }
 

package/src/legend/compute.ts

@@ -289,9 +289,11 @@ export function computeLegend(
   }
 
   // Top/bottom-positioned legend: horizontal flow with overflow protection.
-  // Reserve space on the right so legend entries don't overlap the brand watermark.
+  // Reserve space on the right for bottom legends so they don't overlap the brand
+  // watermark. Top legends don't need this since the brand renders at the bottom.
+  const reserveBrand = watermark && resolvedPosition === 'bottom';
   const availableWidth =
-    chartArea.width - LEGEND_PADDING * 2 - (watermark ? BRAND_RESERVE_WIDTH : 0);
+    chartArea.width - LEGEND_PADDING * 2 - (reserveBrand ? BRAND_RESERVE_WIDTH : 0);
 
   // Apply symbolLimit first if set (minimum 1), then fit remaining entries to available rows.
   if (spec.legend?.symbolLimit != null) {

package/src/sankey/compile-sankey.ts

@@ -140,6 +140,8 @@ function computeNodeLabel(
   theme: ResolvedTheme,
   nodeWidth: number,
   nodeLabelAlign: 'auto' | 'left' | 'right' = 'auto',
+  containerWidth?: number,
+  padding?: number,
 ): SankeyNodeMark['label'] {
   const depth = node.depth ?? 0;
 
@@ -168,6 +170,20 @@ function computeNodeLabel(
   const y1 = node.y1 ?? 0;
   const midY = (y0 + y1) / 2;
 
+  // Compute maxWidth: space from label position to the container edge
+  const pad = padding ?? 0;
+  let maxWidth: number | undefined;
+  if (containerWidth !== undefined) {
+    if (placeLeft) {
+      // Label goes left from x0: available space is from left padding to x0
+      maxWidth = x0 - LABEL_GAP - pad;
+    } else {
+      // Label goes right from x1: available space is from x1 to right edge
+      maxWidth = containerWidth - pad - (x1 + LABEL_GAP);
+    }
+    if (maxWidth !== undefined && maxWidth < 0) maxWidth = 0;
+  }
+
   if (placeLeft) {
     return {
       text: node.label ?? node.id,
@@ -175,6 +191,7 @@ function computeNodeLabel(
       y: midY,
       style: { ...style, textAnchor: 'end', dominantBaseline: 'central' },
       visible: true,
+      maxWidth,
     };
   }
 
@@ -184,6 +201,7 @@ function computeNodeLabel(
     y: midY,
     style: { ...style, textAnchor: 'start', dominantBaseline: 'central' },
     visible: true,
+    maxWidth,
   };
 }
 
@@ -394,7 +412,15 @@ export function compileSankey(spec: unknown, options: CompileOptions): SankeyLay
       height: (node.y1 ?? 0) - (node.y0 ?? 0),
       fill,
       cornerRadius: NODE_CORNER_RADIUS,
-      label: computeNodeLabel(node, maxDepth, theme, sankeySpec.nodeWidth, nodeLabelAlign),
+      label: computeNodeLabel(
+        node,
+        maxDepth,
+        theme,
+        sankeySpec.nodeWidth,
+        nodeLabelAlign,
+        options.width,
+        padding,
+      ),
       nodeId: node.id,
       value: node.value ?? 0,
       depth,