@outputai/cli 0.2.1-next.f1502fb.0 → 0.3.0

package/dist/utils/error_handler.js

@@ -1,11 +1,13 @@
 import { config } from '#config.js';
-const DEFAULT_MESSAGES = {
-    ECONNREFUSED: `Connection refused to ${config.apiUrl}. Is the API server running?`,
-    401: 'Authentication failed. Check your OUTPUT_API_AUTH_TOKEN.',
-    404: 'Resource not found.',
-    500: 'Server error.',
-    UNKNOWN: 'An unknown error occurred.'
-};
+function getDefaultMessages() {
+    return {
+        ECONNREFUSED: `Connection refused to ${config.apiUrl}. Is the API server running?`,
+        401: 'Authentication failed. Check your OUTPUT_API_AUTH_TOKEN.',
+        404: 'Resource not found.',
+        500: 'Server error.',
+        UNKNOWN: 'An unknown error occurred.'
+    };
+}
 /**
  * Extract error type and message from API response data
  */
@@ -49,7 +51,7 @@ function getDetailedErrorMessage(error) {
 }
 export function handleApiError(error, errorFn, overrides = {}) {
     const apiError = error;
-    const errorMessages = { ...DEFAULT_MESSAGES, ...overrides };
+    const errorMessages = { ...getDefaultMessages(), ...overrides };
     if (apiError.code === 'ECONNREFUSED' || apiError.cause?.code === 'ECONNREFUSED') {
         errorFn(errorMessages.ECONNREFUSED, { exit: 1 });
     }

package/dist/utils/validation.d.ts

@@ -11,3 +11,16 @@ export declare function validateWorkflowName(name: string): void;
  * Validate that a directory path is safe to create
  */
 export declare function validateOutputDirectory(outputDir: string): void;
+export declare class InvalidPortError extends Error {
+    constructor(envVarName: string, raw: string, reason: string);
+}
+/**
+ * Parse a port number from an env var. Empty string and undefined fall back to
+ * the default silently (matching Compose's `${VAR:-default}` semantics).
+ * Throws InvalidPortError on anything else: non-numeric, signed, decimal,
+ * trailing junk (e.g. "3001abc"), or out of range 1-65535. Throwing (vs
+ * warn-and-fallback) prevents CLI/Docker disagreement: Compose reads the same
+ * env var via `${VAR:-default}` and uses its own parser, so a CLI fallback
+ * would silently desync from the bound port.
+ */
+export declare function parsePort(raw: string | undefined, defaultPort: number, envVarName: string): number;

package/dist/utils/validation.js

@@ -23,3 +23,34 @@ export function validateOutputDirectory(outputDir) {
         throw new InvalidOutputDirectoryError(outputDir, 'Output directory cannot be empty');
     }
 }
+const MIN_PORT = 1;
+const MAX_PORT = 65535;
+export class InvalidPortError extends Error {
+    constructor(envVarName, raw, reason) {
+        super(`${envVarName}=${raw} is invalid (${reason}). ` +
+            `Set a port in ${MIN_PORT}-${MAX_PORT} in your .env file, or unset the variable to use the default.`);
+        this.name = 'InvalidPortError';
+    }
+}
+/**
+ * Parse a port number from an env var. Empty string and undefined fall back to
+ * the default silently (matching Compose's `${VAR:-default}` semantics).
+ * Throws InvalidPortError on anything else: non-numeric, signed, decimal,
+ * trailing junk (e.g. "3001abc"), or out of range 1-65535. Throwing (vs
+ * warn-and-fallback) prevents CLI/Docker disagreement: Compose reads the same
+ * env var via `${VAR:-default}` and uses its own parser, so a CLI fallback
+ * would silently desync from the bound port.
+ */
+export function parsePort(raw, defaultPort, envVarName) {
+    if (raw === undefined || raw === '') {
+        return defaultPort;
+    }
+    if (!/^\d+$/.test(raw)) {
+        throw new InvalidPortError(envVarName, raw, 'not a positive integer');
+    }
+    const n = parseInt(raw, 10);
+    if (n < MIN_PORT || n > MAX_PORT) {
+        throw new InvalidPortError(envVarName, raw, `out of range ${MIN_PORT}-${MAX_PORT}`);
+    }
+    return n;
+}

package/dist/utils/validation.spec.js

@@ -1,5 +1,5 @@
 import { describe, expect, it } from 'vitest';
-import { isValidWorkflowName } from './validation.js';
+import { isValidWorkflowName, parsePort, InvalidPortError } from './validation.js';
 describe('isValidWorkflowName', () => {
     describe('valid workflow names', () => {
         it('should accept single letter', () => {
@@ -138,3 +138,49 @@ describe('isValidWorkflowName', () => {
         });
     });
 });
+describe('parsePort', () => {
+    it('returns the parsed value for a valid port', () => {
+        expect(parsePort('3001', 3001, 'P')).toBe(3001);
+        expect(parsePort('7234', 7233, 'P')).toBe(7234);
+    });
+    it('falls back to default for undefined silently', () => {
+        expect(parsePort(undefined, 3001, 'P')).toBe(3001);
+    });
+    it('falls back to default for empty string silently (matches Compose ${VAR:-default})', () => {
+        expect(parsePort('', 3001, 'P')).toBe(3001);
+    });
+    it('throws InvalidPortError for non-numeric input', () => {
+        expect(() => parsePort('abc', 3001, 'OUTPUT_API_HOST_PORT')).toThrow(InvalidPortError);
+        expect(() => parsePort('abc', 3001, 'OUTPUT_API_HOST_PORT'))
+            .toThrow(/OUTPUT_API_HOST_PORT=abc/);
+    });
+    it('throws for trailing junk (parseInt would silently truncate)', () => {
+        expect(() => parsePort('3001abc', 3001, 'P')).toThrow(InvalidPortError);
+    });
+    it('throws for negative input', () => {
+        expect(() => parsePort('-1', 3001, 'P')).toThrow(InvalidPortError);
+    });
+    it('throws for port 0 (CLI rejects but Compose would treat as ephemeral - prevents desync)', () => {
+        expect(() => parsePort('0', 3001, 'P')).toThrow(InvalidPortError);
+    });
+    it('throws for port above 65535', () => {
+        expect(() => parsePort('65536', 3001, 'P')).toThrow(InvalidPortError);
+        expect(() => parsePort('99999', 3001, 'P')).toThrow(InvalidPortError);
+    });
+    it('accepts boundary ports 1 and 65535', () => {
+        expect(parsePort('1', 3001, 'P')).toBe(1);
+        expect(parsePort('65535', 3001, 'P')).toBe(65535);
+    });
+    it('error message includes env var name and remediation hint', () => {
+        try {
+            parsePort('abc', 3001, 'OUTPUT_API_HOST_PORT');
+            expect.fail('expected throw');
+        }
+        catch (err) {
+            expect(err).toBeInstanceOf(InvalidPortError);
+            expect(err.message).toContain('OUTPUT_API_HOST_PORT=abc');
+            expect(err.message).toContain('1-65535');
+            expect(err.message).toContain('.env file');
+        }
+    });
+});

package/dist/views/dev.js

@@ -138,10 +138,10 @@ const DevSuccessMessage = ({ services }) => {
     const divider = '─'.repeat(80);
     const sortedNames = services.map(s => s.name).sort().join('|');
     const logsCommand = `docker compose -p ${config.dockerServiceName} logs -f <${sortedNames}>`;
-    return (_jsxs(Box, { flexDirection: "column", marginBottom: 1, children: [_jsx(Box, { marginTop: 1, marginBottom: 1, children: _jsx(Text, { dimColor: true, children: divider }) }), _jsxs(Box, { children: [_jsx(Text, { color: "green", bold: true, children: '✅ SUCCESS! ' }), _jsx(Text, { bold: true, children: "Development services are running" })] }), _jsx(Box, { marginTop: 1, marginBottom: 1, children: _jsx(Text, { dimColor: true, children: divider }) }), _jsx(Box, { marginBottom: 1, children: _jsx(Text, { bold: true, children: "\uD83D\uDC33 SERVICES" }) }), _jsxs(Box, { flexDirection: "column", marginLeft: 2, children: [_jsxs(Box, { children: [_jsx(Text, { color: "white", children: 'Temporal:    ' }), _jsx(Text, { color: "yellow", children: "localhost:7233" })] }), _jsxs(Box, { children: [_jsx(Text, { color: "white", children: 'Temporal UI: ' }), _jsx(Text, { color: "cyan", children: "http://localhost:8080" })] }), _jsxs(Box, { children: [_jsx(Text, { color: "white", children: 'API Server:  ' }), _jsx(Text, { color: "yellow", children: "localhost:3001" })] }), _jsxs(Box, { children: [_jsx(Text, { color: "white", children: 'Redis:       ' }), _jsx(Text, { color: "yellow", children: "localhost:6379" })] })] }), _jsx(Box, { marginTop: 1, marginBottom: 1, children: _jsx(Text, { dimColor: true, children: divider }) }), _jsx(Box, { marginBottom: 1, children: _jsx(Text, { bold: true, children: "\uD83D\uDE80 RUN A WORKFLOW" }) }), _jsxs(Box, { flexDirection: "column", marginLeft: 2, children: [_jsx(Text, { color: "white", children: "In a new terminal, execute:" }), _jsx(Box, { marginLeft: 2, children: _jsx(Text, { color: "cyan", children: "npx output workflow run blog_evaluator paulgraham_hwh" }) })] }), _jsx(Box, { marginTop: 1, marginBottom: 1, children: _jsx(Text, { dimColor: true, children: divider }) }), _jsx(Box, { marginBottom: 1, children: _jsx(Text, { bold: true, children: "\u26A1 USEFUL COMMANDS" }) }), _jsxs(Box, { flexDirection: "column", marginLeft: 2, children: [_jsxs(Box, { children: [_jsx(Text, { color: "white", children: 'Open Temporal UI: ' }), _jsx(Text, { color: "cyan", children: "open http://localhost:8080" })] }), _jsxs(Box, { children: [_jsx(Text, { color: "white", children: 'View logs:        ' }), _jsx(Text, { color: "cyan", children: logsCommand })] }), _jsxs(Box, { children: [_jsx(Text, { color: "white", children: 'Stop services:    ' }), _jsx(Text, { color: "cyan", children: "Press Ctrl+C" })] })] }), _jsx(Box, { marginTop: 1, marginBottom: 1, children: _jsx(Text, { dimColor: true, children: divider }) }), _jsx(Text, { dimColor: true, children: "\uD83D\uDCA1 Tip: The Temporal UI lets you monitor workflow executions in real-time" })] }));
+    return (_jsxs(Box, { flexDirection: "column", marginBottom: 1, children: [_jsx(Box, { marginTop: 1, marginBottom: 1, children: _jsx(Text, { dimColor: true, children: divider }) }), _jsxs(Box, { children: [_jsx(Text, { color: "green", bold: true, children: '✅ SUCCESS! ' }), _jsx(Text, { bold: true, children: "Development services are running" })] }), _jsx(Box, { marginTop: 1, marginBottom: 1, children: _jsx(Text, { dimColor: true, children: divider }) }), _jsx(Box, { marginBottom: 1, children: _jsx(Text, { bold: true, children: "\uD83D\uDC33 SERVICES" }) }), _jsxs(Box, { flexDirection: "column", marginLeft: 2, children: [_jsxs(Box, { children: [_jsx(Text, { color: "white", children: 'Temporal UI: ' }), _jsx(Text, { color: "cyan", children: config.temporalUiUrl })] }), _jsxs(Box, { children: [_jsx(Text, { color: "white", children: 'API Server:  ' }), _jsxs(Text, { color: "yellow", children: ["localhost:", config.ports.api] })] })] }), _jsx(Box, { marginTop: 1, marginBottom: 1, children: _jsx(Text, { dimColor: true, children: divider }) }), _jsx(Box, { marginBottom: 1, children: _jsx(Text, { bold: true, children: "\uD83D\uDE80 RUN A WORKFLOW" }) }), _jsxs(Box, { flexDirection: "column", marginLeft: 2, children: [_jsx(Text, { color: "white", children: "In a new terminal, execute:" }), _jsx(Box, { marginLeft: 2, children: _jsx(Text, { color: "cyan", children: "npx output workflow run blog_evaluator paulgraham_hwh" }) })] }), _jsx(Box, { marginTop: 1, marginBottom: 1, children: _jsx(Text, { dimColor: true, children: divider }) }), _jsx(Box, { marginBottom: 1, children: _jsx(Text, { bold: true, children: "\u26A1 USEFUL COMMANDS" }) }), _jsxs(Box, { flexDirection: "column", marginLeft: 2, children: [_jsxs(Box, { children: [_jsx(Text, { color: "white", children: 'Open Temporal UI: ' }), _jsxs(Text, { color: "cyan", children: ["open ", config.temporalUiUrl] })] }), _jsxs(Box, { children: [_jsx(Text, { color: "white", children: 'View logs:        ' }), _jsx(Text, { color: "cyan", children: logsCommand })] }), _jsxs(Box, { children: [_jsx(Text, { color: "white", children: 'Stop services:    ' }), _jsx(Text, { color: "cyan", children: "Press Ctrl+C" })] })] }), _jsx(Box, { marginTop: 1, marginBottom: 1, children: _jsx(Text, { dimColor: true, children: divider }) }), _jsx(Text, { dimColor: true, children: "\uD83D\uDCA1 Tip: The Temporal UI lets you monitor workflow executions in real-time" })] }));
 };
 const WaitingView = ({ services }) => (_jsxs(Box, { flexDirection: "column", children: [_jsxs(Box, { children: [_jsx(Text, { color: "yellow", children: _jsx(Spinner, { type: "dots" }) }), _jsx(Text, { children: " Waiting for services to become healthy..." })] }), services.length > 0 && (_jsx(Box, { flexDirection: "column", marginTop: 1, children: services.map(s => _jsx(ServiceRow, { service: s }, s.name)) }))] }));
-const RunningView = ({ services, workflowSummary }) => (_jsxs(Box, { flexDirection: "column", children: [_jsx(Text, { bold: true, children: "\uD83D\uDCCA Service Status" }), _jsx(Box, { flexDirection: "column", marginTop: 1, children: services.map(s => _jsx(ServiceRow, { service: s }, s.name)) }), _jsx(FailureWarning, { services: services }), workflowSummary && _jsx(WorkflowSummarySection, { summary: workflowSummary }), _jsxs(Box, { marginTop: 1, children: [_jsx(Text, { color: "cyan", children: '🌐 Temporal UI: ' }), _jsx(Text, { bold: true, children: "http://localhost:8080" })] }), _jsx(CommandFooter, { hints: [
+const RunningView = ({ services, workflowSummary }) => (_jsxs(Box, { flexDirection: "column", children: [_jsx(Text, { bold: true, children: "\uD83D\uDCCA Service Status" }), _jsx(Box, { flexDirection: "column", marginTop: 1, children: services.map(s => _jsx(ServiceRow, { service: s }, s.name)) }), _jsx(FailureWarning, { services: services }), workflowSummary && _jsx(WorkflowSummarySection, { summary: workflowSummary }), _jsxs(Box, { marginTop: 1, children: [_jsx(Text, { color: "cyan", children: '🌐 Temporal UI: ' }), _jsx(Text, { bold: true, children: config.temporalUiUrl })] }), _jsx(CommandFooter, { hints: [
                 { key: 'o', label: 'open ui' },
                 { key: 'w', label: 'view workflow runs' },
                 { key: 'ctrl+c', label: 'stop' }
@@ -173,7 +173,7 @@ export const DevApp = ({ dockerComposePath, onCleanup }) => {
     useStatusRefresh(dockerComposePath, phase === 'running', setServices);
     useWorkflowPolling(phase === 'running' || phase === 'failed', setWorkflowRuns);
     useMainViewInput(activeView === 'main' && phase !== 'waiting', {
-        onOpenTemporal: () => openUrl('http://localhost:8080'),
+        onOpenTemporal: () => openUrl(config.temporalUiUrl),
         onOpenWorkflows: () => setActiveView('workflows')
     });
     useCtrlC(onCleanup);

package/dist/views/workflow/list.js

@@ -2,12 +2,12 @@ import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { useState, useEffect, useRef, useMemo } from 'react';
 import { Box, Text, useInput } from 'ink';
 import Spinner from 'ink-spinner';
-import { getWorkflowIdResult } from '#api/generated/api.js';
+import { getWorkflowIdRunsRidResult } from '#api/generated/api.js';
 import { StatusIcon, statusColor } from '#components/status_icon.js';
 import { elapsedMs, formatDurationCompact } from '#utils/date_formatter.js';
 import { CommandFooter } from '#components/command_footer.js';
 import { openUrl } from '#utils/open_url.js';
-const TEMPORAL_UI_BASE = 'http://localhost:8080';
+import { config } from '#config.js';
 const VISIBLE_ROWS = 15;
 const STATUS_ORDER = {
     running: 0,
@@ -59,17 +59,19 @@ export const WorkflowListView = ({ runs, onBack }) => {
     const clampedIndex = Math.min(selectedIndex, Math.max(0, sortedRuns.length - 1));
     const selectedRun = sortedRuns[clampedIndex];
     const selectedWorkflowId = selectedRun?.workflowId;
+    const selectedRunId = selectedRun?.runId;
     useEffect(() => {
         if (clampedIndex !== selectedIndex) {
             setSelectedIndex(clampedIndex);
         }
     }, [clampedIndex, selectedIndex]);
     useEffect(() => {
-        if (!selectedWorkflowId) {
+        if (!selectedWorkflowId || !selectedRunId) {
             setDetail(null);
             return;
         }
-        const cached = cacheRef.current.get(selectedWorkflowId);
+        const cacheKey = `${selectedWorkflowId}:${selectedRunId}`;
+        const cached = cacheRef.current.get(cacheKey);
         if (cached) {
             setDetail(cached);
             setDetailLoading(false);
@@ -77,13 +79,13 @@ export const WorkflowListView = ({ runs, onBack }) => {
         }
         const currentFetchId = ++fetchIdRef.current;
         setDetailLoading(true);
-        getWorkflowIdResult(selectedWorkflowId)
+        getWorkflowIdRunsRidResult(selectedWorkflowId, selectedRunId)
             .then(response => {
             if (fetchIdRef.current !== currentFetchId) {
                 return;
             }
             const data = response.data;
-            cacheRef.current.set(selectedWorkflowId, data);
+            cacheRef.current.set(cacheKey, data);
             setDetail(data);
             setDetailLoading(false);
         })
@@ -94,7 +96,7 @@ export const WorkflowListView = ({ runs, onBack }) => {
             setDetail(null);
             setDetailLoading(false);
         });
-    }, [selectedWorkflowId]);
+    }, [selectedWorkflowId, selectedRunId]);
     useInput((input, key) => {
         if (key.upArrow) {
             setSelectedIndex(i => Math.max(0, i - 1));
@@ -106,7 +108,7 @@ export const WorkflowListView = ({ runs, onBack }) => {
             onBack();
         }
         else if (input === 'o' && selectedWorkflowId) {
-            openUrl(`${TEMPORAL_UI_BASE}/namespaces/default/workflows/${selectedWorkflowId}`);
+            openUrl(`${config.temporalUiUrl}/namespaces/default/workflows/${selectedWorkflowId}`);
         }
     });
     const windowStart = useMemo(() => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@outputai/cli",
-  "version": "0.2.1-next.f1502fb.0",
+  "version": "0.3.0",
   "description": "CLI for Output.ai workflow generation",
   "type": "module",
   "main": "dist/index.js",
@@ -15,11 +15,11 @@
   },
   "dependencies": {
     "@anthropic-ai/claude-agent-sdk": "0.2.92",
-    "@aws-sdk/client-s3": "3.1031.0",
+    "@aws-sdk/client-s3": "3.1038.0",
     "@hackylabs/deep-redact": "3.0.5",
-    "@inquirer/prompts": "8.4.1",
-    "@oclif/core": "4.10.5",
-    "@oclif/plugin-help": "6.2.44",
+    "@inquirer/prompts": "8.4.2",
+    "@oclif/core": "4.10.6",
+    "@oclif/plugin-help": "6.2.45",
     "change-case": "5.4.4",
     "cli-progress": "3.12.0",
     "cli-table3": "0.6.5",
@@ -34,11 +34,11 @@
     "ky": "1.14.3",
     "react": "19.2.5",
     "semver": "7.7.4",
-    "undici": "8.0.2",
+    "undici": "8.1.0",
     "yaml": "^2.8.3",
-    "@outputai/credentials": "0.2.1-next.f1502fb.0",
-    "@outputai/evals": "0.2.1-next.f1502fb.0",
-    "@outputai/llm": "0.2.1-next.f1502fb.0"
+    "@outputai/credentials": "0.3.0",
+    "@outputai/llm": "0.3.0",
+    "@outputai/evals": "0.3.0"
   },
   "devDependencies": {
     "@types/cli-progress": "3.11.6",
@@ -46,7 +46,7 @@
     "@types/js-yaml": "4.0.9",
     "@types/react": "19.2.14",
     "@types/semver": "7.7.1",
-    "orval": "8.8.0"
+    "orval": "8.9.0"
   },
   "license": "Apache-2.0",
   "publishConfig": {