openzoo 0.49.2 → 0.49.3

package/lib/grokui.mjs

@@ -691,6 +691,7 @@ const AUTO_RACE_RETRY = 'AUTO is still on — the last model call failed (race/e
   + 'Do not stop and do not ask the user to type continue. '
   + 'Emit the next directive now (RUN:/SPAWN:/SEND:/READ:/WRITE:/GLOB:/FETCH:/MCP:/SERVE:), '
   + 'or DONE: if the job is actually finished.';
+const AUTO_EMPTY_RETRY = 'AUTO_EMPTY_RETRY: the command produced no output, try a different command or a different path, do not stop.';
 // Said it would, without a directive line. "Spawned X" and "working on it" are
 // in here because they are FALSE without a SPAWN: in the same reply — the bot
 // reports success for something the harness never saw.
@@ -709,13 +710,43 @@ function isTransientModelFail(text) {
 function isPaymentFailed(text) {
   return /\b(?:payment failed|HTTP 402|wallet is empty|empty wallet)\b/i.test(String(text || ''));
 }
+// Empty stdout, "(no output)", or a directive that found nothing. That is
+// still a command-output hop today, so AUTO used to chain once and then park
+// as if the job had succeeded. It has not — try another command or path.
+function isEmptyExecOutput(text) {
+  const s = String(text ?? '').trim();
+  return !s || s === '(no output)';
+}
+function isEmptyDirectiveAck(text) {
+  const s = String(text ?? '').trim();
+  if (isEmptyExecOutput(s)) return true;
+  if (/:\s*\(empty\)$/i.test(s)) return true;
+  if (/:\s*no matches\s*$/i.test(s)) return true;
+  return false;
+}
+function isEmptyToolResult(text) {
+  if (text == null) return false;
+  const s = String(text).trim();
+  if (isEmptyExecOutput(s)) return true;
+  const cmd = /^\(command output\)\s*([\s\S]*)$/.exec(s);
+  if (cmd) return isEmptyExecOutput(cmd[1]);
+  const dir = /^\(directive result\)\s*([\s\S]*)$/.exec(s);
+  if (dir) return isEmptyDirectiveAck(dir[1]);
+  return false;
+}
+function isEmptyShownRun(text) {
+  const shown = /^\$ [^\n]*\n([\s\S]*)$/.exec(String(text ?? ''));
+  return Boolean(shown && isEmptyExecOutput(shown[1]));
+}
 // Park only: ask mode, pendingRun, DONE:, 402/empty-wallet, or the hard cap.
-function shouldKeepAuto(t, reply) {
+// Empty /(no output) exec is not DONE — keep going with AUTO_EMPTY_RETRY.
+function shouldKeepAuto(t, reply, userText) {
   if (!t || t.runMode !== 'auto') return false;
   if (t.pendingRun) return false;
   if ((t.autoSteps || 0) >= AUTO_MAX_STEPS) return false;
-  if (isDoneReply(reply)) return false;
   if (isPaymentFailed(reply)) return false;
+  if (isEmptyToolResult(userText) || isEmptyShownRun(reply)) return true;
+  if (isDoneReply(reply)) return false;
   return true;
 }
 function enqueueAutoHop(t, threadId, userText, onEvent) {
@@ -725,7 +756,8 @@ function enqueueAutoHop(t, threadId, userText, onEvent) {
   kickTurn(threadId, userText, onEvent).catch(() => {});
   return true;
 }
-function autoHopText(reply) {
+function autoHopText(reply, userText) {
+  if (isEmptyToolResult(userText) || isEmptyShownRun(reply)) return AUTO_EMPTY_RETRY;
   return isTransientModelFail(reply) ? AUTO_RACE_RETRY : AUTO_CONTINUE;
 }
 // PING used to be a read: last-line status, no turn. Idle children stayed idle
@@ -805,7 +837,8 @@ without a fact only the user has.
 
 When the job is actually finished, emit DONE: as the first line. A status
 sentence is not a stop — the harness keeps this thread working until DONE:,
-a real blocking question, or the step cap.`;
+a real blocking question, or the step cap. Empty output, "(no output)", and
+GLOB/GREP with no matches are not finished — try a different command or path.`;
 async function bindThread(t) {
   // Only bind what's NEW since the last successful bind, continuing the
   // existing context_id — previously this rebuilt and re-sent the WHOLE
@@ -992,7 +1025,7 @@ function execCommand(command, cwd) {
     exec(command, { cwd, shell: RUN_SHELL, timeout: RUN_TIMEOUT_MS, maxBuffer: 10 * 1024 * 1024 }, (err, stdout, stderr) => {
       let out = (stdout || '') + (stderr ? '\n' + stderr : '');
       if (err) out += `\n(exit ${err.code ?? 1})`;
-      resolve(keepWhole(out) || '(no output)');
+      resolve(keepWhole(out).trim() || '(no output)');
     });
   });
 }
@@ -1411,7 +1444,7 @@ setInterval(() => {
     const lastBot = [...(t.history || [])].reverse().find((h) => h.who === 'bot');
     const lastText = lastBot?.text || '';
     if (shouldKeepAuto(t, lastText)) {
-      kickTurn(t.id, isTransientModelFail(lastText) ? AUTO_RACE_RETRY : AUTO_CONTINUE).catch(() => {});
+      kickTurn(t.id, autoHopText(lastText)).catch(() => {});
     } else {
       t.status = 'idle';
       t.liveStatus = '';
@@ -1676,7 +1709,9 @@ function isHarnessUserText(text) {
   return /^\((command output|directive result)\)/.test(String(text || ''))
     || String(text || '') === NUDGE
     || String(text || '') === AUTO_CONTINUE
-    || String(text || '') === AUTO_RACE_RETRY;
+    || String(text || '') === AUTO_RACE_RETRY
+    || String(text || '') === AUTO_EMPTY_RETRY
+    || String(text || '').startsWith('AUTO_EMPTY_RETRY:');
 }
 
 function firstUserAsk(t) {
@@ -2502,8 +2537,8 @@ async function runTurn(threadId, userText, onEvent, images) {
     }
     bindThread(t).catch(() => {});
     lastReply = memberReply;
-    if (shouldKeepAuto(t, memberReply)) {
-      chained = enqueueAutoHop(t, threadId, autoHopText(memberReply), onEvent);
+    if (shouldKeepAuto(t, memberReply, userText)) {
+      chained = enqueueAutoHop(t, threadId, autoHopText(memberReply, userText), onEvent);
     }
     return;
   }
@@ -2638,7 +2673,11 @@ async function runTurn(threadId, userText, onEvent, images) {
       // most material (command output, GLOB results, MCP tool lists). The
       // holographic context stopped growing precisely when it mattered.
       lastReply = shown;
-      chained = enqueueAutoHop(t, threadId, condense('(command output)', output), onEvent);
+      chained = enqueueAutoHop(
+        t, threadId,
+        isEmptyExecOutput(output) ? AUTO_EMPTY_RETRY : condense('(command output)', output),
+        onEvent,
+      );
       return;
     }
     {
@@ -2668,8 +2707,12 @@ async function runTurn(threadId, userText, onEvent, images) {
   // Same budget as RUN (shared t.autoSteps, reset when the user speaks), so
   // this cannot spend more than a chained RUN loop already could.
   if (t.runMode === 'auto' && ack !== null && ack !== undefined
-      && !isDoneReply(reply)) {
-    chained = enqueueAutoHop(t, threadId, condense('(directive result)', ack), onEvent);
+      && (isEmptyDirectiveAck(ack) || !isDoneReply(reply))) {
+    chained = enqueueAutoHop(
+      t, threadId,
+      isEmptyDirectiveAck(ack) ? AUTO_EMPTY_RETRY : condense('(directive result)', ack),
+      onEvent,
+    );
     return;
   }
 
@@ -2699,17 +2742,18 @@ async function runTurn(threadId, userText, onEvent, images) {
 
   // After any auto reply that is not DONE: and not waiting on approval,
   // kick immediately. Race/empty/error uses AUTO_RACE_RETRY.
-  if (shouldKeepAuto(t, reply)) {
-    chained = enqueueAutoHop(t, threadId, autoHopText(reply), onEvent);
+  if (shouldKeepAuto(t, reply, userText)) {
+    chained = enqueueAutoHop(t, threadId, autoHopText(reply, userText), onEvent);
     return;
   }
   bindThread(t).catch(() => {});
   } finally {
     // Idle only when this hop should not keep AUTO going: DONE:, pendingRun,
-    // ask mode, 402/empty-wallet, or the hard cap. Otherwise kick again.
+    // ask mode, 402/empty-wallet, or the hard cap. Empty /(no output) is not
+    // DONE — AUTO_EMPTY_RETRY. Otherwise kick again.
     if (stillMine() && !chained && !parked) {
-      if (shouldKeepAuto(t, lastReply)) {
-        enqueueAutoHop(t, threadId, autoHopText(lastReply), onEvent);
+      if (shouldKeepAuto(t, lastReply, userText)) {
+        enqueueAutoHop(t, threadId, autoHopText(lastReply, userText), onEvent);
       } else if (!t.pendingRun) {
         t.status = 'idle';
         t.liveStatus = '';
@@ -4973,12 +5017,12 @@ const APP_HTML = `<!doctype html>
         // is shipping the WHOLE corpus), so 2dp would read as noise up there.
         savedEl.textContent = (mult >= 100 ? Math.round(mult) : mult.toFixed(mult >= 10 ? 1 : 2)) + 'x';
         savedEl.className = mult >= 1 ? 'hlime' : 'hember';
-        // Session direct/spent (never first-call). Cogs-over-paid is the
-        // louder warn — unused grant-back should have kept used cogs ≤ billed.
+        // Session direct/spent (never first-call). Ember when cogs > spent —
+        // house losing. Do not treat race_unused as a user refund.
         if (cogsOver) {
           hintEl.className = 'hhint show';
-          hintEl.innerHTML = '<b>cogs above paid.</b> our cost exceeded what you were billed '
-            + '— this line is used racers, not the N+judge ceiling.';
+          hintEl.innerHTML = '<b>cogs above paid.</b> house is losing — our cost exceeded what you were billed. '
+            + 'you pay for every entrant we actually launched; failures still cost us.';
         } else {
           hintEl.className = mult >= 1 ? 'hhint' : 'hhint show';
           hintEl.innerHTML = '<b>feed it more.</b> you\\'re billed on the slice actually sent, '
@@ -5423,8 +5467,8 @@ export {
   tryDirective, ensureWorkspacePort, isPreviewableRel, previewAck, workspaceFileUrl,
   parseRun, looksLikeMcpAsBash, stripThinkTags, safeResolveIn, inDir, listDir,
   handleSlash, newThread, setRunTurnForTest, setBrainAskForTest, runTurn,
-  AUTO_CONTINUE, AUTO_RACE_RETRY, pingWakeText, pingCanWake, shouldKeepAuto,
-  isDoneReply, isTransientModelFail, enqueueAutoHop, childKickoff, findByName,
+  AUTO_CONTINUE, AUTO_RACE_RETRY, AUTO_EMPTY_RETRY, pingWakeText, pingCanWake, shouldKeepAuto,
+  isDoneReply, isTransientModelFail, isEmptyToolResult, enqueueAutoHop, childKickoff, findByName,
   attachChildDir, finishChildDir,
   lockWorktree, unlockWorktree, parsePrRef, fetchSpecsForOrigin, agentSlug,
 };

package/lib/podagent.mjs

@@ -936,6 +936,7 @@ async function readGatewayRaceStream(r, feed, signal) {
  *
  * Every entrant is paid for, including the abandoned one — this trades money
  * for latency and quality, which is why it is opt-in and capped.
+ * grokui does not grant unused or failed racers back to the user.
  *
  * `hooks` is for tests: `{ stream, classify, pairwise, minScore }`.
  */

package/lib/racesettle.js

@@ -1,6 +1,9 @@
 /**
- * Fly gateway race: one POST { race, race_need, tier }, unused grant-back,
- * and HUD cogs for the racers that actually ran — never the N+judge ceiling.
+ * Fly gateway race: one POST { race, race_need, tier }.
+ * User pays for every entrant we actually launched. Failures still cost us
+ * (OpenRouter was paid). race_unused on a receipt is informational — do not
+ * treat it as a user refund or shrink HUD cogs to hide a house loss.
+ * HUD embers when cogs > spent.
  */
 
 export const FLY_GATEWAY_HOST = 'x402-tokens.fly.dev';
@@ -92,40 +95,23 @@ export function capRaceByCredit(n, { creditUsd, quoteUsd } = {}) {
   return { n: Math.min(want, maxN), reason: 'shrunk' };
 }
 
-function unusedGrant(x) {
-  const u = x?.race_unused ?? x?.raceUnused ?? x?.unused;
-  if (u == null) return { billed: 0, cogs: 0 };
-  if (typeof u === 'number' && Number.isFinite(u)) return { billed: u, cogs: u };
-  if (typeof u !== 'object') return { billed: 0, cogs: 0 };
-  const billed = Number(u.billedUsd ?? u.refundUsd ?? u.unusedBilledUsd ?? u.usd ?? 0);
-  const cogs = Number(u.cogsUsd ?? u.unusedCogsUsd ?? u.refundCogsUsd ?? billed);
-  return {
-    billed: Number.isFinite(billed) && billed > 0 ? billed : 0,
-    cogs: Number.isFinite(cogs) && cogs > 0 ? cogs : 0,
-  };
-}
-
 /**
- * Actual used-racer cogs after unused grant-back.
- * Never the N+judge ceiling. Does not clamp to billed — HUD embers when
- * used cogs still exceed what was paid.
+ * House cost from the receipt. Do not subtract race_unused — unused
+ * grant-back is not a user refund, and shrinking cogs would hide house loss.
+ * Does not clamp to billed — HUD embers when cogs exceed what was paid.
  */
 export function receiptUsedCogs(x, markup = 3) {
   if (!x || typeof x !== 'object') return 0;
   const billedRaw = Number(x.billedUsd);
   const billedOk = Number.isFinite(billedRaw) && billedRaw >= 0;
-  let cogs = typeof x.cogsUsd === 'number' && Number.isFinite(x.cogsUsd)
-    ? x.cogsUsd
-    : (billedOk ? billedRaw / markup : 0);
-  const grant = unusedGrant(x);
-  if (grant.cogs > 0) cogs = Math.max(0, cogs - grant.cogs);
-  return cogs;
+  if (typeof x.cogsUsd === 'number' && Number.isFinite(x.cogsUsd)) return x.cogsUsd;
+  return billedOk ? billedRaw / markup : 0;
 }
 
 /**
- * Session meter. spent/direct are the receipt totals (already net of unused
- * when the gateway refunds into billedUsd) — never a first-call rewrite.
- * cogs is used racers after unused grant-back.
+ * Session meter. spent/direct are the receipt totals as billed — never a
+ * first-call rewrite, never a race_unused user refund.
+ * cogs is the house cost on that receipt (HUD embers when cogs > spent).
  */
 export function meterRaceReceipt(x, markup = 3) {
   const billed = Number(x?.billedUsd);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openzoo",
-  "version": "0.49.2",
+  "version": "0.49.3",
   "description": "Local x402-paying proxy + MCP server for openzoo.fun \u2014 point any OpenAI-compatible harness (Cursor, Claude Code, aider, SDKs) at localhost and it pays per call from a local burner wallet. Solana and Base rails live; Robinhood experimental.",
   "license": "MIT",
   "type": "module",