@allocator-one/rcl 4.4.19

package/dist/resolver/types.d.ts

@@ -0,0 +1,43 @@
+export interface FileChange {
+    filename: string;
+    status: 'added' | 'modified' | 'deleted' | 'renamed';
+    additions: number;
+    deletions: number;
+    patch: string;
+    language: string;
+    previousFilename?: string;
+    /**
+     * Git blob id of the file after the change, when the source provides it
+     * (GitHub does). It binds a patchless file — binary, or too large for a
+     * patch — to its content in the diff digest.
+     */
+    blobSha?: string;
+}
+export interface PRMetadata {
+    owner: string;
+    repo: string;
+    number: number;
+    title: string;
+    body: string;
+    author: string;
+    base: string;
+    head: string;
+    /**
+     * Exact commit ids from `pulls.get` — the evidence ledger's head binding.
+     * GitHub always returns both, so a PR is never bound without them.
+     */
+    headSha: string;
+    baseSha: string;
+    /** Actual compare merge base, explicitly requested for retained PR execution. */
+    mergeBaseSha?: string;
+    mergeCommitSha?: 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,282 @@
+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: '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.`,
+    },
+    {
+        name: 'regression-hunter',
+        description: 'Behavioral regressions from refactoring: changed defaults, weakened guards, lost semantics',
+        isSpecialized: true,
+        focus: ['correctness'],
+        severityBias: { correctness: 1.3 },
+        systemPrompt: `You are a refactoring regression hunter. Your mission is to find behavioral changes silently introduced during code refactoring — things that pass tests but alter production behavior.
+
+Focus areas:
+- Changed defaults (e.g. shadow DOM mode, timeout values, retry counts, delay durations, open/closed, public/private)
+- Removed or weakened guards, validations, or security properties
+- Altered fallback chains (different order, missing fallbacks, new fallbacks that are too broad)
+- API contract drift (renamed fields, changed types, different nullability)
+- Scope changes (closed → open, private → public, restricted → permissive)
+- Lost error handling (try/catch removed, error states no longer reachable)
+- Changed side-effect ordering (operations that now happen in a different sequence)
+- Feature flags or configuration that changed meaning
+- String literals that changed (mode names, CSS classes, attribute values, event names)
+
+Compare old code vs new code line by line when both are visible in the diff. When code moves between files, verify every behavioral detail survived the move. Flag anything where the observable behavior differs, even if tests still pass.
+
+Severity: mark regressions that weaken security or correctness as "critical". Mark changed defaults that alter UX as "important".`,
+    },
+    {
+        name: 'dependency-hygiene',
+        description: 'External runtime fetches, privacy leaks, sensitive console.log, unnecessary permissions',
+        isSpecialized: true,
+        focus: ['security', 'best-practices'],
+        severityBias: { security: 1.2 },
+        systemPrompt: `You are a dependency and external-request auditor. Your mission is to find unnecessary external dependencies, runtime fetches, and privacy-leaking requests in production code.
+
+Focus areas:
+- Runtime fetches to external CDNs or services (Google Fonts, analytics, tracking pixels, external stylesheets) embedded in production code
+- Fonts loaded via @import or <link> that could be bundled or replaced with system fonts
+- External resources fetched on every mount, render, or navigation (cost multiplied by user activity)
+- Privacy leaks: any request to a third party that reveals user browsing behavior (which pages they visit, when, how often)
+- Content Security Policy conflicts: external resources that may be blocked by the host page's CSP
+- Console.log or console.error calls in production code that leak sensitive data (auth tokens, API keys, user PII, profile URLs, full request/response bodies)
+- Dependencies in package.json that are imported nowhere in application code
+- Overly broad permissions in manifests (browser extension host_permissions, OAuth scopes, IAM policies)
+- Hardcoded API keys, tokens, or secrets (even in non-secret positions)
+
+Severity: privacy-leaking external requests are "critical". Unnecessary CDN loads and verbose sensitive logging are "important". Unused deps are "minor".`,
+    },
+    {
+        name: 'edge-case-hunter',
+        description: 'Case sensitivity, incomplete lists, stale state, normalization gaps, broad fallbacks',
+        isSpecialized: true,
+        focus: ['correctness'],
+        severityBias: { correctness: 1.2 },
+        systemPrompt: `You are an edge case hunter. Your mission is to find incomplete handling of real-world input variations that cause subtle bugs in production.
+
+Focus areas:
+- Case sensitivity on identifiers that should be case-insensitive (usernames, email addresses, domain names, URL slugs, social media handles, file extensions)
+- Incomplete exclusion or inclusion lists (missing entries that real users will hit — e.g. reserved route paths, restricted keywords, known special values)
+- Overly broad fallbacks that match unintended targets (e.g. falling back to searching the entire DOM instead of a scoped container, or catching all exceptions instead of specific ones)
+- Missing URL normalization (trailing slashes, query params, fragments, scheme differences, www prefix, percent-encoding, port numbers)
+- Locale and encoding assumptions (assuming ASCII, assuming English, hardcoded locale-specific strings like date formats)
+- Empty, null, or undefined inputs that bypass validation but break downstream logic
+- Boundary values (max-length inputs, zero-length inputs, exactly-at-limit values)
+- Stale DOM or state from SPA navigation (meta tags, canonical links, or cached data from the previous page still present in the DOM after navigation)
+- Time-of-check vs time-of-use gaps where the world changes between reading a value and acting on it
+- Unicode edge cases (zero-width characters, RTL marks, combining characters in identifiers)
+
+Be specific about the exact input that triggers the bug and what incorrect behavior results. Provide a concrete example.
+
+Severity: bugs that cause data corruption or duplication are "critical". Bugs that cause incorrect UI state or wasted backend work are "important". Cosmetic or unlikely edge cases are "minor".`,
+    },
+];
+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,iBAAiB;QACvB,WAAW,EAAE,mDAAmD;QAChE,aAAa,EAAE,IAAI;QACnB,KAAK,EAAE,CAAC,aAAa,EAAE,YAAY,CAAC;QACpC,YAAY,EAAE;;;;;;;;;;;;;sEAaoD;KACnE;IACD;QACE,IAAI,EAAE,mBAAmB;QACzB,WAAW,EAAE,4FAA4F;QACzG,aAAa,EAAE,IAAI;QACnB,KAAK,EAAE,CAAC,aAAa,CAAC;QACtB,YAAY,EAAE,EAAE,WAAW,EAAE,GAAG,EAAE;QAClC,YAAY,EAAE;;;;;;;;;;;;;;;kIAegH;KAC/H;IACD;QACE,IAAI,EAAE,oBAAoB;QAC1B,WAAW,EAAE,yFAAyF;QACtG,aAAa,EAAE,IAAI;QACnB,KAAK,EAAE,CAAC,UAAU,EAAE,gBAAgB,CAAC;QACrC,YAAY,EAAE,EAAE,QAAQ,EAAE,GAAG,EAAE;QAC/B,YAAY,EAAE;;;;;;;;;;;;;0JAawI;KACvJ;IACD;QACE,IAAI,EAAE,kBAAkB;QACxB,WAAW,EAAE,sFAAsF;QACnG,aAAa,EAAE,IAAI;QACnB,KAAK,EAAE,CAAC,aAAa,CAAC;QACtB,YAAY,EAAE,EAAE,WAAW,EAAE,GAAG,EAAE;QAClC,YAAY,EAAE;;;;;;;;;;;;;;;;iMAgB+K;KAC9L;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,26 @@
+import type { Role, ReviewAssignment } from './types.js';
+import type { ReviewerPair } from '../config/schema.js';
+import type { ModelProvider } from '../config/providers.js';
+export type { ModelProvider } from '../config/providers.js';
+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 primary (SOTA) models only
+ * - specialized roles: spread across ALL models (primary + secondary) via shuffled round-robin
+ */
+export declare function buildRoleAssignments(models: string[], roles: Role[], secondaryModels?: string[], deterministic?: boolean): ReviewAssignment[];
+/**
+ * Main dispatch builder — handles explicit pairs or role-based assignment
+ */
+export declare function buildAssignments(opts: {
+    models: string[];
+    roles: Role[];
+    secondaryModels?: string[];
+    explicitReviewers?: ReviewerPair[];
+    roleMap: Map<string, Role>;
+    deterministic?: boolean;
+}): ReviewAssignment[];

package/dist/roles/dispatcher.js

@@ -0,0 +1,96 @@
+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('openrouter/'))
+        return 'openrouter';
+    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) {
+    const assignments = reviewerPairs.flatMap(({ model, role: roleName }) => {
+        const role = roleMap.get(roleName) ?? roleMap.get(roleName.toLowerCase());
+        if (!role) {
+            console.warn(`Warning: unknown role "${roleName}" in reviewer pair, skipping`);
+            return [];
+        }
+        return [{ model, role, provider: detectProvider(model) }];
+    });
+    // All pairs invalid must fail loudly — proceeding would run zero reviews
+    // and emit an empty (green-looking) report.
+    if (reviewerPairs.length > 0 && assignments.length === 0) {
+        throw new Error('No valid --reviewer pairs: every role was unknown. Run `rcl roles list` to see available roles.');
+    }
+    return assignments;
+}
+/**
+ * Build assignments using the dispatch algorithm:
+ * - general role: runs on primary (SOTA) models only
+ * - specialized roles: spread across ALL models (primary + secondary) via shuffled round-robin
+ */
+export function buildRoleAssignments(models, roles, secondaryModels = [], deterministic = false) {
+    const assignments = [];
+    const generalRoles = roles.filter((r) => !r.isSpecialized);
+    const specializedRoles = roles.filter((r) => r.isSpecialized);
+    // General roles: only primary (SOTA) models
+    for (const model of models) {
+        for (const role of generalRoles) {
+            assignments.push({
+                model,
+                role,
+                provider: detectProvider(model),
+            });
+        }
+    }
+    // Specialized roles: shuffled round-robin across ALL models (primary + secondary)
+    const allModels = [...new Set([...models, ...secondaryModels])];
+    if (specializedRoles.length > 0 && allModels.length > 0) {
+        const shuffledModels = deterministic ? allModels : shuffle(allModels);
+        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, opts.secondaryModels, opts.deterministic);
+}
+//# 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":"AAMA,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,aAAa,CAAC;QAAE,OAAO,YAAY,CAAC;IACzD,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,MAAM,WAAW,GAAG,aAAa,CAAC,OAAO,CAAC,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,EAAE,EAAE;QACtE,MAAM,IAAI,GAAG,OAAO,CAAC,GAAG,CAAC,QAAQ,CAAC,IAAI,OAAO,CAAC,GAAG,CAAC,QAAQ,CAAC,WAAW,EAAE,CAAC,CAAC;QAC1E,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;IAEH,yEAAyE;IACzE,4CAA4C;IAC5C,IAAI,aAAa,CAAC,MAAM,GAAG,CAAC,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACzD,MAAM,IAAI,KAAK,CACb,iGAAiG,CAClG,CAAC;IACJ,CAAC;IAED,OAAO,WAAW,CAAC;AACrB,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,oBAAoB,CAClC,MAAgB,EAChB,KAAa,EACb,kBAA4B,EAAE,EAC9B,aAAa,GAAG,KAAK;IAErB,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,4CAA4C;IAC5C,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,kFAAkF;IAClF,MAAM,SAAS,GAAG,CAAC,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,MAAM,EAAE,GAAG,eAAe,CAAC,CAAC,CAAC,CAAC;IAChE,IAAI,gBAAgB,CAAC,MAAM,GAAG,CAAC,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACxD,MAAM,cAAc,GAAG,aAAa,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC;QACtE,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,IAOhC;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,EAAE,IAAI,CAAC,eAAe,EAAE,IAAI,CAAC,aAAa,CAAC,CAAC;AACjG,CAAC"}

package/dist/roles/legacy-4.1.10.d.ts

@@ -0,0 +1,2 @@
+import type { Role } from './types.js';
+export declare const LEGACY_BUILTIN_ROLES_4_1_10: Role[];