Get the FREE Ultimate OpenClaw Setup Guide →

ln-510-quality-coordinator

npx machina-cli add skill levnikolaevich/claude-code-skills/ln-510-quality-coordinator --openclaw
Files (1)
SKILL.md
8.2 KB

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.

Quality Coordinator

Sequential coordinator for code quality pipeline. Invokes 4 workers in index order (511 -> 512 -> 513 -> 514) and returns aggregated results to ln-500.

Purpose & Scope

  • Invoke ln-511-code-quality-checker (metrics, MCP Ref, static analysis)
  • Invoke ln-512-tech-debt-cleaner (auto-fix safe findings from ln-511)
  • Invoke ln-513-agent-reviewer (external agent reviews on cleaned code)
  • Run Criteria Validation (Story dependencies, AC-Task Coverage, DB Creation Principle)
  • Run linters from tech_stack.md
  • Invoke ln-514-regression-checker (test suite after all changes)
  • Return aggregated quality results to ln-500-story-quality-gate
  • No verdict determination — ln-500 decides final Gate verdict

When to Use

  • Invoked by ln-500-story-quality-gate Phase 2
  • All implementation tasks in Story status = Done

Workflow

Phase 1: Discovery

  1. Auto-discover team/config from docs/tasks/kanban_board.md
  2. Load Story + task metadata from Linear (no full descriptions)

Fast-track mode: When invoked with --fast-track flag (readiness 10/10), run Phase 2 with --skip-mcp-ref (metrics + static only, no MCP Ref), skip Phase 3 (ln-512), Phase 4 (ln-513). Run Phase 5 (criteria), Phase 6 (linters), Phase 7 (ln-514).

Input: Story ID from ln-500-story-quality-gate

Phase 2: Code Quality (delegate to ln-511 — ALWAYS runs)

MANDATORY STEP: ln-511 invocation required in ALL modes. Full gate: ln-511 runs everything (metrics + MCP Ref + static analysis). Fast-track: ln-511 runs with --skip-mcp-ref (metrics + static analysis only — catches complexity, DRY, dead code without expensive MCP Ref calls).

  1. Invoke ln-511-code-quality-checker via Skill tool
    • Full: ln-511 runs code metrics, MCP Ref validation (OPT/BP/PERF), static analysis
    • Fast-track: ln-511 runs code metrics + static analysis only (skips OPT-, BP-, PERF- MCP Ref checks)
    • Returns verdict (PASS/CONCERNS/ISSUES_FOUND) + code_quality_score + issues list
  2. If ln-511 returns ISSUES_FOUND -> aggregate issues, continue (ln-500 decides action)

Invocation:

# Full gate:
Skill(skill: "ln-511-code-quality-checker", args: "{storyId}")
# Fast-track:
Skill(skill: "ln-511-code-quality-checker", args: "{storyId} --skip-mcp-ref")

Phase 3: Tech Debt Cleanup (delegate to ln-512 — SKIP if --fast-track)

MANDATORY STEP (full gate): ln-512 invocation required. Safe auto-fixes only (confidence >=90%). Fast-track: SKIP this phase.

  1. Invoke ln-512-tech-debt-cleaner via Skill tool
    • ln-512 consumes findings from ln-511 output (passed via coordinator context)
    • Filters to auto-fixable categories (unused imports, dead code, deprecated aliases)
    • Applies safe fixes, verifies build integrity, creates commit
  2. If ln-512 returns BUILD_FAILED -> all changes reverted, aggregate issue, continue

Invocation:

Skill(skill: "ln-512-tech-debt-cleaner", args: "{storyId}")

Phase 4: Agent Review (delegate to ln-513 — SKIP if --fast-track)

MANDATORY STEP (full gate): ln-513 invocation required. Returns SKIPPED gracefully if agents unavailable. Fast-track: SKIP this phase.

  1. Invoke ln-513-agent-reviewer via Skill tool
    • ln-513 runs external agents (Codex + Gemini) in parallel on cleaned code
    • Critically verifies each suggestion, debates if disagreeing
    • Returns filtered suggestions with confidence scoring
  2. Merge suggestions into issues list (same prefixes: SEC-, PERF-, MNT-, ARCH-, BP-, OPT-)
  3. If verdict = SUGGESTIONS with area=security or area=correctness -> escalate aggregate to CONCERNS

Invocation:

Skill(skill: "ln-513-agent-reviewer", args: "{storyId}")

Phase 5: Criteria Validation

MANDATORY READ: Load references/criteria_validation.md

CheckDescriptionFail Action
#1 Story DependenciesNo forward deps within Epic[DEP-] issue
#2 AC-Task CoverageSTRONG/WEAK/MISSING scoring[COV-]/[BUG-] issue
#3 DB Creation PrincipleSchema scope matches Story[DB-] issue

Phase 6: Linters

MANDATORY READ: shared/references/ci_tool_detection.md (Discovery Hierarchy + Command Registry)

  1. Detect lint/typecheck commands per ci_tool_detection.md discovery hierarchy
  2. Run all detected checks (timeouts per guide: 2min linters, 5min typecheck)
  3. If any check fails -> aggregate issues, continue

Phase 7: Regression Tests (delegate to ln-514)

  1. Invoke ln-514-regression-checker via Skill tool
    • Runs full test suite, reports PASS/FAIL
    • Runs AFTER ln-512 changes to verify nothing broke
  2. If regression FAIL -> aggregate issues, continue

Invocation:

Skill(skill: "ln-514-regression-checker", args: "{storyId}")

Phase 8: Return Results

Return aggregated results to ln-500:

quality_check: PASS | CONCERNS | ISSUES_FOUND
code_quality_score: {0-100}
agent_review: CODE_ACCEPTABLE | SUGGESTIONS | SKIPPED
criteria_validation: PASS | FAIL
linters: PASS | FAIL
tech_debt_cleanup: CLEANED | NOTHING_TO_CLEAN | BUILD_FAILED | SKIPPED
regression: PASS | FAIL
issues:
  - {id: "SEC-001", severity: high, finding: "...", source: "ln-511"}
  - {id: "OPT-001", severity: medium, finding: "...", source: "ln-513"}
  - {id: "DEP-001", severity: medium, finding: "...", source: "criteria"}
  - {id: "LINT-001", severity: low, finding: "...", source: "linters"}

TodoWrite format (mandatory):

- Invoke ln-511-code-quality-checker (in_progress)
- Invoke ln-512-tech-debt-cleaner (pending)
- Invoke ln-513-agent-reviewer (pending)
- Criteria Validation (Story deps, AC coverage, DB schema) (pending)
- Run linters from tech_stack.md (pending)
- Invoke ln-514-regression-checker (pending)
- Return results to ln-500 (pending)

Worker Invocation (MANDATORY)

PhaseWorkerContext
2ln-511-code-quality-checkerShared (Skill tool) — code metrics, MCP Ref, static analysis
3ln-512-tech-debt-cleanerShared (Skill tool) — auto-fix safe findings from ln-511
4ln-513-agent-reviewerShared (Skill tool) — external agent reviews on cleaned code
7ln-514-regression-checkerShared (Skill tool) — full test suite after all changes

All workers: Invoke via Skill tool — workers see coordinator context. Sequential execution: 511 -> 512 -> 513 -> 514.

Anti-Patterns:

  • Running mypy, ruff, pytest directly instead of invoking ln-511/ln-514
  • Running agent reviews directly instead of invoking ln-513
  • Auto-fixing code directly instead of invoking ln-512
  • Marking steps as completed without invoking the actual skill
  • Determining final verdict (that's ln-500's responsibility)

Critical Rules

  • Return all results to ln-500; do NOT determine verdict
  • Single source of truth: rely on Linear metadata for tasks
  • Language preservation in comments (EN/RU)
  • Do not create tasks or change statuses; ln-500 decides next actions

Definition of Done

  • ln-511 invoked (ALWAYS — full or --skip-mcp-ref in fast-track), code quality score returned
  • ln-512 invoked (or skipped if --fast-track), tech debt cleanup results returned
  • ln-513 invoked (or skipped if --fast-track), agent review results returned
  • Criteria Validation completed (3 checks)
  • Linters executed
  • ln-514 invoked, regression results returned
  • Aggregated results returned to ln-500

Reference Files

  • Criteria Validation: references/criteria_validation.md
  • Gate levels: references/gate_levels.md
  • Workers: ../ln-511-code-quality-checker/SKILL.md, ../ln-512-tech-debt-cleaner/SKILL.md, ../ln-513-agent-reviewer/SKILL.md, ../ln-514-regression-checker/SKILL.md
  • Caller: ../ln-500-story-quality-gate/SKILL.md
  • Test planning (separate coordinator): ../ln-520-test-planner/SKILL.md
  • Tech stack/linters: docs/project/tech_stack.md

Version: 7.0.0 Last Updated: 2026-02-09

Source

git clone https://github.com/levnikolaevich/claude-code-skills/blob/master/ln-510-quality-coordinator/SKILL.mdView on GitHub

Overview

ln-510-quality-coordinator orchestrates a sequential code quality pipeline that runs ln-511, ln-512, ln-513, and ln-514 in order, then returns aggregated results to ln-500-story-quality-gate. It centralizes coordination, collects findings, and ensures all quality checks feed into the final gating decision handled by ln-500.

How This Skill Works

The coordinator ALWAYS delegates to ln-511 (code quality checker) to gather metrics, MCP references, and static analysis. Based on results, it may trigger ln-512 for safe tech-debt auto-fixes, then ln-513 for external agent reviews, and finally ln-514 for a regression check. All results are aggregated and returned to ln-500-story-quality-gate; the final verdict is determined by ln-500, not by the coordinator.

When to Use It

  • Invoked by ln-500-story-quality-gate during Phase 2 of a Story lifecycle.
  • You want to run the Full gate to execute 511 → 512 → 513 → 514 in sequence.
  • You need a Fast-track run using --fast-track to skip certain steps and get quicker readiness.
  • ln-511 reports ISSUES_FOUND and you still want to aggregate and continue to higher phases for a complete picture.
  • You require the coordinator to return aggregated results to ln-500-story-quality-gate for the final gating decision.

Quick Start

  1. Step 1: Prepare the story and obtain its ID for the coordinator.
  2. Step 2: Trigger the coordinator to run Phase 2 (and full pipeline or fast-track as needed).
  3. Step 3: Retrieve the aggregated results from ln-500-story-quality-gate (ln-500 makes the final decision).

Best Practices

  • Ensure the storyId is ready and provided when invoking the coordinator.
  • Always run ln-511 as the mandatory first step in ALL modes.
  • Use the fast-track path only when you accept skipping ln-512 and ln-513 in exchange for speed.
  • After ln-512, verify build integrity before proceeding to agent reviews.
  • Review the aggregated issues list and results returned to ln-500 before final gating.

Example Use Cases

  • Onboarding a new feature, the coordinator runs 511-514 to certify code quality before PR merge.
  • A sprint cleanup schedules ln-512 auto-fixes for dead code and unused imports flagged by 511.
  • External agents review cleaned code in ln-513, with suggestions merged into the issues list.
  • Post-refactor regression suite (ln-514) confirms no breaking changes before release.
  • Fast-track mode is used when time is tight and 511 metrics plus static analysis suffice.

Frequently Asked Questions

Add this skill to your agents
Sponsor this space

Reach thousands of developers