adapters.test.ts

import { describe, it, expect } from 'vitest';
import path from 'path';
import { amazonQAdapter } from '../../../src/core/command-generation/adapters/amazon-q.js';
import { antigravityAdapter } from '../../../src/core/command-generation/adapters/antigravity.js';
import { auggieAdapter } from '../../../src/core/command-generation/adapters/auggie.js';
import { bobAdapter } from '../../../src/core/command-generation/adapters/bob.js';
import { claudeAdapter } from '../../../src/core/command-generation/adapters/claude.js';
import { clineAdapter } from '../../../src/core/command-generation/adapters/cline.js';
import { commandCodeAdapter } from '../../../src/core/command-generation/adapters/command-code.js';
import { codebuddyAdapter } from '../../../src/core/command-generation/adapters/codebuddy.js';
import { continueAdapter } from '../../../src/core/command-generation/adapters/continue.js';
import { costrictAdapter } from '../../../src/core/command-generation/adapters/costrict.js';
import { crushAdapter } from '../../../src/core/command-generation/adapters/crush.js';
import { cursorAdapter } from '../../../src/core/command-generation/adapters/cursor.js';
import { devinAdapter } from '../../../src/core/command-generation/adapters/devin.js';
import { factoryAdapter } from '../../../src/core/command-generation/adapters/factory.js';
import { geminiAdapter } from '../../../src/core/command-generation/adapters/gemini.js';
import { githubCopilotAdapter } from '../../../src/core/command-generation/adapters/github-copilot.js';
import { iflowAdapter } from '../../../src/core/command-generation/adapters/iflow.js';
import { junieAdapter } from '../../../src/core/command-generation/adapters/junie.js';
import { kilocodeAdapter } from '../../../src/core/command-generation/adapters/kilocode.js';
import { kiroAdapter } from '../../../src/core/command-generation/adapters/kiro.js';
import { lingmaAdapter } from '../../../src/core/command-generation/adapters/lingma.js';
import { ohMyPiAdapter } from '../../../src/core/command-generation/adapters/oh-my-pi.js';
import { opencodeAdapter } from '../../../src/core/command-generation/adapters/opencode.js';
import { piAdapter } from '../../../src/core/command-generation/adapters/pi.js';
import { qoderAdapter } from '../../../src/core/command-generation/adapters/qoder.js';
import { qwenAdapter } from '../../../src/core/command-generation/adapters/qwen.js';
import { roocodeAdapter } from '../../../src/core/command-generation/adapters/roocode.js';
import { traeAdapter } from '../../../src/core/command-generation/adapters/trae.js';
import { codeassistantAdapter } from '../../../src/core/command-generation/adapters/codeassistant.js';
import { zcodeAdapter } from '../../../src/core/command-generation/adapters/zcode.js';
import type {
  CommandContent,
  ToolCommandAdapter,
} from '../../../src/core/command-generation/types.js';
import { CommandAdapterRegistry } from '../../../src/core/command-generation/registry.js';
import { generateCommand } from '../../../src/core/command-generation/generator.js';
import { getCommandContents } from '../../../src/core/shared/skill-generation.js';
import { parse as parseYaml } from 'yaml';
import { parse as parseToml } from 'smol-toml';

describe('command-generation/adapters', () => {
  const sampleContent: CommandContent = {
    id: 'explore',
    name: 'OpenSpec Explore',
    description: 'Enter explore mode for thinking',
    category: 'Workflow',
    tags: ['workflow', 'explore', 'experimental'],
    body: 'This is the command body.\n\nWith multiple lines.',
  };

  describe('claudeAdapter', () => {
    it('should have correct toolId', () => {
      expect(claudeAdapter.toolId).toBe('claude');
    });

    it('should generate correct file path', () => {
      const filePath = claudeAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.claude', 'commands', 'opsx', 'explore.md'));
    });

    it('should generate correct file path for different command IDs', () => {
      expect(claudeAdapter.getFilePath('new')).toBe(path.join('.claude', 'commands', 'opsx', 'new.md'));
      expect(claudeAdapter.getFilePath('bulk-archive')).toBe(path.join('.claude', 'commands', 'opsx', 'bulk-archive.md'));
    });

    it('should format file with correct YAML frontmatter', () => {
      const output = claudeAdapter.formatFile(sampleContent);

      expect(output).toContain('---\n');
      expect(output).toContain('name: "OpenSpec Explore"');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('allowed-tools: Bash(openspec:*)');
      expect(output).toContain('category: "Workflow"');
      expect(output).toContain('tags: ["workflow", "explore", "experimental"]');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.\n\nWith multiple lines.');
    });

    it('should handle empty tags', () => {
      const contentNoTags: CommandContent = { ...sampleContent, tags: [] };
      const output = claudeAdapter.formatFile(contentNoTags);
      expect(output).toContain('tags: []');
    });
  });

  describe('cursorAdapter', () => {
    it('should have correct toolId', () => {
      expect(cursorAdapter.toolId).toBe('cursor');
    });

    it('should generate correct file path with opsx- prefix', () => {
      const filePath = cursorAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.cursor', 'commands', 'opsx-explore.md'));
    });

    it('should generate correct file paths for different commands', () => {
      expect(cursorAdapter.getFilePath('new')).toBe(path.join('.cursor', 'commands', 'opsx-new.md'));
      expect(cursorAdapter.getFilePath('bulk-archive')).toBe(path.join('.cursor', 'commands', 'opsx-bulk-archive.md'));
    });

    it('should format file with Cursor-specific frontmatter', () => {
      const output = cursorAdapter.formatFile(sampleContent);

      expect(output).toContain('---\n');
      expect(output).toContain('name: "/opsx-explore"');
      expect(output).toContain('id: "opsx-explore"');
      expect(output).toContain('category: "Workflow"');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });

    it('should not include tags in Cursor format', () => {
      const output = cursorAdapter.formatFile(sampleContent);
      expect(output).not.toContain('tags:');
    });
  });

  describe('commandCodeAdapter', () => {
    it('should have correct toolId', () => {
      expect(commandCodeAdapter.toolId).toBe('command-code');
    });

    it('should generate correct file path with opsx- prefix', () => {
      const filePath = commandCodeAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.commandcode', 'commands', 'opsx-explore.md'));
    });

    it('should format the documented plain Markdown command body', () => {
      const output = commandCodeAdapter.formatFile(sampleContent);
      expect(output).toBe(`${sampleContent.body}\n`);
      expect(output).not.toContain('description:');
    });

    it('should pass invocation arguments into the OpenSpec input contract', () => {
      const output = commandCodeAdapter.formatFile({
        ...sampleContent,
        body: '# OpenSpec command\n\n**Input**: A change name or description.\n\nRun the workflow.',
      });
      expect(output).toContain(
        '**Input**: A change name or description.\n**Provided arguments**: $ARGUMENTS'
      );
    });

    it.each(['$ARGUMENTS', '$@', '${ARGUMENTS}', '${@}'])(
      'should not duplicate an existing %s placeholder',
      (placeholder) => {
        const output = commandCodeAdapter.formatFile({
          ...sampleContent,
          body: `**Input**: A change name.\n**Provided arguments**: ${placeholder}`,
        });
        expect(output.match(/\*\*Provided arguments\*\*:/g)).toHaveLength(1);
      }
    );

    it('should preserve invocation arguments for every workflow that accepts them', () => {
      const commandsWithoutArguments = getCommandContents()
        .filter((content) => {
          const output = generateCommand(content, commandCodeAdapter).fileContent;
          return !output.includes('**Provided arguments**: $ARGUMENTS');
        })
        .map((content) => content.id);

      // Onboarding is deliberately interactive and has no invocation input.
      // This list is a tripwire for a new workflow that accidentally drops args.
      expect(commandsWithoutArguments).toEqual(['onboard']);
    });

    it('is generated by generateCommand with hyphen command references', () => {
      const contentWithCommands: CommandContent = {
        ...sampleContent,
        body: 'Use /opsx:new to start, then /opsx:apply to implement.',
      };
      const output = generateCommand(contentWithCommands, commandCodeAdapter).fileContent;
      expect(output).toContain('/opsx-new');
      expect(output).toContain('/opsx-apply');
      expect(output).not.toContain('/opsx:new');
      expect(output).not.toContain('/opsx:apply');
    });
  });

  describe('devinAdapter', () => {
    it('should have correct toolId', () => {
      expect(devinAdapter.toolId).toBe('devin');
    });

    it('should generate correct file path', () => {
      const filePath = devinAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.devin', 'workflows', 'opsx-explore.md'));
    });

    it('should format file with YAML frontmatter', () => {
      const output = devinAdapter.formatFile(sampleContent);

      expect(output).toContain('---\n');
      expect(output).toContain('name: "OpenSpec Explore"');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('category: "Workflow"');
      expect(output).toContain('tags: ["workflow", "explore", "experimental"]');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });

    // The body's `/opsx:*` references are rewritten to the `/opsx-*` form
    // Devin registers by the generator, not here — adapters are pure
    // formatters. Covered for devin in invocation.test.ts.

    // Frontmatter escaping comes from the shared yaml.ts helpers and is
    // covered for every registered adapter by the round-trip matrix in
    // "YAML frontmatter escaping across adapters" below.

    it('should handle empty tags', () => {
      const contentNoTags: CommandContent = { ...sampleContent, tags: [] };
      const output = devinAdapter.formatFile(contentNoTags);
      expect(output).toContain('tags: []');
    });
  });

  describe('amazonQAdapter', () => {
    it('should have correct toolId', () => {
      expect(amazonQAdapter.toolId).toBe('amazon-q');
    });

    it('should generate correct file path', () => {
      const filePath = amazonQAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.amazonq', 'prompts', 'opsx-explore.md'));
    });

    it('should format file with description frontmatter', () => {
      const output = amazonQAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });
  });

  describe('antigravityAdapter', () => {
    it('should have correct toolId', () => {
      expect(antigravityAdapter.toolId).toBe('antigravity');
    });

    it('should generate correct file path', () => {
      const filePath = antigravityAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.agents', 'workflows', 'opsx-explore.md'));
    });

    it('should format file with description frontmatter', () => {
      const output = antigravityAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });
  });

  describe('auggieAdapter', () => {
    it('should have correct toolId', () => {
      expect(auggieAdapter.toolId).toBe('auggie');
    });

    it('should generate correct file path', () => {
      const filePath = auggieAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.augment', 'commands', 'opsx-explore.md'));
    });

    it('should format file with description and argument-hint', () => {
      const output = auggieAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('argument-hint: command arguments');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });
  });


  describe('bobAdapter', () => {
    it('should have correct toolId', () => {
      expect(bobAdapter.toolId).toBe('bob');
    });

    it('should generate correct file path', () => {
      const filePath = bobAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.bob', 'commands', 'opsx-explore.md'));
    });

    it('should generate correct file paths for different commands', () => {
      expect(bobAdapter.getFilePath('new')).toBe(path.join('.bob', 'commands', 'opsx-new.md'));
      expect(bobAdapter.getFilePath('bulk-archive')).toBe(path.join('.bob', 'commands', 'opsx-bulk-archive.md'));
    });

    it('should format file with description and argument-hint frontmatter', () => {
      const output = bobAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('argument-hint: command arguments');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.\n\nWith multiple lines.');
    });

    it('is generated by generateCommand with hyphen command references', () => {
      const contentWithRefs: CommandContent = {
        ...sampleContent,
        body: 'Run /opsx:apply to implement. Then use /opsx:verify.',
      };
      const output = generateCommand(contentWithRefs, bobAdapter).fileContent;
      expect(output).toContain('/opsx-apply');
      expect(output).toContain('/opsx-verify');
      expect(output).not.toContain('/opsx:apply');
      expect(output).not.toContain('/opsx:verify');
    });

    it('should escape YAML special characters in description', () => {
      const contentWithSpecialChars: CommandContent = {
        ...sampleContent,
        description: 'Fix: regression in "auth" feature',
      };
      const output = bobAdapter.formatFile(contentWithSpecialChars);
      expect(output).toContain('description: "Fix: regression in \\"auth\\" feature"');
    });

    it('should escape newlines in description', () => {
      const contentWithNewline: CommandContent = {
        ...sampleContent,
        description: 'Line 1\nLine 2',
      };
      const output = bobAdapter.formatFile(contentWithNewline);
      expect(output).toContain('description: "Line 1\\nLine 2"');
    });

    it('should handle empty description', () => {
      const contentEmptyDesc: CommandContent = {
        ...sampleContent,
        description: '',
      };
      const output = bobAdapter.formatFile(contentEmptyDesc);
      expect(output).toContain('description: ""');
    });
  });

  describe('clineAdapter', () => {
    it('should have correct toolId', () => {
      expect(clineAdapter.toolId).toBe('cline');
    });

    it('should generate correct file path', () => {
      const filePath = clineAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.clinerules', 'workflows', 'opsx-explore.md'));
    });

    it('should format file with markdown header (no YAML frontmatter)', () => {
      const output = clineAdapter.formatFile(sampleContent);
      expect(output).toContain('# OpenSpec Explore');
      expect(output).toContain('Enter explore mode for thinking');
      expect(output).toContain('This is the command body.');
      expect(output).not.toContain('---');
    });
  });

  describe('codebuddyAdapter', () => {
    it('should have correct toolId', () => {
      expect(codebuddyAdapter.toolId).toBe('codebuddy');
    });

    it('should generate correct file path with nested opsx folder', () => {
      const filePath = codebuddyAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.codebuddy', 'commands', 'opsx', 'explore.md'));
    });

    it('should format file with name, description, and argument-hint', () => {
      const output = codebuddyAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('name: "OpenSpec Explore"');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('argument-hint: "[command arguments]"');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });
  });

  describe('continueAdapter', () => {
    it('should have correct toolId', () => {
      expect(continueAdapter.toolId).toBe('continue');
    });

    it('should generate correct file path with .prompt extension', () => {
      const filePath = continueAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.continue', 'prompts', 'opsx-explore.prompt'));
    });

    it('should format file with name, description, and invokable', () => {
      const output = continueAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('name: "opsx-explore"');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('invokable: true');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });
  });

  describe('costrictAdapter', () => {
    it('should have correct toolId', () => {
      expect(costrictAdapter.toolId).toBe('costrict');
    });

    it('should generate correct file path', () => {
      const filePath = costrictAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.cospec', 'openspec', 'commands', 'opsx-explore.md'));
    });

    it('should format file with description and argument-hint', () => {
      const output = costrictAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('argument-hint: command arguments');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });
  });

  describe('crushAdapter', () => {
    it('should have correct toolId', () => {
      expect(crushAdapter.toolId).toBe('crush');
    });

    it('should generate correct file path with nested opsx folder', () => {
      const filePath = crushAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.crush', 'commands', 'opsx', 'explore.md'));
    });

    it('should format file with name, description, category, and tags', () => {
      const output = crushAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('name: "OpenSpec Explore"');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('category: "Workflow"');
      expect(output).toContain('tags: ["workflow", "explore", "experimental"]');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });
  });

  describe('factoryAdapter', () => {
    it('should have correct toolId', () => {
      expect(factoryAdapter.toolId).toBe('factory');
    });

    it('should generate correct file path', () => {
      const filePath = factoryAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.factory', 'commands', 'opsx-explore.md'));
    });

    it('should format file with description and argument-hint', () => {
      const output = factoryAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('argument-hint: command arguments');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });
  });

  describe('geminiAdapter', () => {
    it('should have correct toolId', () => {
      expect(geminiAdapter.toolId).toBe('gemini');
    });

    it('should generate correct file path with .toml extension', () => {
      const filePath = geminiAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.gemini', 'commands', 'opsx', 'explore.toml'));
    });

    it('should format file in TOML format', () => {
      const output = geminiAdapter.formatFile(sampleContent);
      expect(output).toContain('description = "Enter explore mode for thinking"');
      expect(output).toContain('prompt = """');
      expect(output).toContain('This is the command body.');
      expect(output).toContain('"""');
    });

    it('escapes TOML-active characters in the description', () => {
      const output = geminiAdapter.formatFile({
        ...sampleContent,
        description: 'Say "hi" to C:\\Users and\nmore',
      });
      // Basic strings are escape-active: quotes, backslashes, and newlines
      // must be written as escapes or the file stops parsing as TOML.
      expect(output).toContain('description = "Say \\"hi\\" to C:\\\\Users and\\nmore"');
      expect((parseToml(output) as { description: string }).description).toBe(
        'Say "hi" to C:\\Users and\nmore'
      );
    });

    it('keeps the prompt a single multiline string when the body carries fences and backslashes', () => {
      const body = 'Windows path C:\\temp and a quote run: """ done';
      const output = geminiAdapter.formatFile({ ...sampleContent, body });
      // Backslashes must be escaped and no unescaped quote-triple may remain,
      // or the """ delimiter ends the prompt early.
      expect(output).toContain('C:\\\\temp');
      expect(output).toContain('""\\" done');
      const delimiters = output.match(/(?<!\\)"""/g) ?? [];
      expect(delimiters).toHaveLength(2);
      expect((parseToml(output) as { prompt: string }).prompt).toBe(`${body}\n`);
    });

    // Escaping claims are only proven by a real parser: every hostile body
    // must yield a file smol-toml accepts, and the parsed prompt must
    // round-trip to the original (modulo CRLF normalization).
    const HOSTILE_BODIES: Array<[string, string, string]> = [
      ['control characters', 'null:\u0000 vt:\u000b ff:\u000c end', 'null:\u0000 vt:\u000b ff:\u000c end'],
      // A lone CR is illegal raw in a multiline basic string (only LF and
      // CRLF may appear); Python tomllib rejects it — so must never be
      // emitted bare.
      ['a lone carriage return', 'a\rb', 'a\rb'],
      ['CRLF line endings (normalized to LF)', 'line one\r\nline two\r\n', 'line one\nline two\n'],
      ['a CR before a quote run', 'x\r""" y', 'x\r""" y'],
      ['a trailing backslash', 'ends with a backslash \\', 'ends with a backslash \\'],
      ['quote runs of four and five', 'four """" five """""', 'four """" five """""'],
    ];

    for (const [label, body, expected] of HOSTILE_BODIES) {
      it(`emits parseable TOML for a body with ${label}`, () => {
        const output = geminiAdapter.formatFile({ ...sampleContent, body });
        const parsed = parseToml(output) as { description: string; prompt: string };
        expect(parsed.prompt).toBe(`${expected}\n`);
        expect(parsed.description).toBe(sampleContent.description);
      });
    }
  });

  describe('githubCopilotAdapter', () => {
    it('should have correct toolId', () => {
      expect(githubCopilotAdapter.toolId).toBe('github-copilot');
    });

    it('should generate correct file path with .prompt.md extension', () => {
      const filePath = githubCopilotAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.github', 'prompts', 'opsx-explore.prompt.md'));
    });

    it('should format file with description frontmatter', () => {
      const output = githubCopilotAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });
  });

  describe('iflowAdapter', () => {
    it('should have correct toolId', () => {
      expect(iflowAdapter.toolId).toBe('iflow');
    });

    it('should generate correct file path', () => {
      const filePath = iflowAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.iflow', 'commands', 'opsx-explore.md'));
    });

    it('should format file with name, id, category, and description', () => {
      const output = iflowAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('name: "/opsx-explore"');
      expect(output).toContain('id: "opsx-explore"');
      expect(output).toContain('category: "Workflow"');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });
  });

  describe('kilocodeAdapter', () => {
    it('should have correct toolId', () => {
      expect(kilocodeAdapter.toolId).toBe('kilocode');
    });

    it('should generate correct file path', () => {
      const filePath = kilocodeAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.kilocode', 'workflows', 'opsx-explore.md'));
    });

    it('should format file without frontmatter', () => {
      const output = kilocodeAdapter.formatFile(sampleContent);
      expect(output).not.toContain('---');
      expect(output).toContain('This is the command body.');
    });
  });

  describe('opencodeAdapter', () => {
    it('should have correct toolId', () => {
      expect(opencodeAdapter.toolId).toBe('opencode');
    });

    it('should generate correct file path', () => {
      const filePath = opencodeAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.opencode', 'commands', 'opsx-explore.md'));
    });

    it('should format file with description frontmatter', () => {
      const output = opencodeAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });

    it('should pass invocation arguments into the OpenSpec input contract', () => {
      const output = opencodeAdapter.formatFile({
        ...sampleContent,
        body: '# OpenSpec command\n\n**Input**: A change name or description.\n\nRun the workflow.',
      });
      expect(output).toContain(
        '**Input**: A change name or description.\n**Provided arguments**: $ARGUMENTS'
      );
    });

    it('should not duplicate an existing $ARGUMENTS placeholder', () => {
      const output = opencodeAdapter.formatFile({
        ...sampleContent,
        body: '**Input**: A change name.\nExisting input: $ARGUMENTS',
      });
      expect(output.match(/\$ARGUMENTS/g)).toHaveLength(1);
    });

    it('should not duplicate documented positional argument placeholders', () => {
      const output = opencodeAdapter.formatFile({
        ...sampleContent,
        body: '**Input**: Two values.\nFirst: $1\nSecond: $2',
      });
      expect(output).not.toContain('**Provided arguments**: $ARGUMENTS');
      expect(output).toContain('First: $1\nSecond: $2');
    });

    it('should keep multi-line input guidance together before provided arguments', () => {
      const output = opencodeAdapter.formatFile({
        ...sampleContent,
        body: '**Input**: A topic, such as:\n- an idea\n- a problem\n\n**Steps**\n1. Explore.',
      });
      expect(output).toContain(
        '**Input**: A topic, such as:\n- an idea\n- a problem\n**Provided arguments**: $ARGUMENTS'
      );
    });

    it('should preserve CRLF while keeping multi-line input guidance together', () => {
      const output = opencodeAdapter.formatFile({
        ...sampleContent,
        body: '**Input**: A topic, such as:\r\n- an idea\r\n- a problem\r\n\r\nRun it.',
      });
      expect(output).toContain(
        '**Input**: A topic, such as:\r\n- an idea\r\n- a problem\r\n**Provided arguments**: $ARGUMENTS\r\n\r\nRun it.'
      );
    });

    it('should not add invocation arguments to an explicitly input-free workflow', () => {
      const output = opencodeAdapter.formatFile({
        ...sampleContent,
        body: '**Input**: None required (prompts for selection)\n\nPrompt the user.',
      });
      expect(output).not.toContain('$ARGUMENTS');
    });

    it('should preserve exactly one argument placeholder for each workflow that accepts input', () => {
      for (const content of getCommandContents()) {
        const output = generateCommand(content, opencodeAdapter).fileContent;
        const acceptsInput = /^\*\*Input\*\*:(?!\s*None required\b)/im.test(content.body);
        expect(output.match(/\$ARGUMENTS/g) ?? [], content.id).toHaveLength(acceptsInput ? 1 : 0);
      }
    });

    it('is generated by generateCommand with hyphen command references', () => {
      const contentWithCommands: CommandContent = {
        ...sampleContent,
        body: 'Use /opsx:new to start, then /opsx:apply to implement.',
      };
      const output = generateCommand(contentWithCommands, opencodeAdapter).fileContent;
      expect(output).toContain('/opsx-new');
      expect(output).toContain('/opsx-apply');
      expect(output).not.toContain('/opsx:new');
      expect(output).not.toContain('/opsx:apply');
    });

    it('is generated by generateCommand with every reference hyphenated', () => {
      const contentWithMultipleCommands: CommandContent = {
        ...sampleContent,
        body: `/opsx:explore for ideas
/opsx:new to create
/opsx:continue to proceed
/opsx:apply to implement`,
      };
      const output = generateCommand(contentWithMultipleCommands, opencodeAdapter).fileContent;
      expect(output).toContain('/opsx-explore');
      expect(output).toContain('/opsx-new');
      expect(output).toContain('/opsx-continue');
      expect(output).toContain('/opsx-apply');
    });
  });

  describe('qoderAdapter', () => {
    it('should have correct toolId', () => {
      expect(qoderAdapter.toolId).toBe('qoder');
    });

    it('should generate correct file path with nested opsx folder', () => {
      const filePath = qoderAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.qoder', 'commands', 'opsx', 'explore.md'));
    });

    it('should format file with name, description, category, and tags', () => {
      const output = qoderAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('name: "OpenSpec Explore"');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('category: "Workflow"');
      expect(output).toContain('tags: ["workflow", "explore", "experimental"]');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });
  });

  describe('qwenAdapter', () => {
    it('should have correct toolId', () => {
      expect(qwenAdapter.toolId).toBe('qwen');
    });

    it('should generate correct file path with .md extension', () => {
      const filePath = qwenAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.qwen', 'commands', 'opsx-explore.md'));
    });

    it('should format file with description frontmatter', () => {
      const output = qwenAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });

    it('should escape special YAML characters in description', () => {
      const output = qwenAdapter.formatFile({
        ...sampleContent,
        description: 'Review: plan & apply "changes"',
      });
      expect(output).toContain('description: "Review: plan & apply \\"changes\\""');
    });

    it('is generated by generateCommand with hyphen command references', () => {
      // Qwen commands are invoked by filename (/opsx-<id>), like bob/opencode.
      const contentWithRefs: CommandContent = {
        ...sampleContent,
        body: 'Run /opsx:apply to implement. Then use /opsx:archive.',
      };
      const output = generateCommand(contentWithRefs, qwenAdapter).fileContent;
      expect(output).toContain('/opsx-apply');
      expect(output).toContain('/opsx-archive');
      expect(output).not.toContain('/opsx:apply');
      expect(output).not.toContain('/opsx:archive');
    });
  });

  describe('piAdapter', () => {
    it('should have correct toolId', () => {
      expect(piAdapter.toolId).toBe('pi');
    });

    it('should generate correct file path', () => {
      const filePath = piAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.pi', 'prompts', 'opsx-explore.md'));
    });

    it('should generate correct file paths for different commands', () => {
      expect(piAdapter.getFilePath('new')).toBe(path.join('.pi', 'prompts', 'opsx-new.md'));
      expect(piAdapter.getFilePath('bulk-archive')).toBe(path.join('.pi', 'prompts', 'opsx-bulk-archive.md'));
    });

    it('should format file with description frontmatter', () => {
      const output = piAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });

    it('is generated by generateCommand with hyphen command references', () => {
      const contentWithRefs: CommandContent = {
        ...sampleContent,
        body: 'Run /opsx:apply to implement. Then /opsx:archive when done.',
      };

      const output = generateCommand(contentWithRefs, piAdapter).fileContent;
      expect(output).toContain('/opsx-apply');
      expect(output).toContain('/opsx-archive');
      expect(output).not.toContain('/opsx:apply');
    });

    it('should inject template arguments into the input section', () => {
      const contentWithInput: CommandContent = {
        ...sampleContent,
        body: '**Input**: The argument after `/opsx:explore` is the topic.\n\n**Steps**\n1. Think.',
      };

      const output = piAdapter.formatFile(contentWithInput);
      expect(output).toContain('**Provided arguments**: $@');
    });

    it('should escape YAML special characters in description', () => {
      const contentWithSpecialChars: CommandContent = {
        ...sampleContent,
        description: 'Fix: regression in "auth" feature',
      };
      const output = piAdapter.formatFile(contentWithSpecialChars);
      expect(output).toContain('description: "Fix: regression in \\"auth\\" feature"');
    });

    it('should escape newlines in description', () => {
      const contentWithNewline: CommandContent = {
        ...sampleContent,
        description: 'Line 1\nLine 2',
      };
      const output = piAdapter.formatFile(contentWithNewline);
      expect(output).toContain('description: "Line 1\\nLine 2"');
    });
  });

  describe('ohMyPiAdapter', () => {
    it('should have correct toolId', () => {
      expect(ohMyPiAdapter.toolId).toBe('oh-my-pi');
    });

    it('should generate correct file path', () => {
      const filePath = ohMyPiAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.omp', 'commands', 'opsx-explore.md'));
    });

    it('should generate correct file paths for different commands', () => {
      expect(ohMyPiAdapter.getFilePath('new')).toBe(path.join('.omp', 'commands', 'opsx-new.md'));
      expect(ohMyPiAdapter.getFilePath('bulk-archive')).toBe(path.join('.omp', 'commands', 'opsx-bulk-archive.md'));
    });

    it('should format file with description frontmatter', () => {
      const output = ohMyPiAdapter.formatFile(sampleContent);
      expect(output).toContain('---\n');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.');
    });

    it('is generated by generateCommand with hyphen command references', () => {
      const contentWithRefs: CommandContent = {
        ...sampleContent,
        body: 'Run /opsx:apply to implement. Then /opsx:archive when done.',
      };
      const output = generateCommand(contentWithRefs, ohMyPiAdapter).fileContent;
      expect(output).toContain('/opsx-apply');
      expect(output).toContain('/opsx-archive');
      expect(output).not.toContain('/opsx:apply');
    });

    it('should escape YAML special characters in description', () => {
      const contentWithSpecialChars: CommandContent = {
        ...sampleContent,
        description: 'Fix: regression in "auth" feature',
      };
      const output = ohMyPiAdapter.formatFile(contentWithSpecialChars);
      expect(output).toContain('description: "Fix: regression in \\"auth\\" feature"');
    });

    it('should escape newlines in description', () => {
      const contentWithNewline: CommandContent = {
        ...sampleContent,
        description: 'Line 1\nLine 2',
      };
      const output = ohMyPiAdapter.formatFile(contentWithNewline);
      expect(output).toContain('description: "Line 1\\nLine 2"');
    });

    it('should inject $@ after **Input**: heading when not already present', () => {
      const contentWithInput: CommandContent = {
        ...sampleContent,
        body: '**Input**: The argument is the change name.\n\nDo the work.',
      };
      const output = ohMyPiAdapter.formatFile(contentWithInput);
      expect(output).toContain('**Input**: The argument is the change name.\n**Provided arguments**: $@');
    });

    it('injects $@ alongside generateCommand\'s hyphen rewrite', () => {
      const contentWithInput: CommandContent = {
        ...sampleContent,
        body: '**Input**: The argument is the change name.\n\nRun /opsx:apply.',
      };
      const output = generateCommand(contentWithInput, ohMyPiAdapter).fileContent;
      expect(output).toContain('**Provided arguments**: $@');
      expect(output).toContain('/opsx-apply');
    });

    it('should not inject $@ when $@ is already present in the body', () => {
      const contentWithArgs: CommandContent = {
        ...sampleContent,
        body: '**Input**: Accepts arguments.\n\nUser said: $@',
      };
      const output = ohMyPiAdapter.formatFile(contentWithArgs);
      expect(output.match(/\$@/g)?.length).toBe(1);
    });

    it('should not inject $@ when $ARGUMENTS is already present in the body', () => {
      const contentWithArguments: CommandContent = {
        ...sampleContent,
        body: '**Input**: Accepts arguments.\n\nUser said: $ARGUMENTS',
      };
      const output = ohMyPiAdapter.formatFile(contentWithArguments);
      expect(output).not.toContain('$@');
    });
  });

  describe('roocodeAdapter', () => {
    it('should have correct toolId', () => {
      expect(roocodeAdapter.toolId).toBe('roocode');
    });

    it('should generate correct file path', () => {
      const filePath = roocodeAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.roo', 'commands', 'opsx-explore.md'));
    });

    it('should format file with markdown header (no YAML frontmatter)', () => {
      const output = roocodeAdapter.formatFile(sampleContent);
      expect(output).toContain('# OpenSpec Explore');
      expect(output).toContain('Enter explore mode for thinking');
      expect(output).toContain('This is the command body.');
      expect(output).not.toContain('---');
    });
  });

  describe('traeAdapter', () => {
    it('should have correct toolId', () => {
      expect(traeAdapter.toolId).toBe('trae');
    });

    it('should generate correct file path', () => {
      const filePath = traeAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.trae', 'commands', 'opsx-explore.md'));
    });

    it('should generate correct file paths for different commands', () => {
      expect(traeAdapter.getFilePath('new')).toBe(path.join('.trae', 'commands', 'opsx-new.md'));
      expect(traeAdapter.getFilePath('bulk-archive')).toBe(path.join('.trae', 'commands', 'opsx-bulk-archive.md'));
    });

    it('should format file with name and description frontmatter', () => {
      const output = traeAdapter.formatFile(sampleContent);

      expect(output).toContain('---\n');
      expect(output).toContain('name: "OpenSpec Explore"');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.\n\nWith multiple lines.');
    });

    it('should escape YAML special characters in name', () => {
      const contentWithSpecialChars: CommandContent = {
        ...sampleContent,
        name: 'Test: Command',
      };
      const output = traeAdapter.formatFile(contentWithSpecialChars);
      expect(output).toContain('name: "Test: Command"');
    });

    it('should escape YAML special characters in description', () => {
      const contentWithSpecialChars: CommandContent = {
        ...sampleContent,
        description: 'Fix: regression in "auth" feature',
      };
      const output = traeAdapter.formatFile(contentWithSpecialChars);
      expect(output).toContain('description: "Fix: regression in \\"auth\\" feature"');
    });

    it('should escape newlines in description', () => {
      const contentWithNewline: CommandContent = {
        ...sampleContent,
        description: 'Line 1\nLine 2',
      };
      const output = traeAdapter.formatFile(contentWithNewline);
      expect(output).toContain('description: "Line 1\\nLine 2"');
    });

    it('should handle empty description', () => {
      const contentEmptyDesc: CommandContent = {
        ...sampleContent,
        description: '',
      };
      const output = traeAdapter.formatFile(contentEmptyDesc);
      expect(output).toContain('description: ""');
    });

    it('should escape carriage returns in description', () => {
      const contentWithCR: CommandContent = {
        ...sampleContent,
        description: 'Line 1\r\nLine 2',
      };
      const output = traeAdapter.formatFile(contentWithCR);
      expect(output).toContain('description: "Line 1\\r\\nLine 2"');
    });
  });

  describe('zcodeAdapter', () => {
    it('should have correct toolId', () => {
      expect(zcodeAdapter.toolId).toBe('zcode');
    });

    it('should generate correct file path under .zcode/commands/opsx', () => {
      const filePath = zcodeAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.zcode', 'commands', 'opsx', 'explore.md'));
    });

    it('should generate correct file paths for different command IDs', () => {
      expect(zcodeAdapter.getFilePath('new')).toBe(path.join('.zcode', 'commands', 'opsx', 'new.md'));
      expect(zcodeAdapter.getFilePath('bulk-archive')).toBe(path.join('.zcode', 'commands', 'opsx', 'bulk-archive.md'));
    });

    it('should keep command paths under .zcode and never reference .agents', () => {
      for (const id of ['explore', 'new', 'apply', 'sync', 'archive', 'bulk-archive']) {
        const filePath = zcodeAdapter.getFilePath(id);
        expect(filePath).toContain('.zcode');
        expect(filePath).not.toContain('.agents');
      }
    });

    it('should format file with name, description, category, and tags frontmatter', () => {
      const output = zcodeAdapter.formatFile(sampleContent);

      expect(output).toContain('---\n');
      expect(output).toContain('name: "OpenSpec Explore"');
      expect(output).toContain('description: "Enter explore mode for thinking"');
      expect(output).toContain('category: "Workflow"');
      expect(output).toContain('tags: ["workflow", "explore", "experimental"]');
      expect(output).toContain('---\n\n');
      expect(output).toContain('This is the command body.\n\nWith multiple lines.');
    });

    it('should format empty tags as an empty YAML array', () => {
      const output = zcodeAdapter.formatFile({ ...sampleContent, tags: [] });
      expect(output).toContain('tags: []');
    });

    it('should escape colons in description by quoting the YAML value', () => {
      const output = zcodeAdapter.formatFile({
        ...sampleContent,
        description: 'Enter: explore mode',
      });
      expect(output).toContain('description: "Enter: explore mode"');
    });

    it('should escape double quotes in description', () => {
      const output = zcodeAdapter.formatFile({
        ...sampleContent,
        description: 'Enter "explore" mode',
      });
      expect(output).toContain('description: "Enter \\"explore\\" mode"');
    });

    it('should escape newlines in description', () => {
      const output = zcodeAdapter.formatFile({
        ...sampleContent,
        description: 'Line 1\nLine 2',
      });
      expect(output).toContain('description: "Line 1\\nLine 2"');
    });

    it('should escape special characters in name', () => {
      const output = zcodeAdapter.formatFile({
        ...sampleContent,
        name: 'OpenSpec: Explore',
      });
      expect(output).toContain('name: "OpenSpec: Explore"');
    });

    it('should escape special characters in category', () => {
      const output = zcodeAdapter.formatFile({
        ...sampleContent,
        category: 'Work #flow',
      });
      expect(output).toContain('category: "Work #flow"');
    });

    it('should quote individual tags that contain special characters', () => {
      const output = zcodeAdapter.formatFile({
        ...sampleContent,
        tags: ['workflow', 'explore:1', 'experimental'],
      });
      expect(output).toContain('tags: ["workflow", "explore:1", "experimental"]');
    });

    it('should escape backslashes when quoting is triggered by another special char', () => {
      // Backslash alone does not trigger quoting, but once quoting is on (via ':')
      // every backslash must be doubled. Locks the replace(/\\/g, '\\\\') branch.
      const output = zcodeAdapter.formatFile({
        ...sampleContent,
        description: 'path:C:\\foo\\bar',
      });
      expect(output).toContain('description: "path:C:\\\\foo\\\\bar"');
    });

    it('should quote values with leading or trailing whitespace', () => {
      const output = zcodeAdapter.formatFile({
        ...sampleContent,
        description: ' explore mode ',
      });
      expect(output).toContain('description: " explore mode "');
    });
  });

  describe('cross-platform path handling', () => {
    it('Claude adapter uses path.join for paths', () => {
      // path.join handles platform-specific separators
      const filePath = claudeAdapter.getFilePath('test');
      // On any platform, path.join returns the correct separator
      expect(filePath.split(path.sep)).toEqual(['.claude', 'commands', 'opsx', 'test.md']);
    });

    it('Cursor adapter uses path.join for paths', () => {
      const filePath = cursorAdapter.getFilePath('test');
      expect(filePath.split(path.sep)).toEqual(['.cursor', 'commands', 'opsx-test.md']);
    });

    it('Devin adapter uses path.join for paths', () => {
      const filePath = devinAdapter.getFilePath('test');
      expect(filePath.split(path.sep)).toEqual(['.devin', 'workflows', 'opsx-test.md']);
    });

    it('All adapters use path.join for paths', () => {
      // Verify all adapters produce valid paths
      const adapters = [
        amazonQAdapter, antigravityAdapter, auggieAdapter, bobAdapter, clineAdapter,
        codebuddyAdapter, continueAdapter, costrictAdapter,
        crushAdapter, factoryAdapter, geminiAdapter, githubCopilotAdapter,
        iflowAdapter, kilocodeAdapter, kiroAdapter, lingmaAdapter, ohMyPiAdapter,
        opencodeAdapter, piAdapter, qoderAdapter, qwenAdapter, roocodeAdapter,
        traeAdapter, zcodeAdapter
      ];
      for (const adapter of adapters) {
        const filePath = adapter.getFilePath('test');
        expect(filePath.length).toBeGreaterThan(0);
        expect(filePath.includes(path.sep) || filePath.includes('.')).toBe(true);
      }
    });
  });

  describe('codeassistantAdapter', () => {
    it('should have correct toolId', () => {
      expect(codeassistantAdapter.toolId).toBe('codeassistant');
    });

    it('should generate correct file path', () => {
      const filePath = codeassistantAdapter.getFilePath('explore');
      expect(filePath).toBe(path.join('.codeassistant', 'commands', 'opsx-explore.md'));
    });

    it('should generate correct file path for different command IDs', () => {
      expect(codeassistantAdapter.getFilePath('new')).toBe(path.join('.codeassistant', 'commands', 'opsx-new.md'));
      expect(codeassistantAdapter.getFilePath('bulk-archive')).toBe(path.join('.codeassistant', 'commands', 'opsx-bulk-archive.md'));
    });

    it('should format file with correct YAML frontmatter', () => {
      const output = codeassistantAdapter.formatFile(sampleContent);

      const frontmatter = output.match(/^---\n([\s\S]*?)\n---\n\n/);
      expect(frontmatter).not.toBeNull();
      expect(parseYaml(frontmatter![1])).toEqual({ description: sampleContent.description });
      expect(output.slice(frontmatter![0].length)).toBe(`${sampleContent.body}\n`);
    });

    it('generates registered commands with hyphenated workflow references', () => {
      const content: CommandContent = {
        ...sampleContent,
        body: 'Use /opsx:propose, /opsx:update, and /opsx:bulk-archive. Keep /opsx:unknown.',
      };
      const adapter = CommandAdapterRegistry.get('codeassistant');
      expect(adapter).toBe(codeassistantAdapter);
      const generated = generateCommand(content, adapter!);

      expect(generated.path).toBe(path.join('.codeassistant', 'commands', 'opsx-explore.md'));
      expect(generated.fileContent).toContain(
        'Use /opsx-propose, /opsx-update, and /opsx-bulk-archive. Keep /opsx:unknown.'
      );
      expect(content.body).toContain('/opsx:propose');
    });
  });

  describe('YAML frontmatter escaping across adapters', () => {
    // Derived from the registry, not hand-listed: a newly registered adapter
    // must be covered by default. Adding one that emits no YAML frontmatter is
    // then a deliberate act of adding it here.
    const NON_YAML_ADAPTERS = ['cline', 'command-code', 'kilocode', 'roocode', 'gemini'];
    const yamlAdapters = CommandAdapterRegistry.getAll().filter(
      (adapter) => !NON_YAML_ADAPTERS.includes(adapter.toolId)
    );

    /**
     * Builds a CommandContent whose every string field carries `marker`.
     */
    function contentWith(marker: string): CommandContent {
      return {
        id: 'explore',
        name: marker,
        description: marker,
        category: marker,
        tags: [marker, 'explore'],
        body: 'Body text',
      };
    }

    /**
     * Returns the frontmatter fields this adapter fills from CommandContent,
     * found by rendering two different markers and keeping the fields that
     * change. Fields derived from the command id (Cursor's `name`/`id`) or
     * emitted as constants stay put and are excluded.
     */
    function contentDerivedFields(adapter: ToolCommandAdapter): string[] {
      const render = (marker: string): Record<string, unknown> => {
        const match = adapter.formatFile(contentWith(marker)).match(/^---\n([\s\S]*?)\n---/);
        return (parseYaml(match![1]) ?? {}) as Record<string, unknown>;
      };
      // Deliberately different in length and shape. Two same-shaped markers
      // would render identically for a field derived via length or a slice,
      // and such a field would then be silently dropped from every assertion.
      const left = render('AAA');
      const right = render('zz-BBB-9-longer');
      return Object.keys(left).filter(
        (key) => JSON.stringify(left[key]) !== JSON.stringify(right[key])
      );
    }

    it('covers every registered YAML adapter', () => {
      const baseline = contentWith('Baseline');
      expect(yamlAdapters.length).toBeGreaterThan(0);
      for (const adapter of yamlAdapters) {
        expect(adapter.formatFile(baseline), adapter.toolId).toMatch(/^---\n/);
      }
      for (const toolId of NON_YAML_ADAPTERS) {
        const adapter = CommandAdapterRegistry.get(toolId);
        expect(adapter, `${toolId} is excluded but not registered`).toBeDefined();
        expect(adapter!.formatFile(baseline), toolId).not.toMatch(/^---\n/);
      }
    });

    const roundTripCases: Array<[string, string]> = [
      ['plain text', 'Enter explore mode for thinking'],
      ['empty string', ''],
      ['colon and quotes', 'Explore mode: "thinking" & planning (e.g. feature: dark-mode)'],
      ['block literal |', '|'],
      ['block literal |-', '|-'],
      ['block literal |+', '|+'],
      ['block folded >', '>'],
      ['block folded >-', '>-'],
      ['block folded >+', '>+'],
      ['block with text', '| block text'],
      ['folded with text', '> folded text'],
      ['boolean true', 'true'],
      ['boolean false', 'false'],
      ['boolean yes', 'yes'],
      ['boolean no', 'no'],
      ['boolean on', 'on'],
      ['boolean off', 'off'],
      ['null string', 'null'],
      ['tilde null', '~'],
      ['integer', '123'],
      ['zero', '0'],
      ['negative int', '-10'],
      ['float', '1.23'],
      ['scientific notation', '1e5'],
      ['hex integer', '0x12'],
      ['octal integer', '077'],
      ['binary integer', '0b101'],
      ['infinity', '.inf'],
      ['nan', '.nan'],
      ['special characters', '# comment: [a, b] {c: d} - item ? key *ref &anc !tag @at `cmd`'],
      ['leading space', ' leading'],
      ['trailing space', 'trailing '],
      ['multiple spaces', '   '],
      // Without these the matrix drives no control character at all, so the
      // escaping this suite exists to prove gets no adapter-level coverage —
      // and the raw-CR assertion below can never fail.
      ['carriage return', 'line 1\rline 2'],
      ['line feed', 'line 1\nline 2'],
      ['nul', 'a\x00b'],
      ['escape', 'ansi\x1b[0m'],
      ['delete', 'a\x7fb'],
      ['next line', 'a\x85b'],
    ];

    for (const adapter of yamlAdapters) {
      describe(`${adapter.toolId} adapter table-driven round-trip`, () => {
        for (const [label, testVal] of roundTripCases) {
          it(`preserves every string field and its type for ${label}`, () => {
            // Every string field carries the hostile value, not just
            // description: a field an adapter forgot to escape is only caught
            // if the matrix actually drives that field.
            const content: CommandContent = {
              id: 'explore',
              name: testVal,
              description: testVal,
              category: testVal,
              tags: [testVal, 'explore'],
              body: 'Body text',
            };

            const fileContent = adapter.formatFile(content);
            const frontmatterMatch = fileContent.match(/^---\n([\s\S]*?)\n---/);
            expect(frontmatterMatch).not.toBeNull();
            const frontmatter = frontmatterMatch![1];

            // A raw CR survives the parser but corrupts the file for anything
            // that splits on lines, so round-tripping alone would not catch it.
            expect(frontmatter, 'raw carriage return in frontmatter').not.toContain('\r');

            let parsed: Record<string, unknown> | undefined;
            expect(() => {
              parsed = parseYaml(frontmatter);
            }).not.toThrow();

            // Adapters emit different field subsets, and some derive name/id
            // from the command id rather than from the content. Identify the
            // content-derived fields by rendering a second time with a
            // different value and seeing which outputs move — a field that is
            // constant across both renders never carried our input, so it has
            // nothing to round-trip. This must not be softened into "skip the
            // field if it doesn't look like our value": a broken escape mangles
            // the value, and skipping on mismatch would skip the very bug.
            const contentFields = contentDerivedFields(adapter);
            expect(contentFields.length, `${adapter.toolId} emits no content fields`)
              .toBeGreaterThan(0);

            for (const field of contentFields) {
              if (field === 'tags') {
                expect(parsed!.tags, `${adapter.toolId}.tags`).toEqual([testVal, 'explore']);
                continue;
              }
              expect(parsed![field], `${adapter.toolId}.${field}`).toBe(testVal);
              expect(typeof parsed![field], `${adapter.toolId}.${field} type`).toBe('string');
            }
          });
        }
      });
    }
  });
});