ucode-agent 1.17.0 → 1.19.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ucode-agent",
-  "version": "1.17.0",
+  "version": "1.19.0",
   "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

@@ -458,13 +458,19 @@ function systemPrompt({ cwd, skills, mode, check, map, memory }) {
     'already contains it. Do not re-check work the checks have already reported on.',
     'Fast is not sloppy: it is the same work with the waiting taken out.',
     '',
-    'FIRST, EVERY TIME: one short line saying what you are about to do, then the tool',
-    'calls. Never open a turn with a tool call and no words. "Right, the HTML',
-    'structure first." / "Now the state and the render loop." / "That is the layout',
-    'done - onto the animations." / "Let me see what is there." One sentence, your own',
-    'voice, before the actions - not after them, not instead of them, not a',
-    'restatement of the request. The user watches this scroll past, and without those',
-    'lines it is a list of file operations they cannot read intent from.',
+    'SPEAK AS "I", NEVER "WE". You are doing this, not a committee: "I will build',
+    'Tide as a single HTML file", not "we have created the file".',
+    '',
+    'Say three or four things over a whole build, not one per step:',
+    '  - Open with what you are going to make and how, in one line, before anything',
+    '    else: "I will build Tide as a single HTML file - markup, one stylesheet, one',
+    '    module - with the tasks kept in localStorage."',
+    '  - One line when you move between the big pieces of work: "The layout is done,',
+    '    now the animations."',
+    '  - One line at the end saying what it does and how to try it.',
+    'That is all. A line before every tool call is not narration, it is noise: the',
+    'steps already show on screen, and repeating them in words buries the few',
+    'sentences worth reading.',
     '',
     'FIRST, EVERY TIME: write one short line saying what you are about to do, then',
     'make the tool calls. Never open a turn with a tool call and no words. Examples:',
@@ -931,6 +937,7 @@ export class Agent {
 
     this.busy = true;
     this.abort = new AbortController();
+    this.endedSilently = false;
 
     const turnStarted = Date.now();
     let finished = false;
@@ -956,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();
@@ -966,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 });
     }
   }
 
@@ -1163,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

@@ -571,7 +571,9 @@ export class Screen {
     if (line.length < 12) return;   // "Okay." tells nobody anything
 
     this.openedWith = line;
-    this.push(narration('  ' + clip(line, Math.max(30, this.width() - 6))));
+    // Full strength: this is the model talking, and it is the thing on the page
+    // worth reading. The dimmed lines around it are the machinery.
+    this.push('  ' + chalk.white(clip(line, Math.max(30, this.width() - 6))));
   }
 
   thinkingEnd() {
@@ -910,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();
   }