@flikk/ui 1.0.0-beta.30 → 1.0.0-beta.32

package/dist/components/charts/BarChart/BarChart.js

@@ -11,13 +11,15 @@ import { ChartErrorBoundary } from '../shared/ChartErrorBoundary/ChartErrorBound
 import { XAxis } from '../shared/ChartAxis/XAxis.js';
 import { YAxis } from '../shared/ChartAxis/YAxis.js';
 import { HorizontalGrid } from '../shared/ChartGrid/HorizontalGrid.js';
-import { ChartTooltip } from '../shared/ChartTooltip/ChartTooltip.js';
+import { ChartTooltip } from '../../core/shared/ChartTooltip/ChartTooltip.js';
 import { resolveLegendPosition, ChartLegend } from '../shared/ChartLegend/ChartLegend.js';
 import { BAR_CHART_DEFAULTS } from './BarChart.types.js';
 import { X_AXIS_MARGIN } from '../types/chart.types.js';
 import { BAR_GROUP_STAGGER, BAR_SPRING, barEntranceTransition } from '../utils/animation-utils.js';
 import { BarSeriesDefs, BarGlowRing, barStrokeStyle, barFill, barDimOpacity, deriveBarEmphasis } from '../utils/bar-emphasis.js';
-import { ExclamationTriangleIcon } from '@heroicons/react/20/solid';
+import { resolveSeriesColor } from '../utils/series-color.js';
+import { chartTheme } from '../theme/chart.theme.js';
+import { Warning } from '@phosphor-icons/react';
 
 // Pure helper functions moved to module level to avoid recreation per render
 const createRoundedTopBarPath = (x, y, width, height, radius) => {
@@ -59,8 +61,6 @@ showGrid = BAR_CHART_DEFAULTS.showGrid, showXAxis = BAR_CHART_DEFAULTS.showXAxis
 // Standardized visual props with defaults (renamed from bar-specific names)
 barRadius, radius, // @deprecated Use barRadius instead
 gap = BAR_CHART_DEFAULTS.gap, categoryGap = BAR_CHART_DEFAULTS.categoryGap, 
-// Bar-specific props
-orientation, 
 // Animation props
 showAnimation = BAR_CHART_DEFAULTS.showAnimation, 
 // Placeholder bar props
@@ -98,34 +98,15 @@ className, title, description, enableKeyboardNavigation = BAR_CHART_DEFAULTS.ena
     const { validation, sanitizedData, isValid, hasWarnings, safeScale, safeMath, } = useChartValidation(data, config, {
         autoSanitize: true,
     });
-    // Simple color functions based on direct CSS variables (following guidelines)
     const getSeriesFillColor = (index, key) => {
-        var _a, _b, _c;
-        if (key && ((_a = config[key]) === null || _a === void 0 ? void 0 : _a.color)) {
-            const customColor = config[key].color;
-            if (!customColor.startsWith("var(") &&
-                !customColor.startsWith("#") &&
-                !customColor.startsWith("rgb") &&
-                !customColor.startsWith("hsl") &&
-                customColor.includes("-")) {
-                return `var(--color-${customColor})`;
-            }
-            return customColor;
-        }
-        const defaultColors = [
-            "var(--color-primary)",
-            "var(--color-warning)",
-            "var(--color-success)",
-            "var(--color-danger)",
-        ];
-        // Use custom colors if provided, otherwise fall back to defaults
-        const themedColors = (_c = (_b = theme === null || theme === void 0 ? void 0 : theme.baseStyle) === null || _b === void 0 ? void 0 : _b.colors) === null || _c === void 0 ? void 0 : _c.series;
+        var _a, _b;
+        const themedColors = (_b = (_a = theme === null || theme === void 0 ? void 0 : theme.baseStyle) === null || _a === void 0 ? void 0 : _a.colors) === null || _b === void 0 ? void 0 : _b.series;
         const palette = colors && colors.length > 0
             ? colors
             : themedColors && themedColors.length > 0
                 ? themedColors
-                : defaultColors;
-        return palette[index % palette.length];
+                : chartTheme.baseStyle.colors.series;
+        return resolveSeriesColor(index, key, config, palette);
     };
     const resolvedLegendPosition = resolveLegendPosition(legend, config);
     const legendItems = Object.keys(config).map((key, index) => {
@@ -332,7 +313,7 @@ className, title, description, enableKeyboardNavigation = BAR_CHART_DEFAULTS.ena
                     : { scaleY: 1, opacity: 1 }, animate: {
                     scaleY: 1,
                     opacity: 1,
-                }, transition: { ...BAR_SPRING, delay }, className: "transition-opacity duration-200", style: {
+                }, transition: { ...BAR_SPRING, delay }, className: "transition-opacity duration-[var(--motion-duration-base)]", style: {
                     originY: 1,
                     transformOrigin: "bottom",
                     fill: `url(#${hatchPatternId})`,
@@ -364,8 +345,8 @@ className, title, description, enableKeyboardNavigation = BAR_CHART_DEFAULTS.ena
                         // plain style values at mount and ignores re-render changes
                         opacity: barDimOpacity(shouldReduceOpacity, isGradientVariant),
                     }, transition: commonTransition, className: cn(isGradientVariant
-                        ? "transition-all duration-300 ease-out"
-                        : "transition-colors duration-200", !isNull && "cursor-pointer", isFocused && "stroke-2"), style: {
+                        ? "transition-all duration-[var(--motion-duration-slow)] ease-out"
+                        : "transition-colors duration-[var(--motion-duration-base)]", !isNull && "cursor-pointer", isFocused && "stroke-2"), style: {
                         originY: isNegative ? 0 : 1,
                         transformOrigin: isNegative ? "top" : "bottom",
                         fill: barFill({
@@ -449,7 +430,7 @@ className, title, description, enableKeyboardNavigation = BAR_CHART_DEFAULTS.ena
     // Rules of Hooks: "move it before the return."
     // Invalid data: show an error message
     if (!isValid) {
-        return (jsxs("div", { className: cn("chart-container flex flex-col items-center justify-center p-6 border border-[var(--color-border)] rounded-lg text-2xl", className), role: "alert", "aria-live": "polite", ...props, children: [jsxs("div", { className: "flex flex-col items-center gap-4 mb-1", children: [jsx(ExclamationTriangleIcon, { className: "size-6 text-[var(--color-danger)]" }), jsx("div", { className: "text-base font-semibold text-[var(--color-text-secondary)]", children: "Invalid Chart Data" })] }), jsx("p", { className: "text-base text-[var(--color-text-secondary)]/70 mb-2", children: ((_b = validation.errors[0]) === null || _b === void 0 ? void 0 : _b.message) ||
+        return (jsxs("div", { className: cn("chart-container flex flex-col items-center justify-center p-6 border border-[var(--color-border)] rounded-lg text-2xl", className), role: "alert", "aria-live": "polite", ...props, children: [jsxs("div", { className: "flex flex-col items-center gap-4 mb-1", children: [jsx(Warning, { className: "size-6 text-[var(--color-danger)]" }), jsx("div", { className: "text-base font-semibold text-[var(--color-text-secondary)]", children: "Invalid Chart Data" })] }), jsx("p", { className: "text-base text-[var(--color-text-secondary)]/70 mb-2", children: ((_b = validation.errors[0]) === null || _b === void 0 ? void 0 : _b.message) ||
                         "Unable to render chart due to data validation errors" }), process.env.NODE_ENV === "development" &&
                     validation.errors.length > 1 && (jsxs("details", { className: "text-sm text-warning-500", children: [jsxs("summary", { className: "cursor-pointer", children: ["Show all errors (", validation.errors.length, ")"] }), jsx("ul", { className: "mt-2 text-left", children: validation.errors.map((error, index) => (jsxs("li", { children: ["\u2022 ", error.message] }, index))) })] }))] }));
     }
@@ -466,7 +447,7 @@ className, title, description, enableKeyboardNavigation = BAR_CHART_DEFAULTS.ena
                 height: height !== undefined ? (typeof height === "number" ? `${height}px` : height) : undefined,
             }, ...chartAccessibilityProps, onMouseEnter: handleMouseInteraction, onMouseMove: handleMouseInteraction, ...props, children: [resolvedLegendPosition === "top" && (jsx(ChartLegend, { items: legendItems, className: "shrink-0 pb-2" })), descriptionProps && jsx("div", { ...descriptionProps }), showValidationWarnings && hasWarnings && (jsxs("div", { className: "absolute top-2 left-2 right-2 z-10 p-2 bg-[var(--color-warning-50)] border border-[var(--color-warning-200)] rounded text-sm text-[var(--color-warning)] shadow-sm", children: [jsx("strong", { children: "Chart Warnings:" }), jsx("ul", { className: "mt-1", children: validation.warnings
                                 .filter((warning) => !warning.message.includes("has no valid numeric values"))
-                                .map((warning, index) => (jsxs("li", { children: ["\u2022 ", warning.message] }, index))) }), validation.warnings.some((warning) => warning.message.includes("has no valid numeric values")) && (jsx("div", { className: "mt-2 pt-2 border-t border-[var(--color-warning-200)]", children: jsx("span", { className: "text-[var(--color-warning)]", children: "\u2139\uFE0F Null/undefined values are handled by using averages from other data points" }) }))] })), jsx("div", { className: "relative flex-1 min-h-0", children: jsxs("svg", { width: "100%", height: "100%", viewBox: `0 0 ${dimensions.width} ${dimensions.height}`, style: { overflow: isGradientVariant ? "visible" : undefined }, children: [jsxs("defs", { children: [jsxs("pattern", { id: hatchPatternId, patternUnits: "userSpaceOnUse", width: "8", height: "8", children: [jsx("rect", { width: "8", height: "8", fill: "var(--color-neutral-50)" }), jsx("path", { d: "M-2,2 l4,-4\n                   M0,8 l8,-8\n                   M6,10 l4,-4", style: { stroke: "black", strokeWidth: 4, opacity: 0.1 } })] }), isGradientVariant &&
+                                .map((warning, index) => (jsxs("li", { children: ["\u2022 ", warning.message] }, index))) }), validation.warnings.some((warning) => warning.message.includes("has no valid numeric values")) && (jsx("div", { className: "mt-2 pt-2 border-t border-[var(--color-warning-200)]", children: jsx("span", { className: "text-[var(--color-warning)]", children: "\u2139\uFE0F Null/undefined values are handled by using averages from other data points" }) }))] })), jsx("div", { className: "relative flex-1 min-h-0", children: jsxs("svg", { width: "100%", height: "100%", viewBox: `0 0 ${dimensions.width} ${dimensions.height}`, style: { overflow: isGradientVariant ? "visible" : undefined }, children: [jsxs("defs", { children: [jsxs("pattern", { id: hatchPatternId, patternUnits: "userSpaceOnUse", width: "8", height: "8", children: [jsx("rect", { width: "8", height: "8", fill: "var(--color-mono-50)" }), jsx("path", { d: "M-2,2 l4,-4\n                   M0,8 l8,-8\n                   M6,10 l4,-4", style: { stroke: "black", strokeWidth: 4, opacity: 0.1 } })] }), isGradientVariant &&
                                         dataKeys.map((key, index) => {
                                             const color = getSeriesFillColor(index, key);
                                             return (jsx(BarSeriesDefs, { gradientId: getBarGradientId(index), shadowId: getBarShadowId(index), ringId: getBarRingId(index), color: color }, key));

package/dist/components/charts/BarChart/BarChart.types.d.ts

@@ -1,5 +1,13 @@
 import { BaseChartProps, StandardChartDisplayProps, StandardVisualProps, BarChartSpecificProps, ChartVariant } from "../types/chart.types";
-export interface BarChartProps extends BaseChartProps, StandardChartDisplayProps, StandardVisualProps, BarChartSpecificProps {
+/**
+ * Visual props that actually apply to a vertical bar chart.
+ *
+ * 🔴 Narrowed with `Omit` on purpose — see the note on `StandardVisualProps`.
+ * Kept: `barRadius` (+ its deprecated `radius` alias) and `gap`.
+ * Dropped: dot/cell radii, stroke width, and the line/area/stacked-only props.
+ */
+export type BarVisualProps = Omit<StandardVisualProps, "dotRadius" | "strokeWidth" | "curved" | "curveType" | "showDots" | "fillOpacity" | "showStroke" | "stacked">;
+export interface BarChartProps extends BaseChartProps, StandardChartDisplayProps, BarVisualProps, BarChartSpecificProps {
     barRadius?: number;
     radius?: number;
     gap?: number;
@@ -23,27 +31,23 @@ export interface BarChartProps extends BaseChartProps, StandardChartDisplayProps
      */
     darkMode?: boolean;
 }
+/**
+ * Default values using centralized defaults.
+ *
+ * 🔴 Picks explicitly rather than spreading `...CHART_DEFAULTS` — the spread pulled in a
+ * default for every other chart's props, so this object advertised values BarChart never
+ * reads. List only what the component actually consumes.
+ */
 export declare const BAR_CHART_DEFAULTS: {
+    readonly showGrid: true;
+    readonly showXAxis: true;
+    readonly showYAxis: true;
+    readonly enableKeyboardNavigation: true;
     readonly variant: ChartVariant;
     readonly barRadius: 32;
-    readonly radius: 32;
     readonly gap: 4;
     readonly categoryGap: 16;
-    readonly orientation: "horizontal" | "vertical";
     readonly showAnimation: true;
     readonly showPlaceholderBars: false;
-    readonly showGrid: true;
-    readonly showXAxis: true;
-    readonly showYAxis: true;
-    readonly dotRadius: 3;
-    readonly cellRadius: 4;
-    readonly strokeWidth: 2;
-    readonly curved: true;
-    readonly curveType: import("..").CurveType;
-    readonly fillOpacity: 0.3;
-    readonly showStroke: true;
-    readonly showDots: false;
-    readonly stacked: false;
-    readonly enableKeyboardNavigation: true;
 };
 export type BarChartData = BaseChartProps["data"][0];

package/dist/components/charts/BarChart/BarChart.types.js

@@ -1,14 +1,21 @@
 import { CHART_DEFAULTS } from '../types/chart.types.js';
 
-// Default values using centralized defaults
+/**
+ * Default values using centralized defaults.
+ *
+ * 🔴 Picks explicitly rather than spreading `...CHART_DEFAULTS` — the spread pulled in a
+ * default for every other chart's props, so this object advertised values BarChart never
+ * reads. List only what the component actually consumes.
+ */
 const BAR_CHART_DEFAULTS = {
-    ...CHART_DEFAULTS,
+    showGrid: CHART_DEFAULTS.showGrid,
+    showXAxis: CHART_DEFAULTS.showXAxis,
+    showYAxis: CHART_DEFAULTS.showYAxis,
+    enableKeyboardNavigation: CHART_DEFAULTS.enableKeyboardNavigation,
     variant: "gradient", // BarChart defaults to the gradient (premium) look; use variant="default" for flat fills
     barRadius: CHART_DEFAULTS.barRadius, // 24 - For bar corner rounding
-    radius: CHART_DEFAULTS.barRadius, // @deprecated - kept for backward compatibility
     gap: CHART_DEFAULTS.gap,
     categoryGap: CHART_DEFAULTS.categoryGap,
-    orientation: CHART_DEFAULTS.orientation,
     showAnimation: true, // Enable animations by default
     showPlaceholderBars: false, // Opt-in feature for backward compatibility
 };

package/dist/components/charts/ComboChart/BarRenderer.js

@@ -7,14 +7,17 @@ import { BAR_STACK_CATEGORY_DELAY, BAR_STACK_SEGMENT_STAGGER, BAR_GROUP_STAGGER,
 import { BarGlowRing, barStrokeStyle, barFill, barDimOpacity, BarSeriesDefs, deriveBarEmphasis } from '../utils/bar-emphasis.js';
 
 /**
- * Simple color functions based on direct CSS variables
+ * Fallback only — ComboChart always resolves a real color onto each series'
+ * config before it reaches this renderer (see ComboChart.tsx), so this array
+ * exists purely as a defensive default, not a live categorical palette.
  */
 const getSeriesFillColor = (index) => {
     const colorVars = [
-        "var(--color-primary)",
-        "var(--color-warning)",
-        "var(--color-success)",
-        "var(--color-danger)",
+        "var(--color-chart-1)",
+        "var(--color-chart-2)",
+        "var(--color-chart-3)",
+        "var(--color-chart-4)",
+        "var(--color-chart-5)",
     ];
     return colorVars[index % colorVars.length];
 };
@@ -160,8 +163,8 @@ const BarRenderer = ({ data, config, scales, dimensions, margin, stacked = false
                     }, transition: isStacked
                         ? stackedSegmentTransition(delay, hasAnimated)
                         : barEntranceTransition(delay, hasAnimated), className: cn(isGradientVariant
-                        ? "transition-all duration-300 ease-out"
-                        : "transition-colors duration-200", !isNull && "cursor-pointer", isFocused && "stroke-2"), style: {
+                        ? "transition-all duration-[var(--motion-duration-slow)] ease-out"
+                        : "transition-colors duration-[var(--motion-duration-base)]", !isNull && "cursor-pointer", isFocused && "stroke-2"), style: {
                         originY: isNegative ? 0 : 1,
                         transformOrigin: isNegative ? "top" : "bottom",
                         fill: barFill({

package/dist/components/charts/ComboChart/BarRenderer.types.d.ts

@@ -1,7 +1,7 @@
 import type { ChartDataPoint, ChartConfig } from '../types/chart.types';
 import type { ChartScales } from '../hooks/useChartScales';
 import type { ChartDimensions, ChartMargins } from './ComboChartCanvas.types';
-import type { MultiSeriesData } from '../shared/ChartTooltip';
+import type { MultiSeriesData } from '../../core/shared/ChartTooltip';
 export interface BarRendererProps {
     /**
      * Chart data

package/dist/components/charts/ComboChart/ComboChart.js

@@ -8,7 +8,9 @@ import { BarRenderer } from './BarRenderer.js';
 import { LineRenderer } from './LineRenderer.js';
 import { resolveLegendPosition, ChartLegend } from '../shared/ChartLegend/ChartLegend.js';
 import { COMBO_CHART_DEFAULTS } from './ComboChart.types.js';
-import { ExclamationTriangleIcon } from '@heroicons/react/20/solid';
+import { Warning } from '@phosphor-icons/react';
+import { resolveSeriesColor } from '../utils/series-color.js';
+import { chartTheme } from '../theme/chart.theme.js';
 
 /**
  * Internal component that uses context to render bars and lines
@@ -16,31 +18,22 @@ import { ExclamationTriangleIcon } from '@heroicons/react/20/solid';
 const ComboChartRenderers = ({ barKeys, lineKeys, stacked, barRadius, dotRadius, gap, categoryGap, strokeWidth, curved, curveType, showDots, variant, }) => {
     const context = useComboChartContext();
     const dataKeys = Object.keys(context.config);
-    const seriesColors = [
-        "var(--color-primary)",
-        "var(--color-warning)",
-        "var(--color-success)",
-        "var(--color-danger)",
-    ];
+    const seriesColors = chartTheme.baseStyle.colors.series;
     // Filter configs for bars and lines, and assign colors
     const barConfig = {};
     barKeys.forEach((key) => {
-        var _a;
         const keyIndex = dataKeys.indexOf(key);
         barConfig[key] = {
             ...context.config[key],
-            color: ((_a = context.config[key]) === null || _a === void 0 ? void 0 : _a.color) ||
-                seriesColors[keyIndex % seriesColors.length],
+            color: resolveSeriesColor(keyIndex, key, context.config, seriesColors),
         };
     });
     const lineConfig = {};
     lineKeys.forEach((key) => {
-        var _a;
         const keyIndex = dataKeys.indexOf(key);
         lineConfig[key] = {
             ...context.config[key],
-            color: ((_a = context.config[key]) === null || _a === void 0 ? void 0 : _a.color) ||
-                seriesColors[keyIndex % seriesColors.length],
+            color: resolveSeriesColor(keyIndex, key, context.config, seriesColors),
         };
     });
     // Create custom tooltip handler that includes ALL series (bars + lines)
@@ -49,14 +42,14 @@ const ComboChartRenderers = ({ barKeys, lineKeys, stacked, barRadius, dotRadius,
         const dataItem = context.data.find((d) => d.name === content.category);
         if (dataItem) {
             const allSeries = dataKeys.map((key, idx) => {
-                var _a, _b;
+                var _a;
                 const value = dataItem[key];
                 const isNull = value === null || value === undefined || (typeof value === 'number' && !isFinite(value));
                 return {
                     key,
                     label: ((_a = context.config[key]) === null || _a === void 0 ? void 0 : _a.label) || key,
                     value: isNull ? 0 : value,
-                    color: ((_b = context.config[key]) === null || _b === void 0 ? void 0 : _b.color) || seriesColors[idx % seriesColors.length],
+                    color: resolveSeriesColor(idx, key, context.config, seriesColors),
                 };
             }).filter(series => {
                 // Only show series that have valid non-zero values
@@ -117,19 +110,13 @@ className, title, description, enableKeyboardNavigation = COMBO_CHART_DEFAULTS.e
         const seriesConfig = config[key];
         return (seriesConfig === null || seriesConfig === void 0 ? void 0 : seriesConfig.type) === "line";
     });
-    const legendSeriesColors = [
-        "var(--color-primary)",
-        "var(--color-warning)",
-        "var(--color-success)",
-        "var(--color-danger)",
-    ];
     const resolvedLegendPosition = resolveLegendPosition(legend, config || {});
     const legendItems = dataKeys.map((key, index) => {
-        var _a, _b, _c;
+        var _a, _b;
         return ({
             key,
             label: (_b = (_a = config[key]) === null || _a === void 0 ? void 0 : _a.label) !== null && _b !== void 0 ? _b : key,
-            color: ((_c = config[key]) === null || _c === void 0 ? void 0 : _c.color) || legendSeriesColors[index % legendSeriesColors.length],
+            color: resolveSeriesColor(index, key, config, chartTheme.baseStyle.colors.series),
         });
     });
     // Use unified chart scales hook
@@ -146,7 +133,7 @@ className, title, description, enableKeyboardNavigation = COMBO_CHART_DEFAULTS.e
     });
     // Early return for invalid data with error message
     if (!isValid) {
-        return (jsxs("div", { className: cn("chart-container flex flex-col items-center justify-center p-6 border border-border rounded-lg text-2xl", className), role: "alert", "aria-live": "polite", ...props, children: [jsxs("div", { className: "flex flex-col items-center gap-4 mb-1", children: [jsx(ExclamationTriangleIcon, { className: "size-6 text-[var(--color-danger)]" }), jsx("div", { className: "text-base font-semibold text-[var(--color-text-secondary)]", children: "Invalid Chart Data" })] }), jsx("p", { className: "text-base text-[var(--color-text-secondary)]/70 mb-2", children: "Unable to render chart due to data validation errors" })] }));
+        return (jsxs("div", { className: cn("chart-container flex flex-col items-center justify-center p-6 border border-border rounded-lg text-2xl", className), role: "alert", "aria-live": "polite", ...props, children: [jsxs("div", { className: "flex flex-col items-center gap-4 mb-1", children: [jsx(Warning, { className: "size-6 text-[var(--color-danger)]" }), jsx("div", { className: "text-base font-semibold text-[var(--color-text-secondary)]", children: "Invalid Chart Data" })] }), jsx("p", { className: "text-base text-[var(--color-text-secondary)]/70 mb-2", children: "Unable to render chart due to data validation errors" })] }));
     }
     // Early return for empty data
     if (!sanitizedData || sanitizedData.length === 0) {

package/dist/components/charts/ComboChart/ComboChart.types.d.ts

@@ -18,7 +18,7 @@ export interface ComboChartConfig extends ChartConfig {
 /**
  * ComboChart props - extends standard chart interfaces
  */
-export interface ComboChartProps extends BaseChartProps, StandardChartDisplayProps, Omit<StandardVisualProps, 'curved' | 'curveType' | 'showDots'> {
+export interface ComboChartProps extends BaseChartProps, StandardChartDisplayProps, Omit<StandardVisualProps, 'curved' | 'curveType' | 'showDots' | 'fillOpacity' | 'showStroke'> {
     config: ComboChartConfig;
     curved?: boolean;
     curveType?: 'linear' | 'monotone';
@@ -27,7 +27,21 @@ export interface ComboChartProps extends BaseChartProps, StandardChartDisplayPro
     showDualAxis?: boolean;
     showValidationWarnings?: boolean;
 }
+/**
+ * Default values using centralized defaults.
+ *
+ * 🔴 Picks explicitly rather than spreading `...CHART_DEFAULTS` — the spread pulled in a
+ * default for every other chart's props, so this object advertised values ComboChart
+ * never reads. List only what the component actually consumes.
+ */
 export declare const COMBO_CHART_DEFAULTS: {
+    readonly showGrid: true;
+    readonly showXAxis: true;
+    readonly showYAxis: true;
+    readonly enableKeyboardNavigation: true;
+    readonly strokeWidth: 2;
+    readonly gap: 4;
+    readonly stacked: false;
     readonly barRadius: 32;
     readonly dotRadius: 3;
     readonly categoryGap: 16;
@@ -36,16 +50,4 @@ export declare const COMBO_CHART_DEFAULTS: {
     readonly showDots: false;
     readonly showDualAxis: false;
     readonly variant: ChartVariant;
-    readonly showGrid: true;
-    readonly showXAxis: true;
-    readonly showYAxis: true;
-    readonly cellRadius: 4;
-    readonly radius: 20;
-    readonly strokeWidth: 2;
-    readonly gap: 4;
-    readonly fillOpacity: 0.3;
-    readonly showStroke: true;
-    readonly orientation: "vertical" | "horizontal";
-    readonly stacked: false;
-    readonly enableKeyboardNavigation: true;
 };

package/dist/components/charts/ComboChart/ComboChart.types.js

@@ -1,8 +1,20 @@
 import { CHART_DEFAULTS } from '../types/chart.types.js';
 
-// Default values using centralized defaults
+/**
+ * Default values using centralized defaults.
+ *
+ * 🔴 Picks explicitly rather than spreading `...CHART_DEFAULTS` — the spread pulled in a
+ * default for every other chart's props, so this object advertised values ComboChart
+ * never reads. List only what the component actually consumes.
+ */
 const COMBO_CHART_DEFAULTS = {
-    ...CHART_DEFAULTS,
+    showGrid: CHART_DEFAULTS.showGrid,
+    showXAxis: CHART_DEFAULTS.showXAxis,
+    showYAxis: CHART_DEFAULTS.showYAxis,
+    enableKeyboardNavigation: CHART_DEFAULTS.enableKeyboardNavigation,
+    strokeWidth: CHART_DEFAULTS.strokeWidth,
+    gap: CHART_DEFAULTS.gap,
+    stacked: CHART_DEFAULTS.stacked,
     barRadius: CHART_DEFAULTS.barRadius, // 24 - For bar corners
     dotRadius: CHART_DEFAULTS.dotRadius, // 20 - For line markers
     categoryGap: CHART_DEFAULTS.categoryGap,

package/dist/components/charts/ComboChart/ComboChartCanvas.js

@@ -9,7 +9,7 @@ import { ChartErrorBoundary } from '../shared/ChartErrorBoundary/ChartErrorBound
 import { XAxis } from '../shared/ChartAxis/XAxis.js';
 import { YAxis } from '../shared/ChartAxis/YAxis.js';
 import { HorizontalGrid } from '../shared/ChartGrid/HorizontalGrid.js';
-import { ChartTooltip } from '../shared/ChartTooltip/ChartTooltip.js';
+import { ChartTooltip } from '../../core/shared/ChartTooltip/ChartTooltip.js';
 
 const ComboChartContext = createContext(null);
 /**

package/dist/components/charts/ComboChart/LineRenderer.js

@@ -9,26 +9,31 @@ import { generateCurvePath } from '../utils/path-utils.js';
 import { RISE_FROM_BASELINE, riseTransition, SCALE_FROM_CENTER, dotEntranceTransition } from '../utils/animation-utils.js';
 
 /**
- * Get series color class
+ * Get series color class.
+ * Fallback only — ComboChart always resolves a real color onto each series'
+ * config before it reaches this renderer (see ComboChart.tsx).
  */
 const getSeriesColorClass = (index) => {
     const colors = [
-        "stroke-[var(--color-primary)]",
-        "stroke-[var(--color-warning)]",
-        "stroke-[var(--color-success)]",
-        "stroke-[var(--color-danger)]",
+        "stroke-[var(--color-chart-1)]",
+        "stroke-[var(--color-chart-2)]",
+        "stroke-[var(--color-chart-3)]",
+        "stroke-[var(--color-chart-4)]",
+        "stroke-[var(--color-chart-5)]",
     ];
     return colors[index % colors.length];
 };
 /**
- * Get color value for SVG elements and tooltips
+ * Get color value for SVG elements and tooltips.
+ * Fallback only — see getSeriesColorClass above.
  */
 const getSeriesFillColor = (index) => {
     const colorVars = [
-        "var(--color-primary)",
-        "var(--color-warning)",
-        "var(--color-success)",
-        "var(--color-danger)",
+        "var(--color-chart-1)",
+        "var(--color-chart-2)",
+        "var(--color-chart-3)",
+        "var(--color-chart-4)",
+        "var(--color-chart-5)",
     ];
     return colorVars[index % colorVars.length];
 };
@@ -101,7 +106,7 @@ const LineRenderer = ({ data, config, scales, dimensions, margin, strokeWidth =
             const appliedColorClass = customColor ? undefined : fallbackClass;
             return (jsxs("g", { children: [jsxs(motion.g, { initial: { scaleY: shouldReduceMotion ? 1 : 0 }, animate: { scaleY: 1 }, transition: riseTransition, style: RISE_FROM_BASELINE, children: [jsx("rect", { x: margin.left, y: margin.top, width: innerWidth, height: innerHeight, fill: "none", stroke: "none", style: { pointerEvents: "none" } }), jsx("path", { d: targetPath, fill: "none", strokeWidth: strokeWidth, strokeLinecap: "round", 
                                 // Constant stroke width through the scaleY rise (no squash).
-                                vectorEffect: "non-scaling-stroke", className: cn("transition-all duration-200", appliedColorClass), stroke: customColor, filter: "url(#line-shadow)" })] }, shouldReduceMotion ? "line-static" : dataSignature), showDots &&
+                                vectorEffect: "non-scaling-stroke", className: cn("transition-all duration-[var(--motion-duration-base)]", appliedColorClass), stroke: customColor, filter: "url(#line-shadow)" })] }, shouldReduceMotion ? "line-static" : dataSignature), showDots &&
                         data.map((item, dataIndex) => {
                             const rawValue = item[key];
                             const isNull = rawValue === null || rawValue === undefined;

package/dist/components/charts/ComboChart/LineRenderer.types.d.ts

@@ -1,7 +1,7 @@
 import type { ChartDataPoint, ChartConfig, CurveType } from '../types/chart.types';
 import type { ChartScales } from '../hooks/useChartScales';
 import type { ChartDimensions, ChartMargins } from './ComboChartCanvas.types';
-import type { MultiSeriesData } from '../shared/ChartTooltip';
+import type { MultiSeriesData } from '../../core/shared/ChartTooltip';
 export interface LineRendererProps {
     /**
      * Chart data

package/dist/components/charts/DonutChart/DonutChart.theme.js

@@ -3,15 +3,15 @@
  */
 const donutChartTheme = {
     // Base container styles
-    baseStyle: "relative flex items-center justify-center transition-all duration-300 cursor-default",
+    baseStyle: "relative flex items-center justify-center transition-all duration-[var(--motion-duration-slow)] cursor-default",
     // Background track styling
     track: "text-[var(--color-border)]",
     // Progress arc styling
-    progress: "text-[var(--color-primary)] transition-all duration-300",
+    progress: "text-[var(--color-primary)] transition-all duration-[var(--motion-duration-slow)]",
     // Text styling
     text: {
-        label: "text-xl font-bold text-[var(--color-text-primary)] transition-colors duration-300 tracking-tight",
-        subtitle: "text-sm text-[var(--color-text-secondary)] mt-1 transition-colors duration-300",
+        label: "text-xl font-bold text-[var(--color-text-primary)] transition-colors duration-[var(--motion-duration-slow)] tracking-tight",
+        subtitle: "text-sm text-[var(--color-text-secondary)] mt-1 transition-colors duration-[var(--motion-duration-slow)]",
     },
     // Default color palette (CSS variables for dark mode support)
     // Note: These are resolved at runtime via getComputedStyle

package/dist/components/charts/DonutChart/MultiSegmentDonutChart.theme.js

@@ -3,7 +3,7 @@
  */
 const multiSegmentDonutChartTheme = {
     // Base container styles
-    baseStyle: "relative flex items-center justify-center transition-all duration-300",
+    baseStyle: "relative flex items-center justify-center transition-all duration-[var(--motion-duration-slow)]",
     // Background track styling
     track: "text-[var(--color-border)]",
     // Legend styling