@outputai/cli 0.2.1-next.fd72d95.0 → 0.3.1-next.f1dd42f.0

package/dist/commands/workflow/run.spec.js

@@ -27,7 +27,7 @@ describe('workflow run command', () => {
             expect(WorkflowRun.args).toHaveProperty('workflowName');
             expect(WorkflowRun.flags).toHaveProperty('input');
             expect(WorkflowRun.flags).toHaveProperty('format');
-            expect(WorkflowRun.flags).toHaveProperty('task-queue');
+            expect(WorkflowRun.flags).toHaveProperty('catalog');
         });
         it('should have correct flag configuration', async () => {
             const WorkflowRun = (await import('./run.js')).default;
@@ -53,7 +53,7 @@ describe('workflow run command', () => {
             });
             cmd.parse = vi.fn().mockResolvedValue({
                 args: { workflowName: 'my_workflow', scenario: undefined },
-                flags: { input: undefined, 'task-queue': undefined, format: 'text' }
+                flags: { input: undefined, catalog: undefined, format: 'text' }
             });
             return { cmd, postWorkflowRun: vi.mocked(postWorkflowRun), resolveInput: vi.mocked(resolveInput) };
         };
@@ -67,7 +67,7 @@ describe('workflow run command', () => {
             });
             await cmd.run();
             expect(postWorkflowRun).toHaveBeenCalledTimes(1);
-            expect(postWorkflowRun).toHaveBeenCalledWith({ workflowName: 'my_workflow', input: { key: 'value' }, taskQueue: undefined }, expect.objectContaining({ config: { timeout: 600000 } }));
+            expect(postWorkflowRun).toHaveBeenCalledWith({ workflowName: 'my_workflow', input: { key: 'value' }, catalog: undefined }, expect.objectContaining({ config: { timeout: 600000 } }));
             expect(cmd.log).toHaveBeenCalledWith('Executing workflow: my_workflow...');
             expect(cmd.log).toHaveBeenCalledWith(expect.stringMatching(/\n/));
         });

package/dist/commands/workflow/runs/list.d.ts

@@ -6,6 +6,7 @@ export default class WorkflowRunsList extends Command {
         workflowName: import("@oclif/core/interfaces").Arg<string | undefined, Record<string, unknown>>;
     };
     static flags: {
+        catalog: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         limit: import("@oclif/core/interfaces").OptionFlag<number, import("@oclif/core/interfaces").CustomOptions>;
         format: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
     };

package/dist/commands/workflow/runs/list.js

@@ -55,6 +55,7 @@ export default class WorkflowRunsList extends Command {
         '<%= config.bin %> <%= command.id %>',
         '<%= config.bin %> <%= command.id %> simple',
         '<%= config.bin %> <%= command.id %> simple --limit 10',
+        '<%= config.bin %> <%= command.id %> --catalog my-catalog',
         '<%= config.bin %> <%= command.id %> --format json',
         '<%= config.bin %> <%= command.id %> --format table'
     ];
@@ -65,6 +66,11 @@ export default class WorkflowRunsList extends Command {
         })
     };
     static flags = {
+        catalog: Flags.string({
+            char: 'c',
+            description: 'Filter runs by catalog (defaults to OUTPUT_CATALOG_ID)',
+            env: 'OUTPUT_CATALOG_ID'
+        }),
         limit: Flags.integer({
             char: 'l',
             description: 'Maximum number of runs to return',
@@ -81,6 +87,7 @@ export default class WorkflowRunsList extends Command {
         const { args, flags } = await this.parse(WorkflowRunsList);
         const { runs, count } = await fetchWorkflowRuns({
             workflowType: args.workflowName,
+            catalog: flags.catalog,
             limit: flags.limit
         });
         if (runs.length === 0) {

package/dist/commands/workflow/start.d.ts

@@ -8,7 +8,7 @@ export default class WorkflowStart extends Command {
     };
     static flags: {
         input: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
-        'task-queue': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        catalog: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
     };
     run(): Promise<void>;
     catch(error: Error): Promise<void>;

package/dist/commands/workflow/start.js

@@ -8,7 +8,7 @@ export default class WorkflowStart extends Command {
         '<%= 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"}\' --task-queue my-queue'
+        '<%= config.bin %> <%= command.id %> simple --input \'{"key":"value"}\' --catalog my-catalog'
     ];
     static args = {
         workflowName: Args.string({
@@ -26,9 +26,12 @@ export default class WorkflowStart extends Command {
             description: 'Workflow input as JSON string or file path (overrides scenario)',
             required: false
         }),
-        'task-queue': Flags.string({
-            char: 'q',
-            description: 'Task queue name for workflow execution (defaults to OUTPUT_CATALOG_ID)',
+        catalog: Flags.string({
+            char: 'c',
+            aliases: ['task-queue'],
+            charAliases: ['q'],
+            deprecateAliases: true,
+            description: 'Catalog name for workflow execution (defaults to OUTPUT_CATALOG_ID)',
             env: 'OUTPUT_CATALOG_ID'
         })
     };
@@ -39,7 +42,7 @@ export default class WorkflowStart extends Command {
         const response = await postWorkflowStart({
             workflowName: args.workflowName,
             input,
-            taskQueue: flags['task-queue']
+            catalog: flags.catalog
         });
         if (!response || !response.data) {
             this.error('API returned invalid response', { exit: 1 });

package/dist/commands/workflow/start.spec.js

@@ -14,7 +14,7 @@ describe('workflow start command', () => {
             expect(WorkflowStart.description).toContain('Start a workflow');
             expect(WorkflowStart.args).toHaveProperty('workflowName');
             expect(WorkflowStart.flags).toHaveProperty('input');
-            expect(WorkflowStart.flags).toHaveProperty('task-queue');
+            expect(WorkflowStart.flags).toHaveProperty('catalog');
         });
         it('should have correct flag configuration', async () => {
             const WorkflowStart = (await import('./start.js')).default;

package/dist/config.d.ts

@@ -1,44 +1,17 @@
-/**
- * CLI configuration
- */
 export declare const config: {
-    /**
-     * Base URL for the Output.ai API server
-     * Can be overridden with OUTPUT_API_URL environment variable
-     */
-    apiUrl: string;
-    /**
-     * API authentication token
-     * Set via OUTPUT_API_AUTH_TOKEN environment variable
-     */
-    apiToken: string | undefined;
-    /**
-     * Default timeout for API requests (in milliseconds)
-     */
+    readonly apiUrl: string;
+    readonly ports: {
+        temporalUi: number;
+        api: number;
+    };
+    readonly temporalUiUrl: string;
+    readonly apiToken: string | undefined;
     requestTimeout: number;
-    /**
-     * Docker Compose project name
-     * Can be overridden with DOCKER_SERVICE_NAME environment variable
-     */
-    dockerServiceName: string;
-    /**
-     * Set the debug mode
-     */
-    debugMode: boolean;
-    /**
-     * Where the env vars are stored, defaults to `.env`
-     */
-    envFile: string;
-    /**
-     * Agent configuration directory name
-     */
+    readonly dockerServiceName: string;
+    readonly debugMode: boolean;
+    readonly envFile: string;
     agentConfigDir: string;
-    /**
-     * S3 configuration for remote trace storage
-     * Set via OUTPUT_TRACE_REMOTE_S3_BUCKET, OUTPUT_AWS_REGION,
-     * OUTPUT_AWS_ACCESS_KEY_ID, and OUTPUT_AWS_SECRET_ACCESS_KEY environment variables
-     */
-    s3: {
+    readonly s3: {
         bucket: string | undefined;
         region: string | undefined;
         accessKeyId: string | undefined;

package/dist/config.js

@@ -1,47 +1,39 @@
-/**
- * CLI configuration
- */
+import { parsePort } from '#utils/validation.js';
+const DEFAULT_API_PORT = 3001;
+const DEFAULT_TEMPORAL_UI_PORT = 8080;
 export const config = {
-    /**
-     * Base URL for the Output.ai API server
-     * Can be overridden with OUTPUT_API_URL environment variable
-     */
-    apiUrl: process.env.OUTPUT_API_URL || 'http://localhost:3001',
-    /**
-     * API authentication token
-     * Set via OUTPUT_API_AUTH_TOKEN environment variable
-     */
-    apiToken: process.env.OUTPUT_API_AUTH_TOKEN,
-    /**
-     * Default timeout for API requests (in milliseconds)
-     */
+    get apiUrl() {
+        return process.env.OUTPUT_API_URL || `http://localhost:${this.ports.api}`;
+    },
+    get ports() {
+        return {
+            temporalUi: parsePort(process.env.OUTPUT_TEMPORAL_UI_HOST_PORT, DEFAULT_TEMPORAL_UI_PORT, 'OUTPUT_TEMPORAL_UI_HOST_PORT'),
+            api: parsePort(process.env.OUTPUT_API_HOST_PORT, DEFAULT_API_PORT, 'OUTPUT_API_HOST_PORT')
+        };
+    },
+    get temporalUiUrl() {
+        return `http://localhost:${this.ports.temporalUi}`;
+    },
+    get apiToken() {
+        return process.env.OUTPUT_API_AUTH_TOKEN;
+    },
     requestTimeout: 30000,
-    /**
-     * Docker Compose project name
-     * Can be overridden with DOCKER_SERVICE_NAME environment variable
-     */
-    dockerServiceName: process.env.DOCKER_SERVICE_NAME || 'output-sdk',
-    /**
-     * Set the debug mode
-     */
-    debugMode: process.env.OUTPUT_DEBUG === 'true',
-    /**
-     * Where the env vars are stored, defaults to `.env`
-     */
-    envFile: process.env.OUTPUT_CLI_ENV || '.env',
-    /**
-     * Agent configuration directory name
-     */
+    get dockerServiceName() {
+        return process.env.DOCKER_SERVICE_NAME || 'output-sdk';
+    },
+    get debugMode() {
+        return process.env.OUTPUT_DEBUG === 'true';
+    },
+    get envFile() {
+        return process.env.OUTPUT_CLI_ENV || '.env';
+    },
     agentConfigDir: '.outputai',
-    /**
-     * S3 configuration for remote trace storage
-     * Set via OUTPUT_TRACE_REMOTE_S3_BUCKET, OUTPUT_AWS_REGION,
-     * OUTPUT_AWS_ACCESS_KEY_ID, and OUTPUT_AWS_SECRET_ACCESS_KEY environment variables
-     */
-    s3: {
-        bucket: process.env.OUTPUT_TRACE_REMOTE_S3_BUCKET,
-        region: process.env.OUTPUT_AWS_REGION,
-        accessKeyId: process.env.OUTPUT_AWS_ACCESS_KEY_ID,
-        secretAccessKey: process.env.OUTPUT_AWS_SECRET_ACCESS_KEY
+    get s3() {
+        return {
+            bucket: process.env.OUTPUT_TRACE_REMOTE_S3_BUCKET,
+            region: process.env.OUTPUT_AWS_REGION,
+            accessKeyId: process.env.OUTPUT_AWS_ACCESS_KEY_ID,
+            secretAccessKey: process.env.OUTPUT_AWS_SECRET_ACCESS_KEY
+        };
     }
 };

package/dist/config.spec.d.ts

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

package/dist/config.spec.js

@@ -0,0 +1,129 @@
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import { config } from '#config.js';
+import { InvalidPortError } from '#utils/validation.js';
+describe('config', () => {
+    const envVars = [
+        'OUTPUT_API_URL',
+        'OUTPUT_API_HOST_PORT',
+        'OUTPUT_TEMPORAL_UI_HOST_PORT',
+        'OUTPUT_API_AUTH_TOKEN',
+        'DOCKER_SERVICE_NAME',
+        'OUTPUT_DEBUG',
+        'OUTPUT_CLI_ENV',
+        'OUTPUT_TRACE_REMOTE_S3_BUCKET',
+        'OUTPUT_AWS_REGION',
+        'OUTPUT_AWS_ACCESS_KEY_ID',
+        'OUTPUT_AWS_SECRET_ACCESS_KEY'
+    ];
+    const saved = {};
+    beforeEach(() => {
+        for (const key of envVars) {
+            saved[key] = process.env[key];
+        }
+    });
+    afterEach(() => {
+        for (const key of envVars) {
+            if (saved[key] === undefined) {
+                delete process.env[key];
+            }
+            else {
+                process.env[key] = saved[key];
+            }
+        }
+    });
+    it('reads env vars lazily, not at module evaluation time', () => {
+        process.env.OUTPUT_API_URL = 'https://lazy-test.example.com';
+        expect(config.apiUrl).toBe('https://lazy-test.example.com');
+        process.env.OUTPUT_API_URL = 'https://changed.example.com';
+        expect(config.apiUrl).toBe('https://changed.example.com');
+    });
+    it('falls back to defaults when env vars are unset', () => {
+        delete process.env.OUTPUT_API_URL;
+        delete process.env.OUTPUT_API_HOST_PORT;
+        delete process.env.OUTPUT_TEMPORAL_UI_HOST_PORT;
+        delete process.env.DOCKER_SERVICE_NAME;
+        delete process.env.OUTPUT_DEBUG;
+        delete process.env.OUTPUT_CLI_ENV;
+        expect(config.apiUrl).toBe('http://localhost:3001');
+        expect(config.ports).toEqual({ temporalUi: 8080, api: 3001 });
+        expect(config.temporalUiUrl).toBe('http://localhost:8080');
+        expect(config.dockerServiceName).toBe('output-sdk');
+        expect(config.debugMode).toBe(false);
+        expect(config.envFile).toBe('.env');
+    });
+    it('derives apiUrl from OUTPUT_API_HOST_PORT when OUTPUT_API_URL is unset', () => {
+        delete process.env.OUTPUT_API_URL;
+        process.env.OUTPUT_API_HOST_PORT = '3002';
+        expect(config.apiUrl).toBe('http://localhost:3002');
+    });
+    it('OUTPUT_API_URL takes precedence over OUTPUT_API_HOST_PORT', () => {
+        process.env.OUTPUT_API_URL = 'https://api.example.com';
+        process.env.OUTPUT_API_HOST_PORT = '3002';
+        expect(config.apiUrl).toBe('https://api.example.com');
+    });
+    it('empty-string OUTPUT_API_URL falls through to OUTPUT_API_HOST_PORT (|| not ??)', () => {
+        process.env.OUTPUT_API_URL = '';
+        process.env.OUTPUT_API_HOST_PORT = '3002';
+        expect(config.apiUrl).toBe('http://localhost:3002');
+    });
+    it('reads port overrides from env vars', () => {
+        process.env.OUTPUT_TEMPORAL_UI_HOST_PORT = '8081';
+        process.env.OUTPUT_API_HOST_PORT = '3002';
+        expect(config.ports).toEqual({ temporalUi: 8081, api: 3002 });
+        expect(config.temporalUiUrl).toBe('http://localhost:8081');
+    });
+    it('treats empty-string port env vars as unset (matches Compose semantics)', () => {
+        delete process.env.OUTPUT_API_URL;
+        process.env.OUTPUT_API_HOST_PORT = '';
+        process.env.OUTPUT_TEMPORAL_UI_HOST_PORT = '';
+        expect(config.apiUrl).toBe('http://localhost:3001');
+        expect(config.ports).toEqual({ temporalUi: 8080, api: 3001 });
+        expect(config.temporalUiUrl).toBe('http://localhost:8080');
+    });
+    it('throws InvalidPortError on non-numeric port values', () => {
+        delete process.env.OUTPUT_API_URL;
+        process.env.OUTPUT_API_HOST_PORT = 'abc';
+        expect(() => config.ports).toThrow(InvalidPortError);
+        expect(() => config.apiUrl).toThrow(InvalidPortError);
+    });
+    it('throws InvalidPortError on out-of-range port values', () => {
+        process.env.OUTPUT_API_HOST_PORT = '99999';
+        expect(() => config.ports).toThrow(InvalidPortError);
+    });
+    it('throws InvalidPortError on trailing-junk port values', () => {
+        process.env.OUTPUT_TEMPORAL_UI_HOST_PORT = '8080abc';
+        expect(() => config.ports).toThrow(InvalidPortError);
+    });
+    it('throws InvalidPortError on port 0 (Compose treats 0 as ephemeral - prevents CLI/Docker desync)', () => {
+        process.env.OUTPUT_API_HOST_PORT = '0';
+        expect(() => config.ports).toThrow(InvalidPortError);
+    });
+    it('reads apiToken from env', () => {
+        process.env.OUTPUT_API_AUTH_TOKEN = 'test-token-123';
+        expect(config.apiToken).toBe('test-token-123');
+        delete process.env.OUTPUT_API_AUTH_TOKEN;
+        expect(config.apiToken).toBeUndefined();
+    });
+    it('reads debugMode as boolean', () => {
+        process.env.OUTPUT_DEBUG = 'true';
+        expect(config.debugMode).toBe(true);
+        process.env.OUTPUT_DEBUG = 'false';
+        expect(config.debugMode).toBe(false);
+    });
+    it('reads s3 config lazily', () => {
+        process.env.OUTPUT_TRACE_REMOTE_S3_BUCKET = 'my-bucket';
+        process.env.OUTPUT_AWS_REGION = 'us-west-2';
+        process.env.OUTPUT_AWS_ACCESS_KEY_ID = 'AKIA123';
+        process.env.OUTPUT_AWS_SECRET_ACCESS_KEY = 'secret123';
+        expect(config.s3).toEqual({
+            bucket: 'my-bucket',
+            region: 'us-west-2',
+            accessKeyId: 'AKIA123',
+            secretAccessKey: 'secret123'
+        });
+    });
+    it('has static properties that are not env-derived', () => {
+        expect(config.requestTimeout).toBe(30000);
+        expect(config.agentConfigDir).toBe('.outputai');
+    });
+});

package/dist/generated/framework_version.json

@@ -1,3 +1,3 @@
 {
-    "framework": "0.2.1-next.fd72d95.0"
+    "framework": "0.3.1-next.f1dd42f.0"
 }

package/dist/hooks/init.d.ts

@@ -1,3 +1,7 @@
 import { Hook } from '@oclif/core';
+export declare const INTERACTIVE_FLAGS: string[];
+export declare const GLOBAL_FLAGS: Set<string>;
+export declare const hasInteractiveFlag: (argv: string[]) => boolean;
+export declare const stripGlobalFlags: (argv: string[]) => void;
 declare const hook: Hook<'init'>;
 export default hook;

package/dist/hooks/init.js

@@ -1,6 +1,22 @@
 import { ux } from '@oclif/core';
 import { checkForUpdate } from '#services/version_check.js';
-const hook = async function () {
+import { setNonInteractive } from '#utils/interactive.js';
+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));
+export const stripGlobalFlags = (argv) => {
+    const kept = argv.filter(arg => !GLOBAL_FLAGS.has(arg));
+    if (kept.length !== argv.length) {
+        argv.splice(0, argv.length, ...kept);
+    }
+};
+const hook = async function (opts) {
+    const interactive = hasInteractiveFlag(opts.argv) || hasInteractiveFlag(process.argv);
+    stripGlobalFlags(opts.argv);
+    stripGlobalFlags(process.argv);
+    if (interactive) {
+        setNonInteractive(true);
+    }
     try {
         const result = await checkForUpdate(this.config.version, this.config.cacheDir);
         if (!result.updateAvailable) {

package/dist/hooks/init.spec.js

@@ -1,9 +1,13 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
-import { describe, it, expect, beforeEach, vi } from 'vitest';
+import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
 import { checkForUpdate } from '#services/version_check.js';
+import { setNonInteractive } from '#utils/interactive.js';
 vi.mock('#services/version_check.js', () => ({
     checkForUpdate: vi.fn()
 }));
+vi.mock('#utils/interactive.js', () => ({
+    setNonInteractive: vi.fn()
+}));
 vi.mock('@oclif/core', () => ({
     ux: {
         stdout: vi.fn(),
@@ -11,7 +15,7 @@ vi.mock('@oclif/core', () => ({
     }
 }));
 import { ux } from '@oclif/core';
-import hook from './init.js';
+import hook, { hasInteractiveFlag, stripGlobalFlags } from './init.js';
 describe('init hook', () => {
     beforeEach(() => {
         vi.clearAllMocks();
@@ -26,7 +30,7 @@ describe('init hook', () => {
             latestVersion: '1.0.0'
         });
         const ctx = createHookContext();
-        await hook.call(ctx, {});
+        await hook.call(ctx, { argv: [], id: undefined });
         expect(checkForUpdate).toHaveBeenCalledWith('0.8.4', '/tmp/test-cache');
         expect(ux.stdout).toHaveBeenCalled();
         const output = vi.mocked(ux.stdout).mock.calls.map(c => c[0]).join('\n');
@@ -42,13 +46,83 @@ describe('init hook', () => {
             latestVersion: '0.8.4'
         });
         const ctx = createHookContext();
-        await hook.call(ctx, {});
+        await hook.call(ctx, { argv: [], id: undefined });
         expect(ux.stdout).not.toHaveBeenCalled();
     });
     it('should silently handle errors', async () => {
         vi.mocked(checkForUpdate).mockRejectedValue(new Error('network failure'));
         const ctx = createHookContext();
-        await hook.call(ctx, {});
+        await hook.call(ctx, { argv: [], id: undefined });
         expect(ux.stdout).not.toHaveBeenCalled();
     });
+    describe('global interactive flags', () => {
+        const originalArgv = process.argv;
+        beforeEach(() => {
+            vi.mocked(checkForUpdate).mockResolvedValue({
+                updateAvailable: false,
+                currentVersion: '0.8.4',
+                latestVersion: '0.8.4'
+            });
+        });
+        afterEach(() => {
+            process.argv = originalArgv;
+        });
+        it('should mutate opts.argv in place to strip --yes', async () => {
+            process.argv = ['node', 'run.js', 'init', '--yes', 'my-project'];
+            const optsArgv = ['--yes', 'my-project'];
+            const argvRef = optsArgv;
+            const ctx = createHookContext();
+            await hook.call(ctx, { argv: optsArgv, id: 'init' });
+            expect(setNonInteractive).toHaveBeenCalledWith(true);
+            expect(optsArgv).toBe(argvRef);
+            expect(optsArgv).toEqual(['my-project']);
+            expect(process.argv).toEqual(['node', 'run.js', 'init', 'my-project']);
+        });
+        it('should mutate opts.argv in place to strip --non-interactive', async () => {
+            process.argv = ['node', 'run.js', 'init', '--non-interactive'];
+            const optsArgv = ['--non-interactive'];
+            const ctx = createHookContext();
+            await hook.call(ctx, { argv: optsArgv, id: 'init' });
+            expect(setNonInteractive).toHaveBeenCalledWith(true);
+            expect(optsArgv).toEqual([]);
+            expect(process.argv).toEqual(['node', 'run.js', 'init']);
+        });
+        it('should leave argv untouched when no global flag is present', async () => {
+            process.argv = ['node', 'run.js', 'init', '--skip-env'];
+            const optsArgv = ['--skip-env'];
+            const ctx = createHookContext();
+            await hook.call(ctx, { argv: optsArgv, id: 'init' });
+            expect(setNonInteractive).not.toHaveBeenCalled();
+            expect(optsArgv).toEqual(['--skip-env']);
+            expect(process.argv).toEqual(['node', 'run.js', 'init', '--skip-env']);
+        });
+    });
+    describe('hasInteractiveFlag', () => {
+        it('returns true when --yes is present', () => {
+            expect(hasInteractiveFlag(['init', '--yes', 'foo'])).toBe(true);
+        });
+        it('returns true when --non-interactive is present', () => {
+            expect(hasInteractiveFlag(['--non-interactive'])).toBe(true);
+        });
+        it('returns false for unrelated flags', () => {
+            expect(hasInteractiveFlag(['init', '--skip-env', '--skip-git'])).toBe(false);
+        });
+        it('returns false for an empty argv', () => {
+            expect(hasInteractiveFlag([])).toBe(false);
+        });
+    });
+    describe('stripGlobalFlags', () => {
+        it('mutates argv in place to remove global flags', () => {
+            const argv = ['init', '--yes', 'foo', '--non-interactive'];
+            const ref = argv;
+            stripGlobalFlags(argv);
+            expect(argv).toBe(ref);
+            expect(argv).toEqual(['init', 'foo']);
+        });
+        it('leaves argv untouched when no global flag is present', () => {
+            const argv = ['init', '--skip-env'];
+            stripGlobalFlags(argv);
+            expect(argv).toEqual(['init', '--skip-env']);
+        });
+    });
 });

package/dist/services/coding_agents.js

@@ -7,7 +7,8 @@ import { access } from 'node:fs/promises';
 import path from 'node:path';
 import { join } from 'node:path';
 import { ux } from '@oclif/core';
-import { confirm } from '@inquirer/prompts';
+import { confirm } from '#utils/prompt.js';
+import { isInteractive } from '#utils/interactive.js';
 import debugFactory from 'debug';
 import { getTemplateDir } from '#utils/paths.js';
 import { executeClaudeCommand } from '#utils/claude.js';
@@ -142,6 +143,9 @@ async function handlePluginError(error, commandName, silent = false) {
         debug('Plugin error: %s', pluginError.message);
         throw error;
     }
+    if (!isInteractive()) {
+        throw pluginError;
+    }
     ux.warn(pluginError.message);
     try {
         const shouldProceed = await confirm({

package/dist/services/coding_agents.spec.js

@@ -19,9 +19,12 @@ vi.mock('@oclif/core', () => ({
         colorize: vi.fn().mockImplementation((_color, text) => text)
     }
 }));
-vi.mock('@inquirer/prompts', () => ({
+vi.mock('#utils/prompt.js', () => ({
     confirm: vi.fn()
 }));
+vi.mock('#utils/interactive.js', () => ({
+    isInteractive: vi.fn(() => true)
+}));
 describe('coding_agents service', () => {
     beforeEach(() => {
         vi.clearAllMocks();
@@ -157,7 +160,7 @@ describe('coding_agents service', () => {
         });
         it('should show error and prompt user when plugin commands fail', async () => {
             const { executeClaudeCommand } = await import('../utils/claude.js');
-            const { confirm } = await import('@inquirer/prompts');
+            const { confirm } = await import('#utils/prompt.js');
             vi.mocked(executeClaudeCommand)
                 .mockResolvedValueOnce(undefined) // marketplace add
                 .mockRejectedValueOnce(new Error('Plugin update failed')); // marketplace update
@@ -169,7 +172,7 @@ describe('coding_agents service', () => {
         });
         it('should allow user to proceed without plugin setup if they confirm', async () => {
             const { executeClaudeCommand } = await import('../utils/claude.js');
-            const { confirm } = await import('@inquirer/prompts');
+            const { confirm } = await import('#utils/prompt.js');
             vi.mocked(executeClaudeCommand)
                 .mockRejectedValue(new Error('All plugin commands fail'));
             vi.mocked(confirm).mockResolvedValue(true);
@@ -219,7 +222,7 @@ describe('coding_agents service', () => {
         });
         it('should show error and prompt user when registerPluginMarketplace fails', async () => {
             const { executeClaudeCommand } = await import('../utils/claude.js');
-            const { confirm } = await import('@inquirer/prompts');
+            const { confirm } = await import('#utils/prompt.js');
             vi.mocked(executeClaudeCommand)
                 .mockResolvedValueOnce(undefined) // marketplace add
                 .mockRejectedValueOnce(new Error('Plugin update failed')); // marketplace update
@@ -231,7 +234,7 @@ describe('coding_agents service', () => {
         });
         it('should show error and prompt user when installOutputAIPlugin fails', async () => {
             const { executeClaudeCommand } = await import('../utils/claude.js');
-            const { confirm } = await import('@inquirer/prompts');
+            const { confirm } = await import('#utils/prompt.js');
             vi.mocked(executeClaudeCommand)
                 .mockResolvedValueOnce(undefined) // marketplace add
                 .mockResolvedValueOnce(undefined) // marketplace update
@@ -244,7 +247,7 @@ describe('coding_agents service', () => {
         });
         it('should allow user to proceed without plugin setup if they confirm', async () => {
             const { executeClaudeCommand } = await import('../utils/claude.js');
-            const { confirm } = await import('@inquirer/prompts');
+            const { confirm } = await import('#utils/prompt.js');
             vi.mocked(executeClaudeCommand)
                 .mockRejectedValue(new Error('All plugin commands fail'));
             vi.mocked(confirm).mockResolvedValue(true);
@@ -252,5 +255,15 @@ describe('coding_agents service', () => {
             // File operations should still complete
             expect(fs.mkdir).toHaveBeenCalled();
         });
+        it('should rethrow plugin error in non-interactive mode without prompting', async () => {
+            const { executeClaudeCommand } = await import('../utils/claude.js');
+            const { confirm } = await import('#utils/prompt.js');
+            const { isInteractive } = await import('#utils/interactive.js');
+            vi.mocked(isInteractive).mockReturnValueOnce(false);
+            vi.mocked(executeClaudeCommand)
+                .mockRejectedValueOnce(new Error('Plugin marketplace add failed'));
+            await expect(initializeAgentConfig({ projectRoot: '/test/project', force: true })).rejects.toThrow(/plugin marketplace add/i);
+            expect(confirm).not.toHaveBeenCalled();
+        });
     });
 });

package/dist/services/credentials_configurator.js

@@ -1,4 +1,4 @@
-import { password, confirm } from '@inquirer/prompts';
+import { password, confirm } from '#utils/prompt.js';
 import fs from 'node:fs';
 import path from 'node:path';
 import { ux } from '@oclif/core';