ucode-agent 1.18.0 → 1.19.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ucode-agent",
-  "version": "1.18.0",
+  "version": "1.19.1",
   "description": "ucode - a terminal coding agent that reads, edits and runs your code, on NVIDIA and Cohere models.",
   "type": "module",
   "main": "ucode.js",

package/src/core/loop.js

@@ -937,6 +937,7 @@ export class Agent {
 
     this.busy = true;
     this.abort = new AbortController();
+    this.endedSilently = false;
 
     const turnStarted = Date.now();
     let finished = false;
@@ -962,6 +963,7 @@ export class Agent {
       this.stats.workMs += Date.now() - turnStarted;
       this.stats.turns++;
       if (!finished) this.closeInterrupted();
+      const ok = finished && !this.endedSilently;
       this.busy = false;
       this.abort = null;
       this.ui.stopSpinner();
@@ -972,7 +974,7 @@ export class Agent {
       if (finished) this.openWhenReady(turnStarted);
       // Last, so "Done" is the last thing that happens rather than the last
       // thing said before several more things happen.
-      this.ui.turnEnd?.({ ok: finished });
+      this.ui.turnEnd?.({ ok });
     }
   }
 
@@ -1169,14 +1171,21 @@ export class Agent {
           this.push({
             role: 'user',
             content:
-              'You finished without saying anything. In one or two sentences: what did ' +
-              'you change, and does it work? No preamble, no repeating the diffs.',
+              'You stopped without saying anything. If the thing I asked for is not ' +
+              'built yet, carry on and build it. If it is, tell me in one or two ' +
+              'sentences what it does and how to try it. No preamble, no diffs.',
           });
           continue;
         }
 
         this.push({ role: 'assistant', content: reply.text });
-        if (!reply.text?.trim()) this.ui.note('the model ended the turn without a reply');
+        if (!reply.text?.trim()) {
+          // Silence after being asked to speak is not a finished turn. Saying
+          // "Done" here is the worst thing available: the user believes it,
+          // looks, and finds the thing they asked for was never built.
+          this.ui.note('the model stopped without saying anything — the work may be unfinished');
+          this.endedSilently = true;
+        }
         return;
       }
 

package/src/tools/index.js

@@ -639,7 +639,11 @@ export function describe(name, args = {}) {
     case 'deploy':
       return `Deploying ${clip(args.folder || '.', 30)} to Vercel`;
     case 'create_app':
-      return `Creating ${clip(args.name || args.folder, 30)} from the Next.js starter`;
+      // Say which starter it actually is. Hardcoding one of them meant a plain
+      // HTML app announced itself as Next.js, which is a line that is simply
+      // untrue on screen while the opposite happens on disk.
+      return `Creating ${clip(args.name || args.folder, 30)} from the ` +
+        `${args.template === 'plain-html' ? 'HTML' : 'Next.js'} starter`;
     case 'look_at_app':
       return `Looking at ${clip(args.url, 40)} on a phone and a desktop`;
     case 'web_search':

package/src/ui/activity.js

@@ -192,9 +192,12 @@ export function fitActivity({ glyph, label = '', meta = [], hint = '', paint = d
  * theme; the rest dim, because it is a footnote to the answer above it rather
  * than something to read first.
  */
-export function doneLine(ms, steps) {
-  const count = steps > 0 ? ` · ${steps} step${steps === 1 ? '' : 's'}` : '';
-  return `${theme.ok('✓')} ${dim(`Done in ${formatDuration(ms)}${count}`)}`;
+export function doneLine(ms, steps, { ok = true } = {}) {
+  // The step count is bookkeeping: it tells the reader nothing about whether
+  // the thing they asked for exists. /stats has it for anyone who wants it.
+  void steps;
+  if (!ok) return `${theme.warn('!')} ${dim(`Stopped after ${formatDuration(ms)} without finishing`)}`;
+  return `${theme.ok('✓')} ${dim(`Done in ${formatDuration(ms)}`)}`;
 }
 
 /** The step count, brighter for a moment right after it goes up. */

package/src/ui/plain.js

@@ -262,7 +262,7 @@ export class Plain {
   turnEnd({ ok = true } = {}) {
     const t = this.turn;
     this.turn = null;
-    if (t && ok && Date.now() - t.start >= 2000) this.write(`  ${doneLine(Date.now() - t.start, t.steps)}`);
+    if (t && Date.now() - t.start >= 2000) this.write(`  ${doneLine(Date.now() - t.start, t.steps, { ok })}`);
   }
 
   // -- input ---------------------------------------------------------------

package/src/ui/screen.js

@@ -912,7 +912,9 @@ export class Screen {
     const a = this.activity;
     this.activity = null;
     if (!this.status.busy) this.stopTimer();
-    if (a && ok && Date.now() - a.start >= 2000) this.push(`  ${doneLine(Date.now() - a.start, a.steps)}`);
+    // A turn that stopped without finishing says so. Dropping the line entirely
+    // left the transcript looking like the work was still going.
+    if (a && Date.now() - a.start >= 2000) this.push(`  ${doneLine(Date.now() - a.start, a.steps, { ok })}`);
     this.paintStatus();
   }
 

package/ucode.js

@@ -88,7 +88,13 @@ async function main() {
   // have no way to know. So if there is something newer, it goes on now and
   // this process hands over to it. The wait is a few seconds, once per
   // release; every other launch pays one quick question to the registry.
-  if (!args.version && process.argv[2] !== 'login' && process.argv[2] !== 'doctor') {
+  //
+  // Only when a person is sitting there. Handing over means re-running this
+  // process, and a run whose input is a pipe has that input consumed by the
+  // handover — `cat prompt.txt | ucode` printed "updating…" and then did
+  // nothing at all. Piped runs take the background update and carry on.
+  if (!args.version && process.stdin.isTTY &&
+      process.argv[2] !== 'login' && process.argv[2] !== 'doctor') {
     const { pendingUpdate, installNow } = await import('./src/core/updater.js');
     const waiting = await pendingUpdate();
     if (waiting) {