ai-engineering-loop 1.0.11 → 1.0.12

package/tests/sync-hosts.test.js

@@ -0,0 +1,119 @@
+const test = require('node:test');
+const assert = require('node:assert');
+const fs = require('fs');
+const os = require('os');
+const path = require('path');
+const { execFileSync } = require('child_process');
+
+const {
+  applyGrokSkillOverlay,
+  planHostSync,
+  applyHostSync,
+  summarizeHostSync
+} = require('../lib/sync-hosts.js');
+
+const ROOT = path.join(__dirname, '..');
+const CLI = path.join(ROOT, 'bin', 'ai-engineering-loop.js');
+
+function tmpHome() {
+  return fs.mkdtempSync(path.join(os.tmpdir(), 'ael-home-'));
+}
+
+test('applyGrokSkillOverlay inserts user-invocable once', () => {
+  const raw = '---\nname: demo\ndescription: x\n---\n\n# Body\n';
+  const once = applyGrokSkillOverlay(raw);
+  assert.match(once, /user-invocable: true/);
+  const twice = applyGrokSkillOverlay(once);
+  assert.strictEqual(twice, once);
+});
+
+test('planHostSync skips everything when no host roots exist', () => {
+  const home = tmpHome();
+  const plan = planHostSync({ packageRoot: ROOT, home });
+  assert.ok(plan.length > 0);
+  assert.ok(plan.every((item) => item.action === 'skip'));
+  const reasons = new Set(plan.map((item) => item.reason));
+  assert.ok(reasons.has('host-missing'));
+});
+
+test('sync upserts Claude AEL files but does not invent DOT skills', () => {
+  const home = tmpHome();
+  fs.mkdirSync(path.join(home, '.claude'));
+  const results = applyHostSync({ packageRoot: ROOT, home });
+  const claude = results.filter((item) => item.id === 'claude');
+  assert.ok(claude.every((item) => item.action === 'copy'));
+  assert.ok(fs.existsSync(path.join(home, '.claude/skills/ai-engineering-loop/SKILL.md')));
+  assert.ok(fs.existsSync(path.join(home, '.claude/agents/devil-advocate.md')));
+  assert.ok(fs.existsSync(path.join(home, '.claude/agents/judge.md')));
+  assert.ok(fs.existsSync(path.join(home, '.claude/commands/ai-engineering-loop.md')));
+  assert.ok(!fs.existsSync(path.join(home, '.claude/skills/dot-dev-workflow/SKILL.md')));
+  assert.ok(!fs.existsSync(path.join(home, '.claude/settings.local.json')));
+});
+
+test('DOT skills update only when already installed', () => {
+  const home = tmpHome();
+  const dest = path.join(home, '.claude/skills/dot-dev-workflow/SKILL.md');
+  fs.mkdirSync(path.dirname(dest), { recursive: true });
+  fs.writeFileSync(dest, 'stale\n');
+  const results = applyHostSync({ packageRoot: ROOT, home });
+  const dotWf = results.find((item) => item.dest === dest);
+  assert.strictEqual(dotWf.action, 'copy');
+  assert.match(fs.readFileSync(dest, 'utf8'), /Not a substitute for ai-engineering-loop/);
+});
+
+test('Grok skill overlay keeps user-invocable', () => {
+  const home = tmpHome();
+  fs.mkdirSync(path.join(home, '.grok'));
+  const dot = path.join(home, '.grok/skills/dot-dev-workflow/SKILL.md');
+  fs.mkdirSync(path.dirname(dot), { recursive: true });
+  fs.writeFileSync(dot, '---\nname: dot-dev-workflow\ndescription: old\n---\n\nold\n');
+  applyHostSync({ packageRoot: ROOT, home });
+  const skill = fs.readFileSync(path.join(home, '.grok/skills/ai-engineering-loop/SKILL.md'), 'utf8');
+  const wf = fs.readFileSync(dot, 'utf8');
+  assert.match(skill, /^user-invocable: true$/m);
+  assert.match(wf, /^user-invocable: true$/m);
+});
+
+test('dry-run does not write; second apply is current', () => {
+  const home = tmpHome();
+  fs.mkdirSync(path.join(home, '.gemini'));
+  const dry = applyHostSync({ packageRoot: ROOT, home, dryRun: true });
+  const gemini = dry.find((item) => item.id === 'gemini');
+  assert.strictEqual(gemini.action, 'copy');
+  assert.ok(gemini.dryRun);
+  assert.ok(!fs.existsSync(path.join(home, '.gemini/config/skills/ai-engineering-loop/SKILL.md')));
+  applyHostSync({ packageRoot: ROOT, home });
+  const again = applyHostSync({ packageRoot: ROOT, home });
+  assert.strictEqual(summarizeHostSync(again.filter((item) => item.id === 'gemini')).current, 1);
+  assert.strictEqual(summarizeHostSync(again.filter((item) => item.id === 'gemini')).copy, 0);
+});
+
+test('symlink destinations are left alone', () => {
+  const home = tmpHome();
+  const skillDir = path.join(home, '.claude/skills/ai-engineering-loop');
+  fs.mkdirSync(skillDir, { recursive: true });
+  const target = path.join(home, 'outside.md');
+  fs.writeFileSync(target, 'keep me\n');
+  fs.symlinkSync(target, path.join(skillDir, 'SKILL.md'));
+  applyHostSync({ packageRoot: ROOT, home });
+  assert.strictEqual(fs.readFileSync(target, 'utf8'), 'keep me\n');
+  assert.ok(fs.lstatSync(path.join(skillDir, 'SKILL.md')).isSymbolicLink());
+});
+
+test('CLI sync-hosts --dry-run respects AEL_HOME', () => {
+  const home = tmpHome();
+  fs.mkdirSync(path.join(home, '.agents'));
+  const out = execFileSync('node', [CLI, 'sync-hosts', '--dry-run'], {
+    cwd: ROOT,
+    encoding: 'utf8',
+    env: { ...process.env, AEL_HOME: home }
+  });
+  assert.match(out, /dry-run/);
+  assert.match(out, /would-copy|copy:/);
+  assert.ok(!fs.existsSync(path.join(home, '.agents/judge.md')));
+});
+
+test('CLI help lists sync-hosts', () => {
+  const out = execFileSync('node', [CLI, '--help'], { encoding: 'utf8' });
+  assert.match(out, /sync-hosts/);
+});