mindweave 3.4.1 → 3.4.2

package/README.md

@@ -21,8 +21,9 @@ ever go to the model provider you picked.
 Prompts are kept thin on purpose, so the context goes on your code rather than on
 scaffolding.
 
-Prefer a window to a terminal? [mwcode](https://github.com/mindweave-cli/mwcode) is the
-desktop app, built on the same engine, for Windows, macOS and Linux.
+**Names.** Mindweave is the program and the name of this repository and of the npm package (`mindweave`, or `mw` for
+short). [mwcode](https://github.com/mindweave-cli/mwcode) is the desktop app: a window around the same engine, for
+Windows, macOS and Linux, with its own repository and its own download files. The two share sessions and settings.
 
 ## Install
 
@@ -54,7 +55,10 @@ npm link
   Qwen, Kimi, GLM, Meta, MiniMax, Tencent, OpenRouter, and local models through Ollama
   with no key at all. [PROVIDERS.md](src/drivers/PROVIDERS.md)
 - **Real tools** — file reads and edits, search, a shell with background jobs,
-  sub-agents, web search, and a tool that tests your app by using it.
+  sub-agents, web search, and a tool that tests your app by using it: it opens a web page or
+  dev server in a browser, clicks through it and looks at the result. On Windows it can also
+  drive a desktop window and take a screenshot of that one window, never the whole screen. It
+  asks first unless you chose auto-accept.
 - **Code intelligence** — tree-sitter and language servers index the repo in the
   background, at no token cost, and every edit comes back with that file's errors.
 - **Goals that run on their own** — `/marathon` takes a goal and keeps going until it is
@@ -68,6 +72,25 @@ npm link
 `/help` lists every command. `shift+tab` switches modes. Type while it works and your
 message queues.
 
+## Run it from a script
+
+`mindweave -p "task"` does one task with no screen and prints the answer, so it works in a shell script or a CI job.
+Anything piped in is added as context, and `--output-format json` gives one result object with the session, the
+tokens and the cost. The exit code says whether it finished (0), failed (1), had no key (3) or stopped before it was
+done (4). Nobody can answer a question in this mode, so anything that would ask is refused. [docs/HEADLESS.md](docs/HEADLESS.md)
+
+```bash
+git diff | mindweave -p "review this change"
+mindweave -p "list the TODOs" --output-format json | jq -r .result
+```
+
+## Commits
+
+When the agent makes a commit it can add one line to the message, `Co-authored-by: mwcode <...>`, so the commit
+shows the agent beside you on GitHub. It asks once per project the first time, remembers the answer, and never adds
+the line to a commit you make yourself. `/coauthor` shows or changes the answer, and `/coauthor off all` turns it off
+everywhere.
+
 ## Your own rules and hooks
 
 Both live in your state folder (`~/.mindweave`, or `projects/<name>` inside it for one

package/dist/cli/App.js

@@ -29,6 +29,7 @@ import { contextPressure, sharpContextWindow } from "../dynamo/contextWindow.js"
 import { contextBudget, formatBudget } from "../dynamo/contextBudget.js";
 import { contextView, resetContextOverride, setContextOverride } from "../core/contextSettings.js";
 import { createSession, resumeSession } from "../memory/session.js";
+import { coauthorSetting, setCoauthor } from "../tools/coauthor.js";
 import { saveSession, listSessions } from "../memory/store.js";
 import { stopChassis } from "../alternator/lane.js";
 import { loadSkillBody, substituteSkillArgs } from "../governor/skills.js";
@@ -78,6 +79,7 @@ import { applyScreenMode } from "./screenShell.js";
 import { saveScreenMode } from "./screenStore.js";
 import { FULL_TAIL_BLOCKS, needsMeasure, pruneHeights, seedHeights } from "./blockHeights.js";
 import { expandableAt, hitBlock, hitItem, itemHit, revealNode } from "./expandHits.js";
+import { FILE_HIT_BASE, fileRowAt } from "./workLines.js";
 import { isGrouped, nameAt } from "./editGroup.js";
 import { shellNote } from "./shellNotes.js";
 import { pressBackground } from "./commandBackground.js";
@@ -85,7 +87,7 @@ import { readFileSync, statSync } from "node:fs";
 import { compactSearchDetail, compactUiDetail, rebuildFull, recoverReadText, recoverSavedText, searchDetailFromContent, writtenFromDisk } from "../tools/rebuildDetail.js";
 import { isUserShell, runUserCommand } from "../tools/userCommand.js";
 import { isUserCommandText, parseUserCommand } from "../memory/localCommand.js";
-import { shownText } from "../memory/compaction.js";
+import { shownText, splitExpiredImages, userAsSent } from "../memory/compaction.js";
 import { BASE_COMMANDS } from "./commands.js";
 import { attachmentBudget, dismissMarathon } from "../core/turnRunner.js";
 import { describeMarathonEvent, resumeMarathon, startMarathon } from "../dynamo/marathon.js";
@@ -728,8 +730,9 @@ export function App({ resumeSessionId, initialScreen }) {
                 // but they are ours, not the person's. Replaying one draws it as a `>` prompt
                 // the user never typed — seen live as "> That was 3 sentences between tool
                 // calls…" sitting in their own chat history.
+                // The note compaction adds for the model when it drops an image is not something the person typed.
                 if (!e.synthetic)
-                    dispatch({ type: "user", text: collapsePastes(stripAttachments(e.content)) });
+                    dispatch({ type: "user", text: collapsePastes(splitExpiredImages(stripAttachments(userAsSent(e).content)).text) });
                 // A command the person ran with `!` comes back as the row it was: what they ran, how it ended,
                 // and the output behind the click. The screen's record is kept whole even when the model's copy
                 // of the output was cleared to save context.
@@ -2069,7 +2072,19 @@ export function App({ resumeSessionId, initialScreen }) {
             if (block && block.kind === "work" && !block.open && block.items.length === 1)
                 fetchRead(block.items[0]);
         }
-        if (hitItem(hit) >= 0) {
+        if (hitItem(hit) >= FILE_HIT_BASE) {
+            // One file of a read that took several: its text is fetched with the first click on any of them.
+            const st = stateRef.current;
+            const block = [...st.committed, ...st.tail].find((b) => b.id === hitBlock(hit));
+            const at = block && block.kind === "work" ? fileRowAt(block.items, hitItem(hit) - FILE_HIT_BASE) : undefined;
+            if (block && block.kind === "work" && at) {
+                const item = block.items[at.item];
+                if (!item.openFiles?.includes(at.file))
+                    fetchRead(item);
+                dispatch({ type: "toggleFile", id: block.id, item: at.item, file: at.file });
+            }
+        }
+        else if (hitItem(hit) >= 0) {
             // Before the toggle: the state is updated the moment it is dispatched, and a read already open is not fetched for.
             fetchReadText(hitBlock(hit), hitItem(hit));
             dispatch({ type: "toggleItem", id: hitBlock(hit), index: hitItem(hit) });
@@ -3127,12 +3142,14 @@ export function App({ resumeSessionId, initialScreen }) {
     // Hand a free-text directive to the model with an instruction wrapper, as its own
     // turn — used by the manual /rules and /skills commands (the model does the
     // "rewrite it well and save it" work via remember_rule / the skill tool).
-    async function runDirective(instruction, activity) {
+    // `typed` is the command line the person wrote. `instruction` is the engine's wording for the model, which
+    // must never be drawn as if the person had said it, so the message keeps what was typed as its original.
+    async function runDirective(instruction, activity, typed) {
         const s = session.current;
         if (!s)
             return;
         note(activity);
-        s.transcript.push({ role: "user", content: instruction });
+        s.transcript.push({ role: "user", content: instruction, ...(typed ? { original: { content: typed } } : {}) });
         await streamRespond(s);
     }
     async function handleCommand(raw) {
@@ -3308,7 +3325,7 @@ export function App({ resumeSessionId, initialScreen }) {
                 `pull it in with an @-import line rather than making the root file long; if a ` +
                 `convention is true of one folder only, write a MINDWEAVE.md inside that folder ` +
                 `instead, which costs nothing when work is happening elsewhere. ` +
-                `changed. Finish by confirming in one line what you wrote and why.`, "writing MINDWEAVE.md…");
+                `changed. Finish by confirming in one line what you wrote and why.`, "writing MINDWEAVE.md…", raw);
             return;
         }
         if (name === "/compact") {
@@ -3724,7 +3741,7 @@ export function App({ resumeSessionId, initialScreen }) {
             }
             await runDirective(`The user wants this saved as a standing rule for this project. Rewrite it as a clear, ` +
                 `imperative, self-contained rule and save it with remember_rule (add globs only if it ` +
-                `clearly applies to specific files). Then confirm in one line. Directive: "${arg}"`, "saving a rule…");
+                `clearly applies to specific files). Then confirm in one line. Directive: "${arg}"`, "saving a rule…", raw);
             return;
         }
         // /forbidden — list forbidden paths, or (with a path) forbid one outright.
@@ -3756,6 +3773,33 @@ export function App({ resumeSessionId, initialScreen }) {
             }
             return;
         }
+        // /coauthor — whether commits the agent makes carry the "Co-authored-by: mwcode" line.
+        if (name === "/coauthor") {
+            const root = s.governance.forbidden.root;
+            const words = arg.trim().toLowerCase().split(/\s+/).filter(Boolean);
+            const everywhere = words.includes("all");
+            const verb = words.find((w) => w !== "all");
+            if (verb === "on" || verb === "off") {
+                await setCoauthor(root, everywhere ? "global" : "project", verb === "on");
+                const where = everywhere ? "in every project that has no answer of its own" : "in this project";
+                note(verb === "on" ? `commits I make will carry the co-author line ${where}.` : `commits I make will not carry the co-author line ${where}.`);
+                return;
+            }
+            if (verb === "reset") {
+                await setCoauthor(root, "project", null);
+                note("forgot this project's answer. I will ask at the next commit.");
+                return;
+            }
+            if (verb) {
+                say("Use /coauthor to see it, /coauthor on or /coauthor off to set it here, and add 'all' for every project. /coauthor reset forgets this project's answer.");
+                return;
+            }
+            const current = await coauthorSetting(root);
+            say(current
+                ? `The co-author line is ${current.credit ? "on" : "off"}, set for ${current.from === "project" ? "this project" : "all projects"}.`
+                : "Not set yet. I will ask the first time I commit here.");
+            return;
+        }
         // /forbid-command — list forbidden commands, or (with an argument) forbid one.
         if (name === "/forbid-command") {
             if (!arg) {
@@ -3790,7 +3834,7 @@ export function App({ resumeSessionId, initialScreen }) {
             if (arg) {
                 await runDirective(`The user wants a new reusable skill for this project: "${arg}". Design it and save it ` +
                     `with the skill tool — a short invocation name, a one-line description, and the procedure ` +
-                    `as a clear markdown checklist (use $ARGUMENTS/$1 if it should take input). Then confirm.`, "creating a skill…");
+                    `as a clear markdown checklist (use $ARGUMENTS/$1 if it should take input). Then confirm.`, "creating a skill…", raw);
                 return;
             }
             const skills = s.governance.skills;