revu-ai 0.2.1 → 0.3.0

package/dist/types.d.ts

@@ -8,6 +8,40 @@ export interface RuleFile {
     absPath: string;
     relPath: string;
     content: string;
+    /** Glob patterns (repo-root-relative) specifying which changed files this rule applies to.
+     *  Parsed from the YAML frontmatter `files:` field. When absent, the rule applies to all changed files. */
+    filePatterns?: string[];
+}
+export interface RuleResult {
+    id: string;
+    path: string;
+    ok: boolean;
+    durationMs: number;
+    findingCount: number;
+    /** Count of `report_review_summary` calls the agent emitted via the MCP sidecar.
+     *  Expected to be exactly 1 for a healthy review. 0 means the agent never
+     *  signed off — usually a symptom of a broken / silent agent — and surfaces
+     *  as an "incomplete review" warning. >1 means the agent called it multiple
+     *  times; only the first is kept. Always present on non-skipped rules. */
+    summaryCount: number;
+    /** Count of `report_check` calls the agent emitted — incremental compliance
+     *  notes. No required minimum; high values just mean the agent showed its
+     *  working. */
+    checkCount: number;
+    errorMessage?: string;
+    /** True if this rule was stopped by the per-rule timeout. */
+    timedOut?: boolean;
+    /** True if this rule was skipped because no changed files matched its `files:` patterns. */
+    skipped?: boolean;
+    /** Observability counters from the provider — text chars emitted and
+     *  report_finding tool calls observed. The pretty output surfaces a warning
+     *  when textChars is high but findingCount is 0 (model wrote findings as
+     *  prose instead of calling the MCP tool). Absent for skipped rules and for
+     *  providers that don't report them. */
+    diagnostics?: {
+        textChars: number;
+        findingToolCalls: number;
+    };
 }
 export type ReviewTarget = {
     mode: "ref-range";
@@ -53,20 +87,34 @@ export interface Resolution {
     /** Commit at which the agent considered the finding resolved. */
     resolvedAtSha: string;
 }
-export interface RuleResult {
-    id: string;
+export interface Check {
+    ruleId: string;
+    /** Repo-relative file path the agent verified. */
     path: string;
-    ok: boolean;
-    durationMs: number;
-    findingCount: number;
-    errorMessage?: string;
-    /** True if this rule was stopped by the per-rule timeout. */
-    timedOut?: boolean;
+    line?: number;
+    lineEnd?: number;
+    /** What was verified and the evidence that it complies with the rule. */
+    message: string;
+    category?: string;
+}
+export interface ReviewSummary {
+    ruleId: string;
+    /** Agent's explicit declaration. "pass" = no findings reported for this rule;
+     *  "concerns" = one or more findings reported. The runner cross-checks this
+     *  against the recorded finding count. */
+    outcome: "pass" | "concerns";
+    /** Concrete description of what the agent examined (files, behaviours, areas). */
+    checked: string;
+    /** Why the outcome holds, tied to what was checked. */
+    rationale: string;
 }
 export interface RunReport {
     /** Bumped to 2 when prior-run-aware features (resolutions, fingerprint, commentId) were added.
-     *  Readers SHOULD accept v1 reports and treat missing fields as defaults. */
-    schemaVersion: 2;
+     *  Bumped to 3 when the review-summary protocol arrived: required `summaryCount`/`checkCount`
+     *  on each RuleResult, and required `summaries: ReviewSummary[]` / `checks: Check[]` arrays
+     *  on the report.
+     *  Readers SHOULD accept v1, v2, and v3 reports and treat missing fields as defaults. */
+    schemaVersion: 3;
     runId: string;
     startedAt: string;
     completedAt: string;
@@ -77,6 +125,12 @@ export interface RunReport {
     findings: Finding[];
     /** Resolutions emitted by reviewers this run, OR carried forward from `--prior-report`. */
     resolutions: Resolution[];
+    /** Per-rule review summaries — one per rule that called `report_review_summary`.
+     *  A rule with no entry here is flagged as a possibly-incomplete review. */
+    summaries: ReviewSummary[];
+    /** All compliance checks the agents emitted via `report_check`. Granular
+     *  positive evidence; not findings, not subject to resolution. */
+    checks: Check[];
 }
 export interface RevuConfig {
     pattern: string;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "revu-ai",
-  "version": "0.2.1",
+  "version": "0.3.0",
   "description": "Parallel AI code review — pluggable agent harnesses (Claude Code, opencode); pick any provider/model",
   "type": "module",
   "bin": {
@@ -63,10 +63,12 @@
     "commander": "^12.1.0",
     "fast-glob": "^3.3.2",
     "ignore": "^5.3.2",
+    "micromatch": "^4.0.8",
     "undici": "^7.25.0",
     "zod": "^3.23.8"
   },
   "devDependencies": {
+    "@types/micromatch": "^4.0.10",
     "@types/node": "^22.0.0",
     "tsx": "^4.19.0",
     "typescript": "^5.6.0",