@outputai/cli 0.7.1-next.ae5bab4.0 → 0.7.1-next.bd6bd49.0

package/bin/run.js

@@ -7,7 +7,7 @@ import { loadCredentialRefs } from '../dist/utils/credentials_loader.js';
 
 // Load environment variables from .env files before executing CLI
 loadEnvironment();
-bootstrapProxy();
+await bootstrapProxy();
 loadCredentialRefs();
 
 await execute( { dir: import.meta.url } );

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

@@ -226,6 +226,39 @@ export declare const WorkflowResultResponseStatus: {
     readonly timed_out: "timed_out";
     readonly continued: "continued";
 };
+/**
+ * Structured failure details if the workflow failed, null otherwise
+ * @nullable
+ */
+export type WorkflowResultResponseErrorDetails = {
+    /**
+       * Friendly failure message (from the underlying application error)
+       * @nullable
+       */
+    message?: string | null;
+    /**
+       * Error name/type (the original error's class)
+       * @nullable
+       */
+    name?: string | null;
+    /**
+       * Whether Temporal flagged the failure retryable; null if unknown
+       * @nullable
+       */
+    retryable?: boolean | null;
+    /**
+       * Failing activity key ("workflow#step"); null if no activity failed
+       * @nullable
+       */
+    activityId?: string | null;
+    /**
+       * Sanitized error cause chain (name/message per level, no stack)
+       * @nullable
+       */
+    cause?: {
+        [key: string]: unknown;
+    } | null;
+} | null | null;
 export interface WorkflowResultResponse {
     /** The workflow execution id */
     workflowId?: string;
@@ -248,6 +281,11 @@ export interface WorkflowResultResponse {
        * @nullable
        */
     error?: string | null;
+    /**
+       * Structured failure details if the workflow failed, null otherwise
+       * @nullable
+       */
+    errorDetails?: WorkflowResultResponseErrorDetails;
 }
 export interface StopWorkflowResponse {
     workflowId?: string;

package/dist/assets/docker/docker-compose-dev.yml

@@ -80,7 +80,7 @@ services:
         condition: service_healthy
       worker:
         condition: service_healthy
-    image: outputai/api:${OUTPUT_API_VERSION:-0.7.1-next.ae5bab4.0}
+    image: outputai/api:${OUTPUT_API_VERSION:-0.7.1-next.bd6bd49.0}
     init: true
     networks:
       - main

package/dist/commands/update.js

@@ -31,7 +31,7 @@ export default class Update extends Command {
     async updateCli() {
         const latest = await fetchLatestVersion();
         if (!latest) {
-            this.error('Could not fetch the latest version from npm. Check your network connection.');
+            this.error('Could not fetch the latest version from the npm registry. Run with DEBUG=output-cli:npm-update for details.');
         }
         this.log(`\nLatest @outputai/cli version: v${latest}\n`);
         await this.handleGlobalUpdate(latest);

package/dist/generated/framework_version.json

@@ -1,3 +1,3 @@
 {
-    "framework": "0.7.1-next.ae5bab4.0"
+    "framework": "0.7.1-next.bd6bd49.0"
 }

package/dist/hooks/init.js

@@ -1,6 +1,8 @@
 import { ux } from '@oclif/core';
-import { checkForUpdate } from '#services/version_check.js';
+import debugFactory from 'debug';
+import { readCachedResult, spawnBackgroundRefresh } from '#services/version_check.js';
 import { setNonInteractive } from '#utils/interactive.js';
+const debug = debugFactory('output-cli:init');
 export const INTERACTIVE_FLAGS = ['--yes', '--non-interactive'];
 export const GLOBAL_FLAGS = new Set(INTERACTIVE_FLAGS);
 export const hasInteractiveFlag = (argv) => argv.some(arg => INTERACTIVE_FLAGS.includes(arg));
@@ -18,7 +20,13 @@ const hook = async function (opts) {
         setNonInteractive(true);
     }
     try {
-        const result = await checkForUpdate(this.config.version, this.config.cacheDir);
+        // Only the local cache is read here; the registry roundtrip happens in a
+        // detached child so it never delays the invoked command.
+        const result = await readCachedResult(this.config.version, this.config.cacheDir);
+        if (!result) {
+            spawnBackgroundRefresh(this.config.version, this.config.cacheDir);
+            return;
+        }
         if (!result.updateAvailable) {
             return;
         }
@@ -39,8 +47,9 @@ const hook = async function (opts) {
         ux.stdout(border);
         ux.stdout('');
     }
-    catch {
+    catch (error) {
         // Never block CLI execution
+        debug('Version banner failed: %O', error);
     }
 };
 export default hook;

package/dist/hooks/init.spec.js

@@ -1,9 +1,10 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
 import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import { checkForUpdate } from '#services/version_check.js';
+import { readCachedResult, spawnBackgroundRefresh } from '#services/version_check.js';
 import { setNonInteractive } from '#utils/interactive.js';
 vi.mock('#services/version_check.js', () => ({
-    checkForUpdate: vi.fn()
+    readCachedResult: vi.fn(),
+    spawnBackgroundRefresh: vi.fn()
 }));
 vi.mock('#utils/interactive.js', () => ({
     setNonInteractive: vi.fn()
@@ -23,15 +24,16 @@ describe('init hook', () => {
     const createHookContext = (version = '0.8.4') => ({
         config: { version, cacheDir: '/tmp/test-cache' }
     });
-    it('should display warning when update is available', async () => {
-        vi.mocked(checkForUpdate).mockResolvedValue({
+    it('should display warning when cached result says an update is available', async () => {
+        vi.mocked(readCachedResult).mockResolvedValue({
             updateAvailable: true,
             currentVersion: '0.8.4',
             latestVersion: '1.0.0'
         });
         const ctx = createHookContext();
         await hook.call(ctx, { argv: [], id: undefined });
-        expect(checkForUpdate).toHaveBeenCalledWith('0.8.4', '/tmp/test-cache');
+        expect(readCachedResult).toHaveBeenCalledWith('0.8.4', '/tmp/test-cache');
+        expect(spawnBackgroundRefresh).not.toHaveBeenCalled();
         expect(ux.stdout).toHaveBeenCalled();
         const output = vi.mocked(ux.stdout).mock.calls.map(c => c[0]).join('\n');
         expect(output).toContain('Uhoh');
@@ -40,17 +42,25 @@ describe('init hook', () => {
         expect(output).toContain('npx output update');
     });
     it('should not display anything when up to date', async () => {
-        vi.mocked(checkForUpdate).mockResolvedValue({
+        vi.mocked(readCachedResult).mockResolvedValue({
             updateAvailable: false,
             currentVersion: '0.8.4',
             latestVersion: '0.8.4'
         });
         const ctx = createHookContext();
         await hook.call(ctx, { argv: [], id: undefined });
+        expect(spawnBackgroundRefresh).not.toHaveBeenCalled();
+        expect(ux.stdout).not.toHaveBeenCalled();
+    });
+    it('should kick off a background refresh and stay silent when the cache is stale', async () => {
+        vi.mocked(readCachedResult).mockResolvedValue(null);
+        const ctx = createHookContext();
+        await hook.call(ctx, { argv: [], id: undefined });
+        expect(spawnBackgroundRefresh).toHaveBeenCalledWith('0.8.4', '/tmp/test-cache');
         expect(ux.stdout).not.toHaveBeenCalled();
     });
     it('should silently handle errors', async () => {
-        vi.mocked(checkForUpdate).mockRejectedValue(new Error('network failure'));
+        vi.mocked(readCachedResult).mockRejectedValue(new Error('cache failure'));
         const ctx = createHookContext();
         await hook.call(ctx, { argv: [], id: undefined });
         expect(ux.stdout).not.toHaveBeenCalled();
@@ -58,7 +68,7 @@ describe('init hook', () => {
     describe('global interactive flags', () => {
         const originalArgv = process.argv;
         beforeEach(() => {
-            vi.mocked(checkForUpdate).mockResolvedValue({
+            vi.mocked(readCachedResult).mockResolvedValue({
                 updateAvailable: false,
                 currentVersion: '0.8.4',
                 latestVersion: '0.8.4'

package/dist/scripts/refresh_version_check.d.ts

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

package/dist/scripts/refresh_version_check.js

@@ -0,0 +1,9 @@
+// Detached helper spawned by spawnBackgroundRefresh (version_check.ts):
+// refreshes the version-check cache off the critical path.
+// Args: <currentVersion> <cacheDir>
+import debugFactory from 'debug';
+import { bootstrapProxy } from '#utils/proxy.js';
+import { runRefresh } from '#services/version_check.js';
+const debug = debugFactory('output-cli:version-check');
+await bootstrapProxy().catch(error => debug('Proxy bootstrap failed: %O', error));
+process.exitCode = await runRefresh(process.argv);

package/dist/services/cost_calculator.d.ts

@@ -1,12 +1,8 @@
-import type { TraceNode, LLMCall, HTTPCall, TokenUsage, PricingConfig, ModelPricing, ServiceConfig, ServiceCostResult, CostReport } from '#types/cost.js';
+import type { TraceNode, LLMCall, HTTPCall, PricingConfig, ServiceConfig, ServiceCostResult, CostReport } from '#types/cost.js';
 export declare function extractValue(obj: unknown, path: string): unknown;
 export declare function loadPricingConfig(configPath?: string): PricingConfig;
 export declare function findLLMCalls(node: TraceNode, parentStepName?: string | null, seenIds?: Set<string>): LLMCall[];
 export declare function findHTTPCalls(node: TraceNode, parentStepName?: string | null, seenIds?: Set<string>): HTTPCall[];
-export declare function calculateLLMCallCost(usage: TokenUsage, modelPricing: ModelPricing | undefined): {
-    cost: number;
-    warning?: string;
-};
 export declare function identifyService(httpCall: HTTPCall, services: Record<string, ServiceConfig>): {
     serviceName: string;
     config: ServiceConfig;