review-council 1.0.0

package/dist/resolver/local.js

@@ -0,0 +1,75 @@
+import { readFile } from 'fs/promises';
+import { detectLanguage } from '../prepare/language.js';
+function parseDiffText(diffText) {
+    const hunks = [];
+    const fileBlocks = diffText.split(/^diff --git /m).filter(Boolean);
+    for (const block of fileBlocks) {
+        const lines = block.split('\n');
+        const headerLine = lines[0] ?? '';
+        // Extract filenames from "a/path b/path"
+        const headerMatch = headerLine.match(/^a\/(.+) b\/(.+)$/);
+        if (!headerMatch)
+            continue;
+        const aPath = headerMatch[1];
+        const bPath = headerMatch[2];
+        let status = 'modified';
+        let previousFilename;
+        const deletedMatch = block.match(/^deleted file mode/m);
+        const newFileMatch = block.match(/^new file mode/m);
+        const renameFromMatch = block.match(/^rename from (.+)$/m);
+        const renameToMatch = block.match(/^rename to (.+)$/m);
+        if (deletedMatch) {
+            status = 'deleted';
+        }
+        else if (newFileMatch) {
+            status = 'added';
+        }
+        else if (renameFromMatch && renameToMatch) {
+            status = 'renamed';
+            previousFilename = renameFromMatch[1].trim();
+        }
+        const filename = status === 'deleted' ? aPath : bPath;
+        // Extract the actual patch lines (@@...)
+        const patchStart = block.indexOf('\n@@');
+        const patch = patchStart >= 0 ? block.slice(patchStart + 1) : '';
+        let additions = 0;
+        let deletions = 0;
+        for (const line of patch.split('\n')) {
+            if (line.startsWith('+') && !line.startsWith('+++'))
+                additions++;
+            if (line.startsWith('-') && !line.startsWith('---'))
+                deletions++;
+        }
+        hunks.push({
+            filename,
+            status,
+            previousFilename,
+            patch,
+            additions,
+            deletions,
+        });
+    }
+    return hunks;
+}
+export async function loadLocalDiff(filePath) {
+    const content = await readFile(filePath, 'utf-8');
+    return parseDiffFromString(content);
+}
+export function parseDiffFromString(diffText) {
+    const hunks = parseDiffText(diffText);
+    const files = hunks.map((h) => ({
+        filename: h.filename,
+        status: h.status,
+        additions: h.additions,
+        deletions: h.deletions,
+        patch: h.patch,
+        language: detectLanguage(h.filename),
+        previousFilename: h.previousFilename,
+    }));
+    return {
+        files,
+        source: 'local',
+        rawDiff: diffText,
+    };
+}
+//# sourceMappingURL=local.js.map

package/dist/resolver/local.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"local.js","sourceRoot":"","sources":["../../src/resolver/local.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,aAAa,CAAC;AACvC,OAAO,EAAE,cAAc,EAAE,MAAM,wBAAwB,CAAC;AAYxD,SAAS,aAAa,CAAC,QAAgB;IACrC,MAAM,KAAK,GAAiB,EAAE,CAAC;IAC/B,MAAM,UAAU,GAAG,QAAQ,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;IAEnE,KAAK,MAAM,KAAK,IAAI,UAAU,EAAE,CAAC;QAC/B,MAAM,KAAK,GAAG,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QAChC,MAAM,UAAU,GAAG,KAAK,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QAElC,yCAAyC;QACzC,MAAM,WAAW,GAAG,UAAU,CAAC,KAAK,CAAC,mBAAmB,CAAC,CAAC;QAC1D,IAAI,CAAC,WAAW;YAAE,SAAS;QAE3B,MAAM,KAAK,GAAG,WAAW,CAAC,CAAC,CAAE,CAAC;QAC9B,MAAM,KAAK,GAAG,WAAW,CAAC,CAAC,CAAE,CAAC;QAE9B,IAAI,MAAM,GAAyB,UAAU,CAAC;QAC9C,IAAI,gBAAoC,CAAC;QAEzC,MAAM,YAAY,GAAG,KAAK,CAAC,KAAK,CAAC,qBAAqB,CAAC,CAAC;QACxD,MAAM,YAAY,GAAG,KAAK,CAAC,KAAK,CAAC,iBAAiB,CAAC,CAAC;QACpD,MAAM,eAAe,GAAG,KAAK,CAAC,KAAK,CAAC,qBAAqB,CAAC,CAAC;QAC3D,MAAM,aAAa,GAAG,KAAK,CAAC,KAAK,CAAC,mBAAmB,CAAC,CAAC;QAEvD,IAAI,YAAY,EAAE,CAAC;YACjB,MAAM,GAAG,SAAS,CAAC;QACrB,CAAC;aAAM,IAAI,YAAY,EAAE,CAAC;YACxB,MAAM,GAAG,OAAO,CAAC;QACnB,CAAC;aAAM,IAAI,eAAe,IAAI,aAAa,EAAE,CAAC;YAC5C,MAAM,GAAG,SAAS,CAAC;YACnB,gBAAgB,GAAG,eAAe,CAAC,CAAC,CAAE,CAAC,IAAI,EAAE,CAAC;QAChD,CAAC;QAED,MAAM,QAAQ,GAAG,MAAM,KAAK,SAAS,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC;QAEtD,yCAAyC;QACzC,MAAM,UAAU,GAAG,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC;QACzC,MAAM,KAAK,GAAG,UAAU,IAAI,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,KAAK,CAAC,UAAU,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;QAEjE,IAAI,SAAS,GAAG,CAAC,CAAC;QAClB,IAAI,SAAS,GAAG,CAAC,CAAC;QAClB,KAAK,MAAM,IAAI,IAAI,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;YACrC,IAAI,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC;gBAAE,SAAS,EAAE,CAAC;YACjE,IAAI,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC;gBAAE,SAAS,EAAE,CAAC;QACnE,CAAC;QAED,KAAK,CAAC,IAAI,CAAC;YACT,QAAQ;YACR,MAAM;YACN,gBAAgB;YAChB,KAAK;YACL,SAAS;YACT,SAAS;SACV,CAAC,CAAC;IACL,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,aAAa,CAAC,QAAgB;IAClD,MAAM,OAAO,GAAG,MAAM,QAAQ,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAC;IAClD,OAAO,mBAAmB,CAAC,OAAO,CAAC,CAAC;AACtC,CAAC;AAED,MAAM,UAAU,mBAAmB,CAAC,QAAgB;IAClD,MAAM,KAAK,GAAG,aAAa,CAAC,QAAQ,CAAC,CAAC;IAEtC,MAAM,KAAK,GAAiB,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;QAC5C,QAAQ,EAAE,CAAC,CAAC,QAAQ;QACpB,MAAM,EAAE,CAAC,CAAC,MAAM;QAChB,SAAS,EAAE,CAAC,CAAC,SAAS;QACtB,SAAS,EAAE,CAAC,CAAC,SAAS;QACtB,KAAK,EAAE,CAAC,CAAC,KAAK;QACd,QAAQ,EAAE,cAAc,CAAC,CAAC,CAAC,QAAQ,CAAC;QACpC,gBAAgB,EAAE,CAAC,CAAC,gBAAgB;KACrC,CAAC,CAAC,CAAC;IAEJ,OAAO;QACL,KAAK;QACL,MAAM,EAAE,OAAO;QACf,OAAO,EAAE,QAAQ;KAClB,CAAC;AACJ,CAAC"}

package/dist/resolver/types.d.ts

@@ -0,0 +1,28 @@
+export interface FileChange {
+    filename: string;
+    status: 'added' | 'modified' | 'deleted' | 'renamed';
+    additions: number;
+    deletions: number;
+    patch: string;
+    language: string;
+    previousFilename?: string;
+}
+export interface PRMetadata {
+    owner: string;
+    repo: string;
+    number: number;
+    title: string;
+    body: string;
+    author: string;
+    base: string;
+    head: string;
+    url: string;
+    labels: string[];
+    draft: boolean;
+}
+export interface Diff {
+    files: FileChange[];
+    metadata?: PRMetadata;
+    source: 'github' | 'local';
+    rawDiff?: string;
+}

package/dist/resolver/types.js

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=types.js.map

package/dist/resolver/types.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"types.js","sourceRoot":"","sources":["../../src/resolver/types.ts"],"names":[],"mappings":""}

package/dist/roles/builtin.d.ts

@@ -0,0 +1,4 @@
+import type { Role } from './types.js';
+export declare const BUILTIN_ROLES: Role[];
+export declare function getRoleByName(name: string): Role | undefined;
+export declare function getBuiltinRoleNames(): string[];

package/dist/roles/builtin.js

@@ -0,0 +1,234 @@
+export const BUILTIN_ROLES = [
+    {
+        name: 'general',
+        description: 'Comprehensive code review covering all dimensions',
+        isSpecialized: false,
+        focus: ['security', 'correctness', 'best-practices', 'tests', 'api-design'],
+        systemPrompt: `You are an expert code reviewer performing a comprehensive review. Your role is to identify all types of issues: security vulnerabilities, logic errors, performance problems, missing tests, API design flaws, and code quality issues.
+
+Be thorough but focused — only report real issues visible in the diff. Prioritize severity accurately. Consider the full context of each change.`,
+    },
+    {
+        name: 'security-auditor',
+        description: 'Focused on security vulnerabilities: auth, injection, XSS, CSRF, IDOR',
+        isSpecialized: true,
+        focus: ['security'],
+        severityBias: { security: 1.2 },
+        systemPrompt: `You are a security-focused code reviewer with deep expertise in application security. Your primary mission is to identify security vulnerabilities and weaknesses.
+
+Focus areas:
+- Authentication and authorization flaws (broken auth, privilege escalation, IDOR)
+- Injection vulnerabilities (SQL, NoSQL, command, LDAP, XPath injection)
+- Cross-site scripting (XSS) — reflected, stored, and DOM-based
+- Cross-site request forgery (CSRF)
+- Sensitive data exposure (hardcoded secrets, improper encryption, insecure transmission)
+- Security misconfigurations (debug mode in prod, overly permissive CORS, insecure defaults)
+- Insecure deserialization
+- Input validation failures
+- Race conditions with security implications
+- Dependency vulnerabilities (if visible)
+
+Be specific about attack vectors and potential impact. Prefer security findings as "critical" or "important" when there is a real exploitable vulnerability.`,
+    },
+    {
+        name: 'performance-engineer',
+        description: 'Focus on performance: N+1 queries, caching, algorithmic complexity',
+        isSpecialized: true,
+        focus: ['correctness', 'best-practices'],
+        systemPrompt: `You are a performance-focused code reviewer. Your mission is to identify performance bottlenecks, inefficiencies, and scalability concerns.
+
+Focus areas:
+- N+1 query problems (database queries in loops)
+- Missing database indexes (new query patterns without indexes)
+- Inefficient algorithms (O(n²) when O(n log n) is available)
+- Missing caching opportunities (repeated expensive computations)
+- Memory inefficiencies (large object retention, missing streaming)
+- Unnecessary computation (recalculating constants, redundant processing)
+- Synchronous blocking in async contexts
+- Large payload sizes (over-fetching, missing pagination)
+- Connection pool exhaustion
+- Missing batch operations
+
+Estimate impact where possible (e.g., "for 10k rows, this will issue 10k queries").`,
+    },
+    {
+        name: 'api-design',
+        description: 'API contracts, backwards compatibility, interface design',
+        isSpecialized: true,
+        focus: ['api-design'],
+        systemPrompt: `You are an API design expert. Your mission is to evaluate the quality and correctness of API design, with particular attention to backwards compatibility and consumer experience.
+
+Focus areas:
+- Breaking changes to public APIs (changed signatures, removed fields, altered semantics)
+- REST conventions (correct HTTP methods, status codes, resource naming)
+- Backwards compatibility (new required fields, changed field types, deprecated but not removed)
+- Versioning strategy
+- Error response consistency
+- Missing or inadequate API documentation
+- Inconsistent naming conventions
+- Overly coupled interfaces
+- Missing pagination, filtering, or sorting for collection endpoints
+- GraphQL: N+1 in resolvers, missing rate limiting, overly broad queries allowed
+- gRPC/Protobuf: field numbering, backward compatibility of message changes
+
+Flag breaking changes as "critical" severity.`,
+    },
+    {
+        name: 'test-coverage',
+        description: 'Missing tests, edge cases, flawed test logic',
+        isSpecialized: true,
+        focus: ['tests'],
+        systemPrompt: `You are a testing expert. Your mission is to evaluate test coverage quality and identify missing or inadequate tests.
+
+Focus areas:
+- Missing tests for new functionality added in the diff
+- Edge cases not covered (empty inputs, null/undefined, boundary values, max values)
+- Error path testing (what happens when things fail)
+- Flawed test logic (tests that always pass, testing implementation not behavior)
+- Test isolation issues (shared state, ordering dependencies, missing cleanup)
+- Missing integration tests for critical paths
+- Mocking anti-patterns (over-mocking, mocking what you own)
+- Missing negative test cases
+- Tests without assertions
+- Brittle tests (relying on specific timing, order, or environment)
+
+Suggest specific test cases and scenarios that should be covered.`,
+    },
+    {
+        name: 'dx-critic',
+        description: 'Developer experience: readability, naming, documentation',
+        isSpecialized: true,
+        focus: ['best-practices'],
+        systemPrompt: `You are a developer experience (DX) expert. Your mission is to evaluate code readability, maintainability, and developer ergonomics.
+
+Focus areas:
+- Poor naming (unclear variable/function/class names that don't convey intent)
+- Excessive complexity (functions doing too many things, deep nesting)
+- Missing or inadequate documentation (complex algorithms without explanation)
+- Magic numbers and strings without constants
+- Inconsistent patterns within the codebase
+- Long functions that should be decomposed
+- Unclear error messages (from the developer/user perspective)
+- Missing or unhelpful logging
+- Over-engineering (unnecessary abstraction layers)
+- Code duplication that should be extracted
+- Confusing control flow
+
+Be constructive — suggest better names and approaches. Rate issues as "minor" or "nitpick" unless they significantly impact maintainability.`,
+    },
+    {
+        name: 'architecture',
+        description: 'Module boundaries, coupling, architectural patterns',
+        isSpecialized: true,
+        focus: ['best-practices', 'api-design'],
+        systemPrompt: `You are a software architect. Your mission is to evaluate architectural decisions, module boundaries, and coupling.
+
+Focus areas:
+- Violation of module/layer boundaries (e.g., UI code accessing DB directly)
+- Tight coupling between components that should be independent
+- Missing dependency inversion (high-level modules depending on low-level details)
+- Circular dependencies
+- God objects or modules that know too much
+- Missing abstractions (concrete implementations where interfaces are needed)
+- Architectural inconsistencies (mixing patterns in the same layer)
+- Side effects in unexpected places
+- State management issues (global state, inappropriate mutation)
+- Missing or incorrect use of design patterns
+- Scalability concerns at the architectural level
+
+Think in terms of long-term maintainability and the ability to change components independently.`,
+    },
+    {
+        name: 'bug-hunter',
+        description: 'Logic errors, null paths, race conditions, off-by-one',
+        isSpecialized: true,
+        focus: ['correctness'],
+        systemPrompt: `You are a bug hunter specializing in finding logic errors and correctness issues. Your mission is to find bugs in the code.
+
+Focus areas:
+- Logic errors (conditions that are always true/false, incorrect operator precedence)
+- Off-by-one errors (loop bounds, array indices, pagination)
+- Null/undefined dereference without proper checks
+- Race conditions and concurrency bugs (shared mutable state, TOCTOU)
+- Integer overflow/underflow
+- Floating point precision issues
+- Incorrect comparison (== vs ===, reference vs value equality)
+- Missing or incorrect error propagation
+- Incorrect state machine transitions
+- Resource cleanup in error paths
+- Incorrect handling of empty collections
+- Assumption violations (assuming sorted input, assuming specific locale)
+
+Be precise about when and how the bug manifests. Prefer "critical" or "important" severity for bugs that can cause incorrect behavior in production.`,
+    },
+    {
+        name: 'accessibility-auditor',
+        description: 'WCAG compliance, ARIA, keyboard navigation',
+        isSpecialized: true,
+        focus: ['best-practices'],
+        systemPrompt: `You are an accessibility expert. Your mission is to evaluate code for accessibility issues per WCAG 2.1 AA standards.
+
+Focus areas:
+- Missing or incorrect ARIA labels and roles
+- Images without alt text (or with poor alt text)
+- Color contrast issues (where identifiable from code)
+- Keyboard navigation (elements not reachable by keyboard, missing focus management)
+- Screen reader compatibility (missing live regions, incorrect reading order)
+- Form accessibility (labels not associated with inputs, missing error announcements)
+- Interactive elements not usable without mouse
+- Missing skip navigation links
+- Time-based content without controls
+- Missing captions or transcripts references
+- Focus traps (modal dialogs without proper focus management)
+- Semantic HTML violations (using divs where buttons/links are appropriate)
+
+Only flag issues visible in the code diff. Rate WCAG Level A violations as "critical", Level AA as "important".`,
+    },
+    {
+        name: 'project-rules',
+        description: 'Enforces repo conventions from AGENTS.md, CLAUDE.md, etc.',
+        isSpecialized: true,
+        focus: ['best-practices'],
+        systemPrompt: `You are a project conventions enforcer. Your mission is to ensure the code follows the repository's established rules, conventions, and patterns as defined in the project's rules files (AGENTS.md, CLAUDE.md, CONTRIBUTING.md, etc.).
+
+The project rules file content will be provided in the context section above. Review the diff against those rules specifically.
+
+Focus areas:
+- Naming conventions defined in the rules
+- File organization and structure requirements
+- Required patterns or anti-patterns to avoid
+- Commit message or PR conventions (if visible)
+- Testing requirements
+- Documentation requirements
+- Any project-specific architectural decisions
+
+Reference specific rule violations by quoting the rule that is being violated.`,
+    },
+    {
+        name: 'spec-compliance',
+        description: 'Checks implementation against a spec or plan file',
+        isSpecialized: true,
+        focus: ['correctness', 'api-design'],
+        systemPrompt: `You are a specification compliance reviewer. Your mission is to verify that the implementation matches the provided specification or design document.
+
+The specification content will be provided in the context section above. Review the diff against that specification.
+
+Focus areas:
+- Deviations from specified behavior
+- Missing required features or endpoints
+- Incorrect data types or field names vs spec
+- Missing validation described in spec
+- Incorrect error handling vs spec
+- Unspecified behavior that contradicts the spirit of the spec
+- Implementation gaps (things in spec not yet implemented)
+
+Be precise about which part of the spec is being violated or not met.`,
+    },
+];
+export function getRoleByName(name) {
+    return BUILTIN_ROLES.find((r) => r.name === name);
+}
+export function getBuiltinRoleNames() {
+    return BUILTIN_ROLES.map((r) => r.name);
+}
+//# sourceMappingURL=builtin.js.map

package/dist/roles/builtin.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"builtin.js","sourceRoot":"","sources":["../../src/roles/builtin.ts"],"names":[],"mappings":"AAEA,MAAM,CAAC,MAAM,aAAa,GAAW;IACnC;QACE,IAAI,EAAE,SAAS;QACf,WAAW,EAAE,mDAAmD;QAChE,aAAa,EAAE,KAAK;QACpB,KAAK,EAAE,CAAC,UAAU,EAAE,aAAa,EAAE,gBAAgB,EAAE,OAAO,EAAE,YAAY,CAAC;QAC3E,YAAY,EAAE;;iJAE+H;KAC9I;IACD;QACE,IAAI,EAAE,kBAAkB;QACxB,WAAW,EAAE,uEAAuE;QACpF,aAAa,EAAE,IAAI;QACnB,KAAK,EAAE,CAAC,UAAU,CAAC;QACnB,YAAY,EAAE,EAAE,QAAQ,EAAE,GAAG,EAAE;QAC/B,YAAY,EAAE;;;;;;;;;;;;;;6JAc2I;KAC1J;IACD;QACE,IAAI,EAAE,sBAAsB;QAC5B,WAAW,EAAE,oEAAoE;QACjF,aAAa,EAAE,IAAI;QACnB,KAAK,EAAE,CAAC,aAAa,EAAE,gBAAgB,CAAC;QACxC,YAAY,EAAE;;;;;;;;;;;;;;oFAckE;KACjF;IACD;QACE,IAAI,EAAE,YAAY;QAClB,WAAW,EAAE,0DAA0D;QACvE,aAAa,EAAE,IAAI;QACnB,KAAK,EAAE,CAAC,YAAY,CAAC;QACrB,YAAY,EAAE;;;;;;;;;;;;;;;8CAe4B;KAC3C;IACD;QACE,IAAI,EAAE,eAAe;QACrB,WAAW,EAAE,8CAA8C;QAC3D,aAAa,EAAE,IAAI;QACnB,KAAK,EAAE,CAAC,OAAO,CAAC;QAChB,YAAY,EAAE;;;;;;;;;;;;;;kEAcgD;KAC/D;IACD;QACE,IAAI,EAAE,WAAW;QACjB,WAAW,EAAE,0DAA0D;QACvE,aAAa,EAAE,IAAI;QACnB,KAAK,EAAE,CAAC,gBAAgB,CAAC;QACzB,YAAY,EAAE;;;;;;;;;;;;;;;6IAe2H;KAC1I;IACD;QACE,IAAI,EAAE,cAAc;QACpB,WAAW,EAAE,qDAAqD;QAClE,aAAa,EAAE,IAAI;QACnB,KAAK,EAAE,CAAC,gBAAgB,EAAE,YAAY,CAAC;QACvC,YAAY,EAAE;;;;;;;;;;;;;;;gGAe8E;KAC7F;IACD;QACE,IAAI,EAAE,YAAY;QAClB,WAAW,EAAE,uDAAuD;QACpE,aAAa,EAAE,IAAI;QACnB,KAAK,EAAE,CAAC,aAAa,CAAC;QACtB,YAAY,EAAE;;;;;;;;;;;;;;;;qJAgBmI;KAClJ;IACD;QACE,IAAI,EAAE,uBAAuB;QAC7B,WAAW,EAAE,4CAA4C;QACzD,aAAa,EAAE,IAAI;QACnB,KAAK,EAAE,CAAC,gBAAgB,CAAC;QACzB,YAAY,EAAE;;;;;;;;;;;;;;;;gHAgB8F;KAC7G;IACD;QACE,IAAI,EAAE,eAAe;QACrB,WAAW,EAAE,2DAA2D;QACxE,aAAa,EAAE,IAAI;QACnB,KAAK,EAAE,CAAC,gBAAgB,CAAC;QACzB,YAAY,EAAE;;;;;;;;;;;;;+EAa6D;KAC5E;IACD;QACE,IAAI,EAAE,iBAAiB;QACvB,WAAW,EAAE,mDAAmD;QAChE,aAAa,EAAE,IAAI;QACnB,KAAK,EAAE,CAAC,aAAa,EAAE,YAAY,CAAC;QACpC,YAAY,EAAE;;;;;;;;;;;;;sEAaoD;KACnE;CACF,CAAC;AAEF,MAAM,UAAU,aAAa,CAAC,IAAY;IACxC,OAAO,aAAa,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,IAAI,CAAC,CAAC;AACpD,CAAC;AAED,MAAM,UAAU,mBAAmB;IACjC,OAAO,aAAa,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;AAC1C,CAAC"}

package/dist/roles/dispatcher.d.ts

@@ -0,0 +1,23 @@
+import type { Role, ReviewAssignment } from './types.js';
+import type { ReviewerPair } from '../config/schema.js';
+export type ModelProvider = 'anthropic' | 'openai' | 'google' | 'openai-compat';
+export declare function detectProvider(model: string): ModelProvider;
+/**
+ * Build assignments from explicit reviewer pairs (--reviewer model:role)
+ */
+export declare function buildExplicitAssignments(reviewerPairs: ReviewerPair[], roleMap: Map<string, Role>): ReviewAssignment[];
+/**
+ * Build assignments using the dispatch algorithm:
+ * - general role: runs on ALL models
+ * - specialized roles: spread across models via shuffled round-robin
+ */
+export declare function buildRoleAssignments(models: string[], roles: Role[]): ReviewAssignment[];
+/**
+ * Main dispatch builder — handles explicit pairs or role-based assignment
+ */
+export declare function buildAssignments(opts: {
+    models: string[];
+    roles: Role[];
+    explicitReviewers?: ReviewerPair[];
+    roleMap: Map<string, Role>;
+}): ReviewAssignment[];

package/dist/roles/dispatcher.js

@@ -0,0 +1,87 @@
+export function detectProvider(model) {
+    // Handle explicit provider prefix (e.g. "anthropic/claude-sonnet-4-5")
+    if (model.startsWith('anthropic/'))
+        return 'anthropic';
+    if (model.startsWith('openai/'))
+        return 'openai';
+    if (model.startsWith('google/'))
+        return 'google';
+    if (model.startsWith('openai-compat/'))
+        return 'openai-compat';
+    // Detect by model name prefix
+    if (model.startsWith('claude'))
+        return 'anthropic';
+    if (model.startsWith('gpt') ||
+        model.startsWith('o1') ||
+        model.startsWith('o3') ||
+        model.startsWith('o4'))
+        return 'openai';
+    if (model.startsWith('gemini'))
+        return 'google';
+    // OpenAI-compatible for anything else (local models, etc.)
+    return 'openai-compat';
+}
+function shuffle(arr) {
+    const out = [...arr];
+    for (let i = out.length - 1; i > 0; i--) {
+        const j = Math.floor(Math.random() * (i + 1));
+        [out[i], out[j]] = [out[j], out[i]];
+    }
+    return out;
+}
+/**
+ * Build assignments from explicit reviewer pairs (--reviewer model:role)
+ */
+export function buildExplicitAssignments(reviewerPairs, roleMap) {
+    return reviewerPairs.flatMap(({ model, role: roleName }) => {
+        const role = roleMap.get(roleName);
+        if (!role) {
+            console.warn(`Warning: unknown role "${roleName}" in reviewer pair, skipping`);
+            return [];
+        }
+        return [{ model, role, provider: detectProvider(model) }];
+    });
+}
+/**
+ * Build assignments using the dispatch algorithm:
+ * - general role: runs on ALL models
+ * - specialized roles: spread across models via shuffled round-robin
+ */
+export function buildRoleAssignments(models, roles) {
+    const assignments = [];
+    const generalRoles = roles.filter((r) => !r.isSpecialized);
+    const specializedRoles = roles.filter((r) => r.isSpecialized);
+    // General roles: every model runs every general role
+    for (const model of models) {
+        for (const role of generalRoles) {
+            assignments.push({
+                model,
+                role,
+                provider: detectProvider(model),
+            });
+        }
+    }
+    // Specialized roles: shuffled round-robin across models
+    if (specializedRoles.length > 0 && models.length > 0) {
+        const shuffledModels = shuffle(models);
+        specializedRoles.forEach((role, index) => {
+            const model = shuffledModels[index % shuffledModels.length];
+            assignments.push({
+                model,
+                role,
+                provider: detectProvider(model),
+            });
+        });
+    }
+    return assignments;
+}
+/**
+ * Main dispatch builder — handles explicit pairs or role-based assignment
+ */
+export function buildAssignments(opts) {
+    if (opts.explicitReviewers && opts.explicitReviewers.length > 0) {
+        return buildExplicitAssignments(opts.explicitReviewers, opts.roleMap);
+    }
+    return buildRoleAssignments(opts.models, opts.roles);
+}
+//# sourceMappingURL=dispatcher.js.map

package/dist/roles/dispatcher.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"dispatcher.js","sourceRoot":"","sources":["../../src/roles/dispatcher.ts"],"names":[],"mappings":"AAKA,MAAM,UAAU,cAAc,CAAC,KAAa;IAC1C,uEAAuE;IACvE,IAAI,KAAK,CAAC,UAAU,CAAC,YAAY,CAAC;QAAE,OAAO,WAAW,CAAC;IACvD,IAAI,KAAK,CAAC,UAAU,CAAC,SAAS,CAAC;QAAE,OAAO,QAAQ,CAAC;IACjD,IAAI,KAAK,CAAC,UAAU,CAAC,SAAS,CAAC;QAAE,OAAO,QAAQ,CAAC;IACjD,IAAI,KAAK,CAAC,UAAU,CAAC,gBAAgB,CAAC;QAAE,OAAO,eAAe,CAAC;IAC/D,8BAA8B;IAC9B,IAAI,KAAK,CAAC,UAAU,CAAC,QAAQ,CAAC;QAAE,OAAO,WAAW,CAAC;IACnD,IACE,KAAK,CAAC,UAAU,CAAC,KAAK,CAAC;QACvB,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC;QACtB,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC;QACtB,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC;QAEtB,OAAO,QAAQ,CAAC;IAClB,IAAI,KAAK,CAAC,UAAU,CAAC,QAAQ,CAAC;QAAE,OAAO,QAAQ,CAAC;IAChD,2DAA2D;IAC3D,OAAO,eAAe,CAAC;AACzB,CAAC;AAED,SAAS,OAAO,CAAI,GAAQ;IAC1B,MAAM,GAAG,GAAG,CAAC,GAAG,GAAG,CAAC,CAAC;IACrB,KAAK,IAAI,CAAC,GAAG,GAAG,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;QACxC,MAAM,CAAC,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;QAC9C,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,CAAE,EAAE,GAAG,CAAC,CAAC,CAAE,CAAC,CAAC;IACxC,CAAC;IACD,OAAO,GAAG,CAAC;AACb,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,wBAAwB,CACtC,aAA6B,EAC7B,OAA0B;IAE1B,OAAO,aAAa,CAAC,OAAO,CAAC,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,EAAE,EAAE;QACzD,MAAM,IAAI,GAAG,OAAO,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC;QACnC,IAAI,CAAC,IAAI,EAAE,CAAC;YACV,OAAO,CAAC,IAAI,CAAC,0BAA0B,QAAQ,8BAA8B,CAAC,CAAC;YAC/E,OAAO,EAAE,CAAC;QACZ,CAAC;QACD,OAAO,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,cAAc,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;IAC5D,CAAC,CAAC,CAAC;AACL,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,oBAAoB,CAClC,MAAgB,EAChB,KAAa;IAEb,MAAM,WAAW,GAAuB,EAAE,CAAC;IAE3C,MAAM,YAAY,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,aAAa,CAAC,CAAC;IAC3D,MAAM,gBAAgB,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,aAAa,CAAC,CAAC;IAE9D,qDAAqD;IACrD,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC3B,KAAK,MAAM,IAAI,IAAI,YAAY,EAAE,CAAC;YAChC,WAAW,CAAC,IAAI,CAAC;gBACf,KAAK;gBACL,IAAI;gBACJ,QAAQ,EAAE,cAAc,CAAC,KAAK,CAAC;aAChC,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,wDAAwD;IACxD,IAAI,gBAAgB,CAAC,MAAM,GAAG,CAAC,IAAI,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACrD,MAAM,cAAc,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC;QACvC,gBAAgB,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE;YACvC,MAAM,KAAK,GAAG,cAAc,CAAC,KAAK,GAAG,cAAc,CAAC,MAAM,CAAE,CAAC;YAC7D,WAAW,CAAC,IAAI,CAAC;gBACf,KAAK;gBACL,IAAI;gBACJ,QAAQ,EAAE,cAAc,CAAC,KAAK,CAAC;aAChC,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;IACL,CAAC;IAED,OAAO,WAAW,CAAC;AACrB,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,gBAAgB,CAAC,IAKhC;IACC,IAAI,IAAI,CAAC,iBAAiB,IAAI,IAAI,CAAC,iBAAiB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAChE,OAAO,wBAAwB,CAAC,IAAI,CAAC,iBAAiB,EAAE,IAAI,CAAC,OAAO,CAAC,CAAC;IACxE,CAAC;IACD,OAAO,oBAAoB,CAAC,IAAI,CAAC,MAAM,EAAE,IAAI,CAAC,KAAK,CAAC,CAAC;AACvD,CAAC"}

package/dist/roles/loader.d.ts

@@ -0,0 +1,11 @@
+import type { Role } from './types.js';
+import type { Config } from '../config/schema.js';
+export declare function findProjectRulesFile(cwd?: string): Promise<string | null>;
+export declare function loadProjectRulesContent(cwd?: string): Promise<string | null>;
+export declare function buildCustomRole(config: {
+    name: string;
+    systemPrompt?: string;
+    focus?: string[];
+    severityBias?: Record<string, number>;
+}): Role;
+export declare function resolveRoles(config: Config, requestedRoles?: string[], projectRulesContent?: string, specContent?: string): Promise<Role[]>;

package/dist/roles/loader.js

@@ -0,0 +1,107 @@
+import { readFile, access } from 'fs/promises';
+import { join } from 'path';
+import { BUILTIN_ROLES, getRoleByName } from './builtin.js';
+const PROJECT_RULES_FILES = [
+    'AGENTS.md',
+    'CLAUDE.md',
+    'CONTRIBUTING.md',
+    '.github/CONTRIBUTING.md',
+    'DEVELOPMENT.md',
+    'docs/CONTRIBUTING.md',
+];
+export async function findProjectRulesFile(cwd = process.cwd()) {
+    for (const filename of PROJECT_RULES_FILES) {
+        const fullPath = join(cwd, filename);
+        try {
+            await access(fullPath);
+            return fullPath;
+        }
+        catch {
+            // not found, try next
+        }
+    }
+    return null;
+}
+export async function loadProjectRulesContent(cwd) {
+    const filePath = await findProjectRulesFile(cwd);
+    if (!filePath)
+        return null;
+    try {
+        return await readFile(filePath, 'utf-8');
+    }
+    catch {
+        return null;
+    }
+}
+export function buildCustomRole(config) {
+    // If it extends a builtin, start from that base
+    const base = getRoleByName(config.name);
+    return {
+        name: config.name,
+        description: `Custom role: ${config.name}`,
+        isSpecialized: true,
+        focus: config.focus ?? base?.focus ?? ['best-practices'],
+        severityBias: config.severityBias ?? base?.severityBias,
+        systemPrompt: config.systemPrompt ??
+            base?.systemPrompt ??
+            `You are a code reviewer with focus on: ${config.focus?.join(', ') ?? 'general quality'}.`,
+    };
+}
+export async function resolveRoles(config, requestedRoles, projectRulesContent, specContent) {
+    const roles = new Map();
+    // Add all builtin roles
+    for (const role of BUILTIN_ROLES) {
+        roles.set(role.name, role);
+    }
+    // Add custom roles from config (override builtins if same name)
+    if (config.customRoles) {
+        for (const customConfig of config.customRoles) {
+            roles.set(customConfig.name, buildCustomRole(customConfig));
+        }
+    }
+    // Augment project-rules role with actual content
+    if (projectRulesContent) {
+        const projectRulesRole = roles.get('project-rules');
+        roles.set('project-rules', {
+            ...projectRulesRole,
+            systemPrompt: projectRulesRole.systemPrompt +
+                '\n\n## Project Rules File Content\n\n' +
+                projectRulesContent,
+        });
+    }
+    // Augment spec-compliance role with actual spec content
+    if (specContent) {
+        const specRole = roles.get('spec-compliance');
+        roles.set('spec-compliance', {
+            ...specRole,
+            systemPrompt: specRole.systemPrompt +
+                '\n\n## Specification Content\n\n' +
+                specContent,
+        });
+    }
+    // Determine which roles to use
+    const requested = requestedRoles ?? config.roles;
+    if (requested && requested.length > 0) {
+        const hasAll = requested.some((r) => r.toLowerCase() === 'all');
+        if (hasAll) {
+            if (requested.length > 1) {
+                throw new Error(`'all' cannot be combined with other roles. Got: ${requested.join(', ')}`);
+            }
+            return Array.from(roles.values());
+        }
+        const result = [];
+        for (const name of requested) {
+            const role = roles.get(name);
+            if (role) {
+                result.push(role);
+            }
+            else {
+                console.warn(`Warning: unknown role "${name}", skipping`);
+            }
+        }
+        return result;
+    }
+    // Default: general + all specialized
+    return Array.from(roles.values());
+}
+//# sourceMappingURL=loader.js.map

package/dist/roles/loader.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"loader.js","sourceRoot":"","sources":["../../src/roles/loader.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,MAAM,aAAa,CAAC;AAC/C,OAAO,EAAE,IAAI,EAAE,MAAM,MAAM,CAAC;AAC5B,OAAO,EAAE,aAAa,EAAE,aAAa,EAAE,MAAM,cAAc,CAAC;AAI5D,MAAM,mBAAmB,GAAG;IAC1B,WAAW;IACX,WAAW;IACX,iBAAiB;IACjB,yBAAyB;IACzB,gBAAgB;IAChB,sBAAsB;CACvB,CAAC;AAEF,MAAM,CAAC,KAAK,UAAU,oBAAoB,CAAC,MAAc,OAAO,CAAC,GAAG,EAAE;IACpE,KAAK,MAAM,QAAQ,IAAI,mBAAmB,EAAE,CAAC;QAC3C,MAAM,QAAQ,GAAG,IAAI,CAAC,GAAG,EAAE,QAAQ,CAAC,CAAC;QACrC,IAAI,CAAC;YACH,MAAM,MAAM,CAAC,QAAQ,CAAC,CAAC;YACvB,OAAO,QAAQ,CAAC;QAClB,CAAC;QAAC,MAAM,CAAC;YACP,sBAAsB;QACxB,CAAC;IACH,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,uBAAuB,CAAC,GAAY;IACxD,MAAM,QAAQ,GAAG,MAAM,oBAAoB,CAAC,GAAG,CAAC,CAAC;IACjD,IAAI,CAAC,QAAQ;QAAE,OAAO,IAAI,CAAC;IAC3B,IAAI,CAAC;QACH,OAAO,MAAM,QAAQ,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAC;IAC3C,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAC;IACd,CAAC;AACH,CAAC;AAED,MAAM,UAAU,eAAe,CAAC,MAK/B;IACC,gDAAgD;IAChD,MAAM,IAAI,GAAG,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;IAExC,OAAO;QACL,IAAI,EAAE,MAAM,CAAC,IAAI;QACjB,WAAW,EAAE,gBAAgB,MAAM,CAAC,IAAI,EAAE;QAC1C,aAAa,EAAE,IAAI;QACnB,KAAK,EAAE,MAAM,CAAC,KAAK,IAAI,IAAI,EAAE,KAAK,IAAI,CAAC,gBAAgB,CAAC;QACxD,YAAY,EAAE,MAAM,CAAC,YAAY,IAAI,IAAI,EAAE,YAAY;QACvD,YAAY,EACV,MAAM,CAAC,YAAY;YACnB,IAAI,EAAE,YAAY;YAClB,0CAA0C,MAAM,CAAC,KAAK,EAAE,IAAI,CAAC,IAAI,CAAC,IAAI,iBAAiB,GAAG;KAC7F,CAAC;AACJ,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,YAAY,CAChC,MAAc,EACd,cAAyB,EACzB,mBAA4B,EAC5B,WAAoB;IAEpB,MAAM,KAAK,GAAG,IAAI,GAAG,EAAgB,CAAC;IAEtC,wBAAwB;IACxB,KAAK,MAAM,IAAI,IAAI,aAAa,EAAE,CAAC;QACjC,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC;IAC7B,CAAC;IAED,gEAAgE;IAChE,IAAI,MAAM,CAAC,WAAW,EAAE,CAAC;QACvB,KAAK,MAAM,YAAY,IAAI,MAAM,CAAC,WAAW,EAAE,CAAC;YAC9C,KAAK,CAAC,GAAG,CAAC,YAAY,CAAC,IAAI,EAAE,eAAe,CAAC,YAAY,CAAC,CAAC,CAAC;QAC9D,CAAC;IACH,CAAC;IAED,iDAAiD;IACjD,IAAI,mBAAmB,EAAE,CAAC;QACxB,MAAM,gBAAgB,GAAG,KAAK,CAAC,GAAG,CAAC,eAAe,CAAE,CAAC;QACrD,KAAK,CAAC,GAAG,CAAC,eAAe,EAAE;YACzB,GAAG,gBAAgB;YACnB,YAAY,EACV,gBAAgB,CAAC,YAAY;gBAC7B,uCAAuC;gBACvC,mBAAmB;SACtB,CAAC,CAAC;IACL,CAAC;IAED,wDAAwD;IACxD,IAAI,WAAW,EAAE,CAAC;QAChB,MAAM,QAAQ,GAAG,KAAK,CAAC,GAAG,CAAC,iBAAiB,CAAE,CAAC;QAC/C,KAAK,CAAC,GAAG,CAAC,iBAAiB,EAAE;YAC3B,GAAG,QAAQ;YACX,YAAY,EACV,QAAQ,CAAC,YAAY;gBACrB,kCAAkC;gBAClC,WAAW;SACd,CAAC,CAAC;IACL,CAAC;IAED,+BAA+B;IAC/B,MAAM,SAAS,GAAG,cAAc,IAAK,MAAM,CAAC,KAA8B,CAAC;IAC3E,IAAI,SAAS,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACtC,MAAM,MAAM,GAAG,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,WAAW,EAAE,KAAK,KAAK,CAAC,CAAC;QAChE,IAAI,MAAM,EAAE,CAAC;YACX,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;gBACzB,MAAM,IAAI,KAAK,CACb,mDAAmD,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAC1E,CAAC;YACJ,CAAC;YACD,OAAO,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC;QACpC,CAAC;QAED,MAAM,MAAM,GAAW,EAAE,CAAC;QAC1B,KAAK,MAAM,IAAI,IAAI,SAAS,EAAE,CAAC;YAC7B,MAAM,IAAI,GAAG,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;YAC7B,IAAI,IAAI,EAAE,CAAC;gBACT,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACpB,CAAC;iBAAM,CAAC;gBACN,OAAO,CAAC,IAAI,CAAC,0BAA0B,IAAI,aAAa,CAAC,CAAC;YAC5D,CAAC;QACH,CAAC;QACD,OAAO,MAAM,CAAC;IAChB,CAAC;IAED,qCAAqC;IACrC,OAAO,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC;AACpC,CAAC"}

package/dist/roles/types.d.ts

@@ -0,0 +1,13 @@
+export interface Role {
+    name: string;
+    systemPrompt: string;
+    focus: string[];
+    severityBias?: Record<string, number>;
+    description: string;
+    isSpecialized: boolean;
+}
+export interface ReviewAssignment {
+    model: string;
+    role: Role;
+    provider: string;
+}

package/dist/roles/types.js

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=types.js.map

package/dist/roles/types.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"types.js","sourceRoot":"","sources":["../../src/roles/types.ts"],"names":[],"mappings":""}