jaitus 0.3.25 → 0.3.26

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jaitus",
-  "version": "0.3.25",
+  "version": "0.3.26",
   "description": "Jaitus — a coding agent in your terminal.",
   "type": "module",
   "bin": {

package/src/adSlots.js

@@ -51,12 +51,18 @@ export function createAdSlots({ source, isVisible, onChange, now = () => Date.no
 
   function tick() {
     const t = now();
-    const elapsed = Math.max(0, t - lastTick);
+    const prev = lastTick;
     lastTick = t;
     if (!current) { stopTimer(); return; }
+    // Only time since THIS line went up counts: the timer may have been running for an
+    // earlier one, and crediting the whole tick reported views after ~0.8 s on screen.
+    const elapsed = Math.max(0, t - Math.max(prev, current.countFrom ?? current.at));
     if (!current.reported) {
       if (isVisible(current)) {
-        current.visibleMs += elapsed;
+        // Uncovered somewhere since the last tick, but when exactly is unknowable from a
+        // poll: count from this tick on, never from before it (the advertiser's side).
+        if (current.countFrom === Infinity) current.countFrom = t;
+        else current.visibleMs += elapsed;
         if (current.visibleMs >= VIEWABLE_MS) {
           current.reported = true;
           const shown = Math.round(current.visibleMs);
@@ -64,6 +70,7 @@ export function createAdSlots({ source, isVisible, onChange, now = () => Date.no
         }
       } else {
         current.visibleMs = 0;        // continuous, or it does not count
+        current.countFrom = Infinity; // and restarts at the first tick that sees it again
       }
     }
     if (current.placement === 'thinking' && wanted === 'thinking' && t - current.at >= ROTATE_THINKING_MS) {
@@ -102,6 +109,9 @@ export function createAdSlots({ source, isVisible, onChange, now = () => Date.no
       if (current?.placement === wanted) return;
       if (wanted === 'idle' && lastIdle && now() - lastIdle.at < IDLE_REUSE_MS) {
         // Back to waiting moments after the last wait: the same line, not a new serve.
+        // Its clock starts again from now — it was off screen in between.
+        if (!lastIdle.slot.reported) { lastIdle.slot.visibleMs = 0; }
+        lastIdle.slot.countFrom = now();
         set(lastIdle.slot);
         ensureTimer();
         return;
@@ -134,7 +144,10 @@ export function adSourceFor(client, { timeoutMs = 2500 } = {}) {
       return r.ok ? (r.body?.data?.ad || null) : null;
     },
     async seen(token, shownMs) {
-      await call('/ads/cli/seen', { method: 'POST', headers: { 'content-type': 'application/json' }, body: JSON.stringify({ token, shownMs }) });
+      // No content-type here: client.api already sends one, and a second made the header
+      // "application/json, application/json", which the server's JSON parser does not
+      // recognise — so in 0.3.25 every report arrived with no body and was refused.
+      await call('/ads/cli/seen', { method: 'POST', body: JSON.stringify({ token, shownMs }) });
     },
   };
 }

package/src/tui-ink.js

@@ -1493,6 +1493,7 @@ export function createTui({ slashCommands = [], fileProvider = null, meta = {},
     },
   }) : null;
 
+  let adBusyTimer = null;
   let inkInstance = null;
   let update = () => {};
   let pending = null;
@@ -2893,6 +2894,7 @@ export function createTui({ slashCommands = [], fileProvider = null, meta = {},
     stop() {
       state.active = false;
       exited = true;
+      clearTimeout(adBusyTimer);
       adSlots?.stop();
       if (pending) { clearTimeout(pending); pending = null; }
       // Before unmount: the terminal keeps reporting after the process is gone
@@ -3028,7 +3030,19 @@ export function createTui({ slashCommands = [], fileProvider = null, meta = {},
       state.busy = !!value;
       if (value) { state.turnStartedAt = Date.now(); state.turnTokens = 0; state.turnCredits = 0; }
       else { state.status = ''; state.turnStartedAt = null; }
-      adSlots?.want(value ? 'thinking' : 'idle');
+      if (adSlots) {
+        clearTimeout(adBusyTimer);
+        if (value) {
+          // Only work that lasts gets a "thinking" line: an instant command (/help,
+          // /usage) would otherwise fetch one just to throw it away.
+          adBusyTimer = setTimeout(() => { if (state.busy && !state.panel?.adSlot) adSlots.want('thinking'); }, 800);
+          adBusyTimer.unref?.();
+        } else {
+          // /usage runs as a command: its panel is still open when the command ends, and
+          // it keeps its own slot until it is closed.
+          adSlots.want(state.panel?.adSlot ? 'usage' : 'idle');
+        }
+      }
       notifyNow();
     },
     setStatus(value) { state.status = plain(value); notify(); },