review-orchestrator

Files (31)

• SKILL.md
• README.md · agents/
• accessibility.md · agents/
• api-contract.md · agents/
• architecture-reviewer.md · agents/
• bug-hunter.md · agents/
• coderabbit.md · agents/
• complexity.md · agents/
• ddd-structure.md · agents/
• dep-secrets-scan.md · agents/
• doc-freshness.md · agents/
• dx-readability.md · agents/
• ecosystem-compat.md · agents/
• error-handling.md · agents/
• import-graph.md · agents/
• memory-leak.md · agents/
• pattern-compliance.md · agents/
• performance-profiler.md · agents/
• regression-risk.md · agents/
• security-auditor.md · agents/
• semgrep-sast.md · agents/
• sonarjs-lint.md · agents/
• test-quality.md · agents/
• vue-patterns.md · agents/
• eslint.strict.config.js · configs/
• fallback-cascade.md · reference/
• output-format.md · reference/
• severity-definitions.md · reference/
• gather-context.sh · scripts/
• run-headless-review.sh · scripts/
• setup-tools.sh · scripts/

SKILL.md (first lines)

# Review Orchestrator

Dispatches multiple review subagents in parallel, compiles and deduplicates their feedback, and presents a triaged list for human decision.

## Configuration

```bash
# Required environment variables
PIPELINE_API_URL=http://localhost:3000  # Pipeline API endpoint
AGENT_ID=review-orchestrator             # Agent identifier
```

## When to Use

- Quality gates have passed (lint, typecheck, tests)
- Code changes are ready for review
- Before creating a PR

## Prerequisites

- CodeRabbit CLI installed: `which coderabbit`
- Authenticated: `coderabbit auth login`
- Pro tier allows 8 reviews/hour
- Pipeline API running and accessible

## Workflow

### 1. Verify Readiness

```bash
# Check CodeRabbit installed
which coderabbit

# Check there are changes to review
git status --porcelain
git diff --stat HEAD~1  # or vs base branch
```

If no changes or CodeRabbit missing, stop and inform user.

### 2. Gather Context

Get ticket context from the API and collect local data:

```typescript
import { PipelineClient, PipelineAPIError } from '@pipeline/client'

const client = new PipelineClient({
  apiUrl: process.env.PIPELINE_API_URL!,
  agentId: process.env.AGENT_ID!,
})

const ticket = await client.getTicket(ticketId)
const ticketRequirements = ticket.description
```

Also collect:

- List of changed files: `git diff --name-only HEAD~1`
- Plan file if available locally

### 3. Dispatch Review Subagents

Use the Task tool to dispatch three parallel review subagents:

**Subagent 1: CodeRabbit Review**

```
Run CodeRabbit CLI review on the current changes.

Commands:
cd {REPO_PATH}
coderabbit --prompt-only --type uncommitted

Parse output and categorize findings by severity:
- Critical: Security, race conditions, data loss
- Major: Logic errors, performance, missing error handling
- Minor: Code style, naming
- Nitpick: Formatting preferences

Return structured list with file:line, issue, suggested fix.
```

**Subagent 2: Agent Code Review**

```
Perform comprehensive code review of the changes.

Files to review: {FILES_LIST}
Ticket requirements: {TICKET_REQUIREMENTS}

Review Checklist:
1. Functionality: Implements requirements, handles edge cases, proper error handling
2. Code Quality: TypeScript (no any), Vue (Composition API), Tailwind (no dark:), naming conventions
3. Architecture: Follows existing patterns, no over-engineering, clean separation
4. Testing: Tests added, behavioral not change detectors, edge cases covered
5. Security: No secrets exposed, input validation, no XSS/injection
6. Performance: No unnecessary re-renders, efficient data structures

Return findings with severity (Critical/Major/Minor), location (file:line), and fix suggestion.
```

**Subagent 3: Pattern Compliance Review**

```
Check code against target repository patterns and AGENTS.md guidelines.

Files to review: {FILES_LIST}

Check for violations:
- Vue: Reactive props destructuring (not withDefaults), separate type imports
- Tailwind: No dark: variants (use semantic themes), use cn() utility, no !important
- TypeScript: No any types, no as any assertions
- State: No unnecessary computed/watch when simpler works
- Testing: Behavioral tests, not change detectors or mock tests

Return violations with file:line and the correct pattern to use.
```

From DrJKL/ticket-to-pr-pipeline, MIT.

More from DrJKL/ticket-to-pr-pipeline