@querypanel/node-sdk 1.0.53 → 1.0.55

package/dist/index.d.cts

@@ -191,8 +191,8 @@ type QueryErrorCode = (typeof QueryErrorCode)[keyof typeof QueryErrorCode];
  */
 declare class QueryPipelineError extends Error {
     readonly code: QueryErrorCode;
-    readonly details?: Record<string, unknown> | undefined;
-    constructor(message: string, code: QueryErrorCode, details?: Record<string, unknown> | undefined);
+    readonly details?: Record<string, unknown>;
+    constructor(message: string, code: QueryErrorCode, details?: Record<string, unknown>);
     /**
      * Check if this is a moderation error
      */
@@ -737,6 +737,12 @@ interface ChartModifyOptions {
      * - "v2": Improved pipeline with intent planning, hybrid retrieval, schema linking, and SQL reflection
      */
     pipeline?: "v1" | "v2";
+    /**
+     * QueryPanel session ID for context-aware follow-ups.
+     * Pass the querypanelSessionId from a previous ask() response
+     * to preserve conversation history when modifying charts.
+     */
+    querypanelSessionId?: string;
 }
 /**
  * Response from modifyChart(), extending AskResponse with modification metadata.

package/dist/index.d.ts

@@ -191,8 +191,8 @@ type QueryErrorCode = (typeof QueryErrorCode)[keyof typeof QueryErrorCode];
  */
 declare class QueryPipelineError extends Error {
     readonly code: QueryErrorCode;
-    readonly details?: Record<string, unknown> | undefined;
-    constructor(message: string, code: QueryErrorCode, details?: Record<string, unknown> | undefined);
+    readonly details?: Record<string, unknown>;
+    constructor(message: string, code: QueryErrorCode, details?: Record<string, unknown>);
     /**
      * Check if this is a moderation error
      */
@@ -737,6 +737,12 @@ interface ChartModifyOptions {
      * - "v2": Improved pipeline with intent planning, hybrid retrieval, schema linking, and SQL reflection
      */
     pipeline?: "v1" | "v2";
+    /**
+     * QueryPanel session ID for context-aware follow-ups.
+     * Pass the querypanelSessionId from a previous ask() response
+     * to preserve conversation history when modifying charts.
+     */
+    querypanelSessionId?: string;
 }
 /**
  * Response from modifyChart(), extending AskResponse with modification metadata.

package/dist/index.js

@@ -1424,17 +1424,11 @@ function buildModifiedQuestion(originalQuestion, modifications, pipeline) {
       const from = normalizeDateInput(modifications.dateRange.from);
       const to = normalizeDateInput(modifications.dateRange.to);
       if (from && to) {
-        hints.push(
-          `replace any existing date filters with exact date range from ${from} to ${to} (inclusive, do not add extra days)`
-        );
+        hints.push(`change date range to ${from} through ${to}`);
       } else if (from) {
-        hints.push(
-          `replace any existing date filters with exact start date ${from} (do not shift this date)`
-        );
+        hints.push(`change start date to ${from}`);
       } else if (to) {
-        hints.push(
-          `replace any existing date filters with exact end date ${to} (inclusive, do not add extra days)`
-        );
+        hints.push(`change end date to ${to}`);
       }
     } else {
       const parts = [];
@@ -1455,6 +1449,9 @@ function buildModifiedQuestion(originalQuestion, modifications, pipeline) {
   if (hints.length === 0) {
     return originalQuestion;
   }
+  if (pipeline === "v2") {
+    return hints.join(", ");
+  }
   return `${originalQuestion} (${hints.join(", ")})`;
 }
 var START_PARAM_KEY_REGEX = /(^|_)(start|from)(_|$)/i;
@@ -1550,9 +1547,6 @@ function stripVizSpecOnlyHints(hints) {
   const { kind: _kind, ...rest } = hints;
   return rest;
 }
-function isDateRangeOnly(mods) {
-  return !!mods.dateRange && !mods.timeGranularity && !mods.additionalInstructions && !mods.customSql;
-}
 function resolveTenantId5(client, tenantId) {
   const resolved = tenantId ?? client.getDefaultTenantId();
   if (!resolved) {
@@ -1565,6 +1559,7 @@ function resolveTenantId5(client, tenantId) {
 async function modifyChart(client, queryEngine, input, options, signal) {
   const tenantId = resolveTenantId5(client, options?.tenantId);
   const sessionId = crypto4.randomUUID();
+  const querypanelSessionId = options?.querypanelSessionId ?? sessionId;
   const chartType = options?.chartType ?? "vega-lite";
   const hasSqlMods = !!input.sqlModifications;
   const hasVizMods = !!input.vizModifications;
@@ -1597,78 +1592,6 @@ async function modifyChart(client, queryEngine, input, options, signal) {
     finalParams = {};
     paramMetadata = [];
     sqlChanged = true;
-  } else if (hasSqlMods && options?.pipeline === "v2" && isDateRangeOnly(input.sqlModifications)) {
-    let usedFastPath = false;
-    try {
-      const rewriteResponse = await client.post(
-        "/v2/rewrite-datefilter",
-        {
-          previous_sql: input.sql,
-          previous_params: input.params ? Object.entries(input.params).map(([name, value]) => ({
-            name,
-            value
-          })) : [],
-          date_range: input.sqlModifications.dateRange,
-          question: input.question,
-          ...tenantSettings ? { tenant_settings: tenantSettings } : {},
-          ...databaseName ? { database: databaseName } : {},
-          ...metadata?.dialect ? { dialect: metadata.dialect } : {}
-        },
-        tenantId,
-        options?.userId,
-        options?.scopes,
-        signal,
-        sessionId
-      );
-      finalSql = rewriteResponse.sql;
-      paramMetadata = Array.isArray(rewriteResponse.params) ? rewriteResponse.params : [];
-      finalParams = queryEngine.mapGeneratedParams(paramMetadata);
-      applyDateRangeOverrides(
-        input.sqlModifications?.dateRange,
-        finalParams,
-        paramMetadata
-      );
-      rationale = rewriteResponse.rationale;
-      queryId = rewriteResponse.queryId;
-      sqlChanged = finalSql !== input.sql;
-      usedFastPath = true;
-    } catch {
-    }
-    if (!usedFastPath) {
-      const modifiedQuestion = buildModifiedQuestion(
-        input.question,
-        input.sqlModifications,
-        "v2"
-      );
-      finalQuestion = modifiedQuestion;
-      const queryResponse = await client.post(
-        queryEndpoint,
-        {
-          question: modifiedQuestion,
-          previous_sql: input.sql,
-          ...options?.maxRetry ? { max_retry: options.maxRetry } : {},
-          ...tenantSettings ? { tenant_settings: tenantSettings } : {},
-          ...databaseName ? { database: databaseName } : {},
-          ...metadata?.dialect ? { dialect: metadata.dialect } : {}
-        },
-        tenantId,
-        options?.userId,
-        options?.scopes,
-        signal,
-        sessionId
-      );
-      finalSql = queryResponse.sql;
-      paramMetadata = Array.isArray(queryResponse.params) ? queryResponse.params : [];
-      finalParams = queryEngine.mapGeneratedParams(paramMetadata);
-      applyDateRangeOverrides(
-        input.sqlModifications?.dateRange,
-        finalParams,
-        paramMetadata
-      );
-      rationale = queryResponse.rationale;
-      queryId = queryResponse.queryId;
-      sqlChanged = finalSql !== input.sql;
-    }
   } else if (hasSqlMods && !hasCustomSql) {
     const modifiedQuestion = buildModifiedQuestion(
       input.question,
@@ -1682,6 +1605,7 @@ async function modifyChart(client, queryEngine, input, options, signal) {
       queryEndpoint,
       {
         question: modifiedQuestion,
+        session_id: querypanelSessionId,
         previous_sql: input.sql,
         ...options?.maxRetry ? { max_retry: options.maxRetry } : {},
         ...tenantSettings ? { tenant_settings: tenantSettings } : {},
@@ -1784,6 +1708,7 @@ async function modifyChart(client, queryEngine, input, options, signal) {
     rationale,
     dialect: metadata?.dialect ?? "unknown",
     queryId,
+    querypanelSessionId,
     rows,
     fields: execution.fields,
     chart,