@orchagent/cli 0.3.31 → 0.3.33

package/dist/commands/run.test.js

@@ -0,0 +1,330 @@
+"use strict";
+/**
+ * Tests for the run command and LLM utilities.
+ *
+ * These tests cover downloading and running agents locally:
+ * - parseAgentRef() parsing org/agent@version formats
+ * - downloadAgent() fetching from API
+ * - LLM key detection from environment
+ * - Building prompts with variable substitution
+ */
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const vitest_1 = require("vitest");
+const commander_1 = require("commander");
+// Mock modules before importing
+vitest_1.vi.mock('fs/promises');
+vitest_1.vi.mock('../lib/config');
+vitest_1.vi.mock('../lib/api');
+const promises_1 = __importDefault(require("fs/promises"));
+const run_1 = require("./run");
+const config_1 = require("../lib/config");
+const api_1 = require("../lib/api");
+const llm_1 = require("../lib/llm");
+const mockFs = vitest_1.vi.mocked(promises_1.default);
+const mockGetResolvedConfig = vitest_1.vi.mocked(config_1.getResolvedConfig);
+const mockPublicRequest = vitest_1.vi.mocked(api_1.publicRequest);
+const mockGetPublicAgent = vitest_1.vi.mocked(api_1.getPublicAgent);
+(0, vitest_1.describe)('run command - agent ref parsing', () => {
+    let program;
+    let stdoutSpy;
+    let stderrSpy;
+    (0, vitest_1.beforeEach)(() => {
+        vitest_1.vi.clearAllMocks();
+        program = new commander_1.Command();
+        program.exitOverride();
+        (0, run_1.registerRunCommand)(program);
+        stdoutSpy = vitest_1.vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        stderrSpy = vitest_1.vi.spyOn(process.stderr, 'write').mockImplementation(() => true);
+        mockGetResolvedConfig.mockResolvedValue({
+            apiKey: 'sk_test_123',
+            apiUrl: 'https://api.test.com',
+            defaultOrg: 'default-org',
+        });
+        // Mock mkdir for agent saving
+        mockFs.mkdir.mockResolvedValue(undefined);
+        mockFs.writeFile.mockResolvedValue(undefined);
+    });
+    (0, vitest_1.afterEach)(() => {
+        stdoutSpy.mockRestore();
+        stderrSpy.mockRestore();
+        vitest_1.vi.restoreAllMocks();
+    });
+    (0, vitest_1.it)('parses org/agent@version format', async () => {
+        mockPublicRequest.mockResolvedValue({
+            type: 'code',
+            name: 'my-agent',
+            version: 'v2',
+            supported_providers: ['any'],
+        });
+        await program.parseAsync(['node', 'test', 'run', 'myorg/my-agent@v2']);
+        (0, vitest_1.expect)(mockPublicRequest).toHaveBeenCalledWith(vitest_1.expect.any(Object), '/public/agents/myorg/my-agent/v2/download');
+    });
+    (0, vitest_1.it)('parses org/agent format with default version', async () => {
+        mockPublicRequest.mockResolvedValue({
+            type: 'code',
+            name: 'my-agent',
+            version: 'latest',
+            supported_providers: ['any'],
+        });
+        await program.parseAsync(['node', 'test', 'run', 'myorg/my-agent']);
+        (0, vitest_1.expect)(mockPublicRequest).toHaveBeenCalledWith(vitest_1.expect.any(Object), '/public/agents/myorg/my-agent/latest/download');
+    });
+    (0, vitest_1.it)('uses defaultOrg when no org specified', async () => {
+        mockPublicRequest.mockResolvedValue({
+            type: 'code',
+            name: 'my-agent',
+            version: 'latest',
+            supported_providers: ['any'],
+        });
+        await program.parseAsync(['node', 'test', 'run', 'my-agent']);
+        (0, vitest_1.expect)(mockPublicRequest).toHaveBeenCalledWith(vitest_1.expect.any(Object), '/public/agents/default-org/my-agent/latest/download');
+    });
+    (0, vitest_1.it)('parses agent@version format', async () => {
+        mockPublicRequest.mockResolvedValue({
+            type: 'code',
+            name: 'my-agent',
+            version: 'v3',
+            supported_providers: ['any'],
+        });
+        await program.parseAsync(['node', 'test', 'run', 'my-agent@v3']);
+        (0, vitest_1.expect)(mockPublicRequest).toHaveBeenCalledWith(vitest_1.expect.any(Object), '/public/agents/default-org/my-agent/v3/download');
+    });
+    (0, vitest_1.it)('throws error for invalid agent ref format', async () => {
+        mockGetResolvedConfig.mockResolvedValue({
+            apiUrl: 'https://api.test.com',
+            // No defaultOrg
+        });
+        await (0, vitest_1.expect)(program.parseAsync(['node', 'test', 'run', 'just-agent'])).rejects.toThrow('Missing org');
+    });
+    (0, vitest_1.it)('throws error for too many segments', async () => {
+        await (0, vitest_1.expect)(program.parseAsync(['node', 'test', 'run', 'a/b/c/d'])).rejects.toThrow('Invalid agent reference');
+    });
+});
+(0, vitest_1.describe)('run command - download agent', () => {
+    let program;
+    let stdoutSpy;
+    let stderrSpy;
+    (0, vitest_1.beforeEach)(() => {
+        vitest_1.vi.clearAllMocks();
+        program = new commander_1.Command();
+        program.exitOverride();
+        (0, run_1.registerRunCommand)(program);
+        stdoutSpy = vitest_1.vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        stderrSpy = vitest_1.vi.spyOn(process.stderr, 'write').mockImplementation(() => true);
+        mockGetResolvedConfig.mockResolvedValue({
+            apiKey: 'sk_test_123',
+            apiUrl: 'https://api.test.com',
+            defaultOrg: 'test-org',
+        });
+        mockFs.mkdir.mockResolvedValue(undefined);
+        mockFs.writeFile.mockResolvedValue(undefined);
+    });
+    (0, vitest_1.afterEach)(() => {
+        stdoutSpy.mockRestore();
+        stderrSpy.mockRestore();
+        vitest_1.vi.restoreAllMocks();
+    });
+    (0, vitest_1.it)('downloads agent via public API', async () => {
+        mockPublicRequest.mockResolvedValue({
+            type: 'prompt',
+            name: 'test-agent',
+            version: 'v1',
+            description: 'A test agent',
+            prompt: 'You are a test assistant.',
+            supported_providers: ['openai'],
+        });
+        await program.parseAsync(['node', 'test', 'run', 'test-org/test-agent@v1']);
+        (0, vitest_1.expect)(mockPublicRequest).toHaveBeenCalledWith({ apiKey: 'sk_test_123', apiUrl: 'https://api.test.com', defaultOrg: 'test-org' }, '/public/agents/test-org/test-agent/v1/download');
+    });
+    (0, vitest_1.it)('falls back to getPublicAgent if download fails', async () => {
+        mockPublicRequest.mockRejectedValue(new Error('Not found'));
+        mockGetPublicAgent.mockResolvedValue({
+            id: 'agent-123',
+            org_id: 'org-123',
+            name: 'fallback-agent',
+            version: 'v1',
+            type: 'code',
+            supported_providers: ['any'],
+            is_public: true,
+        });
+        await program.parseAsync(['node', 'test', 'run', 'test-org/fallback-agent@v1']);
+        (0, vitest_1.expect)(mockGetPublicAgent).toHaveBeenCalledWith(vitest_1.expect.any(Object), 'test-org', 'fallback-agent', 'v1');
+    });
+    (0, vitest_1.it)('saves agent metadata locally', async () => {
+        mockPublicRequest.mockResolvedValue({
+            type: 'prompt',
+            name: 'saved-agent',
+            version: 'v1',
+            prompt: 'Test prompt',
+            supported_providers: ['any'],
+        });
+        await program.parseAsync(['node', 'test', 'run', 'test-org/saved-agent@v1']);
+        (0, vitest_1.expect)(mockFs.mkdir).toHaveBeenCalled();
+        (0, vitest_1.expect)(mockFs.writeFile).toHaveBeenCalledWith(vitest_1.expect.stringContaining('agent.json'), vitest_1.expect.stringContaining('"name": "saved-agent"'));
+    });
+    (0, vitest_1.it)('saves prompt.md for prompt agents', async () => {
+        mockPublicRequest.mockResolvedValue({
+            type: 'prompt',
+            name: 'prompt-agent',
+            version: 'v1',
+            prompt: 'You are a helpful assistant.',
+            supported_providers: ['any'],
+        });
+        await program.parseAsync(['node', 'test', 'run', 'test-org/prompt-agent@v1']);
+        (0, vitest_1.expect)(mockFs.writeFile).toHaveBeenCalledWith(vitest_1.expect.stringContaining('prompt.md'), 'You are a helpful assistant.');
+    });
+    (0, vitest_1.it)('handles --download-only flag', async () => {
+        mockPublicRequest.mockResolvedValue({
+            type: 'prompt',
+            name: 'download-only-agent',
+            version: 'v1',
+            prompt: 'Test',
+            supported_providers: ['any'],
+        });
+        await program.parseAsync([
+            'node',
+            'test',
+            'run',
+            'test-org/download-only-agent@v1',
+            '--download-only',
+        ]);
+        (0, vitest_1.expect)(stdoutSpy).toHaveBeenCalledWith(vitest_1.expect.stringContaining('Agent downloaded'));
+    });
+});
+(0, vitest_1.describe)('LLM utilities - detectLlmKeyFromEnv', () => {
+    const originalEnv = process.env;
+    (0, vitest_1.beforeEach)(() => {
+        process.env = { ...originalEnv };
+        delete process.env.OPENAI_API_KEY;
+        delete process.env.ANTHROPIC_API_KEY;
+        delete process.env.GEMINI_API_KEY;
+    });
+    (0, vitest_1.afterEach)(() => {
+        process.env = originalEnv;
+    });
+    (0, vitest_1.it)('returns null when no keys found', () => {
+        const result = (0, llm_1.detectLlmKeyFromEnv)(['openai']);
+        (0, vitest_1.expect)(result).toBeNull();
+    });
+    (0, vitest_1.it)('detects OpenAI key', () => {
+        process.env.OPENAI_API_KEY = 'sk-openai-test';
+        const result = (0, llm_1.detectLlmKeyFromEnv)(['openai']);
+        (0, vitest_1.expect)(result).toEqual({ provider: 'openai', key: 'sk-openai-test' });
+    });
+    (0, vitest_1.it)('detects Anthropic key', () => {
+        process.env.ANTHROPIC_API_KEY = 'sk-ant-test';
+        const result = (0, llm_1.detectLlmKeyFromEnv)(['anthropic']);
+        (0, vitest_1.expect)(result).toEqual({ provider: 'anthropic', key: 'sk-ant-test' });
+    });
+    (0, vitest_1.it)('detects Gemini key', () => {
+        process.env.GEMINI_API_KEY = 'AIza-gemini-test';
+        const result = (0, llm_1.detectLlmKeyFromEnv)(['gemini']);
+        (0, vitest_1.expect)(result).toEqual({ provider: 'gemini', key: 'AIza-gemini-test' });
+    });
+    (0, vitest_1.it)('returns first matching provider in order', () => {
+        process.env.ANTHROPIC_API_KEY = 'sk-ant-test';
+        process.env.OPENAI_API_KEY = 'sk-openai-test';
+        const result = (0, llm_1.detectLlmKeyFromEnv)(['openai', 'anthropic']);
+        (0, vitest_1.expect)(result).toEqual({ provider: 'openai', key: 'sk-openai-test' });
+    });
+    (0, vitest_1.it)('handles "any" provider - checks all in order', () => {
+        process.env.GEMINI_API_KEY = 'AIza-gemini-test';
+        const result = (0, llm_1.detectLlmKeyFromEnv)(['any']);
+        // 'any' should find the first available key
+        (0, vitest_1.expect)(result?.key).toBe('AIza-gemini-test');
+    });
+    (0, vitest_1.it)('handles "any" provider - returns first found', () => {
+        process.env.OPENAI_API_KEY = 'sk-openai-test';
+        process.env.ANTHROPIC_API_KEY = 'sk-ant-test';
+        const result = (0, llm_1.detectLlmKeyFromEnv)(['any']);
+        // OpenAI is checked first in the provider order
+        (0, vitest_1.expect)(result).toEqual({ provider: 'openai', key: 'sk-openai-test' });
+    });
+    (0, vitest_1.it)('skips unavailable providers', () => {
+        process.env.ANTHROPIC_API_KEY = 'sk-ant-test';
+        // No OpenAI key
+        const result = (0, llm_1.detectLlmKeyFromEnv)(['openai', 'anthropic']);
+        (0, vitest_1.expect)(result).toEqual({ provider: 'anthropic', key: 'sk-ant-test' });
+    });
+});
+(0, vitest_1.describe)('LLM utilities - getDefaultModel', () => {
+    (0, vitest_1.it)('returns default model for OpenAI', () => {
+        (0, vitest_1.expect)((0, llm_1.getDefaultModel)('openai')).toBe('gpt-4o');
+    });
+    (0, vitest_1.it)('returns default model for Anthropic', () => {
+        (0, vitest_1.expect)((0, llm_1.getDefaultModel)('anthropic')).toBe('claude-sonnet-4-20250514');
+    });
+    (0, vitest_1.it)('returns default model for Gemini', () => {
+        (0, vitest_1.expect)((0, llm_1.getDefaultModel)('gemini')).toBe('gemini-1.5-pro');
+    });
+    (0, vitest_1.it)('returns gpt-4o for unknown provider', () => {
+        (0, vitest_1.expect)((0, llm_1.getDefaultModel)('unknown')).toBe('gpt-4o');
+    });
+});
+(0, vitest_1.describe)('LLM utilities - buildPrompt', () => {
+    (0, vitest_1.it)('substitutes single variable', () => {
+        const template = 'Analyze this: {{input}}';
+        const result = (0, llm_1.buildPrompt)(template, { input: 'Hello world' });
+        (0, vitest_1.expect)(result).toContain('Analyze this: Hello world');
+    });
+    (0, vitest_1.it)('substitutes multiple variables', () => {
+        const template = 'Name: {{name}}, Age: {{age}}';
+        const result = (0, llm_1.buildPrompt)(template, { name: 'Alice', age: 30 });
+        (0, vitest_1.expect)(result).toContain('Name: Alice, Age: 30');
+    });
+    (0, vitest_1.it)('appends JSON input block', () => {
+        const template = 'Process this';
+        const result = (0, llm_1.buildPrompt)(template, { key: 'value' });
+        (0, vitest_1.expect)(result).toContain('Input:');
+        (0, vitest_1.expect)(result).toContain('```json');
+        (0, vitest_1.expect)(result).toContain('"key": "value"');
+    });
+    (0, vitest_1.it)('handles empty input data', () => {
+        const template = 'No inputs needed';
+        const result = (0, llm_1.buildPrompt)(template, {});
+        (0, vitest_1.expect)(result).toBe('No inputs needed');
+    });
+    (0, vitest_1.it)('handles multiple occurrences of same variable', () => {
+        const template = '{{name}} said hello. {{name}} waved goodbye.';
+        const result = (0, llm_1.buildPrompt)(template, { name: 'Bob' });
+        (0, vitest_1.expect)(result).toContain('Bob said hello. Bob waved goodbye.');
+    });
+    (0, vitest_1.it)('preserves unused placeholders', () => {
+        const template = 'Value: {{exists}}, Missing: {{missing}}';
+        const result = (0, llm_1.buildPrompt)(template, { exists: 'here' });
+        (0, vitest_1.expect)(result).toContain('Value: here');
+        (0, vitest_1.expect)(result).toContain('{{missing}}');
+    });
+    (0, vitest_1.it)('handles complex nested objects in JSON', () => {
+        const template = 'Process data';
+        const inputData = {
+            nested: { deep: { value: 42 } },
+            array: [1, 2, 3],
+        };
+        const result = (0, llm_1.buildPrompt)(template, inputData);
+        (0, vitest_1.expect)(result).toContain('"nested"');
+        (0, vitest_1.expect)(result).toContain('"deep"');
+        (0, vitest_1.expect)(result).toContain('"array"');
+    });
+    (0, vitest_1.it)('converts non-string values to strings', () => {
+        const template = 'Count: {{count}}, Active: {{active}}';
+        const result = (0, llm_1.buildPrompt)(template, { count: 42, active: true });
+        (0, vitest_1.expect)(result).toContain('Count: 42');
+        (0, vitest_1.expect)(result).toContain('Active: true');
+    });
+});
+(0, vitest_1.describe)('LLM utilities - constants', () => {
+    (0, vitest_1.it)('has correct provider env vars', () => {
+        (0, vitest_1.expect)(llm_1.PROVIDER_ENV_VARS.openai).toBe('OPENAI_API_KEY');
+        (0, vitest_1.expect)(llm_1.PROVIDER_ENV_VARS.anthropic).toBe('ANTHROPIC_API_KEY');
+        (0, vitest_1.expect)(llm_1.PROVIDER_ENV_VARS.gemini).toBe('GEMINI_API_KEY');
+    });
+    (0, vitest_1.it)('has correct default models', () => {
+        (0, vitest_1.expect)(llm_1.DEFAULT_MODELS.openai).toBe('gpt-4o');
+        (0, vitest_1.expect)(llm_1.DEFAULT_MODELS.anthropic).toBe('claude-sonnet-4-20250514');
+        (0, vitest_1.expect)(llm_1.DEFAULT_MODELS.gemini).toBe('gemini-1.5-pro');
+    });
+});

package/dist/commands/search.js

@@ -14,7 +14,8 @@ function registerSearchCommand(program) {
         .argument('[query]', 'Search query (required unless using --popular or --recent)')
         .option('--popular', 'Show top agents/skills by stars')
         .option('--recent', 'Show most recently published')
-        .option('--type <type>', 'Filter by type: agents, skills, all (default: all)', 'all')
+        .option('--mine', 'Show only your own agents (including private)')
+        .option('--type <type>', 'Filter by type: agents, skills, code, prompt, skill, all (default: all)', 'all')
         .option('--limit <n>', `Max results (default: ${DEFAULT_LIMIT})`, String(DEFAULT_LIMIT))
         .option('--free', 'Show only free agents')
         .option('--paid', 'Show only paid agents')
@@ -23,29 +24,40 @@ function registerSearchCommand(program) {
 Pricing Filters:
   --free    Show only free agents
   --paid    Show only paid agents
+
+Ownership Filters:
+  --mine    Show your own agents (public and private). Requires login.
 `)
         .action(async (query, options) => {
         const config = await (0, config_1.getResolvedConfig)();
         const limit = parseInt(options.limit, 10) || DEFAULT_LIMIT;
-        // Default to popular when no args
-        if (!query && !options.popular && !options.recent) {
-            options.popular = true;
-        }
+        // Map type filter for API (null means no filter)
+        const typeFilter = options.type === 'all' ? undefined : options.type;
+        const sort = options.popular ? 'stars' : options.recent ? 'recent' : undefined;
         let agents;
-        if (query) {
-            agents = await (0, api_1.searchAgents)(config, query);
-            await (0, analytics_1.track)('cli_search', { query, type: options.type });
+        if (options.mine) {
+            // --mine: search within user's own agents (public + private)
+            if (!config.apiKey) {
+                process.stderr.write('Error: --mine requires authentication. Run `orchagent login` first.\n');
+                process.exitCode = 1;
+                return;
+            }
+            agents = await (0, api_1.searchMyAgents)(config, query, { sort, type: typeFilter });
+            await (0, analytics_1.track)('cli_search', { query, type: options.type, mine: true });
         }
         else {
-            agents = await (0, api_1.listPublicAgents)(config);
-            await (0, analytics_1.track)('cli_search', { mode: options.popular ? 'popular' : 'recent', type: options.type });
-        }
-        // Filter by type
-        if (options.type === 'agents') {
-            agents = agents.filter(a => a.type !== 'skill');
-        }
-        else if (options.type === 'skills') {
-            agents = agents.filter(a => a.type === 'skill');
+            // Default to popular when no args
+            if (!query && !options.popular && !options.recent) {
+                options.popular = true;
+            }
+            if (query) {
+                agents = await (0, api_1.searchAgents)(config, query, { sort, type: typeFilter });
+                await (0, analytics_1.track)('cli_search', { query, type: options.type });
+            }
+            else {
+                agents = await (0, api_1.listPublicAgents)(config, { sort, type: typeFilter });
+                await (0, analytics_1.track)('cli_search', { mode: options.popular ? 'popular' : 'recent', type: options.type });
+            }
         }
         // Filter by pricing if requested
         if (options.free) {
@@ -55,7 +67,7 @@ Pricing Filters:
             agents = agents.filter(a => (0, pricing_1.isPaidAgent)(a));
         }
         // Sort results
-        if (options.popular || (!query && !options.recent)) {
+        if (options.popular || (!query && !options.recent && !options.mine)) {
             agents.sort((a, b) => (b.stars_count ?? 0) - (a.stars_count ?? 0));
         }
         else if (options.recent) {
@@ -68,14 +80,27 @@ Pricing Filters:
             return;
         }
         if (agents.length === 0) {
-            process.stdout.write(query ? 'No results found matching your search.\n' : 'No public agents found.\n');
-            process.stdout.write('\nBrowse all agents at: https://orchagent.io/explore\n');
+            if (options.mine) {
+                process.stdout.write(query
+                    ? `No agents found matching "${query}" in your account.\n`
+                    : 'You have no published agents yet.\n');
+                process.stdout.write('\nTip: Run "orchagent init" to create your first agent.\n');
+            }
+            else {
+                process.stdout.write(query ? 'No results found matching your search.\n' : 'No public agents found.\n');
+                process.stdout.write('\nBrowse all agents at: https://orchagent.io/explore\n');
+            }
             return;
         }
-        (0, output_1.printAgentsTable)(agents);
+        (0, output_1.printAgentsTable)(agents, { showVisibility: options.mine });
         if (agents.length === limit) {
             process.stdout.write(`\nShowing top ${limit} results. Use --limit <n> for more.\n`);
         }
-        process.stdout.write('\nTip: Run "orchagent info <agent>" to see input schema and details.\n');
+        if (options.mine) {
+            process.stdout.write('\nTip: Run "orchagent info <agent>" to see details, or "orchagent delete <agent>" to remove.\n');
+        }
+        else {
+            process.stdout.write('\nTip: Run "orchagent info <agent>" to see input schema and details.\n');
+        }
     });
 }

package/dist/commands/skill.js

@@ -269,7 +269,7 @@ Instructions and guidance for AI agents...
         .command('install <skill>')
         .description('Install skill to local AI tool directories (Claude Code, Cursor, etc.)')
         .option('--global', 'Install to home directory (default: current directory)')
-        .option('--scope <scope>', 'Install scope: user or project', 'project')
+        .option('--scope <scope>', 'Install scope: user or project')
         .option('--dry-run', 'Show what would be installed without making changes')
         .option('--format <formats>', 'Comma-separated format IDs (e.g., claude-code,cursor)')
         .option('--all-formats', 'Install to all supported AI tool directories')
@@ -401,8 +401,8 @@ Instructions and guidance for AI agents...
                 'The skill exists but has an empty prompt. This may be a publishing issue.\n' +
                 'Try re-publishing the skill or contact the skill author.');
         }
-        // Determine scope (--global is legacy alias for --scope user)
-        const scope = options.global ? 'user' : (options.scope || 'project');
+        // Determine scope (--global is legacy alias for --scope user; then config default; then 'project')
+        const scope = options.global ? 'user' : (options.scope || await (0, config_1.getDefaultScope)() || 'project');
         result.scope = scope;
         // Build skill content with header
         const skillContent = `# ${skillData.name}
@@ -495,7 +495,10 @@ ${skillData.prompt}
             for (const tool of installed) {
                 log(`  - ${tool}\n`);
             }
-            log(`\nLocation: ${scope === 'user' ? '~/.claude/skills/ (and other AI tool dirs)' : './<tool>/skills/'}\n`);
+            log(`\nLocations:\n`);
+            for (const file of result.files) {
+                log(`  - ${file.tool}: ${file.path}\n`);
+            }
         }
     });
     // orch skill uninstall <skill>
@@ -503,7 +506,7 @@ ${skillData.prompt}
         .command('uninstall <skill>')
         .description('Uninstall skill from local AI tool directories')
         .option('--global', 'Uninstall from home directory (default: current directory)')
-        .option('--scope <scope>', 'Uninstall scope: user or project', 'project')
+        .option('--scope <scope>', 'Uninstall scope: user or project')
         .option('--json', 'Output result as JSON')
         .action(async (skillRef, options) => {
         const jsonMode = options.json === true;
@@ -530,8 +533,8 @@ ${skillData.prompt}
             throw new errors_1.CliError(errMsg);
         }
         result.skill = `${org}/${parsed.skill}`;
-        // Determine scope
-        const scope = options.global ? 'user' : (options.scope || 'project');
+        // Determine scope (--global is legacy alias for --scope user; then config default; then 'project')
+        const scope = options.global ? 'user' : (options.scope || await (0, config_1.getDefaultScope)() || 'project');
         result.scope = scope;
         // Remove from all AI tool directories
         for (const formatId of config_1.VALID_FORMAT_IDS) {