@givolt/i3-composer-agent 2.0.0 → 2.1.0

package/scripts/validate-artifact-paths-test.js

@@ -1,179 +0,0 @@
-#!/usr/bin/env node
-
-'use strict';
-
-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 { afterEach, test } = require('node:test');
-
-const VALIDATOR = path.join(__dirname, 'validate-artifact-paths.js');
-const sandboxes = [];
-
-function makeSandbox() {
-  const root = fs.mkdtempSync(path.join(os.tmpdir(), 'agent-skills-validate-artifact-paths-test-'));
-  const scriptsDir = path.join(root, 'scripts');
-  fs.mkdirSync(scriptsDir, { recursive: true });
-  fs.copyFileSync(VALIDATOR, path.join(scriptsDir, 'validate-artifact-paths.js'));
-  sandboxes.push(root);
-  return root;
-}
-
-function writeFile(root, relativePath, content) {
-  const file = path.join(root, relativePath);
-  fs.mkdirSync(path.dirname(file), { recursive: true });
-  fs.writeFileSync(file, content);
-}
-
-function run(root) {
-  return spawnSync(process.execPath, [path.join(root, 'scripts', 'validate-artifact-paths.js')], {
-    cwd: root,
-    encoding: 'utf8',
-  });
-}
-
-afterEach(() => {
-  for (const root of sandboxes.splice(0)) {
-    fs.rmSync(root, { recursive: true, force: true });
-  }
-});
-
-test('passes when producers and consumers use the canonical artifact paths', () => {
-  const root = makeSandbox();
-  writeFile(root, '.claude/commands/spec.md', 'Save the spec as `SPEC.md` in the project root.\n');
-  writeFile(root, '.claude/commands/plan.md', 'Save the plan to `tasks/plan.md` and task list to `tasks/todo.md`.\n');
-  writeFile(root, '.claude/commands/build.md', 'Look for a spec at `SPEC.md`, `docs/SPEC.md`, or under `spec/`. Require `tasks/plan.md`.\n');
-  writeFile(root, 'skills/i3-spec-driven-development/SKILL.md', 'Save the plan to `tasks/plan.md` and the task list to `tasks/todo.md`.\n');
-  writeFile(root, 'skills/i3-planning-and-task-breakdown/SKILL.md', 'Save to `tasks/plan.md` and `tasks/todo.md`.\n');
-
-  const result = run(root);
-
-  assert.equal(result.status, 0, result.stdout + result.stderr);
-  assert.match(result.stdout, /5 files checked — 0 error\(s\) — PASSED/);
-});
-
-test('fails when a producer drifts to an unapproved artifact path (the #93 regression)', () => {
-  const root = makeSandbox();
-  // Only the producers are changed to the per-feature layout; consumers stay on the old paths.
-  writeFile(root, '.claude/commands/spec.md', 'Save the spec to `docs/features/[feature-name]/spec.md`.\n');
-  writeFile(root, '.claude/commands/plan.md', 'Save the plan to `docs/features/[feature-name]/plan.md`.\n');
-  writeFile(root, '.claude/commands/build.md', 'Require a spec at `SPEC.md` and a plan at `tasks/plan.md`.\n');
-
-  const result = run(root);
-
-  assert.equal(result.status, 1, result.stdout + result.stderr);
-  assert.match(result.stdout, /docs\/features\/\[feature-name\]\/spec\.md/);
-  assert.match(result.stdout, /docs\/features\/\[feature-name\]\/plan\.md/);
-  assert.match(result.stdout, /error\(s\) — FAILED/);
-});
-
-test('reports the offending file and line number', () => {
-  const root = makeSandbox();
-  writeFile(root, '.claude/commands/plan.md', 'Intro line.\nSave the plan to `plan.md` in the root.\n');
-
-  const result = run(root);
-
-  assert.equal(result.status, 1, result.stdout + result.stderr);
-  assert.match(result.stdout, /\.claude\/commands\/plan\.md/);
-  assert.match(result.stdout, /L2:/);
-});
-
-test('accepts the docs/SPEC.md alternate spec location', () => {
-  const root = makeSandbox();
-  writeFile(root, '.claude/commands/build.md', 'Look for the spec at `docs/SPEC.md`.\n');
-
-  const result = run(root);
-
-  assert.equal(result.status, 0, result.stdout + result.stderr);
-  assert.match(result.stdout, /1 files checked — 0 error\(s\) — PASSED/);
-});
-
-test('ignores non-artifact markdown references (no false positives)', () => {
-  const root = makeSandbox();
-  writeFile(
-    root,
-    'skills/i3-spec-driven-development/SKILL.md',
-    'See `SKILL.md` and `references/testing-patterns.md`. Save the plan to `tasks/plan.md`.\n',
-  );
-
-  const result = run(root);
-
-  assert.equal(result.status, 0, result.stdout + result.stderr);
-  assert.match(result.stdout, /1 files checked — 0 error\(s\) — PASSED/);
-});
-
-test('skips guarded files that do not exist', () => {
-  const root = makeSandbox();
-  writeFile(root, '.claude/commands/spec.md', 'Save the spec as `SPEC.md`.\n');
-  // No other guarded files present.
-
-  const result = run(root);
-
-  assert.equal(result.status, 0, result.stdout + result.stderr);
-  assert.match(result.stdout, /1 files checked — 0 error\(s\) — PASSED/);
-});
-
-// ─── Every command surface is guarded, not only the Claude Code copy ────────
-//
-// The Gemini and root TOML command sets carry their own prompt bodies, which
-// validate-commands.js deliberately does not compare. A producer can therefore
-// drift on one surface while `.claude/commands/` stays correct, and before
-// these files were guarded that drift merged silently.
-
-test('fails when a Gemini command drifts while the Claude Code copy stays canonical', () => {
-  const root = makeSandbox();
-  writeFile(root, '.claude/commands/spec.md', 'Save the spec as `SPEC.md` in the project root.\n');
-  writeFile(
-    root,
-    '.gemini/commands/spec.toml',
-    'description = "Start spec-driven development"\n\nprompt = """\nSave the spec to `docs/features/[feature-name]/spec.md`.\n"""\n',
-  );
-
-  const result = run(root);
-
-  assert.equal(result.status, 1, result.stdout + result.stderr);
-  assert.match(result.stdout, /\.gemini\/commands\/spec\.toml/);
-  assert.match(result.stdout, /L4: docs\/features\/\[feature-name\]\/spec\.md/);
-  assert.match(result.stdout, /1 error\(s\) — FAILED/);
-});
-
-test('fails when a root TOML consumer drifts away from the producers', () => {
-  const root = makeSandbox();
-  writeFile(root, 'commands/planning.toml', 'prompt = """\nSave the plan to `tasks/plan.md` and the task list to `tasks/todo.md`.\n"""\n');
-  writeFile(root, 'commands/build.toml', 'prompt = """\nRead the plan from `planning/plan.md` before starting.\n"""\n');
-
-  const result = run(root);
-
-  assert.equal(result.status, 1, result.stdout + result.stderr);
-  assert.match(result.stdout, /commands\/build\.toml/);
-  assert.match(result.stdout, /planning\/plan\.md/);
-});
-
-test('passes when all three command surfaces agree on the canonical paths', () => {
-  const root = makeSandbox();
-  for (const dir of ['.claude/commands', '.gemini/commands', 'commands']) {
-    const ext = dir === '.claude/commands' ? 'md' : 'toml';
-    const planStem = dir === '.claude/commands' ? 'plan' : 'planning';
-    writeFile(root, `${dir}/spec.${ext}`, 'Save the spec as `SPEC.md`.\n');
-    writeFile(root, `${dir}/${planStem}.${ext}`, 'Save to `tasks/plan.md` and `tasks/todo.md`.\n');
-    writeFile(root, `${dir}/build.${ext}`, 'Require `SPEC.md` or `docs/SPEC.md`, and `tasks/plan.md`.\n');
-  }
-
-  const result = run(root);
-
-  assert.equal(result.status, 0, result.stdout + result.stderr);
-  assert.match(result.stdout, /9 files checked — 0 error\(s\) — PASSED/);
-});
-
-test('guards the Copilot setup doc, whose prompt-file aliases name the artifacts', () => {
-  const root = makeSandbox();
-  writeFile(root, 'docs/copilot-setup.md', 'Read `tasks/plan.md`, then write the spec to `specs/[name]/spec.md`.\n');
-
-  const result = run(root);
-
-  assert.equal(result.status, 1, result.stdout + result.stderr);
-  assert.match(result.stdout, /docs\/copilot-setup\.md/);
-  assert.match(result.stdout, /specs\/\[name\]\/spec\.md/);
-});

package/scripts/validate-artifact-paths.js

@@ -1,130 +0,0 @@
-#!/usr/bin/env node
-/**
- * validate-artifact-paths.js
- *
- * Guards the spec -> plan -> build pipeline against silent artifact-path drift.
- *
- * The `/spec` and `/plan` commands (producers) write their artifacts to a set
- * of paths that the `/build` command and the spec/plan skills (consumers) read
- * back. When a producer moves an artifact without updating the consumers — as
- * in PR #93, which pointed `/spec` and `/plan` at docs/features/[name]/ while
- * `/build` still required SPEC.md and tasks/plan.md — the pipeline breaks, and
- * nothing else in CI catches it (command parity only compares descriptions).
- *
- * This validator enforces one canonical set of spec/plan/todo artifact paths
- * across every file in the pipeline. Changing the convention means updating
- * ARTIFACT_ALLOWLIST *and* every guarded file in the same change; CI fails
- * until they agree.
- *
- * Scope is deliberately narrow: only spec/plan/todo artifacts, only the files
- * that define the pipeline. It is not a general markdown path linter.
- *
- * Exit codes: 0 = all clear, 1 = one or more drifted paths.
- */
-
-'use strict';
-
-const fs = require('fs');
-const path = require('path');
-
-const ROOT = path.resolve(__dirname, '..');
-
-// The canonical spec/plan/todo artifact paths. These are the only artifact
-// file paths the pipeline files may reference. To change the convention, edit
-// this list and update every guarded file to match — CI enforces the pairing.
-const ARTIFACT_ALLOWLIST = new Set([
-  'SPEC.md',        // spec, project root (produced by /spec, read by /build)
-  'docs/SPEC.md',   // spec, alternate location accepted by /build
-  'tasks/plan.md',  // plan (produced by /plan, read by /build)
-  'tasks/todo.md',  // task list (produced by /plan)
-]);
-
-// The files that make up the spec -> plan -> build pipeline. Absent files are
-// skipped, not failed: this validator checks path consistency, not presence.
-//
-// The command bodies exist once per host surface (validate-commands.js checks
-// that the three sets stay in step on descriptions, but their prompt bodies
-// are allowed to differ), so a producer or consumer can drift on one surface
-// while the Claude Code copy stays correct. Every surface is guarded here.
-const GUARDED_FILES = [
-  // Claude Code commands
-  '.claude/commands/spec.md',
-  '.claude/commands/plan.md',
-  '.claude/commands/build.md',
-  // Gemini CLI commands
-  '.gemini/commands/spec.toml',
-  '.gemini/commands/planning.toml',
-  '.gemini/commands/build.toml',
-  // Root command set (Antigravity, Codex, and other TOML-based hosts)
-  'commands/spec.toml',
-  'commands/planning.toml',
-  'commands/build.toml',
-  // Skills the commands invoke
-  'skills/i3-spec-driven-development/SKILL.md',
-  'skills/i3-planning-and-task-breakdown/SKILL.md',
-  // Docs that tell users where the artifacts live
-  'docs/getting-started.md',
-  'docs/adoption-guide.md',
-  'docs/copilot-setup.md', // its prompt-file aliases name the artifacts directly
-];
-
-// Matches a path-like token ending in a spec/plan/todo artifact filename,
-// including an optional directory prefix with bracket placeholders like
-// docs/features/[feature-name]/spec.md. Case-insensitive so SPEC.md and a
-// drifted spec.md are both caught, then compared against the allowlist.
-const ARTIFACT_RE = /(?:[A-Za-z0-9._[\]-]+\/)*(?:spec|plan|todo)\.md/gi;
-
-GUARDED_FILES.push(...GUARDED_FILES.filter(relative => /(?:^|\/)commands\//.test(relative)).map(relative => relative.replace(/(commands\/)([^/]+)(\.md|\.toml)$/, '$1i3-$2$3')));
-
-function findViolations(relPath) {
-  const abs = path.join(ROOT, relPath);
-  if (!fs.existsSync(abs)) return null; // skipped
-
-  const violations = [];
-  const lines = fs.readFileSync(abs, 'utf8').split(/\r?\n/);
-  lines.forEach((line, i) => {
-    const matches = line.match(ARTIFACT_RE);
-    if (!matches) return;
-    for (const match of matches) {
-      if (!ARTIFACT_ALLOWLIST.has(match)) {
-        violations.push({ line: i + 1, match });
-      }
-    }
-  });
-  return violations;
-}
-
-function main() {
-  console.log('Checking spec/plan/todo artifact paths...\n');
-
-  let checked = 0;
-  let errors = 0;
-
-  for (const relPath of GUARDED_FILES) {
-    const violations = findViolations(relPath);
-    if (violations === null) continue; // file not present, skip
-    checked++;
-
-    if (violations.length === 0) {
-      console.log(`  ✓  ${relPath}`);
-    } else {
-      console.log(`  ✗  ${relPath}`);
-      for (const { line, match } of violations) {
-        console.log(`       L${line}: ${match} — not an approved spec/plan/todo artifact path`);
-        errors++;
-      }
-    }
-  }
-
-  const status = errors > 0 ? 'FAILED' : 'PASSED';
-  console.log(`\n${checked} files checked — ${errors} error(s) — ${status}`);
-
-  if (errors > 0) {
-    console.log('\nThe spec -> plan -> build pipeline expects one set of artifact paths.');
-    console.log('Either use a path from ARTIFACT_ALLOWLIST, or change the convention');
-    console.log('across every guarded file and update the allowlist in the same change.');
-    process.exit(1);
-  }
-}
-
-main();

package/scripts/validate-brand.cjs

@@ -1,44 +0,0 @@
-#!/usr/bin/env node
-'use strict';
-
-const assert = require('node:assert/strict');
-const fs = require('node:fs');
-const path = require('node:path');
-const root = path.resolve(__dirname, '..');
-const brand = require('../rebrand/brand.json');
-const pkg = require('../package.json');
-const upstream = require('../rebrand/upstream.json');
-const { validateCatalog } = require('../rebrand/catalog.cjs');
-validateCatalog(root, upstream, { branded: true });
-function json(relative) { return JSON.parse(fs.readFileSync(path.join(root, relative), 'utf8')); }
-assert.equal(pkg.name, brand.npmName);
-assert.equal(pkg.version, brand.releaseVersion);
-assert.equal(brand.upstreamCommit, upstream.upstreamCommit);
-assert.equal(pkg.i3ComposerAgent.upstreamVersion, upstream.upstreamVersion);
-assert.equal(pkg.i3ComposerAgent.upstreamCommit, upstream.upstreamCommit);
-assert.equal(pkg.i3ComposerAgent.upstreamRepository, brand.upstream);
-assert.equal(Object.keys(pkg.bin).length, 1, 'npx must resolve one executable');
-assert.ok(fs.existsSync(path.join(root, pkg.bin[brand.name])));
-for (const relative of ['plugin.json', '.claude-plugin/plugin.json', '.codex-plugin/plugin.json']) {
-  assert.equal(json(relative).name, brand.name, `${relative}: inconsistent plugin identity`);
-}
-for (const relative of ['.claude-plugin/marketplace.json', '.agents/plugins/marketplace.json']) {
-  const marketplace = json(relative);
-  assert.equal(marketplace.name, brand.name);
-  assert.equal(marketplace.plugins[0].name, brand.name);
-  assert.ok(marketplace.plugins[0].source === './' || marketplace.plugins[0].source.path === './', 'marketplace must load this distribution');
-}
-assert.ok(json('.codex-plugin/plugin.json').interface.shortDescription.length <= 30);
-const skills = fs.readdirSync(path.join(root, 'skills')).filter(name => fs.existsSync(path.join(root, 'skills', name, 'SKILL.md')));
-assert.equal(skills.length, upstream.skills.length, 'brand conversion must preserve the reviewed upstream skills');
-assert.ok(skills.includes('i3-using-agent-skills'), 'meta-skill ID must use the branded prefix');
-const commands = fs.readdirSync(path.join(root, '.claude/commands')).filter(name => name.endsWith('.md'));
-assert.equal(commands.length, upstream.commands.length, 'brand conversion must preserve the reviewed upstream commands');
-for (const command of commands) {
-  const body = fs.readFileSync(path.join(root, '.claude/commands', command), 'utf8');
-  assert.ok(!body.includes('agent-skills:'), `${command}: stale plugin namespace`);
-  for (const match of body.matchAll(/(?<!\/)i3-composer-agent:([a-z0-9-]+)/g)) assert.ok(skills.includes(match[1]), `${command}: unknown skill ${match[1]}`);
-}
-for (const relative of pkg.i3ComposerAgent.pluginFiles) assert.ok(fs.existsSync(path.join(root, relative)), `bundle is missing ${relative}`);
-assert.match(fs.readFileSync(path.join(root, 'LICENSE'), 'utf8'), /Copyright \(c\) 2025 Addy Osmani/);
-console.log('Brand identity, package entrypoint, reviewed upstream catalog, and attribution are consistent.');

package/scripts/validate-commands-test.js

@@ -1,216 +0,0 @@
-#!/usr/bin/env node
-
-'use strict';
-
-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 { afterEach, test } = require('node:test');
-
-const VALIDATOR = path.join(__dirname, 'validate-commands.js');
-// The validator shares the frontmatter-validity rules with validate-skills, so
-// the sandbox needs the lib alongside it, not just the script.
-const SKILL_LINT = path.join(__dirname, 'lib', 'skill-lint.js');
-const sandboxes = [];
-
-function makeSandbox() {
-  const root = fs.mkdtempSync(path.join(os.tmpdir(), 'agent-skills-validate-commands-test-'));
-  const scriptsDir = path.join(root, 'scripts');
-  fs.mkdirSync(scriptsDir, { recursive: true });
-  fs.copyFileSync(VALIDATOR, path.join(scriptsDir, 'validate-commands.js'));
-  fs.mkdirSync(path.join(scriptsDir, 'lib'), { recursive: true });
-  fs.copyFileSync(SKILL_LINT, path.join(scriptsDir, 'lib', 'skill-lint.js'));
-  sandboxes.push(root);
-  return root;
-}
-
-function writeFile(root, relativePath, content) {
-  const file = path.join(root, relativePath);
-  fs.mkdirSync(path.dirname(file), { recursive: true });
-  fs.writeFileSync(file, content);
-}
-
-function writeClaudeCommand(root, stem, descriptionLine) {
-  writeFile(
-    root,
-    path.join('.claude', 'commands', `${stem}.md`),
-    `---\n${descriptionLine}\n---\n\n# Command\n`,
-  );
-}
-
-function writeTomlCommand(root, directory, stem, descriptionLine) {
-  writeFile(root, path.join(directory, `${stem}.toml`), `${descriptionLine}\nprompt = "Run command"\n`);
-}
-
-function writeMatchingCommands(root, stem, description) {
-  writeClaudeCommand(root, stem, `description: ${description}`);
-  writeTomlCommand(root, path.join('.gemini', 'commands'), stem, `description = "${description}"`);
-  writeTomlCommand(root, 'commands', stem, `description = "${description}"`);
-}
-
-function run(root) {
-  return spawnSync(process.execPath, [path.join(root, 'scripts', 'validate-commands.js')], {
-    cwd: root,
-    encoding: 'utf8',
-  });
-}
-
-afterEach(() => {
-  for (const root of sandboxes.splice(0)) {
-    fs.rmSync(root, { recursive: true, force: true });
-  }
-});
-
-test('passes matching command twins and maps plan to planning', () => {
-  const root = makeSandbox();
-  const description = 'Break work into ordered tasks';
-  writeClaudeCommand(root, 'plan', `description: ${description}`);
-  writeTomlCommand(root, path.join('.gemini', 'commands'), 'planning', `description = '${description}'`);
-  writeTomlCommand(root, 'commands', 'planning', `description = '${description}'`);
-
-  const result = run(root);
-
-  assert.equal(result.status, 0, result.stdout + result.stderr);
-  assert.match(result.stdout, /✓  plan \(planning in toml dirs\)/);
-  assert.match(result.stdout, /1 commands checked — 0 error\(s\) — PASSED/);
-});
-
-test('fails when a Claude command is missing a TOML twin', () => {
-  const root = makeSandbox();
-  const description = 'Review a change';
-  writeClaudeCommand(root, 'review', `description: ${description}`);
-  writeTomlCommand(root, path.join('.gemini', 'commands'), 'review', `description = "${description}"`);
-
-  const result = run(root);
-
-  assert.equal(result.status, 1, result.stdout + result.stderr);
-  assert.match(result.stdout, /review — missing in: commands/);
-  assert.match(result.stdout, /1 commands checked — 1 error\(s\) — FAILED/);
-});
-
-test('fails when a TOML command has no Claude twin', () => {
-  const root = makeSandbox();
-  writeTomlCommand(root, path.join('.gemini', 'commands'), 'deploy', 'description = "Deploy a change"');
-  writeTomlCommand(root, 'commands', 'deploy', 'description = "Deploy a change"');
-
-  const result = run(root);
-
-  assert.equal(result.status, 1, result.stdout + result.stderr);
-  assert.match(result.stdout, /deploy — present in toml dirs but missing in \.claude\/commands/);
-  assert.match(result.stdout, /1 commands checked — 1 error\(s\) — FAILED/);
-});
-
-test('reports all descriptions when command twins drift', () => {
-  const root = makeSandbox();
-  writeClaudeCommand(root, 'review', 'description: Review a change');
-  writeTomlCommand(root, path.join('.gemini', 'commands'), 'review', 'description = "Inspect a change"');
-  writeTomlCommand(root, 'commands', 'review', 'description = "Audit a change"');
-
-  const result = run(root);
-
-  assert.equal(result.status, 1, result.stdout + result.stderr);
-  assert.match(result.stdout, /\.claude:\s+Review a change/);
-  assert.match(result.stdout, /\.gemini:\s+Inspect a change/);
-  assert.match(result.stdout, /commands\/:\s+Audit a change/);
-});
-
-test('fails with an actionable error for a malformed description', () => {
-  const root = makeSandbox();
-  writeMatchingCommands(root, 'review', 'Review a change');
-  writeFile(
-    root,
-    path.join('.gemini', 'commands', 'review.toml'),
-    'prompt = "Missing description"\n',
-  );
-
-  const result = run(root);
-
-  assert.equal(result.status, 1, result.stdout + result.stderr);
-  assert.match(result.stdout, /\.gemini\/commands\/review — missing or malformed description/);
-  assert.match(result.stdout, /1 commands checked — 1 error\(s\) — FAILED/);
-});
-
-test('parses escaped quotes in double-quoted TOML descriptions', () => {
-  const root = makeSandbox();
-  writeClaudeCommand(root, 'review', 'description: Review "important" changes');
-  writeTomlCommand(
-    root,
-    path.join('.gemini', 'commands'),
-    'review',
-    'description = "Review \\"important\\" changes"',
-  );
-  writeTomlCommand(
-    root,
-    'commands',
-    'review',
-    'description = "Review \\"important\\" changes"',
-  );
-
-  const result = run(root);
-
-  assert.equal(result.status, 0, result.stdout + result.stderr);
-  assert.match(result.stdout, /1 commands checked — 0 error\(s\) — PASSED/);
-});
-
-// Claude parses a command's frontmatter as YAML when the command is loaded, and
-// `descriptionFromMd` splits each line on its first colon — so a command whose
-// frontmatter is not valid YAML passed every check here. Same class as the
-// SKILL.md gap, on the other set of files the #494 thread checked by hand.
-
-test('a command whose frontmatter is valid YAML passes', () => {
-  const root = makeSandbox();
-  writeMatchingCommands(root, 'build', 'Build the thing');
-  const result = run(root);
-  assert.equal(result.status, 0, result.stdout);
-  assert.match(result.stdout, /Checking Claude command frontmatter/);
-});
-
-test('an unquoted colon in a command description is rejected', () => {
-  const root = makeSandbox();
-  // Valid to the splitter, rejected by a real YAML parser: it reads a nested
-  // mapping. The TOML siblings keep the same text so only the .md is at fault.
-  writeClaudeCommand(root, 'build', 'description: Build the thing: quickly');
-  writeTomlCommand(root, path.join('.gemini', 'commands'), 'build', 'description = "Build the thing: quickly"');
-  writeTomlCommand(root, 'commands', 'build', 'description = "Build the thing: quickly"');
-
-  const result = run(root);
-  assert.equal(result.status, 1, result.stdout);
-  assert.match(result.stdout, /unquoted value containing/);
-});
-
-test('quoting the same description makes it pass', () => {
-  const root = makeSandbox();
-  writeClaudeCommand(root, 'build', 'description: "Build the thing: quickly"');
-  writeTomlCommand(root, path.join('.gemini', 'commands'), 'build', 'description = "Build the thing: quickly"');
-  writeTomlCommand(root, 'commands', 'build', 'description = "Build the thing: quickly"');
-
-  const result = run(root);
-  assert.equal(result.status, 0, result.stdout);
-});
-
-test('a tab-indented command frontmatter is rejected', () => {
-  const root = makeSandbox();
-  writeFile(
-    root,
-    path.join('.claude', 'commands', 'build.md'),
-    '---\ndescription: Build the thing\nmeta:\n\tlevel: core\n---\n\n# Command\n',
-  );
-  writeTomlCommand(root, path.join('.gemini', 'commands'), 'build', 'description = "Build the thing"');
-  writeTomlCommand(root, 'commands', 'build', 'description = "Build the thing"');
-
-  const result = run(root);
-  assert.equal(result.status, 1, result.stdout);
-  assert.match(result.stdout, /indents with a tab/);
-});
-
-test('the TOML directories are not put through the YAML rules', () => {
-  const root = makeSandbox();
-  // A TOML description legitimately carries a colon. Running the YAML rules over
-  // these files would fail every command that has one.
-  writeMatchingCommands(root, 'build', 'Build the thing quickly');
-  const result = run(root);
-  assert.equal(result.status, 0, result.stdout);
-  assert.doesNotMatch(result.stdout, /\.gemini.*unquoted value/);
-});