@givolt/i3-composer-agent 2.0.0 → 2.1.1

package/scripts/validate-skills.js

@@ -1,69 +0,0 @@
-#!/usr/bin/env node
-/**
- * validate-skills.js
- *
- * CLI that validates every skill in skills/ against the rules in
- * docs/skill-anatomy.md. The rules themselves live in scripts/lib/skill-lint.js
- * (a single source of truth, importable and unit-testable); this file is a thin
- * wrapper that walks the skills directory, runs the linter, prints the report,
- * and sets the exit code.
- *
- * Exit codes: 0 = all clear, 1 = one or more errors
- */
-
-'use strict';
-
-const fs   = require('fs');
-const path = require('path');
-
-const { lintSkill } = require('./lib/skill-lint');
-
-const SKILLS_DIR = path.resolve(__dirname, '..', 'skills');
-
-// ─── Main ────────────────────────────────────────────────────────────────────
-
-function main() {
-  if (!fs.existsSync(SKILLS_DIR)) {
-    console.error(`ERROR: skills directory not found at ${SKILLS_DIR}`);
-    process.exit(1);
-  }
-
-  const skillDirs = fs.readdirSync(SKILLS_DIR)
-    .filter(d => fs.statSync(path.join(SKILLS_DIR, d)).isDirectory())
-    .sort();
-
-  const knownSkills = new Set(skillDirs);
-
-  let totalErrors   = 0;
-  let totalWarnings = 0;
-
-  for (const dirName of skillDirs) {
-    const { errors, warnings, exempt } = lintSkill(dirName, SKILLS_DIR, knownSkills);
-    totalErrors   += errors.length;
-    totalWarnings += warnings.length;
-
-    if (errors.length === 0 && warnings.length === 0) {
-      const tag = exempt ? ' (section checks exempt)' : '';
-      console.log(`  ✓  ${dirName}${tag}`);
-    } else {
-      const icon = errors.length > 0 ? '  ✗ ' : '  ⚠ ';
-      console.log(`${icon} ${dirName}`);
-      for (const msg of errors)   console.log(`       ERROR: ${msg}`);
-      for (const msg of warnings) console.log(`       WARN:  ${msg}`);
-    }
-  }
-
-  const status = totalErrors > 0 ? 'FAILED' : totalWarnings > 0 ? 'PASSED WITH WARNINGS' : 'PASSED';
-  console.log(`\n${skillDirs.length} skills checked — ${totalErrors} error(s), ${totalWarnings} warning(s) — ${status}`);
-
-  if (totalErrors > 0) process.exit(1);
-}
-
-// Surface unexpected failures (fs errors, bad symlinks, …) as a structured
-// one-line CI error instead of an uncaught stack trace.
-try {
-  main();
-} catch (err) {
-  console.error(`\nERROR: validate-skills failed unexpectedly: ${err.message}`);
-  process.exit(1);
-}

package/scripts/validate-versions-test.js

@@ -1,32 +0,0 @@
-"use strict";
-
-const assert = require("node:assert/strict");
-const { readFileSync } = require("node:fs");
-const test = require("node:test");
-
-const manifestPaths = [
-  "package.json",
-  "plugin.json",
-  ".codex-plugin/plugin.json",
-  ".claude-plugin/plugin.json",
-  ".claude-plugin/marketplace.json",
-  ".agents/plugins/marketplace.json",
-];
-
-function readManifestVersion(manifestPath) {
-  const manifest = JSON.parse(readFileSync(manifestPath, "utf8"));
-  return manifest.version ?? manifest.plugins?.[0]?.version;
-}
-
-test("all plugin manifests use the root plugin.json version", () => {
-  const expectedVersion = readManifestVersion("plugin.json");
-  assert.ok(expectedVersion, "plugin.json must define a version");
-
-  for (const manifestPath of manifestPaths) {
-    assert.equal(
-      readManifestVersion(manifestPath),
-      expectedVersion,
-      `${manifestPath} must use version ${expectedVersion}`,
-    );
-  }
-});

package/scripts/validate-versions.js

@@ -1,35 +0,0 @@
-#!/usr/bin/env node
-
-"use strict";
-
-const { readFileSync } = require("node:fs");
-
-const manifestPaths = [
-  "package.json",
-  "plugin.json",
-  ".codex-plugin/plugin.json",
-  ".claude-plugin/plugin.json",
-  ".claude-plugin/marketplace.json",
-  ".agents/plugins/marketplace.json",
-];
-
-function readManifestVersion(manifestPath) {
-  const manifest = JSON.parse(readFileSync(manifestPath, "utf8"));
-  return manifest.version ?? manifest.plugins?.[0]?.version;
-}
-
-const expectedVersion = readManifestVersion("plugin.json");
-if (!expectedVersion) {
-  throw new Error("plugin.json is missing a version field");
-}
-
-for (const manifestPath of manifestPaths) {
-  const version = readManifestVersion(manifestPath);
-  if (version !== expectedVersion) {
-    throw new Error(
-      `${manifestPath} has version ${version ?? "<missing>"}; expected ${expectedVersion}`,
-    );
-  }
-}
-
-console.log(`All plugin manifests use version ${expectedVersion}.`);

package/tests/bundle.test.cjs

@@ -1,85 +0,0 @@
-'use strict';
-
-const test = require('node:test');
-const assert = require('node:assert/strict');
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-const { spawnSync } = require('node:child_process');
-const root = path.resolve(__dirname, '..');
-const upstream = require('../rebrand/upstream.json');
-const cli = path.join(root, 'bin/i3-composer-agent.cjs');
-function run(args) { return spawnSync(process.execPath, [cli, ...args], { encoding: 'utf8' }); }
-function workspace(t) {
-  const directory = fs.mkdtempSync(path.join(os.tmpdir(), 'i3-agent-bundle-'));
-  t.after(() => fs.rmSync(directory, { recursive: true, force: true }));
-  return directory;
-}
-test('help and version identify the package without preparing files', () => {
-  const help = run(['--help']).stdout;
-  assert.match(help, /@givolt\/i3-composer-agent/);
-  assert.ok(help.includes(`i3 Composer Agent v${require('../package.json').version}`));
-  assert.ok(help.includes(`Based on agent-skills v${upstream.upstreamVersion}`));
-  assert.equal(run(['--version']).stdout.trim(), require('../package.json').version);
-});
-test('dry run and invalid arguments leave the target untouched', t => {
-  const target = path.join(workspace(t), 'plugin');
-  assert.equal(run(['--codex', '--dir', target, '--dry-run']).status, 0);
-  assert.equal(fs.existsSync(target), false);
-  assert.equal(run(['--codebuddy', '--codex', '--dir', target]).status, 1);
-  assert.equal(run(['--extract', '--dir']).status, 1);
-  assert.equal(fs.existsSync(target), false);
-});
-test('complete bundle preserves reference links and all host assets at a path with spaces', t => {
-  const target = path.join(workspace(t), 'plugin with spaces');
-  const result = run(['--codebuddy', '--dir', target]);
-  assert.equal(result.status, 0, result.stderr);
-  assert.ok(result.stdout.includes(`i3 Composer Agent v${require('../package.json').version}`));
-  assert.ok(result.stdout.includes(`Based on agent-skills v${upstream.upstreamVersion}`));
-  assert.match(result.stdout, /\/plugin install i3-composer-agent@i3-composer-agent/);
-  assert.equal(fs.readdirSync(path.join(target, 'skills')).length, upstream.skills.length);
-  assert.equal(fs.readdirSync(path.join(target, '.claude/commands')).length, upstream.commands.length);
-  assert.equal(fs.readdirSync(path.join(target, 'agents')).filter(name => name.endsWith('.md')).length, upstream.personas.length);
-  assert.equal(JSON.parse(fs.readFileSync(path.join(target, '.codex-plugin/plugin.json'), 'utf8')).name, 'i3-composer-agent');
-  const check = spawnSync(process.execPath, [path.join(target, 'scripts/validate-reference-links.js')], { cwd: target, encoding: 'utf8' });
-  assert.equal(check.status, 0, check.stdout + check.stderr);
-  assert.equal(fs.existsSync(path.join(target, 'AGENTS.md')), false);
-  assert.equal(fs.existsSync(path.join(target, 'CLAUDE.md')), false);
-});
-test('existing unrelated files cannot be replaced even with force', t => {
-  const target = path.join(workspace(t), 'existing');
-  fs.mkdirSync(target);
-  fs.writeFileSync(path.join(target, 'plugin.json'), '{"name":"another-plugin"}\n');
-  fs.writeFileSync(path.join(target, 'keep.txt'), 'business data');
-  assert.equal(run(['--extract', '--dir', target]).status, 1);
-  assert.equal(run(['--extract', '--dir', target, '--force']).status, 1);
-  assert.equal(fs.readFileSync(path.join(target, 'keep.txt'), 'utf8'), 'business data');
-});
-test('installed packages beneath node_modules remain fully extractable', t => {
-  const parent = workspace(t);
-  const installed = path.join(parent, 'node_modules', '@givolt', 'i3-composer-agent');
-  assert.equal(run(['--extract', '--dir', installed]).status, 0);
-  const target = path.join(parent, 'plugin');
-  const result = spawnSync(process.execPath, [path.join(installed, 'bin/i3-composer-agent.cjs'), '--extract', '--dir', target], { encoding: 'utf8' });
-  assert.equal(result.status, 0, result.stderr);
-  assert.equal(JSON.parse(fs.readFileSync(path.join(target, 'package.json'), 'utf8')).name, '@givolt/i3-composer-agent');
-  assert.equal(fs.readdirSync(path.join(target, 'skills')).length, upstream.skills.length);
-});
-test('replacement preserves the complete previous bundle in a backup', t => {
-  const parent = workspace(t);
-  const target = path.join(parent, 'plugin');
-  assert.equal(run(['--extract', '--dir', target]).status, 0);
-  fs.writeFileSync(path.join(target, 'custom-note.txt'), 'preserve me');
-  const result = run(['--extract', '--dir', target, '--force']);
-  assert.equal(result.status, 0, result.stderr);
-  const backups = fs.readdirSync(parent).filter(name => name.startsWith('plugin.backup-'));
-  assert.equal(backups.length, 1);
-  assert.equal(fs.readFileSync(path.join(parent, backups[0], 'custom-note.txt'), 'utf8'), 'preserve me');
-  assert.equal(fs.existsSync(path.join(target, 'custom-note.txt')), false);
-});
-test('source directories and aliases cannot be overwritten', t => {
-  assert.equal(run(['--extract', '--dir', root, '--force']).status, 1);
-  const alias = path.join(workspace(t), 'source-alias');
-  fs.symlinkSync(root, alias, 'dir');
-  assert.equal(run(['--extract', '--dir', alias, '--force']).status, 1);
-});

package/tests/installer.test.cjs

@@ -1,241 +0,0 @@
-'use strict';
-
-const test = require('node:test');
-const assert = require('node:assert/strict');
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-const { spawnSync } = require('node:child_process');
-const root = path.resolve(__dirname, '..');
-const cli = path.join(root, 'bin/i3-composer-agent.cjs');
-const upstream = require('../rebrand/upstream.json');
-function workspace(t) {
-  const directory = fs.mkdtempSync(path.join(os.tmpdir(), 'i3-agent-install-'));
-  t.after(() => fs.rmSync(directory, { recursive: true, force: true }));
-  return fs.realpathSync(directory);
-}
-function run(directory, args) {
-  return spawnSync(process.execPath, [cli, ...args], { cwd: directory, encoding: 'utf8' });
-}
-test('native CodeBuddy installation includes short commands, personas and usable references', t => {
-  const directory = workspace(t);
-  const result = run(directory, ['--codebuddy', '--scope', 'project', '--yes']);
-  assert.equal(result.status, 0, result.stderr);
-  const base = path.join(directory, '.codebuddy');
-  assert.equal(fs.readdirSync(path.join(base, 'skills')).length, upstream.skills.length);
-  assert.equal(fs.readdirSync(path.join(base, 'commands')).length, upstream.commands.length);
-  assert.equal(fs.readdirSync(path.join(base, 'agents')).length, upstream.personas.length);
-  const command = fs.readFileSync(path.join(base, 'commands/i3-spec.md'), 'utf8');
-  assert.match(command, /Invoke the i3-spec-driven-development skill/);
-  assert.doesNotMatch(command, /i3-composer-agent:/);
-  const ship = fs.readFileSync(path.join(base, 'commands/i3-ship.md'), 'utf8');
-  assert.match(ship, /\.codebuddy\/agents\//);
-  assert.doesNotMatch(ship, /\.claude\/agents\//);
-  assert.ok(fs.existsSync(path.join(base, 'references/definition-of-done.md')));
-  assert.equal(fs.readFileSync(path.join(base, 'i3-composer-agent-LICENSE'), 'utf8'), fs.readFileSync(path.join(root, 'LICENSE'), 'utf8'));
-  assert.match(fs.readFileSync(path.join(base, 'i3-composer-agent-NOTICE.md'), 'utf8'), /Addy Osmani/);
-  assert.equal(fs.existsSync(path.join(directory, 'AGENTS.md')), false);
-  assert.equal(fs.existsSync(path.join(directory, 'i3-composer-agent')), false);
-  assert.match(result.stdout, /\/i3-spec/);
-});
-test('multiple tools retain their own destinations and share universal skills without duplicate work', t => {
-  const directory = workspace(t);
-  const result = run(directory, ['--agents', 'codebuddy,codex,cursor,claude-code', '--scope', 'project', '--yes']);
-  assert.equal(result.status, 0, result.stderr);
-  for (const base of ['.codebuddy', '.agents', '.claude']) {
-    assert.equal(fs.readdirSync(path.join(directory, base, 'skills')).length, upstream.skills.length);
-    assert.ok(fs.existsSync(path.join(directory, base, 'references/testing-patterns.md')));
-  }
-  const manifest = JSON.parse(fs.readFileSync(path.join(directory, '.agents/.i3-composer-agent.json')));
-  assert.deepEqual(manifest.agents.sort(), ['codex', 'cursor']);
-  assert.equal(fs.existsSync(path.join(directory, '.agents/commands')), false);
-});
-test('installation dry run describes scope and destinations without changing project files', t => {
-  const directory = workspace(t);
-  const result = run(directory, ['--agents', 'codebuddy,codex', '--scope', 'project', '--dry-run']);
-  assert.equal(result.status, 0, result.stderr);
-  const plan = JSON.parse(result.stdout);
-  assert.equal(plan.mode, 'install');
-  assert.equal(plan.scope, 'project');
-  assert.equal(plan.targets.length, 2);
-  assert.deepEqual(fs.readdirSync(directory), []);
-});
-test('all copied Markdown reference links and stable skill frontmatter remain valid', t => {
-  const directory = workspace(t);
-  assert.equal(run(directory, ['--codebuddy', '--scope', 'project', '--yes']).status, 0);
-  const base = path.join(directory, '.codebuddy');
-  function visit(folder) {
-    for (const entry of fs.readdirSync(folder, { withFileTypes: true })) {
-      const file = path.join(folder, entry.name);
-      if (entry.isDirectory()) visit(file);
-      else if (entry.name.endsWith('.md')) {
-        const content = fs.readFileSync(file, 'utf8');
-        for (const match of content.matchAll(/\]\(([^)]+)\)/g)) {
-          const relative = match[1].split('#')[0];
-          if (relative.includes('references/') && !/^[a-z]+:/i.test(relative)) {
-            assert.ok(fs.existsSync(path.resolve(path.dirname(file), relative)), `${file}: ${relative}`);
-          }
-        }
-      }
-    }
-  }
-  visit(path.join(base, 'skills'));
-  for (const id of upstream.skills.map(id => `i3-${id}`)) {
-    const head = content => content.match(/^---\r?\n[\s\S]*?\r?\n---/)[0];
-    assert.equal(head(fs.readFileSync(path.join(base, 'skills', id, 'SKILL.md'), 'utf8')), head(fs.readFileSync(path.join(root, 'skills', id, 'SKILL.md'), 'utf8')));
-  }
-});
-function legacyInstallation(directory) {
-  const base = path.join(directory, '.codebuddy');
-  fs.mkdirSync(path.join(base, 'commands'), { recursive: true });
-  fs.mkdirSync(path.join(base, 'skills/spec-driven-development'), { recursive: true });
-  fs.writeFileSync(path.join(base, 'commands/spec.md'), 'original owned command');
-  fs.writeFileSync(path.join(base, 'skills/spec-driven-development/SKILL.md'), 'original owned Skill');
-  fs.writeFileSync(path.join(base, 'commands/build.md'), 'unowned custom command');
-  fs.writeFileSync(path.join(base, '.i3-composer-agent.json'), JSON.stringify({ name: 'i3-composer-agent', version: '1.0.1', assets: ['commands/spec.md', 'skills/spec-driven-development'] }));
-  return base;
-}
-test('prefix migration retires only owned legacy paths and preserves their backups', t => {
-  const directory = workspace(t);
-  const base = legacyInstallation(directory);
-  const args = ['--codebuddy', '--scope', 'project', '--yes'];
-  assert.equal(run(directory, args).status, 1);
-  assert.equal(fs.readFileSync(path.join(base, 'commands/spec.md'), 'utf8'), 'original owned command');
-  const result = run(directory, [...args, '--force']);
-  assert.equal(result.status, 0, result.stderr);
-  assert.equal(fs.existsSync(path.join(base, 'commands/spec.md')), false);
-  assert.equal(fs.existsSync(path.join(base, 'skills/spec-driven-development')), false);
-  assert.ok(fs.existsSync(path.join(base, 'commands/i3-spec.md')));
-  assert.match(fs.readFileSync(path.join(base, 'skills/i3-spec-driven-development/SKILL.md'), 'utf8'), /^name: i3-spec-driven-development$/m);
-  assert.equal(fs.readFileSync(path.join(base, 'commands/build.md'), 'utf8'), 'unowned custom command');
-  const backup = fs.readdirSync(base).find(file => file.startsWith('.i3-composer-agent-backup-'));
-  assert.equal(fs.readFileSync(path.join(base, backup, 'commands/spec.md'), 'utf8'), 'original owned command');
-  assert.equal(fs.readFileSync(path.join(base, backup, 'skills/spec-driven-development/SKILL.md'), 'utf8'), 'original owned Skill');
-  const manifest = JSON.parse(fs.readFileSync(path.join(base, '.i3-composer-agent.json'), 'utf8'));
-  assert.equal(manifest.entryPrefix, 'i3-');
-  assert.ok(!manifest.assets.includes('commands/spec.md'));
-});
-test('a failure after retiring legacy paths rolls the migration back', t => {
-  const directory = workspace(t);
-  const base = legacyInstallation(directory);
-  const before = fs.readFileSync(path.join(base, '.i3-composer-agent.json'), 'utf8');
-  const { planInstallation, executeInstallation } = require('../bin/install.cjs');
-  const plan = planInstallation({ agents: ['codebuddy'], scope: 'project', projectDir: directory });
-  const realRename = fs.renameSync;
-  fs.renameSync = (source, destination) => {
-    if (destination === path.join(base, '.i3-composer-agent.json') && source.includes('-stage-')) throw new Error('simulated manifest failure');
-    return realRename(source, destination);
-  };
-  try { assert.throws(() => executeInstallation(plan, { force: true }), /simulated manifest failure/); }
-  finally { fs.renameSync = realRename; }
-  assert.equal(fs.readFileSync(path.join(base, '.i3-composer-agent.json'), 'utf8'), before);
-  assert.equal(fs.readFileSync(path.join(base, 'commands/spec.md'), 'utf8'), 'original owned command');
-  assert.equal(fs.readFileSync(path.join(base, 'skills/spec-driven-development/SKILL.md'), 'utf8'), 'original owned Skill');
-  assert.equal(fs.existsSync(path.join(base, 'commands/i3-spec.md')), false);
-});
-test('existing assets are protected; confirmed replacement backs them up and preserves unrelated settings', t => {
-  const directory = workspace(t);
-  const base = path.join(directory, '.codebuddy');
-  fs.mkdirSync(path.join(base, 'commands'), { recursive: true });
-  fs.writeFileSync(path.join(base, 'commands/i3-spec.md'), 'my custom command');
-  fs.writeFileSync(path.join(base, 'settings.json'), '{"keep":"my settings"}\n');
-  const args = ['--codebuddy', '--scope', 'project', '--yes'];
-  assert.equal(run(directory, args).status, 1);
-  assert.equal(fs.existsSync(path.join(base, 'skills')), false);
-  const result = run(directory, [...args, '--force']);
-  assert.equal(result.status, 0, result.stderr);
-  const backups = fs.readdirSync(base).filter(file => file.startsWith('.i3-composer-agent-backup-'));
-  assert.equal(backups.length, 1);
-  assert.equal(fs.readFileSync(path.join(base, backups[0], 'commands/i3-spec.md'), 'utf8'), 'my custom command');
-  assert.equal(fs.readFileSync(path.join(base, 'settings.json'), 'utf8'), '{"keep":"my settings"}\n');
-  assert.equal(fs.existsSync(path.join(base, backups[0], 'settings.json')), false);
-});
-test('global installation uses injected home and config roots without touching the real user configuration', t => {
-  const directory = workspace(t);
-  const { planInstallation, executeInstallation } = require('../bin/install.cjs');
-  const plan = planInstallation({ agents: ['codebuddy', 'codex', 'claude-code', 'opencode'], scope: 'global', home: directory, env: { XDG_CONFIG_HOME: path.join(directory, 'config'), CLAUDE_CONFIG_DIR: path.join(directory, 'custom-claude') } });
-  assert.ok(plan.targets.every(target => target.skillsDirectory.startsWith(directory + path.sep)));
-  assert.ok(plan.targets.some(target => target.skillsDirectory === path.join(directory, 'custom-claude/skills')));
-  assert.ok(plan.targets.some(target => target.skillsDirectory === path.join(directory, 'config/opencode/skills')));
-  executeInstallation(plan);
-  for (const target of plan.targets) {
-    assert.equal(fs.readdirSync(target.skillsDirectory).length, upstream.skills.length);
-    assert.ok(fs.existsSync(path.join(target.referencesDirectory, 'definition-of-done.md')));
-  }
-});
-test('the pinned tool catalog plans all project choices and rejects unsupported global choices before writing', t => {
-  const directory = workspace(t);
-  const { registry, planInstallation } = require('../bin/install.cjs');
-  assert.equal(registry.agents.length, 79);
-  assert.equal(registry.commit, '48dc9e8eb8aa19040e91cb90ba0b642c9cbad57e');
-  const plan = planInstallation({ agents: registry.agents.map(agent => agent.id), scope: 'project', projectDir: directory });
-  assert.equal(plan.agents.length, registry.agents.length);
-  assert.ok(plan.targets.length < plan.agents.length);
-  assert.equal(new Set(plan.assets.map(asset => asset.destination)).size, plan.assets.length);
-  for (const agent of registry.agents) {
-    if (agent.globalSkillsDir) assert.ok(planInstallation({ agents: [agent.id], scope: 'global', home: directory, env: {} }).targets[0].skillsDirectory.startsWith(directory + path.sep));
-    else assert.throws(() => planInstallation({ agents: [agent.id], scope: 'global', home: directory, env: {} }), /project installation only/);
-  }
-  assert.deepEqual(fs.readdirSync(directory), []);
-});
-test('failed installation restores prior assets and removes all newly written files', t => {
-  const directory = workspace(t);
-  const { planInstallation, executeInstallation } = require('../bin/install.cjs');
-  const prior = path.join(directory, '.codebuddy/skills/i3-api-and-interface-design');
-  fs.mkdirSync(prior, { recursive: true });
-  fs.writeFileSync(path.join(prior, 'custom.md'), 'prior skill content');
-  const plan = planInstallation({ agents: ['codebuddy', 'codex'], scope: 'project', projectDir: directory });
-  const realRename = fs.renameSync;
-  let failed = false;
-  fs.renameSync = (source, destination) => {
-    if (!failed && destination === plan.assets[1].destination) { failed = true; throw new Error('simulated disk failure'); }
-    return realRename(source, destination);
-  };
-  try { assert.throws(() => executeInstallation(plan, { force: true }), /simulated disk failure/); }
-  finally { fs.renameSync = realRename; }
-  assert.equal(fs.readFileSync(path.join(prior, 'custom.md'), 'utf8'), 'prior skill content');
-  assert.deepEqual(fs.readdirSync(path.dirname(prior)), ['i3-api-and-interface-design']);
-  assert.deepEqual(fs.readdirSync(path.join(directory, '.codebuddy')), ['skills']);
-  assert.equal(fs.existsSync(path.join(directory, '.agents')), false);
-});
-test('replacing a skill symlink backs up the link and leaves its external source untouched', t => {
-  const directory = workspace(t);
-  const external = path.join(directory, 'external-skill');
-  const skills = path.join(directory, '.codebuddy/skills');
-  fs.mkdirSync(external);
-  fs.writeFileSync(path.join(external, 'custom.md'), 'external source');
-  fs.mkdirSync(skills, { recursive: true });
-  fs.symlinkSync(external, path.join(skills, 'i3-spec-driven-development'), 'dir');
-  const result = run(directory, ['--codebuddy', '--scope', 'project', '--yes', '--force']);
-  assert.equal(result.status, 0, result.stderr);
-  assert.deepEqual(fs.readdirSync(external), ['custom.md']);
-  assert.equal(fs.lstatSync(path.join(skills, 'i3-spec-driven-development')).isSymbolicLink(), false);
-  const base = path.dirname(skills);
-  const backup = fs.readdirSync(base).find(file => file.startsWith('.i3-composer-agent-backup-'));
-  assert.equal(fs.lstatSync(path.join(base, backup, 'skills/i3-spec-driven-development')).isSymbolicLink(), true);
-});
-test('package destinations, shared directory symlinks and ambiguous noninteractive flags fail without writes', t => {
-  const directory = workspace(t);
-  for (const args of [ ['--codebuddy'], ['--codebuddy', '--yes'], ['--agents', 'unknown', '--scope', 'project', '--yes'], ['--codebuddy', '--scope', 'invalid', '--yes'], ['--codebuddy', '--scope', 'project', '--global', '--yes'] ]) {
-    assert.equal(run(directory, args).status, 1);
-    assert.deepEqual(fs.readdirSync(directory), []);
-  }
-  assert.equal(run(directory, ['--codebuddy', '--scope', 'project', '--yes', '--project-dir', root]).status, 1);
-  fs.mkdirSync(path.join(directory, '.codebuddy'));
-  fs.symlinkSync(path.join(root, 'skills'), path.join(directory, '.codebuddy/skills'), 'dir');
-  assert.equal(run(directory, ['--codebuddy', '--scope', 'project', '--yes', '--force']).status, 1);
-  assert.deepEqual(fs.readdirSync(path.join(directory, '.codebuddy')), ['skills']);
-});
-test('explicit plugin preparation remains available and does not install native host files', t => {
-  const directory = workspace(t);
-  const result = run(directory, ['--prepare', '--codebuddy']);
-  assert.equal(result.status, 0, result.stderr);
-  assert.match(result.stdout, /\/plugin install/);
-  assert.equal(fs.existsSync(path.join(directory, 'i3-composer-agent/plugin.json')), true);
-  assert.equal(fs.existsSync(path.join(directory, '.codebuddy')), false);
-  const listing = run(directory, ['--list-agents']);
-  assert.equal(listing.status, 0, listing.stderr);
-  assert.equal(listing.stdout.trim().split('\n').length, 79);
-  assert.match(listing.stdout, /codebuddy/);
-  assert.match(listing.stdout, /windsurf/);
-});