@mrclrchtr/supi-review 1.8.0 → 1.9.0

package/README.md

@@ -1,3 +1,5 @@
+![SuPi](assets/logo.png)
+
 # @mrclrchtr/supi-review
 
 Adds an interactive `/supi-review` command to the [pi coding agent](https://github.com/earendil-works/pi) for session-aware code review.
@@ -16,8 +18,6 @@ For local development:
 pi install ./packages/supi-review
 ```
 
-After editing the source, run `/reload`.
-
 ## What you get
 
 After install, pi gets one command:
@@ -29,6 +29,14 @@ The reviewer runs in managed child agent sessions:
 - a **brief synthesizer** creates a structured review brief from the active session branch
 - a **read-only reviewer** inspects the selected snapshot (without receiving bulk inline diffs) and submits structured findings
 
+![Review target selection](https://raw.githubusercontent.com/mrclrchtr/supi/main/screenshots/supi-review-1.png)
+
+![Review brief preview](https://raw.githubusercontent.com/mrclrchtr/supi/main/screenshots/supi-review-2.png)
+
+![Review result](https://raw.githubusercontent.com/mrclrchtr/supi/main/screenshots/supi-review-3.png)
+
+![Review progress](https://raw.githubusercontent.com/mrclrchtr/supi/main/screenshots/supi-review-4.png)
+
 ## Review flow
 
 `/supi-review` walks you through:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrclrchtr/supi-review",
-  "version": "1.8.0",
+  "version": "1.9.0",
   "description": "SuPi Review extension — structured code review via /supi-review command",
   "license": "MIT",
   "repository": {
@@ -42,7 +42,8 @@
   "pi": {
     "extensions": [
       "./src/extension.ts"
-    ]
+    ],
+    "image": "https://raw.githubusercontent.com/mrclrchtr/supi/main/packages/supi-review/assets/logo.png"
   },
   "main": "src/api.ts",
   "exports": {

package/src/review.ts

@@ -65,6 +65,8 @@ async function handleInteractive(ctx: CommandContext, pi: ExtensionAPI): Promise
       }),
   });
 
+  notifyBriefDone(pi, synthesis, snapshot, model.canonicalId);
+
   if (synthesis.kind !== "success") {
     notifySynthesisFailure(synthesis, ctx);
     return;
@@ -82,6 +84,8 @@ async function handleInteractive(ctx: CommandContext, pi: ExtensionAPI): Promise
   if (!approved) return;
 
   const result = await runReviewWithLoader(plan, ctx, pi);
+
+  notifyReviewDone(pi, result);
   injectReviewMessage(pi, result);
 }
 
@@ -223,6 +227,44 @@ function notifySynthesisFailure(
   }
 }
 
+/** Ring the terminal bell so the user knows to check back. */
+function ringBell(): void {
+  process.stdout.write("\x07");
+}
+
+/**
+ * Emit a `supi:review:brief-done` event and ring the terminal bell after
+ * brief synthesis completes (success, failure, cancel, or timeout).
+ */
+function notifyBriefDone(
+  pi: ExtensionAPI,
+  result: BriefSynthesisRunResult,
+  snapshot: ReviewSnapshot,
+  modelId: string,
+): void {
+  pi.events.emit("supi:review:brief-done", {
+    kind: result.kind,
+    snapshot: snapshot.title,
+    modelId,
+    brief: result.kind === "success" ? result.brief : undefined,
+  });
+  ringBell();
+}
+
+/**
+ * Emit a `supi:review:review-done` event and ring the terminal bell after
+ * the review child session completes.
+ */
+function notifyReviewDone(pi: ExtensionAPI, result: ReviewResult): void {
+  pi.events.emit("supi:review:review-done", {
+    kind: result.kind,
+    snapshot: result.snapshot.title,
+    modelId: result.modelId,
+    findingsCount: result.kind === "success" ? result.output.findings.length : 0,
+  });
+  ringBell();
+}
+
 function injectReviewMessage(pi: ExtensionAPI, result: ReviewResult): void {
   pi.sendMessage({
     customType: "supi-review",