opencode-reviewer 0.5.1 → 0.6.1

package/README.md

@@ -92,6 +92,12 @@ The rating remains provisional until the full response is validated. A format re
 clears the preview and restores the loading indicator. Auto-approval
 starts only after the full response is validated and rendered.
 
+Transient connection failures and HTTP 408/429/500/502/503/504 responses get up to
+two internal retries within `timeoutMs`, honoring server cooldowns. A stream that
+has already delivered assessment text is not restarted. This is separate from
+`formatRetries`; no additional setting is needed. Retried requests can incur
+additional provider charges, and unreported usage remains unknown.
+
 During an auto-approval countdown, click the countdown to allow once immediately,
 or Cancel to leave the request manual. Hiding or covering the panel also cancels
 that request's countdown. Cancel before using native Allow always or rejection
@@ -110,6 +116,12 @@ costs use reported charges; other endpoints use available catalog estimates.
 Received usage counts even if a review fails or is interrupted. Unreported charges
 are missing from the totals.
 
+Lifetime statistics also show **Safe** and **Unsafe** review counts. Each completed,
+validated review counts once, even without usage data. Previews, retries and failed
+or interrupted analyses do not count. Re-enabling a conversation can produce a new
+review, which counts again. The totals persist across restarts; earlier ratings
+were not recorded and cannot be reconstructed.
+
 Reviews send the pending request, latest user prompt, project context, and relevant
 file snapshots, diffs, or tool arguments to your endpoint before approval. Shell
 file snapshots can follow symlinks outside the project. Missing evidence is noted
@@ -117,21 +129,19 @@ in the report; ratings are advice based on the supplied evidence.
 
 ### Desktop notifications
 
-Transient banners have an **Opencode** heading and a small status icon: green
-checkmark for approvals, orange exclamation mark for attention/countdowns, red X
+Transient banners have an **Opencode (Session name)** heading and a small status icon: green
+checkmark for approvals, orange exclamation mark for attention, red X
 for errors, and a neutral code mark for completed responses. GNOME controls the
-heading's font weight. The event
-message and conversation title appear beneath it, even while the terminal is focused.
-An explicit separator keeps them distinct on desktops that flatten banner text:
+heading's font weight. The event message appears beneath it, even while the
+terminal is focused:
 
 - **Session needs attention:** questions and manual permissions. Reviewed requests
   wait for a final validated assessment; failures, canceled automation, and a Safe
   review blocked from starting its countdown also notify. Unreviewed requests
   notify immediately, including when conversation review is disabled.
-- **Reviewer will approve permission in X s:** one announcement when a positive
-  countdown starts, using the attention sound. Confirmed automatic approval then
-  sends **Reviewer approved a permission** with the approval sound. Zero delay
-  sends only the confirmed approval notification. Approval sounds are limited to
+- **Reviewer approved a permission:** sent with the approval sound after confirmed
+  automatic approval, for both positive and zero delays. The countdown is silent.
+  Approval sounds are limited to
   one every two seconds; every eligible banner is retained for delivery.
 - **Session error:** an unrecovered session/provider failure, not a review failure.
 - **Session ended:** a completed root-agent response, not a question/permission

package/dist/tui.js

@@ -1181,6 +1181,9 @@ var uiText = {
     tokenCoverage: (available, total) => `Token counts available: ${available}/${total} requests`,
     pricingCoverage: (available, total) => `Pricing available: ${available}/${total} requests`,
     since: (date) => `Recorded since: ${date}`,
+    ratings: (safe, unsafe) => `${safe} \u2713 ${unsafe} \u2717`,
+    ratingsSince: (date) => `Ratings recorded since: ${date}`,
+    ratingsExplanation: "Ratings count completed, validated reviews, including those without usage data. Previews, retries and failed or interrupted analyses do not count. A fresh review after re-enabling counts again. Earlier ratings were not recorded and cannot be recovered.",
     explanation: "Received usage only, including interrupted or unsuccessful reviews. Unreported charges remain unknown. Costs combine endpoint-reported amounts and catalog estimates; legacy history retains its original estimates. Earlier unrecorded usage cannot be recovered."
   },
   notifications: {
@@ -1189,7 +1192,6 @@ var uiText = {
     openSession: "Open session",
     fallbackSession: "OpenCode session",
     attention: "Session needs attention",
-    countdown: (seconds) => `Reviewer will approve permission in ${seconds} s`,
     approved: "Reviewer approved a permission",
     error: "Session error",
     ended: "Session ended",
@@ -1603,6 +1605,74 @@ function measured(trace, phase, run) {
   return trace ? trace.measure(phase, run) : run();
 }
 
+// src/transport-retry.ts
+init_define_REVIEW_PROMPTS();
+init_define_REVIEW_SOUNDS();
+import { setTimeout as sleep } from "node:timers/promises";
+var statuses = /* @__PURE__ */ new Set([408, 429, 500, 502, 503, 504]);
+var networkCodes = /* @__PURE__ */ new Set([
+  "EAI_AGAIN",
+  "ECONNRESET",
+  "ECONNREFUSED",
+  "ECONNABORTED",
+  "EPIPE",
+  "ETIMEDOUT",
+  "ENETUNREACH",
+  "EHOSTUNREACH",
+  "UND_ERR_CONNECT_TIMEOUT",
+  "UND_ERR_HEADERS_TIMEOUT",
+  "UND_ERR_BODY_TIMEOUT",
+  "UND_ERR_SOCKET"
+]);
+var TransientTransportError = class extends Error {
+  constructor(message, notBefore = 0) {
+    super(message);
+    this.notBefore = notBefore;
+  }
+  notBefore;
+};
+function networkFailure(error, message) {
+  for (let depth = 0; depth < 4 && error && typeof error === "object"; depth++) {
+    const value = error;
+    if (typeof value.code === "string") {
+      return networkCodes.has(value.code) ? new TransientTransportError(message) : new Error(message);
+    }
+    error = value.cause;
+  }
+  return new Error(message);
+}
+function retryAfter(value, now = Date.now()) {
+  if (value === null) return 0;
+  value = value.trim();
+  if (/^\d+$/.test(value)) return Number(value) * 1e3;
+  if (!/^(?:[A-Za-z]{3}, \d{2} [A-Za-z]{3} \d{4} \d{2}:\d{2}:\d{2} GMT|[A-Za-z]+, \d{2}-[A-Za-z]{3}-\d{2} \d{2}:\d{2}:\d{2} GMT|[A-Za-z]{3} [A-Za-z]{3} [ \d]\d \d{2}:\d{2}:\d{2} \d{4})$/.test(value)) return 0;
+  const date = Date.parse(value.endsWith("GMT") ? value : `${value} GMT`);
+  return Number.isFinite(date) ? Math.max(0, date - now) : 0;
+}
+function httpFailure(response) {
+  const message = `Reviewer HTTP ${response.status}`;
+  return statuses.has(response.status) ? new TransientTransportError(message, performance.now() + retryAfter(response.headers.get("retry-after"))) : new Error(message);
+}
+var TransportRetries = class {
+  used = 0;
+  async wait(error, signal) {
+    reviewStage(signal, "Reviewer response");
+    if (!(error instanceof TransientTransportError) || this.used >= 2) return false;
+    const base = 250 * 2 ** this.used;
+    const delay = Math.max(base + Math.random() * base, error.notBefore - performance.now());
+    if (delay + 250 >= remainingTime(signal)) return false;
+    this.used++;
+    try {
+      await sleep(Math.ceil(delay), void 0, { signal });
+    } catch {
+      signal.throwIfAborted();
+      throw error;
+    }
+    reviewStage(signal, "Reviewer response");
+    return remainingTime(signal) > 250;
+  }
+};
+
 // src/reviewer.ts
 function abortable(promise, signal, late) {
   return new Promise((resolve, reject) => {
@@ -1647,9 +1717,9 @@ async function readBody(response, signal, consume) {
       let part;
       try {
         part = await abortable(reader2.read(), signal);
-      } catch {
+      } catch (error) {
         signal.throwIfAborted();
-        throw new Error("Reviewer response read failed");
+        throw networkFailure(error, "Reviewer response read failed");
       }
       signal.throwIfAborted();
       if (part.done) break;
@@ -1748,6 +1818,15 @@ async function streamedAssessment(response, signal, accounting, progress, firstC
   return assessment;
 }
 async function review(evidence, config, signal, fetcher = fetch, environment = process.env, prompts = BUILTIN_PROMPTS, pricing, onUsage, onProgress, onDiagnostics) {
+  if (remainingTime(signal) === Infinity) {
+    let worker;
+    try {
+      return await withDeadline(signal, config.timeoutMs, (bounded) => worker = review(evidence, config, bounded, fetcher, environment, prompts, pricing, onUsage, onProgress, onDiagnostics));
+    } finally {
+      await worker?.catch(() => {
+      });
+    }
+  }
   const key = config.apiKey ?? (config.apiKeyEnv ? environment[config.apiKeyEnv]?.trim() : void 0);
   if (config.apiKeyEnv && !key) throw new Error(`API key environment variable ${config.apiKeyEnv} is unset or empty`);
   const prompt2 = prompts[evidence.kind];
@@ -1756,7 +1835,9 @@ async function review(evidence, config, signal, fetcher = fetch, environment = p
     { role: "user", content: JSON.stringify(evidence) }
   ];
   let usage;
-  for (let attempt = 0; attempt <= config.formatRetries; attempt++) {
+  let corrections = 0;
+  const retries = new TransportRetries();
+  for (let attempt = 0; ; attempt++) {
     signal.throwIfAborted();
     const progress = (phase, preview) => {
       if (signal.aborted) return;
@@ -1768,6 +1849,10 @@ async function review(evidence, config, signal, fetcher = fetch, environment = p
     };
     progress(attempt ? "retrying" : "evaluating");
     const accounting = usageAttempt(config.baseURL, config.model, pricing, onUsage);
+    const requestAbort = new AbortController();
+    let contentStarted = false;
+    let failed = false;
+    let failure;
     try {
       reviewStage(signal, "Reviewer response");
       let response;
@@ -1776,31 +1861,36 @@ async function review(evidence, config, signal, fetcher = fetch, environment = p
         const body = JSON.stringify({
           model: config.model,
           messages,
+          max_tokens: 2e3,
           stream: config.stream,
           ...config.stream ? { stream_options: { include_usage: true } } : {}
         });
+        reviewStage(signal, "Reviewer response");
         diagnose = onDiagnostics ? diagnosticAttempt(onDiagnostics, attempt) : void 0;
         response = await abortable(fetcher(`${config.baseURL}/chat/completions`, {
           method: "POST",
           redirect: "error",
-          signal,
+          signal: AbortSignal.any([signal, requestAbort.signal]),
           headers: { "Content-Type": "application/json", ...key ? { Authorization: `Bearer ${key}` } : {} },
           body
         }), signal, cancelBody);
         diagnose?.("headers");
-      } catch {
+      } catch (error) {
         signal.throwIfAborted();
-        throw new Error("Reviewer network request failed");
+        throw networkFailure(error, "Reviewer network request failed");
       }
       if (!response.ok) {
         cancelBody(response);
-        throw new Error(`Reviewer HTTP ${response.status}`);
+        throw httpFailure(response);
       }
       let assessment;
       if (config.stream) assessment = await streamedAssessment(response, signal, accounting, (preview) => {
         if (typeof preview?.safe === "boolean") diagnose?.("first-rating");
         progress("streaming", preview);
-      }, diagnose ? () => diagnose("first-content") : void 0);
+      }, () => {
+        contentStarted = true;
+        diagnose?.("first-content");
+      });
       else {
         let envelope;
         try {
@@ -1818,14 +1908,17 @@ async function review(evidence, config, signal, fetcher = fetch, environment = p
         if (!object(choice) || !object(message) || typeof message.content !== "string" || !textMetadata(message) || object(envelope) && envelope.error != null || choice.error != null || choice.index !== void 0 && choice.index !== 0 || choice.finish_reason !== void 0 && choice.finish_reason !== "stop") {
           throw new Error("Reviewer API did not return a text assessment");
         }
-        if (message.content) diagnose?.("first-content");
+        if (message.content) {
+          contentStarted = true;
+          diagnose?.("first-content");
+        }
         assessment = new StreamingAssessment();
         assessment.push(message.content);
         if (diagnose && typeof assessment.preview()?.safe === "boolean") diagnose("first-rating");
       }
       reviewStage(signal, "Assessment validation");
       const currentUsage = accounting.current();
-      usage = attempt === 0 ? currentUsage : sumUsage(usage, currentUsage);
+      const reportUsage = attempt === 0 ? currentUsage : sumUsage(usage, currentUsage);
       try {
         let result;
         try {
@@ -1835,20 +1928,26 @@ async function review(evidence, config, signal, fetcher = fetch, environment = p
         }
         if (config.stream) progress("streaming", result);
         signal.throwIfAborted();
-        return { ...result, ...usage ? { usage } : {} };
+        return { ...result, ...reportUsage ? { usage: reportUsage } : {} };
       } catch (error) {
         if (!(error instanceof AssessmentFormatError)) throw error;
-        if (attempt === config.formatRetries) throw new Error("Reviewer response format invalid after configured attempts");
+        if (corrections++ === config.formatRetries) throw new Error("Reviewer response format invalid after configured attempts");
         messages.push(
           { role: "assistant", content: assessment.content },
           { role: "user", content: correctionPrompt(error.message) }
         );
       }
+    } catch (error) {
+      requestAbort.abort();
+      if (contentStarted) throw error;
+      failed = true;
+      failure = error;
     } finally {
-      accounting.finalize();
+      const currentUsage = accounting.finalize();
+      usage = attempt === 0 ? currentUsage : sumUsage(usage, currentUsage);
     }
+    if (failed && !await retries.wait(failure, signal)) throw failure;
   }
-  throw new Error("Reviewer exhausted attempts");
 }
 
 // src/classification.ts
@@ -1980,7 +2079,7 @@ var clock = {
 };
 var permissionResolved = Object.freeze({ reason: "permission resolved" });
 var Controller = class {
-  constructor(evaluate, changed, options = { reviewBash: true, reviewEdits: true }, approval, time = clock, modes, approvalObserver) {
+  constructor(evaluate, changed, options = { reviewBash: true, reviewEdits: true }, approval, time = clock, modes, approvalObserver, ratingObserver) {
     this.evaluate = evaluate;
     this.changed = changed;
     this.options = options;
@@ -1988,6 +2087,7 @@ var Controller = class {
     this.time = time;
     this.modes = modes;
     this.approvalObserver = approvalObserver;
+    this.ratingObserver = ratingObserver;
   }
   evaluate;
   changed;
@@ -1996,6 +2096,7 @@ var Controller = class {
   time;
   modes;
   approvalObserver;
+  ratingObserver;
   entries = /* @__PURE__ */ new Map();
   stopped = false;
   version = 0;
@@ -2288,6 +2389,13 @@ var Controller = class {
       this.clearProgress(entry);
       if (active()) {
         entry.view = assessment ? { ...entry.view, status: "complete", assessment, progress: void 0 } : { ...entry.view, status: "unrelated", progress: void 0 };
+        if (assessment) {
+          try {
+            void Promise.resolve(this.ratingObserver?.(assessment.safe)).catch(() => {
+            });
+          } catch {
+          }
+        }
         this.publish();
       }
     }, (error) => {
@@ -3538,12 +3646,25 @@ import { constants as constants3 } from "node:fs";
 import { mkdir, open as open3, readdir as readdir2, rename, unlink } from "node:fs/promises";
 import { randomUUID } from "node:crypto";
 import path8 from "node:path";
-var empty = () => ({ requests: 0, tokenRequests: 0, input: 0, output: 0, priced: 0, cost: 0, since: null });
+var empty = () => ({
+  requests: 0,
+  tokenRequests: 0,
+  input: 0,
+  output: 0,
+  priced: 0,
+  cost: 0,
+  since: null,
+  safe: 0,
+  unsafe: 0,
+  ratingsSince: null
+});
 var count2 = (value) => typeof value === "number" && Number.isSafeInteger(value) && value >= 0;
 var amount = (value) => typeof value === "number" && Number.isFinite(value) && value >= 0;
 var snapshotName = /^[0-9a-f]{8}(?:-[0-9a-f]{4}){3}-[0-9a-f]{12}\.json$/;
 function validate(value) {
-  if (!value || ![value.requests, value.tokenRequests, value.input, value.output, value.priced].every(count2) || !amount(value.cost) || value.priced > value.requests || value.tokenRequests > value.requests || value.requests > value.tokenRequests + value.priced || !value.tokenRequests && (value.input !== 0 || value.output !== 0) || (value.requests > 0 ? !count2(value.since) || value.since > 864e13 : value.since !== null || value.input !== 0 || value.output !== 0) || !value.priced && value.cost !== 0) throw new Error("Invalid lifetime usage totals");
+  if (!value || ![value.requests, value.tokenRequests, value.input, value.output, value.priced, value.safe, value.unsafe, value.safe + value.unsafe].every(count2) || !amount(value.cost) || value.priced > value.requests || value.tokenRequests > value.requests || value.requests > value.tokenRequests + value.priced || !value.tokenRequests && (value.input !== 0 || value.output !== 0) || (value.requests > 0 ? !count2(value.since) || value.since > 864e13 : value.since !== null || value.input !== 0 || value.output !== 0) || !value.priced && value.cost !== 0 || (value.safe + value.unsafe > 0 ? !count2(value.ratingsSince) || value.ratingsSince > 864e13 : value.ratingsSince !== null)) {
+    throw new Error("Invalid lifetime usage totals");
+  }
   return value;
 }
 function add(left, right) {
@@ -3554,28 +3675,32 @@ function add(left, right) {
     tokenRequests: left.tokenRequests + right.tokenRequests,
     priced: left.priced + right.priced,
     cost: left.cost + right.cost,
-    since: left.since === null ? right.since : right.since === null ? left.since : Math.min(left.since, right.since)
+    since: left.since === null ? right.since : right.since === null ? left.since : Math.min(left.since, right.since),
+    safe: left.safe + right.safe,
+    unsafe: left.unsafe + right.unsafe,
+    ratingsSince: left.ratingsSince === null ? right.ratingsSince : right.ratingsSince === null ? left.ratingsSince : Math.min(left.ratingsSince, right.ratingsSince)
   });
 }
 var LifetimeUsage = class {
-  constructor(directory, legacyDirectory) {
+  constructor(directory, ...legacyDirectories) {
     this.directory = directory;
-    this.legacyDirectory = legacyDirectory;
-    if (!path8.isAbsolute(directory) || legacyDirectory !== void 0 && !path8.isAbsolute(legacyDirectory)) {
+    if (![directory, ...legacyDirectories].every((value) => path8.isAbsolute(value))) {
       throw new Error("Lifetime usage directory must be absolute");
     }
+    this.legacyDirectories = legacyDirectories;
   }
   directory;
-  legacyDirectory;
   id = randomUUID();
   local = empty();
   pending = Promise.resolve();
+  legacyDirectories;
   record(usage) {
     const tokens = count2(usage.input) && count2(usage.output);
     if (!tokens && (usage.input !== void 0 || usage.output !== void 0 || usage.cost === void 0) || usage.cost !== void 0 && !amount(usage.cost)) {
       return Promise.reject(new Error("Invalid request usage"));
     }
-    const increment = {
+    return this.increment({
+      ...empty(),
       requests: 1,
       tokenRequests: tokens ? 1 : 0,
       input: usage.input ?? 0,
@@ -3583,7 +3708,14 @@ var LifetimeUsage = class {
       priced: usage.cost === void 0 ? 0 : 1,
       cost: usage.cost ?? 0,
       since: Date.now()
-    };
+    });
+  }
+  /** One accepted final review, independent of POST count or received usage. */
+  recordRating(safe) {
+    if (typeof safe !== "boolean") return Promise.reject(new Error("Invalid review rating"));
+    return this.increment({ ...empty(), safe: safe ? 1 : 0, unsafe: safe ? 0 : 1, ratingsSince: Date.now() });
+  }
+  increment(increment) {
     this.pending = this.pending.catch(() => {
     }).then(async () => {
       this.local = add(this.local, increment);
@@ -3592,7 +3724,7 @@ var LifetimeUsage = class {
       try {
         const file = await open3(temporary, "wx", 384);
         try {
-          await file.writeFile(JSON.stringify({ version: 2, ...this.local }));
+          await file.writeFile(JSON.stringify({ version: 3, ...this.local }));
           await file.sync();
         } finally {
           await file.close();
@@ -3620,7 +3752,7 @@ var LifetimeUsage = class {
     await this.flush();
     signal?.throwIfAborted();
     let total = empty();
-    const directories = new Set([this.directory, ...this.legacyDirectory ? [this.legacyDirectory] : []].map((directory) => path8.resolve(directory)));
+    const directories = new Set([this.directory, ...this.legacyDirectories].map((directory) => path8.resolve(directory)));
     for (const directory of directories) total = add(total, await this.readDirectory(directory, signal));
     signal?.throwIfAborted();
     return total;
@@ -3652,7 +3784,7 @@ var LifetimeUsage = class {
         }
         if (size > 1024) throw new Error("Invalid lifetime usage snapshot");
         const value = JSON.parse(bytes.subarray(0, size).toString("utf8"));
-        if (value?.version !== 1 && value?.version !== 2) throw new Error("Unsupported lifetime usage snapshot");
+        if (value?.version !== 1 && value?.version !== 2 && value?.version !== 3) throw new Error("Unsupported lifetime usage snapshot");
         total = add(total, validate({
           requests: value.requests,
           input: value.input,
@@ -3660,7 +3792,10 @@ var LifetimeUsage = class {
           tokenRequests: value.version === 1 ? value.requests : value.tokenRequests,
           priced: value.priced,
           cost: value.cost,
-          since: value.since
+          since: value.since,
+          safe: value.version === 3 ? value.safe : 0,
+          unsafe: value.version === 3 ? value.unsafe : 0,
+          ratingsSince: value.version === 3 ? value.ratingsSince : null
         }));
       } finally {
         await file.close();
@@ -3679,13 +3814,17 @@ function lifetimeCost(totals) {
 function lifetimeReport(totals) {
   return [
     lifetimeCost(totals),
+    uiText.lifetime.ratings(totals.safe, totals.unsafe),
     uiText.lifetime.requests(totals.requests),
     totals.tokenRequests ? uiText.lifetime.tokens(totals.input, totals.output, totals.tokenRequests < totals.requests) : uiText.lifetime.tokensUnavailable,
     uiText.lifetime.tokenCoverage(totals.tokenRequests, totals.requests),
     uiText.lifetime.pricingCoverage(totals.priced, totals.requests),
     ...totals.since === null ? [] : [uiText.lifetime.since(new Date(totals.since).toISOString().slice(0, 10))],
+    ...totals.ratingsSince === null ? [] : [uiText.lifetime.ratingsSince(new Date(totals.ratingsSince).toISOString().slice(0, 10))],
+    "",
+    uiText.lifetime.explanation,
     "",
-    uiText.lifetime.explanation
+    uiText.lifetime.ratingsExplanation
   ].join("\n");
 }
 
@@ -3771,7 +3910,7 @@ var LifetimeRefresh = class {
 // src/lifetime-view.tsx
 function lifetimeTracker(api) {
   const directory = path9.join(api.state.path.state, "opencode-reviewer");
-  const store = new LifetimeUsage(path9.join(directory, "usage-v2"), path9.join(directory, "usage-v1"));
+  const store = new LifetimeUsage(path9.join(directory, "usage-v3"), path9.join(directory, "usage-v2"), path9.join(directory, "usage-v1"));
   const [totals, setTotals] = createSignal();
   const [unavailable, setUnavailable] = createSignal(false);
   const refresh = new LifetimeRefresh((signal) => store.totals(signal), (value) => {
@@ -3808,9 +3947,12 @@ function lifetimeTracker(api) {
     // The controller must settle aborted review workers before this final queue drain.
     flush: () => store.flush().catch(() => {
     }),
-    text: () => unavailable() ? uiText.lifetime.inlineUnavailable : totals()?.requests ? lifetimeCost(totals()) : void 0,
+    text: () => unavailable() ? uiText.lifetime.inlineUnavailable : totals()?.requests ? [lifetimeCost(totals()), uiText.lifetime.ratings(totals().safe, totals().unsafe)].join("\n") : void 0,
     record: (usage) => {
       void store.record(usage).then(() => refresh.refresh(), () => refresh.failed());
+    },
+    recordRating: (safe) => {
+      void store.recordRating(safe).then(() => refresh.refresh(), () => refresh.failed());
     }
   };
 }
@@ -4066,16 +4208,14 @@ function notificationMarkup(value) {
 
 // src/notification-policy.ts
 var NotificationPolicy = class {
-  constructor(config, auto, delay, backend, clock2 = notificationClock) {
+  constructor(config, auto, backend, clock2 = notificationClock) {
     this.config = config;
     this.auto = auto;
-    this.delay = delay;
     this.backend = backend;
     this.clock = clock2;
   }
   config;
   auto;
-  delay;
   backend;
   clock;
   permissions = /* @__PURE__ */ new Map();
@@ -4176,7 +4316,7 @@ var NotificationPolicy = class {
       if (entry.phase !== "automatic") {
         this.withdraw(entry.banner);
         entry.phase = "automatic";
-        entry.banner = state === "countdown" && this.delay > 0 ? this.show("attention", uiText.notifications.countdown(view.autoApproval.status === "countdown" ? view.autoApproval.seconds : this.delay), entry.target) : void 0;
+        entry.banner = void 0;
       }
       return;
     }
@@ -6188,7 +6328,7 @@ async function reviewTui(api, options, fileIO, observer, notificationBackend) {
     try {
       const click = (id) => notifications?.click(id);
       const backend = notificationBackend ? notificationBackend(click, notificationConfig) : new LinuxNotifications(notificationConfig, click);
-      notifications = new NotificationHost(api, new NotificationPolicy(notificationConfig, reviewOptions?.autoApprove === true, reviewOptions?.autoApproveDelaySeconds ?? 15, backend), (id, signal) => modes.root(id, signal));
+      notifications = new NotificationHost(api, new NotificationPolicy(notificationConfig, reviewOptions?.autoApprove === true, backend), (id, signal) => modes.root(id, signal));
     } catch {
     }
   }
@@ -6229,7 +6369,7 @@ async function reviewTui(api, options, fileIO, observer, notificationBackend) {
       once: (request, signal) => measured(traces?.get(request), "approval-reply", () => approval.once(request, signal))
     } : approval,
     visibleID: () => visibleApproval()
-  }, void 0, modes, (fact) => notifications?.fact(fact));
+  }, void 0, modes, (fact) => notifications?.fact(fact), lifetime.recordRating);
   api.event.on("permission.asked", (event) => controller.asked(event.properties));
   api.event.on("permission.replied", (event) => controller.replied(event.properties.requestID));
   api.event.on("session.deleted", (event) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-reviewer",
-  "version": "0.5.1",
+  "version": "0.6.1",
   "license": "MIT",
   "publishConfig": {
     "access": "public",