@bytesbrains/pi-review-gate 1.1.2 → 1.1.3

package/README.md

@@ -1,7 +1,7 @@
 # Review Gate for Pi
 
-[![npm version](https://img.shields.io/npm/v/pi-review-gate)](https://www.npmjs.com/package/pi-review-gate)
-[![license](https://img.shields.io/npm/l/pi-review-gate)](./LICENSE)
+[![npm version](https://img.shields.io/npm/v/@bytesbrains/pi-review-gate)](https://www.npmjs.com/package/@bytesbrains/pi-review-gate)
+[![license](https://img.shields.io/npm/l/@bytesbrains/pi-review-gate)](./LICENSE)
 
 > CI-level merge guardian for AI agents — required reviewers, breaking change detection, stale PR cleanup, and review automation. **Agents don't `git merge` — CI merges only after all gates pass.**
 
@@ -15,7 +15,7 @@ Core enforcement runs in Gitea Actions — agents **cannot bypass** these gates.
 ## Install
 
 ```bash
-pi install npm:pi-review-gate
+pi install npm:@bytesbrains/pi-review-gate
 ```
 
 ## Tools
@@ -76,7 +76,7 @@ breakingChangePatterns: export interface,export type,export function,export clas
 ## Workflow
 
 ```
-contrib_submit() from pi-contrib-gate
+contrib_submit() from @bytesbrains/pi-contrib-gate
   │
   ▼
 PR opened on Gitea
@@ -120,12 +120,12 @@ Every CI run diffs against the base branch and scans for:
 
 If detected, CI adds a `⚠️ breaking-change` label and requires human review.
 
-## Integration with pi-contrib-gate
+## Integration with @bytesbrains/pi-contrib-gate
 
 The two gates work together:
 
 ```
-pi-contrib-gate          pi-review-gate
+pi-contrib-gate          @bytesbrains/pi-review-gate
      │                        │
      │  contrib_start_work    │
      │  contrib_propose       │
@@ -141,8 +141,8 @@ pi-contrib-gate          pi-review-gate
 Install both for full agent governance:
 
 ```bash
-pi install npm:pi-contrib-gate
-pi install npm:pi-review-gate
+pi install npm:@bytesbrains/pi-contrib-gate
+pi install npm:@bytesbrains/pi-review-gate
 ```
 
 ## License

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bytesbrains/pi-review-gate",
-  "version": "1.1.2",
+  "version": "1.1.3",
   "description": "CI-level merge guardian for AI agents \u2014 required reviewers, breaking change detection, stale PR cleanup, and review automation.",
   "keywords": [
     "pi-package",

package/src/config.ts

@@ -37,10 +37,12 @@ export function loadConfig(cwd: string): ReviewConfig {
         reviewers[key.replace("requiredReviewers.", "")] = (val as string).split(",").map(s => s.trim()).filter(Boolean);
       }
     }
+    const minDiverseReviews = parseInt(result["minDiverseReviews"] as string);
+    const staleDays = parseInt(result["staleDays"] as string);
     return {
-      minDiverseReviews: parseInt(result["minDiverseReviews"] as string) || DEFAULT_CONFIG.minDiverseReviews,
+      minDiverseReviews: isNaN(minDiverseReviews) ? DEFAULT_CONFIG.minDiverseReviews : minDiverseReviews,
       requiredReviewers: reviewers,
-      staleDays: parseInt(result["staleDays"] as string) || DEFAULT_CONFIG.staleDays,
+      staleDays: isNaN(staleDays) ? DEFAULT_CONFIG.staleDays : staleDays,
       protectedPaths: (result["protectedPaths"] as string)?.split(",").map(s => s.trim()).filter(Boolean) || DEFAULT_CONFIG.protectedPaths,
       breakingChangePatterns: (result["breakingChangePatterns"] as string)?.split(",").map(s => s.trim()).filter(Boolean) || DEFAULT_CONFIG.breakingChangePatterns,
     };

package/src/helpers.ts

@@ -1,5 +1,7 @@
 import * as cp from "node:child_process";
 
+// ⚠️ SYNC-MARKER: exec(), resolveGitea(), giteaApi() are duplicated across packages.
+// If changing behavior here, update all copies in: ci-gate, contrib-gate, review-gate, project-gate
 export function exec(cmd: string, cwd?: string): { ok: boolean; stdout: string; stderr: string } {
   try {
     const r = cp.execSync(cmd, { cwd, encoding: "utf-8", timeout: 30000 });
@@ -23,7 +25,7 @@ export function resolveGitea(cwd: string): { repo: string; token: string } {
   return { repo, token };
 }
 
-export async function giteaApi(path: string, method: string, body: Record<string, unknown> | null, opts: { repo: string; token?: string }, _cwd: string): Promise<{ ok: boolean; data: unknown; error?: string }> {
+export async function giteaApi(path: string, method: string, body: Record<string, unknown> | null, opts: { repo: string; token?: string }, _cwd: string): Promise<{ ok: boolean; data: unknown; error?: string; statusCode?: number }> {
   const base = `http://127.0.0.1:3001/api/v1/repos/${opts.repo}`;
   const url = `${base}${path}`;
   const headers: Record<string, string> = { "Content-Type": "application/json", "Accept": "application/json" };
@@ -36,8 +38,9 @@ export async function giteaApi(path: string, method: string, body: Record<string
       body: body ? JSON.stringify(body) : undefined,
     });
     const text = await res.text();
-    if (!res.ok) return { ok: false, data: null, error: text || `HTTP ${res.status}` };
-    try { return { ok: true, data: JSON.parse(text) }; } catch { return { ok: true, data: text }; }
+    const statusCode = res.status;
+    if (!res.ok) return { ok: false, data: null, statusCode, error: `Gitea API error: HTTP ${statusCode} ${method} ${path}` };
+    try { return { ok: true, data: JSON.parse(text), statusCode }; } catch { return { ok: true, data: text, statusCode }; }
   } catch (e: any) {
     return { ok: false, data: null, error: e.message || "Network error" };
   }