@outputai/cli 0.10.1-next.09ed166.0 → 0.10.1-next.2caa4a1.0

package/dist/utils/resolve_input.js

@@ -1,7 +1,8 @@
 import { ux } from '@oclif/core';
 import { parseInputFlag } from '#utils/input_parser.js';
 import { resolveScenarioPath, getScenarioNotFoundMessage } from '#utils/scenario_resolver.js';
-export async function resolveInput(workflowName, scenario, inputFlag, commandName, catalog) {
+export async function resolveInput(options) {
+    const { workflowName, scenario, inputFlag, commandName, catalog, json } = options;
     if (inputFlag && scenario) {
         return ux.error('Cannot use both scenario argument and --input flag. Choose one.', { exit: 1 });
     }
@@ -13,7 +14,12 @@ export async function resolveInput(workflowName, scenario, inputFlag, commandNam
         if (!resolution.found) {
             return ux.error(getScenarioNotFoundMessage(workflowName, scenario, resolution.searchedPaths), { exit: 1 });
         }
-        ux.stdout(`Using scenario: ${resolution.path}\n`);
+        // Advisory notice goes to stderr so stdout stays clean for piping, and is
+        // skipped entirely under --json where even stderr is noise to a script
+        // consuming the structured output (same rule as the init hook's banner).
+        if (!json) {
+            ux.stderr(`Using scenario: ${resolution.path}`);
+        }
         return parseInputFlag(resolution.path);
     }
     return ux.error('Input required. Provide either:\n' +

package/dist/utils/resolve_input.spec.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/utils/resolve_input.spec.js

@@ -0,0 +1,75 @@
+/* eslint-disable @typescript-eslint/no-explicit-any */
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+vi.mock('@oclif/core', () => ({
+    ux: {
+        stdout: vi.fn(),
+        stderr: vi.fn(),
+        error: vi.fn(() => {
+            throw new Error('ux.error called');
+        })
+    }
+}));
+vi.mock('#utils/input_parser.js', () => ({
+    parseInputFlag: vi.fn()
+}));
+vi.mock('#utils/scenario_resolver.js', () => ({
+    resolveScenarioPath: vi.fn(),
+    getScenarioNotFoundMessage: vi.fn()
+}));
+describe('resolveInput', () => {
+    beforeEach(() => {
+        vi.clearAllMocks();
+    });
+    it('emits the scenario notice on stderr so --json stdout stays clean', async () => {
+        const { ux } = await import('@oclif/core');
+        const { parseInputFlag } = await import('#utils/input_parser.js');
+        const { resolveScenarioPath } = await import('#utils/scenario_resolver.js');
+        const { resolveInput } = await import('./resolve_input.js');
+        vi.mocked(resolveScenarioPath).mockResolvedValue({
+            found: true,
+            path: '/scenarios/happy_path.json'
+        });
+        vi.mocked(parseInputFlag).mockReturnValue({ key: 'value' });
+        const result = await resolveInput({
+            workflowName: 'web_search',
+            scenario: 'happy_path',
+            commandName: 'start'
+        });
+        expect(result).toEqual({ key: 'value' });
+        expect(ux.stderr).toHaveBeenCalledWith('Using scenario: /scenarios/happy_path.json');
+        expect(ux.stdout).not.toHaveBeenCalled();
+    });
+    it('suppresses the scenario notice entirely under --json', async () => {
+        const { ux } = await import('@oclif/core');
+        const { parseInputFlag } = await import('#utils/input_parser.js');
+        const { resolveScenarioPath } = await import('#utils/scenario_resolver.js');
+        const { resolveInput } = await import('./resolve_input.js');
+        vi.mocked(resolveScenarioPath).mockResolvedValue({
+            found: true,
+            path: '/scenarios/happy_path.json'
+        });
+        vi.mocked(parseInputFlag).mockReturnValue({ key: 'value' });
+        const result = await resolveInput({
+            workflowName: 'web_search',
+            scenario: 'happy_path',
+            commandName: 'start',
+            json: true
+        });
+        expect(result).toEqual({ key: 'value' });
+        expect(ux.stderr).not.toHaveBeenCalled();
+        expect(ux.stdout).not.toHaveBeenCalled();
+    });
+    it('does not emit the scenario notice when input comes from --input', async () => {
+        const { ux } = await import('@oclif/core');
+        const { parseInputFlag } = await import('#utils/input_parser.js');
+        const { resolveInput } = await import('./resolve_input.js');
+        vi.mocked(parseInputFlag).mockReturnValue({ key: 'value' });
+        await resolveInput({
+            workflowName: 'web_search',
+            inputFlag: '{"key":"value"}',
+            commandName: 'start'
+        });
+        expect(ux.stderr).not.toHaveBeenCalled();
+        expect(ux.stdout).not.toHaveBeenCalled();
+    });
+});

package/dist/views/dev/chrome/footer.d.ts

@@ -8,5 +8,7 @@ export interface FooterState {
     hints?: CommandHint[];
     itemCount?: number;
     itemLabel?: string;
+    /** Attached to a pre-existing stack — quitting leaves services running. */
+    attached?: boolean;
 }
 export declare const Footer: React.FC<FooterState>;

package/dist/views/dev/chrome/footer.js

@@ -5,16 +5,16 @@ import packageJson from '../../../../package.json' with { type: 'json' };
 const GLOBAL_HINT_ROWS = 1;
 const LOCAL_HINT_ROWS = 1;
 export const getHeight = () => GLOBAL_HINT_ROWS + LOCAL_HINT_ROWS;
-const GLOBAL_HINTS = [
+const globalHints = (attached) => [
     { key: 'tab', label: 'next tab' },
     { key: 'shift-tab', label: 'prev tab' },
     { key: '1-4', label: 'tabs' },
     { key: '/', label: 'search' },
     { key: '?', label: 'help' },
-    { key: 'ctrl+c', label: 'quit' }
+    { key: 'ctrl+c', label: attached ? 'detach (keeps running)' : 'quit' }
 ];
 const VERSION = packageJson.version;
 const HintRow = ({ hints }) => (_jsx(Box, { flexDirection: "row", children: hints.length === 0 ? (_jsx(Text, { children: " " })) : hints.map((hint, i) => (_jsxs(React.Fragment, { children: [i > 0 && _jsx(Text, { dimColor: true, children: '  ' }), _jsx(Text, { bold: true, children: hint.key }), _jsx(Text, { dimColor: true, children: ` ${hint.label}` })] }, hint.key))) }));
-export const Footer = ({ hints = [], itemCount, itemLabel }) => {
-    return (_jsxs(Box, { flexDirection: "column", children: [_jsxs(Box, { flexDirection: "row", justifyContent: "space-between", children: [_jsx(HintRow, { hints: GLOBAL_HINTS }), typeof itemCount === 'number' && itemLabel && (_jsx(Box, { children: _jsxs(Text, { dimColor: true, children: [itemCount, " ", itemLabel] }) }))] }), _jsxs(Box, { flexDirection: "row", justifyContent: "space-between", children: [_jsx(HintRow, { hints: hints }), _jsxs(Text, { color: "blackBright", children: ["v", VERSION] })] })] }));
+export const Footer = ({ hints = [], itemCount, itemLabel, attached = false }) => {
+    return (_jsxs(Box, { flexDirection: "column", children: [_jsxs(Box, { flexDirection: "row", justifyContent: "space-between", children: [_jsx(HintRow, { hints: globalHints(attached) }), typeof itemCount === 'number' && itemLabel && (_jsx(Box, { children: _jsxs(Text, { dimColor: true, children: [itemCount, " ", itemLabel] }) }))] }), _jsxs(Box, { flexDirection: "row", justifyContent: "space-between", children: [_jsx(HintRow, { hints: hints }), _jsxs(Text, { color: "blackBright", children: ["v", VERSION] })] })] }));
 };

package/dist/views/dev/dev_app.d.ts

@@ -3,4 +3,5 @@ export type Phase = 'waiting' | 'running' | 'failed';
 export declare const DevApp: React.FC<{
     dockerComposePath: string;
     onCleanup: () => Promise<void>;
+    attached?: boolean;
 }>;

package/dist/views/dev/dev_app.js

@@ -150,7 +150,7 @@ const overlayFor = (opts) => {
     }
     return null;
 };
-const Shell = ({ dockerComposePath, onCleanup }) => {
+const Shell = ({ dockerComposePath, onCleanup, attached }) => {
     const { exit } = useApp();
     const ui = useUiState();
     const [phase, setPhase] = useState('waiting');
@@ -160,7 +160,16 @@ const Shell = ({ dockerComposePath, onCleanup }) => {
         onServices: setServices,
         onAllHealthy: () => setPhase('running'),
         onFailure: () => setPhase('failed'),
-        onTimeout: () => exit(new Error('Timeout waiting for services to become healthy'))
+        // An attach/reconcile session monitors a stack it doesn't own, so a slow
+        // health check must not be fatal — drop into the dashboard and keep
+        // polling status. Only an owned fresh start treats the timeout as an error.
+        onTimeout: () => {
+            if (attached) {
+                setPhase('running');
+                return;
+            }
+            exit(new Error('Timeout waiting for services to become healthy'));
+        }
     });
     useStatusRefresh(dockerComposePath, phase !== 'waiting', setServices);
     useWorkflowRunsPolling(phase !== 'waiting', setRuns);
@@ -231,6 +240,6 @@ const Shell = ({ dockerComposePath, onCleanup }) => {
     if (overlay) {
         return (_jsx(Box, { flexDirection: "column", height: rows, paddingX: 1, children: overlay }));
     }
-    return (_jsxs(Box, { flexDirection: "column", height: rows, paddingX: 1, children: [_jsx(Header, { counters: counters }), _jsx(TabBar, { active: ui.tab, borderColor: RULE_PURPLE }), _jsx(SearchBar, { active: ui.search.open }), _jsxs(Box, { flexDirection: "column", flexGrow: 1, height: contentRows, overflow: "hidden", children: [ui.tab === 'workflows' && _jsx(WorkflowsPanel, { workflows: workflows, runs: runs }), ui.tab === 'runs' && _jsx(RunsPanel, { runs: runs, height: contentRows }), ui.tab === 'services' && (_jsx(ServicesPanel, { height: contentRows, phase: phase, services: services, dockerComposePath: dockerComposePath })), ui.tab === 'help' && _jsx(HelpPanel, {})] }), _jsx(Toasts, {}), _jsx(Footer, { hints: footer.hints, itemCount: footer.itemCount, itemLabel: footer.itemLabel })] }));
+    return (_jsxs(Box, { flexDirection: "column", height: rows, paddingX: 1, children: [_jsx(Header, { counters: counters }), _jsx(TabBar, { active: ui.tab, borderColor: RULE_PURPLE }), _jsx(SearchBar, { active: ui.search.open }), _jsxs(Box, { flexDirection: "column", flexGrow: 1, height: contentRows, overflow: "hidden", children: [ui.tab === 'workflows' && _jsx(WorkflowsPanel, { workflows: workflows, runs: runs }), ui.tab === 'runs' && _jsx(RunsPanel, { runs: runs, height: contentRows }), ui.tab === 'services' && (_jsx(ServicesPanel, { height: contentRows, phase: phase, services: services, dockerComposePath: dockerComposePath })), ui.tab === 'help' && _jsx(HelpPanel, {})] }), _jsx(Toasts, {}), _jsx(Footer, { hints: footer.hints, itemCount: footer.itemCount, itemLabel: footer.itemLabel, attached: attached })] }));
 };
-export const DevApp = ({ dockerComposePath, onCleanup }) => (_jsx(UiStateProvider, { children: _jsx(Shell, { dockerComposePath: dockerComposePath, onCleanup: onCleanup }) }));
+export const DevApp = ({ dockerComposePath, onCleanup, attached = false }) => (_jsx(UiStateProvider, { children: _jsx(Shell, { dockerComposePath: dockerComposePath, onCleanup: onCleanup, attached: attached }) }));

package/dist/views/dev/hooks/use_run_detail.js

@@ -3,13 +3,15 @@ import { readFile } from 'node:fs/promises';
 import { getWorkflowIdResult, getWorkflowIdRunsRidResult, getWorkflowIdTraceLog, getWorkflowIdRunsRidTraceLog } from '#api/generated/api.js';
 import { normalizeWorkflowStatus } from '#utils/normalize_workflow_status.js';
 import { TERMINAL_STATUSES } from '#utils/format_workflow_result.js';
+import { createBoundedCache } from '#views/dev/utils/bounded_cache.js';
 const EMPTY_DETAIL = {
     result: null,
     trace: null,
     steps: [],
     loading: false
 };
-const runDetailCache = new Map();
+const RUN_DETAIL_CACHE_MAX = 50;
+const runDetailCache = createBoundedCache(RUN_DETAIL_CACHE_MAX);
 const stepNameOf = (node) => {
     if (node.name) {
         return node.name;

package/dist/views/dev/hooks/use_step_graph.js

@@ -3,6 +3,7 @@ import { fetchWorkflowHistory } from '#services/workflow_history.js';
 import buildSpanLabels from '#utils/span_labels.js';
 import { isTerminalRunStatus } from '#views/dev/hooks/use_run_detail.js';
 import { usePoll, POLL_INTERVAL_MS } from '#views/dev/hooks/use_poll.js';
+import { createBoundedCache } from '#views/dev/utils/bounded_cache.js';
 const EMPTY_GRAPH = {
     spans: [],
     totalDurationMs: 0,
@@ -11,7 +12,8 @@ const EMPTY_GRAPH = {
     loading: false,
     error: null
 };
-const stepGraphCache = new Map();
+const STEP_GRAPH_CACHE_MAX = 50;
+const stepGraphCache = createBoundedCache(STEP_GRAPH_CACHE_MAX);
 /**
  * Fetches a run's correlated step spans for the dev TUI's waterfall overlay,
  * reusing the same `fetchWorkflowHistory` path as the `workflow history` CLI

package/dist/views/dev/utils/bounded_cache.d.ts

@@ -0,0 +1,14 @@
+export interface BoundedCache<K, V extends {}> {
+    get(key: K): V | undefined;
+    set(key: K, value: V): void;
+    has(key: K): boolean;
+    clear(): void;
+    size(): number;
+}
+/**
+ * A small LRU cache backed by an insertion-ordered `Map`, capped at `maxSize`
+ * entries. Reading a key refreshes its recency; once the cap is exceeded the
+ * least-recently-used entries are evicted. Drop-in compatible with the
+ * `Map.get` / `Map.set` calls it replaces.
+ */
+export declare const createBoundedCache: <K, V extends {}>(maxSize: number) => BoundedCache<K, V>;

package/dist/views/dev/utils/bounded_cache.js

@@ -0,0 +1,42 @@
+/**
+ * A small LRU cache backed by an insertion-ordered `Map`, capped at `maxSize`
+ * entries. Reading a key refreshes its recency; once the cap is exceeded the
+ * least-recently-used entries are evicted. Drop-in compatible with the
+ * `Map.get` / `Map.set` calls it replaces.
+ */
+export const createBoundedCache = (maxSize) => {
+    if (maxSize < 1) {
+        throw new Error('createBoundedCache: maxSize must be >= 1');
+    }
+    const entries = new Map();
+    return {
+        get(key) {
+            const value = entries.get(key);
+            if (value === undefined) {
+                return value;
+            }
+            entries.delete(key);
+            entries.set(key, value);
+            return value;
+        },
+        set(key, value) {
+            entries.delete(key);
+            entries.set(key, value);
+            if (entries.size > maxSize) {
+                const oldest = entries.keys().next();
+                if (!oldest.done) {
+                    entries.delete(oldest.value);
+                }
+            }
+        },
+        has(key) {
+            return entries.has(key);
+        },
+        clear() {
+            entries.clear();
+        },
+        size() {
+            return entries.size;
+        }
+    };
+};

package/dist/views/dev/utils/bounded_cache.spec.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/views/dev/utils/bounded_cache.spec.js

@@ -0,0 +1,53 @@
+import { describe, expect, it } from 'vitest';
+import { createBoundedCache } from './bounded_cache.js';
+describe('createBoundedCache', () => {
+    it('round-trips set and get', () => {
+        const cache = createBoundedCache(3);
+        cache.set('a', 1);
+        expect(cache.get('a')).toBe(1);
+        expect(cache.get('missing')).toBeUndefined();
+    });
+    it('evicts the oldest entry once maxSize is exceeded', () => {
+        const cache = createBoundedCache(2);
+        cache.set('a', 1);
+        cache.set('b', 2);
+        cache.set('c', 3);
+        expect(cache.has('a')).toBe(false);
+        expect(cache.get('b')).toBe(2);
+        expect(cache.get('c')).toBe(3);
+    });
+    it('refreshes recency on get so the touched entry survives eviction', () => {
+        const cache = createBoundedCache(2);
+        cache.set('a', 1);
+        cache.set('b', 2);
+        // Touch 'a' so 'b' becomes the least-recently-used entry.
+        expect(cache.get('a')).toBe(1);
+        cache.set('c', 3);
+        expect(cache.has('a')).toBe(true);
+        expect(cache.has('b')).toBe(false);
+        expect(cache.has('c')).toBe(true);
+    });
+    it('updates an existing key in place without growing', () => {
+        const cache = createBoundedCache(2);
+        cache.set('a', 1);
+        cache.set('a', 2);
+        expect(cache.get('a')).toBe(2);
+        expect(cache.size()).toBe(1);
+    });
+    it('supports has and clear', () => {
+        const cache = createBoundedCache(2);
+        cache.set('a', 1);
+        expect(cache.has('a')).toBe(true);
+        cache.clear();
+        expect(cache.has('a')).toBe(false);
+        expect(cache.size()).toBe(0);
+    });
+    it('never exceeds maxSize', () => {
+        const cache = createBoundedCache(3);
+        Array.from({ length: 100 }, (_, i) => i).forEach(i => {
+            cache.set(`key-${i}`, i);
+            expect(cache.size()).toBeLessThanOrEqual(3);
+        });
+        expect(cache.size()).toBe(3);
+    });
+});

package/oclif.manifest.json

@@ -416,6 +416,41 @@
         "show.js"
       ]
     },
+    "dev:down": {
+      "aliases": [],
+      "args": {},
+      "description": "Stop Output development services started by `output dev`\n\nUseful after `output dev -d`, or when an attached `output dev` session\nleft the services running on quit.",
+      "examples": [
+        "<%= config.bin %> <%= command.id %>",
+        "<%= config.bin %> <%= command.id %> --compose-file ./custom-docker-compose.yml"
+      ],
+      "flags": {
+        "compose-file": {
+          "char": "f",
+          "description": "Path to a custom docker-compose file",
+          "name": "compose-file",
+          "required": false,
+          "hasDynamicHelp": false,
+          "multiple": false,
+          "type": "option"
+        }
+      },
+      "hasDynamicHelp": false,
+      "hiddenAliases": [],
+      "id": "dev:down",
+      "pluginAlias": "@outputai/cli",
+      "pluginName": "@outputai/cli",
+      "pluginType": "core",
+      "strict": true,
+      "enableJsonFlag": false,
+      "isESM": true,
+      "relativePath": [
+        "dist",
+        "commands",
+        "dev",
+        "down.js"
+      ]
+    },
     "dev:eject": {
       "aliases": [],
       "args": {},
@@ -463,9 +498,10 @@
     "dev": {
       "aliases": [],
       "args": {},
-      "description": "Start Output development services (auto-restarts worker on file changes)\n\nTo run a second dev stack concurrently, override host ports in .env:\n\n  OUTPUT_API_HOST_PORT=3002\n  OUTPUT_TEMPORAL_UI_HOST_PORT=8081\n  OUTPUT_TEMPORAL_HOST_PORT=7234",
+      "description": "Start Output development services (auto-restarts worker on file changes)\n\nIf services are already running (e.g. after `output dev -d`), this attaches\nto monitor them rather than treating our own containers as a port collision.\nQuitting an attached session leaves the services running — stop them with\n`output dev down`.\n\nTo run a second dev stack concurrently, give it its own compose project and\nhost ports in .env — without DOCKER_SERVICE_NAME both checkouts share one\nstack, and the second will attach to the first instead of starting:\n\n  DOCKER_SERVICE_NAME=output-sdk-two\n  OUTPUT_API_HOST_PORT=3002\n  OUTPUT_TEMPORAL_UI_HOST_PORT=8081\n  OUTPUT_TEMPORAL_HOST_PORT=7234",
       "examples": [
         "<%= config.bin %> <%= command.id %>",
+        "<%= config.bin %> <%= command.id %> --detached",
         "<%= config.bin %> <%= command.id %> --compose-file ./custom-docker-compose.yml",
         "<%= config.bin %> <%= command.id %> --image-pull-policy missing"
       ],
@@ -1200,9 +1236,17 @@
         "<%= config.bin %> <%= command.id %> simple basic_input",
         "<%= config.bin %> <%= command.id %> simple --input '{\"values\":[1,2,3]}'",
         "<%= config.bin %> <%= command.id %> simple --input input.json",
-        "<%= config.bin %> <%= command.id %> simple --input '{\"key\":\"value\"}' --catalog my-catalog"
+        "<%= config.bin %> <%= command.id %> simple --input '{\"key\":\"value\"}' --catalog my-catalog",
+        "<%= config.bin %> <%= command.id %> simple --json"
       ],
       "flags": {
+        "json": {
+          "description": "Format output as json.",
+          "helpGroup": "GLOBAL",
+          "name": "json",
+          "allowNo": false,
+          "type": "boolean"
+        },
         "input": {
           "char": "i",
           "description": "Workflow input as JSON string or file path (overrides scenario)",
@@ -1236,7 +1280,7 @@
       "pluginName": "@outputai/cli",
       "pluginType": "core",
       "strict": true,
-      "enableJsonFlag": false,
+      "enableJsonFlag": true,
       "isESM": true,
       "relativePath": [
         "dist",
@@ -1671,5 +1715,5 @@
       ]
     }
   },
-  "version": "0.10.1-next.09ed166.0"
+  "version": "0.10.1-next.2caa4a1.0"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@outputai/cli",
-  "version": "0.10.1-next.09ed166.0",
+  "version": "0.10.1-next.2caa4a1.0",
   "description": "CLI for Output.ai workflow generation",
   "type": "module",
   "main": "dist/index.js",
@@ -31,16 +31,15 @@
     "handlebars": "4.7.9",
     "ink": "7.0.1",
     "ink-spinner": "5.0.0",
-    "js-yaml": "4.1.1",
+    "js-yaml": "4.3.0",
     "json-schema-library": "11.4.0",
-    "ky": "1.14.3",
+    "ky": "2.0.2",
     "react": "19.2.5",
     "semver": "7.7.4",
-    "undici": "8.5.0",
-    "yaml": "^2.8.3",
-    "@outputai/credentials": "0.10.1-next.09ed166.0",
-    "@outputai/llm": "0.10.1-next.09ed166.0",
-    "@outputai/evals": "0.10.1-next.09ed166.0"
+    "undici": "8.9.0",
+    "@outputai/credentials": "0.10.1-next.2caa4a1.0",
+    "@outputai/evals": "0.10.1-next.2caa4a1.0",
+    "@outputai/llm": "0.10.1-next.2caa4a1.0"
   },
   "devDependencies": {
     "@types/cli-progress": "3.11.6",