@superatomai/sdk-node 0.0.50-dsp → 0.0.51-dsp

package/dist/index.d.mts

@@ -291,6 +291,14 @@ type MatchTier = 'high' | 'near' | 'edit' | 'none';
  */
 interface ScriptBinding {
     recipeId: string;
+    /**
+     * The recipe's own version number at the moment THIS turn ran — not
+     * necessarily the recipe's current version. A later edit bumps the live
+     * version, so without this a past turn's binding would silently point at
+     * whatever the recipe looks like today instead of what actually produced
+     * that answer. Absent on bindings created before this field existed.
+     */
+    version?: number;
     /** Params the script ran with — the edit's starting point. */
     params: Record<string, any>;
     /** Recipe name at bind time (matcher catalog + user-facing confirmation). */
@@ -2548,6 +2556,18 @@ declare class ScriptStore {
     search(prompt: string, limit?: number): Promise<ScriptRecipe[]>;
     /** Fetch one recipe by id with its body loaded from disk. */
     get(id: string): Promise<ScriptRecipe | null>;
+    /**
+     * Fetch one recipe by id, pinned to a SPECIFIC past version rather than
+     * whatever is live today. Needed to render a past turn's conversation
+     * history accurately after the script has since been edited — get() alone
+     * always returns the CURRENT body, which may no longer be what that turn
+     * actually ran.
+     *
+     * Returns null (never a silently-wrong version) when the requested version
+     * isn't the current one and its archived body can't be found — e.g. the
+     * archive was cleaned up after the whole recipe was later deleted.
+     */
+    getVersion(id: string, version: number): Promise<ScriptRecipe | null>;
     /** Create or update a recipe (metadata upsert + body write when changed). */
     save(recipe: ScriptRecipe): Promise<void>;
     /**
@@ -3470,7 +3490,14 @@ declare class Thread {
      * See backend/docs/SCRIPT-EDIT-DESIGN.md § Component 1.
      */
     getScriptBindings(limit?: number, currentUIBlockId?: string): Record<string, any>[];
-    getConversationContext(limit?: number, currentUIBlockId?: string): string;
+    /**
+     * @param getScriptBody Resolves a script-backed turn's EXACT script, pinned
+     * to the version that turn actually ran — never today's (possibly
+     * since-edited) live body. Owned by the caller (needs a ScriptStore, which
+     * Thread doesn't hold) — omit it and every turn falls back to the
+     * no-script summary below.
+     */
+    getConversationContext(limit?: number, currentUIBlockId?: string, getScriptBody?: (recipeId: string, version: number) => Promise<string | null>): Promise<string>;
     /**
      * Convert Thread to JSON-serializable object
      */

package/dist/index.d.ts

@@ -291,6 +291,14 @@ type MatchTier = 'high' | 'near' | 'edit' | 'none';
  */
 interface ScriptBinding {
     recipeId: string;
+    /**
+     * The recipe's own version number at the moment THIS turn ran — not
+     * necessarily the recipe's current version. A later edit bumps the live
+     * version, so without this a past turn's binding would silently point at
+     * whatever the recipe looks like today instead of what actually produced
+     * that answer. Absent on bindings created before this field existed.
+     */
+    version?: number;
     /** Params the script ran with — the edit's starting point. */
     params: Record<string, any>;
     /** Recipe name at bind time (matcher catalog + user-facing confirmation). */
@@ -2548,6 +2556,18 @@ declare class ScriptStore {
     search(prompt: string, limit?: number): Promise<ScriptRecipe[]>;
     /** Fetch one recipe by id with its body loaded from disk. */
     get(id: string): Promise<ScriptRecipe | null>;
+    /**
+     * Fetch one recipe by id, pinned to a SPECIFIC past version rather than
+     * whatever is live today. Needed to render a past turn's conversation
+     * history accurately after the script has since been edited — get() alone
+     * always returns the CURRENT body, which may no longer be what that turn
+     * actually ran.
+     *
+     * Returns null (never a silently-wrong version) when the requested version
+     * isn't the current one and its archived body can't be found — e.g. the
+     * archive was cleaned up after the whole recipe was later deleted.
+     */
+    getVersion(id: string, version: number): Promise<ScriptRecipe | null>;
     /** Create or update a recipe (metadata upsert + body write when changed). */
     save(recipe: ScriptRecipe): Promise<void>;
     /**
@@ -3470,7 +3490,14 @@ declare class Thread {
      * See backend/docs/SCRIPT-EDIT-DESIGN.md § Component 1.
      */
     getScriptBindings(limit?: number, currentUIBlockId?: string): Record<string, any>[];
-    getConversationContext(limit?: number, currentUIBlockId?: string): string;
+    /**
+     * @param getScriptBody Resolves a script-backed turn's EXACT script, pinned
+     * to the version that turn actually ran — never today's (possibly
+     * since-edited) live body. Owned by the caller (needs a ScriptStore, which
+     * Thread doesn't hold) — omit it and every turn falls back to the
+     * no-script summary below.
+     */
+    getConversationContext(limit?: number, currentUIBlockId?: string, getScriptBody?: (recipeId: string, version: number) => Promise<string | null>): Promise<string>;
     /**
      * Convert Thread to JSON-serializable object
      */

package/dist/index.js

@@ -1502,7 +1502,14 @@ var Thread = class {
     }
     return out;
   }
-  getConversationContext(limit = 5, currentUIBlockId) {
+  /**
+   * @param getScriptBody Resolves a script-backed turn's EXACT script, pinned
+   * to the version that turn actually ran — never today's (possibly
+   * since-edited) live body. Owned by the caller (needs a ScriptStore, which
+   * Thread doesn't hold) — omit it and every turn falls back to the
+   * no-script summary below.
+   */
+  async getConversationContext(limit = 5, currentUIBlockId, getScriptBody) {
     if (limit === 0) {
       return "";
     }
@@ -1512,53 +1519,62 @@ var Thread = class {
     }
     const recentBlocks = allBlocks.slice(-limit);
     const contextLines = [];
-    recentBlocks.forEach((block, index) => {
-      const questionNum = index + 1;
+    for (const block of recentBlocks) {
       const question = block.getUserQuestion();
       const metadata = block.getComponentMetadata();
       const programAnswer = block.getProgramAnswer();
       const textResponse = block.getTextResponse();
-      let assistantResponse = "";
+      const scriptBinding = block.getScriptBinding();
       const responseParts = [];
+      let analysisText = "";
       const hasProgramAnswer = programAnswer && programAnswer.trim().length > 0;
       if (hasProgramAnswer) {
         const trimmed = programAnswer.trim();
-        responseParts.push(truncateKeepingMethodLine(trimmed, 2e3));
+        analysisText = truncateKeepingMethodLine(trimmed, 2e3);
       } else {
         const hasTextResponse = textResponse && textResponse.trim().length > 0;
         if (hasTextResponse) {
           const cleaned = cleanAssistantText(textResponse);
           if (cleaned) {
-            const trimmedText = cleaned.length > 800 ? cleaned.substring(0, 800) + "...[truncated]" : cleaned;
-            responseParts.push(trimmedText);
+            analysisText = cleaned.length > 800 ? cleaned.substring(0, 800) + "...[truncated]" : cleaned;
           }
         }
       }
-      const scriptBinding = block.getScriptBinding();
-      if (scriptBinding?.params && Object.keys(scriptBinding.params).length > 0) {
-        responseParts.push(`Parameters used: ${JSON.stringify(scriptBinding.params)}`);
-      }
-      const hasComponent = metadata && Object.keys(metadata).length > 0 && metadata.type;
-      if (hasComponent) {
-        const label = metadata.name ? `${metadata.type} \u2014 ${metadata.name}` : metadata.type;
-        responseParts.push(`[Rendered: ${label}]`);
-        const innerComponents = metadata.props?.config?.components;
-        if (Array.isArray(innerComponents) && innerComponents.length > 0) {
-          const summary = innerComponents.map((c) => c?.props?.title ? `${c.name} ("${c.props.title}")` : c?.name).filter(Boolean).join(", ");
-          if (summary) responseParts.push(`  Components: ${summary}`);
+      if (analysisText) responseParts.push(analysisText);
+      let scriptAttached = false;
+      if (scriptBinding?.recipeId && scriptBinding.version !== void 0 && getScriptBody) {
+        try {
+          const body = await getScriptBody(scriptBinding.recipeId, scriptBinding.version);
+          if (body) {
+            responseParts.push(`Script used:
+${body}`);
+            scriptAttached = true;
+          }
+        } catch {
         }
       }
-      if (responseParts.length > 0) {
-        assistantResponse = responseParts.join("\n");
-      } else {
-        assistantResponse = "No response generated";
+      if (!scriptAttached) {
+        if (scriptBinding?.params && Object.keys(scriptBinding.params).length > 0) {
+          responseParts.push(`Parameters used: ${JSON.stringify(scriptBinding.params)}`);
+        }
+        const hasComponent = metadata && Object.keys(metadata).length > 0 && metadata.type;
+        if (hasComponent) {
+          const label = metadata.name ? `${metadata.type} \u2014 ${metadata.name}` : metadata.type;
+          responseParts.push(`[Rendered: ${label}]`);
+          const innerComponents = metadata.props?.config?.components;
+          if (Array.isArray(innerComponents) && innerComponents.length > 0) {
+            const summary = innerComponents.map((c) => c?.props?.title ? `${c.name} ("${c.props.title}")` : c?.name).filter(Boolean).join(", ");
+            if (summary) responseParts.push(`  Components: ${summary}`);
+          }
+        }
       }
+      const assistantResponse = responseParts.length > 0 ? responseParts.join("\n") : "No response generated";
       contextLines.push(`User:
  ${question}`);
       contextLines.push(`Assistant:
  ${assistantResponse}`);
       contextLines.push("---");
-    });
+    }
     return contextLines.join("\n").trim();
   }
   /**
@@ -10232,16 +10248,20 @@ ${formatted}`;
         } catch (execError) {
           const errorMsg = execError instanceof Error ? execError.message : typeof execError === "object" && execError !== null ? execError.message || execError.error || JSON.stringify(execError) : String(execError);
           logger.warn(`[SourceAgent:${this.tool.name}] Tool execution failed (attempt ${this.attempts}/${this.config.maxRetries}): ${errorMsg}`);
+          let hint = "";
+          if (this.extractSourceType() === "netsuite" && /"o:errorCode"\s*:\s*"UNEXPECTED_ERROR"/.test(errorMsg)) {
+            hint = ` This gives no column/syntax detail \u2014 it usually means a column that doesn't exist on that table (wrong field name), or a column referenced outside the CTE that selects it (a CTE only exposes columns it projects). Recheck every column name against the schema.`;
+          }
           if (this.streamBuffer.hasCallback()) {
             const querySeconds = ((Date.now() - queryStartTime) / 1e3).toFixed(1);
             this.streamBuffer.write(`__QUERY_TIMER_DONE_${querySeconds}__
 
 `);
-            this.streamBuffer.write(`\u274C **Query failed:** ${errorMsg}
+            this.streamBuffer.write(`\u274C **Query failed:** ${errorMsg}${hint}
 
 `);
           }
-          return `\u274C ERROR: ${errorMsg}
+          return `\u274C ERROR: ${errorMsg}${hint}
 
 Analyze the error and try again with a corrected query.`;
         }
@@ -11254,13 +11274,34 @@ function findHardcodedComponentText(scriptBody) {
   }
   return violations;
 }
+function findHardcodedEntityNames(source, entityDisplayNames) {
+  const names = entityDisplayNames.filter((n) => n && n.trim().length >= 2);
+  if (!names.length) return [];
+  const violations = [];
+  const seen = /* @__PURE__ */ new Set();
+  for (const chunk of literalChunks(source)) {
+    const trimmed = chunk.trim();
+    if (!trimmed) continue;
+    for (const name of names) {
+      if (seen.has(name)) continue;
+      if (trimmed.toLowerCase().includes(name.toLowerCase())) {
+        seen.add(name);
+        violations.push({
+          text: trimmed.slice(0, 120),
+          reason: `"${name}" \u2014 a name resolved via resolve_entities this turn \u2014 is written as fixed text. Read it from \`data\`/\`datasets\` instead (e.g. \`data[0]?.<name_column>\`), or filter/join on the resolved id (\`params.<name>Id\`) rather than the name string \u2014 a name typed as a literal is exactly as wrong on replay as a number typed as a literal: the id a future run resolves will differ, but the printed/filtered name will not.`
+        });
+      }
+    }
+  }
+  return violations;
+}
 function formatLintError(violations) {
   if (!violations.length) return null;
   const lines = violations.map((v) => `  \u2022 "${v.text}" \u2014 ${v.reason}`).join("\n");
-  return `The analysis body hardcodes ${violations.length} number${violations.length > 1 ? "s" : ""} as text:
+  return `The analysis body hardcodes ${violations.length} value${violations.length > 1 ? "s" : ""} as text:
 ${lines}
 
-FIX: every figure must come from the data via a fmt helper \u2014 fmt.money, fmt.num, fmt.pct, fmt.delta, fmt.count, fmt.date, fmt.range. A number typed into a string is frozen at authoring time and will be wrong on replay when the data changes. Rewording will not clear this \u2014 only moving the number out of the string will. A constant the code itself uses is fine once interpolated from a variable; a figure read off the data is not.`;
+FIX: every figure must come from the data via a fmt helper \u2014 fmt.money, fmt.num, fmt.pct, fmt.delta, fmt.count, fmt.date, fmt.range \u2014 and every resolved name must come from \`data\`/\`datasets\`, not be typed in. A value typed into a string is frozen at authoring time and will be wrong on replay when the data or params change. Rewording will not clear this \u2014 only moving the value out of the string will. A constant the code itself uses is fine once interpolated from a variable; a figure or name read off the data is not.`;
 }
 
 // src/userResponse/scripts/component-validator.ts
@@ -12031,7 +12072,8 @@ ${catalog}`,
     this.renderComponentAttempts++;
     const hardcodedAggregates = findHardcodedAggregates(body);
     const hardcodedComponentText = findHardcodedComponentText(body);
-    const componentViolations = [...hardcodedAggregates, ...hardcodedComponentText];
+    const hardcodedEntityNames = findHardcodedEntityNames(body, this.resolvedEntities.map((e) => e.displayName));
+    const componentViolations = [...hardcodedAggregates, ...hardcodedComponentText, ...hardcodedEntityNames];
     if (componentViolations.length > 0) {
       const named = componentViolations.map((v) => `"${v.text}"`).join(", ");
       const repeatedTexts = componentViolations.map((v) => v.text).filter((t) => this.lastComponentsViolationTexts.includes(t));
@@ -12114,7 +12156,10 @@ Now write your answer for the user. Do not restate the component list.`;
       return `Reached the max of ${MAX_ANALYSIS_ATTEMPTS} write_analysis attempts. Write your final analysis as prose instead, using the values from the script result exactly as shown.`;
     }
     this.scriptState.analysisAttempts++;
-    const violations = lintAnalysisBody(body);
+    const violations = [
+      ...lintAnalysisBody(body),
+      ...findHardcodedEntityNames(body, this.resolvedEntities.map((e) => e.displayName))
+    ];
     const repeatedTexts = violations.map((v) => v.text).filter((t) => this.scriptState.lastAnalysisViolationTexts.includes(t));
     this.scriptState.lastAnalysisViolationTexts = violations.map((v) => v.text);
     const lintError = formatLintError(violations);
@@ -12192,6 +12237,14 @@ FIX: derive every such value from the data itself \u2014 \`data.length\`, a real
 ${lines}
 
 FIX: build the title/description with a template literal that reads the value from \`data\`/\`datasets\` at render time (e.g. \` \\\`Total: \\\${fmt.money(datasets.summary[0].total)}\\\` \`), never a number typed directly into the string.`;
+    }
+    const hardcodedEntityNames = findHardcodedEntityNames(scriptBody, this.resolvedEntities.map((e) => e.displayName));
+    if (hardcodedEntityNames.length > 0) {
+      const lines = hardcodedEntityNames.map((v) => `  \u2022 "${v.text}" \u2014 ${v.reason}`).join("\n");
+      return `write_script rejected: the script hardcodes ${hardcodedEntityNames.length} resolved entity name(s) as fixed text:
+${lines}
+
+FIX: read the name from \`data\`/\`datasets\` at render time, or filter/join on the resolved id (\`params.<name>Id\`) instead of the name string.`;
     }
     const draft = await this.scriptStore.saveDraft({
       recipeId: this.scriptState.recipeId ?? void 0,
@@ -13397,6 +13450,22 @@ function stripNamedExport(body, fnName) {
   }
   return body.slice(0, start);
 }
+function extractNamedExport(body, fnName) {
+  const start = body.search(new RegExp(`export\\s+(?:async\\s+)?function\\s+${fnName}\\b`));
+  if (start === -1) return null;
+  const open = body.indexOf("{", start);
+  if (open === -1) return null;
+  let depth = 0;
+  for (let i = open; i < body.length; i++) {
+    const c = body[i];
+    if (c === "{") depth++;
+    else if (c === "}") {
+      depth--;
+      if (depth === 0) return body.slice(start, i + 1);
+    }
+  }
+  return null;
+}
 function normalizeScriptBody(scriptBody) {
   let i = 0;
   const n = scriptBody.length;
@@ -13496,6 +13565,37 @@ var ScriptStore = class {
       return null;
     }
   }
+  /**
+   * Fetch one recipe by id, pinned to a SPECIFIC past version rather than
+   * whatever is live today. Needed to render a past turn's conversation
+   * history accurately after the script has since been edited — get() alone
+   * always returns the CURRENT body, which may no longer be what that turn
+   * actually ran.
+   *
+   * Returns null (never a silently-wrong version) when the requested version
+   * isn't the current one and its archived body can't be found — e.g. the
+   * archive was cleaned up after the whole recipe was later deleted.
+   */
+  async getVersion(id, version) {
+    if (!this.store) return null;
+    try {
+      const row = await this.store.getById(id);
+      if (!row) return null;
+      if (version === row.version) {
+        return this.rowToRecipe(row, this.readBody(row.fileBase));
+      }
+      const archivePath = path6.join(this.archiveDir, `${row.fileBase}.v${version}.ts`);
+      if (!fs6.existsSync(archivePath)) {
+        logger.warn(`[ScriptStore] getVersion(${id}, v${version}) \u2014 archived body not found, recipe is now at v${row.version}`);
+        return null;
+      }
+      const body = fs6.readFileSync(archivePath, "utf-8");
+      return this.rowToRecipe(row, body);
+    } catch (err) {
+      logger.warn(`[ScriptStore] getVersion(${id}, v${version}) failed: ${err}`);
+      return null;
+    }
+  }
   // ============================================
   // Write
   // ============================================
@@ -13535,7 +13635,20 @@ var ScriptStore = class {
     const reuse = existing && existing.status === "draft" && existing.turnId === input.turnId;
     const id = reuse ? existing.id : `script_${Date.now()}_${(0, import_crypto5.randomBytes)(3).toString("hex")}`;
     const fileBase = reuse ? existing.fileBase : input.parentId ? `${this.toSlug(input.name)}-${id.slice(-6)}` : await this.computeFileBase(input.name, id);
-    const normalized = normalizeScriptBody(input.scriptBody);
+    let normalized = normalizeScriptBody(input.scriptBody);
+    if (reuse) {
+      const existingBody = this.readBody(fileBase);
+      if (existingBody) {
+        for (const fnName of ["getComponents", "getAnalysis"]) {
+          if (new RegExp(`export\\s+(?:async\\s+)?function\\s+${fnName}\\b`).test(normalized)) continue;
+          const carried = extractNamedExport(existingBody, fnName);
+          if (carried) normalized = `${normalized.trimEnd()}
+
+${carried.trim()}
+`;
+        }
+      }
+    }
     const bodyHash = this.hash(normalized);
     this.writeBody(fileBase, normalized);
     const recipe = {
@@ -17785,6 +17898,7 @@ RENDERING REQUIREMENT (overrides the default choice of component): ${scriptMatch
                 // ("use mode instead") can edit THIS script.
                 scriptBinding: {
                   recipeId: scriptMatch.recipe.id,
+                  version: scriptMatch.recipe.version,
                   params: scriptMatch.extractedParams || {},
                   name: scriptMatch.recipe.name,
                   columns: scriptResult.executedQueries?.[0]?.data?.[0] ? Object.keys(scriptResult.executedQueries[0].data[0]) : void 0
@@ -18212,6 +18326,7 @@ RENDERING REQUIREMENT (overrides the default choice of component): ${carriedRequ
         if (committed) {
           scriptBinding = {
             recipeId: committed.id,
+            version: committed.version,
             params: {},
             name: committed.name,
             columns: authored.executedQueries?.[0]?.data?.[0] ? Object.keys(authored.executedQueries[0].data[0]) : void 0
@@ -18269,6 +18384,7 @@ RENDERING REQUIREMENT (overrides the default choice of component): ${carriedRequ
         }
         scriptBinding = {
           recipeId: authored.recipeId,
+          version: promoted?.version,
           params: {},
           name: authored.name,
           columns: authored.executedQueries?.[0]?.data?.[0] ? Object.keys(authored.executedQueries[0].data[0]) : void 0
@@ -18587,7 +18703,7 @@ var CONTEXT_CONFIG = {
    * Set to 0 to disable conversation history
    * Higher values provide more context but may increase token usage
    */
-  MAX_CONVERSATION_CONTEXT_BLOCKS: 3
+  MAX_CONVERSATION_CONTEXT_BLOCKS: 2
 };
 
 // src/handlers/user-prompt-request.ts
@@ -18685,7 +18801,12 @@ var get_user_request = async (data, components, sendMessage, anthropicApiKey, gr
   }
   const requestStartTime = Date.now();
   logger.info(`Starting user prompt request with ${components.length} components`);
-  const conversationHistory = thread.getConversationContext(CONTEXT_CONFIG.MAX_CONVERSATION_CONTEXT_BLOCKS, existingUiBlockId);
+  const conversationScriptStore = new ScriptStore({ collections, projectId: process.env.SUPERATOM_PROJECT_ID || "" });
+  const conversationHistory = await thread.getConversationContext(
+    CONTEXT_CONFIG.MAX_CONVERSATION_CONTEXT_BLOCKS,
+    existingUiBlockId,
+    async (recipeId, version) => (await conversationScriptStore.getVersion(recipeId, version))?.scriptBody ?? null
+  );
   const MAX_EDIT_CANDIDATES = 5;
   let activeScriptBindings = thread.getScriptBindings(MAX_EDIT_CANDIDATES, existingUiBlockId);
   if (activeScriptBindings.length === 0 && collections?.["conversation-history"]?.["threadScriptBindings"]) {
@@ -19333,7 +19454,7 @@ async function handleActionsRequest(data, sendMessage, anthropicApiKey, groqApiK
     logger.debug(`[ACTIONS_REQ ${id}] Component: ${component?.name || "unknown"} (${component?.type || "unknown"})`);
     logger.debug(`[ACTIONS_REQ ${id}] Component data available: ${componentData ? "yes" : "no"}`);
     logger.debug(`[ACTIONS_REQ ${id}] Extracting conversation history (max ${CONTEXT_CONFIG.MAX_CONVERSATION_CONTEXT_BLOCKS} blocks)`);
-    const conversationHistory = thread.getConversationContext(CONTEXT_CONFIG.MAX_CONVERSATION_CONTEXT_BLOCKS, uiBlockId);
+    const conversationHistory = await thread.getConversationContext(CONTEXT_CONFIG.MAX_CONVERSATION_CONTEXT_BLOCKS, uiBlockId);
     const historyLineCount = conversationHistory.split("\n").filter((l) => l.trim()).length;
     logger.info(`[ACTIONS_REQ ${id}] Conversation history extracted: ${historyLineCount} lines`);
     logger.debug(`[ACTIONS_REQ ${id}] Conversation history preview:

package/dist/index.mjs

@@ -1443,7 +1443,14 @@ var Thread = class {
     }
     return out;
   }
-  getConversationContext(limit = 5, currentUIBlockId) {
+  /**
+   * @param getScriptBody Resolves a script-backed turn's EXACT script, pinned
+   * to the version that turn actually ran — never today's (possibly
+   * since-edited) live body. Owned by the caller (needs a ScriptStore, which
+   * Thread doesn't hold) — omit it and every turn falls back to the
+   * no-script summary below.
+   */
+  async getConversationContext(limit = 5, currentUIBlockId, getScriptBody) {
     if (limit === 0) {
       return "";
     }
@@ -1453,53 +1460,62 @@ var Thread = class {
     }
     const recentBlocks = allBlocks.slice(-limit);
     const contextLines = [];
-    recentBlocks.forEach((block, index) => {
-      const questionNum = index + 1;
+    for (const block of recentBlocks) {
       const question = block.getUserQuestion();
       const metadata = block.getComponentMetadata();
       const programAnswer = block.getProgramAnswer();
       const textResponse = block.getTextResponse();
-      let assistantResponse = "";
+      const scriptBinding = block.getScriptBinding();
       const responseParts = [];
+      let analysisText = "";
       const hasProgramAnswer = programAnswer && programAnswer.trim().length > 0;
       if (hasProgramAnswer) {
         const trimmed = programAnswer.trim();
-        responseParts.push(truncateKeepingMethodLine(trimmed, 2e3));
+        analysisText = truncateKeepingMethodLine(trimmed, 2e3);
       } else {
         const hasTextResponse = textResponse && textResponse.trim().length > 0;
         if (hasTextResponse) {
           const cleaned = cleanAssistantText(textResponse);
           if (cleaned) {
-            const trimmedText = cleaned.length > 800 ? cleaned.substring(0, 800) + "...[truncated]" : cleaned;
-            responseParts.push(trimmedText);
+            analysisText = cleaned.length > 800 ? cleaned.substring(0, 800) + "...[truncated]" : cleaned;
           }
         }
       }
-      const scriptBinding = block.getScriptBinding();
-      if (scriptBinding?.params && Object.keys(scriptBinding.params).length > 0) {
-        responseParts.push(`Parameters used: ${JSON.stringify(scriptBinding.params)}`);
-      }
-      const hasComponent = metadata && Object.keys(metadata).length > 0 && metadata.type;
-      if (hasComponent) {
-        const label = metadata.name ? `${metadata.type} \u2014 ${metadata.name}` : metadata.type;
-        responseParts.push(`[Rendered: ${label}]`);
-        const innerComponents = metadata.props?.config?.components;
-        if (Array.isArray(innerComponents) && innerComponents.length > 0) {
-          const summary = innerComponents.map((c) => c?.props?.title ? `${c.name} ("${c.props.title}")` : c?.name).filter(Boolean).join(", ");
-          if (summary) responseParts.push(`  Components: ${summary}`);
+      if (analysisText) responseParts.push(analysisText);
+      let scriptAttached = false;
+      if (scriptBinding?.recipeId && scriptBinding.version !== void 0 && getScriptBody) {
+        try {
+          const body = await getScriptBody(scriptBinding.recipeId, scriptBinding.version);
+          if (body) {
+            responseParts.push(`Script used:
+${body}`);
+            scriptAttached = true;
+          }
+        } catch {
         }
       }
-      if (responseParts.length > 0) {
-        assistantResponse = responseParts.join("\n");
-      } else {
-        assistantResponse = "No response generated";
+      if (!scriptAttached) {
+        if (scriptBinding?.params && Object.keys(scriptBinding.params).length > 0) {
+          responseParts.push(`Parameters used: ${JSON.stringify(scriptBinding.params)}`);
+        }
+        const hasComponent = metadata && Object.keys(metadata).length > 0 && metadata.type;
+        if (hasComponent) {
+          const label = metadata.name ? `${metadata.type} \u2014 ${metadata.name}` : metadata.type;
+          responseParts.push(`[Rendered: ${label}]`);
+          const innerComponents = metadata.props?.config?.components;
+          if (Array.isArray(innerComponents) && innerComponents.length > 0) {
+            const summary = innerComponents.map((c) => c?.props?.title ? `${c.name} ("${c.props.title}")` : c?.name).filter(Boolean).join(", ");
+            if (summary) responseParts.push(`  Components: ${summary}`);
+          }
+        }
       }
+      const assistantResponse = responseParts.length > 0 ? responseParts.join("\n") : "No response generated";
       contextLines.push(`User:
  ${question}`);
       contextLines.push(`Assistant:
  ${assistantResponse}`);
       contextLines.push("---");
-    });
+    }
     return contextLines.join("\n").trim();
   }
   /**
@@ -10173,16 +10189,20 @@ ${formatted}`;
         } catch (execError) {
           const errorMsg = execError instanceof Error ? execError.message : typeof execError === "object" && execError !== null ? execError.message || execError.error || JSON.stringify(execError) : String(execError);
           logger.warn(`[SourceAgent:${this.tool.name}] Tool execution failed (attempt ${this.attempts}/${this.config.maxRetries}): ${errorMsg}`);
+          let hint = "";
+          if (this.extractSourceType() === "netsuite" && /"o:errorCode"\s*:\s*"UNEXPECTED_ERROR"/.test(errorMsg)) {
+            hint = ` This gives no column/syntax detail \u2014 it usually means a column that doesn't exist on that table (wrong field name), or a column referenced outside the CTE that selects it (a CTE only exposes columns it projects). Recheck every column name against the schema.`;
+          }
           if (this.streamBuffer.hasCallback()) {
             const querySeconds = ((Date.now() - queryStartTime) / 1e3).toFixed(1);
             this.streamBuffer.write(`__QUERY_TIMER_DONE_${querySeconds}__
 
 `);
-            this.streamBuffer.write(`\u274C **Query failed:** ${errorMsg}
+            this.streamBuffer.write(`\u274C **Query failed:** ${errorMsg}${hint}
 
 `);
           }
-          return `\u274C ERROR: ${errorMsg}
+          return `\u274C ERROR: ${errorMsg}${hint}
 
 Analyze the error and try again with a corrected query.`;
         }
@@ -11195,13 +11215,34 @@ function findHardcodedComponentText(scriptBody) {
   }
   return violations;
 }
+function findHardcodedEntityNames(source, entityDisplayNames) {
+  const names = entityDisplayNames.filter((n) => n && n.trim().length >= 2);
+  if (!names.length) return [];
+  const violations = [];
+  const seen = /* @__PURE__ */ new Set();
+  for (const chunk of literalChunks(source)) {
+    const trimmed = chunk.trim();
+    if (!trimmed) continue;
+    for (const name of names) {
+      if (seen.has(name)) continue;
+      if (trimmed.toLowerCase().includes(name.toLowerCase())) {
+        seen.add(name);
+        violations.push({
+          text: trimmed.slice(0, 120),
+          reason: `"${name}" \u2014 a name resolved via resolve_entities this turn \u2014 is written as fixed text. Read it from \`data\`/\`datasets\` instead (e.g. \`data[0]?.<name_column>\`), or filter/join on the resolved id (\`params.<name>Id\`) rather than the name string \u2014 a name typed as a literal is exactly as wrong on replay as a number typed as a literal: the id a future run resolves will differ, but the printed/filtered name will not.`
+        });
+      }
+    }
+  }
+  return violations;
+}
 function formatLintError(violations) {
   if (!violations.length) return null;
   const lines = violations.map((v) => `  \u2022 "${v.text}" \u2014 ${v.reason}`).join("\n");
-  return `The analysis body hardcodes ${violations.length} number${violations.length > 1 ? "s" : ""} as text:
+  return `The analysis body hardcodes ${violations.length} value${violations.length > 1 ? "s" : ""} as text:
 ${lines}
 
-FIX: every figure must come from the data via a fmt helper \u2014 fmt.money, fmt.num, fmt.pct, fmt.delta, fmt.count, fmt.date, fmt.range. A number typed into a string is frozen at authoring time and will be wrong on replay when the data changes. Rewording will not clear this \u2014 only moving the number out of the string will. A constant the code itself uses is fine once interpolated from a variable; a figure read off the data is not.`;
+FIX: every figure must come from the data via a fmt helper \u2014 fmt.money, fmt.num, fmt.pct, fmt.delta, fmt.count, fmt.date, fmt.range \u2014 and every resolved name must come from \`data\`/\`datasets\`, not be typed in. A value typed into a string is frozen at authoring time and will be wrong on replay when the data or params change. Rewording will not clear this \u2014 only moving the value out of the string will. A constant the code itself uses is fine once interpolated from a variable; a figure or name read off the data is not.`;
 }
 
 // src/userResponse/scripts/component-validator.ts
@@ -11972,7 +12013,8 @@ ${catalog}`,
     this.renderComponentAttempts++;
     const hardcodedAggregates = findHardcodedAggregates(body);
     const hardcodedComponentText = findHardcodedComponentText(body);
-    const componentViolations = [...hardcodedAggregates, ...hardcodedComponentText];
+    const hardcodedEntityNames = findHardcodedEntityNames(body, this.resolvedEntities.map((e) => e.displayName));
+    const componentViolations = [...hardcodedAggregates, ...hardcodedComponentText, ...hardcodedEntityNames];
     if (componentViolations.length > 0) {
       const named = componentViolations.map((v) => `"${v.text}"`).join(", ");
       const repeatedTexts = componentViolations.map((v) => v.text).filter((t) => this.lastComponentsViolationTexts.includes(t));
@@ -12055,7 +12097,10 @@ Now write your answer for the user. Do not restate the component list.`;
       return `Reached the max of ${MAX_ANALYSIS_ATTEMPTS} write_analysis attempts. Write your final analysis as prose instead, using the values from the script result exactly as shown.`;
     }
     this.scriptState.analysisAttempts++;
-    const violations = lintAnalysisBody(body);
+    const violations = [
+      ...lintAnalysisBody(body),
+      ...findHardcodedEntityNames(body, this.resolvedEntities.map((e) => e.displayName))
+    ];
     const repeatedTexts = violations.map((v) => v.text).filter((t) => this.scriptState.lastAnalysisViolationTexts.includes(t));
     this.scriptState.lastAnalysisViolationTexts = violations.map((v) => v.text);
     const lintError = formatLintError(violations);
@@ -12133,6 +12178,14 @@ FIX: derive every such value from the data itself \u2014 \`data.length\`, a real
 ${lines}
 
 FIX: build the title/description with a template literal that reads the value from \`data\`/\`datasets\` at render time (e.g. \` \\\`Total: \\\${fmt.money(datasets.summary[0].total)}\\\` \`), never a number typed directly into the string.`;
+    }
+    const hardcodedEntityNames = findHardcodedEntityNames(scriptBody, this.resolvedEntities.map((e) => e.displayName));
+    if (hardcodedEntityNames.length > 0) {
+      const lines = hardcodedEntityNames.map((v) => `  \u2022 "${v.text}" \u2014 ${v.reason}`).join("\n");
+      return `write_script rejected: the script hardcodes ${hardcodedEntityNames.length} resolved entity name(s) as fixed text:
+${lines}
+
+FIX: read the name from \`data\`/\`datasets\` at render time, or filter/join on the resolved id (\`params.<name>Id\`) instead of the name string.`;
     }
     const draft = await this.scriptStore.saveDraft({
       recipeId: this.scriptState.recipeId ?? void 0,
@@ -13338,6 +13391,22 @@ function stripNamedExport(body, fnName) {
   }
   return body.slice(0, start);
 }
+function extractNamedExport(body, fnName) {
+  const start = body.search(new RegExp(`export\\s+(?:async\\s+)?function\\s+${fnName}\\b`));
+  if (start === -1) return null;
+  const open = body.indexOf("{", start);
+  if (open === -1) return null;
+  let depth = 0;
+  for (let i = open; i < body.length; i++) {
+    const c = body[i];
+    if (c === "{") depth++;
+    else if (c === "}") {
+      depth--;
+      if (depth === 0) return body.slice(start, i + 1);
+    }
+  }
+  return null;
+}
 function normalizeScriptBody(scriptBody) {
   let i = 0;
   const n = scriptBody.length;
@@ -13437,6 +13506,37 @@ var ScriptStore = class {
       return null;
     }
   }
+  /**
+   * Fetch one recipe by id, pinned to a SPECIFIC past version rather than
+   * whatever is live today. Needed to render a past turn's conversation
+   * history accurately after the script has since been edited — get() alone
+   * always returns the CURRENT body, which may no longer be what that turn
+   * actually ran.
+   *
+   * Returns null (never a silently-wrong version) when the requested version
+   * isn't the current one and its archived body can't be found — e.g. the
+   * archive was cleaned up after the whole recipe was later deleted.
+   */
+  async getVersion(id, version) {
+    if (!this.store) return null;
+    try {
+      const row = await this.store.getById(id);
+      if (!row) return null;
+      if (version === row.version) {
+        return this.rowToRecipe(row, this.readBody(row.fileBase));
+      }
+      const archivePath = path6.join(this.archiveDir, `${row.fileBase}.v${version}.ts`);
+      if (!fs6.existsSync(archivePath)) {
+        logger.warn(`[ScriptStore] getVersion(${id}, v${version}) \u2014 archived body not found, recipe is now at v${row.version}`);
+        return null;
+      }
+      const body = fs6.readFileSync(archivePath, "utf-8");
+      return this.rowToRecipe(row, body);
+    } catch (err) {
+      logger.warn(`[ScriptStore] getVersion(${id}, v${version}) failed: ${err}`);
+      return null;
+    }
+  }
   // ============================================
   // Write
   // ============================================
@@ -13476,7 +13576,20 @@ var ScriptStore = class {
     const reuse = existing && existing.status === "draft" && existing.turnId === input.turnId;
     const id = reuse ? existing.id : `script_${Date.now()}_${randomBytes(3).toString("hex")}`;
     const fileBase = reuse ? existing.fileBase : input.parentId ? `${this.toSlug(input.name)}-${id.slice(-6)}` : await this.computeFileBase(input.name, id);
-    const normalized = normalizeScriptBody(input.scriptBody);
+    let normalized = normalizeScriptBody(input.scriptBody);
+    if (reuse) {
+      const existingBody = this.readBody(fileBase);
+      if (existingBody) {
+        for (const fnName of ["getComponents", "getAnalysis"]) {
+          if (new RegExp(`export\\s+(?:async\\s+)?function\\s+${fnName}\\b`).test(normalized)) continue;
+          const carried = extractNamedExport(existingBody, fnName);
+          if (carried) normalized = `${normalized.trimEnd()}
+
+${carried.trim()}
+`;
+        }
+      }
+    }
     const bodyHash = this.hash(normalized);
     this.writeBody(fileBase, normalized);
     const recipe = {
@@ -17726,6 +17839,7 @@ RENDERING REQUIREMENT (overrides the default choice of component): ${scriptMatch
                 // ("use mode instead") can edit THIS script.
                 scriptBinding: {
                   recipeId: scriptMatch.recipe.id,
+                  version: scriptMatch.recipe.version,
                   params: scriptMatch.extractedParams || {},
                   name: scriptMatch.recipe.name,
                   columns: scriptResult.executedQueries?.[0]?.data?.[0] ? Object.keys(scriptResult.executedQueries[0].data[0]) : void 0
@@ -18153,6 +18267,7 @@ RENDERING REQUIREMENT (overrides the default choice of component): ${carriedRequ
         if (committed) {
           scriptBinding = {
             recipeId: committed.id,
+            version: committed.version,
             params: {},
             name: committed.name,
             columns: authored.executedQueries?.[0]?.data?.[0] ? Object.keys(authored.executedQueries[0].data[0]) : void 0
@@ -18210,6 +18325,7 @@ RENDERING REQUIREMENT (overrides the default choice of component): ${carriedRequ
         }
         scriptBinding = {
           recipeId: authored.recipeId,
+          version: promoted?.version,
           params: {},
           name: authored.name,
           columns: authored.executedQueries?.[0]?.data?.[0] ? Object.keys(authored.executedQueries[0].data[0]) : void 0
@@ -18528,7 +18644,7 @@ var CONTEXT_CONFIG = {
    * Set to 0 to disable conversation history
    * Higher values provide more context but may increase token usage
    */
-  MAX_CONVERSATION_CONTEXT_BLOCKS: 3
+  MAX_CONVERSATION_CONTEXT_BLOCKS: 2
 };
 
 // src/handlers/user-prompt-request.ts
@@ -18626,7 +18742,12 @@ var get_user_request = async (data, components, sendMessage, anthropicApiKey, gr
   }
   const requestStartTime = Date.now();
   logger.info(`Starting user prompt request with ${components.length} components`);
-  const conversationHistory = thread.getConversationContext(CONTEXT_CONFIG.MAX_CONVERSATION_CONTEXT_BLOCKS, existingUiBlockId);
+  const conversationScriptStore = new ScriptStore({ collections, projectId: process.env.SUPERATOM_PROJECT_ID || "" });
+  const conversationHistory = await thread.getConversationContext(
+    CONTEXT_CONFIG.MAX_CONVERSATION_CONTEXT_BLOCKS,
+    existingUiBlockId,
+    async (recipeId, version) => (await conversationScriptStore.getVersion(recipeId, version))?.scriptBody ?? null
+  );
   const MAX_EDIT_CANDIDATES = 5;
   let activeScriptBindings = thread.getScriptBindings(MAX_EDIT_CANDIDATES, existingUiBlockId);
   if (activeScriptBindings.length === 0 && collections?.["conversation-history"]?.["threadScriptBindings"]) {
@@ -19274,7 +19395,7 @@ async function handleActionsRequest(data, sendMessage, anthropicApiKey, groqApiK
     logger.debug(`[ACTIONS_REQ ${id}] Component: ${component?.name || "unknown"} (${component?.type || "unknown"})`);
     logger.debug(`[ACTIONS_REQ ${id}] Component data available: ${componentData ? "yes" : "no"}`);
     logger.debug(`[ACTIONS_REQ ${id}] Extracting conversation history (max ${CONTEXT_CONFIG.MAX_CONVERSATION_CONTEXT_BLOCKS} blocks)`);
-    const conversationHistory = thread.getConversationContext(CONTEXT_CONFIG.MAX_CONVERSATION_CONTEXT_BLOCKS, uiBlockId);
+    const conversationHistory = await thread.getConversationContext(CONTEXT_CONFIG.MAX_CONVERSATION_CONTEXT_BLOCKS, uiBlockId);
     const historyLineCount = conversationHistory.split("\n").filter((l) => l.trim()).length;
     logger.info(`[ACTIONS_REQ ${id}] Conversation history extracted: ${historyLineCount} lines`);
     logger.debug(`[ACTIONS_REQ ${id}] Conversation history preview:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@superatomai/sdk-node",
-  "version": "0.0.50-dsp",
+  "version": "0.0.51-dsp",
   "description": "Node.js TypeScript SDK for Superatom",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",