@querypanel/node-sdk 1.0.54 → 1.0.56

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>;
-    constructor(message: string, code: QueryErrorCode, details?: Record<string, unknown>);
+    readonly details?: Record<string, unknown> | undefined;
+    constructor(message: string, code: QueryErrorCode, details?: Record<string, unknown> | undefined);
     /**
      * Check if this is a moderation error
      */
@@ -741,6 +741,8 @@ interface ChartModifyOptions {
      * QueryPanel session ID for context-aware follow-ups.
      * Pass the querypanelSessionId from a previous ask() response
      * to preserve conversation history when modifying charts.
+     * If omitted, modifyChart starts a new QueryPanel session and sends
+     * the original question with modification hints.
      */
     querypanelSessionId?: string;
 }

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>;
-    constructor(message: string, code: QueryErrorCode, details?: Record<string, unknown>);
+    readonly details?: Record<string, unknown> | undefined;
+    constructor(message: string, code: QueryErrorCode, details?: Record<string, unknown> | undefined);
     /**
      * Check if this is a moderation error
      */
@@ -741,6 +741,8 @@ interface ChartModifyOptions {
      * QueryPanel session ID for context-aware follow-ups.
      * Pass the querypanelSessionId from a previous ask() response
      * to preserve conversation history when modifying charts.
+     * If omitted, modifyChart starts a new QueryPanel session and sends
+     * the original question with modification hints.
      */
     querypanelSessionId?: string;
 }

package/dist/index.js

@@ -1414,7 +1414,7 @@ function anonymizeResults(rows) {
 }
 
 // src/routes/modify.ts
-function buildModifiedQuestion(originalQuestion, modifications, pipeline) {
+function buildModifiedQuestion(originalQuestion, modifications, pipeline, hasSessionContext = false) {
   const hints = [];
   if (modifications.timeGranularity) {
     hints.push(`group results by ${modifications.timeGranularity}`);
@@ -1449,6 +1449,9 @@ function buildModifiedQuestion(originalQuestion, modifications, pipeline) {
   if (hints.length === 0) {
     return originalQuestion;
   }
+  if (pipeline === "v2" && hasSessionContext) {
+    return hints.join(", ");
+  }
   return `${originalQuestion} (${hints.join(", ")})`;
 }
 var START_PARAM_KEY_REGEX = /(^|_)(start|from)(_|$)/i;
@@ -1557,6 +1560,7 @@ async function modifyChart(client, queryEngine, input, options, signal) {
   const tenantId = resolveTenantId5(client, options?.tenantId);
   const sessionId = crypto4.randomUUID();
   const querypanelSessionId = options?.querypanelSessionId ?? sessionId;
+  const hasQuerypanelSessionContext = !!options?.querypanelSessionId;
   const chartType = options?.chartType ?? "vega-lite";
   const hasSqlMods = !!input.sqlModifications;
   const hasVizMods = !!input.vizModifications;
@@ -1593,7 +1597,8 @@ async function modifyChart(client, queryEngine, input, options, signal) {
     const modifiedQuestion = buildModifiedQuestion(
       input.question,
       input.sqlModifications,
-      options?.pipeline
+      options?.pipeline,
+      hasQuerypanelSessionContext
     );
     if (options?.pipeline === "v2") {
       finalQuestion = modifiedQuestion;