@tiens.nguyen/gonext-local-worker 1.0.184 → 1.0.186

package/gonext-repl.mjs

@@ -43,6 +43,11 @@ const CLEAR_LINE = "\r\x1b[K"; // carriage-return + erase-to-end-of-line
 // also drop the worker's ~~~ stream fences (they only matter to the web renderer).
 const isHeartbeatLine = (line) => /…\s?\(\d+s\)\s*$/.test(line);
 const isFenceLine = (line) => line.trim() === "~~~";
+// The model's own raw Python code blob, delimited by <code>/</code> lines in its
+// Thought+Code output. It's redundant with the step summary that follows it (e.g.
+// "Downloading → url"), so we don't print it — only the "Thought:" prose survives.
+const isCodeOpenLine = (line) => line.trim() === "<code>";
+const isCodeCloseLine = (line) => line.trim() === "</code>";
 
 // Playful words for the local "thinking…" ticker, reused verbatim from the worker's
 // thinking_words.txt sibling (falls back to a tiny built-in list if it's missing).
@@ -290,7 +295,10 @@ async function runAgentTurn(history) {
   };
 
   // Split newly-arrived text into lines: heartbeats + ~~~ fences are swallowed (the ticker
-  // shows progress); blank lines keep spacing; everything else commits dim to scrollback.
+  // shows progress); the model's raw <code>…</code> blob is swallowed too (redundant with
+  // the step summary that follows it, e.g. "Downloading → url"); blank lines keep spacing;
+  // everything else (the "Thought:" prose) commits dim to scrollback.
+  let inCode = false;
   const consume = (chunk) => {
     carry += stripThinkTags(chunk);
     let nl;
@@ -298,6 +306,18 @@ async function runAgentTurn(history) {
       const line = carry.slice(0, nl);
       carry = carry.slice(nl + 1);
       if (isHeartbeatLine(line) || isFenceLine(line)) continue; // swallow
+      if (!inCode && isCodeOpenLine(line)) {
+        // Reasoning just ended and code is about to run — stamp the thought complete,
+        // then suppress the code body itself.
+        onContent();
+        inCode = true;
+        continue;
+      }
+      if (inCode) {
+        lastContentAt = Date.now(); // content IS arriving, just not printed
+        if (isCodeCloseLine(line)) inCode = false;
+        continue;
+      }
       if (line.trim() === "") {
         if (!thinking && !statusShown) process.stdout.write("\n");
         continue;

package/gonext_agent_chat.py

@@ -488,8 +488,14 @@ def _summarise_step(step_log):
         name = getattr(tc, "name", "")
         args = getattr(tc, "arguments", None)
 
-        if name == "python_interpreter" and isinstance(args, dict):
-            code = args.get("code", "")
+        if name == "python_interpreter":
+            # smolagents' CodeAgent passes the raw code STRING as `arguments` (NOT a
+            # dict — only ToolCallingAgent's structured calls use {"code": ...}). Handle
+            # both shapes, else this always fell through to the useless "python_interpreter"
+            # label below and never showed what was actually called.
+            code = args if isinstance(args, str) else (
+                args.get("code", "") if isinstance(args, dict) else ""
+            )
             # Show the http_request call if present, else first meaningful line
             m = re.search(r'http_request\s*\(\s*(?:method\s*=\s*)?[\'"]?(\w+)[\'"]?\s*,\s*(?:url\s*=\s*)?[\'"]([^\'"]+)', code)
             if m:
@@ -514,9 +520,13 @@ def _summarise_step(step_log):
 
     if observations:
         obs = str(observations).strip()
-        # smolagents prefixes output with "Execution logs:" — skip that header line
-        # and show the first line of actual content.
-        lines = [l for l in obs.splitlines() if l.strip() and l.strip() != "Execution logs:"]
+        # smolagents wraps observations as "Execution logs:\n<print output>\nLast output
+        # from code snippet:\n<return value>" — both header lines are pure boilerplate.
+        # Without filtering "Last output from code snippet:" too, a call with no print()
+        # output (most of our @tool functions just `return`, not print) surfaced that
+        # HEADER as the shown text instead of the actual return value on the next line.
+        _OBS_BOILERPLATE = ("Execution logs:", "Last output from code snippet:")
+        lines = [l for l in obs.splitlines() if l.strip() and l.strip() not in _OBS_BOILERPLATE]
         if lines:
             parts.append(f"→ {_clip(lines[0])}")
 
@@ -3277,8 +3287,10 @@ def run_agent_chat(cfg):
                 for dirpath, dirnames, filenames in os.walk(root):
                     dirnames[:] = [d for d in dirnames if d not in _RAG_SKIP_DIRS]
                     for fn in filenames:
-                        rel = os.path.relpath(os.path.join(dirpath, fn), root)
-                        entries.append(rel)
+                        # ABSOLUTE path — read_file_lines/read_text_file resolve relative
+                        # paths against the WORKSPACE root, not whatever subfolder was
+                        # listed here, so a name relative to `root` could resolve wrong.
+                        entries.append(os.path.join(dirpath, fn))
                         if len(entries) >= 300:
                             break
                     if len(entries) >= 300:
@@ -3447,7 +3459,11 @@ def run_agent_chat(cfg):
                         with open(abs_path, "r", encoding="utf-8", errors="replace") as fh:
                             for i, line in enumerate(fh, 1):
                                 if rx.search(line):
-                                    hits.append(f"{rel}:{i}: {line.rstrip()[:200]}")
+                                    # ABSOLUTE path — read_file_lines/read_text_file resolve
+                                    # relative paths against the WORKSPACE root, not this
+                                    # search root, so a path relative to `root` (e.g. when
+                                    # searching a subfolder) would be silently wrong there.
+                                    hits.append(f"{abs_path}:{i}: {line.rstrip()[:200]}")
                                     if len(hits) >= 100:
                                         break
                     except OSError:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tiens.nguyen/gonext-local-worker",
-  "version": "1.0.184",
+  "version": "1.0.186",
   "description": "Polls GoNext cloud API for async local LLM jobs and runs them against Ollama/OpenAI-compatible servers on this Mac",
   "type": "module",
   "license": "MIT",