limn-review 0.1.173 → 0.1.174

package/dist/review/claude-review/manager.js

@@ -349,13 +349,19 @@ async function startAutoItem(a) {
         const detect = preflightFor(ctx);
         const nowMs = Date.now();
         const wantsPair = (await readAutoReviewProvidersForPr(ctx, accountId, prId)) === 'both';
-        const providers = automaticRunProviders({
+        // ⚠ A PAIR drops a locked half EVEN ON A PRESS while the other half can run: that half would
+        // fail at once (2026-11-03: six pressed Codex halves over the limit, nothing reviewed). Only when
+        // every provider is locked does a press run anyway (the reader's choice).
+        const pick = (lockedOk) => automaticRunProviders({
             defaultProvider: activeProviderId(),
             pair: wantsPair,
             order: PAIR_ORDER,
             ready: (p) => detect(p, 'review').status === 'ok',
-            locked: (p) => !a.pressed && isLockedOut(p, nowMs),
+            locked: (p) => !lockedOk && isLockedOut(p, nowMs),
         });
+        let providers = pick(a.pressed === true && !wantsPair);
+        if (providers.length === 0 && a.pressed)
+            providers = pick(true);
         if (providers.length === 0) {
             ctx.log.info(`auto claude review pr ${prId}: skipped — no provider outside its usage limit`);
             return;

package/dist/review/claude-review/team-claim.js

@@ -43,13 +43,17 @@ export function releasedLine(login, by) {
 // A FREE line: a success starting "Cleared by " (a Clear, a release of ours or of a teammate's expired
 // line), a success "free to claim" or "Released:" (a 2026-10-28 build's wording), or a pending "free to
 // claim" (older Limn versions).
+// A teammate's review KEPT IN LIMN put nothing on GitHub: the team got nothing from it, so it holds
+// nothing (2026-11-03 — robin-dunn's kept-in-Limn Claude reviews held six PRs, the pair fell back to a
+// Codex over its usage limit, and nobody got a review).
+const KEPT_PHRASE = ' reviewed · kept in Limn';
 const isFreeLine = (l) => {
     const d = l.description ?? '';
     if (l.state === 'pending')
         return d.includes(FREE_PHRASE);
     if (l.state !== 'success')
         return false;
-    return d.includes(FREE_PHRASE) || d.startsWith(CLEARED_PREFIX) || d.startsWith(RELEASED_PREFIX);
+    return d.includes(FREE_PHRASE) || d.startsWith(CLEARED_PREFIX) || d.startsWith(RELEASED_PREFIX) || d.endsWith(KEPT_PHRASE);
 };
 const settles = (l) => l.state !== 'pending' || isFreeLine(l);
 const sameLogin = (a, b) => a != null && b != null && a.toLowerCase() === b.toLowerCase();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "limn-review",
-  "version": "0.1.173",
+  "version": "0.1.174",
   "description": "Dashboard for tracking your team's GitHub PR activity across repos. Runs on your machine with your gh login.",
   "type": "module",
   "author": "Alex Wakeman",