@krmxd/onegpt 0.2.0-beta → 0.2.1-beta

package/package.json

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

package/src/agent.js

@@ -131,7 +131,15 @@ const EXECUTE_NUDGE =
 
 function wantsAction(userInput) {
   // True when the user asked for something to be built (not explained).
-  return TASK_RE.test(userInput) && !QUESTION_RE.test(userInput.trim());
+  const t = String(userInput || "").trim();
+  const tm = TASK_RE.exec(t);
+  if (!tm) return false;
+  // A question word BEFORE the build verb means they're ASKING, not
+  // requesting work: "how do I fix the login bug?" must be answered,
+  // not turned into a flurry of file edits.
+  const qm = /\b(how|what|why|when|where|which|who|whom|explain|describe)\b/i.exec(t);
+  if (qm && qm.index < tm.index) return false;
+  return true;
 }
 
 function redirectMsg(task) {
@@ -631,8 +639,11 @@ class Agent {
         // Self-correction: instructions/code instead of done work gets one
         // firm push to act - on ANY round, not just the first (tiny models
         // relapse into how-to mode mid-task). An EMPTY reply after partial
-        // work means it stalled - push it too.
-        const replyLazy = repliedWithoutActing(resp.content);
+        // work means it stalled - push it too. BUT a reply is never "lazy"
+        // once real work happened this turn: closing summaries often quote
+        // commands in fences ("run it with ```npm start```") - nudging then
+        // made the model edit files over and over instead of answering.
+        const replyLazy = repliedWithoutActing(resp.content) && doneBits.length === 0;
         const wentSilent = !resp.content.trim() && (rounds > 1 || doneBits.length > 0);
         if (nudges < 3 && !this._cancelled
             && wantsAction(userInput) && (replyLazy || wentSilent || stalledHalfway(resp.content, userInput, doneBits))) {
@@ -784,8 +795,11 @@ class Agent {
         // Self-correction: instructions/code instead of done work gets one
         // firm push to act - on ANY round, not just the first (tiny models
         // relapse into how-to mode mid-task). An EMPTY reply after partial
-        // work means it stalled - push it too.
-        const replyLazy = repliedWithoutActing(full);
+        // work means it stalled - push it too. BUT a reply is never "lazy"
+        // once real work happened this turn: closing summaries often quote
+        // commands in fences ("run it with ```npm start```") - nudging then
+        // made the model edit files over and over instead of answering.
+        const replyLazy = repliedWithoutActing(full) && doneBits.length === 0;
         const wentSilent = !full.trim() && (rounds > 1 || doneBits.length > 0);
         if (nudges < 3 && !this._cancelled
             && wantsAction(userInput) && (replyLazy || wentSilent || stalledHalfway(full, userInput, doneBits))) {
@@ -892,4 +906,4 @@ class Agent {
   }
 }
 
-module.exports = { Agent, extractJsonTools, repairToolJson };
+module.exports = { Agent, extractJsonTools, repairToolJson, wantsAction, repliedWithoutActing, stalledHalfway };