@myclaw163/clawclaw-cli 0.6.56 → 0.6.57

package/scripts/sync-bundled-skill.test.mjs

@@ -1,152 +1,152 @@
-import { existsSync, mkdirSync, mkdtempSync, readFileSync, writeFileSync } from 'fs';
-import { tmpdir } from 'os';
-import { join } from 'path';
-import { describe, expect, it, vi } from 'vitest';
-import { backupInstalledSkillTargets, syncBundledSkill } from './sync-bundled-skill.mjs';
-
-function writeSkill(dir, version, body) {
-  mkdirSync(dir, { recursive: true });
-  writeFileSync(join(dir, 'SKILL.md'), `---\nname: clawclaw\nversion: ${version}\n---\n${body}\n`);
-}
-
-function writeInstallDir(root) {
-  const installDir = join(root, 'pkg');
-  writeSkill(join(installDir, 'skills', 'clawclaw'), '4.8.18', 'new');
-  mkdirSync(join(installDir, 'node_modules', 'skills'), { recursive: true });
-  writeFileSync(join(installDir, 'node_modules', 'skills', 'package.json'), JSON.stringify({
-    bin: { skills: 'bin/cli.mjs' },
-  }));
-  mkdirSync(join(installDir, 'node_modules', 'skills', 'bin'), { recursive: true });
-  writeFileSync(join(installDir, 'node_modules', 'skills', 'bin', 'cli.mjs'), '');
-  return installDir;
-}
-
-describe('sync-bundled-skill', () => {
-  it('backs up only unique skill contents', () => {
-    const root = mkdtempSync(join(tmpdir(), 'skill-backup-'));
-    const one = join(root, 'home', '.claude', 'skills', 'clawclaw');
-    const two = join(root, 'home', '.codex', 'skills', 'clawclaw');
-    const three = join(root, 'home', '.openclaw', 'skills', 'clawclaw');
-    const newSkill = join(root, 'pkg', 'skills', 'clawclaw');
-    writeSkill(one, '4.8.17', 'same');
-    writeSkill(two, '4.8.17', 'same');
-    writeSkill(three, '4.8.16', 'different');
-    writeSkill(newSkill, '4.8.18', 'new');
-
-    const result = backupInstalledSkillTargets({
-      sourceDir: newSkill,
-      backupRoot: join(root, 'backups'),
-      candidateDirs: [one, two, three],
-      now: new Date('2026-06-11T08:09:10Z'),
-    });
-
-    expect(result.backupDir).toBe(join(root, 'backups', 'clawclaw-20260611-080910Z'));
-    expect(result.copies).toHaveLength(2);
-    expect(result.entries).toHaveLength(3);
-    expect(result.entries[0].copy).toBe(result.entries[1].copy);
-    expect(result.entries[2].copy).not.toBe(result.entries[0].copy);
-    const manifest = JSON.parse(readFileSync(join(result.backupDir, 'manifest.json'), 'utf8'));
-    expect(manifest.newSkillVersion).toBe('4.8.18');
-  });
-
-  it('does not back up installed skills that already match the bundled skill', () => {
-    const root = mkdtempSync(join(tmpdir(), 'skill-backup-same-'));
-    const installed = join(root, 'home', '.codex', 'skills', 'clawclaw');
-    const newSkill = join(root, 'pkg', 'skills', 'clawclaw');
-    writeSkill(installed, '4.8.18', 'new');
-    writeSkill(newSkill, '4.8.18', 'new');
-
-    const result = backupInstalledSkillTargets({
-      sourceDir: newSkill,
-      backupRoot: join(root, 'backups'),
-      candidateDirs: [installed],
-      now: new Date('2026-06-11T08:09:10Z'),
-    });
-
-    expect(result).toBeNull();
-    expect(existsSync(join(root, 'backups'))).toBe(false);
-  });
-
-  it('backs up only installed skills that differ from the bundled skill', () => {
-    const root = mkdtempSync(join(tmpdir(), 'skill-backup-mixed-'));
-    const same = join(root, 'home', '.claude', 'skills', 'clawclaw');
-    const different = join(root, 'home', '.codex', 'skills', 'clawclaw');
-    const newSkill = join(root, 'pkg', 'skills', 'clawclaw');
-    writeSkill(same, '4.8.18', 'new');
-    writeSkill(different, '4.8.17', 'old');
-    writeSkill(newSkill, '4.8.18', 'new');
-
-    const result = backupInstalledSkillTargets({
-      sourceDir: newSkill,
-      backupRoot: join(root, 'backups'),
-      candidateDirs: [same, different],
-      now: new Date('2026-06-11T08:09:10Z'),
-    });
-
-    expect(result.copies).toHaveLength(1);
-    expect(result.entries).toHaveLength(1);
-    expect(result.entries[0].sourcePath).toBe(different);
-  });
-
-  it('runs skills add after a successful backup', () => {
-    const root = mkdtempSync(join(tmpdir(), 'skill-sync-'));
-    const installDir = writeInstallDir(root);
-    const old = join(root, 'home', '.codex', 'skills', 'clawclaw');
-    writeSkill(old, '4.8.17', 'old');
-    const spawn = vi.fn(() => ({ status: 0 }));
-
-    const result = syncBundledSkill({
-      installDir,
-      backupRoot: join(root, 'backups'),
-      candidateDirs: [old],
-      spawnSync: spawn,
-      stdio: 'ignore',
-    });
-
-    expect(result.copyCount).toBe(1);
-    expect(spawn).toHaveBeenCalledOnce();
-    expect(spawn.mock.calls[0][1].slice(1)).toEqual([
-      'add',
-      join(installDir, 'skills', 'clawclaw'),
-      '-y',
-      '-g',
-    ]);
-  });
-
-  it('does not run skills add when backup fails', () => {
-    const root = mkdtempSync(join(tmpdir(), 'skill-backup-fail-'));
-    const installDir = writeInstallDir(root);
-    const old = join(root, 'home', '.codex', 'skills', 'clawclaw');
-    writeSkill(old, '4.8.17', 'old');
-    const backupRootFile = join(root, 'backups-file');
-    writeFileSync(backupRootFile, 'not a directory');
-    const spawn = vi.fn(() => ({ status: 0 }));
-
-    expect(() => syncBundledSkill({
-      installDir,
-      backupRoot: backupRootFile,
-      candidateDirs: [old],
-      spawnSync: spawn,
-      stdio: 'ignore',
-    })).toThrow();
-    expect(spawn).not.toHaveBeenCalled();
-  });
-
-  it('keeps backup when skills add fails', () => {
-    const root = mkdtempSync(join(tmpdir(), 'skill-add-fail-'));
-    const installDir = writeInstallDir(root);
-    const old = join(root, 'home', '.codex', 'skills', 'clawclaw');
-    const backupRoot = join(root, 'backups');
-    writeSkill(old, '4.8.17', 'old');
-    const spawn = vi.fn(() => ({ status: 1 }));
-
-    expect(() => syncBundledSkill({
-      installDir,
-      backupRoot,
-      candidateDirs: [old],
-      spawnSync: spawn,
-      stdio: 'ignore',
-    })).toThrow(/skills add exited/);
-    expect(existsSync(backupRoot)).toBe(true);
-  });
-});
+import { existsSync, mkdirSync, mkdtempSync, readFileSync, writeFileSync } from 'fs';
+import { tmpdir } from 'os';
+import { join } from 'path';
+import { describe, expect, it, vi } from 'vitest';
+import { backupInstalledSkillTargets, syncBundledSkill } from './sync-bundled-skill.mjs';
+
+function writeSkill(dir, version, body) {
+  mkdirSync(dir, { recursive: true });
+  writeFileSync(join(dir, 'SKILL.md'), `---\nname: clawclaw\nversion: ${version}\n---\n${body}\n`);
+}
+
+function writeInstallDir(root) {
+  const installDir = join(root, 'pkg');
+  writeSkill(join(installDir, 'skills', 'clawclaw'), '4.8.18', 'new');
+  mkdirSync(join(installDir, 'node_modules', 'skills'), { recursive: true });
+  writeFileSync(join(installDir, 'node_modules', 'skills', 'package.json'), JSON.stringify({
+    bin: { skills: 'bin/cli.mjs' },
+  }));
+  mkdirSync(join(installDir, 'node_modules', 'skills', 'bin'), { recursive: true });
+  writeFileSync(join(installDir, 'node_modules', 'skills', 'bin', 'cli.mjs'), '');
+  return installDir;
+}
+
+describe('sync-bundled-skill', () => {
+  it('backs up only unique skill contents', () => {
+    const root = mkdtempSync(join(tmpdir(), 'skill-backup-'));
+    const one = join(root, 'home', '.claude', 'skills', 'clawclaw');
+    const two = join(root, 'home', '.codex', 'skills', 'clawclaw');
+    const three = join(root, 'home', '.openclaw', 'skills', 'clawclaw');
+    const newSkill = join(root, 'pkg', 'skills', 'clawclaw');
+    writeSkill(one, '4.8.17', 'same');
+    writeSkill(two, '4.8.17', 'same');
+    writeSkill(three, '4.8.16', 'different');
+    writeSkill(newSkill, '4.8.18', 'new');
+
+    const result = backupInstalledSkillTargets({
+      sourceDir: newSkill,
+      backupRoot: join(root, 'backups'),
+      candidateDirs: [one, two, three],
+      now: new Date('2026-06-11T08:09:10Z'),
+    });
+
+    expect(result.backupDir).toBe(join(root, 'backups', 'clawclaw-20260611-080910Z'));
+    expect(result.copies).toHaveLength(2);
+    expect(result.entries).toHaveLength(3);
+    expect(result.entries[0].copy).toBe(result.entries[1].copy);
+    expect(result.entries[2].copy).not.toBe(result.entries[0].copy);
+    const manifest = JSON.parse(readFileSync(join(result.backupDir, 'manifest.json'), 'utf8'));
+    expect(manifest.newSkillVersion).toBe('4.8.18');
+  });
+
+  it('does not back up installed skills that already match the bundled skill', () => {
+    const root = mkdtempSync(join(tmpdir(), 'skill-backup-same-'));
+    const installed = join(root, 'home', '.codex', 'skills', 'clawclaw');
+    const newSkill = join(root, 'pkg', 'skills', 'clawclaw');
+    writeSkill(installed, '4.8.18', 'new');
+    writeSkill(newSkill, '4.8.18', 'new');
+
+    const result = backupInstalledSkillTargets({
+      sourceDir: newSkill,
+      backupRoot: join(root, 'backups'),
+      candidateDirs: [installed],
+      now: new Date('2026-06-11T08:09:10Z'),
+    });
+
+    expect(result).toBeNull();
+    expect(existsSync(join(root, 'backups'))).toBe(false);
+  });
+
+  it('backs up only installed skills that differ from the bundled skill', () => {
+    const root = mkdtempSync(join(tmpdir(), 'skill-backup-mixed-'));
+    const same = join(root, 'home', '.claude', 'skills', 'clawclaw');
+    const different = join(root, 'home', '.codex', 'skills', 'clawclaw');
+    const newSkill = join(root, 'pkg', 'skills', 'clawclaw');
+    writeSkill(same, '4.8.18', 'new');
+    writeSkill(different, '4.8.17', 'old');
+    writeSkill(newSkill, '4.8.18', 'new');
+
+    const result = backupInstalledSkillTargets({
+      sourceDir: newSkill,
+      backupRoot: join(root, 'backups'),
+      candidateDirs: [same, different],
+      now: new Date('2026-06-11T08:09:10Z'),
+    });
+
+    expect(result.copies).toHaveLength(1);
+    expect(result.entries).toHaveLength(1);
+    expect(result.entries[0].sourcePath).toBe(different);
+  });
+
+  it('runs skills add after a successful backup', () => {
+    const root = mkdtempSync(join(tmpdir(), 'skill-sync-'));
+    const installDir = writeInstallDir(root);
+    const old = join(root, 'home', '.codex', 'skills', 'clawclaw');
+    writeSkill(old, '4.8.17', 'old');
+    const spawn = vi.fn(() => ({ status: 0 }));
+
+    const result = syncBundledSkill({
+      installDir,
+      backupRoot: join(root, 'backups'),
+      candidateDirs: [old],
+      spawnSync: spawn,
+      stdio: 'ignore',
+    });
+
+    expect(result.copyCount).toBe(1);
+    expect(spawn).toHaveBeenCalledOnce();
+    expect(spawn.mock.calls[0][1].slice(1)).toEqual([
+      'add',
+      join(installDir, 'skills', 'clawclaw'),
+      '-y',
+      '-g',
+    ]);
+  });
+
+  it('does not run skills add when backup fails', () => {
+    const root = mkdtempSync(join(tmpdir(), 'skill-backup-fail-'));
+    const installDir = writeInstallDir(root);
+    const old = join(root, 'home', '.codex', 'skills', 'clawclaw');
+    writeSkill(old, '4.8.17', 'old');
+    const backupRootFile = join(root, 'backups-file');
+    writeFileSync(backupRootFile, 'not a directory');
+    const spawn = vi.fn(() => ({ status: 0 }));
+
+    expect(() => syncBundledSkill({
+      installDir,
+      backupRoot: backupRootFile,
+      candidateDirs: [old],
+      spawnSync: spawn,
+      stdio: 'ignore',
+    })).toThrow();
+    expect(spawn).not.toHaveBeenCalled();
+  });
+
+  it('keeps backup when skills add fails', () => {
+    const root = mkdtempSync(join(tmpdir(), 'skill-add-fail-'));
+    const installDir = writeInstallDir(root);
+    const old = join(root, 'home', '.codex', 'skills', 'clawclaw');
+    const backupRoot = join(root, 'backups');
+    writeSkill(old, '4.8.17', 'old');
+    const spawn = vi.fn(() => ({ status: 1 }));
+
+    expect(() => syncBundledSkill({
+      installDir,
+      backupRoot,
+      candidateDirs: [old],
+      spawnSync: spawn,
+      stdio: 'ignore',
+    })).toThrow(/skills add exited/);
+    expect(existsSync(backupRoot)).toBe(true);
+  });
+});