@myclaw163/clawclaw-cli 0.6.70 → 0.6.74

package/src/commands/tts.ts

@@ -1,128 +1,128 @@
-import { Command } from 'commander';
-import { AuthStore } from '../lib/auth.js';
-import { GameClient } from '../lib/game-client.js';
-import { setMeta } from '../lib/command-meta.js';
-import { DEFAULT_TTS_VOICE, TTS_PROVIDER_LEIHUO } from '../lib/tts-keys.js';
-import { synthesizeAndUploadSpeechAudio } from '../lib/tts-speech.js';
-
-const AVAILABLE_VOICES = [
-  { id: "male-qn-qingse", language: "中文 (普通话)", description: "青涩青年音色" },
-  { id: "male-qn-jingying", language: "中文 (普通话)", description: "精英青年音色" },
-  { id: "male-qn-badao", language: "中文 (普通话)", description: "霸道青年音色" },
-  { id: "male-qn-daxuesheng", language: "中文 (普通话)", description: "青年大学生音色" },
-  { id: "female-shaonv", language: "中文 (普通话)", description: "少女音色" },
-  { id: "female-yujie", language: "中文 (普通话)", description: "御姐音色" },
-  { id: "female-chengshu", language: "中文 (普通话)", description: "成熟女性音色" },
-  { id: "female-tianmei", language: "中文 (普通话)", description: "甜美女性音色" },
-  { id: "male-qn-qingse-jingpin", language: "中文 (普通话)", description: "青涩青年音色-beta" },
-  { id: "male-qn-jingying-jingpin", language: "中文 (普通话)", description: "精英青年音色-beta" },
-  { id: "male-qn-badao-jingpin", language: "中文 (普通话)", description: "霸道青年音色-beta" },
-  { id: "male-qn-daxuesheng-jingpin", language: "中文 (普通话)", description: "青年大学生音色-beta" },
-  { id: "female-shaonv-jingpin", language: "中文 (普通话)", description: "少女音色-beta" },
-  { id: "female-yujie-jingpin", language: "中文 (普通话)", description: "御姐音色-beta" },
-  { id: "female-chengshu-jingpin", language: "中文 (普通话)", description: "成熟女性音色-beta" },
-  { id: "female-tianmei-jingpin", language: "中文 (普通话)", description: "甜美女性音色-beta" },
-  { id: "clever_boy", language: "中文 (普通话)", description: "聪明男童" },
-  { id: "cute_boy", language: "中文 (普通话)", description: "可爱男童" },
-  { id: "lovely_girl", language: "中文 (普通话)", description: "萌萌女童" },
-  { id: "cartoon_pig", language: "中文 (普通话)", description: "卡通猪小琪" },
-  { id: "bingjiao_didi", language: "中文 (普通话)", description: "病娇弟弟" },
-  { id: "junlang_nanyou", language: "中文 (普通话)", description: "俊朗男友" },
-  { id: "chunzhen_xuedi", language: "中文 (普通话)", description: "纯真学弟" },
-  { id: "lengdan_xiongzhang", language: "中文 (普通话)", description: "冷淡学长" },
-  { id: "badao_shaoye", language: "中文 (普通话)", description: "霸道少爷" },
-  { id: "tianxin_xiaoling", language: "中文 (普通话)", description: "甜心小玲" },
-  { id: "qiaopi_mengmei", language: "中文 (普通话)", description: "俏皮萌妹" },
-  { id: "wumei_yujie", language: "中文 (普通话)", description: "妩媚御姐" },
-  { id: "diadia_xuemei", language: "中文 (普通话)", description: "嗲嗲学妹" },
-  { id: "danya_xuejie", language: "中文 (普通话)", description: "淡雅学姐" },
-  { id: "Chinese (Mandarin)_Reliable_Executive", language: "中文 (普通话)", description: "沉稳高管" },
-  { id: "Chinese (Mandarin)_News_Anchor", language: "中文 (普通话)", description: "新闻女声" },
-  { id: "Chinese (Mandarin)_Mature_Woman", language: "中文 (普通话)", description: "傲娇御姐" },
-  { id: "Chinese (Mandarin)_Unrestrained_Young_Man", language: "中文 (普通话)", description: "不羁青年" },
-  { id: "Arrogant_Miss", language: "中文 (普通话)", description: "嚣张小姐" },
-  { id: "Robot_Armor", language: "中文 (普通话)", description: "机械战甲" },
-  { id: "Chinese (Mandarin)_Kind-hearted_Antie", language: "中文 (普通话)", description: "热心大婶" },
-  { id: "Chinese (Mandarin)_HK_Flight_Attendant", language: "中文 (普通话)", description: "港普空姐" },
-  { id: "Chinese (Mandarin)_Humorous_Elder", language: "中文 (普通话)", description: "搞笑大爷" },
-  { id: "Chinese (Mandarin)_Gentleman", language: "中文 (普通话)", description: "温润男声" },
-  { id: "Chinese (Mandarin)_Warm_Bestie", language: "中文 (普通话)", description: "温暖闺蜜" },
-  { id: "Chinese (Mandarin)_Male_Announcer", language: "中文 (普通话)", description: "播报男声" },
-  { id: "Chinese (Mandarin)_Sweet_Lady", language: "中文 (普通话)", description: "甜美女声" },
-  { id: "Chinese (Mandarin)_Southern_Young_Man", language: "中文 (普通话)", description: "南方小哥" },
-  { id: "Chinese (Mandarin)_Wise_Women", language: "中文 (普通话)", description: "阅历姐姐" },
-  { id: "Chinese (Mandarin)_Gentle_Youth", language: "中文 (普通话)", description: "温润青年" },
-  { id: "Chinese (Mandarin)_Warm_Girl", language: "中文 (普通话)", description: "温暖少女" },
-  { id: "Chinese (Mandarin)_Kind-hearted_Elder", language: "中文 (普通话)", description: "花甲奶奶" },
-  { id: "Chinese (Mandarin)_Cute_Spirit", language: "中文 (普通话)", description: "憨憨萌兽" },
-  { id: "Chinese (Mandarin)_Radio_Host", language: "中文 (普通话)", description: "电台男主播" },
-  { id: "Chinese (Mandarin)_Lyrical_Voice", language: "中文 (普通话)", description: "抒情男声" },
-  { id: "Chinese (Mandarin)_Straightforward_Boy", language: "中文 (普通话)", description: "率真弟弟" },
-  { id: "Chinese (Mandarin)_Sincere_Adult", language: "中文 (普通话)", description: "真诚青年" },
-  { id: "Chinese (Mandarin)_Gentle_Senior", language: "中文 (普通话)", description: "温柔学姐" },
-  { id: "Chinese (Mandarin)_Stubborn_Friend", language: "中文 (普通话)", description: "嘴硬竹马" },
-  { id: "Chinese (Mandarin)_Crisp_Girl", language: "中文 (普通话)", description: "清脆少女" },
-  { id: "Chinese (Mandarin)_Pure-hearted_Boy", language: "中文 (普通话)", description: "清澈邻家弟弟" },
-  { id: "Chinese (Mandarin)_Soft_Girl", language: "中文 (普通话)", description: "柔和少女" },
-  { id: "Cantonese_ProfessionalHost（F)", language: "中文 (粤语)", description: "专业女主持" },
-  { id: "Cantonese_GentleLady", language: "中文 (粤语)", description: "温柔女声" },
-  { id: "Cantonese_ProfessionalHost（M)", language: "中文 (粤语)", description: "专业男主持" },
-  { id: "Cantonese_PlayfulMan", language: "中文 (粤语)", description: "活泼男声" },
-  { id: "Cantonese_CuteGirl", language: "中文 (粤语)", description: "可爱女孩" },
-  { id: "Cantonese_KindWoman", language: "中文 (粤语)", description: "善良女声" }
-];
-
-export function createTTSCommand(): Command {
-  const tts = new Command('tts');
-  tts.description('NetEase TTS configuration and requests');
-
-  tts
-    .command('config [apiKey]')
-    .description('Configure active account NetEase TTS API key and default voice')
-    .option('-v, --voice <voice>', 'Default voice ID for this account')
-    .action((apiKey: string | undefined, opts: { voice?: string }) => {
-      if (!apiKey && !opts.voice) {
-        throw new Error('Provide an API key, a default voice with --voice, or both.');
-      }
-      const authStore = new AuthStore();
-      const profile = authStore.getActive();
-      if (!profile) throw new Error('No active account. Run `clawclaw-cli account register` or `clawclaw-cli account switch <apiKey>` first.');
-      if (apiKey) authStore.setTtsKey(TTS_PROVIDER_LEIHUO, apiKey);
-      if (opts.voice) authStore.setTtsDefaultVoice(opts.voice);
-      const config = authStore.getTtsConfig();
-      console.log(JSON.stringify({
-        message: 'TTS configured successfully for active account',
-        account: profile.agentName,
-        provider: TTS_PROVIDER_LEIHUO,
-        apiKeyConfigured: Boolean(config?.keys?.[TTS_PROVIDER_LEIHUO]),
-        defaultVoice: config?.defaultVoice ?? DEFAULT_TTS_VOICE,
-      }, null, 2));
-    });
-
-  tts
-    .command('list')
-    .description('List all available voices')
-    .action(() => {
-      console.log(JSON.stringify(AVAILABLE_VOICES, null, 2));
-    });
-
-  tts
-    .command('request <text>')
-    .description('Request local NetEase TTS synthesis and upload audio to OSS')
-    .option('-v, --voice <voice>', 'Voice ID; defaults to the active account default voice')
-    .action(async (text: string, opts: { voice?: string }) => {
-      const client = GameClient.fromAuth();
-      const authStore = new AuthStore();
-      const voice = opts.voice ?? authStore.getTtsDefaultVoice() ?? DEFAULT_TTS_VOICE;
-      const audioUrl = await synthesizeAndUploadSpeechAudio(text, client, { voice, authStore });
-      console.log(JSON.stringify({
-        audio_url: audioUrl,
-        voice,
-      }, null, 2));
-    });
-
-  // ── Adapter metadata ──
-  setMeta(tts.commands.find((c) => c.name() === 'config')!, { requiresConfirm: true, sensitive: true });
-
-  return tts;
-}
+import { Command } from 'commander';
+import { AuthStore } from '../lib/auth.js';
+import { GameClient } from '../lib/game-client.js';
+import { setMeta } from '../lib/command-meta.js';
+import { DEFAULT_TTS_VOICE, TTS_PROVIDER_LEIHUO } from '../lib/tts-keys.js';
+import { synthesizeAndUploadSpeechAudio } from '../lib/tts-speech.js';
+
+const AVAILABLE_VOICES = [
+  { id: "male-qn-qingse", language: "中文 (普通话)", description: "青涩青年音色" },
+  { id: "male-qn-jingying", language: "中文 (普通话)", description: "精英青年音色" },
+  { id: "male-qn-badao", language: "中文 (普通话)", description: "霸道青年音色" },
+  { id: "male-qn-daxuesheng", language: "中文 (普通话)", description: "青年大学生音色" },
+  { id: "female-shaonv", language: "中文 (普通话)", description: "少女音色" },
+  { id: "female-yujie", language: "中文 (普通话)", description: "御姐音色" },
+  { id: "female-chengshu", language: "中文 (普通话)", description: "成熟女性音色" },
+  { id: "female-tianmei", language: "中文 (普通话)", description: "甜美女性音色" },
+  { id: "male-qn-qingse-jingpin", language: "中文 (普通话)", description: "青涩青年音色-beta" },
+  { id: "male-qn-jingying-jingpin", language: "中文 (普通话)", description: "精英青年音色-beta" },
+  { id: "male-qn-badao-jingpin", language: "中文 (普通话)", description: "霸道青年音色-beta" },
+  { id: "male-qn-daxuesheng-jingpin", language: "中文 (普通话)", description: "青年大学生音色-beta" },
+  { id: "female-shaonv-jingpin", language: "中文 (普通话)", description: "少女音色-beta" },
+  { id: "female-yujie-jingpin", language: "中文 (普通话)", description: "御姐音色-beta" },
+  { id: "female-chengshu-jingpin", language: "中文 (普通话)", description: "成熟女性音色-beta" },
+  { id: "female-tianmei-jingpin", language: "中文 (普通话)", description: "甜美女性音色-beta" },
+  { id: "clever_boy", language: "中文 (普通话)", description: "聪明男童" },
+  { id: "cute_boy", language: "中文 (普通话)", description: "可爱男童" },
+  { id: "lovely_girl", language: "中文 (普通话)", description: "萌萌女童" },
+  { id: "cartoon_pig", language: "中文 (普通话)", description: "卡通猪小琪" },
+  { id: "bingjiao_didi", language: "中文 (普通话)", description: "病娇弟弟" },
+  { id: "junlang_nanyou", language: "中文 (普通话)", description: "俊朗男友" },
+  { id: "chunzhen_xuedi", language: "中文 (普通话)", description: "纯真学弟" },
+  { id: "lengdan_xiongzhang", language: "中文 (普通话)", description: "冷淡学长" },
+  { id: "badao_shaoye", language: "中文 (普通话)", description: "霸道少爷" },
+  { id: "tianxin_xiaoling", language: "中文 (普通话)", description: "甜心小玲" },
+  { id: "qiaopi_mengmei", language: "中文 (普通话)", description: "俏皮萌妹" },
+  { id: "wumei_yujie", language: "中文 (普通话)", description: "妩媚御姐" },
+  { id: "diadia_xuemei", language: "中文 (普通话)", description: "嗲嗲学妹" },
+  { id: "danya_xuejie", language: "中文 (普通话)", description: "淡雅学姐" },
+  { id: "Chinese (Mandarin)_Reliable_Executive", language: "中文 (普通话)", description: "沉稳高管" },
+  { id: "Chinese (Mandarin)_News_Anchor", language: "中文 (普通话)", description: "新闻女声" },
+  { id: "Chinese (Mandarin)_Mature_Woman", language: "中文 (普通话)", description: "傲娇御姐" },
+  { id: "Chinese (Mandarin)_Unrestrained_Young_Man", language: "中文 (普通话)", description: "不羁青年" },
+  { id: "Arrogant_Miss", language: "中文 (普通话)", description: "嚣张小姐" },
+  { id: "Robot_Armor", language: "中文 (普通话)", description: "机械战甲" },
+  { id: "Chinese (Mandarin)_Kind-hearted_Antie", language: "中文 (普通话)", description: "热心大婶" },
+  { id: "Chinese (Mandarin)_HK_Flight_Attendant", language: "中文 (普通话)", description: "港普空姐" },
+  { id: "Chinese (Mandarin)_Humorous_Elder", language: "中文 (普通话)", description: "搞笑大爷" },
+  { id: "Chinese (Mandarin)_Gentleman", language: "中文 (普通话)", description: "温润男声" },
+  { id: "Chinese (Mandarin)_Warm_Bestie", language: "中文 (普通话)", description: "温暖闺蜜" },
+  { id: "Chinese (Mandarin)_Male_Announcer", language: "中文 (普通话)", description: "播报男声" },
+  { id: "Chinese (Mandarin)_Sweet_Lady", language: "中文 (普通话)", description: "甜美女声" },
+  { id: "Chinese (Mandarin)_Southern_Young_Man", language: "中文 (普通话)", description: "南方小哥" },
+  { id: "Chinese (Mandarin)_Wise_Women", language: "中文 (普通话)", description: "阅历姐姐" },
+  { id: "Chinese (Mandarin)_Gentle_Youth", language: "中文 (普通话)", description: "温润青年" },
+  { id: "Chinese (Mandarin)_Warm_Girl", language: "中文 (普通话)", description: "温暖少女" },
+  { id: "Chinese (Mandarin)_Kind-hearted_Elder", language: "中文 (普通话)", description: "花甲奶奶" },
+  { id: "Chinese (Mandarin)_Cute_Spirit", language: "中文 (普通话)", description: "憨憨萌兽" },
+  { id: "Chinese (Mandarin)_Radio_Host", language: "中文 (普通话)", description: "电台男主播" },
+  { id: "Chinese (Mandarin)_Lyrical_Voice", language: "中文 (普通话)", description: "抒情男声" },
+  { id: "Chinese (Mandarin)_Straightforward_Boy", language: "中文 (普通话)", description: "率真弟弟" },
+  { id: "Chinese (Mandarin)_Sincere_Adult", language: "中文 (普通话)", description: "真诚青年" },
+  { id: "Chinese (Mandarin)_Gentle_Senior", language: "中文 (普通话)", description: "温柔学姐" },
+  { id: "Chinese (Mandarin)_Stubborn_Friend", language: "中文 (普通话)", description: "嘴硬竹马" },
+  { id: "Chinese (Mandarin)_Crisp_Girl", language: "中文 (普通话)", description: "清脆少女" },
+  { id: "Chinese (Mandarin)_Pure-hearted_Boy", language: "中文 (普通话)", description: "清澈邻家弟弟" },
+  { id: "Chinese (Mandarin)_Soft_Girl", language: "中文 (普通话)", description: "柔和少女" },
+  { id: "Cantonese_ProfessionalHost（F)", language: "中文 (粤语)", description: "专业女主持" },
+  { id: "Cantonese_GentleLady", language: "中文 (粤语)", description: "温柔女声" },
+  { id: "Cantonese_ProfessionalHost（M)", language: "中文 (粤语)", description: "专业男主持" },
+  { id: "Cantonese_PlayfulMan", language: "中文 (粤语)", description: "活泼男声" },
+  { id: "Cantonese_CuteGirl", language: "中文 (粤语)", description: "可爱女孩" },
+  { id: "Cantonese_KindWoman", language: "中文 (粤语)", description: "善良女声" }
+];
+
+export function createTTSCommand(): Command {
+  const tts = new Command('tts');
+  tts.description('NetEase TTS configuration and requests');
+
+  tts
+    .command('config [apiKey]')
+    .description('Configure active account NetEase TTS API key and default voice')
+    .option('-v, --voice <voice>', 'Default voice ID for this account')
+    .action((apiKey: string | undefined, opts: { voice?: string }) => {
+      if (!apiKey && !opts.voice) {
+        throw new Error('Provide an API key, a default voice with --voice, or both.');
+      }
+      const authStore = new AuthStore();
+      const profile = authStore.getActive();
+      if (!profile) throw new Error('No active account. Run `clawclaw-cli account register` or `clawclaw-cli account switch <apiKey>` first.');
+      if (apiKey) authStore.setTtsKey(TTS_PROVIDER_LEIHUO, apiKey);
+      if (opts.voice) authStore.setTtsDefaultVoice(opts.voice);
+      const config = authStore.getTtsConfig();
+      console.log(JSON.stringify({
+        message: 'TTS configured successfully for active account',
+        account: profile.agentName,
+        provider: TTS_PROVIDER_LEIHUO,
+        apiKeyConfigured: Boolean(config?.keys?.[TTS_PROVIDER_LEIHUO]),
+        defaultVoice: config?.defaultVoice ?? DEFAULT_TTS_VOICE,
+      }, null, 2));
+    });
+
+  tts
+    .command('list')
+    .description('List all available voices')
+    .action(() => {
+      console.log(JSON.stringify(AVAILABLE_VOICES, null, 2));
+    });
+
+  tts
+    .command('request <text>')
+    .description('Request local NetEase TTS synthesis and upload audio to OSS')
+    .option('-v, --voice <voice>', 'Voice ID; defaults to the active account default voice')
+    .action(async (text: string, opts: { voice?: string }) => {
+      const client = GameClient.fromAuth();
+      const authStore = new AuthStore();
+      const voice = opts.voice ?? authStore.getTtsDefaultVoice() ?? DEFAULT_TTS_VOICE;
+      const audioUrl = await synthesizeAndUploadSpeechAudio(text, client, { voice, authStore });
+      console.log(JSON.stringify({
+        audio_url: audioUrl,
+        voice,
+      }, null, 2));
+    });
+
+  // ── Adapter metadata ──
+  setMeta(tts.commands.find((c) => c.name() === 'config')!, { requiresConfirm: true, sensitive: true });
+
+  return tts;
+}

package/src/commands/upgrade.test.ts

@@ -1,82 +1,82 @@
-import { mkdirSync, mkdtempSync, writeFileSync } from 'fs';
-import { tmpdir } from 'os';
-import { join, resolve } from 'path';
-import { describe, expect, it } from 'vitest';
-import { installLatestFromNpm, runUpgrade, syncSkillFromInstalledPackage } from './upgrade.js';
-
-describe('runUpgrade', () => {
-  it('installs and returns the installed cli version without syncing skill again', () => {
-    let installed = false;
-    const result = runUpgrade({
-      installLatest: () => { installed = true; },
-      readInstalledCliVersion: () => '0.4.7',
-    });
-    expect(installed).toBe(true);
-    expect(result).toEqual({ version: '0.4.7' });
-  });
-
-  it('reports error when npm install fails', () => {
-    const result = runUpgrade({
-      installLatest: () => { throw new Error('EACCES'); },
-      readInstalledCliVersion: () => '0.3.4',
-    });
-    expect(result.version).toBeUndefined();
-    expect(result.error).toMatch(/npm install failed/);
-  });
-
-  it('installs latest from npm without registry flag', () => {
-    const calls: Array<{ file: string; args: string[]; options: Record<string, unknown> }> = [];
-    installLatestFromNpm('clawclaw-cli', (file, args, options) => {
-      calls.push({ file, args, options });
-    });
-
-    expect(calls).toHaveLength(1);
-    if (process.platform === 'win32') {
-      expect(calls[0].file).toBe('cmd.exe');
-      expect(calls[0].args).toEqual([
-        '/d',
-        '/c',
-        'npm.cmd install -g clawclaw-cli@latest --foreground-scripts --loglevel=error',
-      ]);
-    } else {
-      expect(calls[0].args).toEqual([
-        'install',
-        '-g',
-        'clawclaw-cli@latest',
-        '--foreground-scripts',
-        '--loglevel=error',
-      ]);
-    }
-    expect(calls[0].options.stdio).toEqual(['ignore', 'ignore', 'inherit']);
-    expect(calls[0].options.shell).toBe(false);
-  });
-
-  it('runs bundled skill sync script for explicit skill update', () => {
-    const root = mkdtempSync(join(tmpdir(), 'upgrade-skill-'));
-    const installDir = join(root, 'clawclaw-cli');
-    const scriptPath = join(installDir, 'scripts', 'sync-bundled-skill.mjs');
-    mkdirSync(join(installDir, 'scripts'), { recursive: true });
-    writeFileSync(scriptPath, '');
-
-    const calls: Array<{ file: string; args: string[]; options: Record<string, unknown> }> = [];
-    syncSkillFromInstalledPackage('clawclaw-cli', (file, args, options) => {
-      calls.push({ file, args, options });
-      if (args[0] === 'root' || args.at(-1)?.includes('root -g')) return root;
-      return '';
-    });
-
-    expect(calls).toHaveLength(2);
-    expect(calls[1].file).toBe(process.execPath);
-    expect(calls[1].args).toEqual([resolve(scriptPath)]);
-    expect(calls[1].options.stdio).toBe('inherit');
-  });
-
-  it('reports error when reading installed version fails', () => {
-    const result = runUpgrade({
-      installLatest: () => {},
-      readInstalledCliVersion: () => { throw new Error('missing package'); },
-    });
-    expect(result.version).toBeUndefined();
-    expect(result.error).toMatch(/read installed cli version failed/);
-  });
-});
+import { mkdirSync, mkdtempSync, writeFileSync } from 'fs';
+import { tmpdir } from 'os';
+import { join, resolve } from 'path';
+import { describe, expect, it } from 'vitest';
+import { installLatestFromNpm, runUpgrade, syncSkillFromInstalledPackage } from './upgrade.js';
+
+describe('runUpgrade', () => {
+  it('installs and returns the installed cli version without syncing skill again', () => {
+    let installed = false;
+    const result = runUpgrade({
+      installLatest: () => { installed = true; },
+      readInstalledCliVersion: () => '0.4.7',
+    });
+    expect(installed).toBe(true);
+    expect(result).toEqual({ version: '0.4.7' });
+  });
+
+  it('reports error when npm install fails', () => {
+    const result = runUpgrade({
+      installLatest: () => { throw new Error('EACCES'); },
+      readInstalledCliVersion: () => '0.3.4',
+    });
+    expect(result.version).toBeUndefined();
+    expect(result.error).toMatch(/npm install failed/);
+  });
+
+  it('installs latest from npm without registry flag', () => {
+    const calls: Array<{ file: string; args: string[]; options: Record<string, unknown> }> = [];
+    installLatestFromNpm('clawclaw-cli', (file, args, options) => {
+      calls.push({ file, args, options });
+    });
+
+    expect(calls).toHaveLength(1);
+    if (process.platform === 'win32') {
+      expect(calls[0].file).toBe('cmd.exe');
+      expect(calls[0].args).toEqual([
+        '/d',
+        '/c',
+        'npm.cmd install -g clawclaw-cli@latest --foreground-scripts --loglevel=error',
+      ]);
+    } else {
+      expect(calls[0].args).toEqual([
+        'install',
+        '-g',
+        'clawclaw-cli@latest',
+        '--foreground-scripts',
+        '--loglevel=error',
+      ]);
+    }
+    expect(calls[0].options.stdio).toEqual(['ignore', 'ignore', 'inherit']);
+    expect(calls[0].options.shell).toBe(false);
+  });
+
+  it('runs bundled skill sync script for explicit skill update', () => {
+    const root = mkdtempSync(join(tmpdir(), 'upgrade-skill-'));
+    const installDir = join(root, 'clawclaw-cli');
+    const scriptPath = join(installDir, 'scripts', 'sync-bundled-skill.mjs');
+    mkdirSync(join(installDir, 'scripts'), { recursive: true });
+    writeFileSync(scriptPath, '');
+
+    const calls: Array<{ file: string; args: string[]; options: Record<string, unknown> }> = [];
+    syncSkillFromInstalledPackage('clawclaw-cli', (file, args, options) => {
+      calls.push({ file, args, options });
+      if (args[0] === 'root' || args.at(-1)?.includes('root -g')) return root;
+      return '';
+    });
+
+    expect(calls).toHaveLength(2);
+    expect(calls[1].file).toBe(process.execPath);
+    expect(calls[1].args).toEqual([resolve(scriptPath)]);
+    expect(calls[1].options.stdio).toBe('inherit');
+  });
+
+  it('reports error when reading installed version fails', () => {
+    const result = runUpgrade({
+      installLatest: () => {},
+      readInstalledCliVersion: () => { throw new Error('missing package'); },
+    });
+    expect(result.version).toBeUndefined();
+    expect(result.error).toMatch(/read installed cli version failed/);
+  });
+});