ucode-agent 1.50.0 → 1.52.0

package/package.json

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

package/src/core/loop.js

@@ -1911,6 +1911,31 @@ export class Agent {
       this.reads.set(key, seen + 1);
     }
 
+    // create_app again for the app this turn already made. DeepSeek scaffolded
+    // tasker with no files, then called create_app twice more — once with no
+    // folder or name at all — and was refused both times while the empty
+    // starter sat there. The folder and name are known, so fill them in; with
+    // no files there is nothing to create, and the next step is batch_write.
+    if (call.name === 'create_app' && this.apps?.length === 1) {
+      const made = this.apps[0];
+      const args = (call.args ??= {});
+      if (!args.folder) args.folder = path.relative(this.cwd, made) || '.';
+      if (!args.name) args.name = path.basename(made);
+      const files = args.files;
+      const none = files == null || files === ''
+        || (Array.isArray(files) ? !files.length : typeof files === 'object' && !Object.keys(files).length);
+      if (none && path.resolve(this.cwd, String(args.folder)) === made) {
+        const show = path.relative(this.cwd, made) || '.';
+        throw new ToolFailure({
+          kind: 'already_made',
+          attempted: `creating ${show}`,
+          failed: `${show} was already created this turn, with the starter files in it.`,
+          fix: `Do not call create_app again. Write the app into ${show} with one batch_write, `
+            + 'replacing the starter files with the finished ones.',
+        });
+      }
+    }
+
     // Starting a second app instead of fixing the first.
     //
     // A traced build hit a problem in todo/, abandoned it and made todo-fixed/

package/src/core/provider.js

@@ -94,6 +94,10 @@ export const MODELS = {
   'deepseek/deepseek-v4-flash-0731:free': {
     name: 'DeepSeek V4 Flash',
     context: 1_048_576,
+    // Left to itself it thinks for minutes before every step — over two
+    // minutes of nothing on screen before its first word. Low effort answers
+    // in seconds and still plans the build.
+    reasoning: { effort: 'low' },
     note: 'fast DeepSeek coder, 1M context — on trial',
   },
   'qwen/qwen3.8-27b:free': {
@@ -162,6 +166,9 @@ export const stallLimit = () => Number(process.env.UCODE_STALL_MS) || 60_000;
 
 /** Freezes in a row before ucode stops asking and says to switch models. */
 export const MAX_STALLS = 3;
+
+/** How long a reply may go quiet once it has started writing a tool call. */
+const writingLimit = () => Number(process.env.UCODE_STALL_MS) || 300_000;
 let stalls = 0;
 
 let current = process.env.UCODE_MODEL || DEFAULT_MODEL;
@@ -632,7 +639,8 @@ export async function ask(messages, tools = [], opts = {}) {
   }
   if (opts.temperature !== undefined) request.temperature = opts.temperature;
   if (opts.maxOutputTokens) request.max_tokens = opts.maxOutputTokens;
-  if (opts.reasoning) request.reasoning = opts.reasoning;
+  const reasoning = opts.reasoning ?? MODELS[id]?.reasoning;
+  if (reasoning) request.reasoning = reasoning;
 
   // A side call (the design review) passes fewer: it is better skipped than
   // waited on through a string of rate-limit pauses.
@@ -728,9 +736,14 @@ async function streamed(request, opts, id) {
   opts.signal?.addEventListener('abort', stop, { once: true });
   let stalled = false;
   let timer;
+  // Once a tool call has begun, the silence may be the call being written.
+  // DeepSeek's free upstream holds a create_app back until the whole app is
+  // done — two or three quiet minutes — and a one-minute watchdog killed every
+  // build it started. So a call in progress gets five minutes instead.
   const alive = () => {
     clearTimeout(timer);
-    timer = setTimeout(() => { stalled = true; quiet.abort(); }, stallLimit());
+    const limit = partial.size ? writingLimit() : stallLimit();
+    timer = setTimeout(() => { stalled = true; quiet.abort(); }, limit);
   };
 
   let text = '';

package/src/tools/index.js

@@ -581,6 +581,18 @@ function check(name, args) {
     return ['the arguments must be a JSON object'];
   }
 
+  // One file named as "path" where the tool takes a list called "paths".
+  // DeepSeek did it to four read_files at once, a whole round trip spent being
+  // told a plural it could simply have been given.
+  for (const key of schema.required ?? []) {
+    const one = key.endsWith('s') ? key.slice(0, -1) : '';
+    if (args[key] == null && one && !schema.properties[one] && args[one] != null
+      && schema.properties[key]?.type === 'array') {
+      args[key] = Array.isArray(args[one]) ? args[one] : [args[one]];
+      delete args[one];
+    }
+  }
+
   for (const key of schema.required ?? []) {
     if (args[key] === undefined || args[key] === null) problems.push(`"${key}" is required and missing`);
   }
@@ -709,7 +721,7 @@ export function describe(name, args = {}) {
       // untrue on screen while the opposite happens on disk.
       return `Creating ${clip(args.name || args.folder, 30)} from the ` +
         `${args.template === 'next-shadcn' ? 'Next.js' : 'HTML'} starter` +
-        `${files ? ` with ${files} file${files === 1 ? '' : 's'}` : ''}`;
+        `${files ? ` with ${files} file${files === 1 ? '' : 's'}` : ''}`;
     }
     case 'look_at_app':
       return `Looking at ${clip(args.url, 40)} on a phone and a desktop`;

package/src/tools/scaffold.js

@@ -431,6 +431,18 @@ export async function createApp({ folder, name, description, template = 'plain-h
   // The app's own files, written in this same call. Two round trips become
   // one, and round trips are nearly all of the time a build takes.
   // Next.js only: paths that have missed src/ are corrected before they land.
+  // Paths are relative to the project root, but a model will name a file
+  // "index.html" meaning the app's own. DeepSeek did: its tasker went to the
+  // project root and the starter was left in tasker/ as the app. A path that
+  // is not already under the folder is put under it.
+  const home = resolveIn(folder, 'create_app', 'folder').abs;
+  for (const f of given) {
+    const abs = resolveIn(f.path, 'create_app', 'files').abs;
+    const rel = path.relative(home, abs);
+    if (rel.startsWith('..') || path.isAbsolute(rel)) {
+      f.path = path.join(String(folder), String(f.path).replace(/^[./\\]+/, ''));
+    }
+  }
   const placed = template === 'next-shadcn' ? placeForNext(given, path.basename(target.abs)) : { files: given, moved: [] };
   const mine = placed.files;
   const wrote = mine.length ? await batchWrite({ files: mine }) : null;