@nanhara/hara 0.109.3 → 0.109.4

package/CHANGELOG.md

@@ -5,6 +5,16 @@ 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

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.3",
+  "version": "0.109.4",
   "description": "hara — a coding agent CLI that runs like an engineering org.",
   "bin": {
     "hara": "dist/index.js"