@nanhara/hara 0.109.2 → 0.109.4

package/CHANGELOG.md

@@ -5,6 +5,29 @@ All notable changes to `@nanhara/hara`.
 > Versioning (pre-1.0, SemVer-style): the **minor** (middle) number bumps for a **new feature**; the
 > **patch** (last) number bumps for **optimizations/fixes of existing features**.
 
+## 0.109.4 — a dropped file path is read, not "Unknown command"
+
+- **Dragging/pasting a file into the prompt no longer errors.** A dropped file pastes as an absolute path
+  (`/Users/…/spec.md`, often with trailing text or `[Image #N]` tokens). Because it starts with `/`, hara
+  treated it as a slash command and replied `Unknown command /Users/…`. Now the command parser only fires
+  when the first token has **no embedded slash** (real commands — `/help`, `/design` — never contain one),
+  so a path falls through to the normal turn. And when a message *begins* with an existing absolute path,
+  that path is rewritten to an `@`-mention so its content is **read into the turn** — i.e. "interpret this
+  file" just works. Fixed in both the TUI and the readline REPL.
+
+## 0.109.3 — an empty model response no longer looks like a hang
+
+- **A turn that comes back with no text and no tool calls is no longer silently dropped.** The agent
+  loop treated an empty completion as "done" and returned with zero feedback — so after e.g. a "继续"
+  the box just sat there, idle, looking frozen for hours (CPU 0%, waiting at the prompt) when really
+  the turn had vanished. It now **retries once** with a nudge (an empty response is usually a transient
+  hiccup), and if it's still empty says so plainly (`✻ empty response — nothing to do. Rephrase…`)
+  instead of disappearing. Matches how Claude Code / codex refuse to end on a blank turn.
+- **Closed a matching spin:** a `tool_use` stop reason carrying an *empty* tool list used to push an
+  empty tool round and re-request in a loop; it's now bounded by the same single-retry guard. (The
+  120s stall-watchdog already covered a genuinely dead/silent socket — this covers the case where the
+  request *succeeds* but returns nothing.)
+
 ## 0.109.2 — long paste no longer freezes the input box
 
 - **The input box now draws a bottom-anchored viewport, not every wrapped row.** A long multi-line

package/dist/agent/loop.js

@@ -97,6 +97,7 @@ export async function runAgent(history, opts) {
     const permRules = loadPermissionRules(ctx.cwd); // command-level allow/ask/deny policy for the bash tool
     let activeProvider = provider; // may switch to a fallback model on a recoverable error (app-failover)
     let triedFallback = false;
+    let emptyRetried = false; // one-shot: a genuinely empty model turn gets a single nudge before we give up
     // Stuck/loop guard — only in headless chat (`hara gateway`), where a wrong approach can grind forever with
     // nobody to hit Esc (e.g. screenshots it can't read). Once per run, when the agent keeps repeating one
     // non-read tool or acting blind, we inject a reflection nudge so it steps back instead of spinning.
@@ -281,7 +282,37 @@ export async function runAgent(history, opts) {
             }
             return;
         }
-        if (r.stop !== "tool_use")
+        // Empty-turn guard. The model returned nothing actionable — no text AND no tool calls (a blank
+        // completion, or a "tool_use" stop with an empty tool list). Silently returning here leaves the
+        // user at a dead prompt with ZERO feedback: it reads as a 15-hour hang when really the turn just
+        // vanished. Retry ONCE with a nudge (usually a transient hiccup), then, if still empty, say so
+        // plainly and end — never loop forever, never disappear. (Claude Code / codex both guard this.)
+        if (!r.text.trim() && r.toolUses.length === 0) {
+            if (!emptyRetried) {
+                emptyRetried = true;
+                history.pop(); // drop the empty assistant turn before re-asking
+                history.push({ role: "user", content: "(Your previous response was empty. Continue the task now — take the next concrete step with a tool, or reply with text. Do not return an empty response.)" });
+                if (!opts.quiet) {
+                    const note = "✻ empty response — retrying once…";
+                    if (sink)
+                        sink.notice(note);
+                    else
+                        out(c.dim(`${note}\n`));
+                }
+                continue;
+            }
+            const note = "✻ the model returned an empty response — nothing to do. Rephrase your request, or press Enter to try again.";
+            if (!opts.quiet) {
+                if (sink)
+                    sink.notice(note);
+                else
+                    out(c.dim(`${note}\n`));
+            }
+            return;
+        }
+        // A "tool_use" stop with text but no tools (rare) has nothing to execute — end after showing the text
+        // rather than pushing an empty tool round and re-requesting in a loop.
+        if (r.stop !== "tool_use" || r.toolUses.length === 0)
             return;
         const plans = [];
         // Extra (per-run) tools win over the registry so a run-scoped tool can't be shadowed by a global one.

package/dist/context/mentions.js

@@ -8,6 +8,23 @@ import { mediaTypeFor } from "../images.js";
 const MAX_FILE = 50_000;
 // @ at start-of-string or after whitespace; capture a path with no spaces/@ (avoids emails like a@b.com)
 const MENTION_RE = /(?:^|\s)@([^\s@]+)/g;
+/** A submitted line is a slash COMMAND only if it starts with '/' AND its first whitespace-delimited
+ *  token has no *embedded* '/'. A dropped/pasted absolute file path (`/Users/…/doc.md`, possibly followed
+ *  by text or `[Image #N]` tokens) has an embedded slash → it is NOT a command. Without this, dragging a
+ *  file into the prompt errored with "Unknown command /Users/…". Exported pure for tests. */
+export function isSlashCommand(line) {
+    return line.startsWith("/") && !line.slice(1).split(/\s+/)[0].includes("/");
+}
+/** If a message BEGINS with an absolute file path (a dragged/pasted file), rewrite that leading path into
+ *  an `@`-mention so expandMentions inlines the file's content into the turn — the "read/interpret this
+ *  file" the user meant. Space-free leading paths only (a mention can't contain spaces); a non-existent or
+ *  spaced path is left untouched. `exists` is injected so this stays pure/testable. */
+export function inlineLeadingPath(line, exists) {
+    if (!line.startsWith("/"))
+        return line;
+    const first = line.split(/\s+/)[0];
+    return exists(first) ? "@" + line : line;
+}
 /** Expand `@path` references **in place** — the file/dir content lands where it's referenced, not
  *  dumped at the bottom (so "compare @a.ts with @b.ts" reads in context). A repeated mention keeps
  *  the bare `@path` the second time (no double-inlining), and a non-readable ref is left untouched. */

package/dist/index.js

@@ -47,7 +47,7 @@ import { loadAgentsMd, hasAgentsMd, INIT_PROMPT, findProjectRoot } from "./conte
 import { getEmbedder } from "./search/embed.js";
 import { collectRepoChunks, collectDirChunks, buildIndex, indexPath, indexExists } from "./search/semindex.js";
 import { searchHybrid } from "./search/hybrid.js";
-import { expandMentions, fileCandidates } from "./context/mentions.js";
+import { expandMentions, fileCandidates, isSlashCommand, inlineLeadingPath } from "./context/mentions.js";
 import { newSessionId, shortId, resolveSessionId, saveSession, loadSession, listSessions, latestForCwd, titleFrom, slugify, } from "./session/store.js";
 import { setSessionForceModel, isSessionForceModel, effectiveRoleModel } from "./session/session-model.js";
 import { loadRoles, scaffoldRoles, subagentToolFilter } from "./org/roles.js";
@@ -2814,7 +2814,9 @@ program.action(async (opts) => {
             onClipboardImage: readClipboardImage,
             vim: cfg.vimMode,
             onSubmit: async (line, h, images) => {
-                if (line.startsWith("/")) {
+                // A dropped/pasted file path (`/Users/…/doc.md`, maybe trailing text/images) starts with '/' but
+                // is NOT a command — treat it as a file to read, not "Unknown command" (see isSlashCommand).
+                if (isSlashCommand(line)) {
                     const [nm, ...rest] = line.slice(1).split(/\s+/);
                     const arg = rest.join(" ").trim();
                     if (nm === "exit" || nm === "quit") {
@@ -3074,6 +3076,9 @@ program.action(async (opts) => {
                     const near = nearest(nm, [...byName.keys()]);
                     return void h.sink.notice(`Unknown command /${nm}.${near.length ? " Did you mean " + near.map((n) => "/" + n).join(", ") + "?" : ""}`);
                 }
+                // A message that begins with an absolute file path (the case skipped above) → inline it as an
+                // @-mention so its content is read into the turn (drag-a-file-in → interpret it).
+                line = inlineLeadingPath(line, existsSync);
                 const ui = { text: h.sink.assistantDelta, reasoning: h.sink.reasoningDelta, tool: h.sink.tool, diff: h.sink.diff, notice: h.sink.notice };
                 const appr = h.approval;
                 // Type-ahead steering: fold messages typed mid-turn into the next model call (codex-style) so a
@@ -3237,7 +3242,8 @@ program.action(async (opts) => {
         bar.renderBottom(); // bottom border + modes/usage
         if (!line)
             continue;
-        if (line.startsWith("/")) {
+        // A dropped/pasted file path starts with '/' but isn't a command — read it, don't error (see TUI path).
+        if (isSlashCommand(line)) {
             const [name, ...rest] = line.slice(1).split(/\s+/);
             const cmd = byName.get(name);
             if (!cmd) {
@@ -3275,6 +3281,7 @@ program.action(async (opts) => {
                 break;
             continue;
         }
+        line = inlineLeadingPath(line, existsSync); // leading dropped file path → @-mention so it's read in
         const userContent = (recalledContext ? `${recalledContext}\n\n---\n\n` : "") + expandMentions(line, cwd);
         recalledContext = "";
         history.push({ role: "user", content: userContent });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nanhara/hara",
-  "version": "0.109.2",
+  "version": "0.109.4",
   "description": "hara — a coding agent CLI that runs like an engineering org.",
   "bin": {
     "hara": "dist/index.js"