claude-dev-kit 2.1.0

package/.claude/skills/build-and-fix/SKILL.md

@@ -0,0 +1,271 @@
+---
+version: 1.1.0
+name: build-and-fix
+description: This skill should be used when the user asks to "build the project", "run build", "build and fix errors", "fix lint errors", "fix formatting", "run cargo build", "npm run build", "pnpm build", "yarn build", "poetry build", "cmake build", "make it compile", "check if code compiles", "verify build", "fix build failures", "validate before commit", "pre-commit validation", or "check compilation". Auto-detects project type (Rust/JS/TS/Python/C++), runs build, and automatically fixes simple errors like formatting and linting. Does NOT fix type errors, logic bugs, or missing dependencies - bails to user for complex issues.
+---
+
+# Build & Fix
+
+**Notification:** At skill start, output: "Using build-and-fix to detect and build project..."
+
+## When to Use
+
+- Build a project and ensure it compiles successfully
+- Fix simple build errors automatically (formatting, linting)
+- Validate changes before committing
+- Debug build failures efficiently
+
+## When NOT to Use
+
+- Complex refactoring requiring semantic understanding
+- Fixing type errors, logic bugs, or missing imports
+- Projects requiring manual build configuration
+- When user needs to understand the error (don't hide it)
+
+## Supported Languages
+
+| Language | Build Systems | Priority |
+|----------|--------------|----------|
+| **Rust** | cargo | 1 (highest) |
+| **JavaScript/TypeScript** | npm/yarn/pnpm | 2 |
+| **Python** | pip/poetry/pipenv | 3 |
+| **C++** | cmake only | 4 (lowest) |
+
+## Workflow
+
+### 1. Pre-flight Checks
+
+Before building, verify environment is ready:
+
+```
+[PREFLIGHT] Checking build environment...
+```
+
+- **Verify build tool exists**: Run the `preflight` command from config (e.g., `cargo --version`)
+- **Check dependencies**: Verify lockfile/node_modules/target directory exists
+- **If tool missing (exit 127)**: Bail immediately with installation instructions
+- **If deps missing**: Suggest running install command (don't auto-install without permission)
+
+### 2. Detect Project Type
+
+```
+[DETECT] Scanning for project manifests...
+```
+
+- Search current working directory for manifest files using Glob
+- Match against `detection.required_files` in each language config
+- If multiple languages detected, use `priority` field (lower = higher priority)
+- For monorepos: Check for workspace configs (see Workspace Detection below)
+
+### 3. Load Language Config
+
+- Read `language-configs/{language}.yaml` using Read tool
+- Extract commands, fix patterns, bail patterns, and limits
+- Determine package manager if applicable (from lockfiles)
+
+### 4. Configure Build (C++ only)
+
+```
+[CONFIGURE] Configuring build system...
+```
+
+- Run `cmake -S . -B build` before building
+- Exit codes: 0 = success, 127 = cmake not found (bail), other = config error (bail)
+
+### 5. Build Project
+
+```
+[BUILD] Building project...
+```
+
+- Run build command from config via Bash
+- Apply timeout from config (default: 300s for C++, 120s for others)
+- Capture stdout + stderr for error parsing
+- Check exit code:
+  - 0 = success
+  - 127 = tool not found (bail with install instructions)
+  - 130 = user interrupted (stop gracefully)
+  - Other = build failure (proceed to error parsing)
+
+### 6. Parse Errors (if build failed)
+
+```
+[ANALYZE] Parsing build output...
+```
+
+**Error Extraction Process:**
+
+1. **Check bail patterns FIRST** - If any `bail_on` regex matches, stop immediately
+2. **Extract error locations** using `error_parsing` patterns from config:
+   - Pattern groups: `{file}`, `{line}`, `{column}`, `{message}`
+3. **Match fix patterns** - Compare errors against `fixes[]` list
+4. **Classify errors**:
+   - Auto-fixable: formatting, linting, whitespace
+   - Bail immediately: type errors, syntax errors, missing deps
+
+### 7. Apply Fixes
+
+```
+[FIX] Applying {count} fixes...
+```
+
+- Look up `action` in commands (e.g., 'format' → 'cargo fmt')
+- Substitute captured values: `{1}` = first capture group, `{file}` = matched file
+- Execute fix via Bash
+- Track which fixes were applied
+
+### 8. Retry Build
+
+```
+[RETRY] Retrying build (attempt {n}/{max})...
+```
+
+- Maximum 2 retry cycles
+- **Stop immediately if:**
+  - Same errors recur (compare error text, ignore line numbers)
+  - Total errors exceed `max_errors` from config
+  - Any `bail_on` pattern matches
+
+### 9. Report Results
+
+**Success:**
+```
+[SUCCESS] Build passed!
+- Fixes applied: {list}
+- Build time: {duration}
+- Retry cycles: {count}
+```
+
+**Failure:**
+```
+[BAIL] Build failed - manual intervention required
+- Error: {summary}
+- Location: {file}:{line}
+- Reason: {why auto-fix stopped}
+- Suggestion: {next steps}
+```
+
+## Config Schema
+
+See `references/config-schema.md` for the complete schema reference.
+
+**Quick summary:** Each `language-configs/{language}.yaml` contains:
+- `detection` - Files that identify the project type
+- `preflight` - Pre-build verification commands
+- `commands` - Build, format, lint commands
+- `fixes` - Auto-fixable error patterns
+- `bail_on` - Errors requiring manual intervention
+- `max_errors`, `max_retries` - Safety limits
+
+## Workspace Detection
+
+For monorepos with multiple projects:
+
+| Language | Workspace Indicator | Behavior |
+|----------|-------------------|----------|
+| Rust | `[workspace]` in Cargo.toml | Build from workspace root |
+| JavaScript | `workspaces` in package.json | Detect package manager workspace commands |
+| Python | `tool.poetry.packages` in pyproject.toml | Build from monorepo root |
+
+If workspace detected:
+1. Note it in output: `[DETECT] Rust workspace (3 members)`
+2. Build from workspace root, not individual package
+3. Errors may reference multiple packages
+
+## Safety Guidelines
+
+**DO auto-fix:**
+- Formatting issues (rustfmt, black, prettier)
+- Simple linting errors with `--fix` flag
+- Trailing whitespace
+
+**DO NOT auto-fix:**
+- Dependency changes (npm audit fix, cargo update)
+- Auto-upgrades of any kind
+- Destructive file operations
+- Anything requiring user confirmation
+
+**Always bail on:**
+- Type errors
+- Syntax errors
+- Missing imports/modules
+- Linker errors
+- Permission errors
+
+## Tool Usage
+
+| Tool | Use For | Never Use For |
+|------|---------|---------------|
+| **Glob** | Find manifest files | - |
+| **Read** | Load configs, examine source | - |
+| **Edit** | Apply code fixes | - |
+| **Bash** | Run build/fix commands | File operations, sed, awk |
+| **Grep** | Parse build output | - |
+
+## Progress Markers
+
+Output these markers during execution:
+
+| Marker | When |
+|--------|------|
+| `[PREFLIGHT]` | Checking build environment |
+| `[DETECT]` | Project type detection |
+| `[CONFIGURE]` | CMake configuration (C++ only) |
+| `[BUILD]` | Build command execution |
+| `[ANALYZE]` | Parsing build errors |
+| `[FIX]` | Applying auto-fixes |
+| `[RETRY]` | Retrying build after fixes |
+| `[SUCCESS]` | Build passed |
+| `[BAIL]` | Stopping due to unfixable errors |
+
+## Example Flows
+
+### Success with Auto-Fix
+```
+User: Build the project and fix errors
+
+[PREFLIGHT] Checking build environment... cargo v1.75.0 found
+[DETECT] Rust project (Cargo.toml found)
+[BUILD] Building project...
+[ANALYZE] Build failed - 2 errors found
+[FIX] Applying 2 formatting fixes... Running cargo fmt
+[RETRY] Retrying build (attempt 1/2)...
+[SUCCESS] Build passed! Fixed 2 formatting issues in 1 retry cycle.
+```
+
+### Bail on Complex Error
+```
+User: Run cargo build
+
+[PREFLIGHT] Checking build environment... cargo v1.75.0 found
+[DETECT] Rust project (Cargo.toml found)
+[BUILD] Building project...
+[ANALYZE] Build failed - 1 error found
+[BAIL] Type error detected (E0308) - cannot auto-fix
+
+Error at src/main.rs:15:20
+  mismatched types: expected `i32`, found `&str`
+
+This requires manual code changes. The variable is declared as i32 but assigned a string.
+```
+
+## Additional Resources
+
+Detailed reference documentation:
+- `references/config-schema.md` - Complete configuration schema
+- `references/error-patterns.md` - Common error patterns by language
+- `references/troubleshooting.md` - Build tool issues and solutions
+- `references/extending.md` - How to add new language support
+
+Language configurations in `language-configs/`:
+- `rust.yaml`, `javascript.yaml`, `python.yaml`, `cpp.yaml`
+
+Example scenarios in `examples/`:
+- `rust-format-fix.md` - Rust formatting fix
+- `javascript-lint-fix.md` - ESLint auto-fix
+- `python-format-fix.md` - Black formatting
+- `bail-on-type-error.md` - Bail on complex error
+- `tool-not-found.md` - Missing build tool handling
+- `monorepo-detection.md` - Workspace project handling
+- `max-errors-exceeded.md` - Error threshold exceeded
+- `cpp-clang-format.md` - C++ clang-format fix

package/.claude/skills/build-and-fix/examples/javascript-lint-fix.md

@@ -0,0 +1,37 @@
+# Example: JavaScript ESLint Fix
+
+## Scenario
+User runs build on a JS/TS project with fixable ESLint errors.
+
+## Build Output
+```
+$ npm run build
+
+> project@1.0.0 build
+> eslint . && tsc
+
+/src/utils.ts
+  5:1  error  Expected indentation of 2 spaces  indent
+  12:20  error  Missing semicolon  semi
+
+2 problems (2 errors, 0 warnings)
+  2 errors and 0 warnings potentially fixable with the `--fix` option.
+```
+
+## Expected Response
+```
+Using build-and-fix to detect and build project...
+
+[DETECT] JavaScript/TypeScript project (package.json found, using npm)
+[BUILD] Building project...
+[ERROR] Build failed - 2 ESLint errors
+[ANALYZE] Analyzing build errors...
+[FIX] Applying fixes... Running npm run lint -- --fix...
+[RETRY] Building project...
+[SUCCESS] Build passed! Fixed 2 linting issues in 1 retry cycle.
+```
+
+## Key Points
+- Pattern `\d+ fixable with.*--fix` detects ESLint fixable errors
+- Package manager detected from lock files (npm/yarn/pnpm)
+- Uses appropriate lint command for detected package manager

package/.claude/skills/build-and-fix/language-configs/javascript.yaml

@@ -0,0 +1,139 @@
+# JavaScript/TypeScript Language Configuration
+version: "1.1.0"
+name: javascript
+display_name: JavaScript/TypeScript
+priority: 2
+
+# Detection: Files that identify this as a JS/TS project
+detection:
+  required_files: [package.json]
+  confidence_boost_files: [tsconfig.json, package-lock.json, yarn.lock, pnpm-lock.yaml]
+  workspace_indicators:
+    file: package.json
+    pattern: '"workspaces"'
+
+# Package manager detection (check lock files in this order)
+package_manager_detection:
+  - lockfile: pnpm-lock.yaml
+    manager: pnpm
+  - lockfile: yarn.lock
+    manager: yarn
+  - lockfile: package-lock.json
+    manager: npm
+  - default: npm
+
+# Pre-flight checks (by package manager)
+preflight:
+  npm:
+    check_tool: npm --version
+    check_deps: test -d node_modules
+    install_hint: "Install Node.js from https://nodejs.org"
+  yarn:
+    check_tool: yarn --version
+    check_deps: test -d node_modules
+    install_hint: "Install Yarn: npm install -g yarn"
+  pnpm:
+    check_tool: pnpm --version
+    check_deps: test -d node_modules
+    install_hint: "Install pnpm: npm install -g pnpm"
+
+# Timeouts (seconds)
+timeouts:
+  build: 180
+  fix: 60
+
+# Build commands by package manager
+commands:
+  npm:
+    build: npm run build
+    check: npm run build --dry-run 2>/dev/null || npm run typecheck 2>/dev/null || true
+    lint: npm run lint
+    lint_fix: npm run lint -- --fix
+    format: npm run format 2>/dev/null || npx prettier --write .
+    test: npm test
+  yarn:
+    build: yarn build
+    check: yarn build --dry-run 2>/dev/null || yarn typecheck 2>/dev/null || true
+    lint: yarn lint
+    lint_fix: yarn lint --fix
+    format: yarn format 2>/dev/null || yarn prettier --write .
+    test: yarn test
+  pnpm:
+    build: pnpm build
+    check: pnpm build --dry-run 2>/dev/null || pnpm typecheck 2>/dev/null || true
+    lint: pnpm lint
+    lint_fix: pnpm lint --fix
+    format: pnpm format 2>/dev/null || pnpm prettier --write .
+    test: pnpm test
+
+# Error parsing - extract file:line:column from various tools
+error_parsing:
+  patterns:
+    # TypeScript errors: src/file.ts(10,5): error TS2322
+    - regex: '(.+)\((\d+),(\d+)\): error (TS\d+): (.+)'
+      file: 1
+      line: 2
+      column: 3
+      code: 4
+      message: 5
+    # ESLint errors: /path/file.js:10:5 error Message
+    - regex: '(.+):(\d+):(\d+)\s+(error|warning)\s+(.+)'
+      file: 1
+      line: 2
+      column: 3
+      severity: 4
+      message: 5
+    # Webpack/Vite errors
+    - regex: 'ERROR in (.+):(\d+):(\d+)'
+      file: 1
+      line: 2
+      column: 3
+    # Generic error with file path
+    - regex: 'Error: (.+) in (.+):(\d+)'
+      message: 1
+      file: 2
+      line: 3
+
+# Error patterns with auto-fixes (only include fixable errors)
+fixes:
+  - pattern: '\d+ errors? and \d+ warnings? potentially fixable with the `--fix` option'
+    action: lint_fix
+    description: Auto-fix ESLint errors with --fix flag
+    confidence: high
+  - pattern: '\d+ fixable with.*--fix'
+    action: lint_fix
+    description: Auto-fix ESLint errors
+    confidence: high
+  - pattern: 'Run Prettier to fix'
+    action: format
+    description: Fix formatting with Prettier
+    confidence: high
+  - pattern: 'style:.*\(prettier'
+    action: format
+    description: Fix Prettier formatting issues
+    confidence: medium
+
+# Bail immediately on these errors (no auto-fix possible)
+bail_on:
+  - pattern: 'error TS\d+:.*is not assignable to type'
+    description: TypeScript type mismatch
+  - pattern: 'error TS\d+:.*Property .* does not exist'
+    description: TypeScript missing property
+  - pattern: 'error TS\d+:.*Cannot find module'
+    description: TypeScript missing module
+  - pattern: 'SyntaxError:'
+    description: JavaScript syntax error
+  - pattern: 'ReferenceError:'
+    description: Undefined variable reference
+  - pattern: 'TypeError:'
+    description: Type error at runtime
+  - pattern: 'The engine "node" is incompatible'
+    description: Node.js version mismatch
+  - pattern: 'Module not found'
+    description: Missing dependency
+  - pattern: 'ENOENT.*package\.json'
+    description: Missing package.json
+
+# Limits
+max_errors: 15
+max_retries: 2

package/.claude/skills/build-and-fix/references/config-schema.md

@@ -0,0 +1,120 @@
+# Config Schema Reference
+
+Each `language-configs/{language}.yaml` follows this structure:
+
+```yaml
+# Metadata
+version: "1.1.0"
+name: rust                    # Internal identifier
+display_name: Rust (Cargo)    # Human-readable name
+priority: 1                   # Detection priority (lower = higher)
+
+# Detection
+detection:
+  required_files: [Cargo.toml]           # Must exist (any one)
+  confidence_boost_files: [Cargo.lock]   # Optional, increases confidence
+  workspace_indicators: [members]         # For monorepo detection
+
+# Pre-flight checks
+preflight:
+  check_tool: cargo --version            # Verify tool exists
+  check_deps: test -f Cargo.lock         # Verify deps installed
+
+# Timeouts (seconds)
+timeouts:
+  build: 120
+  fix: 30
+
+# Commands (by package manager if applicable)
+commands:
+  build: cargo build
+  check: cargo check              # Fast pre-build verification
+  format: cargo fmt
+  test: cargo test
+
+# Error parsing - extract file:line:column from output
+error_parsing:
+  patterns:
+    - regex: 'error.*--> (.+):(\d+):(\d+)'
+      file: 1
+      line: 2
+      column: 3
+    - regex: '^error: (.+)$'
+      message: 1
+
+# Auto-fixable patterns
+fixes:
+  - pattern: 'Diff in .+\.rs'    # Regex to match
+    action: format               # Command to run
+    description: Fix formatting  # Human-readable
+    confidence: high             # high, medium, low
+
+# Bail patterns - stop immediately, don't retry
+bail_on:
+  - pattern: 'error\[E0308\]'
+    description: Type mismatch
+  - pattern: 'error\[E0597\]'
+    description: Lifetime error
+
+# Limits
+max_errors: 10
+max_retries: 2
+```
+
+## Pattern Syntax
+
+- **Patterns use regex** (Python `re` module compatible)
+- **Capture groups**: `()` creates groups, referenced as `{1}`, `{2}`, etc.
+- **Named groups**: `(?P<file>.+)` can be referenced as `{file}`
+- **Bail patterns checked first**, before any fix patterns
+
+## Command Substitution
+
+- `{1}`, `{2}`: Numbered capture groups from pattern match
+- `{file}`: Matched filename (if extracted)
+- `{line}`: Matched line number (if extracted)
+
+## Required Fields
+
+| Field | Type | Description |
+|-------|------|-------------|
+| `version` | string | Config version (e.g., "1.1.0") |
+| `name` | string | Internal identifier |
+| `display_name` | string | Human-readable name |
+| `priority` | int | Detection priority (1 = highest) |
+| `detection.required_files` | list | Files that identify this language |
+| `commands.build` | string | Build command |
+| `max_errors` | int | Error threshold before bailing |
+| `max_retries` | int | Maximum retry cycles |
+
+## Optional Fields
+
+| Field | Type | Description |
+|-------|------|-------------|
+| `detection.confidence_boost_files` | list | Additional files that increase confidence |
+| `detection.at_least_one` | bool | If true, any required file matches |
+| `detection.workspace_indicators` | object | Monorepo detection config |
+| `preflight` | object | Pre-build verification commands |
+| `timeouts` | object | Timeout values in seconds |
+| `error_parsing` | object | Patterns to extract error locations |
+| `fixes` | list | Auto-fixable error patterns |
+| `bail_on` | list | Patterns that trigger immediate bail |
+
+## Fix Confidence Levels
+
+| Level | Meaning | Example |
+|-------|---------|---------|
+| `high` | Safe, always works | `cargo fmt`, `black .` |
+| `medium` | Usually works, may need verification | `eslint --fix` |
+| `low` | Experimental, requires user confirmation | - |
+
+## Error Parsing Groups
+
+| Group Name | Purpose |
+|------------|---------|
+| `file` | Captured file path |
+| `line` | Line number |
+| `column` | Column number |
+| `message` | Error message text |
+| `code` | Error code (e.g., E0308, TS2322) |
+| `severity` | error/warning level |