deepline 0.1.25 → 0.1.26

package/dist/repo/apps/play-runner-workers/src/coordinator-entry.ts

@@ -28,7 +28,7 @@ import type {
   PlayRuntimeManifest,
   PlayRuntimeManifestMap,
 } from '../../../shared_libs/plays/compiler-manifest';
-import { runtimeRunActions } from '../../../shared_libs/play-runtime/runtime-actions';
+import type { PlayRunLedgerEvent } from '../../../shared_libs/play-runtime/run-ledger';
 import {
   COORDINATOR_INTERNAL_TOKEN_HEADER,
   COORDINATOR_RUN_SCOPE_HEADER,
@@ -45,11 +45,22 @@ export type PlayWorkflowParams = {
   artifactHash: string;
   graphHash: string;
   input: Record<string, unknown>;
-  inputFile?: { name?: string; r2Key?: string; path?: string } | null;
+  inputFile?: {
+    name?: string;
+    r2Key?: string;
+    storageKey?: string;
+    path?: string;
+    fileName?: string;
+    logicalPath?: string;
+    contentType?: string;
+    bytes?: number;
+  } | null;
   inlineCsv?: { name: string; rows: Record<string, unknown>[] } | null;
   packagedFiles?: Array<{
     playPath: string;
     storageKey: string;
+    contentType?: string;
+    bytes?: number;
     inlineText?: string;
   }> | null;
   contractSnapshot?: unknown;
@@ -508,6 +519,8 @@ type DynamicWorkflowMetadata = {
   packagedFiles?: Array<{
     playPath: string;
     storageKey: string;
+    contentType?: string;
+    bytes?: number;
     inlineText?: string;
   }> | null;
 };
@@ -1311,16 +1324,21 @@ async function markWorkflowRuntimeFailure(input: {
   });
   const bypass = input.env.VERCEL_PROTECTION_BYPASS_TOKEN?.trim();
   if (bypass) headers.set('x-vercel-protection-bypass', bypass);
-  const body = JSON.stringify(
-    runtimeRunActions.updateStatus({
-      playId: runId,
-      status: 'failed',
-      error: `DynamicWorkflow runner failed: ${errorName}: ${errorMessage}${
-        errorStack ? `\n${errorStack}` : ''
-      }`,
-      runtimeBackend: 'cf_workflows_dynamic_worker',
-    }),
-  );
+  const body = JSON.stringify({
+    action: 'append_run_events',
+    playId: runId,
+    events: [
+      {
+        type: 'run.failed',
+        runId,
+        source: 'coordinator',
+        occurredAt: Date.now(),
+        error: `DynamicWorkflow runner failed: ${errorName}: ${errorMessage}${
+          errorStack ? `\n${errorStack}` : ''
+        }`,
+      } satisfies PlayRunLedgerEvent,
+    ],
+  });
   const url = `${baseUrl.replace(/\/$/, '')}/api/v2/plays/internal/runtime`;
   const backoffMs = [200, 500, 1500];
   let lastError: unknown = null;
@@ -1621,6 +1639,15 @@ function buildChildWorkflowParams(input: {
 function runRequestFromPlayWorkflowParams(
   params: PlayWorkflowParams,
 ): Record<string, unknown> {
+  const inputFileName = String(
+    params.inputFile?.name ??
+      params.inputFile?.fileName ??
+      params.inputFile?.logicalPath ??
+      params.inputFile?.path ??
+      '',
+  );
+  const inputStorageKey =
+    params.inputFile?.r2Key ?? params.inputFile?.storageKey ?? null;
   return {
     runId: params.runId,
     callbackUrl: params.baseUrl,
@@ -1632,12 +1659,22 @@ function runRequestFromPlayWorkflowParams(
     userEmail: params.userEmail,
     runtimeInput: params.input,
     inlineCsv: params.inlineCsv ?? null,
-    inputR2Keys:
-      params.inputFile?.r2Key &&
-      (params.inputFile.name || params.inputFile.path)
+    inputFiles:
+      inputStorageKey && inputFileName
         ? {
-            [String(params.inputFile.name ?? params.inputFile.path)]:
-              params.inputFile.r2Key,
+            [inputFileName]: {
+              logicalPath:
+                params.inputFile?.logicalPath ??
+                params.inputFile?.path ??
+                inputFileName,
+              fileName: params.inputFile?.fileName ?? inputFileName,
+              storageKey: inputStorageKey,
+              contentType: params.inputFile?.contentType ?? null,
+              bytes:
+                typeof params.inputFile?.bytes === 'number'
+                  ? params.inputFile.bytes
+                  : null,
+            },
           }
         : null,
     packagedFiles: params.packagedFiles ?? null,
@@ -1996,13 +2033,6 @@ export class RuntimeApi extends WorkerEntrypoint<CoordinatorEnv, undefined> {
         ? this.env.DEEPLINE_API_BASE_URL.trim()
         : 'https://code.deepline.com';
     const target = new URL(incoming.pathname + incoming.search, apiBaseUrl);
-    const runtimeStatusBody =
-      incoming.pathname === '/api/v2/plays/internal/runtime'
-        ? await request
-            .clone()
-            .json()
-            .catch(() => null)
-        : null;
     const forwarded = new Request(target.toString(), request);
     const bypassToken = this.env.VERCEL_PROTECTION_BYPASS_TOKEN;
     if (typeof bypassToken === 'string' && bypassToken) {
@@ -2018,41 +2048,9 @@ export class RuntimeApi extends WorkerEntrypoint<CoordinatorEnv, undefined> {
         `[RUNTIME_API] ${incoming.pathname} failed: status=${res.status} ` +
           `target=${target.toString()} body=${body.slice(0, 500)}`,
       );
-    } else {
-      await this.recordRuntimeStatusEvent(runtimeStatusBody).catch(() => null);
     }
     return res;
   }
-
-  private async recordRuntimeStatusEvent(body: unknown): Promise<void> {
-    if (!isRecord(body) || body.action !== 'update_run_status') {
-      return;
-    }
-    const runId = typeof body.playId === 'string' ? body.playId : '';
-    const status = typeof body.status === 'string' ? body.status : '';
-    if (!runId || !status) {
-      return;
-    }
-    await appendCoordinatorRunEvent(this.env, {
-      runId,
-      type: 'progress',
-      status,
-      ts: Date.now(),
-      logs: sanitizeLiveLogLines(body.liveLogs) ?? undefined,
-      activeNodeId:
-        typeof body.activeNodeId === 'string' ? body.activeNodeId : null,
-      activeArtifactTableNamespace:
-        typeof body.activeArtifactTableNamespace === 'string'
-          ? body.activeArtifactTableNamespace
-          : null,
-      updatedAt:
-        typeof body.lastCheckpointAt === 'number'
-          ? body.lastCheckpointAt
-          : null,
-      liveNodeProgress:
-        body.liveNodeProgress !== undefined ? body.liveNodeProgress : undefined,
-    });
-  }
 }
 
 export class CoordinatorControl extends WorkerEntrypoint<
@@ -3252,7 +3250,7 @@ function stableHash(value: string): string {
   return (hash >>> 0).toString(36);
 }
 
-const DYNAMIC_PLAY_WORKER_HARNESS_VERSION = 'h7-skip-high-volume-tool-traces';
+const DYNAMIC_PLAY_WORKER_HARNESS_VERSION = 'h11-full-row-dataset-handles';
 const DYNAMIC_WORKER_BUNDLED_CODE_CACHE_MAX_ENTRIES = 64;
 const dynamicWorkerBundledCodeCache = new Map<string, string>();
 
@@ -3548,6 +3546,14 @@ function normalizePackagedFiles(
     .map((entry) => ({
       playPath: String(entry.playPath ?? '').replace(/^\.\//, ''),
       storageKey: String(entry.storageKey ?? ''),
+      contentType:
+        typeof entry.contentType === 'string' ? entry.contentType : undefined,
+      bytes:
+        typeof entry.bytes === 'number' &&
+        Number.isSafeInteger(entry.bytes) &&
+        entry.bytes >= 0
+          ? entry.bytes
+          : undefined,
       inlineText:
         typeof entry.inlineText === 'string' ? entry.inlineText : undefined,
     }))
@@ -4062,8 +4068,13 @@ function mapWorkflowResult(
   runId: string,
   status: InstanceStatus,
 ): Record<string, unknown> {
-  const error = readWorkflowError(status);
-  const mapped = resolveTerminalStatus(status, error);
+  const rawError = readWorkflowError(status);
+  const mapped = resolveTerminalStatus(status, rawError);
+  const error =
+    rawError ??
+    (mapped === 'failed'
+      ? `Cloudflare workflow reported ${String(status.status ?? 'failed')} without an error payload.`
+      : null);
   const output =
     status.output && typeof status.output === 'object'
       ? (status.output as Record<string, unknown>)
@@ -4131,9 +4142,11 @@ function resolveTerminalStatus(
 function readWorkflowError(status: InstanceStatus): string | null {
   const error = status.error as unknown;
   if (!error) return null;
-  if (typeof error === 'string') return error;
+  if (typeof error === 'string') return error.trim() || null;
   if (typeof error === 'object' && 'message' in error) {
-    return String((error as { message?: unknown }).message ?? '');
+    const message = String((error as { message?: unknown }).message ?? '');
+    return message.trim() || null;
   }
-  return String(error);
+  const message = String(error);
+  return message.trim() || null;
 }

package/dist/repo/apps/play-runner-workers/src/dedup-do.ts

@@ -1061,7 +1061,12 @@ export class PlayDedup implements DurableObject {
       runId: body.runId,
       status: body.status,
       result: body.result,
-      error: typeof body.error === 'string' ? body.error : null,
+      error:
+        typeof body.error === 'string' && body.error.trim()
+          ? body.error.trim()
+          : body.status === 'failed'
+            ? 'Cloudflare workflow failed before the runner reported an error.'
+            : null,
       totalRows: body.totalRows,
       durationMs: body.durationMs,
       playName: typeof body.playName === 'string' ? body.playName : null,