diff --git a/packages/cli/src/commands/skills/disable.test.ts b/packages/cli/src/commands/skills/disable.test.ts index b7bc8805c8..f4ae0d954e 100644 --- a/packages/cli/src/commands/skills/disable.test.ts +++ b/packages/cli/src/commands/skills/disable.test.ts @@ -108,7 +108,7 @@ describe('skills disable command', () => { describe('disableCommand', () => { it('should have correct command and describe', () => { - expect(disableCommand.command).toBe('disable '); + expect(disableCommand.command).toBe('disable [--scope]'); expect(disableCommand.describe).toBe('Disables an agent skill.'); }); }); diff --git a/packages/cli/src/commands/skills/disable.ts b/packages/cli/src/commands/skills/disable.ts index e0b657afbc..fcb69c087d 100644 --- a/packages/cli/src/commands/skills/disable.ts +++ b/packages/cli/src/commands/skills/disable.ts @@ -31,7 +31,7 @@ export async function handleDisable(args: DisableArgs) { } export const disableCommand: CommandModule = { - command: 'disable ', + command: 'disable [--scope]', describe: 'Disables an agent skill.', builder: (yargs) => yargs diff --git a/packages/cli/src/commands/skills/install.test.ts b/packages/cli/src/commands/skills/install.test.ts index d3f36fbac3..e0621c8028 100644 --- a/packages/cli/src/commands/skills/install.test.ts +++ b/packages/cli/src/commands/skills/install.test.ts @@ -17,7 +17,7 @@ vi.mock('@google/gemini-cli-core', () => ({ })); import { debugLogger } from '@google/gemini-cli-core'; -import { handleInstall } from './install.js'; +import { handleInstall, installCommand } from './install.js'; describe('skill install command', () => { beforeEach(() => { @@ -25,6 +25,17 @@ describe('skill install command', () => { vi.spyOn(process, 'exit').mockImplementation(() => undefined as never); }); + describe('installCommand', () => { + it('should have correct command and describe', () => { + expect(installCommand.command).toBe( + 'install [--scope] [--path]', + ); + expect(installCommand.describe).toBe( + 'Installs an agent skill from a git repository URL or a local path.', + ); + }); + }); + it('should call installSkill with correct arguments for user scope', async () => { mockInstallSkill.mockResolvedValue([ { name: 'test-skill', location: '/mock/user/skills/test-skill' }, diff --git a/packages/cli/src/commands/skills/install.ts b/packages/cli/src/commands/skills/install.ts index 9dbc0007bf..bdf8402de7 100644 --- a/packages/cli/src/commands/skills/install.ts +++ b/packages/cli/src/commands/skills/install.ts @@ -46,7 +46,7 @@ export async function handleInstall(args: InstallArgs) { } export const installCommand: CommandModule = { - command: 'install ', + command: 'install [--scope] [--path]', describe: 'Installs an agent skill from a git repository URL or a local path.', builder: (yargs) => diff --git a/packages/cli/src/commands/skills/list.test.ts b/packages/cli/src/commands/skills/list.test.ts index 81230b33a2..e7e25a2736 100644 --- a/packages/cli/src/commands/skills/list.test.ts +++ b/packages/cli/src/commands/skills/list.test.ts @@ -184,7 +184,7 @@ describe('skills list command', () => { const command = listCommand; it('should have correct command and describe', () => { - expect(command.command).toBe('list'); + expect(command.command).toBe('list [--all]'); expect(command.describe).toBe('Lists discovered agent skills.'); }); }); diff --git a/packages/cli/src/commands/skills/list.ts b/packages/cli/src/commands/skills/list.ts index 17f225c510..c262f39b9b 100644 --- a/packages/cli/src/commands/skills/list.ts +++ b/packages/cli/src/commands/skills/list.ts @@ -63,7 +63,7 @@ export async function handleList(args: { all?: boolean }) { } export const listCommand: CommandModule = { - command: 'list', + command: 'list [--all]', describe: 'Lists discovered agent skills.', builder: (yargs) => yargs.option('all', { diff --git a/packages/cli/src/commands/skills/uninstall.test.ts b/packages/cli/src/commands/skills/uninstall.test.ts index d1feaf7838..74f1730590 100644 --- a/packages/cli/src/commands/skills/uninstall.test.ts +++ b/packages/cli/src/commands/skills/uninstall.test.ts @@ -17,7 +17,7 @@ vi.mock('@google/gemini-cli-core', () => ({ })); import { debugLogger } from '@google/gemini-cli-core'; -import { handleUninstall } from './uninstall.js'; +import { handleUninstall, uninstallCommand } from './uninstall.js'; describe('skill uninstall command', () => { beforeEach(() => { @@ -25,6 +25,15 @@ describe('skill uninstall command', () => { vi.spyOn(process, 'exit').mockImplementation(() => undefined as never); }); + describe('uninstallCommand', () => { + it('should have correct command and describe', () => { + expect(uninstallCommand.command).toBe('uninstall [--scope]'); + expect(uninstallCommand.describe).toBe( + 'Uninstalls an agent skill by name.', + ); + }); + }); + it('should call uninstallSkill with correct arguments for user scope', async () => { mockUninstallSkill.mockResolvedValue({ location: '/mock/user/skills/test-skill', diff --git a/packages/cli/src/commands/skills/uninstall.ts b/packages/cli/src/commands/skills/uninstall.ts index 99f9091e3c..1ab0c130b9 100644 --- a/packages/cli/src/commands/skills/uninstall.ts +++ b/packages/cli/src/commands/skills/uninstall.ts @@ -41,7 +41,7 @@ export async function handleUninstall(args: UninstallArgs) { } export const uninstallCommand: CommandModule = { - command: 'uninstall ', + command: 'uninstall [--scope]', describe: 'Uninstalls an agent skill by name.', builder: (yargs) => yargs