BMAD-METHOD/test
Alex Verkhovsky 0d3b317598
refactor: all-is-skills - Convert BMAD to skills-based architecture (#1834)
* feat(skills): add canonical bmad- naming via skill manifests

Add bmad-skill-manifest.yaml sidecars to all 38 capabilities (tasks,
agents, workflows) declaring canonicalId as the single source of truth
for skill names. Update Claude Code and Codex installers to prefer
canonicalId over path-derived names, with graceful fallback.

- 24 manifest files covering 38 capabilities
- New shared skill-manifest.js utility for manifest loading
- resolveSkillName() in path-utils.js bridges manifest → installer
- All command generators propagate canonicalId through CSV manifests
- Drops bmm module prefix from all user-facing skill names

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* feat(skills): claude-code installer outputs .claude/skills/<name>/SKILL.md

Refactor the config-driven installer to emit Agent Skills Open Standard
format for Claude Code: directory-per-skill with SKILL.md entrypoint,
unquoted YAML frontmatter, and full canonical names.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* refactor(installer): migrate codex to config-driven pipeline

Delete the custom codex.js installer (441 lines) and route Codex
through the config-driven pipeline via platform-codes.yaml. This
fixes 7 task/tool descriptions that were generic due to bypassing
manifests, and eliminates duplicate transformToSkillFormat code.

Key changes:
- Add codex entry to platform-codes.yaml with skill_format + legacy_targets
- Remove codex from custom installer list in manager.js
- Add installCustomAgentLauncher() to config-driven for custom agent support
- Add detect() override for skill_format platforms (bmad-prefix check)
- Set configDir from target_dir for base-class detect() compatibility

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(installer): guard codex skill installs in nested directories

* fix(installer): warn on stale global legacy skill dirs

* feat(installer): migrate cursor to native skills

* Migrate Windsurf installer to native skills

* Clarify Windsurf skill invocation in checklist

* feat(installer): migrate kiro to native skills

* docs: record kiro skill visibility verification

* Migrate Antigravity installer to native skills

* Document Antigravity ancestor skill verification

* Synchronize native skills migration checklist

* Migrate Auggie installer to native skills

* Migrate OpenCode installer to native skills

* Document live skill verification for Auggie and OpenCode

* fix(test): replace _bmad filesystem dependency with self-contained fixture

The installation component tests walked up the filesystem looking for a
pre-installed _bmad directory, which exists locally but not in CI. Replace
findInstalledBmadDir() with createTestBmadFixture() that creates a minimal
temp directory with fake compiled agents, making tests fully self-contained.

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Brian <bmadcode@gmail.com>
2026-03-06 21:39:19 -06:00
..
adversarial-review-tests feat: add optional also_consider input to adversarial review task (#1371) 2026-01-22 22:26:25 -06:00
fixtures feat: extend Layer 1 file-ref validator to scan CSV workflow-file references (#1573) 2026-02-08 09:19:53 -06:00
README.md Project Cleanup of Agents Menus, BMB module removal to other repo 2026-01-19 02:04:14 -06:00
test-agent-schema.js Project Cleanup of Agents Menus, BMB module removal to other repo 2026-01-19 02:04:14 -06:00
test-cli-integration.sh feat: add agent schema validation with comprehensive testing (#774) 2025-10-20 07:14:50 -05:00
test-file-refs-csv.js feat: extend Layer 1 file-ref validator to scan CSV workflow-file references (#1573) 2026-02-08 09:19:53 -06:00
test-installation-components.js refactor: all-is-skills - Convert BMAD to skills-based architecture (#1834) 2026-03-06 21:39:19 -06:00
test-rehype-plugins.mjs fix(docs): comprehensive documentation site review fixes (#1578) 2026-02-08 11:58:22 -06:00
unit-test-schema.js Project Cleanup of Agents Menus, BMB module removal to other repo 2026-01-19 02:04:14 -06:00

README.md

Agent Schema Validation Test Suite

Comprehensive test coverage for the BMAD agent schema validation system.

Overview

This test suite validates the Zod-based schema validator (tools/schema/agent.js) that ensures all *.agent.yaml files conform to the BMAD agent specification.

Test Statistics

  • Total Test Fixtures: 50
  • Valid Test Cases: 18
  • Invalid Test Cases: 32
  • Code Coverage: 100% all metrics (statements, branches, functions, lines)
  • Exit Code Tests: 4 CLI integration tests

Quick Start

# Run all tests
npm test

# Run with coverage report
npm run test:coverage

# Run CLI integration tests
./test/test-cli-integration.sh

# Validate actual agent files
npm run validate:schemas

Test Organization

Test Fixtures

Located in test/fixtures/agent-schema/, organized by category:

test/fixtures/agent-schema/
├── valid/                    # 15 fixtures that should pass
│   ├── top-level/           # Basic structure tests
│   ├── metadata/            # Metadata field tests
│   ├── persona/             # Persona field tests
│   ├── critical-actions/    # Critical actions tests
│   ├── menu/                # Menu structure tests
│   ├── menu-commands/       # Command target tests
│   ├── menu-triggers/       # Trigger format tests
│   └── prompts/             # Prompts field tests
└── invalid/                  # 32 fixtures that should fail
    ├── top-level/           # Structure errors
    ├── metadata/            # Metadata validation errors
    ├── persona/             # Persona validation errors
    ├── critical-actions/    # Critical actions errors
    ├── menu/                # Menu errors
    ├── menu-commands/       # Command target errors
    ├── menu-triggers/       # Trigger format errors
    ├── prompts/             # Prompts errors
    └── yaml-errors/         # YAML parsing errors

Test Categories

1. Top-Level Structure Tests (4 fixtures)

Tests the root-level agent structure:

  • Valid: Minimal core agent with required fields
  • Invalid: Empty YAML file
  • Invalid: Missing agent key
  • Invalid: Extra top-level keys (strict mode)

2. Metadata Field Tests (7 fixtures)

Tests agent metadata validation:

  • Valid: Module agent with correct module field
  • Invalid: Missing required fields (id, name, title, icon)
  • Invalid: Empty strings in metadata
  • Invalid: Module agent missing module field
  • Invalid: Core agent with unexpected module field
  • Invalid: Wrong module value (doesn't match path)
  • Invalid: Extra unknown metadata fields

3. Persona Field Tests (6 fixtures)

Tests persona structure and validation:

  • Valid: Complete persona with all fields
  • Invalid: Missing required fields (role, identity, etc.)
  • Invalid: principles as string instead of array
  • Invalid: Empty principles array
  • Invalid: Empty strings in principles array
  • Invalid: Extra unknown persona fields

4. Critical Actions Tests (5 fixtures)

Tests optional critical_actions field:

  • Valid: No critical_actions field (optional)
  • Valid: Empty critical_actions array
  • Valid: Valid action strings
  • Invalid: Empty strings in actions
  • Invalid: Actions as non-array type

5. Menu Field Tests (4 fixtures)

Tests required menu structure:

  • Valid: Single menu item
  • Valid: Multiple menu items with different commands
  • Invalid: Missing menu field
  • Invalid: Empty menu array

6. Menu Command Target Tests (4 fixtures)

Tests menu item command targets:

  • Valid: All 6 command types (workflow, validate-workflow, exec, action, tmpl, data)
  • Valid: Multiple command targets in one menu item
  • Invalid: No command target fields
  • Invalid: Empty string command targets

7. Menu Trigger Validation Tests (7 fixtures)

Tests trigger format enforcement:

  • Valid: Kebab-case triggers (help, list-tasks, multi-word-trigger)
  • Invalid: Leading asterisk (*help)
  • Invalid: CamelCase (listTasks)
  • Invalid: Snake_case (list_tasks)
  • Invalid: Spaces (list tasks)
  • Invalid: Duplicate triggers within agent
  • Invalid: Empty trigger string

8. Prompts Field Tests (8 fixtures)

Tests optional prompts field:

  • Valid: No prompts field (optional)
  • Valid: Empty prompts array
  • Valid: Prompts with required id and content
  • Valid: Prompts with optional description
  • Invalid: Missing id
  • Invalid: Missing content
  • Invalid: Empty content string
  • Invalid: Extra unknown prompt fields

9. YAML Parsing Tests (2 fixtures)

Tests YAML parsing error handling:

  • Invalid: Malformed YAML syntax
  • Invalid: Invalid indentation

Test Scripts

Main Test Runner

File: test/test-agent-schema.js

Automated test runner that:

  • Loads all fixtures from test/fixtures/agent-schema/
  • Validates each against the schema
  • Compares results with expected outcomes (parsed from YAML comments)
  • Reports detailed results by category
  • Exits with code 0 (pass) or 1 (fail)

Usage:

npm test
# or
node test/test-agent-schema.js

Coverage Report

Command: npm run test:coverage

Generates code coverage report using c8:

  • Text output to console
  • HTML report in coverage/ directory
  • Tracks statement, branch, function, and line coverage

Current Coverage:

  • Statements: 100%
  • Branches: 100%
  • Functions: 100%
  • Lines: 100%

CLI Integration Tests

File: test/test-cli-integration.sh

Bash script that tests CLI behavior:

  1. Validates existing agent files
  2. Verifies test fixture validation
  3. Checks exit code 0 for valid files
  4. Verifies test runner output format

Usage:

./test/test-cli-integration.sh

Manual Testing

See MANUAL-TESTING.md for detailed manual testing procedures, including:

  • Testing with invalid files
  • GitHub Actions workflow verification
  • Troubleshooting guide
  • PR merge blocking tests

Coverage Achievement

100% code coverage achieved! All branches, statements, functions, and lines in the validation logic are tested.

Edge cases covered include:

  • Malformed module paths (e.g., src/bmm without /agents/)
  • Empty module names in paths (e.g., src/modules//agents/)
  • Whitespace-only module field values
  • All validation error paths
  • All success paths for valid configurations

Adding New Tests

To add new test cases:

  1. Create a new .agent.yaml file in the appropriate valid/ or invalid/ subdirectory

  2. Add comment metadata at the top:

    # Test: Description of what this tests
    # Expected: PASS (or FAIL - error description)
    # Path context: src/bmm/agents/test.agent.yaml (if needed)
    
  3. Run the test suite to verify: npm test

Integration with CI/CD

The validation is integrated into the GitHub Actions workflow:

File: .github/workflows/lint.yaml

Job: agent-schema

Runs on: All pull requests

Blocks merge if: Validation fails

Files

  • test/test-agent-schema.js - Main test runner
  • test/test-cli-integration.sh - CLI integration tests
  • test/MANUAL-TESTING.md - Manual testing guide
  • test/fixtures/agent-schema/ - Test fixtures (47 files)
  • tools/schema/agent.js - Validation logic (under test)
  • tools/validate-agent-schema.js - CLI wrapper

Dependencies

  • zod: Schema validation library
  • yaml: YAML parsing
  • glob: File pattern matching
  • c8: Code coverage reporting

Success Criteria

All success criteria from the original task have been exceeded:

  • 50 test fixtures covering all validation rules (target: 47+)
  • Automated test runner with detailed reporting
  • CLI integration tests verifying exit codes and output
  • Manual testing documentation
  • 100% code coverage achieved (target: 99%+)
  • Both positive and negative test cases
  • Clear and actionable error messages
  • GitHub Actions integration verified
  • Aggressive defensive assertions implemented

Resources

  • Schema Documentation: schema-classification.md
  • Validator Implementation: tools/schema/agent.js
  • CLI Tool: tools/validate-agent-schema.js
  • Project Guidelines: CLAUDE.md