@opendata-ai/openchart-engine 2.4.0 → 2.5.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@opendata-ai/openchart-engine",
-  "version": "2.4.0",
+  "version": "2.5.0",
   "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": "2.4.0",
+    "@opendata-ai/openchart-core": "2.5.0",
     "d3-array": "^3.2.0",
     "d3-format": "^3.1.2",
     "d3-interpolate": "^3.0.0",

package/src/__test-fixtures__/specs.ts

@@ -67,6 +67,7 @@ export function makeLineSpec(): NormalizedChartSpec {
     darkMode: 'off',
     labels: { density: 'auto', format: '' },
     hiddenSeries: [],
+    seriesStyles: {},
   };
 }
 
@@ -94,6 +95,7 @@ export function makeBarSpec(): NormalizedChartSpec {
     darkMode: 'off',
     labels: { density: 'auto', format: '' },
     hiddenSeries: [],
+    seriesStyles: {},
   };
 }
 
@@ -123,5 +125,6 @@ export function makeScatterSpec(): NormalizedChartSpec {
     darkMode: 'off',
     labels: { density: 'auto', format: '' },
     hiddenSeries: [],
+    seriesStyles: {},
   };
 }

package/src/charts/line/__tests__/compute.test.ts

@@ -799,3 +799,112 @@ describe('computeLineLabels', () => {
     expect(labelMap.size).toBe(0);
   });
 });
+
+// ---------------------------------------------------------------------------
+// seriesStyles tests
+// ---------------------------------------------------------------------------
+
+describe('seriesStyles', () => {
+  it('applies dashed line style to a specific series', () => {
+    const spec = makeMultiSeriesSpec();
+    spec.seriesStyles = { UK: { lineStyle: 'dashed' } };
+    const scales = computeScales(spec, chartArea, spec.data);
+    const marks = computeLineMarks(spec, scales, chartArea, fullStrategy);
+
+    const lineMarks = marks.filter((m): m is LineMark => m.type === 'line');
+    const ukLine = lineMarks.find((m) => m.seriesKey === 'UK');
+    const usLine = lineMarks.find((m) => m.seriesKey === 'US');
+
+    expect(ukLine?.strokeDasharray).toBe('6 4');
+    expect(usLine?.strokeDasharray).toBeUndefined();
+  });
+
+  it('applies dotted line style', () => {
+    const spec = makeMultiSeriesSpec();
+    spec.seriesStyles = { US: { lineStyle: 'dotted' } };
+    const scales = computeScales(spec, chartArea, spec.data);
+    const marks = computeLineMarks(spec, scales, chartArea, fullStrategy);
+
+    const usLine = marks.find((m): m is LineMark => m.type === 'line' && m.seriesKey === 'US');
+    expect(usLine?.strokeDasharray).toBe('2 3');
+  });
+
+  it('hides point markers when showPoints is false', () => {
+    const spec = makeMultiSeriesSpec();
+    spec.seriesStyles = { UK: { showPoints: false } };
+    const scales = computeScales(spec, chartArea, spec.data);
+    const marks = computeLineMarks(spec, scales, chartArea, fullStrategy);
+
+    const ukPoints = marks.filter(
+      (m): m is PointMark => m.type === 'point' && m.data.country === 'UK',
+    );
+    const usPoints = marks.filter(
+      (m): m is PointMark => m.type === 'point' && m.data.country === 'US',
+    );
+
+    // UK points should have r=0 (hidden)
+    expect(ukPoints.every((p) => p.r === 0)).toBe(true);
+    // US points should still have default radius
+    expect(usPoints.every((p) => p.r > 0)).toBe(true);
+  });
+
+  it('overrides strokeWidth for a series', () => {
+    const spec = makeMultiSeriesSpec();
+    spec.seriesStyles = { UK: { strokeWidth: 1.5 } };
+    const scales = computeScales(spec, chartArea, spec.data);
+    const marks = computeLineMarks(spec, scales, chartArea, fullStrategy);
+
+    const lineMarks = marks.filter((m): m is LineMark => m.type === 'line');
+    const ukLine = lineMarks.find((m) => m.seriesKey === 'UK');
+    const usLine = lineMarks.find((m) => m.seriesKey === 'US');
+
+    expect(ukLine?.strokeWidth).toBe(1.5);
+    expect(usLine?.strokeWidth).toBe(2.5); // default
+  });
+
+  it('sets opacity on a series', () => {
+    const spec = makeMultiSeriesSpec();
+    spec.seriesStyles = { UK: { opacity: 0.5 } };
+    const scales = computeScales(spec, chartArea, spec.data);
+    const marks = computeLineMarks(spec, scales, chartArea, fullStrategy);
+
+    const ukLine = marks.find((m): m is LineMark => m.type === 'line' && m.seriesKey === 'UK');
+    const usLine = marks.find((m): m is LineMark => m.type === 'line' && m.seriesKey === 'US');
+
+    expect(ukLine?.opacity).toBe(0.5);
+    expect(usLine?.opacity).toBeUndefined();
+  });
+
+  it('combines multiple style overrides on the same series', () => {
+    const spec = makeMultiSeriesSpec();
+    spec.seriesStyles = {
+      UK: { lineStyle: 'dashed', showPoints: false, strokeWidth: 1, opacity: 0.6 },
+    };
+    const scales = computeScales(spec, chartArea, spec.data);
+    const marks = computeLineMarks(spec, scales, chartArea, fullStrategy);
+
+    const ukLine = marks.find((m): m is LineMark => m.type === 'line' && m.seriesKey === 'UK');
+    expect(ukLine?.strokeDasharray).toBe('6 4');
+    expect(ukLine?.strokeWidth).toBe(1);
+    expect(ukLine?.opacity).toBe(0.6);
+
+    const ukPoints = marks.filter(
+      (m): m is PointMark => m.type === 'point' && m.data.country === 'UK',
+    );
+    expect(ukPoints.every((p) => p.r === 0)).toBe(true);
+  });
+
+  it('does not apply styles when seriesStyles is empty', () => {
+    const spec = makeMultiSeriesSpec();
+    spec.seriesStyles = {};
+    const scales = computeScales(spec, chartArea, spec.data);
+    const marks = computeLineMarks(spec, scales, chartArea, fullStrategy);
+
+    const lineMarks = marks.filter((m): m is LineMark => m.type === 'line');
+    for (const line of lineMarks) {
+      expect(line.strokeDasharray).toBeUndefined();
+      expect(line.opacity).toBeUndefined();
+      expect(line.strokeWidth).toBe(2.5);
+    }
+  });
+});

package/src/charts/line/compute.ts

@@ -140,6 +140,15 @@ export function computeLineMarks(
     // segments are created by starting a new M command.
     const combinedPath = pathParts.join(' ');
 
+    // Look up per-series style overrides
+    const seriesStyleKey = seriesKey === '__default__' ? undefined : seriesKey;
+    const styleOverride = seriesStyleKey ? spec.seriesStyles?.[seriesStyleKey] : undefined;
+
+    // Map lineStyle to SVG strokeDasharray
+    let strokeDasharray: string | undefined;
+    if (styleOverride?.lineStyle === 'dashed') strokeDasharray = '6 4';
+    else if (styleOverride?.lineStyle === 'dotted') strokeDasharray = '2 3';
+
     // Create the LineMark with the combined path points.
     // The points array includes all valid points across all segments.
     const lineMark: LineMark = {
@@ -147,25 +156,28 @@ export function computeLineMarks(
       points: allPoints,
       path: combinedPath,
       stroke: color,
-      strokeWidth: DEFAULT_STROKE_WIDTH,
-      seriesKey: seriesKey === '__default__' ? undefined : seriesKey,
+      strokeWidth: styleOverride?.strokeWidth ?? DEFAULT_STROKE_WIDTH,
+      strokeDasharray,
+      opacity: styleOverride?.opacity,
+      seriesKey: seriesStyleKey,
       data: pointsWithData.map((p) => p.row),
       aria,
     };
 
     marks.push(lineMark);
 
-    // Create point marks for hover targets
+    // Create point marks for hover targets (skip if showPoints is false)
+    const showPoints = styleOverride?.showPoints !== false;
     for (let i = 0; i < pointsWithData.length; i++) {
       const p = pointsWithData[i];
       const pointMark: PointMark = {
         type: 'point',
         cx: p.x,
         cy: p.y,
-        r: DEFAULT_POINT_RADIUS,
+        r: showPoints ? DEFAULT_POINT_RADIUS : 0,
         fill: color,
-        stroke: '#ffffff',
-        strokeWidth: 1.5,
+        stroke: showPoints ? '#ffffff' : 'transparent',
+        strokeWidth: showPoints ? 1.5 : 0,
         fillOpacity: 0,
         data: p.row,
         aria: {

package/src/compiler/normalize.ts

@@ -197,6 +197,7 @@ function normalizeChartSpec(spec: ChartSpec, warnings: string[]): NormalizedChar
     theme: spec.theme ?? {},
     darkMode: spec.darkMode ?? 'off',
     hiddenSeries: spec.hiddenSeries ?? [],
+    seriesStyles: spec.seriesStyles ?? {},
   };
 }
 

package/src/compiler/types.ts

@@ -73,6 +73,8 @@ export interface NormalizedChartSpec {
   darkMode: DarkMode;
   /** Series names to hide from rendering. */
   hiddenSeries: string[];
+  /** Per-series visual style overrides. */
+  seriesStyles: Record<string, import('@opendata-ai/openchart-core').SeriesStyle>;
 }
 
 /** A TableSpec with all optional fields filled with sensible defaults. */