deepline 0.1.145 → 0.1.147

package/dist/bundling-sources/shared_libs/play-runtime/tool-result.ts

@@ -38,6 +38,8 @@ import type {
   ToolResultTargetAccessor,
   ToolResultTargetMetadata,
 } from './tool-result-types';
+import { createPlayDataset, type PlayDataset } from '../plays/dataset';
+import { normalizeTableNamespace, sha256Hex } from '../plays/row-identity';
 
 type PathSegment = string | number | '*';
 
@@ -737,12 +739,22 @@ function resolveListRows(
     );
     let resolvedPath: string | null = null;
     let rows: Record<string, unknown>[] | null = null;
+    let emptyMatch: { path: string; rows: Record<string, unknown>[] } | null =
+      null;
     for (const candidate of candidates) {
       rows = normalizeRows(getAtPath(result, candidate));
-      if (rows) {
+      if (!rows) {
+        continue;
+      }
+      if (rows.length > 0) {
         resolvedPath = candidate;
         break;
       }
+      emptyMatch ??= { path: candidate, rows };
+    }
+    if (!rows && emptyMatch) {
+      resolvedPath = emptyMatch.path;
+      rows = emptyMatch.rows;
     }
     if (!rows) continue;
     const storedPath = resolvedPath ?? path;
@@ -751,7 +763,12 @@ function resolveListRows(
       .filter(Boolean)
       .at(-1)
       ?.replace(/\[\d+\]$/, '');
-    lists[name || storedPath] = { path: storedPath, rows };
+    const listName = name || storedPath;
+    const existing = lists[listName];
+    if (existing?.rows.length && rows.length === 0) {
+      continue;
+    }
+    lists[listName] = { path: storedPath, rows };
   }
   return lists;
 }
@@ -935,10 +952,26 @@ function buildExtractedAccessors(
 function buildListAccessors(
   resolved: Record<string, { path: string; rows: Record<string, unknown>[] }>,
   lists: Record<string, ToolResultListMetadata>,
+  toolId: string,
+  executionDiscriminator: string,
 ): Record<string, ToolResultListAccessor> {
   return Object.fromEntries(
     Object.entries(lists).map(([name, metadata]) => {
       const rows = resolved[name]?.rows ?? [];
+      const datasetDiscriminator = `${executionDiscriminator}:${listRowsFingerprint(rows)}`;
+      const dataset = createPlayDataset(rows, {
+        kind: 'csv',
+        sourceLabel: metadata.path,
+        tableNamespace: listTableNamespace(
+          toolId,
+          name,
+          metadata.path,
+          datasetDiscriminator,
+        ),
+        datasetId: `tool-list:${sha256Hex(
+          `${toolId}:${metadata.path}:${datasetDiscriminator}`,
+        )}`,
+      });
       const accessor = {
         path: metadata.path,
         count: metadata.count,
@@ -946,7 +979,7 @@ function buildListAccessors(
       } as ToolResultListAccessor;
       Object.defineProperty(accessor, 'get', {
         value() {
-          return rows;
+          return dataset;
         },
         enumerable: false,
       });
@@ -955,6 +988,44 @@ function buildListAccessors(
   );
 }
 
+function listRowsFingerprint(rows: readonly Record<string, unknown>[]): string {
+  try {
+    return sha256Hex(
+      JSON.stringify(
+        {
+          count: rows.length,
+          rows,
+        },
+        (_key, value) => (typeof value === 'bigint' ? value.toString() : value),
+      ),
+    ).slice(0, 12);
+  } catch {
+    return sha256Hex(String(rows.length)).slice(0, 12);
+  }
+}
+
+function listTableNamespace(
+  toolId: string,
+  name: string,
+  path: string,
+  discriminator: string,
+): string {
+  const raw = `${toolId}_${name || path || 'rows'}_${sha256Hex(discriminator).slice(0, 10)}`;
+  try {
+    return normalizeTableNamespace(raw);
+  } catch {
+    const hash = sha256Hex(raw).slice(0, 10);
+    const leaf = name || path.split('.').filter(Boolean).at(-1) || 'rows';
+    let prefix = 'rows';
+    try {
+      prefix = normalizeTableNamespace(leaf).slice(0, 52) || 'rows';
+    } catch {
+      prefix = 'rows';
+    }
+    return normalizeTableNamespace(`${prefix}_${hash}`);
+  }
+}
+
 export function createToolExecuteResult<TResult = unknown>(input: {
   status: string;
   jobId?: string;
@@ -994,7 +1065,12 @@ export function createToolExecuteResult<TResult = unknown>(input: {
     ...(result.meta ? { meta: result.meta } : {}),
   };
   const extractedValues = buildExtractedAccessors(targets);
-  const extractedLists = buildListAccessors(resolvedLists, lists);
+  const extractedLists = buildListAccessors(
+    resolvedLists,
+    lists,
+    input.metadata.toolId,
+    input.jobId ?? input.execution.cacheKey ?? 'inline',
+  );
   const wrapper = {
     status: input.status,
     ...(input.jobId ? { job_id: input.jobId } : {}),
@@ -1063,7 +1139,7 @@ export function readValue(
 export function readList(
   result: ToolExecuteResult,
   selector?: readonly string[] | string,
-): Record<string, unknown>[] {
+): PlayDataset<Record<string, unknown>> | Record<string, unknown>[] {
   if (selector) {
     const paths = Array.isArray(selector) ? selector : [selector];
     const found = findFirstTargetByPath(resultRootOf(result), paths)?.value;