@fullstackhouse/cerber 0.5.0 → 0.6.0

package/dist/server/index.js

@@ -8,9 +8,12 @@ import { fileURLToPath } from "node:url";
 import { randomUUID } from "node:crypto";
 import { ArtifactStatusSchema, VerdictSchema, artifactKey } from "../core/artifact.js";
 import { toMarkdown } from "../core/export.js";
-import { submitReview } from "../core/gh.js";
+import { fetchPrDiff, fetchPrInfo, submitReview } from "../core/gh.js";
+import { refreshArtifact } from "../core/refresh.js";
 import { buildReviewPayload, computeCalibration } from "../core/send.js";
-import { listArtifacts, loadArtifactByKey, updateArtifactByKey } from "../core/state.js";
+import { listArtifacts, loadArtifactByKey, reconcileRunning, updateArtifactByKey, } from "../core/state.js";
+import { isReviewRunning } from "../runner/inflight.js";
+import { reviewPr } from "../runner/review.js";
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 export async function buildApp(opts) {
     const app = new Hono();
@@ -117,6 +120,8 @@ export async function buildApp(opts) {
                     origin: "user",
                     status: "draft",
                     editedByUser: false,
+                    originalLine: null,
+                    drifted: false,
                 },
             ],
         }));
@@ -134,6 +139,76 @@ export async function buildApp(opts) {
             return c.json({ error: "not found" }, 404);
         return c.json(updated);
     });
+    // ---- Freshness: pull a review forward onto the PR's current head ----
+    // Read-only against GitHub (pr view + pr diff); writes only the local artifact.
+    app.post("/api/reviews/:key/refresh", async (c) => {
+        const artifact = await loadArtifactByKey(c.req.param("key"));
+        if (!artifact)
+            return c.json({ error: "not found" }, 404);
+        const ref = { owner: artifact.pr.owner, repo: artifact.pr.repo, number: artifact.pr.number };
+        let pr;
+        try {
+            pr = await fetchPrInfo(ref);
+        }
+        catch (err) {
+            return c.json({ error: err instanceof Error ? err.message : String(err) }, 502);
+        }
+        const stale = artifact.pr.headSha !== "" && artifact.pr.headSha !== pr.headSha;
+        // A sent review is a record of what was posted — never rewrite it. A running
+        // one is mid-run and would be overwritten from under the runner.
+        if (!stale || artifact.sent || artifact.status === "running") {
+            return c.json({ stale, changed: false, prState: pr.state, artifact });
+        }
+        try {
+            const diff = await fetchPrDiff(ref);
+            const result = refreshArtifact(artifact, pr, diff);
+            const saved = await updateArtifactByKey(c.req.param("key"), () => result.artifact);
+            return c.json({
+                stale: true,
+                changed: result.changed,
+                prState: pr.state,
+                moved: result.moved,
+                drifted: result.drifted,
+                artifact: saved,
+            });
+        }
+        catch (err) {
+            return c.json({ error: err instanceof Error ? err.message : String(err) }, 502);
+        }
+    });
+    // ---- Re-review: start an AI run for a PR that has moved on ----
+    app.post("/api/reviews/:key/rerun", async (c) => {
+        const key = c.req.param("key");
+        const artifact = await loadArtifactByKey(key);
+        if (!artifact)
+            return c.json({ error: "not found" }, 404);
+        if (artifact.sent)
+            return c.json({ error: `already sent at ${artifact.sent.at}` }, 409);
+        if (isReviewRunning(artifact.id)) {
+            return c.json({ error: "a review of this PR is already running" }, 409);
+        }
+        const ref = { owner: artifact.pr.owner, repo: artifact.pr.repo, number: artifact.pr.number };
+        // Mark it running before responding, so the cockpit's next poll can't catch
+        // the old "ready" status and conclude the run already finished.
+        const running = await updateArtifactByKey(key, (a) => ({
+            ...a,
+            status: "running",
+            run: { model: null, startedAt: new Date().toISOString(), finishedAt: null, costUsd: null, error: null },
+        }));
+        // Minutes-long: run it detached and let the cockpit poll the artifact.
+        void reviewPr(ref, { force: true, onProgress: (m) => console.log(`[rerun ${artifact.id}] ${m}`) })
+            .catch(async (err) => {
+            const message = err instanceof Error ? err.message : String(err);
+            console.error(`[rerun ${artifact.id}] failed: ${message}`);
+            // The runner marks the artifact failed once it owns it; a failure before
+            // that (fetching the PR, or a run already in flight) would otherwise
+            // leave it stuck at "running" with nothing to clear it.
+            await updateArtifactByKey(key, (a) => a.status === "running"
+                ? { ...a, status: "failed", run: a.run ? { ...a.run, error: message } : null }
+                : a).catch(() => { });
+        });
+        return c.json(running, 202);
+    });
     // ---- Export (local file download) ----
     app.get("/api/reviews/:key/export", async (c) => {
         const artifact = await loadArtifactByKey(c.req.param("key"));
@@ -160,7 +235,7 @@ export async function buildApp(opts) {
         }
         const payload = buildReviewPayload(artifact, event);
         try {
-            const { url } = await submitReview({ owner: artifact.pr.owner, repo: artifact.pr.repo, number: artifact.pr.number }, { event: payload.event, body: payload.body, comments: payload.comments });
+            const { url } = await submitReview({ owner: artifact.pr.owner, repo: artifact.pr.repo, number: artifact.pr.number }, { event: payload.event, body: payload.body, comments: payload.comments, commitId: payload.commitId });
             const updated = await updateArtifactByKey(c.req.param("key"), (a) => ({
                 ...a,
                 status: "sent",
@@ -198,6 +273,10 @@ export async function buildApp(opts) {
     return app;
 }
 export async function startServer(opts) {
+    const cleared = await reconcileRunning();
+    if (cleared > 0) {
+        console.log(`Cleared ${cleared} review(s) left mid-run by a previous process — re-review to retry.`);
+    }
     const app = await buildApp(opts);
     serve({ fetch: app.fetch, port: opts.port, hostname: opts.host }, (info) => {
         const tokenHint = opts.token ? `/?token=${opts.token}` : "";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fullstackhouse/cerber",
-  "version": "0.5.0",
+  "version": "0.6.0",
   "description": "AI code-review cockpit: Claude reviews your PRs into local artifacts; nothing reaches GitHub until you say so.",
   "license": "MIT",
   "repository": "github:fullstackhouse/cerber",