npm - @krmxd/onegpt - Versions diffs - 0.2.3-beta → 0.2.4-beta - Mend

@krmxd/onegpt 0.2.3-beta → 0.2.4-beta

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (3) hide show

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@krmxd/onegpt",
-  "version": "0.2.3-beta",
+  "version": "0.2.4-beta",
   "description": "OGPT - AI coding assistant for the terminal with a built-in local engine",
   "main": "src/index.js",
   "bin": {

package/src/agent.js CHANGED Viewed

@@ -109,6 +109,9 @@ const TOOL_GUIDANCE = (
   "'/home/user/...' or '/Users/...' - they do not exist on this machine.\n" +
   "- Tool arguments are REAL values: 'path': 'my_folder/app.py'. Never send the " +
   "parameter schema ({'type': 'string'}) as a value, and never leave arguments empty.\n" +
+  "- Call tools ONLY as exact JSON: {\"name\": \"write_file\", \"arguments\": {...}}. " +
+  "NEVER invent formats like 'name_file:path' or bare '<html>' dumps - those are NOT " +
+  "tool calls, they will be ignored, and you must redo them properly.\n" +
   "- To act, output plain JSON tool calls (never shell commands in code fences):\n" +
   '{"name": "write_file", "arguments": {"path": "widgets/gadget.py", ' +
   '"content": "print(\'hi\')"}}\n' +
@@ -195,6 +198,21 @@ function repliedWithoutActing(reply) {
   return SLOTH_RE.test(reply) || reply.includes("```");
 }
+// Tiny models degenerate into printing the same chunk over and over
+// ("name_file:web <html>..." x20). Detect any meaningful line repeated
+// too many times so the agent can cut the run instead of spamming chat.
+function degenerateRepeat(text, minLen = 12, maxHits = 4) {
+  const counts = new Map();
+  for (const rawLine of String(text || "").split("\n")) {
+    const line = rawLine.trim();
+    if (line.length < minLen) continue;
+    const n = (counts.get(line) || 0) + 1;
+    counts.set(line, n);
+    if (n >= maxHits) return line;
+  }
+  return null;
+}
 // "Created the folder." style announcements. Harmless on their own, but a
 // red flag when the request named files that were never actually created
 // (e.g. 'a folder AND an index.html' - folder done, file skipped).
@@ -607,6 +625,7 @@ class Agent {
     let nudges = 0;
     this._autoChecks = 0;
     let lastSig = null;
+    let collectedText = "";
     let loopBreaks = 0;
     const doneBits = [];
@@ -733,6 +752,7 @@ class Agent {
     let nTokens = 0;
     this.lastRun = {};
     let lastSig = null;
+    let collectedText = "";
     let loopBreaks = 0;
     const doneBits = [];
     let sawText = false;
@@ -781,6 +801,20 @@ class Agent {
       }
       const full = contentParts.join("");
+      // Degeneration breaker: same meaningful line 4+ times in ONE round
+      // (or across the whole reply) -> stop instead of spamming chat.
+      const rep = degenerateRepeat(full)
+        || (full.trim().length < 40 && degenerateRepeat(collectedText, 8, 6));
+      if (rep) {
+        this.history.push({ role: "assistant", content: full, ts: Date.now() / 1000 });
+        yield "\n· repeated output detected - cutting it off\n";
+        if (wantsAction(userInput)) {
+          this.history.push({ role: "user", content: redirectMsg(userInput), ts: Date.now() / 1000 });
+          if (loopBreaks < 2) { loopBreaks++; continue; }
+        }
+        break;
+      }
+      collectedText = (collectedText ? collectedText + "\n" : "") + full;
       this.usage.prompt += usage.prompt;
       this.usage.completion += usage.completion;
       this.usage.total += usage.total;
@@ -966,4 +1000,5 @@ class Agent {
 }
 module.exports = { Agent, extractJsonTools, repairToolJson, wantsAction,
-  repliedWithoutActing, stalledHalfway, syntaxCheckError, autoCheckMsg };
+  repliedWithoutActing, stalledHalfway, syntaxCheckError, autoCheckMsg,
+  degenerateRepeat };

package/src/render.js CHANGED Viewed

@@ -257,18 +257,19 @@ class SmoothPrinter {
     let preview = this.buf;
     if (preview.length > MAX_PREVIEW_CHARS) preview = preview.slice(-MAX_PREVIEW_CHARS);
     const rendered = renderMarkdown(preview, width);
-    const lines = rendered.split("\n");
-    const rows = Math.max(0, lines.length - 1);
-    const maxRows = (this.out.rows || 2000) - 1;
-    if (rows > maxRows) {
-      // Taller than the screen: scrolling would already have eaten the top
-      // rows - repainting can't reach them. Degrade instead of duplicating.
-      this._degrade(true);
-      return;
-    }
+    let lines = rendered.split("\n");
+    const target = Math.max(0, lines.length - 1);
+    const maxRows = Math.max(3, (this.out.rows || 2000) - 2);
+    // Viewport with INCREMENTAL REVEAL: grow by one row per tick so the
+    // painted block never pushes past the screen edge (which would scroll
+    // ghosts in and desync the next erase). Once at maxRows it repaints a
+    // stable window; finish() prints the complete styled message.
+    let take = Math.min(target, maxRows, this.drawn + 1);
+    if (target < take) take = target;
+    lines = lines.slice(-(take + 1));
     this._erase();
-    this.out.write(rendered);
-    this.drawn = rows;
+    this.out.write(lines.join("\n"));
+    this.drawn = Math.max(0, lines.length - 1);
     this.paintWidth = cols || width;
     this.lastPaint = now;
   }