@tiens.nguyen/gonext-local-worker 1.0.200 → 1.0.202

package/gonext-repl.mjs

@@ -206,6 +206,19 @@ const ask = async (q) => {
   rl.setPrompt(PROMPT);
   return (a ?? "").trim();
 };
+// Yes/no prompt with an explicit default. Empty input (just Enter) → the default;
+// otherwise the FIRST non-space character decides (y*→yes, n*→no), and anything else
+// re-uses the default. This replaces a fragile `(await ask()) || "y"` idiom where any
+// leftover character (a stray escape byte, a stray space) silently defeated the
+// default — the reported "[Y/n] + Enter registered as No" bug.
+const askYesNo = async (question, defaultYes) => {
+  const suffix = defaultYes ? " [Y/n] " : " [y/N] ";
+  const raw = (await ask(question + suffix)).trim().toLowerCase();
+  if (!raw) return defaultYes; // just Enter → the default
+  if (raw[0] === "y") return true;
+  if (raw[0] === "n") return false;
+  return defaultYes; // unrecognized → default (never silently flip it)
+};
 
 // ---------- workspace registration for the current folder ----------
 async function readWorkspaces() {
@@ -229,16 +242,21 @@ async function ensureWorkspace() {
   }
   if (!existsSync(cwd) || !statSync(cwd).isDirectory()) return;
   console.log(`This folder is not a registered workspace:\n  ${cwd}`);
-  const reg = (await ask("Register it so the agent can read/edit code here? [Y/n] ")) || "y";
-  if (!/^y(es)?$/i.test(reg)) {
+  const register = await askYesNo(
+    "Register it so the agent can read/edit code here?", true /* default Yes */
+  );
+  if (!register) {
     console.log(dim("Skipped — the agent will not be able to touch files here."));
     return;
   }
-  const trust = (await ask(
+  const allowRun = await askYesNo(
     "Do you trust this folder — allow the agent to RUN build/test commands\n" +
-      "(npm test, mvn, pytest…)? [y/N] "
-  )) || "n";
-  const allowRun = /^y(es)?$/i.test(trust);
+      "(npm test, mvn, pytest…)?",
+    true /* default Yes — you opened THIS folder with gonext deliberately, and the
+            action-first workflows (start/test/build the project) need it; commands are
+            still allowlisted (npm/mvn/pytest/…) and run with a scrubbed env. Type n to
+            keep a workspace read/edit-only. */
+  );
   const next = list.filter((w) => w.path !== cwd);
   next.push({
     name: basename(cwd),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tiens.nguyen/gonext-local-worker",
-  "version": "1.0.200",
+  "version": "1.0.202",
   "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",