mulmoterminal 2.8.0 → 2.9.0

package/README.md

@@ -74,15 +74,19 @@ npx mulmoterminal@latest        # starts on http://localhost:34567 and opens you
 
 *Zoomed in, the **cockpit roster** replaces thumbnails with information: every session as a
 text row — directory, AI summary, your last prompt, the agent's latest reply, a status word,
-and the branch's **PR phase** badge. Click a row to swap the enlarged terminal.*
+and the branch's **PR phase** badge. A row whose agent is **waiting on you** rings amber and
+blinks; one that has merely **finished** rings green and stays still (Settings → Waiting rows
+turns the movement off). Click a row to swap the enlarged terminal.*
 
 ### What it is, under the hood
 
 Each session runs as a real PTY on the server (the agent CLI in a pseudo-terminal) and is
 streamed to an [xterm.js](https://xtermjs.org/) terminal in the browser over a WebSocket. A
 sidebar lists every session for the project and reflects, in real time, which are **working**
-(the agent is thinking) and which **need attention** (waiting for input, or finished with
-output you haven't seen) — driven by Claude/Codex activity hooks the server injects per spawn.
+(the agent is thinking, a spinner), which are **waiting on you** (a permission prompt or a
+question — an amber dot; nothing proceeds until you answer) and which are **finished with output
+you haven't seen** (a green dot) — driven by Claude/Codex activity hooks the server injects per
+spawn. The horizontal tab bar carries the same two dots.
 
 ![Single view — one agent in focus, terminal on the left and a GUI panel on the right](https://raw.githubusercontent.com/receptron/mulmoterminal/main/docs/guide/images/single-view.png)
 
@@ -392,7 +396,9 @@ today — **Claude Code** (the default), **Codex**, and **Antigravity** (`agy`).
   `--model`). Antigravity runs on its own WebSocket (`/ws/antigravity`). Like Codex it mints its
   own conversation id, so the server watches `~/.gemini/antigravity-cli/brain/` (home overridable
   via `ANTIGRAVITY_HOME`) for the directory the new conversation creates — attributed only when
-  unambiguous — and cold-resumes it with `--conversation <id>`.
+  unambiguous — and cold-resumes it with `--conversation <id>`. That mapping is appended to
+  `~/.mulmoterminal/antigravity-conversations.jsonl`, so a conversation is still resumable after
+  the server restarts.
 
   Its **GUI tools work differently**, because `agy` takes no MCP flag: it reads its servers from
   `.agents/mcp_config.json` in the working directory. MulmoTerminal writes that file from the
@@ -1028,8 +1034,8 @@ your **own HTTP MCP servers** into the single-view session via Settings → `use
 **Wiki.** The toolbar **Wiki** button opens a read-only browser over `<workspace>/data/wiki/`
 — an **index** (tag-filterable page catalog), rendered **pages** with `[[wiki links]]` and
 backlinks, a **graph** view (pages ranked by references), and a **lint** report (orphans /
-broken links / tag drift). Read-only endpoints: `GET /api/wiki`, `/api/wiki/graph`,
-`/api/wiki/lint`.
+broken links / tag drift) whose `[[links]]` are clickable too. Read-only endpoints:
+`GET /api/wiki`, `/api/wiki/graph`, `/api/wiki/lint`.
 
 **Collections.** The toolbar **Collections** button browses the workspace's collection
 "cards" (`@mulmoclaude/collection-plugin`). Running a collection **action** fetches a seed
@@ -1136,13 +1142,32 @@ newest first, including freshly-created sessions that aren't yet written to disk
 - `500 { "error": string }` on an unexpected filesystem error. A missing project
   directory is **not** an error — it yields an empty `sessions` array.
 
+### Directories that cannot be used
+
+Every `?cwd=` — on the terminal sockets and on the read routes alike — names the directory
+the request is about. When one is named and cannot be used, the server says so instead of
+quietly answering about the **default workspace** (#1151):
+
+| Where | What happens |
+| --- | --- |
+| `/ws`, `/ws/codex`, `/ws/antigravity`, `/ws/launch`, `/ws/run` | The socket is closed with `{ type: "error", message }`, which the terminal shows as a red banner and does not retry. |
+| A session that is still running (`?session=` names a live PTY or a surviving tmux session) | **Attaches anyway**, with a warning in the server log. Moving or renaming a directory must not shut you out of an agent that is still working in it — and the cwd reported back comes from the running PTY, not from the request. |
+| `GET /api/scripts`, `/api/skills`, `/api/dir-config`, `/api/dir-sound`, `/api/git-status`, `/api/pr-phase`, `/api/header`, `/api/sessions`, `/api/codex/sessions`, `/api/session/:id`, `/api/transcript/*`, `/api/cost` | `404 { error, cwd }` — a directory that is not there. |
+| A `?cwd=` that cannot name a directory at all (relative, or repeated as `?cwd=a&cwd=b`) | `400 { error, cwd }`. |
+
+A request that names **no** directory is unaffected: `CLAUDE_CWD` is then the answer it
+asked for. The wording of the refusal is the one a refused spawn already uses, so the same
+condition reads the same whether it is caught here or by `ptySpawn` itself.
+
 ### HTTP: `GET /api/scripts`
 
 The runnable entries from `<cwd>/script.json` for a cell's chosen directory
-(`?cwd=<dir>`, falling back to `CLAUDE_CWD`); see
-[Scripts (Run menu)](#scripts-run-menu). The resolved `cwd` is echoed back (the
-server may fall back from a bad path), and each entry carries its `index` (the
-position the client sends back to `/ws/run`).
+(`?cwd=<dir>`, or `CLAUDE_CWD` when none is named); see
+[Scripts (Run menu)](#scripts-run-menu). The resolved `cwd` is echoed back, and each
+entry carries its `index` (the position the client sends back to `/ws/run`). A `?cwd=`
+that names a directory the server cannot enter is answered `404 { error, cwd }` rather
+than with the default workspace's scripts — see
+[Directories that cannot be used](#directories-that-cannot-be-used).
 
 ```jsonc
 // GET /api/scripts?cwd=/Users/me/proj
@@ -1160,8 +1185,8 @@ A missing or invalid `script.json` is **not** an error — it yields an empty
 
 ### HTTP: `GET /api/skills`
 
-The Claude skills discoverable for a terminal's chosen directory (`?cwd=<dir>`,
-falling back to `CLAUDE_CWD`) — project scope (`<cwd>/.claude/skills`) plus user scope
+The Claude skills discoverable for a terminal's chosen directory (`?cwd=<dir>`, or
+`CLAUDE_CWD` when none is named) — project scope (`<cwd>/.claude/skills`) plus user scope
 (`~/.claude/skills`), deduped by slug (project shadows user), **working-dir skills
 first**; see [Skills (Skill menu)](#skills-skill-menu). A `skills` allowlist in that
 directory's `.mulmoterminal.json` narrows and reorders the result; absent → all. The
@@ -1628,13 +1653,19 @@ src/
     Terminal.vue                             xterm.js terminal; /ws, /ws/codex, /ws/run
     AppToolbar.vue                           shared header + toolbar buttons
     GridView.vue, TerminalGrid.vue, TerminalCell.vue, CommandCell.vue, LauncherCell.vue
+    CellLaunchForm.vue                       what an EMPTY cell shows: dir + target + resume /
+                                             scripts / worktrees / tool groups
     GuiPanel.vue, PluginFrame.vue            GUI panel (Canvas) + Shadow-DOM plugin host
     FilesOverlay.vue                         file browser + CodeMirror editor
     GitBranchChip.vue, ModelContextBadge.vue header chips / badges
     PrsOverlay.vue                           cross-repo PRs & Issues
     Wiki*View.vue, Collections*.vue, AccountingOverlay.vue   workspace views
     TimelineOverlay.vue, ToolsPane.vue, NotificationBell.vue, RemoteHostControl.vue
-    SettingsModal.vue                        ⚙ settings
+    SettingsModal.vue                        ⚙ settings — the dialog shell + section order
+    settings/                                one file per settings section (theme, sounds,
+                                             web push, google, PR repos, launchers, quick
+                                             commands, MCP, cost, shortcuts, …), plus the
+                                             shared SettingsStepper / SettingsListRow
   composables/                  useSessions, usePubSub, useGitStatus, useCost,
                                 useChatLauncher, useFilesView, useWikiBrowse,
                                 useCollectionBrowse, useNotifications, useVoiceInput, …

package/common/fileWriteChannel.ts

@@ -1,3 +1,5 @@
+import { isRecord } from "./isRecord.js";
+
 // The pub/sub channel carrying "an agent just wrote this file". Both sides decide from it —
 // the server publishes, the editor listens to know its open file moved under it — so the name
 // and the payload shape live here rather than as a string literal on each side.
@@ -7,3 +9,7 @@ export const FILE_WRITE_CHANNEL = "file-write";
 export interface FileWriteEvent {
   file: string;
 }
+
+/** Both sides go through this rather than re-spelling `{ file: string }`: the publisher pins its
+ *  payload with `satisfies`, the subscriber narrows an untyped pub/sub frame with it. */
+export const isFileWriteEvent = (data: unknown): data is FileWriteEvent => isRecord(data) && typeof data.file === "string";

package/common/sessionAgent.ts

@@ -18,12 +18,25 @@ export type TerminalAgent = (typeof TERMINAL_AGENTS)[number];
 // persisted cell (written before the field existed) means.
 export const asTerminalAgent = (value: unknown): TerminalAgent => (TERMINAL_AGENTS.some((agent) => agent === value) ? (value as TerminalAgent) : "claude");
 
-// A cell / connection / header button carries its agent as booleans, because Claude is the
-// default and the default is the absence of a flag. One function turns them back into the name,
-// so the precedence lives in one place rather than in a ternary repeated at every call site —
-// and adding a fourth agent is one edit here.
-export function terminalAgent(flags: { codex?: boolean | null; antigravity?: boolean | null }): TerminalAgent {
-  if (flags.antigravity) return "antigravity";
-  if (flags.codex) return "codex";
-  return "claude";
+export interface AgentBadge {
+  /** Where there is room for it — a sidebar row, a header bar. */
+  full: string;
+  /** For the tab bar, which fits about two characters between the dot and the title. */
+  short: string;
+}
+
+// Claude has no badge on purpose: it is the default everywhere, so badging it would put one on
+// nearly every row and stop the badge meaning "this one is different". A shell is not an agent
+// and reaches this with no entry, which is the same answer.
+const AGENT_BADGES: Partial<Record<TerminalAgent, AgentBadge>> = {
+  codex: { full: "codex", short: "cx" },
+  antigravity: { full: "agy", short: "agy" },
+};
+
+/** How a session row announces which agent it is running, or null when it wears no badge. The one
+ *  place the wording lives: the sidebar and the tab bar show the SAME list, so a literal compared
+ *  in each is how the two came to disagree about a third agent. */
+export function agentBadge(agent: string | null | undefined): AgentBadge | null {
+  const known = TERMINAL_AGENTS.find((candidate) => candidate === agent);
+  return known ? (AGENT_BADGES[known] ?? null) : null;
 }

package/common/terminalSubmit.ts

@@ -30,6 +30,35 @@ export const newlineSequence = (mode: TerminalSubmitMode): string => (mode === "
 // where ESC+CR is Alt+Enter, not submit.
 export const submitSequenceForAgent = (agent: string | undefined, mode: TerminalSubmitMode): string => (agent === "claude" ? submitSequence(mode) : CR);
 
+// Text we are about to submit ON the user's behalf, ended so Claude Code has no completion menu
+// open when the submit byte(s) arrive.
+//
+// Claude Code holds a completion menu open while the cursor sits at the end of a completion token
+// — a `/command` or an `@path` — and an open menu consumes the key that should have submitted.
+// Measured against Claude Code 2.1.220:
+//
+//   `/help` + CR        submits          (the one case that always worked)
+//   `/help` + ESC CR    does NOT submit  — the menu's own `escape` binding eats the ESC, so the
+//                                          "esc-cr" submit never lands, however often it is sent
+//   `@path` + CR        does NOT submit  — the file picker takes that CR for itself
+//   either + a trailing space            submits: the space ends the token, closing the menu
+//
+// So this is NOT specific to the reversed submit mapping (#1142 was reported there, but the
+// `@path` half hits the default mapping too), and it is deliberately not a list of Claude Code's
+// trigger characters: a trigger we failed to enumerate would silently restore a dead end that
+// looks like a broken feature.
+//
+// For auto-submitted text only. A draft the user still edits keeps exactly what was handed over —
+// their own Enter is a keystroke they can retry once they see the menu.
+export const submittableLine = (text: string): string => (/\S$/.test(text) ? `${text} ` : text);
+
+// The same Claude-only scoping as submitSequenceForAgent, and for a sharper reason: the completion
+// menu is Claude Code's behaviour, while for every other target the trailing space would be REAL
+// input. A shell line ending in `\` escapes the newline and waits for more (`echo foo\` → a
+// continuation prompt); with a space appended it is an escaped space and runs instead. So a
+// shell/codex/command session's bytes stay exactly what the sender wrote.
+export const submittableLineForAgent = (agent: string | undefined, text: string): string => (agent === "claude" ? submittableLine(text) : text);
+
 // The structural shape of a keydown the override needs. A real DOM KeyboardEvent satisfies
 // it, and so does a plain test object — no DOM dependency, so this stays testable and shared.
 export interface EnterKeyEvent {

package/dist/assets/{abnfDiagram-VRR7QNED-RjiYivmv-rGHffQn8.js → abnfDiagram-VRR7QNED-RjiYivmv-FRF8kLeT.js}

@@ -1 +1 @@
-import{n as e}from"./chunk-Y2CYZVJY-Bdt8pFDJ-DsF7k-Jl.js";import{f as t}from"./src-Brzfja-q-ch1VcTiG.js";import{_ as n,g as r}from"./mermaid-parser.core-DxEa8E3F-DteVxl8X.js";import"./chunk-WYO6CB5R-BlzOfotS-B9k6Do_6.js";import"./chunk-VAUOI2AC-CS9QJ4yz-FXdeVJH3.js";import{n as i,r as a,t as o}from"./chunk-MOJQB5TN-CMZRaeqt-LKJvD71K.js";import{t as s}from"./chunk-JWPE2WC7-BQ3zXr2k-DSd3Ct95.js";var c=r().RailroadAbnf.parser.LangiumParser,l=e(e=>{let t=e.alternatives.map(u);return t.length===1?t[0]:{type:`choice`,alternatives:t}},`transformAlternation`),u=e(e=>{let t=e.elements.map(f);return t.length===1?t[0]:{type:`sequence`,elements:t}},`transformConcatenation`),d=e(e=>{if(e.includes(`*`)){let[t,n]=e.split(`*`);return{min:t?parseInt(t,10):0,max:n?parseInt(n,10):1/0}}let t=parseInt(e,10);return{min:t,max:t}},`parseRepeat`),f=e(e=>{let t=p(e.primary);if(!e.repeat)return t;let{min:n,max:r}=d(e.repeat);return n===0&&r===1?{type:`optional`,element:t}:{type:`repetition`,element:t,min:n,max:r}},`transformElement`),p=e(e=>{switch(e.$type){case`AbnfStringLiteral`:return{type:`terminal`,value:e.value};case`AbnfNumVal`:return{type:`terminal`,value:e.value};case`AbnfRuleName`:return{type:`nonterminal`,name:e.name};case`AbnfGroup`:return l(e.element);case`AbnfOptionalGroup`:return{type:`optional`,element:l(e.element)};default:throw Error(`Unsupported ABNF primary node: ${e.$type}`)}},`transformPrimary`),m=e(e=>({name:e.name,definition:l(e.definition)}),`transformRule`),h=e(e=>{s(e,a),e.title&&a.setTitle(e.title),e.rules.map(e=>a.addRule(m(e)))},`populateDb`),g={parser:{parse:e(e=>{a.clear(),t.debug(`[ABNF Parser] Starting Langium parse`);let r=c.parse(e);if(r.lexerErrors.length>0||r.parserErrors.length>0)throw new n(r);let i=r.value;t.debug(`[ABNF Parser] Parsed rules:`,i.rules.length),h(i),t.debug(`[ABNF Parser] Parse complete`)},`parse`),parser:{yy:a}},db:a,renderer:o,styles:i};export{g as diagram};
+import{n as e}from"./chunk-Y2CYZVJY-Bdt8pFDJ-DsF7k-Jl.js";import{f as t}from"./src-Brzfja-q-ch1VcTiG.js";import{_ as n,g as r}from"./mermaid-parser.core-DxEa8E3F-C7AoyBVj.js";import"./chunk-WYO6CB5R-BlzOfotS-Bx9N57Se.js";import"./chunk-VAUOI2AC-CS9QJ4yz-BQv37MCq.js";import{n as i,r as a,t as o}from"./chunk-MOJQB5TN-CMZRaeqt-BPbi0qBZ.js";import{t as s}from"./chunk-JWPE2WC7-BQ3zXr2k-DSd3Ct95.js";var c=r().RailroadAbnf.parser.LangiumParser,l=e(e=>{let t=e.alternatives.map(u);return t.length===1?t[0]:{type:`choice`,alternatives:t}},`transformAlternation`),u=e(e=>{let t=e.elements.map(f);return t.length===1?t[0]:{type:`sequence`,elements:t}},`transformConcatenation`),d=e(e=>{if(e.includes(`*`)){let[t,n]=e.split(`*`);return{min:t?parseInt(t,10):0,max:n?parseInt(n,10):1/0}}let t=parseInt(e,10);return{min:t,max:t}},`parseRepeat`),f=e(e=>{let t=p(e.primary);if(!e.repeat)return t;let{min:n,max:r}=d(e.repeat);return n===0&&r===1?{type:`optional`,element:t}:{type:`repetition`,element:t,min:n,max:r}},`transformElement`),p=e(e=>{switch(e.$type){case`AbnfStringLiteral`:return{type:`terminal`,value:e.value};case`AbnfNumVal`:return{type:`terminal`,value:e.value};case`AbnfRuleName`:return{type:`nonterminal`,name:e.name};case`AbnfGroup`:return l(e.element);case`AbnfOptionalGroup`:return{type:`optional`,element:l(e.element)};default:throw Error(`Unsupported ABNF primary node: ${e.$type}`)}},`transformPrimary`),m=e(e=>({name:e.name,definition:l(e.definition)}),`transformRule`),h=e(e=>{s(e,a),e.title&&a.setTitle(e.title),e.rules.map(e=>a.addRule(m(e)))},`populateDb`),g={parser:{parse:e(e=>{a.clear(),t.debug(`[ABNF Parser] Starting Langium parse`);let r=c.parse(e);if(r.lexerErrors.length>0||r.parserErrors.length>0)throw new n(r);let i=r.value;t.debug(`[ABNF Parser] Parsed rules:`,i.rules.length),h(i),t.debug(`[ABNF Parser] Parse complete`)},`parse`),parser:{yy:a}},db:a,renderer:o,styles:i};export{g as diagram};

package/dist/assets/architecture-TIHT7OUA-0NsD2xW1-DrblApEd.js

@@ -0,0 +1 @@
+import{f as e}from"./mermaid-parser.core-DxEa8E3F-C7AoyBVj.js";export{e as createArchitectureServices};