@sqlrooms/mosaic 0.29.0-rc.8 → 0.29.0-rc.9

package/dist/charts/chart-types/line-chart/tool.d.ts

@@ -1,9 +1,8 @@
 import { z } from 'zod';
-import type { DashboardToolDeps } from '../base-types';
-export declare const LineChartToolParameters: z.ZodObject<{
-    artifactId: z.ZodOptional<z.ZodString>;
-    tableName: z.ZodOptional<z.ZodString>;
-    createArtifactIfMissing: z.ZodDefault<z.ZodOptional<z.ZodBoolean>>;
+import { ChartToolParams, ChartToolOutput } from '../tool-types';
+export declare const LineChartToolInput: z.ZodObject<{
+    tableName: z.ZodString;
+    title: z.ZodOptional<z.ZodString>;
     panelId: z.ZodOptional<z.ZodString>;
     reasoning: z.ZodString;
     settings: z.ZodObject<{
@@ -31,9 +30,9 @@ export declare const LineChartToolParameters: z.ZodObject<{
         showLegend: z.ZodNonOptional<z.ZodDefault<z.ZodOptional<z.ZodBoolean>>>;
     }, z.core.$strip>;
 }, z.core.$strip>;
-export type LineChartToolParams = z.infer<typeof LineChartToolParameters>;
-export declare function createLineChartAiTool(deps: DashboardToolDeps): import("ai").Tool<{
-    createArtifactIfMissing: boolean;
+export type LineChartToolInput = z.infer<typeof LineChartToolInput>;
+export declare function createLineChartAiTool({ databaseAdapter, addChart, maxDataPoints, }: ChartToolParams): import("ai").Tool<{
+    tableName: string;
     reasoning: string;
     settings: {
         x: string;
@@ -45,22 +44,25 @@ export declare function createLineChartAiTool(deps: DashboardToolDeps): import("
         }[];
         showLegend: boolean;
     };
-    artifactId?: string | undefined;
-    tableName?: string | undefined;
+    title?: string | undefined;
     panelId?: string | undefined;
-}, {
-    llmResult: {
-        success: boolean;
-        details: string;
-        data: import("..").PanelResult;
-        errorMessage?: undefined;
-    };
-} | {
-    llmResult: {
-        success: boolean;
-        errorMessage: string;
-        details?: undefined;
-        data?: undefined;
+}, ChartToolOutput<{
+    chartType: "line-chart";
+    settings: {
+        showLegend: boolean;
+        x?: string | undefined;
+        xInterval?: "second" | "minute" | "hour" | "day" | "week" | "month" | "quarter" | "year" | undefined;
+        yFields?: {
+            field: string;
+            aggregate: "sum" | "max" | "min" | "avg";
+            color?: string | undefined;
+        }[] | undefined;
     };
-}>;
+    settingsOpen?: boolean | undefined;
+    dataPolicy?: {
+        disabled?: boolean | undefined;
+        maxRows?: number | undefined;
+        reason?: string | undefined;
+    } | undefined;
+}>>;
 //# sourceMappingURL=tool.d.ts.map

package/dist/charts/chart-types/line-chart/tool.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"tool.d.ts","sourceRoot":"","sources":["../../../../src/charts/chart-types/line-chart/tool.ts"],"names":[],"mappings":"AACA,OAAO,EAAC,CAAC,EAAC,MAAM,KAAK,CAAC;AAItB,OAAO,KAAK,EAAC,iBAAiB,EAAC,MAAM,eAAe,CAAC;AAYrD,eAAO,MAAM,uBAAuB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;iBAElC,CAAC;AAEH,MAAM,MAAM,mBAAmB,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,uBAAuB,CAAC,CAAC;AAE1E,wBAAgB,qBAAqB,CAAC,IAAI,EAAE,iBAAiB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAkF5D"}
+{"version":3,"file":"tool.d.ts","sourceRoot":"","sources":["../../../../src/charts/chart-types/line-chart/tool.ts"],"names":[],"mappings":"AACA,OAAO,EAAC,CAAC,EAAC,MAAM,KAAK,CAAC;AAStB,OAAO,EAAC,eAAe,EAAE,eAAe,EAAC,MAAM,eAAe,CAAC;AAO/D,eAAO,MAAM,kBAAkB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;iBAE7B,CAAC;AAEH,MAAM,MAAM,kBAAkB,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,kBAAkB,CAAC,CAAC;AAEpE,wBAAgB,qBAAqB,CAAC,EACpC,eAAe,EACf,QAAQ,EACR,aAAa,GACd,EAAE,eAAe;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;IAoDjB"}

package/dist/charts/chart-types/line-chart/tool.js

@@ -1,16 +1,16 @@
 import { tool } from 'ai';
 import { LineChartSettings } from './schema';
 import { AggregateFunction, TemporalInterval } from '../../../schemas';
-import { BaseChartToolParameters } from '../../../ai/tool-schemas';
-import { validateColumnExists } from '../../../ai/tool-validation';
+import { BaseChartToolInput } from '../../../ai/tool-schemas';
 import { NUMERIC_COLUMN_TYPES, QUANTITATIVE_COLUMN_TYPES, TEMPORAL_COLUMN_TYPES, } from '../../../column-types-utils';
-import { createOrUpdateChartPanel } from '../../../ai/tool-helpers';
+import { validateLineChartSettings } from './validation';
+import { ensureTable } from '../../../ai/tool-helpers';
 const AGGREGATE_FUNCTIONS = AggregateFunction.options;
 const TEMPORAL_INTERVALS = TemporalInterval.options;
-export const LineChartToolParameters = BaseChartToolParameters.extend({
+export const LineChartToolInput = BaseChartToolInput.extend({
     settings: LineChartSettings.required(),
 });
-export function createLineChartAiTool(deps) {
+export function createLineChartAiTool({ databaseAdapter, addChart, maxDataPoints, }) {
     return tool({
         description: `Line chart: shows trends and changes over time or ordered continuous variable. Connects data points to show progression.
 
@@ -24,49 +24,37 @@ Required:
 Optional: xInterval for temporal grouping (${TEMPORAL_INTERVALS.join(', ')}) when x is temporal (${TEMPORAL_COLUMN_TYPES.join(', ')}).
 Multiple yFields create multi-line chart for comparing metrics.
 
-NOTE: Line charts with aggregation (xInterval or aggregate functions) handle large datasets well. Without aggregation, line charts plot individual points and should not be used for tables with more than ${deps.maxDataPoints.toLocaleString()} rows - use aggregated visualizations instead.
-
-To UPDATE an existing line chart: provide the panelId parameter. Otherwise creates new panel.
+NOTE: Line charts with aggregation (xInterval or aggregate functions) handle large datasets well. Without aggregation, line charts plot individual points and should not be used for tables with more than ${maxDataPoints.toLocaleString()} rows - use aggregated visualizations instead.
 
 Do NOT use for: single point distributions (use histogram), categorical counts (use count-plot), two-variable correlations (use scatter-plot).`,
-        inputSchema: LineChartToolParameters,
-        execute: async (params, context) => {
+        inputSchema: LineChartToolInput,
+        execute: async ({ tableName, title, settings, panelId }) => {
             try {
-                const artifactId = deps.resolveArtifact(params.artifactId, params.createArtifactIfMissing, context);
-                const { tableName, columns } = deps.resolveTable(artifactId, params.tableName);
-                // Validate settings
-                validateColumnExists(params.settings.x, QUANTITATIVE_COLUMN_TYPES, columns, 'x');
-                for (const yField of params.settings.yFields) {
-                    validateColumnExists(yField.field, NUMERIC_COLUMN_TYPES, columns, 'yFields');
-                }
-                const result = createOrUpdateChartPanel(deps, {
-                    panelId: params.panelId,
-                    dashboardId: artifactId,
+                const dataTable = ensureTable(databaseAdapter, tableName);
+                validateLineChartSettings({
+                    dataTable,
+                    settings,
+                });
+                const chartConfig = {
+                    chartType: 'line-chart',
+                    settings,
+                };
+                await addChart({
                     tableName,
-                    title: params.settings.x
-                        ? `Line chart - ${params.settings.yFields?.map((f) => f.field).join(', ') || ''} over ${params.settings.x}`
-                        : 'Line chart',
-                    config: {
-                        chartType: 'line-chart',
-                        settings: params.settings,
-                    },
+                    panelId,
+                    title,
+                    config: chartConfig,
                 });
                 return {
-                    llmResult: {
-                        success: true,
-                        details: params.panelId
-                            ? `Updated line chart "${result.title}".`
-                            : `Created line chart "${result.title}".`,
-                        data: result,
-                    },
+                    success: true,
+                    details: `Generated line chart configuration.`,
+                    data: chartConfig,
                 };
             }
             catch (error) {
                 return {
-                    llmResult: {
-                        success: false,
-                        errorMessage: error instanceof Error ? error.message : String(error),
-                    },
+                    success: false,
+                    errorMessage: error instanceof Error ? error.message : String(error),
                 };
             }
         },

package/dist/charts/chart-types/line-chart/tool.js.map

@@ -1 +1 @@
-{"version":3,"file":"tool.js","sourceRoot":"","sources":["../../../../src/charts/chart-types/line-chart/tool.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,IAAI,EAAC,MAAM,IAAI,CAAC;AAExB,OAAO,EAAC,iBAAiB,EAAC,MAAM,UAAU,CAAC;AAC3C,OAAO,EAAC,iBAAiB,EAAE,gBAAgB,EAAC,MAAM,kBAAkB,CAAC;AACrE,OAAO,EAAC,uBAAuB,EAAC,MAAM,0BAA0B,CAAC;AAEjE,OAAO,EAAC,oBAAoB,EAAC,MAAM,6BAA6B,CAAC;AACjE,OAAO,EACL,oBAAoB,EACpB,yBAAyB,EACzB,qBAAqB,GACtB,MAAM,6BAA6B,CAAC;AACrC,OAAO,EAAC,wBAAwB,EAAC,MAAM,0BAA0B,CAAC;AAElE,MAAM,mBAAmB,GAAG,iBAAiB,CAAC,OAAO,CAAC;AACtD,MAAM,kBAAkB,GAAG,gBAAgB,CAAC,OAAO,CAAC;AAEpD,MAAM,CAAC,MAAM,uBAAuB,GAAG,uBAAuB,CAAC,MAAM,CAAC;IACpE,QAAQ,EAAE,iBAAiB,CAAC,QAAQ,EAAE;CACvC,CAAC,CAAC;AAIH,MAAM,UAAU,qBAAqB,CAAC,IAAuB;IAC3D,OAAO,IAAI,CAAC;QACV,WAAW,EAAE;;;;;;4BAMW,yBAAyB,CAAC,IAAI,CAAC,IAAI,CAAC;+CACjB,oBAAoB,CAAC,IAAI,CAAC,IAAI,CAAC,kBAAkB,mBAAmB,CAAC,IAAI,CAAC,GAAG,CAAC;;6CAEhF,kBAAkB,CAAC,IAAI,CAAC,IAAI,CAAC,yBAAyB,qBAAqB,CAAC,IAAI,CAAC,IAAI,CAAC;;;6MAG0E,IAAI,CAAC,aAAa,CAAC,cAAc,EAAE;;;;+IAIjG;QAC3I,WAAW,EAAE,uBAAuB;QACpC,OAAO,EAAE,KAAK,EAAE,MAAM,EAAE,OAAO,EAAE,EAAE;YACjC,IAAI,CAAC;gBACH,MAAM,UAAU,GAAG,IAAI,CAAC,eAAe,CACrC,MAAM,CAAC,UAAU,EACjB,MAAM,CAAC,uBAAuB,EAC9B,OAAO,CACR,CAAC;gBACF,MAAM,EAAC,SAAS,EAAE,OAAO,EAAC,GAAG,IAAI,CAAC,YAAY,CAC5C,UAAU,EACV,MAAM,CAAC,SAAS,CACjB,CAAC;gBAEF,oBAAoB;gBACpB,oBAAoB,CAClB,MAAM,CAAC,QAAQ,CAAC,CAAC,EACjB,yBAAyB,EACzB,OAAO,EACP,GAAG,CACJ,CAAC;gBAEF,KAAK,MAAM,MAAM,IAAI,MAAM,CAAC,QAAQ,CAAC,OAAO,EAAE,CAAC;oBAC7C,oBAAoB,CAClB,MAAM,CAAC,KAAK,EACZ,oBAAoB,EACpB,OAAO,EACP,SAAS,CACV,CAAC;gBACJ,CAAC;gBAED,MAAM,MAAM,GAAG,wBAAwB,CAAC,IAAI,EAAE;oBAC5C,OAAO,EAAE,MAAM,CAAC,OAAO;oBACvB,WAAW,EAAE,UAAU;oBACvB,SAAS;oBACT,KAAK,EAAE,MAAM,CAAC,QAAQ,CAAC,CAAC;wBACtB,CAAC,CAAC,gBAAgB,MAAM,CAAC,QAAQ,CAAC,OAAO,EAAE,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,SAAS,MAAM,CAAC,QAAQ,CAAC,CAAC,EAAE;wBAC3G,CAAC,CAAC,YAAY;oBAChB,MAAM,EAAE;wBACN,SAAS,EAAE,YAAY;wBACvB,QAAQ,EAAE,MAAM,CAAC,QAAQ;qBAC1B;iBACF,CAAC,CAAC;gBAEH,OAAO;oBACL,SAAS,EAAE;wBACT,OAAO,EAAE,IAAI;wBACb,OAAO,EAAE,MAAM,CAAC,OAAO;4BACrB,CAAC,CAAC,uBAAuB,MAAM,CAAC,KAAK,IAAI;4BACzC,CAAC,CAAC,uBAAuB,MAAM,CAAC,KAAK,IAAI;wBAC3C,IAAI,EAAE,MAAM;qBACb;iBACF,CAAC;YACJ,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,OAAO;oBACL,SAAS,EAAE;wBACT,OAAO,EAAE,KAAK;wBACd,YAAY,EACV,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC;qBACzD;iBACF,CAAC;YACJ,CAAC;QACH,CAAC;KACF,CAAC,CAAC;AACL,CAAC","sourcesContent":["import {tool} from 'ai';\nimport {z} from 'zod';\nimport {LineChartSettings} from './schema';\nimport {AggregateFunction, TemporalInterval} from '../../../schemas';\nimport {BaseChartToolParameters} from '../../../ai/tool-schemas';\nimport type {DashboardToolDeps} from '../base-types';\nimport {validateColumnExists} from '../../../ai/tool-validation';\nimport {\n  NUMERIC_COLUMN_TYPES,\n  QUANTITATIVE_COLUMN_TYPES,\n  TEMPORAL_COLUMN_TYPES,\n} from '../../../column-types-utils';\nimport {createOrUpdateChartPanel} from '../../../ai/tool-helpers';\n\nconst AGGREGATE_FUNCTIONS = AggregateFunction.options;\nconst TEMPORAL_INTERVALS = TemporalInterval.options;\n\nexport const LineChartToolParameters = BaseChartToolParameters.extend({\n  settings: LineChartSettings.required(),\n});\n\nexport type LineChartToolParams = z.infer<typeof LineChartToolParameters>;\n\nexport function createLineChartAiTool(deps: DashboardToolDeps) {\n  return tool({\n    description: `Line chart: shows trends and changes over time or ordered continuous variable. Connects data points to show progression.\n\nUse when: user asks about \"trend\", \"over time\", \"changes in\", \"time series\", \"progression of\", \"track X over Y\".\nExample queries: \"population growth over time\", \"temperature trend by month\", \"show land development over years\", \"elevation changes along route\", \"average precipitation by season\".\n\nRequired:\n- x: quantitative column (${QUANTITATIVE_COLUMN_TYPES.join(', ')})\n- yFields: array of {field: string (numeric: ${NUMERIC_COLUMN_TYPES.join(', ')}), aggregate?: ${AGGREGATE_FUNCTIONS.join('|')}}\n\nOptional: xInterval for temporal grouping (${TEMPORAL_INTERVALS.join(', ')}) when x is temporal (${TEMPORAL_COLUMN_TYPES.join(', ')}).\nMultiple yFields create multi-line chart for comparing metrics.\n\nNOTE: Line charts with aggregation (xInterval or aggregate functions) handle large datasets well. Without aggregation, line charts plot individual points and should not be used for tables with more than ${deps.maxDataPoints.toLocaleString()} rows - use aggregated visualizations instead.\n\nTo UPDATE an existing line chart: provide the panelId parameter. Otherwise creates new panel.\n\nDo NOT use for: single point distributions (use histogram), categorical counts (use count-plot), two-variable correlations (use scatter-plot).`,\n    inputSchema: LineChartToolParameters,\n    execute: async (params, context) => {\n      try {\n        const artifactId = deps.resolveArtifact(\n          params.artifactId,\n          params.createArtifactIfMissing,\n          context,\n        );\n        const {tableName, columns} = deps.resolveTable(\n          artifactId,\n          params.tableName,\n        );\n\n        // Validate settings\n        validateColumnExists(\n          params.settings.x,\n          QUANTITATIVE_COLUMN_TYPES,\n          columns,\n          'x',\n        );\n\n        for (const yField of params.settings.yFields) {\n          validateColumnExists(\n            yField.field,\n            NUMERIC_COLUMN_TYPES,\n            columns,\n            'yFields',\n          );\n        }\n\n        const result = createOrUpdateChartPanel(deps, {\n          panelId: params.panelId,\n          dashboardId: artifactId,\n          tableName,\n          title: params.settings.x\n            ? `Line chart - ${params.settings.yFields?.map((f) => f.field).join(', ') || ''} over ${params.settings.x}`\n            : 'Line chart',\n          config: {\n            chartType: 'line-chart',\n            settings: params.settings,\n          },\n        });\n\n        return {\n          llmResult: {\n            success: true,\n            details: params.panelId\n              ? `Updated line chart \"${result.title}\".`\n              : `Created line chart \"${result.title}\".`,\n            data: result,\n          },\n        };\n      } catch (error) {\n        return {\n          llmResult: {\n            success: false,\n            errorMessage:\n              error instanceof Error ? error.message : String(error),\n          },\n        };\n      }\n    },\n  });\n}\n"]}
+{"version":3,"file":"tool.js","sourceRoot":"","sources":["../../../../src/charts/chart-types/line-chart/tool.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,IAAI,EAAC,MAAM,IAAI,CAAC;AAExB,OAAO,EAAkB,iBAAiB,EAAC,MAAM,UAAU,CAAC;AAC5D,OAAO,EAAC,iBAAiB,EAAE,gBAAgB,EAAC,MAAM,kBAAkB,CAAC;AACrE,OAAO,EAAC,kBAAkB,EAAC,MAAM,0BAA0B,CAAC;AAC5D,OAAO,EACL,oBAAoB,EACpB,yBAAyB,EACzB,qBAAqB,GACtB,MAAM,6BAA6B,CAAC;AAErC,OAAO,EAAC,yBAAyB,EAAC,MAAM,cAAc,CAAC;AACvD,OAAO,EAAC,WAAW,EAAC,MAAM,0BAA0B,CAAC;AAErD,MAAM,mBAAmB,GAAG,iBAAiB,CAAC,OAAO,CAAC;AACtD,MAAM,kBAAkB,GAAG,gBAAgB,CAAC,OAAO,CAAC;AAEpD,MAAM,CAAC,MAAM,kBAAkB,GAAG,kBAAkB,CAAC,MAAM,CAAC;IAC1D,QAAQ,EAAE,iBAAiB,CAAC,QAAQ,EAAE;CACvC,CAAC,CAAC;AAIH,MAAM,UAAU,qBAAqB,CAAC,EACpC,eAAe,EACf,QAAQ,EACR,aAAa,GACG;IAChB,OAAO,IAAI,CAAuD;QAChE,WAAW,EAAE;;;;;;4BAMW,yBAAyB,CAAC,IAAI,CAAC,IAAI,CAAC;+CACjB,oBAAoB,CAAC,IAAI,CAAC,IAAI,CAAC,kBAAkB,mBAAmB,CAAC,IAAI,CAAC,GAAG,CAAC;;6CAEhF,kBAAkB,CAAC,IAAI,CAAC,IAAI,CAAC,yBAAyB,qBAAqB,CAAC,IAAI,CAAC,IAAI,CAAC;;;6MAG0E,aAAa,CAAC,cAAc,EAAE;;+IAE5F;QAC3I,WAAW,EAAE,kBAAkB;QAC/B,OAAO,EAAE,KAAK,EAAE,EAAC,SAAS,EAAE,KAAK,EAAE,QAAQ,EAAE,OAAO,EAAC,EAAE,EAAE;YACvD,IAAI,CAAC;gBACH,MAAM,SAAS,GAAG,WAAW,CAAC,eAAe,EAAE,SAAS,CAAC,CAAC;gBAE1D,yBAAyB,CAAC;oBACxB,SAAS;oBACT,QAAQ;iBACT,CAAC,CAAC;gBAEH,MAAM,WAAW,GAAoB;oBACnC,SAAS,EAAE,YAAqB;oBAChC,QAAQ;iBACT,CAAC;gBAEF,MAAM,QAAQ,CAAC;oBACb,SAAS;oBACT,OAAO;oBACP,KAAK;oBACL,MAAM,EAAE,WAAW;iBACpB,CAAC,CAAC;gBAEH,OAAO;oBACL,OAAO,EAAE,IAAI;oBACb,OAAO,EAAE,qCAAqC;oBAC9C,IAAI,EAAE,WAAW;iBAClB,CAAC;YACJ,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,OAAO;oBACL,OAAO,EAAE,KAAK;oBACd,YAAY,EAAE,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC;iBACrE,CAAC;YACJ,CAAC;QACH,CAAC;KACF,CAAC,CAAC;AACL,CAAC","sourcesContent":["import {tool} from 'ai';\nimport {z} from 'zod';\nimport {LineChartConfig, LineChartSettings} from './schema';\nimport {AggregateFunction, TemporalInterval} from '../../../schemas';\nimport {BaseChartToolInput} from '../../../ai/tool-schemas';\nimport {\n  NUMERIC_COLUMN_TYPES,\n  QUANTITATIVE_COLUMN_TYPES,\n  TEMPORAL_COLUMN_TYPES,\n} from '../../../column-types-utils';\nimport {ChartToolParams, ChartToolOutput} from '../tool-types';\nimport {validateLineChartSettings} from './validation';\nimport {ensureTable} from '../../../ai/tool-helpers';\n\nconst AGGREGATE_FUNCTIONS = AggregateFunction.options;\nconst TEMPORAL_INTERVALS = TemporalInterval.options;\n\nexport const LineChartToolInput = BaseChartToolInput.extend({\n  settings: LineChartSettings.required(),\n});\n\nexport type LineChartToolInput = z.infer<typeof LineChartToolInput>;\n\nexport function createLineChartAiTool({\n  databaseAdapter,\n  addChart,\n  maxDataPoints,\n}: ChartToolParams) {\n  return tool<LineChartToolInput, ChartToolOutput<LineChartConfig>>({\n    description: `Line chart: shows trends and changes over time or ordered continuous variable. Connects data points to show progression.\n\nUse when: user asks about \"trend\", \"over time\", \"changes in\", \"time series\", \"progression of\", \"track X over Y\".\nExample queries: \"population growth over time\", \"temperature trend by month\", \"show land development over years\", \"elevation changes along route\", \"average precipitation by season\".\n\nRequired:\n- x: quantitative column (${QUANTITATIVE_COLUMN_TYPES.join(', ')})\n- yFields: array of {field: string (numeric: ${NUMERIC_COLUMN_TYPES.join(', ')}), aggregate?: ${AGGREGATE_FUNCTIONS.join('|')}}\n\nOptional: xInterval for temporal grouping (${TEMPORAL_INTERVALS.join(', ')}) when x is temporal (${TEMPORAL_COLUMN_TYPES.join(', ')}).\nMultiple yFields create multi-line chart for comparing metrics.\n\nNOTE: Line charts with aggregation (xInterval or aggregate functions) handle large datasets well. Without aggregation, line charts plot individual points and should not be used for tables with more than ${maxDataPoints.toLocaleString()} rows - use aggregated visualizations instead.\n\nDo NOT use for: single point distributions (use histogram), categorical counts (use count-plot), two-variable correlations (use scatter-plot).`,\n    inputSchema: LineChartToolInput,\n    execute: async ({tableName, title, settings, panelId}) => {\n      try {\n        const dataTable = ensureTable(databaseAdapter, tableName);\n\n        validateLineChartSettings({\n          dataTable,\n          settings,\n        });\n\n        const chartConfig: LineChartConfig = {\n          chartType: 'line-chart' as const,\n          settings,\n        };\n\n        await addChart({\n          tableName,\n          panelId,\n          title,\n          config: chartConfig,\n        });\n\n        return {\n          success: true,\n          details: `Generated line chart configuration.`,\n          data: chartConfig,\n        };\n      } catch (error) {\n        return {\n          success: false,\n          errorMessage: error instanceof Error ? error.message : String(error),\n        };\n      }\n    },\n  });\n}\n"]}

package/dist/charts/chart-types/line-chart/validation.d.ts

@@ -0,0 +1,16 @@
+import { LineChartSettings } from './schema';
+import { ValidateSpecOptions } from '../base-types';
+import { TableColumn } from '@sqlrooms/duckdb';
+import { AggregateFunction, TemporalInterval } from '../../../schemas';
+export type ValidatedLineChartSettings = {
+    xColumn: TableColumn;
+    yColumns: {
+        field: string;
+        column: TableColumn;
+        aggregate: AggregateFunction;
+        color: string;
+    }[];
+    xInterval?: TemporalInterval;
+};
+export declare function validateLineChartSettings({ dataTable, settings: { x, yFields, xInterval }, }: ValidateSpecOptions<LineChartSettings>): ValidatedLineChartSettings;
+//# sourceMappingURL=validation.d.ts.map

package/dist/charts/chart-types/line-chart/validation.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"validation.d.ts","sourceRoot":"","sources":["../../../../src/charts/chart-types/line-chart/validation.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,iBAAiB,EAAC,MAAM,UAAU,CAAC;AAC3C,OAAO,EAAC,mBAAmB,EAAC,MAAM,eAAe,CAAC;AAOlD,OAAO,EAAC,WAAW,EAAC,MAAM,kBAAkB,CAAC;AAC7C,OAAO,EAAC,iBAAiB,EAAE,gBAAgB,EAAC,MAAM,kBAAkB,CAAC;AAGrE,MAAM,MAAM,0BAA0B,GAAG;IACvC,OAAO,EAAE,WAAW,CAAC;IACrB,QAAQ,EAAE;QACR,KAAK,EAAE,MAAM,CAAC;QACd,MAAM,EAAE,WAAW,CAAC;QACpB,SAAS,EAAE,iBAAiB,CAAC;QAC7B,KAAK,EAAE,MAAM,CAAC;KACf,EAAE,CAAC;IACJ,SAAS,CAAC,EAAE,gBAAgB,CAAC;CAC9B,CAAC;AAEF,wBAAgB,yBAAyB,CAAC,EACxC,SAAS,EACT,QAAQ,EAAE,EAAC,CAAC,EAAE,OAAY,EAAE,SAAS,EAAC,GACvC,EAAE,mBAAmB,CAAC,iBAAiB,CAAC,GAAG,0BAA0B,CAgDrE"}

package/dist/charts/chart-types/line-chart/validation.js

@@ -0,0 +1,43 @@
+import { InvalidColumnTypeError, MissingColumnsError, RequiredFieldsError, } from '../errors';
+import { isNumericType, isQuantitativeType } from '../../../column-types-utils';
+import { DEFAULT_CHART_FALLBACK_COLOR } from '../../../constants/chart-colors';
+export function validateLineChartSettings({ dataTable, settings: { x, yFields = [], xInterval }, }) {
+    // Basic validation for required fields
+    if (!x || yFields.length === 0) {
+        throw new RequiredFieldsError([
+            ...(x ? [] : ['X-axis']),
+            ...(yFields.length > 0 ? [] : ['Y-axis']),
+        ]);
+    }
+    // Validate X and Y field existence
+    const xColumn = dataTable.columns.find((col) => col.name === x);
+    const yColumns = yFields.map((y) => ({
+        field: y.field,
+        column: dataTable.columns.find((col) => col.name === y.field),
+        aggregate: y.aggregate ?? 'sum',
+        color: y.color ?? DEFAULT_CHART_FALLBACK_COLOR,
+    }));
+    const missingYColumns = yColumns.filter((y) => !y.column);
+    if (!xColumn || missingYColumns.length > 0) {
+        throw new MissingColumnsError([
+            ...(xColumn ? [] : ['X-axis']),
+            ...missingYColumns.map((y) => y.field),
+        ]);
+    }
+    // Validate X and Y field types
+    if (!isQuantitativeType(xColumn.type)) {
+        throw new InvalidColumnTypeError(xColumn.name, 'quantitative');
+    }
+    const invalidYFields = yColumns.filter((y) => {
+        return y.column && !isNumericType(y.column.type);
+    });
+    if (invalidYFields.length > 0) {
+        throw new InvalidColumnTypeError(invalidYFields.map(({ field }) => field), 'numeric');
+    }
+    return {
+        xColumn,
+        yColumns,
+        xInterval,
+    };
+}
+//# sourceMappingURL=validation.js.map

package/dist/charts/chart-types/line-chart/validation.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"validation.js","sourceRoot":"","sources":["../../../../src/charts/chart-types/line-chart/validation.ts"],"names":[],"mappings":"AAEA,OAAO,EACL,sBAAsB,EACtB,mBAAmB,EACnB,mBAAmB,GACpB,MAAM,WAAW,CAAC;AACnB,OAAO,EAAC,aAAa,EAAE,kBAAkB,EAAC,MAAM,6BAA6B,CAAC;AAG9E,OAAO,EAAC,4BAA4B,EAAC,MAAM,iCAAiC,CAAC;AAa7E,MAAM,UAAU,yBAAyB,CAAC,EACxC,SAAS,EACT,QAAQ,EAAE,EAAC,CAAC,EAAE,OAAO,GAAG,EAAE,EAAE,SAAS,EAAC,GACC;IACvC,uCAAuC;IACvC,IAAI,CAAC,CAAC,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC/B,MAAM,IAAI,mBAAmB,CAAC;YAC5B,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC;YACxB,GAAG,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC;SAC1C,CAAC,CAAC;IACL,CAAC;IAED,mCAAmC;IACnC,MAAM,OAAO,GAAG,SAAS,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,KAAK,CAAC,CAAC,CAAC;IAChE,MAAM,QAAQ,GAAG,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;QACnC,KAAK,EAAE,CAAC,CAAC,KAAK;QACd,MAAM,EAAE,SAAS,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,KAAK,CAAC,CAAC,KAAK,CAAC;QAC7D,SAAS,EAAE,CAAC,CAAC,SAAS,IAAI,KAAK;QAC/B,KAAK,EAAE,CAAC,CAAC,KAAK,IAAI,4BAA4B;KAC/C,CAAC,CAAC,CAAC;IAEJ,MAAM,eAAe,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;IAE1D,IAAI,CAAC,OAAO,IAAI,eAAe,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC3C,MAAM,IAAI,mBAAmB,CAAC;YAC5B,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC;YAC9B,GAAG,eAAe,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC;SACvC,CAAC,CAAC;IACL,CAAC;IAED,+BAA+B;IAC/B,IAAI,CAAC,kBAAkB,CAAC,OAAO,CAAC,IAAI,CAAC,EAAE,CAAC;QACtC,MAAM,IAAI,sBAAsB,CAAC,OAAO,CAAC,IAAI,EAAE,cAAc,CAAC,CAAC;IACjE,CAAC;IAED,MAAM,cAAc,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE;QAC3C,OAAO,CAAC,CAAC,MAAM,IAAI,CAAC,aAAa,CAAC,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;IACnD,CAAC,CAAC,CAAC;IAEH,IAAI,cAAc,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC9B,MAAM,IAAI,sBAAsB,CAC9B,cAAc,CAAC,GAAG,CAAC,CAAC,EAAC,KAAK,EAAC,EAAE,EAAE,CAAC,KAAK,CAAC,EACtC,SAAS,CACV,CAAC;IACJ,CAAC;IAED,OAAO;QACL,OAAO;QACP,QAAQ;QACR,SAAS;KACoB,CAAC;AAClC,CAAC","sourcesContent":["import {LineChartSettings} from './schema';\nimport {ValidateSpecOptions} from '../base-types';\nimport {\n  InvalidColumnTypeError,\n  MissingColumnsError,\n  RequiredFieldsError,\n} from '../errors';\nimport {isNumericType, isQuantitativeType} from '../../../column-types-utils';\nimport {TableColumn} from '@sqlrooms/duckdb';\nimport {AggregateFunction, TemporalInterval} from '../../../schemas';\nimport {DEFAULT_CHART_FALLBACK_COLOR} from '../../../constants/chart-colors';\n\nexport type ValidatedLineChartSettings = {\n  xColumn: TableColumn;\n  yColumns: {\n    field: string;\n    column: TableColumn;\n    aggregate: AggregateFunction;\n    color: string;\n  }[];\n  xInterval?: TemporalInterval;\n};\n\nexport function validateLineChartSettings({\n  dataTable,\n  settings: {x, yFields = [], xInterval},\n}: ValidateSpecOptions<LineChartSettings>): ValidatedLineChartSettings {\n  // Basic validation for required fields\n  if (!x || yFields.length === 0) {\n    throw new RequiredFieldsError([\n      ...(x ? [] : ['X-axis']),\n      ...(yFields.length > 0 ? [] : ['Y-axis']),\n    ]);\n  }\n\n  // Validate X and Y field existence\n  const xColumn = dataTable.columns.find((col) => col.name === x);\n  const yColumns = yFields.map((y) => ({\n    field: y.field,\n    column: dataTable.columns.find((col) => col.name === y.field),\n    aggregate: y.aggregate ?? 'sum',\n    color: y.color ?? DEFAULT_CHART_FALLBACK_COLOR,\n  }));\n\n  const missingYColumns = yColumns.filter((y) => !y.column);\n\n  if (!xColumn || missingYColumns.length > 0) {\n    throw new MissingColumnsError([\n      ...(xColumn ? [] : ['X-axis']),\n      ...missingYColumns.map((y) => y.field),\n    ]);\n  }\n\n  // Validate X and Y field types\n  if (!isQuantitativeType(xColumn.type)) {\n    throw new InvalidColumnTypeError(xColumn.name, 'quantitative');\n  }\n\n  const invalidYFields = yColumns.filter((y) => {\n    return y.column && !isNumericType(y.column.type);\n  });\n\n  if (invalidYFields.length > 0) {\n    throw new InvalidColumnTypeError(\n      invalidYFields.map(({field}) => field),\n      'numeric',\n    );\n  }\n\n  return {\n    xColumn,\n    yColumns,\n    xInterval,\n  } as ValidatedLineChartSettings;\n}\n"]}

package/dist/charts/chart-types/resolveChartTypes.d.ts

@@ -0,0 +1,3 @@
+import { ChartTypeDefinition } from './base-types';
+export declare function resolveChartTypes(chartTypes: ChartTypeDefinition<any>[] | undefined): ChartTypeDefinition<any>[];
+//# sourceMappingURL=resolveChartTypes.d.ts.map

package/dist/charts/chart-types/resolveChartTypes.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"resolveChartTypes.d.ts","sourceRoot":"","sources":["../../../src/charts/chart-types/resolveChartTypes.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,mBAAmB,EAAC,MAAM,cAAc,CAAC;AAGjD,wBAAgB,iBAAiB,CAC/B,UAAU,EAAE,mBAAmB,CAAC,GAAG,CAAC,EAAE,GAAG,SAAS,GACjD,mBAAmB,CAAC,GAAG,CAAC,EAAE,CAM5B"}

package/dist/charts/chart-types/resolveChartTypes.js

@@ -0,0 +1,8 @@
+import { createDefaultChartTypes } from './createDefaultChartTypes';
+export function resolveChartTypes(chartTypes) {
+    if (chartTypes === undefined) {
+        return createDefaultChartTypes({ includeCustomSpec: false });
+    }
+    return chartTypes;
+}
+//# sourceMappingURL=resolveChartTypes.js.map

package/dist/charts/chart-types/resolveChartTypes.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"resolveChartTypes.js","sourceRoot":"","sources":["../../../src/charts/chart-types/resolveChartTypes.ts"],"names":[],"mappings":"AACA,OAAO,EAAC,uBAAuB,EAAC,MAAM,2BAA2B,CAAC;AAElE,MAAM,UAAU,iBAAiB,CAC/B,UAAkD;IAElD,IAAI,UAAU,KAAK,SAAS,EAAE,CAAC;QAC7B,OAAO,uBAAuB,CAAC,EAAC,iBAAiB,EAAE,KAAK,EAAC,CAAC,CAAC;IAC7D,CAAC;IAED,OAAO,UAAU,CAAC;AACpB,CAAC","sourcesContent":["import {ChartTypeDefinition} from './base-types';\nimport {createDefaultChartTypes} from './createDefaultChartTypes';\n\nexport function resolveChartTypes(\n  chartTypes: ChartTypeDefinition<any>[] | undefined,\n): ChartTypeDefinition<any>[] {\n  if (chartTypes === undefined) {\n    return createDefaultChartTypes({includeCustomSpec: false});\n  }\n\n  return chartTypes;\n}\n"]}

package/dist/charts/chart-types/scatter-plot/definition.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"definition.d.ts","sourceRoot":"","sources":["../../../../src/charts/chart-types/scatter-plot/definition.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAC,uBAAuB,EAAC,MAAM,eAAe,CAAC;AAC3D,OAAO,EAAC,sBAAsB,EAA2B,MAAM,UAAU,CAAC;AAU1E,eAAO,MAAM,oBAAoB,EAAE,uBAAuB,CAAC,sBAAsB,CAgB9E,CAAC"}
+{"version":3,"file":"definition.d.ts","sourceRoot":"","sources":["../../../../src/charts/chart-types/scatter-plot/definition.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAC,uBAAuB,EAAC,MAAM,eAAe,CAAC;AAC3D,OAAO,EAAC,sBAAsB,EAA2B,MAAM,UAAU,CAAC;AAU1E,eAAO,MAAM,oBAAoB,EAAE,uBAAuB,CAAC,sBAAsB,CAiB9E,CAAC"}

package/dist/charts/chart-types/scatter-plot/definition.js

@@ -10,6 +10,7 @@ export const scatterPlotChartType = {
     id: 'scatter-plot',
     label: 'Scatter Plot',
     description: DESCRIPTION,
+    aiDescription: `${DESCRIPTION} - relationship between two numeric columns (avoid for >10k rows, use heatmap instead)`,
     icon: Workflow,
     schema: ScatterPlotChartSettings,
     settingsComponent: ScatterPlotSettingsComponent,

package/dist/charts/chart-types/scatter-plot/definition.js.map

@@ -1 +1 @@
-{"version":3,"file":"definition.js","sourceRoot":"","sources":["../../../../src/charts/chart-types/scatter-plot/definition.ts"],"names":[],"mappings":"AACA,OAAO,EAAyB,wBAAwB,EAAC,MAAM,UAAU,CAAC;AAC1E,OAAO,EAAC,oBAAoB,EAAC,MAAM,wCAAwC,CAAC;AAC5E,OAAO,EAAC,4BAA4B,EAAC,MAAM,uBAAuB,CAAC;AACnE,OAAO,EAAC,uBAAuB,EAAC,MAAM,QAAQ,CAAC;AAC/C,OAAO,EAAC,QAAQ,EAAC,MAAM,cAAc,CAAC;AACtC,OAAO,EAAC,qBAAqB,EAAC,MAAM,QAAQ,CAAC;AAC7C,OAAO,EAAC,6BAA6B,EAAC,MAAM,wBAAwB,CAAC;AAErE,MAAM,WAAW,GAAG,6BAA6B,CAAC;AAElD,MAAM,CAAC,MAAM,oBAAoB,GAC/B;IACE,EAAE,EAAE,cAAc;IAClB,KAAK,EAAE,cAAc;IACrB,WAAW,EAAE,WAAW;IACxB,IAAI,EAAE,QAAQ;IACd,MAAM,EAAE,wBAAwB;IAChC,iBAAiB,EAAE,4BAA4B;IAC/C,UAAU,EAAE,oBAAoB,CAAC,WAAW,CAAC;IAC7C,UAAU,EAAE,uBAAuB;IACnC,aAAa,EAAE,GAAG,EAAE,CAAC,CAAC;QACpB,OAAO,EAAE,6BAA6B;QACtC,MAAM,EACJ,yGAAyG;KAC5G,CAAC;IACF,UAAU,EAAE,qBAAqB;CAClC,CAAC","sourcesContent":["import type {SpecChartTypeDefinition} from '../base-types';\nimport {ScatterPlotChartConfig, ScatterPlotChartSettings} from './schema';\nimport {titleFromDescription} from '../../../chart-builders/chartTypeUtils';\nimport {ScatterPlotSettingsComponent} from './ScatterPlotSettings';\nimport {createScatterPlotAiTool} from './tool';\nimport {Workflow} from 'lucide-react';\nimport {createScatterPlotSpec} from './spec';\nimport {DEFAULT_CHART_MAX_DATA_POINTS} from '../../../chart-runtime';\n\nconst DESCRIPTION = 'Create a scatter plot chart';\n\nexport const scatterPlotChartType: SpecChartTypeDefinition<ScatterPlotChartConfig> =\n  {\n    id: 'scatter-plot',\n    label: 'Scatter Plot',\n    description: DESCRIPTION,\n    icon: Workflow,\n    schema: ScatterPlotChartSettings,\n    settingsComponent: ScatterPlotSettingsComponent,\n    buildTitle: titleFromDescription(DESCRIPTION),\n    createTool: createScatterPlotAiTool,\n    getDataPolicy: () => ({\n      maxRows: DEFAULT_CHART_MAX_DATA_POINTS,\n      reason:\n        'Scatter charts render one point per row. Use a heatmap or another aggregated chart for larger datasets.',\n    }),\n    createSpec: createScatterPlotSpec,\n  };\n"]}
+{"version":3,"file":"definition.js","sourceRoot":"","sources":["../../../../src/charts/chart-types/scatter-plot/definition.ts"],"names":[],"mappings":"AACA,OAAO,EAAyB,wBAAwB,EAAC,MAAM,UAAU,CAAC;AAC1E,OAAO,EAAC,oBAAoB,EAAC,MAAM,wCAAwC,CAAC;AAC5E,OAAO,EAAC,4BAA4B,EAAC,MAAM,uBAAuB,CAAC;AACnE,OAAO,EAAC,uBAAuB,EAAC,MAAM,QAAQ,CAAC;AAC/C,OAAO,EAAC,QAAQ,EAAC,MAAM,cAAc,CAAC;AACtC,OAAO,EAAC,qBAAqB,EAAC,MAAM,QAAQ,CAAC;AAC7C,OAAO,EAAC,6BAA6B,EAAC,MAAM,wBAAwB,CAAC;AAErE,MAAM,WAAW,GAAG,6BAA6B,CAAC;AAElD,MAAM,CAAC,MAAM,oBAAoB,GAC/B;IACE,EAAE,EAAE,cAAc;IAClB,KAAK,EAAE,cAAc;IACrB,WAAW,EAAE,WAAW;IACxB,aAAa,EAAE,GAAG,WAAW,wFAAwF;IACrH,IAAI,EAAE,QAAQ;IACd,MAAM,EAAE,wBAAwB;IAChC,iBAAiB,EAAE,4BAA4B;IAC/C,UAAU,EAAE,oBAAoB,CAAC,WAAW,CAAC;IAC7C,UAAU,EAAE,uBAAuB;IACnC,aAAa,EAAE,GAAG,EAAE,CAAC,CAAC;QACpB,OAAO,EAAE,6BAA6B;QACtC,MAAM,EACJ,yGAAyG;KAC5G,CAAC;IACF,UAAU,EAAE,qBAAqB;CAClC,CAAC","sourcesContent":["import type {SpecChartTypeDefinition} from '../base-types';\nimport {ScatterPlotChartConfig, ScatterPlotChartSettings} from './schema';\nimport {titleFromDescription} from '../../../chart-builders/chartTypeUtils';\nimport {ScatterPlotSettingsComponent} from './ScatterPlotSettings';\nimport {createScatterPlotAiTool} from './tool';\nimport {Workflow} from 'lucide-react';\nimport {createScatterPlotSpec} from './spec';\nimport {DEFAULT_CHART_MAX_DATA_POINTS} from '../../../chart-runtime';\n\nconst DESCRIPTION = 'Create a scatter plot chart';\n\nexport const scatterPlotChartType: SpecChartTypeDefinition<ScatterPlotChartConfig> =\n  {\n    id: 'scatter-plot',\n    label: 'Scatter Plot',\n    description: DESCRIPTION,\n    aiDescription: `${DESCRIPTION} - relationship between two numeric columns (avoid for >10k rows, use heatmap instead)`,\n    icon: Workflow,\n    schema: ScatterPlotChartSettings,\n    settingsComponent: ScatterPlotSettingsComponent,\n    buildTitle: titleFromDescription(DESCRIPTION),\n    createTool: createScatterPlotAiTool,\n    getDataPolicy: () => ({\n      maxRows: DEFAULT_CHART_MAX_DATA_POINTS,\n      reason:\n        'Scatter charts render one point per row. Use a heatmap or another aggregated chart for larger datasets.',\n    }),\n    createSpec: createScatterPlotSpec,\n  };\n"]}

package/dist/charts/chart-types/scatter-plot/spec.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"spec.d.ts","sourceRoot":"","sources":["../../../../src/charts/chart-types/scatter-plot/spec.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAC,IAAI,EAAC,MAAM,qBAAqB,CAAC;AAC9C,OAAO,EAAC,wBAAwB,EAAC,MAAM,UAAU,CAAC;AAMlD,OAAO,EAAC,iBAAiB,EAAyB,MAAM,eAAe,CAAC;AAMxE,wBAAgB,qBAAqB,CACnC,OAAO,EAAE,iBAAiB,CAAC,wBAAwB,CAAC,GACnD,IAAI,CAoCN"}
+{"version":3,"file":"spec.d.ts","sourceRoot":"","sources":["../../../../src/charts/chart-types/scatter-plot/spec.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAC,IAAI,EAAC,MAAM,qBAAqB,CAAC;AAC9C,OAAO,EAAC,wBAAwB,EAAC,MAAM,UAAU,CAAC;AAIlD,OAAO,EAAC,iBAAiB,EAAyB,MAAM,eAAe,CAAC;AAKxE,wBAAgB,qBAAqB,CACnC,OAAO,EAAE,iBAAiB,CAAC,wBAAwB,CAAC,GACnD,IAAI,CAoCN"}

package/dist/charts/chart-types/scatter-plot/spec.js

@@ -1,6 +1,5 @@
-import { InvalidColumnTypeError, MissingColumnsError, RequiredFieldsError, } from '../errors';
+import { validateScatterPlotSettings } from './validation';
 import { getChartTableReference } from '../base-types';
-import { isNumericType } from '../../../column-types-utils';
 const FG_COLOR = 'var(--color-chart-1)';
 const DEFAULT_POINT_SIZE = 3;
 export function createScatterPlotSpec(options) {
@@ -35,47 +34,4 @@ export function createScatterPlotSpec(options) {
         params: { brush: { select: 'crossfilter' } },
     };
 }
-function validateScatterPlotSettings({ dataTable, settings: { x, y, size }, }) {
-    // Basic validation for required fields
-    if (!x || !y) {
-        throw new RequiredFieldsError([
-            ...(x ? [] : ['X field']),
-            ...(y ? [] : ['Y field']),
-        ]);
-    }
-    // Validate X and Y field existence
-    const xColumn = dataTable.columns.find((col) => col.name === x);
-    const yColumn = dataTable.columns.find((col) => col.name === y);
-    if (!xColumn || !yColumn) {
-        throw new MissingColumnsError([
-            ...(xColumn ? [] : [x]),
-            ...(yColumn ? [] : [y]),
-        ]);
-    }
-    // Validate X and Y field are numeric
-    const xIsNumeric = isNumericType(xColumn.type);
-    const yIsNumeric = isNumericType(yColumn.type);
-    if (!xIsNumeric || !yIsNumeric) {
-        throw new InvalidColumnTypeError([
-            ...(!xIsNumeric ? [xColumn.name] : []),
-            ...(!yIsNumeric ? [yColumn.name] : []),
-        ], 'numeric');
-    }
-    // Validate size field if provided
-    let sizeColumn = undefined;
-    if (size) {
-        sizeColumn = dataTable.columns.find((col) => col.name === size);
-        if (!sizeColumn) {
-            throw new MissingColumnsError([size]);
-        }
-        if (!isNumericType(sizeColumn.type)) {
-            throw new InvalidColumnTypeError([sizeColumn.name], 'numeric');
-        }
-    }
-    return {
-        xColumn,
-        yColumn,
-        sizeColumn,
-    };
-}
 //# sourceMappingURL=spec.js.map

package/dist/charts/chart-types/scatter-plot/spec.js.map

@@ -1 +1 @@
-{"version":3,"file":"spec.js","sourceRoot":"","sources":["../../../../src/charts/chart-types/scatter-plot/spec.ts"],"names":[],"mappings":"AAEA,OAAO,EACL,sBAAsB,EACtB,mBAAmB,EACnB,mBAAmB,GACpB,MAAM,WAAW,CAAC;AACnB,OAAO,EAAoB,sBAAsB,EAAC,MAAM,eAAe,CAAC;AACxE,OAAO,EAAC,aAAa,EAAC,MAAM,6BAA6B,CAAC;AAE1D,MAAM,QAAQ,GAAG,sBAAsB,CAAC;AACxC,MAAM,kBAAkB,GAAG,CAAC,CAAC;AAE7B,MAAM,UAAU,qBAAqB,CACnC,OAAoD;IAEpD,MAAM,EAAC,SAAS,EAAE,aAAa,EAAC,GAAG,OAAO,CAAC;IAE3C,MAAM,EAAC,OAAO,EAAE,OAAO,EAAE,UAAU,EAAC,GAAG,2BAA2B,CAAC,OAAO,CAAC,CAAC;IAE5E,MAAM,OAAO,GAA4B;QACvC,IAAI,EAAE,KAAK;QACX,IAAI,EAAE,EAAC,IAAI,EAAE,sBAAsB,CAAC,SAAS,CAAC,EAAE,QAAQ,EAAE,QAAQ,EAAC;QACnE,CAAC,EAAE,OAAO,CAAC,IAAI;QACf,CAAC,EAAE,OAAO,CAAC,IAAI;QACf,IAAI,EAAE,QAAQ;QACd,WAAW,EAAE,GAAG;KACjB,CAAC;IAEF,gFAAgF;IAChF,IAAI,UAAU,EAAE,CAAC;QACf,OAAO,CAAC,CAAC,GAAG,UAAU,CAAC,IAAI,CAAC;IAC9B,CAAC;SAAM,CAAC;QACN,OAAO,CAAC,CAAC,GAAG,kBAAkB,CAAC;IACjC,CAAC;IAED,MAAM,IAAI,GAAc,CAAC,OAAO,CAAC,CAAC;IAElC,IAAI,aAAa,EAAE,CAAC;QAClB,IAAI,CAAC,IAAI,CAAC,EAAC,MAAM,EAAE,YAAY,EAAE,EAAE,EAAE,QAAQ,EAAC,CAAC,CAAC;IAClD,CAAC;IAED,OAAO;QACL,IAAI;QACJ,MAAM,EAAE,OAAO,CAAC,IAAI;QACpB,MAAM,EAAE,OAAO,CAAC,IAAI;QACpB,MAAM,EAAE,GAAG;QACX,KAAK,EAAE,GAAG;QACV,OAAO,EAAE,EAAC,IAAI,EAAE,EAAE,EAAE,KAAK,EAAE,EAAE,EAAE,GAAG,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAC;QACnD,MAAM,EAAE,EAAC,KAAK,EAAE,EAAC,MAAM,EAAE,aAAa,EAAC,EAAC;KACjC,CAAC;AACZ,CAAC;AAED,SAAS,2BAA2B,CAAC,EACnC,SAAS,EACT,QAAQ,EAAE,EAAC,CAAC,EAAE,CAAC,EAAE,IAAI,EAAC,GACsB;IAC5C,uCAAuC;IACvC,IAAI,CAAC,CAAC,IAAI,CAAC,CAAC,EAAE,CAAC;QACb,MAAM,IAAI,mBAAmB,CAAC;YAC5B,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC;YACzB,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC;SAC1B,CAAC,CAAC;IACL,CAAC;IAED,mCAAmC;IACnC,MAAM,OAAO,GAAG,SAAS,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,KAAK,CAAC,CAAC,CAAC;IAChE,MAAM,OAAO,GAAG,SAAS,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,KAAK,CAAC,CAAC,CAAC;IAEhE,IAAI,CAAC,OAAO,IAAI,CAAC,OAAO,EAAE,CAAC;QACzB,MAAM,IAAI,mBAAmB,CAAC;YAC5B,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;YACvB,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;SACxB,CAAC,CAAC;IACL,CAAC;IAED,qCAAqC;IACrC,MAAM,UAAU,GAAG,aAAa,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;IAC/C,MAAM,UAAU,GAAG,aAAa,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;IAC/C,IAAI,CAAC,UAAU,IAAI,CAAC,UAAU,EAAE,CAAC;QAC/B,MAAM,IAAI,sBAAsB,CAC9B;YACE,GAAG,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;YACtC,GAAG,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;SACvC,EACD,SAAS,CACV,CAAC;IACJ,CAAC;IAED,kCAAkC;IAClC,IAAI,UAAU,GAAG,SAAS,CAAC;IAC3B,IAAI,IAAI,EAAE,CAAC;QACT,UAAU,GAAG,SAAS,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,KAAK,IAAI,CAAC,CAAC;QAChE,IAAI,CAAC,UAAU,EAAE,CAAC;YAChB,MAAM,IAAI,mBAAmB,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC;QACxC,CAAC;QACD,IAAI,CAAC,aAAa,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC;YACpC,MAAM,IAAI,sBAAsB,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,SAAS,CAAC,CAAC;QACjE,CAAC;IACH,CAAC;IAED,OAAO;QACL,OAAO;QACP,OAAO;QACP,UAAU;KACX,CAAC;AACJ,CAAC","sourcesContent":["import type {Spec} from '@uwdata/mosaic-spec';\nimport {ScatterPlotChartSettings} from './schema';\nimport {\n  InvalidColumnTypeError,\n  MissingColumnsError,\n  RequiredFieldsError,\n} from '../errors';\nimport {CreateSpecOptions, getChartTableReference} from '../base-types';\nimport {isNumericType} from '../../../column-types-utils';\n\nconst FG_COLOR = 'var(--color-chart-1)';\nconst DEFAULT_POINT_SIZE = 3;\n\nexport function createScatterPlotSpec(\n  options: CreateSpecOptions<ScatterPlotChartSettings>,\n): Spec {\n  const {dataTable, selectionName} = options;\n\n  const {xColumn, yColumn, sizeColumn} = validateScatterPlotSettings(options);\n\n  const dotMark: Record<string, unknown> = {\n    mark: 'dot',\n    data: {from: getChartTableReference(dataTable), filterBy: '$brush'},\n    x: xColumn.name,\n    y: yColumn.name,\n    fill: FG_COLOR,\n    fillOpacity: 0.5,\n  };\n\n  // If size column is provided, use it for point radius; otherwise use fixed size\n  if (sizeColumn) {\n    dotMark.r = sizeColumn.name;\n  } else {\n    dotMark.r = DEFAULT_POINT_SIZE;\n  }\n\n  const plot: unknown[] = [dotMark];\n\n  if (selectionName) {\n    plot.push({select: 'intervalXY', as: '$brush'});\n  }\n\n  return {\n    plot,\n    xLabel: xColumn.name,\n    yLabel: yColumn.name,\n    height: 250,\n    width: 380,\n    margins: {left: 50, right: 20, top: 20, bottom: 50},\n    params: {brush: {select: 'crossfilter'}},\n  } as Spec;\n}\n\nfunction validateScatterPlotSettings({\n  dataTable,\n  settings: {x, y, size},\n}: CreateSpecOptions<ScatterPlotChartSettings>) {\n  // Basic validation for required fields\n  if (!x || !y) {\n    throw new RequiredFieldsError([\n      ...(x ? [] : ['X field']),\n      ...(y ? [] : ['Y field']),\n    ]);\n  }\n\n  // Validate X and Y field existence\n  const xColumn = dataTable.columns.find((col) => col.name === x);\n  const yColumn = dataTable.columns.find((col) => col.name === y);\n\n  if (!xColumn || !yColumn) {\n    throw new MissingColumnsError([\n      ...(xColumn ? [] : [x]),\n      ...(yColumn ? [] : [y]),\n    ]);\n  }\n\n  // Validate X and Y field are numeric\n  const xIsNumeric = isNumericType(xColumn.type);\n  const yIsNumeric = isNumericType(yColumn.type);\n  if (!xIsNumeric || !yIsNumeric) {\n    throw new InvalidColumnTypeError(\n      [\n        ...(!xIsNumeric ? [xColumn.name] : []),\n        ...(!yIsNumeric ? [yColumn.name] : []),\n      ],\n      'numeric',\n    );\n  }\n\n  // Validate size field if provided\n  let sizeColumn = undefined;\n  if (size) {\n    sizeColumn = dataTable.columns.find((col) => col.name === size);\n    if (!sizeColumn) {\n      throw new MissingColumnsError([size]);\n    }\n    if (!isNumericType(sizeColumn.type)) {\n      throw new InvalidColumnTypeError([sizeColumn.name], 'numeric');\n    }\n  }\n\n  return {\n    xColumn,\n    yColumn,\n    sizeColumn,\n  };\n}\n"]}
+{"version":3,"file":"spec.js","sourceRoot":"","sources":["../../../../src/charts/chart-types/scatter-plot/spec.ts"],"names":[],"mappings":"AAGA,OAAO,EAAC,2BAA2B,EAAC,MAAM,cAAc,CAAC;AAEzD,OAAO,EAAoB,sBAAsB,EAAC,MAAM,eAAe,CAAC;AAExE,MAAM,QAAQ,GAAG,sBAAsB,CAAC;AACxC,MAAM,kBAAkB,GAAG,CAAC,CAAC;AAE7B,MAAM,UAAU,qBAAqB,CACnC,OAAoD;IAEpD,MAAM,EAAC,SAAS,EAAE,aAAa,EAAC,GAAG,OAAO,CAAC;IAE3C,MAAM,EAAC,OAAO,EAAE,OAAO,EAAE,UAAU,EAAC,GAAG,2BAA2B,CAAC,OAAO,CAAC,CAAC;IAE5E,MAAM,OAAO,GAA4B;QACvC,IAAI,EAAE,KAAK;QACX,IAAI,EAAE,EAAC,IAAI,EAAE,sBAAsB,CAAC,SAAS,CAAC,EAAE,QAAQ,EAAE,QAAQ,EAAC;QACnE,CAAC,EAAE,OAAO,CAAC,IAAI;QACf,CAAC,EAAE,OAAO,CAAC,IAAI;QACf,IAAI,EAAE,QAAQ;QACd,WAAW,EAAE,GAAG;KACjB,CAAC;IAEF,gFAAgF;IAChF,IAAI,UAAU,EAAE,CAAC;QACf,OAAO,CAAC,CAAC,GAAG,UAAU,CAAC,IAAI,CAAC;IAC9B,CAAC;SAAM,CAAC;QACN,OAAO,CAAC,CAAC,GAAG,kBAAkB,CAAC;IACjC,CAAC;IAED,MAAM,IAAI,GAAc,CAAC,OAAO,CAAC,CAAC;IAElC,IAAI,aAAa,EAAE,CAAC;QAClB,IAAI,CAAC,IAAI,CAAC,EAAC,MAAM,EAAE,YAAY,EAAE,EAAE,EAAE,QAAQ,EAAC,CAAC,CAAC;IAClD,CAAC;IAED,OAAO;QACL,IAAI;QACJ,MAAM,EAAE,OAAO,CAAC,IAAI;QACpB,MAAM,EAAE,OAAO,CAAC,IAAI;QACpB,MAAM,EAAE,GAAG;QACX,KAAK,EAAE,GAAG;QACV,OAAO,EAAE,EAAC,IAAI,EAAE,EAAE,EAAE,KAAK,EAAE,EAAE,EAAE,GAAG,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAC;QACnD,MAAM,EAAE,EAAC,KAAK,EAAE,EAAC,MAAM,EAAE,aAAa,EAAC,EAAC;KACjC,CAAC;AACZ,CAAC","sourcesContent":["import type {Spec} from '@uwdata/mosaic-spec';\nimport {ScatterPlotChartSettings} from './schema';\n\nimport {validateScatterPlotSettings} from './validation';\n\nimport {CreateSpecOptions, getChartTableReference} from '../base-types';\n\nconst FG_COLOR = 'var(--color-chart-1)';\nconst DEFAULT_POINT_SIZE = 3;\n\nexport function createScatterPlotSpec(\n  options: CreateSpecOptions<ScatterPlotChartSettings>,\n): Spec {\n  const {dataTable, selectionName} = options;\n\n  const {xColumn, yColumn, sizeColumn} = validateScatterPlotSettings(options);\n\n  const dotMark: Record<string, unknown> = {\n    mark: 'dot',\n    data: {from: getChartTableReference(dataTable), filterBy: '$brush'},\n    x: xColumn.name,\n    y: yColumn.name,\n    fill: FG_COLOR,\n    fillOpacity: 0.5,\n  };\n\n  // If size column is provided, use it for point radius; otherwise use fixed size\n  if (sizeColumn) {\n    dotMark.r = sizeColumn.name;\n  } else {\n    dotMark.r = DEFAULT_POINT_SIZE;\n  }\n\n  const plot: unknown[] = [dotMark];\n\n  if (selectionName) {\n    plot.push({select: 'intervalXY', as: '$brush'});\n  }\n\n  return {\n    plot,\n    xLabel: xColumn.name,\n    yLabel: yColumn.name,\n    height: 250,\n    width: 380,\n    margins: {left: 50, right: 20, top: 20, bottom: 50},\n    params: {brush: {select: 'crossfilter'}},\n  } as Spec;\n}\n"]}

package/dist/charts/chart-types/scatter-plot/tool.d.ts

@@ -1,9 +1,8 @@
 import { z } from 'zod';
-import { type DashboardToolDeps } from '../base-types';
-export declare const ScatterPlotToolParameters: z.ZodObject<{
-    artifactId: z.ZodOptional<z.ZodString>;
-    tableName: z.ZodOptional<z.ZodString>;
-    createArtifactIfMissing: z.ZodDefault<z.ZodOptional<z.ZodBoolean>>;
+import { ChartToolParams, ChartToolOutput } from '../tool-types';
+export declare const ScatterPlotToolInput: z.ZodObject<{
+    tableName: z.ZodString;
+    title: z.ZodOptional<z.ZodString>;
     panelId: z.ZodOptional<z.ZodString>;
     reasoning: z.ZodString;
     settings: z.ZodObject<{
@@ -12,31 +11,29 @@ export declare const ScatterPlotToolParameters: z.ZodObject<{
         size: z.ZodNonOptional<z.ZodOptional<z.ZodString>>;
     }, z.core.$strip>;
 }, z.core.$strip>;
-export type ScatterPlotToolParams = z.infer<typeof ScatterPlotToolParameters>;
-export declare function createScatterPlotAiTool(deps: DashboardToolDeps): import("ai").Tool<{
-    createArtifactIfMissing: boolean;
+export type ScatterPlotToolInput = z.infer<typeof ScatterPlotToolInput>;
+export declare function createScatterPlotAiTool({ databaseAdapter, addChart, maxDataPoints, }: ChartToolParams): import("ai").Tool<{
+    tableName: string;
     reasoning: string;
     settings: {
         x: string;
         y: string;
         size: string;
     };
-    artifactId?: string | undefined;
-    tableName?: string | undefined;
+    title?: string | undefined;
     panelId?: string | undefined;
-}, {
-    llmResult: {
-        success: boolean;
-        details: string;
-        data: import("..").PanelResult;
-        errorMessage?: undefined;
-    };
-} | {
-    llmResult: {
-        success: boolean;
-        errorMessage: string;
-        details?: undefined;
-        data?: undefined;
+}, ChartToolOutput<{
+    chartType: "scatter-plot";
+    settings: {
+        x?: string | undefined;
+        y?: string | undefined;
+        size?: string | undefined;
     };
-}>;
+    settingsOpen?: boolean | undefined;
+    dataPolicy?: {
+        disabled?: boolean | undefined;
+        maxRows?: number | undefined;
+        reason?: string | undefined;
+    } | undefined;
+}>>;
 //# sourceMappingURL=tool.d.ts.map

package/dist/charts/chart-types/scatter-plot/tool.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"tool.d.ts","sourceRoot":"","sources":["../../../../src/charts/chart-types/scatter-plot/tool.ts"],"names":[],"mappings":"AACA,OAAO,EAAC,CAAC,EAAC,MAAM,KAAK,CAAC;AAGtB,OAAO,EAAC,KAAK,iBAAiB,EAAC,MAAM,eAAe,CAAC;AAKrD,eAAO,MAAM,yBAAyB;;;;;;;;;;;iBAEpC,CAAC;AAEH,MAAM,MAAM,qBAAqB,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,yBAAyB,CAAC,CAAC;AAE9E,wBAAgB,uBAAuB,CAAC,IAAI,EAAE,iBAAiB;;;;;;;;;;;;;;;;;;;;;;;;;GAuF9D"}
+{"version":3,"file":"tool.d.ts","sourceRoot":"","sources":["../../../../src/charts/chart-types/scatter-plot/tool.ts"],"names":[],"mappings":"AACA,OAAO,EAAC,CAAC,EAAC,MAAM,KAAK,CAAC;AAItB,OAAO,EAAC,eAAe,EAAE,eAAe,EAAC,MAAM,eAAe,CAAC;AAI/D,eAAO,MAAM,oBAAoB;;;;;;;;;;iBAE/B,CAAC;AAEH,MAAM,MAAM,oBAAoB,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,oBAAoB,CAAC,CAAC;AAExE,wBAAgB,uBAAuB,CAAC,EACtC,eAAe,EACf,QAAQ,EACR,aAAa,GACd,EAAE,eAAe;;;;;;;;;;;;;;;;;;;;;;;IAiDjB"}

package/dist/charts/chart-types/scatter-plot/tool.js

@@ -1,13 +1,13 @@
 import { tool } from 'ai';
 import { ScatterPlotChartSettings } from './schema';
-import { BaseChartToolParameters } from '../../../ai/tool-schemas';
-import { validateColumnExists } from '../../../ai/tool-validation';
+import { BaseChartToolInput } from '../../../ai/tool-schemas';
 import { NUMERIC_COLUMN_TYPES } from '../../../column-types-utils';
-import { createOrUpdateChartPanel } from '../../../ai/tool-helpers';
-export const ScatterPlotToolParameters = BaseChartToolParameters.extend({
+import { validateScatterPlotSettings } from './validation';
+import { ensureTable } from '../../../ai/tool-helpers';
+export const ScatterPlotToolInput = BaseChartToolInput.extend({
     settings: ScatterPlotChartSettings.required(),
 });
-export function createScatterPlotAiTool(deps) {
+export function createScatterPlotAiTool({ databaseAdapter, addChart, maxDataPoints, }) {
     return tool({
         description: `Scatter chart: plots individual points positioned by two numeric columns (x, y), with optional size dimension.
 
@@ -17,51 +17,38 @@ Example queries: "plot latitude vs longitude", "show correlation between elevati
 Required: x and y must be numeric (${NUMERIC_COLUMN_TYPES.join(', ')}).
 Optional: size can encode a third numeric dimension (magnitude, frequency, count).
 
-IMPORTANT: Scatter charts render ALL rows as individual points. Do NOT create scatter charts for tables with more than ${deps.maxDataPoints.toLocaleString()} rows - use aggregated visualizations instead (histogram, count-plot, line-chart with time intervals, or heatmap).
+IMPORTANT: Scatter charts render ALL rows as individual points. Do NOT create scatter charts for tables with more than ${maxDataPoints.toLocaleString()} rows - use aggregated visualizations instead (histogram, count-plot, line-chart with time intervals, or heatmap).
 
-To UPDATE an existing scatter chart: provide the panelId parameter. Otherwise creates new panel.
-
-Do NOT use for: distributions (use histogram), categorical counts (use count-plot), trends over time (use line-chart), or large datasets (>${deps.maxDataPoints.toLocaleString()} rows).`,
-        inputSchema: ScatterPlotToolParameters,
-        execute: async (params, context) => {
+Do NOT use for: distributions (use histogram), categorical counts (use count-plot), trends over time (use line-chart), or large datasets (>${maxDataPoints.toLocaleString()} rows).`,
+        inputSchema: ScatterPlotToolInput,
+        execute: async ({ tableName, title, settings, panelId }) => {
             try {
-                const artifactId = deps.resolveArtifact(params.artifactId, params.createArtifactIfMissing, context);
-                const { tableName, columns } = deps.resolveTable(artifactId, params.tableName);
-                // Validate settings
-                validateColumnExists(params.settings.x, NUMERIC_COLUMN_TYPES, columns, 'x');
-                validateColumnExists(params.settings.y, NUMERIC_COLUMN_TYPES, columns, 'y');
-                // Validate size if provided
-                if (params.settings.size) {
-                    validateColumnExists(params.settings.size, NUMERIC_COLUMN_TYPES, columns, 'size');
-                }
-                const result = createOrUpdateChartPanel(deps, {
-                    panelId: params.panelId,
-                    dashboardId: artifactId,
+                const dataTable = ensureTable(databaseAdapter, tableName);
+                validateScatterPlotSettings({
+                    dataTable,
+                    settings,
+                    maxDataPoints,
+                });
+                const chartConfig = {
+                    chartType: 'scatter-plot',
+                    settings,
+                };
+                await addChart({
                     tableName,
-                    title: params.settings.x && params.settings.y
-                        ? `Scatter chart - ${params.settings.x} vs ${params.settings.y}`
-                        : 'Scatter chart',
-                    config: {
-                        chartType: 'scatter-plot',
-                        settings: params.settings,
-                    },
+                    panelId,
+                    config: chartConfig,
+                    title,
                 });
                 return {
-                    llmResult: {
-                        success: true,
-                        details: params.panelId
-                            ? `Updated scatter chart "${result.title}".`
-                            : `Created scatter chart "${result.title}".`,
-                        data: result,
-                    },
+                    success: true,
+                    details: `Generated scatter chart configuration.`,
+                    data: chartConfig,
                 };
             }
             catch (error) {
                 return {
-                    llmResult: {
-                        success: false,
-                        errorMessage: error instanceof Error ? error.message : String(error),
-                    },
+                    success: false,
+                    errorMessage: error instanceof Error ? error.message : String(error),
                 };
             }
         },