@myclaw163/clawclaw-cli 0.6.70 → 0.6.74

package/src/lib/strategy-export.ts

@@ -1,242 +1,242 @@
-// src/lib/strategy-export.ts
-// Export official strategies to workspace for customization.
-import { existsSync, readFileSync, writeFileSync, mkdirSync, renameSync, mkdtempSync, rmSync } from 'fs';
-import { join, dirname } from 'path';
-import { tmpdir } from 'os';
-import { fileURLToPath } from 'url';
-import { createHash } from 'crypto';
-import { getWorkspaceDir } from './init-command.js';
-import { importUserFile } from '../strategies/loader.js';
-
-// ── Constants ──────────────────────────────────────────────────────
-const OFFICIAL_DIR = '.official';
-const MANIFEST_FILE = 'manifest.json';
-
-// ── Types ──────────────────────────────────────────────────────────
-export interface ManifestEntry {
-  sourceHash: string;
-  sidecarHash?: string;
-}
-
-export type Manifest = Record<string, ManifestEntry>;
-
-// ── Helpers ────────────────────────────────────────────────────────
-function strategiesDir(): string {
-  return join(getWorkspaceDir(), 'strategies');
-}
-
-function officialDir(): string {
-  return join(strategiesDir(), OFFICIAL_DIR);
-}
-
-function manifestPath(): string {
-  return join(officialDir(), MANIFEST_FILE);
-}
-
-export function readManifest(): Manifest {
-  const path = manifestPath();
-  if (!existsSync(path)) return {};
-  try {
-    return JSON.parse(readFileSync(path, 'utf8')) as Manifest;
-  } catch {
-    return {};
-  }
-}
-
-function writeManifest(manifest: Manifest): void {
-  mkdirSync(officialDir(), { recursive: true });
-  writeFileSync(manifestPath(), JSON.stringify(manifest, null, 2) + '\n', 'utf8');
-}
-
-function locateBuiltinSource(id: string): string | null {
-  const currentDir = dirname(fileURLToPath(import.meta.url));
-  const strategiesDir = join(currentDir, '..', 'strategies');
-  const filePath = join(strategiesDir, `${id}.ts`);
-  return existsSync(filePath) ? filePath : null;
-}
-
-function hashContent(content: string): string {
-  return createHash('sha256').update(content).digest('hex').slice(0, 16);
-}
-
-export function rewriteBuiltinImports(source: string, id: string, version: string): string {
-  // Rewrite ALL relative imports (./... and ../...) to '@myclaw163/clawclaw-cli'.
-  // The SDK must export every symbol used; validateExportable catches gaps.
-  const rewritten = source.replace(/from\s+['"]\.\.?\/[^'"]+['"]/g, `from '@myclaw163/clawclaw-cli'`);
-  const provenance = `// Exported from ${id}@${version} — customize freely\n`;
-  return provenance + rewritten;
-}
-
-async function validateExportable(rewritten: string): Promise<{ ok: boolean; error?: string }> {
-  try {
-    const tmpDir = mkdtempSync(join(tmpdir(), 'ccl-export-validate-'));
-    const tmpFile = join(tmpDir, 'strategy.ts');
-    try {
-      writeFileSync(tmpFile, rewritten, 'utf8');
-      const mod = await importUserFile(tmpFile);
-      const entry = mod.strategy as any;
-      if (!entry || typeof entry !== 'object') {
-        return { ok: false, error: 'Missing or invalid strategy export' };
-      }
-      if (typeof entry.id !== 'string') {
-        return { ok: false, error: 'strategy.id must be a string' };
-      }
-      if (typeof entry.description !== 'string') {
-        return { ok: false, error: 'strategy.description must be a string' };
-      }
-      if (typeof entry.create !== 'function') {
-        return { ok: false, error: 'strategy.create must be a function' };
-      }
-      // Try calling create() to catch runtime errors.
-      // Some strategies require args (move-room, social-task, etc.) — those throw
-      // "requires X" errors, which are expected and not validation failures.
-      try {
-        entry.create();
-      } catch (err: any) {
-        const msg = err?.message ?? String(err);
-        if (msg.includes('requires') || msg.includes('argument') || msg.includes('param')) {
-          // Expected: strategy needs args, but the import resolution worked.
-        } else {
-          return { ok: false, error: `create() failed: ${msg}` };
-        }
-      }
-      // Static check: parse imports from 'clawclaw-cli' and verify each symbol
-      // is actually exported by the SDK. esbuild/tsx don't validate named exports
-      // at link time (missing symbols resolve to undefined, not errors).
-      const sdkExports = await getSdkExports();
-      const importPattern = /import\s+(?:type\s+)?\{([^}]+)\}\s+from\s+['"](?:@myclaw163\/)?clawclaw-cli['"]/g;
-      let match;
-      while ((match = importPattern.exec(rewritten)) !== null) {
-        const names = match[1].split(',').map(s => s.trim().replace(/^type\s+/, '').replace(/\s+as\s+.*$/, '').trim()).filter(Boolean);
-        for (const name of names) {
-          if (!sdkExports.has(name)) {
-            return { ok: false, error: `'${name}' is not exported by @myclaw163/clawclaw-cli SDK` };
-          }
-        }
-      }
-      return { ok: true };
-    } finally {
-      rmSync(tmpDir, { recursive: true, force: true });
-    }
-  } catch (err: any) {
-    return { ok: false, error: err?.message ?? String(err) };
-  }
-}
-
-// Cache SDK exports to avoid re-parsing on every validation.
-let sdkExportsCache: Set<string> | null = null;
-
-async function getSdkExports(): Promise<Set<string>> {
-  if (sdkExportsCache) return sdkExportsCache;
-  const sdkPath = join(dirname(fileURLToPath(import.meta.url)), '..', 'sdk', 'index.ts');
-  const source = readFileSync(sdkPath, 'utf8');
-  const exports = new Set<string>();
-  // Match: export { X, Y, Z } from '...'
-  const exportPattern = /export\s+(?:type\s+)?\{([^}]+)\}/g;
-  let match;
-  while ((match = exportPattern.exec(source)) !== null) {
-    const names = match[1].split(',').map(s => s.trim().replace(/^type\s+/, '').replace(/\s+as\s+.*$/, '').trim()).filter(Boolean);
-    for (const name of names) {
-      exports.add(name);
-    }
-  }
-  sdkExportsCache = exports;
-  return exports;
-}
-
-function writeAtomic(path: string, content: string): void {
-  const tmpPath = path + '.tmp';
-  writeFileSync(tmpPath, content, 'utf8');
-  renameSync(tmpPath, path);
-}
-
-function readPackageVersion(): string {
-  try {
-    const currentDir = dirname(fileURLToPath(import.meta.url));
-    const pkgDir = join(currentDir, '..', '..');
-    const pkgPath = join(pkgDir, 'package.json');
-    const pkg = JSON.parse(readFileSync(pkgPath, 'utf8'));
-    return pkg.version ?? '0.0.0';
-  } catch {
-    return '0.0.0';
-  }
-}
-
-// ── Main API ───────────────────────────────────────────────────────
-export async function exportStrategy(id: string, { force = false } = {}): Promise<void> {
-  const sourcePath = locateBuiltinSource(id);
-  if (!sourcePath) {
-    throw new Error(`Built-in strategy '${id}' not found.`);
-  }
-
-  const strategyFile = join(strategiesDir(), `${id}.ts`);
-  const manifest = readManifest();
-
-  if (existsSync(strategyFile) && !manifest[id] && !force) {
-    throw new Error(`${strategyFile} already exists and is not tracked. Use --force to overwrite.`);
-  }
-
-  const source = readFileSync(sourcePath, 'utf8');
-  const version = readPackageVersion();
-  const rewritten = rewriteBuiltinImports(source, id, version);
-
-  const validation = await validateExportable(rewritten);
-  if (!validation.ok) {
-    throw new Error(`Validation failed: ${validation.error}`);
-  }
-
-  // Compute content hashes for staleness detection.
-  const sourceHash = hashContent(source);
-  const sidecarSource = sourcePath.replace(/\.ts$/, '.knowledge.md');
-  const sidecarHash = existsSync(sidecarSource) ? hashContent(readFileSync(sidecarSource, 'utf8')) : undefined;
-
-  // Write metadata first, then strategy file last.
-  mkdirSync(officialDir(), { recursive: true });
-
-  const entry: ManifestEntry = { sourceHash };
-  if (sidecarHash !== undefined) entry.sidecarHash = sidecarHash;
-  manifest[id] = entry;
-  writeManifest(manifest);
-
-  mkdirSync(strategiesDir(), { recursive: true });
-  writeAtomic(strategyFile, rewritten);
-
-  // Copy the knowledge sidecar (<id>.knowledge.md) so the exported strategy
-  // keeps its contract — otherwise `ccl strategy --info` returns null for it.
-  if (sidecarHash !== undefined) {
-    writeAtomic(join(strategiesDir(), `${id}.knowledge.md`), readFileSync(sidecarSource, 'utf8'));
-  }
-}
-
-export function detectStaleOfficialStrategies(): string[] {
-  const manifest = readManifest();
-  const staleIds: string[] = [];
-
-  for (const [id, entry] of Object.entries(manifest)) {
-    const sourcePath = locateBuiltinSource(id);
-    if (!sourcePath) {
-      // Built-in strategy was removed — mark as stale so user can clean up.
-      staleIds.push(id);
-      continue;
-    }
-
-    const currentSourceHash = hashContent(readFileSync(sourcePath, 'utf8'));
-    if (currentSourceHash !== entry.sourceHash) {
-      staleIds.push(id);
-      continue;
-    }
-
-    const sidecarSource = sourcePath.replace(/\.ts$/, '.knowledge.md');
-    if (existsSync(sidecarSource)) {
-      const currentSidecarHash = hashContent(readFileSync(sidecarSource, 'utf8'));
-      if (currentSidecarHash !== entry.sidecarHash) {
-        staleIds.push(id);
-      }
-    } else if (entry.sidecarHash !== undefined) {
-      // Sidecar was removed in the new version.
-      staleIds.push(id);
-    }
-  }
-
-  return staleIds;
-}
+// src/lib/strategy-export.ts
+// Export official strategies to workspace for customization.
+import { existsSync, readFileSync, writeFileSync, mkdirSync, renameSync, mkdtempSync, rmSync } from 'fs';
+import { join, dirname } from 'path';
+import { tmpdir } from 'os';
+import { fileURLToPath } from 'url';
+import { createHash } from 'crypto';
+import { getWorkspaceDir } from './init-command.js';
+import { importUserFile } from '../strategies/loader.js';
+
+// ── Constants ──────────────────────────────────────────────────────
+const OFFICIAL_DIR = '.official';
+const MANIFEST_FILE = 'manifest.json';
+
+// ── Types ──────────────────────────────────────────────────────────
+export interface ManifestEntry {
+  sourceHash: string;
+  sidecarHash?: string;
+}
+
+export type Manifest = Record<string, ManifestEntry>;
+
+// ── Helpers ────────────────────────────────────────────────────────
+function strategiesDir(): string {
+  return join(getWorkspaceDir(), 'strategies');
+}
+
+function officialDir(): string {
+  return join(strategiesDir(), OFFICIAL_DIR);
+}
+
+function manifestPath(): string {
+  return join(officialDir(), MANIFEST_FILE);
+}
+
+export function readManifest(): Manifest {
+  const path = manifestPath();
+  if (!existsSync(path)) return {};
+  try {
+    return JSON.parse(readFileSync(path, 'utf8')) as Manifest;
+  } catch {
+    return {};
+  }
+}
+
+function writeManifest(manifest: Manifest): void {
+  mkdirSync(officialDir(), { recursive: true });
+  writeFileSync(manifestPath(), JSON.stringify(manifest, null, 2) + '\n', 'utf8');
+}
+
+function locateBuiltinSource(id: string): string | null {
+  const currentDir = dirname(fileURLToPath(import.meta.url));
+  const strategiesDir = join(currentDir, '..', 'strategies');
+  const filePath = join(strategiesDir, `${id}.ts`);
+  return existsSync(filePath) ? filePath : null;
+}
+
+function hashContent(content: string): string {
+  return createHash('sha256').update(content).digest('hex').slice(0, 16);
+}
+
+export function rewriteBuiltinImports(source: string, id: string, version: string): string {
+  // Rewrite ALL relative imports (./... and ../...) to '@myclaw163/clawclaw-cli'.
+  // The SDK must export every symbol used; validateExportable catches gaps.
+  const rewritten = source.replace(/from\s+['"]\.\.?\/[^'"]+['"]/g, `from '@myclaw163/clawclaw-cli'`);
+  const provenance = `// Exported from ${id}@${version} — customize freely\n`;
+  return provenance + rewritten;
+}
+
+async function validateExportable(rewritten: string): Promise<{ ok: boolean; error?: string }> {
+  try {
+    const tmpDir = mkdtempSync(join(tmpdir(), 'ccl-export-validate-'));
+    const tmpFile = join(tmpDir, 'strategy.ts');
+    try {
+      writeFileSync(tmpFile, rewritten, 'utf8');
+      const mod = await importUserFile(tmpFile);
+      const entry = mod.strategy as any;
+      if (!entry || typeof entry !== 'object') {
+        return { ok: false, error: 'Missing or invalid strategy export' };
+      }
+      if (typeof entry.id !== 'string') {
+        return { ok: false, error: 'strategy.id must be a string' };
+      }
+      if (typeof entry.description !== 'string') {
+        return { ok: false, error: 'strategy.description must be a string' };
+      }
+      if (typeof entry.create !== 'function') {
+        return { ok: false, error: 'strategy.create must be a function' };
+      }
+      // Try calling create() to catch runtime errors.
+      // Some strategies require args (move-room, social-task, etc.) — those throw
+      // "requires X" errors, which are expected and not validation failures.
+      try {
+        entry.create();
+      } catch (err: any) {
+        const msg = err?.message ?? String(err);
+        if (msg.includes('requires') || msg.includes('argument') || msg.includes('param')) {
+          // Expected: strategy needs args, but the import resolution worked.
+        } else {
+          return { ok: false, error: `create() failed: ${msg}` };
+        }
+      }
+      // Static check: parse imports from 'clawclaw-cli' and verify each symbol
+      // is actually exported by the SDK. esbuild/tsx don't validate named exports
+      // at link time (missing symbols resolve to undefined, not errors).
+      const sdkExports = await getSdkExports();
+      const importPattern = /import\s+(?:type\s+)?\{([^}]+)\}\s+from\s+['"](?:@myclaw163\/)?clawclaw-cli['"]/g;
+      let match;
+      while ((match = importPattern.exec(rewritten)) !== null) {
+        const names = match[1].split(',').map(s => s.trim().replace(/^type\s+/, '').replace(/\s+as\s+.*$/, '').trim()).filter(Boolean);
+        for (const name of names) {
+          if (!sdkExports.has(name)) {
+            return { ok: false, error: `'${name}' is not exported by @myclaw163/clawclaw-cli SDK` };
+          }
+        }
+      }
+      return { ok: true };
+    } finally {
+      rmSync(tmpDir, { recursive: true, force: true });
+    }
+  } catch (err: any) {
+    return { ok: false, error: err?.message ?? String(err) };
+  }
+}
+
+// Cache SDK exports to avoid re-parsing on every validation.
+let sdkExportsCache: Set<string> | null = null;
+
+async function getSdkExports(): Promise<Set<string>> {
+  if (sdkExportsCache) return sdkExportsCache;
+  const sdkPath = join(dirname(fileURLToPath(import.meta.url)), '..', 'sdk', 'index.ts');
+  const source = readFileSync(sdkPath, 'utf8');
+  const exports = new Set<string>();
+  // Match: export { X, Y, Z } from '...'
+  const exportPattern = /export\s+(?:type\s+)?\{([^}]+)\}/g;
+  let match;
+  while ((match = exportPattern.exec(source)) !== null) {
+    const names = match[1].split(',').map(s => s.trim().replace(/^type\s+/, '').replace(/\s+as\s+.*$/, '').trim()).filter(Boolean);
+    for (const name of names) {
+      exports.add(name);
+    }
+  }
+  sdkExportsCache = exports;
+  return exports;
+}
+
+function writeAtomic(path: string, content: string): void {
+  const tmpPath = path + '.tmp';
+  writeFileSync(tmpPath, content, 'utf8');
+  renameSync(tmpPath, path);
+}
+
+function readPackageVersion(): string {
+  try {
+    const currentDir = dirname(fileURLToPath(import.meta.url));
+    const pkgDir = join(currentDir, '..', '..');
+    const pkgPath = join(pkgDir, 'package.json');
+    const pkg = JSON.parse(readFileSync(pkgPath, 'utf8'));
+    return pkg.version ?? '0.0.0';
+  } catch {
+    return '0.0.0';
+  }
+}
+
+// ── Main API ───────────────────────────────────────────────────────
+export async function exportStrategy(id: string, { force = false } = {}): Promise<void> {
+  const sourcePath = locateBuiltinSource(id);
+  if (!sourcePath) {
+    throw new Error(`Built-in strategy '${id}' not found.`);
+  }
+
+  const strategyFile = join(strategiesDir(), `${id}.ts`);
+  const manifest = readManifest();
+
+  if (existsSync(strategyFile) && !manifest[id] && !force) {
+    throw new Error(`${strategyFile} already exists and is not tracked. Use --force to overwrite.`);
+  }
+
+  const source = readFileSync(sourcePath, 'utf8');
+  const version = readPackageVersion();
+  const rewritten = rewriteBuiltinImports(source, id, version);
+
+  const validation = await validateExportable(rewritten);
+  if (!validation.ok) {
+    throw new Error(`Validation failed: ${validation.error}`);
+  }
+
+  // Compute content hashes for staleness detection.
+  const sourceHash = hashContent(source);
+  const sidecarSource = sourcePath.replace(/\.ts$/, '.knowledge.md');
+  const sidecarHash = existsSync(sidecarSource) ? hashContent(readFileSync(sidecarSource, 'utf8')) : undefined;
+
+  // Write metadata first, then strategy file last.
+  mkdirSync(officialDir(), { recursive: true });
+
+  const entry: ManifestEntry = { sourceHash };
+  if (sidecarHash !== undefined) entry.sidecarHash = sidecarHash;
+  manifest[id] = entry;
+  writeManifest(manifest);
+
+  mkdirSync(strategiesDir(), { recursive: true });
+  writeAtomic(strategyFile, rewritten);
+
+  // Copy the knowledge sidecar (<id>.knowledge.md) so the exported strategy
+  // keeps its contract — otherwise `ccl strategy --info` returns null for it.
+  if (sidecarHash !== undefined) {
+    writeAtomic(join(strategiesDir(), `${id}.knowledge.md`), readFileSync(sidecarSource, 'utf8'));
+  }
+}
+
+export function detectStaleOfficialStrategies(): string[] {
+  const manifest = readManifest();
+  const staleIds: string[] = [];
+
+  for (const [id, entry] of Object.entries(manifest)) {
+    const sourcePath = locateBuiltinSource(id);
+    if (!sourcePath) {
+      // Built-in strategy was removed — mark as stale so user can clean up.
+      staleIds.push(id);
+      continue;
+    }
+
+    const currentSourceHash = hashContent(readFileSync(sourcePath, 'utf8'));
+    if (currentSourceHash !== entry.sourceHash) {
+      staleIds.push(id);
+      continue;
+    }
+
+    const sidecarSource = sourcePath.replace(/\.ts$/, '.knowledge.md');
+    if (existsSync(sidecarSource)) {
+      const currentSidecarHash = hashContent(readFileSync(sidecarSource, 'utf8'));
+      if (currentSidecarHash !== entry.sidecarHash) {
+        staleIds.push(id);
+      }
+    } else if (entry.sidecarHash !== undefined) {
+      // Sidecar was removed in the new version.
+      staleIds.push(id);
+    }
+  }
+
+  return staleIds;
+}

package/src/lib/tts-keys.ts

@@ -1,7 +1,7 @@
-/** NetEase Leihuo TTS provider id (stored under each account's `tts.keys.leihuo`). */
-export const TTS_PROVIDER_LEIHUO = 'leihuo';
-
-export const DEFAULT_TTS_VOICE = 'female-shaonv';
-
-/** NetEase TTS input limit for CLI-side synthesis. */
-export const TTS_TEXT_MAX_LENGTH = 100;
+/** NetEase Leihuo TTS provider id (stored under each account's `tts.keys.leihuo`). */
+export const TTS_PROVIDER_LEIHUO = 'leihuo';
+
+export const DEFAULT_TTS_VOICE = 'female-shaonv';
+
+/** NetEase TTS input limit for CLI-side synthesis. */
+export const TTS_TEXT_MAX_LENGTH = 100;

package/src/lib/tts-speech.test.ts

@@ -1,63 +1,63 @@
-import { describe, expect, it, vi, beforeEach, afterEach } from 'vitest';
-import { mkdtempSync, rmSync } from 'fs';
-import { tmpdir } from 'os';
-import { join } from 'path';
-import { AuthStore } from './auth.js';
-import { TTS_PROVIDER_LEIHUO } from './tts-keys.js';
-import { synthesizeNeteaseTTS } from './netease-tts.js';
-import { maybeSynthesizeSpeechAudioUrl } from './tts-speech.js';
-
-vi.mock('./netease-tts.js', () => ({
-  synthesizeNeteaseTTS: vi.fn(async () => ({
-    audio: Buffer.from('audio'),
-    contentType: 'audio/mpeg',
-  })),
-}));
-
-describe('maybeSynthesizeSpeechAudioUrl', () => {
-  let dir: string;
-  let authFile: string;
-
-  beforeEach(() => {
-    dir = mkdtempSync(join(tmpdir(), 'clawclaw-tts-'));
-    authFile = join(dir, '.auth.json');
-  });
-
-  afterEach(() => {
-    rmSync(dir, { recursive: true, force: true });
-    vi.clearAllMocks();
-  });
-
-  it('returns existing url without calling TTS', async () => {
-    const client = { uploadAudio: vi.fn() } as any;
-    const url = await maybeSynthesizeSpeechAudioUrl('hi', 'https://example.com/a.mp3', client);
-    expect(url).toBe('https://example.com/a.mp3');
-    expect(client.uploadAudio).not.toHaveBeenCalled();
-  });
-
-  it('returns undefined when no TTS key is configured', async () => {
-    const store = new AuthStore(authFile);
-    store.addProfile({ agentName: 'lobster-1', apiKey: 'claw_1', serverUrl: 'https://example.com' });
-    const client = { uploadAudio: vi.fn() } as any;
-    const url = await maybeSynthesizeSpeechAudioUrl('hi', undefined, client, { authStore: store });
-    expect(url).toBeUndefined();
-  });
-
-  it('synthesizes and uploads when leihuo key is configured', async () => {
-    const store = new AuthStore(authFile);
-    store.addProfile({ agentName: 'lobster-1', apiKey: 'claw_1', serverUrl: 'https://example.com' });
-    store.setTtsKey(TTS_PROVIDER_LEIHUO, 'sk-test');
-    store.setTtsDefaultVoice('male-qn-qingse');
-    const client = {
-      uploadAudio: vi.fn(async () => ({ audio_url: 'https://cdn.example.com/a.mp3' })),
-    } as any;
-    const url = await maybeSynthesizeSpeechAudioUrl('你好', undefined, client, { authStore: store });
-    expect(url).toBe('https://cdn.example.com/a.mp3');
-    expect(client.uploadAudio).toHaveBeenCalledOnce();
-    expect(synthesizeNeteaseTTS).toHaveBeenCalledWith(expect.objectContaining({
-      apiKey: 'sk-test',
-      text: '你好',
-      voice: 'male-qn-qingse',
-    }));
-  });
-});
+import { describe, expect, it, vi, beforeEach, afterEach } from 'vitest';
+import { mkdtempSync, rmSync } from 'fs';
+import { tmpdir } from 'os';
+import { join } from 'path';
+import { AuthStore } from './auth.js';
+import { TTS_PROVIDER_LEIHUO } from './tts-keys.js';
+import { synthesizeNeteaseTTS } from './netease-tts.js';
+import { maybeSynthesizeSpeechAudioUrl } from './tts-speech.js';
+
+vi.mock('./netease-tts.js', () => ({
+  synthesizeNeteaseTTS: vi.fn(async () => ({
+    audio: Buffer.from('audio'),
+    contentType: 'audio/mpeg',
+  })),
+}));
+
+describe('maybeSynthesizeSpeechAudioUrl', () => {
+  let dir: string;
+  let authFile: string;
+
+  beforeEach(() => {
+    dir = mkdtempSync(join(tmpdir(), 'clawclaw-tts-'));
+    authFile = join(dir, '.auth.json');
+  });
+
+  afterEach(() => {
+    rmSync(dir, { recursive: true, force: true });
+    vi.clearAllMocks();
+  });
+
+  it('returns existing url without calling TTS', async () => {
+    const client = { uploadAudio: vi.fn() } as any;
+    const url = await maybeSynthesizeSpeechAudioUrl('hi', 'https://example.com/a.mp3', client);
+    expect(url).toBe('https://example.com/a.mp3');
+    expect(client.uploadAudio).not.toHaveBeenCalled();
+  });
+
+  it('returns undefined when no TTS key is configured', async () => {
+    const store = new AuthStore(authFile);
+    store.addProfile({ agentName: 'lobster-1', apiKey: 'claw_1', serverUrl: 'https://example.com' });
+    const client = { uploadAudio: vi.fn() } as any;
+    const url = await maybeSynthesizeSpeechAudioUrl('hi', undefined, client, { authStore: store });
+    expect(url).toBeUndefined();
+  });
+
+  it('synthesizes and uploads when leihuo key is configured', async () => {
+    const store = new AuthStore(authFile);
+    store.addProfile({ agentName: 'lobster-1', apiKey: 'claw_1', serverUrl: 'https://example.com' });
+    store.setTtsKey(TTS_PROVIDER_LEIHUO, 'sk-test');
+    store.setTtsDefaultVoice('male-qn-qingse');
+    const client = {
+      uploadAudio: vi.fn(async () => ({ audio_url: 'https://cdn.example.com/a.mp3' })),
+    } as any;
+    const url = await maybeSynthesizeSpeechAudioUrl('你好', undefined, client, { authStore: store });
+    expect(url).toBe('https://cdn.example.com/a.mp3');
+    expect(client.uploadAudio).toHaveBeenCalledOnce();
+    expect(synthesizeNeteaseTTS).toHaveBeenCalledWith(expect.objectContaining({
+      apiKey: 'sk-test',
+      text: '你好',
+      voice: 'male-qn-qingse',
+    }));
+  });
+});