@motivation-labs/crosscheck 0.2.1-beta.f07295b.0 → 0.3.0

package/AGENT.md

@@ -0,0 +1,207 @@
+# AGENT.md — crosscheck optimize harness
+
+You are improving crosscheck's AI code review instructions. crosscheck runs `codex` and
+`claude` CLIs to review pull requests automatically. Both reviewers read
+`~/.crosscheck/instructions.md` before every review. Your job is to read the diagnostic
+report below and produce an improved `instructions.md` that increases review quality and
+reduces failures.
+
+> **Note:** `crosscheck optimize` selects which agent runs you based on your local config
+> and log history — whichever reviewer has the highest success rate, or `claude` if there
+> is no data. The instructions you produce are reviewer-agnostic: they are read by both
+> `claude` and `codex`, so write in plain language that both understand.
+
+---
+
+## Input you will receive
+
+1. **Diagnostic JSON** from `crosscheck diagnose --json` — error patterns, review outcomes,
+   repos seen, language signals, and suggestions.
+2. **Current `instructions.md`** — may be empty on first run.
+
+## Output you must produce
+
+Respond with only the new content of `instructions.md`. No explanation, no preamble, no
+markdown fences — just the file content. The file uses plain Markdown.
+
+---
+
+## Required sections (always present)
+
+### `## Constraints`
+What reviewers must NOT do. Each constraint is one bullet:
+```
+- Do not run [specific command].
+```
+
+### `## Focus`
+What reviewers should prioritize. Free-form prose or bullets.
+
+### `## Verdict format` (never modify this section)
+This section must always be preserved exactly as written in the current file. If it is
+missing from the current file, add it verbatim:
+
+```markdown
+## Verdict format
+
+On the very last line of your response, write exactly one of:
+
+VERDICT: APPROVE
+VERDICT: NEEDS WORK
+VERDICT: BLOCK
+
+Use APPROVE for no issues or trivial nits only.
+Use NEEDS WORK for addressable issues that are not blocking.
+Use BLOCK for security risks, data loss, broken API contracts, or correctness bugs.
+```
+
+---
+
+## Rules for the `## Constraints` section
+
+### Add a constraint when
+- `diagnostic.errors` contains a `command_not_found` pattern with `count >= 1` for a
+  specific command.
+- `diagnostic.errors` contains a `timeout` pattern with `count >= 2` for a reviewer that
+  is also producing `command_not_found` entries (likely spinning on a failed tool call).
+
+### Remove a constraint when
+- It was previously added for a specific command but that command no longer appears in
+  `diagnostic.errors` and has not appeared for the full log period analyzed.
+
+### Phrasing rules
+- Use the exact command name, not a category. Write `Do not run tsc.` not `Do not run
+  TypeScript build tools.`
+- One command per bullet. Do not combine: `Do not run tsc or npm.` → split into two.
+- Do not restrict reading. `Do not open package.json` is wrong. Constraints are for
+  execution only.
+- Do not restrict security analysis. Never add a constraint that would prevent a reviewer
+  from flagging a vulnerability.
+
+---
+
+## Language detection → constraint mapping
+
+Use `diagnostic.languages_detected` (list of detected language/tool identifiers) to seed
+the initial constraints. Add the corresponding constraint only if the language is detected
+AND a related `command_not_found` error is present OR this is the first run with no
+existing constraints.
+
+| Detected signal | Constraint to add |
+|---|---|
+| `typescript` / `tsconfig.json` / `package.json` | `Do not run tsc.` |
+| `nodejs` / `package.json` | `Do not run npm, npx, yarn, or pnpm.` |
+| `jest` / `vitest` in devDependencies | `Do not run jest or vitest.` |
+| `python` / `requirements.txt` / `pyproject.toml` | `Do not run pytest, pip, or python scripts.` |
+| `rust` / `Cargo.toml` | `Do not run cargo build or cargo test.` |
+| `go` / `go.mod` | `Do not run go build or go test.` |
+| `java` / `pom.xml` | `Do not run mvn.` |
+| `kotlin` / `gradle` / `build.gradle` | `Do not run gradle.` |
+| `ruby` / `Gemfile` | `Do not run bundle exec or rspec.` |
+
+Do not add constraints for languages not detected. Do not add all of the above blindly.
+
+---
+
+## Rules for the `## Focus` section
+
+### Update focus when
+- `diagnostic.verdict_distribution.APPROVE` percentage > 80% across >= 10 reviews → reviews
+  may be too lenient → add: "Apply strict scrutiny to error handling and edge cases."
+- `diagnostic.verdict_distribution.BLOCK` percentage > 30% across >= 10 reviews → reviews
+  may be too strict → add: "Prefer NEEDS WORK over BLOCK unless the issue causes data loss
+  or a security vulnerability."
+- `diagnostic.repos_seen` consistently includes infrastructure repos (name contains
+  `-infra`, `-deploy`, `-k8s`) → add: "Pay extra attention to secrets, env vars, and IAM
+  permissions in infrastructure changes."
+
+### Default focus (use when no signals override it)
+```markdown
+## Focus
+
+Review for correctness, security, and maintainability. Flag issues that would cause bugs
+in production, expose sensitive data, or make the code significantly harder to maintain.
+Nits and style preferences should be NEEDS WORK, not BLOCK.
+```
+
+---
+
+## Quality principles
+
+1. **Minimal** — fewer instructions produce better reviews. Each line must earn its place.
+2. **Specific** — exact command names, concrete criteria, not vague categories.
+3. **Evidence-based** — add only what the diagnostic shows is needed or what language
+   detection justifies.
+4. **Reversible** — stale constraints (no matching errors in the log period) should be
+   removed, not accumulated.
+5. **Reviewer-agnostic** — these instructions are read by both `codex` and `claude`. Write
+   them in plain language that both understand.
+
+---
+
+## What must never change
+
+- The `## Verdict format` section content and label.
+- Instructions that the user has manually annotated with `<!-- keep -->`.
+- The overall Markdown structure (## headings, bullet lists).
+
+---
+
+## Worked example
+
+### Input diagnostic (excerpt)
+```json
+{
+  "summary": { "total_reviews": 6, "successful": 3, "failed": 3, "failure_rate": 0.5 },
+  "errors": [
+    { "pattern": "command_not_found", "command": "tsc", "count": 2, "reviewer": "codex" },
+    { "pattern": "command_not_found", "command": "jest", "count": 1, "reviewer": "codex" },
+    { "pattern": "base_branch_missing", "branch": "staging", "count": 2 }
+  ],
+  "verdict_distribution": { "APPROVE": 2, "NEEDS_WORK": 1, "BLOCK": 0 },
+  "languages_detected": ["typescript", "nodejs"],
+  "suggestions": [
+    { "type": "add_constraint", "instruction": "Do not run tsc.", "reason": "tsc not found ×2" },
+    { "type": "add_constraint", "instruction": "Do not run jest.", "reason": "jest not found ×1" }
+  ]
+}
+```
+
+### Input current instructions.md
+```markdown
+## Verdict format
+
+On the very last line of your response, write exactly one of:
+...
+```
+
+### Correct output
+```markdown
+## Constraints
+
+- Do not run tsc.
+- Do not run jest.
+- Do not run npm, npx, yarn, or pnpm.
+
+## Focus
+
+Review for correctness, security, and maintainability. Flag issues that would cause bugs
+in production, expose sensitive data, or make the code significantly harder to maintain.
+Nits and style preferences should be NEEDS WORK, not BLOCK.
+
+## Verdict format
+
+On the very last line of your response, write exactly one of:
+
+VERDICT: APPROVE
+VERDICT: NEEDS WORK
+VERDICT: BLOCK
+
+Use APPROVE for no issues or trivial nits only.
+Use NEEDS WORK for addressable issues that are not blocking.
+Use BLOCK for security risks, data loss, broken API contracts, or correctness bugs.
+```
+
+Note: `Do not run npm, npx, yarn, or pnpm.` was added because `nodejs` was detected, even
+though no npm error appeared yet — this is the language-detection pre-emptive path for
+first-run seeding.

package/dist/__tests__/diagnose.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=diagnose.test.d.ts.map

package/dist/__tests__/diagnose.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"diagnose.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/diagnose.test.ts"],"names":[],"mappings":""}

package/dist/__tests__/diagnose.test.js

@@ -0,0 +1,164 @@
+import { describe, it, expect } from 'vitest';
+import { join } from 'path';
+import { fileURLToPath } from 'url';
+import { classifyError, buildDiagnoseReport } from '../commands/diagnose.js';
+import { INDICATORS } from '../lib/languages.js';
+const FIXTURES_DIR = join(fileURLToPath(import.meta.url), '..', 'fixtures');
+describe('classifyError', () => {
+    it('detects command_not_found for tsc', () => {
+        const r = classifyError('sh: tsc: command not found');
+        expect(r.pattern).toBe('command_not_found');
+        expect(r.command).toBe('tsc');
+    });
+    it('detects command_not_found (bash format)', () => {
+        const r = classifyError('bash: jest: not found');
+        expect(r.pattern).toBe('command_not_found');
+        expect(r.command).toBe('jest');
+    });
+    it('detects base_branch_missing with quoted branch', () => {
+        const r = classifyError("fatal: no such branch: 'staging'");
+        expect(r.pattern).toBe('base_branch_missing');
+        expect(r.branch).toBe('staging');
+    });
+    it('detects base_branch_missing without quotes', () => {
+        const r = classifyError('fatal: no such branch: main');
+        expect(r.pattern).toBe('base_branch_missing');
+        expect(r.branch).toBe('main');
+    });
+    it('detects timeout', () => {
+        const r = classifyError('Request timed out after 120000ms');
+        expect(r.pattern).toBe('timeout');
+    });
+    it('detects timeout (ETIMEDOUT)', () => {
+        const r = classifyError('connect ETIMEDOUT 10.0.0.1:443');
+        expect(r.pattern).toBe('timeout');
+    });
+    it('detects auth_failure on 401', () => {
+        const r = classifyError('Response 401: bad credentials');
+        expect(r.pattern).toBe('auth_failure');
+    });
+    it('detects auth_failure on unauthorized', () => {
+        const r = classifyError('Error: Unauthorized — check your token');
+        expect(r.pattern).toBe('auth_failure');
+    });
+    it('falls back to other for unrecognised message', () => {
+        const r = classifyError('some completely unknown error');
+        expect(r.pattern).toBe('other');
+    });
+});
+describe('LANGUAGE_CONSTRAINT_MAP / INDICATORS consistency', () => {
+    it('every language id in INDICATORS has a corresponding constraint entry in diagnose', () => {
+        // Import the internal map indirectly by checking that buildDiagnoseReport produces
+        // suggestions for every language id that INDICATORS can produce.
+        // We do this by asserting INDICATORS covers only known language ids, detected via
+        // a fixture with command_not_found errors for each mapped command.
+        // The key assertion: INDICATORS values are a subset of the constraint map keys.
+        // We expose this by checking that no language id from INDICATORS is silently dropped.
+        const indicatorLangs = new Set(INDICATORS.map(([, lang]) => lang));
+        // Build a minimal report with one error per language to trigger suggestions
+        // then assert the suggestions cover all indicator languages that are also
+        // in COMMAND_LANGUAGE_MAP (those have constraint entries).
+        // Simpler: just assert the known set matches expectation so any future addition
+        // to INDICATORS without a matching constraint fails here.
+        const expected = new Set(['typescript', 'nodejs', 'python', 'rust', 'golang', 'java', 'ruby']);
+        expect(indicatorLangs).toEqual(expected);
+    });
+});
+describe('buildDiagnoseReport', () => {
+    it('returns empty report when log dir does not exist', () => {
+        const r = buildDiagnoseReport(undefined, '/nonexistent/path/that/does/not/exist');
+        expect(r.summary.total_reviews).toBe(0);
+        expect(r.period.log_files).toBe(0);
+        expect(r.errors).toHaveLength(0);
+    });
+    it('parses fixture and counts reviews correctly', () => {
+        const r = buildDiagnoseReport(undefined, FIXTURES_DIR);
+        // 2026-01-10: 5 started, 3 complete; 2026-01-11: 3 started, 1 complete
+        expect(r.summary.total_reviews).toBe(8);
+        expect(r.summary.successful).toBe(4);
+        expect(r.summary.failed).toBe(4);
+    });
+    it('detects languages from review_started events', () => {
+        const r = buildDiagnoseReport(undefined, FIXTURES_DIR);
+        expect(r.languages_detected).toContain('typescript');
+        expect(r.languages_detected).toContain('nodejs');
+    });
+    it('identifies command_not_found errors', () => {
+        const r = buildDiagnoseReport(undefined, FIXTURES_DIR);
+        const tscErr = r.errors.find(e => e.pattern === 'command_not_found' && e.command === 'tsc');
+        expect(tscErr).toBeDefined();
+        expect(tscErr.count).toBe(1);
+        const jestErr = r.errors.find(e => e.pattern === 'command_not_found' && e.command === 'jest');
+        expect(jestErr).toBeDefined();
+    });
+    it('identifies base_branch_missing errors', () => {
+        const r = buildDiagnoseReport(undefined, FIXTURES_DIR);
+        const err = r.errors.find(e => e.pattern === 'base_branch_missing');
+        expect(err).toBeDefined();
+        expect(err.branch).toBe('staging');
+    });
+    it('identifies timeout errors', () => {
+        const r = buildDiagnoseReport(undefined, FIXTURES_DIR);
+        const err = r.errors.find(e => e.pattern === 'timeout');
+        expect(err).toBeDefined();
+    });
+    it('builds constraint suggestions from errors', () => {
+        const r = buildDiagnoseReport(undefined, FIXTURES_DIR);
+        const constraints = r.suggestions.filter(s => s.type === 'add_constraint');
+        expect(constraints.length).toBeGreaterThan(0);
+        expect(constraints.some(s => s.instruction?.includes('tsc'))).toBe(true);
+    });
+    it('tracks verdict distribution', () => {
+        const r = buildDiagnoseReport(undefined, FIXTURES_DIR);
+        expect(r.verdict_distribution.APPROVE).toBe(2);
+        expect(r.verdict_distribution.NEEDS_WORK).toBe(1);
+        expect(r.verdict_distribution.BLOCK).toBe(1);
+    });
+    it('aggregates reviewer performance', () => {
+        const r = buildDiagnoseReport(undefined, FIXTURES_DIR);
+        const claude = r.reviewer_performance['claude'];
+        const codex = r.reviewer_performance['codex'];
+        expect(claude).toBeDefined();
+        expect(codex).toBeDefined();
+        // 2026-01-10: claude 2 starts, 2 complete; 2026-01-11: claude 1 start, 1 complete
+        expect(claude.attempts).toBe(3);
+        expect(claude.successes).toBe(3);
+        // 2026-01-10: codex 3 starts, 1 complete; 2026-01-11: codex 2 starts, 0 complete
+        expect(codex.attempts).toBe(5);
+        expect(codex.successes).toBe(1);
+    });
+    it('--since filters to the specified date file only', () => {
+        const r = buildDiagnoseReport('2026-01-11', FIXTURES_DIR);
+        expect(r.period.log_files).toBe(1);
+        expect(r.summary.total_reviews).toBe(3);
+    });
+    it('tolerates malformed NDJSON lines', () => {
+        // 2026-01-11 fixture contains one malformed line — should not throw
+        expect(() => buildDiagnoseReport(undefined, FIXTURES_DIR)).not.toThrow();
+    });
+    it('repos_seen collects unique repo names', () => {
+        const r = buildDiagnoseReport(undefined, FIXTURES_DIR);
+        expect(r.repos_seen).toContain('acme/api');
+        expect(r.repos_seen).toContain('acme/web');
+    });
+    it('period.from and period.to reflect file names', () => {
+        const r = buildDiagnoseReport(undefined, FIXTURES_DIR);
+        expect(r.period.from).toBe('2026-01-10');
+        expect(r.period.to).toBe('2026-01-11');
+    });
+    it('failedReviews is never negative when --since truncates review_started events', () => {
+        // 2026-01-11 file has 3 starts and 1 complete.
+        // Using --since on 2026-01-11 only reads that file — no negative possible here,
+        // but we assert the invariant holds regardless.
+        const r = buildDiagnoseReport('2026-01-11', FIXTURES_DIR);
+        expect(r.summary.failed).toBeGreaterThanOrEqual(0);
+        expect(r.summary.failure_rate).toBeGreaterThanOrEqual(0);
+    });
+    it('reviewer attribution on errors uses review_started entry', () => {
+        // 2026-01-10: PR #1 was started by codex and errored with tsc not found
+        const r = buildDiagnoseReport('2026-01-10', FIXTURES_DIR);
+        const tscErr = r.errors.find(e => e.pattern === 'command_not_found' && e.command === 'tsc');
+        expect(tscErr?.reviewer).toBe('codex');
+    });
+});
+//# sourceMappingURL=diagnose.test.js.map

package/dist/__tests__/diagnose.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"diagnose.test.js","sourceRoot":"","sources":["../../src/__tests__/diagnose.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAA;AAC7C,OAAO,EAAE,IAAI,EAAE,MAAM,MAAM,CAAA;AAC3B,OAAO,EAAE,aAAa,EAAE,MAAM,KAAK,CAAA;AACnC,OAAO,EAAE,aAAa,EAAE,mBAAmB,EAAE,MAAM,yBAAyB,CAAA;AAC5E,OAAO,EAAE,UAAU,EAAE,MAAM,qBAAqB,CAAA;AAEhD,MAAM,YAAY,GAAG,IAAI,CAAC,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,EAAE,IAAI,EAAE,UAAU,CAAC,CAAA;AAE3E,QAAQ,CAAC,eAAe,EAAE,GAAG,EAAE;IAC7B,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC3C,MAAM,CAAC,GAAG,aAAa,CAAC,4BAA4B,CAAC,CAAA;QACrD,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAA;QAC3C,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IAC/B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,MAAM,CAAC,GAAG,aAAa,CAAC,uBAAuB,CAAC,CAAA;QAChD,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAA;QAC3C,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;IAChC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,MAAM,CAAC,GAAG,aAAa,CAAC,kCAAkC,CAAC,CAAA;QAC3D,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAA;QAC7C,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;IAClC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;QACpD,MAAM,CAAC,GAAG,aAAa,CAAC,6BAA6B,CAAC,CAAA;QACtD,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAA;QAC7C,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;IAC/B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,iBAAiB,EAAE,GAAG,EAAE;QACzB,MAAM,CAAC,GAAG,aAAa,CAAC,kCAAkC,CAAC,CAAA;QAC3D,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;IACnC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,6BAA6B,EAAE,GAAG,EAAE;QACrC,MAAM,CAAC,GAAG,aAAa,CAAC,gCAAgC,CAAC,CAAA;QACzD,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;IACnC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,6BAA6B,EAAE,GAAG,EAAE;QACrC,MAAM,CAAC,GAAG,aAAa,CAAC,+BAA+B,CAAC,CAAA;QACxD,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAA;IACxC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;QAC9C,MAAM,CAAC,GAAG,aAAa,CAAC,wCAAwC,CAAC,CAAA;QACjE,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAA;IACxC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,8CAA8C,EAAE,GAAG,EAAE;QACtD,MAAM,CAAC,GAAG,aAAa,CAAC,+BAA+B,CAAC,CAAA;QACxD,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;IACjC,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,kDAAkD,EAAE,GAAG,EAAE;IAChE,EAAE,CAAC,kFAAkF,EAAE,GAAG,EAAE;QAC1F,mFAAmF;QACnF,iEAAiE;QACjE,kFAAkF;QAClF,mEAAmE;QACnE,gFAAgF;QAChF,sFAAsF;QACtF,MAAM,cAAc,GAAG,IAAI,GAAG,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,IAAI,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,CAAC,CAAA;QAClE,4EAA4E;QAC5E,0EAA0E;QAC1E,2DAA2D;QAC3D,gFAAgF;QAChF,0DAA0D;QAC1D,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAC,CAAC,YAAY,EAAE,QAAQ,EAAE,QAAQ,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,MAAM,CAAC,CAAC,CAAA;QAC9F,MAAM,CAAC,cAAc,CAAC,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAA;IAC1C,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACnC,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,MAAM,CAAC,GAAG,mBAAmB,CAAC,SAAS,EAAE,uCAAuC,CAAC,CAAA;QACjF,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACvC,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAClC,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;IAClC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;QACrD,MAAM,CAAC,GAAG,mBAAmB,CAAC,SAAS,EAAE,YAAY,CAAC,CAAA;QACtD,uEAAuE;QACvE,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACvC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACpC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IAClC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,8CAA8C,EAAE,GAAG,EAAE;QACtD,MAAM,CAAC,GAAG,mBAAmB,CAAC,SAAS,EAAE,YAAY,CAAC,CAAA;QACtD,MAAM,CAAC,CAAC,CAAC,kBAAkB,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAA;QACpD,MAAM,CAAC,CAAC,CAAC,kBAAkB,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAA;IAClD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC7C,MAAM,CAAC,GAAG,mBAAmB,CAAC,SAAS,EAAE,YAAY,CAAC,CAAA;QACtD,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,KAAK,mBAAmB,IAAI,CAAC,CAAC,OAAO,KAAK,KAAK,CAAC,CAAA;QAC3F,MAAM,CAAC,MAAM,CAAC,CAAC,WAAW,EAAE,CAAA;QAC5B,MAAM,CAAC,MAAO,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAC7B,MAAM,OAAO,GAAG,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,KAAK,mBAAmB,IAAI,CAAC,CAAC,OAAO,KAAK,MAAM,CAAC,CAAA;QAC7F,MAAM,CAAC,OAAO,CAAC,CAAC,WAAW,EAAE,CAAA;IAC/B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;QAC/C,MAAM,CAAC,GAAG,mBAAmB,CAAC,SAAS,EAAE,YAAY,CAAC,CAAA;QACtD,MAAM,GAAG,GAAG,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,KAAK,qBAAqB,CAAC,CAAA;QACnE,MAAM,CAAC,GAAG,CAAC,CAAC,WAAW,EAAE,CAAA;QACzB,MAAM,CAAC,GAAI,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;IACrC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,2BAA2B,EAAE,GAAG,EAAE;QACnC,MAAM,CAAC,GAAG,mBAAmB,CAAC,SAAS,EAAE,YAAY,CAAC,CAAA;QACtD,MAAM,GAAG,GAAG,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,KAAK,SAAS,CAAC,CAAA;QACvD,MAAM,CAAC,GAAG,CAAC,CAAC,WAAW,EAAE,CAAA;IAC3B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACnD,MAAM,CAAC,GAAG,mBAAmB,CAAC,SAAS,EAAE,YAAY,CAAC,CAAA;QACtD,MAAM,WAAW,GAAG,CAAC,CAAC,WAAW,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,gBAAgB,CAAC,CAAA;QAC1E,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAA;QAC7C,MAAM,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,WAAW,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IAC1E,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,6BAA6B,EAAE,GAAG,EAAE;QACrC,MAAM,CAAC,GAAG,mBAAmB,CAAC,SAAS,EAAE,YAAY,CAAC,CAAA;QACtD,MAAM,CAAC,CAAC,CAAC,oBAAoB,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAC9C,MAAM,CAAC,CAAC,CAAC,oBAAoB,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACjD,MAAM,CAAC,CAAC,CAAC,oBAAoB,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IAC9C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,iCAAiC,EAAE,GAAG,EAAE;QACzC,MAAM,CAAC,GAAG,mBAAmB,CAAC,SAAS,EAAE,YAAY,CAAC,CAAA;QACtD,MAAM,MAAM,GAAG,CAAC,CAAC,oBAAoB,CAAC,QAAQ,CAAC,CAAA;QAC/C,MAAM,KAAK,GAAG,CAAC,CAAC,oBAAoB,CAAC,OAAO,CAAC,CAAA;QAC7C,MAAM,CAAC,MAAM,CAAC,CAAC,WAAW,EAAE,CAAA;QAC5B,MAAM,CAAC,KAAK,CAAC,CAAC,WAAW,EAAE,CAAA;QAC3B,kFAAkF;QAClF,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAC/B,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAChC,iFAAiF;QACjF,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAC9B,MAAM,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IACjC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,iDAAiD,EAAE,GAAG,EAAE;QACzD,MAAM,CAAC,GAAG,mBAAmB,CAAC,YAAY,EAAE,YAAY,CAAC,CAAA;QACzD,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAClC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IACzC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,kCAAkC,EAAE,GAAG,EAAE;QAC1C,oEAAoE;QACpE,MAAM,CAAC,GAAG,EAAE,CAAC,mBAAmB,CAAC,SAAS,EAAE,YAAY,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAA;IAC1E,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;QAC/C,MAAM,CAAC,GAAG,mBAAmB,CAAC,SAAS,EAAE,YAAY,CAAC,CAAA;QACtD,MAAM,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAA;QAC1C,MAAM,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAA;IAC5C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,8CAA8C,EAAE,GAAG,EAAE;QACtD,MAAM,CAAC,GAAG,mBAAmB,CAAC,SAAS,EAAE,YAAY,CAAC,CAAA;QACtD,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAA;QACxC,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAA;IACxC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,8EAA8E,EAAE,GAAG,EAAE;QACtF,+CAA+C;QAC/C,gFAAgF;QAChF,gDAAgD;QAChD,MAAM,CAAC,GAAG,mBAAmB,CAAC,YAAY,EAAE,YAAY,CAAC,CAAA;QACzD,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,sBAAsB,CAAC,CAAC,CAAC,CAAA;QAClD,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,sBAAsB,CAAC,CAAC,CAAC,CAAA;IAC1D,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,0DAA0D,EAAE,GAAG,EAAE;QAClE,wEAAwE;QACxE,MAAM,CAAC,GAAG,mBAAmB,CAAC,YAAY,EAAE,YAAY,CAAC,CAAA;QACzD,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,KAAK,mBAAmB,IAAI,CAAC,CAAC,OAAO,KAAK,KAAK,CAAC,CAAA;QAC3F,MAAM,CAAC,MAAM,EAAE,QAAQ,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;IACxC,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/__tests__/optimize.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=optimize.test.d.ts.map

package/dist/__tests__/optimize.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"optimize.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/optimize.test.ts"],"names":[],"mappings":""}

package/dist/__tests__/optimize.test.js

@@ -0,0 +1,88 @@
+import { describe, it, expect } from 'vitest';
+import { selectOptimizeAgent } from '../commands/optimize.js';
+function makeConfig(claudeEnabled, codexEnabled) {
+    return {
+        mode: 'cross-vendor',
+        orgs: [],
+        repos: [],
+        routing: { codex_reviews_patterns: [], claude_reviews_patterns: [] },
+        server: { port: 7892, webhook_path: '/webhook' },
+        quality: { tier: 'balanced', focus: [], custom_prompt: undefined },
+        budget: { codex_monthly_usd: null, per_review_usd: 1 },
+        vendors: {
+            claude: { enabled: claudeEnabled, auth: 'subscription', effort: 'medium' },
+            codex: { enabled: codexEnabled, auth: 'subscription', effort: 'medium' },
+        },
+        logs: { enabled: false, retention_days: 7 },
+    };
+}
+function makeReport(claudeAttempts = 0, claudeSuccesses = 0, codexAttempts = 0, codexSuccesses = 0) {
+    const performance = {};
+    if (claudeAttempts > 0) {
+        performance['claude'] = {
+            attempts: claudeAttempts,
+            successes: claudeSuccesses,
+            failure_rate: (claudeAttempts - claudeSuccesses) / claudeAttempts,
+        };
+    }
+    if (codexAttempts > 0) {
+        performance['codex'] = {
+            attempts: codexAttempts,
+            successes: codexSuccesses,
+            failure_rate: (codexAttempts - codexSuccesses) / codexAttempts,
+        };
+    }
+    return {
+        period: { from: 'N/A', to: 'N/A', log_files: 0 },
+        summary: { total_reviews: 0, successful: 0, failed: 0, failure_rate: 0 },
+        errors: [],
+        verdict_distribution: { APPROVE: 0, NEEDS_WORK: 0, BLOCK: 0 },
+        repos_seen: [],
+        languages_detected: [],
+        reviewer_performance: performance,
+        suggestions: [],
+    };
+}
+describe('selectOptimizeAgent', () => {
+    it('returns codex when only codex is enabled', () => {
+        const { agent } = selectOptimizeAgent(makeConfig(false, true), makeReport());
+        expect(agent).toBe('codex');
+    });
+    it('returns claude when only claude is enabled', () => {
+        const { agent } = selectOptimizeAgent(makeConfig(true, false), makeReport());
+        expect(agent).toBe('claude');
+    });
+    it('returns claude as default when both enabled and no log data', () => {
+        const { agent, reason } = selectOptimizeAgent(makeConfig(true, true), makeReport());
+        expect(agent).toBe('claude');
+        expect(reason).toMatch(/default/);
+    });
+    it('returns codex when both enabled and codex has higher success rate', () => {
+        // codex 80%, claude 50%
+        const { agent } = selectOptimizeAgent(makeConfig(true, true), makeReport(10, 5, 10, 8));
+        expect(agent).toBe('codex');
+    });
+    it('returns claude when both enabled and claude has higher success rate', () => {
+        // claude 90%, codex 60%
+        const { agent } = selectOptimizeAgent(makeConfig(true, true), makeReport(10, 9, 10, 6));
+        expect(agent).toBe('claude');
+    });
+    it('returns claude (default) when both enabled and rates are equal', () => {
+        // both 70%
+        const { agent } = selectOptimizeAgent(makeConfig(true, true), makeReport(10, 7, 10, 7));
+        expect(agent).toBe('claude');
+    });
+    it('throws when no vendors are enabled', () => {
+        expect(() => selectOptimizeAgent(makeConfig(false, false), makeReport())).toThrow(/No vendors enabled/);
+    });
+    it('reason string mentions source when only one vendor enabled', () => {
+        const { reason } = selectOptimizeAgent(makeConfig(true, false), makeReport());
+        expect(reason).toMatch(/only enabled vendor/);
+    });
+    it('reason string includes success rates when chosen by data', () => {
+        const { reason } = selectOptimizeAgent(makeConfig(true, true), makeReport(10, 5, 10, 8));
+        expect(reason).toMatch(/80%/);
+        expect(reason).toMatch(/50%/);
+    });
+});
+//# sourceMappingURL=optimize.test.js.map

package/dist/__tests__/optimize.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"optimize.test.js","sourceRoot":"","sources":["../../src/__tests__/optimize.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAA;AAC7C,OAAO,EAAE,mBAAmB,EAAE,MAAM,yBAAyB,CAAA;AAI7D,SAAS,UAAU,CAAC,aAAsB,EAAE,YAAqB;IAC/D,OAAO;QACL,IAAI,EAAE,cAAc;QACpB,IAAI,EAAE,EAAE;QACR,KAAK,EAAE,EAAE;QACT,OAAO,EAAE,EAAE,sBAAsB,EAAE,EAAE,EAAE,uBAAuB,EAAE,EAAE,EAAE;QACpE,MAAM,EAAE,EAAE,IAAI,EAAE,IAAI,EAAE,YAAY,EAAE,UAAU,EAAE;QAChD,OAAO,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,KAAK,EAAE,EAAE,EAAE,aAAa,EAAE,SAAS,EAAE;QAClE,MAAM,EAAE,EAAE,iBAAiB,EAAE,IAAI,EAAE,cAAc,EAAE,CAAC,EAAE;QACtD,OAAO,EAAE;YACP,MAAM,EAAE,EAAE,OAAO,EAAE,aAAa,EAAE,IAAI,EAAE,cAAc,EAAE,MAAM,EAAE,QAAQ,EAAE;YAC1E,KAAK,EAAE,EAAE,OAAO,EAAE,YAAY,EAAE,IAAI,EAAE,cAAc,EAAE,MAAM,EAAE,QAAQ,EAAE;SACzE;QACD,IAAI,EAAE,EAAE,OAAO,EAAE,KAAK,EAAE,cAAc,EAAE,CAAC,EAAE;KAC5C,CAAA;AACH,CAAC;AAED,SAAS,UAAU,CACjB,cAAc,GAAG,CAAC,EAAE,eAAe,GAAG,CAAC,EACvC,aAAa,GAAG,CAAC,EAAE,cAAc,GAAG,CAAC;IAErC,MAAM,WAAW,GAAkF,EAAE,CAAA;IACrG,IAAI,cAAc,GAAG,CAAC,EAAE,CAAC;QACvB,WAAW,CAAC,QAAQ,CAAC,GAAG;YACtB,QAAQ,EAAE,cAAc;YACxB,SAAS,EAAE,eAAe;YAC1B,YAAY,EAAE,CAAC,cAAc,GAAG,eAAe,CAAC,GAAG,cAAc;SAClE,CAAA;IACH,CAAC;IACD,IAAI,aAAa,GAAG,CAAC,EAAE,CAAC;QACtB,WAAW,CAAC,OAAO,CAAC,GAAG;YACrB,QAAQ,EAAE,aAAa;YACvB,SAAS,EAAE,cAAc;YACzB,YAAY,EAAE,CAAC,aAAa,GAAG,cAAc,CAAC,GAAG,aAAa;SAC/D,CAAA;IACH,CAAC;IACD,OAAO;QACL,MAAM,EAAE,EAAE,IAAI,EAAE,KAAK,EAAE,EAAE,EAAE,KAAK,EAAE,SAAS,EAAE,CAAC,EAAE;QAChD,OAAO,EAAE,EAAE,aAAa,EAAE,CAAC,EAAE,UAAU,EAAE,CAAC,EAAE,MAAM,EAAE,CAAC,EAAE,YAAY,EAAE,CAAC,EAAE;QACxE,MAAM,EAAE,EAAE;QACV,oBAAoB,EAAE,EAAE,OAAO,EAAE,CAAC,EAAE,UAAU,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE;QAC7D,UAAU,EAAE,EAAE;QACd,kBAAkB,EAAE,EAAE;QACtB,oBAAoB,EAAE,WAAW;QACjC,WAAW,EAAE,EAAE;KAChB,CAAA;AACH,CAAC;AAED,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACnC,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;QAClD,MAAM,EAAE,KAAK,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,KAAK,EAAE,IAAI,CAAC,EAAE,UAAU,EAAE,CAAC,CAAA;QAC5E,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;IAC7B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;QACpD,MAAM,EAAE,KAAK,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,IAAI,EAAE,KAAK,CAAC,EAAE,UAAU,EAAE,CAAC,CAAA;QAC5E,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;IAC9B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,6DAA6D,EAAE,GAAG,EAAE;QACrE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,IAAI,EAAE,IAAI,CAAC,EAAE,UAAU,EAAE,CAAC,CAAA;QACnF,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;QAC5B,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,SAAS,CAAC,CAAA;IACnC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,mEAAmE,EAAE,GAAG,EAAE;QAC3E,wBAAwB;QACxB,MAAM,EAAE,KAAK,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,IAAI,EAAE,IAAI,CAAC,EAAE,UAAU,CAAC,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC,CAAA;QACvF,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;IAC7B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,qEAAqE,EAAE,GAAG,EAAE;QAC7E,wBAAwB;QACxB,MAAM,EAAE,KAAK,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,IAAI,EAAE,IAAI,CAAC,EAAE,UAAU,CAAC,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC,CAAA;QACvF,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;IAC9B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,gEAAgE,EAAE,GAAG,EAAE;QACxE,WAAW;QACX,MAAM,EAAE,KAAK,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,IAAI,EAAE,IAAI,CAAC,EAAE,UAAU,CAAC,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC,CAAA;QACvF,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;IAC9B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;QAC5C,MAAM,CAAC,GAAG,EAAE,CAAC,mBAAmB,CAAC,UAAU,CAAC,KAAK,EAAE,KAAK,CAAC,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,oBAAoB,CAAC,CAAA;IACzG,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;QACpE,MAAM,EAAE,MAAM,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,IAAI,EAAE,KAAK,CAAC,EAAE,UAAU,EAAE,CAAC,CAAA;QAC7E,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,qBAAqB,CAAC,CAAA;IAC/C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,0DAA0D,EAAE,GAAG,EAAE;QAClE,MAAM,EAAE,MAAM,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,IAAI,EAAE,IAAI,CAAC,EAAE,UAAU,CAAC,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC,CAAA;QACxF,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,KAAK,CAAC,CAAA;QAC7B,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,KAAK,CAAC,CAAA;IAC/B,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/cli.js

@@ -5,6 +5,8 @@ import { runServe } from './commands/serve.js';
 import { runWatch } from './commands/watch.js';
 import { runReview } from './commands/review.js';
 import { runStatus } from './commands/status.js';
+import { runDiagnose } from './commands/diagnose.js';
+import { runOptimize } from './commands/optimize.js';
 const program = new Command();
 program
     .name('crosscheck')
@@ -36,5 +38,20 @@ program
     .description('Show auth state, config summary, and CLI versions')
     .option('-c, --config <path>', 'config file path')
     .action((opts) => void runStatus(opts.config));
+program
+    .command('diagnose')
+    .description('Analyze review logs — surface failure patterns, error trends, and improvement suggestions')
+    .option('--json', 'output full report as JSON')
+    .option('--since <date>', 'only analyze logs from this date onward (YYYY-MM-DD)')
+    .action((opts) => void runDiagnose(opts));
+program
+    .command('optimize')
+    .description('Use AI to improve review instructions based on diagnose output')
+    .option('--apply', 'write the improved instructions to ~/.crosscheck/instructions.md')
+    .option('--dry-run', 'show diff without writing (default behavior)')
+    .option('--agent <vendor>', 'force a specific agent: claude | codex')
+    .option('--since <date>', 'limit the diagnose window (YYYY-MM-DD)')
+    .option('-c, --config <path>', 'config file path')
+    .action((opts) => void runOptimize(opts));
 program.parse();
 //# sourceMappingURL=cli.js.map

package/dist/cli.js.map

@@ -1 +1 @@
-{"version":3,"file":"cli.js","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":";AACA,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAA;AACnC,OAAO,EAAE,OAAO,EAAE,MAAM,oBAAoB,CAAA;AAC5C,OAAO,EAAE,QAAQ,EAAE,MAAM,qBAAqB,CAAA;AAC9C,OAAO,EAAE,QAAQ,EAAE,MAAM,qBAAqB,CAAA;AAC9C,OAAO,EAAE,SAAS,EAAE,MAAM,sBAAsB,CAAA;AAChD,OAAO,EAAE,SAAS,EAAE,MAAM,sBAAsB,CAAA;AAEhD,MAAM,OAAO,GAAG,IAAI,OAAO,EAAE,CAAA;AAE7B,OAAO;KACJ,IAAI,CAAC,YAAY,CAAC;KAClB,WAAW,CAAC,mDAAmD,CAAC;KAChE,OAAO,CAAC,OAAO,CAAC,CAAA;AAEnB,OAAO;KACJ,OAAO,CAAC,MAAM,CAAC;KACf,WAAW,CAAC,0DAA0D,CAAC;KACvE,MAAM,CAAC,qBAAqB,EAAE,2BAA2B,CAAC;KAC1D,MAAM,CAAC,CAAC,IAAyB,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAA;AAE9D,OAAO;KACJ,OAAO,CAAC,OAAO,CAAC;KAChB,WAAW,CAAC,+DAA+D,CAAC;KAC5E,MAAM,CAAC,qBAAqB,EAAE,kBAAkB,CAAC;KACjD,MAAM,CAAC,CAAC,IAAyB,EAAE,EAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAA;AAE/D,OAAO;KACJ,OAAO,CAAC,OAAO,CAAC;KAChB,WAAW,CAAC,wDAAwD,CAAC;KACrE,MAAM,CAAC,qBAAqB,EAAE,kBAAkB,CAAC;KACjD,MAAM,CAAC,CAAC,IAAyB,EAAE,EAAE,CAAC,KAAK,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAA;AAEpE,OAAO;KACJ,OAAO,CAAC,iBAAiB,CAAC;KAC1B,WAAW,CAAC,+CAA+C,CAAC;KAC5D,MAAM,CAAC,qBAAqB,EAAE,kBAAkB,CAAC;KACjD,MAAM,CAAC,yBAAyB,EAAE,qEAAqE,CAAC;KACxG,MAAM,CAAC,CAAC,KAAa,EAAE,IAA4C,EAAE,EAAE,CAAC,KAAK,SAAS,CAAC,KAAK,EAAE,IAAI,CAAC,MAAM,EAAE,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAA;AAE7H,OAAO;KACJ,OAAO,CAAC,QAAQ,CAAC;KACjB,WAAW,CAAC,mDAAmD,CAAC;KAChE,MAAM,CAAC,qBAAqB,EAAE,kBAAkB,CAAC;KACjD,MAAM,CAAC,CAAC,IAAyB,EAAE,EAAE,CAAC,KAAK,SAAS,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAA;AAErE,OAAO,CAAC,KAAK,EAAE,CAAA"}
+{"version":3,"file":"cli.js","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":";AACA,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAA;AACnC,OAAO,EAAE,OAAO,EAAE,MAAM,oBAAoB,CAAA;AAC5C,OAAO,EAAE,QAAQ,EAAE,MAAM,qBAAqB,CAAA;AAC9C,OAAO,EAAE,QAAQ,EAAE,MAAM,qBAAqB,CAAA;AAC9C,OAAO,EAAE,SAAS,EAAE,MAAM,sBAAsB,CAAA;AAChD,OAAO,EAAE,SAAS,EAAE,MAAM,sBAAsB,CAAA;AAChD,OAAO,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAA;AACpD,OAAO,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAA;AAEpD,MAAM,OAAO,GAAG,IAAI,OAAO,EAAE,CAAA;AAE7B,OAAO;KACJ,IAAI,CAAC,YAAY,CAAC;KAClB,WAAW,CAAC,mDAAmD,CAAC;KAChE,OAAO,CAAC,OAAO,CAAC,CAAA;AAEnB,OAAO;KACJ,OAAO,CAAC,MAAM,CAAC;KACf,WAAW,CAAC,0DAA0D,CAAC;KACvE,MAAM,CAAC,qBAAqB,EAAE,2BAA2B,CAAC;KAC1D,MAAM,CAAC,CAAC,IAAyB,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAA;AAE9D,OAAO;KACJ,OAAO,CAAC,OAAO,CAAC;KAChB,WAAW,CAAC,+DAA+D,CAAC;KAC5E,MAAM,CAAC,qBAAqB,EAAE,kBAAkB,CAAC;KACjD,MAAM,CAAC,CAAC,IAAyB,EAAE,EAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAA;AAE/D,OAAO;KACJ,OAAO,CAAC,OAAO,CAAC;KAChB,WAAW,CAAC,wDAAwD,CAAC;KACrE,MAAM,CAAC,qBAAqB,EAAE,kBAAkB,CAAC;KACjD,MAAM,CAAC,CAAC,IAAyB,EAAE,EAAE,CAAC,KAAK,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAA;AAEpE,OAAO;KACJ,OAAO,CAAC,iBAAiB,CAAC;KAC1B,WAAW,CAAC,+CAA+C,CAAC;KAC5D,MAAM,CAAC,qBAAqB,EAAE,kBAAkB,CAAC;KACjD,MAAM,CAAC,yBAAyB,EAAE,qEAAqE,CAAC;KACxG,MAAM,CAAC,CAAC,KAAa,EAAE,IAA4C,EAAE,EAAE,CAAC,KAAK,SAAS,CAAC,KAAK,EAAE,IAAI,CAAC,MAAM,EAAE,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAA;AAE7H,OAAO;KACJ,OAAO,CAAC,QAAQ,CAAC;KACjB,WAAW,CAAC,mDAAmD,CAAC;KAChE,MAAM,CAAC,qBAAqB,EAAE,kBAAkB,CAAC;KACjD,MAAM,CAAC,CAAC,IAAyB,EAAE,EAAE,CAAC,KAAK,SAAS,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAA;AAErE,OAAO;KACJ,OAAO,CAAC,UAAU,CAAC;KACnB,WAAW,CAAC,2FAA2F,CAAC;KACxG,MAAM,CAAC,QAAQ,EAAE,4BAA4B,CAAC;KAC9C,MAAM,CAAC,gBAAgB,EAAE,sDAAsD,CAAC;KAChF,MAAM,CAAC,CAAC,IAAwC,EAAE,EAAE,CAAC,KAAK,WAAW,CAAC,IAAI,CAAC,CAAC,CAAA;AAE/E,OAAO;KACJ,OAAO,CAAC,UAAU,CAAC;KACnB,WAAW,CAAC,gEAAgE,CAAC;KAC7E,MAAM,CAAC,SAAS,EAAE,kEAAkE,CAAC;KACrF,MAAM,CAAC,WAAW,EAAE,8CAA8C,CAAC;KACnE,MAAM,CAAC,kBAAkB,EAAE,wCAAwC,CAAC;KACpE,MAAM,CAAC,gBAAgB,EAAE,wCAAwC,CAAC;KAClE,MAAM,CAAC,qBAAqB,EAAE,kBAAkB,CAAC;KACjD,MAAM,CAAC,CAAC,IAA4F,EAAE,EAAE,CAAC,KAAK,WAAW,CAAC,IAAI,CAAC,CAAC,CAAA;AAEnI,OAAO,CAAC,KAAK,EAAE,CAAA"}

package/dist/commands/diagnose.d.ts

@@ -0,0 +1,53 @@
+type ErrorPattern = 'command_not_found' | 'base_branch_missing' | 'timeout' | 'auth_failure' | 'other';
+interface ErrorEntry {
+    pattern: ErrorPattern;
+    command?: string;
+    branch?: string;
+    count: number;
+    reviewer?: string;
+}
+interface ReviewerPerf {
+    attempts: number;
+    successes: number;
+    failure_rate: number;
+}
+interface Suggestion {
+    type: 'add_constraint' | 'investigate' | 'config_change';
+    instruction?: string;
+    reason: string;
+}
+export interface DiagnoseReport {
+    period: {
+        from: string;
+        to: string;
+        log_files: number;
+    };
+    summary: {
+        total_reviews: number;
+        successful: number;
+        failed: number;
+        failure_rate: number;
+    };
+    errors: ErrorEntry[];
+    verdict_distribution: {
+        APPROVE: number;
+        NEEDS_WORK: number;
+        BLOCK: number;
+    };
+    repos_seen: string[];
+    languages_detected: string[];
+    reviewer_performance: Record<string, ReviewerPerf>;
+    suggestions: Suggestion[];
+}
+export declare function classifyError(message: string): {
+    pattern: ErrorPattern;
+    command?: string;
+    branch?: string;
+};
+export declare function buildDiagnoseReport(since?: string, logDir?: string): DiagnoseReport;
+export declare function runDiagnose(opts: {
+    json?: boolean;
+    since?: string;
+}): Promise<void>;
+export {};
+//# sourceMappingURL=diagnose.d.ts.map

package/dist/commands/diagnose.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"diagnose.d.ts","sourceRoot":"","sources":["../../src/commands/diagnose.ts"],"names":[],"mappings":"AAQA,KAAK,YAAY,GAAG,mBAAmB,GAAG,qBAAqB,GAAG,SAAS,GAAG,cAAc,GAAG,OAAO,CAAA;AAEtG,UAAU,UAAU;IAClB,OAAO,EAAE,YAAY,CAAA;IACrB,OAAO,CAAC,EAAE,MAAM,CAAA;IAChB,MAAM,CAAC,EAAE,MAAM,CAAA;IACf,KAAK,EAAE,MAAM,CAAA;IACb,QAAQ,CAAC,EAAE,MAAM,CAAA;CAClB;AAED,UAAU,YAAY;IACpB,QAAQ,EAAE,MAAM,CAAA;IAChB,SAAS,EAAE,MAAM,CAAA;IACjB,YAAY,EAAE,MAAM,CAAA;CACrB;AAED,UAAU,UAAU;IAClB,IAAI,EAAE,gBAAgB,GAAG,aAAa,GAAG,eAAe,CAAA;IACxD,WAAW,CAAC,EAAE,MAAM,CAAA;IACpB,MAAM,EAAE,MAAM,CAAA;CACf;AAED,MAAM,WAAW,cAAc;IAC7B,MAAM,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,EAAE,EAAE,MAAM,CAAC;QAAC,SAAS,EAAE,MAAM,CAAA;KAAE,CAAA;IACvD,OAAO,EAAE;QAAE,aAAa,EAAE,MAAM,CAAC;QAAC,UAAU,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAC;QAAC,YAAY,EAAE,MAAM,CAAA;KAAE,CAAA;IAC5F,MAAM,EAAE,UAAU,EAAE,CAAA;IACpB,oBAAoB,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,UAAU,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,CAAA;IAC5E,UAAU,EAAE,MAAM,EAAE,CAAA;IACpB,kBAAkB,EAAE,MAAM,EAAE,CAAA;IAC5B,oBAAoB,EAAE,MAAM,CAAC,MAAM,EAAE,YAAY,CAAC,CAAA;IAClD,WAAW,EAAE,UAAU,EAAE,CAAA;CAC1B;AAkDD,wBAAgB,aAAa,CAAC,OAAO,EAAE,MAAM,GAAG;IAAE,OAAO,EAAE,YAAY,CAAC;IAAC,OAAO,CAAC,EAAE,MAAM,CAAC;IAAC,MAAM,CAAC,EAAE,MAAM,CAAA;CAAE,CAY3G;AA8DD,wBAAgB,mBAAmB,CAAC,KAAK,CAAC,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,GAAG,cAAc,CAkGnF;AAuED,wBAAsB,WAAW,CAAC,IAAI,EAAE;IAAE,IAAI,CAAC,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAkBzF"}