@output.ai/cli 0.3.0-dev.pr156.05c9aa2 → 0.3.0-dev.pr156.696d4dd

package/dist/api/generated/api.d.ts

@@ -42,11 +42,27 @@ export type PostWorkflowRunBody = {
     /** The name of the task queue to send the workflow to */
     taskQueue?: string;
 };
+/**
+ * File destinations for trace data
+ */
+export type PostWorkflowRun200TraceDestinations = {
+    /**
+     * Absolute path to local trace file, or null if not saved locally
+     * @nullable
+     */
+    local: string | null;
+    /**
+     * Remote trace location (e.g., S3 URI), or null if not saved remotely
+     * @nullable
+     */
+    remote: string | null;
+};
 /**
  * An object with information about the trace generated by the execution
  */
 export type PostWorkflowRun200Trace = {
-    [key: string]: unknown;
+    /** File destinations for trace data */
+    destinations?: PostWorkflowRun200TraceDestinations;
 };
 export type PostWorkflowRun200 = {
     /** The workflow execution id */
@@ -94,11 +110,27 @@ export type GetWorkflowIdStatus200 = {
     /** An epoch timestamp representing when the workflow ended */
     completedAt?: number;
 };
+/**
+ * File destinations for trace data
+ */
+export type GetWorkflowIdOutput200TraceDestinations = {
+    /**
+     * Absolute path to local trace file, or null if not saved locally
+     * @nullable
+     */
+    local: string | null;
+    /**
+     * Remote trace location (e.g., S3 URI), or null if not saved remotely
+     * @nullable
+     */
+    remote: string | null;
+};
 /**
  * An object with information about the trace generated by the execution
  */
 export type GetWorkflowIdOutput200Trace = {
-    [key: string]: unknown;
+    /** File destinations for trace data */
+    destinations?: GetWorkflowIdOutput200TraceDestinations;
 };
 export type GetWorkflowIdOutput200 = {
     /** The workflow execution id */

package/dist/commands/workflow/debug.js

@@ -1,7 +1,7 @@
 import { Args, Command, Flags } from '@oclif/core';
 import { OUTPUT_FORMAT } from '#utils/constants.js';
 import { traceFormatter } from '#utils/trace_formatter.js';
-import { TraceReader } from '#services/trace_reader.js';
+import { findTraceFile, readTraceFile } from '#services/trace_reader.js';
 export default class WorkflowDebug extends Command {
     static description = 'Get and display workflow execution trace for debugging';
     static examples = [
@@ -39,9 +39,8 @@ export default class WorkflowDebug extends Command {
         this.displayTextTrace(traceData);
     }
     async getTrace(workflowId) {
-        const reader = new TraceReader();
-        const tracePath = await reader.findTraceFile(workflowId);
-        return reader.readTraceFile(tracePath);
+        const tracePath = await findTraceFile(workflowId);
+        return readTraceFile(tracePath);
     }
     outputJson(data) {
         this.log(JSON.stringify(data, null, 2));

package/dist/services/trace_reader.d.ts

@@ -1,15 +1,8 @@
-export declare class TraceReader {
-    /**
-     * Find trace file from workflow metadata
-     */
-    findTraceFile(workflowId: string): Promise<string>;
-    /**
-     * Read and parse trace file
-     */
-    readTraceFile(path: string): Promise<any>;
-    /**
-     * Check if a file exists
-     */
-    private fileExists;
-}
-export declare const traceReader: TraceReader;
+/**
+ * Find trace file from workflow metadata
+ */
+export declare function findTraceFile(workflowId: string): Promise<string>;
+/**
+ * Read and parse trace file
+ */
+export declare function readTraceFile(path: string): Promise<any>;

package/dist/services/trace_reader.js

@@ -1,52 +1,51 @@
 import { readFile, stat } from 'node:fs/promises';
-import { getWorkflowIdStatus } from '#api/generated/api.js';
-export class TraceReader {
-    /**
-     * Find trace file from workflow metadata
-     */
-    async findTraceFile(workflowId) {
-        const status = await getWorkflowIdStatus(workflowId);
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        const tracePath = status.data?.trace?.destinations?.local;
-        if (!tracePath) {
-            throw new Error(`No trace file path found for workflow ${workflowId}`);
-        }
-        if (!await this.fileExists(tracePath)) {
-            throw new Error(`Trace file not found at path: ${tracePath}`);
-        }
-        return tracePath;
+import { getWorkflowIdOutput } from '#api/generated/api.js';
+/**
+ * Check if a file exists
+ */
+async function fileExists(path) {
+    try {
+        await stat(path);
+        return true;
     }
-    /**
-     * Read and parse trace file
-     */
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    async readTraceFile(path) {
-        try {
-            const content = await readFile(path, 'utf-8');
-            return JSON.parse(content);
-        }
-        catch (error) {
-            if (error && typeof error === 'object' && 'code' in error && error.code === 'ENOENT') {
-                throw new Error(`Trace file not found at path: ${path}`);
-            }
-            if (error instanceof SyntaxError) {
-                throw new Error(`Invalid JSON in trace file: ${path}`);
-            }
-            throw error;
-        }
+    catch {
+        return false;
+    }
+}
+/**
+ * Find trace file from workflow metadata
+ */
+export async function findTraceFile(workflowId) {
+    const response = await getWorkflowIdOutput(workflowId);
+    // Check if we got a successful response
+    if (response.status !== 200) {
+        throw new Error(`Failed to get workflow output for ${workflowId}`);
+    }
+    const tracePath = response.data.trace?.destinations?.local;
+    if (!tracePath) {
+        throw new Error(`No trace file path found for workflow ${workflowId}`);
+    }
+    if (!await fileExists(tracePath)) {
+        throw new Error(`Trace file not found at path: ${tracePath}`);
+    }
+    return tracePath;
+}
+/**
+ * Read and parse trace file
+ */
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
+export async function readTraceFile(path) {
+    try {
+        const content = await readFile(path, 'utf-8');
+        return JSON.parse(content);
     }
-    /**
-     * Check if a file exists
-     */
-    async fileExists(path) {
-        try {
-            await stat(path);
-            return true;
+    catch (error) {
+        if (error && typeof error === 'object' && 'code' in error && error.code === 'ENOENT') {
+            throw new Error(`Trace file not found at path: ${path}`);
         }
-        catch {
-            return false;
+        if (error instanceof SyntaxError) {
+            throw new Error(`Invalid JSON in trace file: ${path}`);
         }
+        throw error;
     }
 }
-// Export singleton instance for convenience
-export const traceReader = new TraceReader();

package/dist/services/trace_reader.test.js

@@ -1,5 +1,5 @@
 import { describe, it, expect, vi, afterEach } from 'vitest';
-import { TraceReader } from './trace_reader.js';
+import { findTraceFile, readTraceFile } from './trace_reader.js';
 // Mock file system operations
 vi.mock('node:fs/promises', () => ({
     readFile: vi.fn(),
@@ -7,7 +7,7 @@ vi.mock('node:fs/promises', () => ({
 }));
 // Mock API
 vi.mock('../api/generated/api.js', () => ({
-    getWorkflowIdStatus: vi.fn()
+    getWorkflowIdOutput: vi.fn()
 }));
 describe('TraceReader', () => {
     const getMocks = async () => {
@@ -19,75 +19,110 @@ describe('TraceReader', () => {
             // eslint-disable-next-line @typescript-eslint/no-explicit-any
             mockStat: fsModule.stat,
             // eslint-disable-next-line @typescript-eslint/no-explicit-any
-            mockGetWorkflowIdStatus: apiModule.getWorkflowIdStatus
+            mockGetWorkflowIdOutput: apiModule.getWorkflowIdOutput
         };
     };
     afterEach(() => {
         vi.clearAllMocks();
     });
     describe('findTraceFile', () => {
-        it('should find trace file from workflow metadata', async () => {
-            const traceReader = new TraceReader();
-            const { mockGetWorkflowIdStatus, mockStat } = await getMocks();
+        it('should find trace file from workflow output metadata', async () => {
+            const { mockGetWorkflowIdOutput, mockStat } = await getMocks();
             const workflowId = 'test-workflow-123';
             const expectedPath = '/app/logs/runs/test/2024-01-01_test-workflow-123.json';
-            mockGetWorkflowIdStatus.mockResolvedValue({
+            mockGetWorkflowIdOutput.mockResolvedValue({
+                status: 200,
                 data: {
+                    workflowId,
+                    output: { result: 'test result' },
                     trace: {
                         destinations: {
-                            local: expectedPath
+                            local: expectedPath,
+                            remote: null
                         }
                     }
                 }
             });
             mockStat.mockResolvedValue({ isFile: () => true });
-            const result = await traceReader.findTraceFile(workflowId);
+            const result = await findTraceFile(workflowId);
             expect(result).toBe(expectedPath);
-            expect(mockGetWorkflowIdStatus).toHaveBeenCalledWith(workflowId);
+            expect(mockGetWorkflowIdOutput).toHaveBeenCalledWith(workflowId);
             expect(mockStat).toHaveBeenCalledWith(expectedPath);
         });
         it('should throw error when no trace path in metadata', async () => {
-            const traceReader = new TraceReader();
-            const { mockGetWorkflowIdStatus } = await getMocks();
+            const { mockGetWorkflowIdOutput } = await getMocks();
             const workflowId = 'test-workflow-456';
-            mockGetWorkflowIdStatus.mockResolvedValue({
+            mockGetWorkflowIdOutput.mockResolvedValue({
+                status: 200,
                 data: {
-                // No trace field
+                    workflowId,
+                    output: { result: 'test result' },
+                    trace: {
+                        destinations: {
+                            local: null,
+                            remote: null
+                        }
+                    }
                 }
             });
-            await expect(traceReader.findTraceFile(workflowId))
+            await expect(findTraceFile(workflowId))
                 .rejects.toThrow(`No trace file path found for workflow ${workflowId}`);
         });
         it('should throw error when trace file not on disk', async () => {
-            const traceReader = new TraceReader();
-            const { mockGetWorkflowIdStatus, mockStat } = await getMocks();
+            const { mockGetWorkflowIdOutput, mockStat } = await getMocks();
             const workflowId = 'test-workflow-789';
             const expectedPath = '/app/logs/runs/test/2024-01-01_test-workflow-789.json';
-            mockGetWorkflowIdStatus.mockResolvedValue({
+            mockGetWorkflowIdOutput.mockResolvedValue({
+                status: 200,
                 data: {
+                    workflowId,
+                    output: { result: 'test result' },
                     trace: {
                         destinations: {
-                            local: expectedPath
+                            local: expectedPath,
+                            remote: null
                         }
                     }
                 }
             });
             mockStat.mockRejectedValue(new Error('ENOENT'));
-            await expect(traceReader.findTraceFile(workflowId))
+            await expect(findTraceFile(workflowId))
                 .rejects.toThrow(`Trace file not found at path: ${expectedPath}`);
         });
         it('should throw error when API call fails', async () => {
-            const traceReader = new TraceReader();
-            const { mockGetWorkflowIdStatus } = await getMocks();
+            const { mockGetWorkflowIdOutput } = await getMocks();
             const workflowId = 'non-existent';
-            mockGetWorkflowIdStatus.mockRejectedValue(new Error('Workflow not found'));
-            await expect(traceReader.findTraceFile(workflowId))
+            mockGetWorkflowIdOutput.mockRejectedValue(new Error('Workflow not found'));
+            await expect(findTraceFile(workflowId))
                 .rejects.toThrow('Workflow not found');
         });
+        it('should handle missing trace property gracefully', async () => {
+            const { mockGetWorkflowIdOutput } = await getMocks();
+            const workflowId = 'test-workflow-no-trace';
+            mockGetWorkflowIdOutput.mockResolvedValue({
+                status: 200,
+                data: {
+                    workflowId,
+                    output: { result: 'test result' }
+                    // No trace property at all
+                }
+            });
+            await expect(findTraceFile(workflowId))
+                .rejects.toThrow(`No trace file path found for workflow ${workflowId}`);
+        });
+        it('should throw error when workflow not found (404)', async () => {
+            const { mockGetWorkflowIdOutput } = await getMocks();
+            const workflowId = 'non-existent-workflow';
+            mockGetWorkflowIdOutput.mockResolvedValue({
+                status: 404,
+                data: void 0
+            });
+            await expect(findTraceFile(workflowId))
+                .rejects.toThrow(`Failed to get workflow output for ${workflowId}`);
+        });
     });
     describe('readTraceFile', () => {
         it('should read and parse JSON file successfully', async () => {
-            const traceReader = new TraceReader();
             const { mockReadFile } = await getMocks();
             const path = '/logs/test.json';
             const traceData = {
@@ -95,36 +130,33 @@ describe('TraceReader', () => {
                 events: []
             };
             mockReadFile.mockResolvedValue(JSON.stringify(traceData));
-            const result = await traceReader.readTraceFile(path);
+            const result = await readTraceFile(path);
             expect(result).toEqual(traceData);
             expect(mockReadFile).toHaveBeenCalledWith(path, 'utf-8');
         });
         it('should throw error for non-existent file', async () => {
-            const traceReader = new TraceReader();
             const { mockReadFile } = await getMocks();
             const path = '/logs/missing.json';
             const error = new Error('ENOENT');
             // eslint-disable-next-line @typescript-eslint/no-explicit-any
             error.code = 'ENOENT';
             mockReadFile.mockRejectedValue(error);
-            await expect(traceReader.readTraceFile(path))
+            await expect(readTraceFile(path))
                 .rejects.toThrow(`Trace file not found at path: ${path}`);
         });
         it('should throw error for invalid JSON', async () => {
-            const traceReader = new TraceReader();
             const { mockReadFile } = await getMocks();
             const path = '/logs/invalid.json';
             mockReadFile.mockResolvedValue('invalid json {');
-            await expect(traceReader.readTraceFile(path))
+            await expect(readTraceFile(path))
                 .rejects.toThrow(`Invalid JSON in trace file: ${path}`);
         });
         it('should rethrow other errors', async () => {
-            const traceReader = new TraceReader();
             const { mockReadFile } = await getMocks();
             const path = '/logs/test.json';
             const error = new Error('Permission denied');
             mockReadFile.mockRejectedValue(error);
-            await expect(traceReader.readTraceFile(path))
+            await expect(readTraceFile(path))
                 .rejects.toThrow('Permission denied');
         });
     });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@output.ai/cli",
-  "version": "0.3.0-dev.pr156.05c9aa2",
+  "version": "0.3.0-dev.pr156.696d4dd",
   "description": "CLI for Output.ai workflow generation",
   "type": "module",
   "main": "dist/index.js",