junecoder 1.0.11 → 1.0.13

package/agent.mjs

@@ -263,7 +263,7 @@ function defaultCallbacks(overrides = {}) {
     onToolCall: () => {},
     onToolResult: () => {},
     onToolOutput: () => {},
-    onPermissionRequest: async () => true,
+    onPermissionRequest: async () => ({ allowed: true }),
     onCompress: () => {},
     onUsage: () => {},
     onTurnEnd: () => {},

package/executor.mjs

@@ -66,10 +66,16 @@ export async function executeToolCalls(agent, toolByName, toolCalls, callbacks,
 
     // Permission request for non-readonly tools
     let permissionDenied = false;
+    let denyReason = '';
     if (!tool.readonly && cb.onPermissionRequest) {
       try {
-        const allowed = await cb.onPermissionRequest(tool, args);
-        if (!allowed) {
+        const result = await cb.onPermissionRequest(tool, args);
+        if (typeof result === 'object' && result !== null) {
+          if (!result.allowed) {
+            permissionDenied = true;
+            denyReason = result.reason || '';
+          }
+        } else if (!result) {
           permissionDenied = true;
         }
       } catch {
@@ -78,11 +84,12 @@ export async function executeToolCalls(agent, toolByName, toolCalls, callbacks,
     }
 
     if (permissionDenied) {
+      const reasonSuffix = denyReason ? ` Reason: ${denyReason}` : '';
       prepared.push({
         id,
         name,
         denied: true,
-        output: `Permission denied for "${name}".`,
+        output: `Permission denied for "${name}".${reasonSuffix}`,
       });
       continue;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "junecoder",
-  "version": "1.0.11",
+  "version": "1.0.13",
   "description": "Zero Npm Dependencies Agent Framework",
   "main": "agent.mjs",
   "type": "module",

package/tui.mjs

@@ -227,6 +227,7 @@ export async function startTUI(agent, opts = {}) {
 
     let borderColor = C.tool, title;
     if (state.question) { borderColor = C.tool; title = " " + sliceByWidth(state.question.text, W - 6) + " "; }
+    else if (state.permission?.reasonMode) { borderColor = C.warn; title = " Denied \u2014 why? (Enter to send, Esc to skip) "; }
     else if (state.permission) { borderColor = C.warn; title = state.permission.name === "continue" ? " Continue? (y/n) " : " Allow " + state.permission.name + "? (y/n/a) "; }
     else if (state.processing) title = " Processing... ";
     else if (setupMode) title = " API Key ";
@@ -271,12 +272,32 @@ export async function startTUI(agent, opts = {}) {
 
   keyStream.on("keypress", (str, key) => {
     if (state.permission) {
-      const ch = (str || key.name || "").toLowerCase();
-      if (ch === "y" || key.name === "y") state.permission.resolve(true);
-      else if (ch === "n" || key.name === "n") state.permission.resolve(false);
-      else if (ch === "a" || key.name === "a") { agent.autoApprove = true; state.permission.resolve(true); pushLine("  [auto] Auto-approve ON.", C.warn); }
-      else return;
-      state.permission = null; state.permissionPreview = []; state.status = state.processing ? "Processing..." : "Ready"; render(); return;
+      if (state.permission.reasonMode) {
+        // In reason mode: Escape cancels, everything else passes through for typing
+        if (key.name === "escape") {
+          state.input = []; state.cursor = 0;
+          const resolve = state.permission.resolve;
+          state.permission = null; state.permissionPreview = [];
+          state.status = state.processing ? "Processing..." : "Ready";
+          resolve({ allowed: false });
+          render(); return;
+        }
+        // Fall through to normal key handling (typing, Enter→submit, Ctrl+C, etc.)
+      } else {
+        const ch = (str || key.name || "").toLowerCase();
+        if (ch === "y" || key.name === "y") state.permission.resolve({ allowed: true });
+        else if (ch === "a" || key.name === "a") { agent.autoApprove = true; state.permission.resolve({ allowed: true }); pushLine("  [auto] Auto-approve ON.", C.warn); }
+        else if (ch === "n" || key.name === "n") {
+          // Enter reason mode
+          state.permission.reasonMode = true;
+          state.input = []; state.cursor = 0;
+          state.status = "Denied — why? (Enter to send, Esc to skip)";
+          render();
+          return;
+        }
+        else return;
+        state.permission = null; state.permissionPreview = []; state.status = state.processing ? "Processing..." : "Ready"; render(); return;
+      }
     }
     if (state.question) {
       if (key.name === "return" || key.name === "enter") {
@@ -344,7 +365,20 @@ export async function startTUI(agent, opts = {}) {
 
   async function submit() {
     const text = state.input.join("").trim();
-    if (!text || state.processing) return;
+    if (!text && !state.permission?.reasonMode) return;
+    if (state.processing && !state.permission?.reasonMode) return;
+
+    // ─── Permission reason mode ────────────────────────────────────────────
+    if (state.permission?.reasonMode) {
+      const reason = text;
+      state.input = []; state.cursor = 0;
+      const resolve = state.permission.resolve;
+      state.permission = null; state.permissionPreview = [];
+      state.status = state.processing ? "Processing..." : "Ready";
+      pushLine("  Denied: " + (reason || "(no reason)"), C.warn);
+      resolve({ allowed: false, reason: reason || undefined });
+      render(); return;
+    }
 
     // ─── Setup mode: first input is the API key ──────────────────────────────
     if (setupMode) {
@@ -423,7 +457,7 @@ export async function startTUI(agent, opts = {}) {
       try { await runAgent(agent, text, callbacks, { signal: state.controller.signal, resume }); flushStream(); break; }
       catch (error) { flushStream();
         if (error.name === "AbortError" || state.controller?.signal.aborted) { pushLine("[Aborted]", C.warn); break; }
-        if (error instanceof ContinueError) { pushLabel("\u276f Continue", ansi.bold + C.warn); pushLine("Ran " + error.turn + " turns (limit " + error.turn + "). Continue?", C.warn); const willContinue = await new Promise(resolve => { state.permission = { name: "continue", args: { turns: error.turn }, resolve }; state.status = "Continue after " + error.turn + " turns?"; render(); }); state.permission = null; if (!willContinue) { pushLine("[Cancelled]", C.warn); break; } pushLine("[Continuing\u2026]", C.tool); state.controller = new AbortController(); continue; }
+        if (error instanceof ContinueError) { pushLabel("\u276f Continue", ansi.bold + C.warn); pushLine("Ran " + error.turn + " turns (limit " + error.turn + "). Continue?", C.warn); const willContinue = await new Promise(resolve => { state.permission = { name: "continue", args: { turns: error.turn }, resolve, reasonMode: false }; state.status = "Continue after " + error.turn + " turns?"; render(); }); state.permission = null; state.permissionPreview = []; if (!willContinue.allowed) { pushLine("[Cancelled]", C.warn); break; } pushLine("[Continuing\u2026]", C.tool); state.controller = new AbortController(); continue; }
         pushLine("[error] " + error.message, C.error); break;
       }
     }
@@ -432,7 +466,7 @@ export async function startTUI(agent, opts = {}) {
     try { saveSession(agent, state.lines); } catch {} render();
   }
   function flushStream() { if (state.reasoning) { pushLine(state.reasoning, C.reason); state.reasoning = ""; } if (state.streaming) { pushLine(state.streaming, C.text); state.streaming = ""; } }
-  function askPermission(name, args) { if (agent.autoApprove) { pushLine("  [auto] " + name + " " + summarize(args), C.warn); return Promise.resolve(true); } state.permissionPreview = formatPermission(name, args); return new Promise(resolve => { state.permission = { name, args, resolve }; state.status = "Waiting: " + name; render(); }); }
+  function askPermission(name, args) { if (agent.autoApprove) { pushLine("  [auto] " + name + " " + summarize(args), C.warn); return Promise.resolve({ allowed: true }); } state.permissionPreview = formatPermission(name, args); return new Promise(resolve => { state.permission = { name, args, resolve, reasonMode: false }; state.status = "Waiting: " + name; render(); }); }
   function askQuestion(text) { if (state.question) return Promise.resolve("(already waiting)"); pushLabel("\u276f Question", ansi.bold + C.tool); for (const line of text.split("\n")) pushLine("  " + line, C.text); return new Promise(resolve => { state.question = { text, options: [], resolve }; state.status = "Waiting..."; render(); }); }
 
   async function handleSlash(text) {
@@ -474,8 +508,8 @@ export async function startTUI(agent, opts = {}) {
           for (const c of candidates) {
             pushLine(`\n  [${c.type}] ${c.title}`, C.text);
             pushLine(`    ${c.content.slice(0, 200)}...`, C.dim);
-            const accept = await askPermission("distill-save", { title: c.title });
-            if (accept) {
+            const result = await askPermission("distill-save", { title: c.title });
+            if (result.allowed) {
               const status = await saveCandidate(agent.memory, c, {});
               pushLine(`  ✓ ${status}`, C.tool);
               saved++;