mirror of
https://github.com/bmadcode/BMAD-METHOD.git
synced 2025-12-17 17:55:34 +00:00
chore: added CC PR review (#871)
* chore: added CC PR review * remove CLAUDE.md --------- Co-authored-by: Murat Ozcan <murat@mac.lan>
This commit is contained in:
parent
c20ead1acb
commit
bc76d25be6
203
.github/workflows/claude-code-review.yaml
vendored
Normal file
203
.github/workflows/claude-code-review.yaml
vendored
Normal file
@ -0,0 +1,203 @@
|
|||||||
|
# Sample Claude Code Review Workflow
|
||||||
|
#
|
||||||
|
# This is a template workflow that demonstrates how to set up automated code reviews
|
||||||
|
# using Claude via GitHub Actions. Customize the prompt and focus areas for your project.
|
||||||
|
#
|
||||||
|
# To use this workflow:
|
||||||
|
# 1. Use Claude Code command in your terminal: /install-github-app , this holds your hand throughout the setup
|
||||||
|
# 2. Copy this file over to your repository's .github/workflows/claude-code-review.yml , which gets auto-generated
|
||||||
|
# 3. Add ANTHROPIC_API_KEY to your repository secrets
|
||||||
|
# 4. Customize the prompt section for your project's specific needs
|
||||||
|
# 5. Adjust the focus areas, tools, and model as needed
|
||||||
|
|
||||||
|
name: Claude Code Review - BMAD Method
|
||||||
|
|
||||||
|
on:
|
||||||
|
pull_request:
|
||||||
|
types: [opened, synchronize, ready_for_review, reopened]
|
||||||
|
|
||||||
|
# if this branch is pushed back to back, cancel the older branch's workflow
|
||||||
|
concurrency:
|
||||||
|
group: ${{ github.workflow }}-${{ github.head_ref || github.ref }}
|
||||||
|
cancel-in-progress: true
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
claude-review:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
permissions:
|
||||||
|
contents: read
|
||||||
|
pull-requests: write
|
||||||
|
issues: read
|
||||||
|
id-token: write
|
||||||
|
|
||||||
|
steps:
|
||||||
|
- name: Checkout repository
|
||||||
|
uses: actions/checkout@v4
|
||||||
|
with:
|
||||||
|
fetch-depth: 1
|
||||||
|
|
||||||
|
- name: Run Claude Code Review
|
||||||
|
id: claude-review
|
||||||
|
uses: anthropics/claude-code-action@v1
|
||||||
|
with:
|
||||||
|
# Using API key for per-token billing plan
|
||||||
|
anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
|
||||||
|
|
||||||
|
# Track progress creates a comment showing review progress
|
||||||
|
track_progress: true
|
||||||
|
|
||||||
|
prompt: |
|
||||||
|
REPO: ${{ github.repository }}
|
||||||
|
PR NUMBER: ${{ github.event.pull_request.number }}
|
||||||
|
|
||||||
|
# BMAD-METHOD Repository - AI Agent Framework
|
||||||
|
|
||||||
|
IMPORTANT: Skip reviewing files in these directories:
|
||||||
|
- docs/ (user-facing documentation)
|
||||||
|
- bmad/ (compiled installation output, not source)
|
||||||
|
- test/fixtures/ (test data files)
|
||||||
|
- node_modules/ (dependencies)
|
||||||
|
|
||||||
|
**Context:** This is BMAD-CORE, a universal human-AI collaboration framework with YAML-based agent definitions and XML-tagged workflow instructions.
|
||||||
|
|
||||||
|
Perform comprehensive code review focusing on BMAD-specific patterns:
|
||||||
|
|
||||||
|
## 1. Agent YAML Schema Compliance (CRITICAL)
|
||||||
|
|
||||||
|
**For files in `src/modules/*/agents/*.agent.yaml`:**
|
||||||
|
- ✅ Required fields: metadata (id, name, title, icon, module), persona (role, identity, communication_style, principles), menu items
|
||||||
|
- ✅ Menu triggers must reference valid workflow paths: `{project-root}/bmad/{module}/workflows/{path}/workflow.yaml`
|
||||||
|
- ✅ Critical actions syntax (if TEA agent): Must reference tea-index.csv and knowledge fragments
|
||||||
|
- ✅ Schema validation: Run `npm run validate:schemas` to verify compliance
|
||||||
|
- ❌ No hardcoded file paths outside {project-root} or {installed_path}
|
||||||
|
- ❌ No duplicate menu triggers within an agent
|
||||||
|
|
||||||
|
## 2. Workflow Definition Integrity
|
||||||
|
|
||||||
|
**For files in `src/modules/*/workflows/**/workflow.yaml`:**
|
||||||
|
- ✅ Required fields: name, config_source, instructions, default_output_file (if template-based)
|
||||||
|
- ✅ Variable resolution: Use {config_source}, {project-root}, {installed_path}, {output_folder}
|
||||||
|
- ✅ Instructions path must exist: `{installed_path}/instructions.md`
|
||||||
|
- ✅ Template path (if template workflow): `{installed_path}/template.md`
|
||||||
|
- ❌ No absolute paths - use variable placeholders
|
||||||
|
|
||||||
|
**For `instructions.md` files:**
|
||||||
|
- ✅ XML tag syntax: `<step n="1">`, `<action>`, `<template-output>section</template-output>`, `<check if="condition">`
|
||||||
|
- ✅ Steps must have sequential numbering (1, 2, 3...)
|
||||||
|
- ✅ All XML tags must close properly (e.g., `</check>`, `</step>`)
|
||||||
|
- ✅ Template-output tags reference actual template sections
|
||||||
|
- ❌ No malformed XML that breaks workflow execution engine
|
||||||
|
|
||||||
|
## 3. TEA Knowledge Base Integrity
|
||||||
|
|
||||||
|
**For changes in `src/modules/bmm/testarch/`:**
|
||||||
|
- ✅ tea-index.csv must match knowledge/ directory (21 fragments indexed)
|
||||||
|
- ✅ Fragment file names match csv entries exactly
|
||||||
|
- ✅ TEA agent critical_actions reference tea-index.csv correctly
|
||||||
|
- ✅ Knowledge fragments maintain consistent format
|
||||||
|
- ❌ Don't break the index-fragment relationship
|
||||||
|
|
||||||
|
## 4. Documentation Consistency (Phase & Track Terminology)
|
||||||
|
|
||||||
|
**For changes in `src/modules/bmm/docs/`:**
|
||||||
|
- ✅ Use 3-track terminology: Quick Flow, BMad Method, Enterprise Method (not Level 0-4)
|
||||||
|
- ✅ Phase numbering: Phase 1 (Analysis), Phase 2 (Planning), Phase 3 (Solutioning), Phase 4 (Implementation)
|
||||||
|
- ✅ TEA operates in Phase 2 and Phase 4 only (not "all phases")
|
||||||
|
- ✅ `*test-design` is per-epic in Phase 4 (not per-project in Phase 2/3)
|
||||||
|
- ❌ Don't mix YAML phase numbers (0-indexed) with doc phase numbers (1-indexed) without context
|
||||||
|
|
||||||
|
**For changes in workflow-status YAML paths:**
|
||||||
|
- ✅ Only include phase-gate workflows (prd, architecture, sprint-planning)
|
||||||
|
- ❌ Don't include per-epic/per-story workflows (test-design, create-story, atdd, automate)
|
||||||
|
- Note: Per-epic/per-story workflows tracked in sprint-status.yaml, not workflow-status.yaml
|
||||||
|
|
||||||
|
## 5. Cross-Module Dependencies
|
||||||
|
|
||||||
|
- ✅ Verify workflow invocations reference valid paths
|
||||||
|
- ✅ Module dependencies declared in installer-manifest.yaml
|
||||||
|
- ✅ Shared task references resolve correctly
|
||||||
|
- ❌ No circular dependencies between modules
|
||||||
|
|
||||||
|
## 6. Compilation & Installation
|
||||||
|
|
||||||
|
**For changes affecting `tools/cli/`:**
|
||||||
|
- ✅ Agent compilation: YAML → Markdown/XML for both IDE and web bundle targets
|
||||||
|
- ✅ forWebBundle flag changes compilation behavior (inline vs file paths)
|
||||||
|
- ✅ Manifest generation creates agent-manifest.csv and workflow-manifest.csv
|
||||||
|
- ✅ Platform-specific hooks execute for IDE integrations
|
||||||
|
|
||||||
|
## 7. Code Quality (Node.js/JavaScript)
|
||||||
|
|
||||||
|
- ✅ Modern JavaScript (ES6+, async/await, proper error handling)
|
||||||
|
- ✅ Schema validation with Zod where applicable
|
||||||
|
- ✅ Proper YAML parsing with js-yaml
|
||||||
|
- ✅ File operations use fs-extra for better error handling
|
||||||
|
- ❌ No synchronous file I/O in async contexts
|
||||||
|
|
||||||
|
## Review Guidelines
|
||||||
|
|
||||||
|
- Reference CLAUDE.md for repository architecture
|
||||||
|
- Check CONTRIBUTING.md for contribution guidelines
|
||||||
|
- **Validation commands** (deterministic tests):
|
||||||
|
- `npm test` - Comprehensive quality checks (all validations + linting + formatting)
|
||||||
|
- `npm run test:schemas` - Agent schema validation tests (fixture-based)
|
||||||
|
- `npm run test:install` - Installation component tests (compilation)
|
||||||
|
- `npm run validate:schemas` - YAML schema validation
|
||||||
|
- `npm run validate:bundles` - Web bundle integrity
|
||||||
|
- `npm run lint` - ESLint compliance
|
||||||
|
- `npm run format:check` - Prettier formatting
|
||||||
|
- Prioritize issues: **Critical** (breaks workflows/compilation) > **High** (schema violations) > **Medium** (inconsistency) > **Low** (style)
|
||||||
|
- Be specific with file paths and line numbers
|
||||||
|
|
||||||
|
Use `gh pr comment` with your Bash tool to leave your review as a comment on the PR.
|
||||||
|
|
||||||
|
# Using Sonnet 4.5 for comprehensive reviews
|
||||||
|
# Available models: claude-opus-4-1-20250805, claude-sonnet-4-5-20250929, etc.
|
||||||
|
# Tools can be restricted based on what review actions you want to allow
|
||||||
|
claude_args: '--model claude-sonnet-4-5-20250929 --allowed-tools "mcp__github_inline_comment__create_inline_comment,Bash(gh issue view:*),Bash(gh search:*),Bash(gh issue list:*),Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*)"'
|
||||||
|
|
||||||
|
# SETUP INSTRUCTIONS
|
||||||
|
# ==================
|
||||||
|
#
|
||||||
|
# 1. Repository Secrets Setup:
|
||||||
|
# - Go to your repository <20> Settings <20> Secrets and variables <20> Actions
|
||||||
|
# - Click "New repository secret"
|
||||||
|
# - Name: ANTHROPIC_API_KEY
|
||||||
|
# - Value: Your Anthropic API key (get one from https://console.anthropic.com/)
|
||||||
|
#
|
||||||
|
# 2. Permissions:
|
||||||
|
# - The workflow needs 'pull-requests: write' to comment on PRs
|
||||||
|
# - The workflow needs 'contents: read' to access repository code
|
||||||
|
# - The workflow needs 'issues: read' for GitHub CLI operations
|
||||||
|
#
|
||||||
|
# 3. Customization:
|
||||||
|
# - Update the prompt section to match your project's needs
|
||||||
|
# - Add project-specific file/directory exclusions
|
||||||
|
# - Customize the focus areas based on your tech stack
|
||||||
|
# - Adjust the model (opus for more thorough reviews, sonnet for faster)
|
||||||
|
# - Modify allowed tools based on what actions you want Claude to perform
|
||||||
|
#
|
||||||
|
# 4. Testing:
|
||||||
|
# - Create a test PR to verify the workflow runs correctly
|
||||||
|
# - Check that Claude can comment on the PR
|
||||||
|
# - Ensure the review quality meets your standards
|
||||||
|
#
|
||||||
|
# 5. Advanced Customization:
|
||||||
|
# - Add conditional logic based on file types or changes
|
||||||
|
# - Integrate with other GitHub Actions (linting, testing, etc.)
|
||||||
|
# - Set up different review levels based on PR size or author
|
||||||
|
# - Add custom review templates for different types of changes
|
||||||
|
#
|
||||||
|
# TROUBLESHOOTING
|
||||||
|
# ===============
|
||||||
|
#
|
||||||
|
# Common Issues:
|
||||||
|
# - "Authentication failed" <20> Check ANTHROPIC_API_KEY secret
|
||||||
|
# - "Permission denied" <20> Verify workflow permissions in job definition
|
||||||
|
# - "No comments posted" <20> Check allowed tools and gh CLI permissions
|
||||||
|
# - "Review too generic" <20> Customize prompt with project-specific guidance
|
||||||
|
#
|
||||||
|
# For more help:
|
||||||
|
# - GitHub Actions documentation: https://docs.github.com/en/actions
|
||||||
|
# - Claude Code Action: https://github.com/anthropics/claude-code-action
|
||||||
|
# - Anthropic API documentation: https://docs.anthropic.com/
|
||||||
61
.github/workflows/lint.yaml
vendored
61
.github/workflows/lint.yaml
vendored
@ -1,61 +0,0 @@
|
|||||||
name: lint
|
|
||||||
|
|
||||||
"on":
|
|
||||||
pull_request:
|
|
||||||
branches: ["**"]
|
|
||||||
workflow_dispatch:
|
|
||||||
|
|
||||||
jobs:
|
|
||||||
prettier:
|
|
||||||
runs-on: ubuntu-latest
|
|
||||||
steps:
|
|
||||||
- name: Checkout
|
|
||||||
uses: actions/checkout@v4
|
|
||||||
|
|
||||||
- name: Setup Node
|
|
||||||
uses: actions/setup-node@v4
|
|
||||||
with:
|
|
||||||
node-version-file: ".nvmrc"
|
|
||||||
cache: "npm"
|
|
||||||
|
|
||||||
- name: Install dependencies
|
|
||||||
run: npm ci
|
|
||||||
|
|
||||||
- name: Prettier format check
|
|
||||||
run: npm run format:check
|
|
||||||
|
|
||||||
eslint:
|
|
||||||
runs-on: ubuntu-latest
|
|
||||||
steps:
|
|
||||||
- name: Checkout
|
|
||||||
uses: actions/checkout@v4
|
|
||||||
|
|
||||||
- name: Setup Node
|
|
||||||
uses: actions/setup-node@v4
|
|
||||||
with:
|
|
||||||
node-version-file: ".nvmrc"
|
|
||||||
cache: "npm"
|
|
||||||
|
|
||||||
- name: Install dependencies
|
|
||||||
run: npm ci
|
|
||||||
|
|
||||||
- name: ESLint
|
|
||||||
run: npm run lint
|
|
||||||
|
|
||||||
schema-validation:
|
|
||||||
runs-on: ubuntu-latest
|
|
||||||
steps:
|
|
||||||
- name: Checkout
|
|
||||||
uses: actions/checkout@v4
|
|
||||||
|
|
||||||
- name: Setup Node
|
|
||||||
uses: actions/setup-node@v4
|
|
||||||
with:
|
|
||||||
node-version-file: ".nvmrc"
|
|
||||||
cache: "npm"
|
|
||||||
|
|
||||||
- name: Install dependencies
|
|
||||||
run: npm ci
|
|
||||||
|
|
||||||
- name: Validate YAML schemas
|
|
||||||
run: npm run validate:schemas
|
|
||||||
123
.github/workflows/quality.yaml
vendored
Normal file
123
.github/workflows/quality.yaml
vendored
Normal file
@ -0,0 +1,123 @@
|
|||||||
|
name: Quality & Validation
|
||||||
|
|
||||||
|
# Runs comprehensive quality checks on all PRs:
|
||||||
|
# - Prettier (formatting)
|
||||||
|
# - ESLint (linting)
|
||||||
|
# - Schema validation (YAML structure)
|
||||||
|
# - Agent schema tests (fixture-based validation)
|
||||||
|
# - Installation component tests (compilation)
|
||||||
|
# - Bundle validation (web bundle integrity)
|
||||||
|
|
||||||
|
"on":
|
||||||
|
pull_request:
|
||||||
|
branches: ["**"]
|
||||||
|
workflow_dispatch:
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
prettier:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- name: Checkout
|
||||||
|
uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- name: Setup Node
|
||||||
|
uses: actions/setup-node@v4
|
||||||
|
with:
|
||||||
|
node-version-file: ".nvmrc"
|
||||||
|
cache: "npm"
|
||||||
|
|
||||||
|
- name: Install dependencies
|
||||||
|
run: npm ci
|
||||||
|
|
||||||
|
- name: Prettier format check
|
||||||
|
run: npm run format:check
|
||||||
|
|
||||||
|
eslint:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- name: Checkout
|
||||||
|
uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- name: Setup Node
|
||||||
|
uses: actions/setup-node@v4
|
||||||
|
with:
|
||||||
|
node-version-file: ".nvmrc"
|
||||||
|
cache: "npm"
|
||||||
|
|
||||||
|
- name: Install dependencies
|
||||||
|
run: npm ci
|
||||||
|
|
||||||
|
- name: ESLint
|
||||||
|
run: npm run lint
|
||||||
|
|
||||||
|
schema-validation:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- name: Checkout
|
||||||
|
uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- name: Setup Node
|
||||||
|
uses: actions/setup-node@v4
|
||||||
|
with:
|
||||||
|
node-version-file: ".nvmrc"
|
||||||
|
cache: "npm"
|
||||||
|
|
||||||
|
- name: Install dependencies
|
||||||
|
run: npm ci
|
||||||
|
|
||||||
|
- name: Validate YAML schemas
|
||||||
|
run: npm run validate:schemas
|
||||||
|
|
||||||
|
agent-schema-tests:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- name: Checkout
|
||||||
|
uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- name: Setup Node
|
||||||
|
uses: actions/setup-node@v4
|
||||||
|
with:
|
||||||
|
node-version-file: ".nvmrc"
|
||||||
|
cache: "npm"
|
||||||
|
|
||||||
|
- name: Install dependencies
|
||||||
|
run: npm ci
|
||||||
|
|
||||||
|
- name: Run agent schema validation tests
|
||||||
|
run: npm run test:schemas
|
||||||
|
|
||||||
|
installation-components:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- name: Checkout
|
||||||
|
uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- name: Setup Node
|
||||||
|
uses: actions/setup-node@v4
|
||||||
|
with:
|
||||||
|
node-version-file: ".nvmrc"
|
||||||
|
cache: "npm"
|
||||||
|
|
||||||
|
- name: Install dependencies
|
||||||
|
run: npm ci
|
||||||
|
|
||||||
|
- name: Test agent compilation components
|
||||||
|
run: npm run test:install
|
||||||
|
|
||||||
|
bundle-validation:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- name: Checkout
|
||||||
|
uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- name: Setup Node
|
||||||
|
uses: actions/setup-node@v4
|
||||||
|
with:
|
||||||
|
node-version-file: ".nvmrc"
|
||||||
|
cache: "npm"
|
||||||
|
|
||||||
|
- name: Install dependencies
|
||||||
|
run: npm ci
|
||||||
|
|
||||||
|
- name: Validate web bundles
|
||||||
|
run: npm run validate:bundles
|
||||||
2
.gitignore
vendored
2
.gitignore
vendored
@ -34,7 +34,7 @@ Thumbs.db
|
|||||||
.bmad*/.cursor/
|
.bmad*/.cursor/
|
||||||
|
|
||||||
# AI assistant files
|
# AI assistant files
|
||||||
# CLAUDE.md # we need this for Claude Code reviews
|
CLAUDE.md
|
||||||
.ai/*
|
.ai/*
|
||||||
cursor
|
cursor
|
||||||
.gemini
|
.gemini
|
||||||
|
|||||||
@ -1,3 +1,7 @@
|
|||||||
#!/usr/bin/env sh
|
#!/usr/bin/env sh
|
||||||
|
|
||||||
|
# Auto-fix changed files and stage them
|
||||||
npx --no-install lint-staged
|
npx --no-install lint-staged
|
||||||
|
|
||||||
|
# Validate everything
|
||||||
|
npm test
|
||||||
|
|||||||
60
README.md
60
README.md
@ -35,11 +35,15 @@ The **BMad-CORE** powers the **BMad Method** (probably why you're here!), but yo
|
|||||||
- [C.O.R.E. Philosophy](#core-philosophy)
|
- [C.O.R.E. Philosophy](#core-philosophy)
|
||||||
- [Modules](#modules)
|
- [Modules](#modules)
|
||||||
- [BMad Method (BMM) - AI-Driven Agile Development](#bmad-method-bmm---ai-driven-agile-development)
|
- [BMad Method (BMM) - AI-Driven Agile Development](#bmad-method-bmm---ai-driven-agile-development)
|
||||||
|
- [v6 Highlights](#v6-highlights)
|
||||||
|
- [🚀 Quick Start](#-quick-start)
|
||||||
- [BMad Builder (BMB) - Create Custom Solutions](#bmad-builder-bmb---create-custom-solutions)
|
- [BMad Builder (BMB) - Create Custom Solutions](#bmad-builder-bmb---create-custom-solutions)
|
||||||
- [Creative Intelligence Suite (CIS) - Innovation \& Creativity](#creative-intelligence-suite-cis---innovation--creativity)
|
- [Creative Intelligence Suite (CIS) - Innovation \& Creativity](#creative-intelligence-suite-cis---innovation--creativity)
|
||||||
- [🚀 Quick Start](#-quick-start)
|
|
||||||
- [Installation](#installation)
|
- [Installation](#installation)
|
||||||
- [🎯 Working with Agents \& Commands](#-working-with-agents--commands)
|
- [🎯 Working with Agents \& Commands](#-working-with-agents--commands)
|
||||||
|
- [Method 1: Agent Menu (Recommended for Beginners)](#method-1-agent-menu-recommended-for-beginners)
|
||||||
|
- [Method 2: Direct Slash Commands](#method-2-direct-slash-commands)
|
||||||
|
- [Method 3: Party Mode Execution](#method-3-party-mode-execution)
|
||||||
- [Key Features](#key-features)
|
- [Key Features](#key-features)
|
||||||
- [🎨 Update-Safe Customization](#-update-safe-customization)
|
- [🎨 Update-Safe Customization](#-update-safe-customization)
|
||||||
- [🚀 Intelligent Installation](#-intelligent-installation)
|
- [🚀 Intelligent Installation](#-intelligent-installation)
|
||||||
@ -47,6 +51,10 @@ The **BMad-CORE** powers the **BMad Method** (probably why you're here!), but yo
|
|||||||
- [📄 Document Sharding (Advanced)](#-document-sharding-advanced)
|
- [📄 Document Sharding (Advanced)](#-document-sharding-advanced)
|
||||||
- [Documentation](#documentation)
|
- [Documentation](#documentation)
|
||||||
- [Community \& Support](#community--support)
|
- [Community \& Support](#community--support)
|
||||||
|
- [Development \& Quality Checks](#development--quality-checks)
|
||||||
|
- [Testing \& Validation](#testing--validation)
|
||||||
|
- [Code Quality](#code-quality)
|
||||||
|
- [Build \& Development](#build--development)
|
||||||
- [Contributing](#contributing)
|
- [Contributing](#contributing)
|
||||||
- [License](#license)
|
- [License](#license)
|
||||||
|
|
||||||
@ -352,6 +360,56 @@ Optional optimization for large projects (BMad Method and Enterprise tracks):
|
|||||||
|
|
||||||
---
|
---
|
||||||
|
|
||||||
|
## Development & Quality Checks
|
||||||
|
|
||||||
|
**For contributors working on the BMAD codebase:**
|
||||||
|
|
||||||
|
**Requirements:** Node.js 22+ (see `.nvmrc`). Run `nvm use` to switch to the correct version.
|
||||||
|
|
||||||
|
### Testing & Validation
|
||||||
|
|
||||||
|
```bash
|
||||||
|
# Run all quality checks (comprehensive - use before pushing)
|
||||||
|
npm test
|
||||||
|
|
||||||
|
# Individual test suites
|
||||||
|
npm run test:schemas # Agent schema validation (fixture-based)
|
||||||
|
npm run test:install # Installation component tests (compilation)
|
||||||
|
npm run validate:schemas # YAML schema validation
|
||||||
|
npm run validate:bundles # Web bundle integrity
|
||||||
|
```
|
||||||
|
|
||||||
|
### Code Quality
|
||||||
|
|
||||||
|
```bash
|
||||||
|
# Lint check
|
||||||
|
npm run lint
|
||||||
|
|
||||||
|
# Auto-fix linting issues
|
||||||
|
npm run lint:fix
|
||||||
|
|
||||||
|
# Format check
|
||||||
|
npm run format:check
|
||||||
|
|
||||||
|
# Auto-format all files
|
||||||
|
npm run format:fix
|
||||||
|
```
|
||||||
|
|
||||||
|
### Build & Development
|
||||||
|
|
||||||
|
```bash
|
||||||
|
# Bundle for web deployment
|
||||||
|
npm run bundle
|
||||||
|
|
||||||
|
# Test local installation
|
||||||
|
npm run install:bmad
|
||||||
|
```
|
||||||
|
|
||||||
|
**Pre-commit Hook:** Auto-fixes changed files (lint-staged) + validates everything (npm test)
|
||||||
|
**CI:** GitHub Actions runs all quality checks in parallel on every PR
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
## Contributing
|
## Contributing
|
||||||
|
|
||||||
We welcome contributions! See **[CONTRIBUTING.md](CONTRIBUTING.md)** for:
|
We welcome contributions! See **[CONTRIBUTING.md](CONTRIBUTING.md)** for:
|
||||||
|
|||||||
4
package-lock.json
generated
4
package-lock.json
generated
@ -1,12 +1,12 @@
|
|||||||
{
|
{
|
||||||
"name": "bmad-method",
|
"name": "bmad-method",
|
||||||
"version": "6.0.0-alpha.5",
|
"version": "6.0.0-alpha.6",
|
||||||
"lockfileVersion": 3,
|
"lockfileVersion": 3,
|
||||||
"requires": true,
|
"requires": true,
|
||||||
"packages": {
|
"packages": {
|
||||||
"": {
|
"": {
|
||||||
"name": "bmad-method",
|
"name": "bmad-method",
|
||||||
"version": "6.0.0-alpha.5",
|
"version": "6.0.0-alpha.6",
|
||||||
"license": "MIT",
|
"license": "MIT",
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
"@kayvan/markdown-tree-parser": "^1.6.1",
|
"@kayvan/markdown-tree-parser": "^1.6.1",
|
||||||
|
|||||||
@ -39,8 +39,10 @@
|
|||||||
"release:minor": "gh workflow run \"Manual Release\" -f version_bump=minor",
|
"release:minor": "gh workflow run \"Manual Release\" -f version_bump=minor",
|
||||||
"release:patch": "gh workflow run \"Manual Release\" -f version_bump=patch",
|
"release:patch": "gh workflow run \"Manual Release\" -f version_bump=patch",
|
||||||
"release:watch": "gh run watch",
|
"release:watch": "gh run watch",
|
||||||
"test": "node test/test-agent-schema.js",
|
"test": "npm run test:schemas && npm run test:install && npm run validate:bundles && npm run validate:schemas && npm run lint && npm run format:check",
|
||||||
"test:coverage": "c8 --reporter=text --reporter=html node test/test-agent-schema.js",
|
"test:coverage": "c8 --reporter=text --reporter=html npm run test:schemas",
|
||||||
|
"test:install": "node test/test-installation-components.js",
|
||||||
|
"test:schemas": "node test/test-agent-schema.js",
|
||||||
"validate:bundles": "node tools/validate-bundles.js",
|
"validate:bundles": "node tools/validate-bundles.js",
|
||||||
"validate:schemas": "node tools/validate-agent-schema.js"
|
"validate:schemas": "node tools/validate-agent-schema.js"
|
||||||
},
|
},
|
||||||
|
|||||||
214
test/test-installation-components.js
Normal file
214
test/test-installation-components.js
Normal file
@ -0,0 +1,214 @@
|
|||||||
|
/**
|
||||||
|
* Installation Component Tests
|
||||||
|
*
|
||||||
|
* Tests individual installation components in isolation:
|
||||||
|
* - Agent YAML → XML compilation
|
||||||
|
* - Manifest generation
|
||||||
|
* - Path resolution
|
||||||
|
* - Customization merging
|
||||||
|
*
|
||||||
|
* These are deterministic unit tests that don't require full installation.
|
||||||
|
* Usage: node test/test-installation-components.js
|
||||||
|
*/
|
||||||
|
|
||||||
|
const path = require('node:path');
|
||||||
|
const fs = require('fs-extra');
|
||||||
|
const { YamlXmlBuilder } = require('../tools/cli/lib/yaml-xml-builder');
|
||||||
|
const { ManifestGenerator } = require('../tools/cli/installers/lib/core/manifest-generator');
|
||||||
|
|
||||||
|
// ANSI colors
|
||||||
|
const colors = {
|
||||||
|
reset: '\u001B[0m',
|
||||||
|
green: '\u001B[32m',
|
||||||
|
red: '\u001B[31m',
|
||||||
|
yellow: '\u001B[33m',
|
||||||
|
cyan: '\u001B[36m',
|
||||||
|
dim: '\u001B[2m',
|
||||||
|
};
|
||||||
|
|
||||||
|
let passed = 0;
|
||||||
|
let failed = 0;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Test helper: Assert condition
|
||||||
|
*/
|
||||||
|
function assert(condition, testName, errorMessage = '') {
|
||||||
|
if (condition) {
|
||||||
|
console.log(`${colors.green}✓${colors.reset} ${testName}`);
|
||||||
|
passed++;
|
||||||
|
} else {
|
||||||
|
console.log(`${colors.red}✗${colors.reset} ${testName}`);
|
||||||
|
if (errorMessage) {
|
||||||
|
console.log(` ${colors.dim}${errorMessage}${colors.reset}`);
|
||||||
|
}
|
||||||
|
failed++;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Test Suite
|
||||||
|
*/
|
||||||
|
async function runTests() {
|
||||||
|
console.log(`${colors.cyan}========================================`);
|
||||||
|
console.log('Installation Component Tests');
|
||||||
|
console.log(`========================================${colors.reset}\n`);
|
||||||
|
|
||||||
|
const projectRoot = path.join(__dirname, '..');
|
||||||
|
|
||||||
|
// ============================================================
|
||||||
|
// Test 1: YAML → XML Agent Compilation (In-Memory)
|
||||||
|
// ============================================================
|
||||||
|
console.log(`${colors.yellow}Test Suite 1: Agent Compilation${colors.reset}\n`);
|
||||||
|
|
||||||
|
try {
|
||||||
|
const builder = new YamlXmlBuilder();
|
||||||
|
const pmAgentPath = path.join(projectRoot, 'src/modules/bmm/agents/pm.agent.yaml');
|
||||||
|
|
||||||
|
// Create temp output path
|
||||||
|
const tempOutput = path.join(__dirname, 'temp-pm-agent.md');
|
||||||
|
|
||||||
|
try {
|
||||||
|
const result = await builder.buildAgent(pmAgentPath, null, tempOutput, { includeMetadata: true });
|
||||||
|
|
||||||
|
assert(result && result.outputPath === tempOutput, 'Agent compilation returns result object with outputPath');
|
||||||
|
|
||||||
|
// Read the output
|
||||||
|
const compiled = await fs.readFile(tempOutput, 'utf8');
|
||||||
|
|
||||||
|
assert(compiled.includes('<agent'), 'Compiled agent contains <agent> tag');
|
||||||
|
|
||||||
|
assert(compiled.includes('<persona>'), 'Compiled agent contains <persona> tag');
|
||||||
|
|
||||||
|
assert(compiled.includes('<menu>'), 'Compiled agent contains <menu> tag');
|
||||||
|
|
||||||
|
assert(compiled.includes('Product Manager'), 'Compiled agent contains agent title');
|
||||||
|
|
||||||
|
// Cleanup
|
||||||
|
await fs.remove(tempOutput);
|
||||||
|
} catch (error) {
|
||||||
|
assert(false, 'Agent compilation succeeds', error.message);
|
||||||
|
}
|
||||||
|
} catch (error) {
|
||||||
|
assert(false, 'YamlXmlBuilder instantiates', error.message);
|
||||||
|
}
|
||||||
|
|
||||||
|
console.log('');
|
||||||
|
|
||||||
|
// ============================================================
|
||||||
|
// Test 2: Customization Merging
|
||||||
|
// ============================================================
|
||||||
|
console.log(`${colors.yellow}Test Suite 2: Customization Merging${colors.reset}\n`);
|
||||||
|
|
||||||
|
try {
|
||||||
|
const builder = new YamlXmlBuilder();
|
||||||
|
|
||||||
|
// Test deepMerge function
|
||||||
|
const base = {
|
||||||
|
agent: {
|
||||||
|
metadata: { name: 'John', title: 'PM' },
|
||||||
|
persona: { role: 'Product Manager', style: 'Analytical' },
|
||||||
|
},
|
||||||
|
};
|
||||||
|
|
||||||
|
const customize = {
|
||||||
|
agent: {
|
||||||
|
metadata: { name: 'Sarah' }, // Override name only
|
||||||
|
persona: { style: 'Concise' }, // Override style only
|
||||||
|
},
|
||||||
|
};
|
||||||
|
|
||||||
|
const merged = builder.deepMerge(base, customize);
|
||||||
|
|
||||||
|
assert(merged.agent.metadata.name === 'Sarah', 'Deep merge overrides customized name');
|
||||||
|
|
||||||
|
assert(merged.agent.metadata.title === 'PM', 'Deep merge preserves non-overridden title');
|
||||||
|
|
||||||
|
assert(merged.agent.persona.role === 'Product Manager', 'Deep merge preserves non-overridden role');
|
||||||
|
|
||||||
|
assert(merged.agent.persona.style === 'Concise', 'Deep merge overrides customized style');
|
||||||
|
} catch (error) {
|
||||||
|
assert(false, 'Customization merging works', error.message);
|
||||||
|
}
|
||||||
|
|
||||||
|
console.log('');
|
||||||
|
|
||||||
|
// ============================================================
|
||||||
|
// Test 3: Path Resolution
|
||||||
|
// ============================================================
|
||||||
|
console.log(`${colors.yellow}Test Suite 3: Path Variable Resolution${colors.reset}\n`);
|
||||||
|
|
||||||
|
try {
|
||||||
|
const builder = new YamlXmlBuilder();
|
||||||
|
|
||||||
|
// Test path resolution logic (if exposed)
|
||||||
|
// This would test {project-root}, {installed_path}, {config_source} resolution
|
||||||
|
|
||||||
|
const testPath = '{project-root}/bmad/bmm/config.yaml';
|
||||||
|
const expectedPattern = /\/bmad\/bmm\/config\.yaml$/;
|
||||||
|
|
||||||
|
assert(
|
||||||
|
true, // Placeholder - would test actual resolution
|
||||||
|
'Path variable resolution pattern matches expected format',
|
||||||
|
'Note: This test validates path resolution logic exists',
|
||||||
|
);
|
||||||
|
} catch (error) {
|
||||||
|
assert(false, 'Path resolution works', error.message);
|
||||||
|
}
|
||||||
|
|
||||||
|
console.log('');
|
||||||
|
|
||||||
|
// ============================================================
|
||||||
|
// Test 5: TEA Agent Special Handling
|
||||||
|
// ============================================================
|
||||||
|
console.log(`${colors.yellow}Test Suite 5: TEA Agent Compilation${colors.reset}\n`);
|
||||||
|
|
||||||
|
try {
|
||||||
|
const builder = new YamlXmlBuilder();
|
||||||
|
const teaAgentPath = path.join(projectRoot, 'src/modules/bmm/agents/tea.agent.yaml');
|
||||||
|
const tempOutput = path.join(__dirname, 'temp-tea-agent.md');
|
||||||
|
|
||||||
|
try {
|
||||||
|
const result = await builder.buildAgent(teaAgentPath, null, tempOutput, { includeMetadata: true });
|
||||||
|
const compiled = await fs.readFile(tempOutput, 'utf8');
|
||||||
|
|
||||||
|
assert(compiled.includes('tea-index.csv'), 'TEA agent compilation includes critical_actions with tea-index.csv reference');
|
||||||
|
|
||||||
|
assert(compiled.includes('testarch/knowledge'), 'TEA agent compilation includes knowledge base path');
|
||||||
|
|
||||||
|
assert(compiled.includes('*test-design'), 'TEA agent menu includes test-design workflow');
|
||||||
|
|
||||||
|
// Cleanup
|
||||||
|
await fs.remove(tempOutput);
|
||||||
|
} catch (error) {
|
||||||
|
assert(false, 'TEA agent compiles successfully', error.message);
|
||||||
|
}
|
||||||
|
} catch (error) {
|
||||||
|
assert(false, 'TEA compilation test setup', error.message);
|
||||||
|
}
|
||||||
|
|
||||||
|
console.log('');
|
||||||
|
|
||||||
|
// ============================================================
|
||||||
|
// Summary
|
||||||
|
// ============================================================
|
||||||
|
console.log(`${colors.cyan}========================================`);
|
||||||
|
console.log('Test Results:');
|
||||||
|
console.log(` Passed: ${colors.green}${passed}${colors.reset}`);
|
||||||
|
console.log(` Failed: ${colors.red}${failed}${colors.reset}`);
|
||||||
|
console.log(`========================================${colors.reset}\n`);
|
||||||
|
|
||||||
|
if (failed === 0) {
|
||||||
|
console.log(`${colors.green}✨ All installation component tests passed!${colors.reset}\n`);
|
||||||
|
process.exit(0);
|
||||||
|
} else {
|
||||||
|
console.log(`${colors.red}❌ Some installation component tests failed${colors.reset}\n`);
|
||||||
|
process.exit(1);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// Run tests
|
||||||
|
runTests().catch((error) => {
|
||||||
|
console.error(`${colors.red}Test runner failed:${colors.reset}`, error.message);
|
||||||
|
console.error(error.stack);
|
||||||
|
process.exit(1);
|
||||||
|
});
|
||||||
Loading…
x
Reference in New Issue
Block a user