ln-622-build-auditor
npx skills add https://github.com/levnikolaevich/claude-code-skills --skill ln-622-build-auditor
Agent 安装分布
Skill 文档
Paths: File paths (
shared/,references/,../ln-*) are relative to skills repo root. If not found at CWD, locate this SKILL.md directory and go up one level for repo root.
Build Health Auditor (L3 Worker)
Specialized worker auditing build health and code quality tooling.
Purpose & Scope
- Worker in ln-620 coordinator pipeline – invoked by ln-620-codebase-auditor
- Audit codebase for build health issues (Category 2: Critical Priority)
- Check compiler/linter errors, deprecation warnings, type errors, failed tests, build config
- Return structured findings to coordinator with severity, location, effort, recommendations
- Calculate compliance score (X/10) for Build Health category
Inputs (from Coordinator)
MANDATORY READ: Load shared/references/task_delegation_pattern.md#audit-coordinator--worker-contract for contextStore structure.
Receives contextStore with: tech_stack (including build_tool, test_framework), best_practices, principles, codebase_root.
Workflow
- Parse Context: Extract tech stack, build tools, test framework from contextStore
- Run Build Checks: Execute compiler, linter, type checker, tests (see Audit Rules below)
- Collect Findings: Record each violation with severity, location, effort, recommendation
- Calculate Score: Count violations by severity, calculate compliance score (X/10)
- Return Results: Return JSON with category, score, findings to coordinator
Audit Rules (Priority: CRITICAL)
1. Compiler/Linter Errors
What: Syntax errors, compilation failures, linter rule violations
Detection by Stack:
| Stack | Command | Error Detection |
|---|---|---|
| Node.js/TypeScript | npm run build or tsc --noEmit |
Check exit code, parse stderr for errors |
| Python | python -m py_compile *.py |
Check exit code, parse stderr |
| Go | go build ./... |
Check exit code, parse stderr |
| Rust | cargo build |
Check exit code, parse stderr |
| Java | mvn compile |
Check exit code, parse build log |
Linters:
- ESLint (JS/TS):
npx eslint . --format jsonâ parse JSON for errors - Pylint (Python):
pylint **/*.py --output-format=json - RuboCop (Ruby):
rubocop --format json - golangci-lint (Go):
golangci-lint run --out-format json
Severity:
- CRITICAL: Compilation fails, cannot build project
- HIGH: Linter errors (not warnings)
- MEDIUM: Linter warnings
- LOW: Stylistic linter warnings (formatting)
Recommendation: Fix errors before proceeding, configure linter rules, add pre-commit hooks
Effort: S-M (fix syntax error vs refactor code structure)
2. Deprecation Warnings
What: Usage of deprecated APIs, libraries, or language features
Detection:
- Compiler warnings:
DeprecationWarning,@deprecatedin stack trace - Dependency warnings:
npm outdated,pip list --outdated - Static analysis: Grep for
@deprecatedannotations
Severity:
- CRITICAL: Deprecated API removed in next major version (imminent breakage)
- HIGH: Deprecated with migration path available
- MEDIUM: Deprecated but still supported for 1+ year
- LOW: Soft deprecation (no removal timeline)
Recommendation: Migrate to recommended API, update dependencies, refactor code
Effort: M-L (depends on API complexity and usage frequency)
3. Type Errors
What: Type mismatches, missing type annotations, type checker failures
Detection by Stack:
| Stack | Tool | Command |
|---|---|---|
| TypeScript | tsc | tsc --noEmit --strict |
| Python | mypy | mypy . --strict |
| Python | pyright | pyright --warnings |
| Go | go vet | go vet ./... |
| Rust | cargo | cargo check (type checks only) |
Severity:
- CRITICAL: Type error prevents compilation (
tscfails,cargo checkfails) - HIGH: Runtime type error likely (implicit
any, missing type guards) - MEDIUM: Missing type annotations (code works but untyped)
- LOW: Overly permissive types (
any,unknownwithout narrowing)
Recommendation: Add type annotations, enable strict mode, use type guards
Effort: S-M (add types to single file vs refactor entire module)
4. Failed or Skipped Tests
What: Test suite failures, skipped tests, missing test coverage
Detection by Stack:
| Stack | Framework | Command |
|---|---|---|
| Node.js | Jest | npm test -- --json --outputFile=test-results.json |
| Node.js | Mocha | mocha --reporter json > test-results.json |
| Python | Pytest | pytest --json-report --json-report-file=test-results.json |
| Go | go test | go test ./... -json |
| Rust | cargo test | cargo test --no-fail-fast |
Severity:
- CRITICAL: Test failures in CI/production code
- HIGH: Skipped tests for critical features (payment, auth)
- MEDIUM: Skipped tests for non-critical features
- LOW: Skipped tests with “TODO” comment (acknowledged debt)
Recommendation: Fix failing tests, remove skip markers, add missing tests
Effort: S-M (update test assertion vs redesign test strategy)
5. Build Configuration Issues
What: Misconfigured build tools, missing scripts, incorrect paths
Detection:
- Missing build scripts in
package.json,Makefile,build.gradle - Incorrect paths in
tsconfig.json,webpack.config.js,Cargo.toml - Missing environment-specific configs (dev, staging, prod)
- Unused or conflicting build dependencies
Severity:
- CRITICAL: Build fails due to misconfiguration
- HIGH: Build succeeds but outputs incorrect artifacts (wrong target, missing assets)
- MEDIUM: Suboptimal config (no minification, missing source maps)
- LOW: Unused config options
Recommendation: Fix config paths, add missing build scripts, optimize build settings
Effort: S-M (update config file vs redesign build pipeline)
Scoring Algorithm
MANDATORY READ: Load shared/references/audit_scoring.md for unified scoring formula.
Output Format
MANDATORY READ: Load shared/references/audit_output_schema.md for JSON structure.
Return JSON with category: "Build Health" and checks: compilation_errors, linter_warnings, type_errors, test_failures, build_config.
Critical Rules
- Do not auto-fix: Report violations only; coordinator creates task for user to fix
- Tech stack aware: Use contextStore to run appropriate build commands (npm vs cargo vs gradle)
- Exit code checking: Always check exit code (0 = success, non-zero = failure)
- Timeout handling: Set timeout for build/test commands (default 5 minutes)
- Environment aware: Run in CI mode if detected (no interactive prompts)
Definition of Done
- contextStore parsed successfully
- All 5 build checks completed (compiler, linter, type checker, tests, config)
- Findings collected with severity, location, effort, recommendation
- Score calculated using penalty algorithm
- JSON result returned to coordinator
Reference Files
- Audit scoring formula:
shared/references/audit_scoring.md - Audit output schema:
shared/references/audit_output_schema.md - Build audit rules: references/build_rules.md
Version: 3.0.0 Last Updated: 2025-12-23