mulmoterminal 2.2.0 → 2.4.0

package/README.md

@@ -93,6 +93,13 @@ than as bytes (files within the session's working directory only):
 | source, config, logs, and `.txt` — 46 extensions | the app's own **Files** view (`/files?path=`), where CodeMirror highlights it, the tree is right there, and it can be edited |
 | everything else — images, PDF, SVG, HTML, video | raw bytes in a new tab, which the browser renders better than an editor would |
 
+**While a grid cell is enlarged, the [Files pane](#files-view-browse--edit) takes the click first** — every
+row above except the last one, since the pane is the same editor plus a Markdown preview. The
+file opens *beside* the terminal that printed it, and the pane opens itself if it was closed.
+It declines, leaving the routing above untouched, when nothing is enlarged, when the path is
+not under that cell's own directory (the pane cannot walk above its root), or for the raw-bytes
+row, where it would only show an empty editor.
+
 Highlighting in the Files view covers the JS/TS family, JSON and Markdown (the modes
 `cmEditor.ts` bundles); other languages open as plain text.
 
@@ -223,6 +230,7 @@ The launcher detects it and prints the exact, OS-appropriate removal command; ru
 - [Session model](#session-model)
 - [Session lifecycle](#session-lifecycle)
 - [Claude hook injection](#claude-hook-injection)
+- [Closing summary](#closing-summary)
 - [Session discovery & titles](#session-discovery--titles)
 - [Project structure](#project-structure)
 - [Testing](#testing)
@@ -287,7 +295,8 @@ today — **Claude Code** (the default) and **Codex**.
 - **Claude** — spawned as `claude` (override with `CLAUDE_BIN`). The server passes
   `--session-id <uuid>`, so it knows the live session's id even before its transcript
   file exists, and injects activity hooks + the GUI MCP per spawn (see
-  [Claude hook injection](#claude-hook-injection)).
+  [Claude hook injection](#claude-hook-injection)) plus the
+  [closing summary](#closing-summary) instruction.
 - **Codex** — spawned as `codex` (override with `CODEX_BIN`; `CODEX_MODEL` sets
   `--model`). Codex runs on its own WebSocket (`/ws/codex`) and its sessions appear in the
   sidebar next to Claude's. Because Codex only mints its rollout id **after** the first
@@ -465,6 +474,7 @@ The Settings modal (⚙) persists per-user UI choices to `~/.mulmoterminal/confi
 | `worklogEnabled` | `true` to run the built-in **dev worklog** batch (see below). Off by default (each run spawns an LLM session, so it costs tokens). |
 | `worklogIntervalHours` | Worklog cadence in hours (default `6`, clamped to `1`–`168`). |
 | `terminalSubmit` | Which bytes Claude reads as **submit** vs **newline**: `"cr"` (default — Enter submits, Shift+Enter makes a newline) or `"esc-cr"` (for a Claude Code rebound the other way). Applies to the keyboard **and** the phone remote-view submit, for **Claude sessions only** (shell/codex keep plain Enter). See the [Configuration guide](https://receptron.github.io/mulmoterminal/guide/en/config.html#terminal-submit). |
+| `copyOnSelect` | `true` puts a **mouse selection on the clipboard the moment it settles**, with no key pressed (the PuTTY / iTerm2 behaviour). **Off by default** — it changes the clipboard when you may only have meant to highlight something. No Settings UI: edit the file and reload the tab. Composes with the `copy` keymap action rather than replacing it. Over plain `http://` the browser gives a page no clipboard access, so a fallback asks xterm to copy instead; see the [Configuration guide](https://receptron.github.io/mulmoterminal/guide/en/config.html#copy-on-select). |
 | `prWorkdirFooter` | Ends the body of a PR **⧉ Open PR** creates with `work in <clone>` — the directory name of the clone the work happened in, so a PR says which of several side-by-side checkouts produced it. **On by default**; set `false` to opt out — read from the file per PR, so no restart is needed (there is no Settings control for it). Only applied to PRs this app creates (pressing the button again on an existing PR never re-appends). |
 | `fontFamily` | The **terminal font** every session renders in — a CSS font-family stack, e.g. `"'Cica', 'MS Gothic', monospace"`. No Settings UI: edit the file, then **restart** (this config is read once at startup). Unset uses the built-in stack (JetBrains Mono / Fira Code / Menlo / Consolas, then CJK faces for Japanese, Korean and Chinese). Unlike the per-browser font **size**, this is one value for the whole host — it names fonts, and which fonts exist is a property of the machine. A directory can override it. See the [Configuration guide](https://receptron.github.io/mulmoterminal/guide/en/config.html#font-family). |
 
@@ -617,6 +627,12 @@ as the reload signal, so colors, palette, font size and grid order update withou
 anything. There is no filesystem watcher, so an edit made **outside** a session (your own
 editor) is picked up when the terminal is next opened.
 
+**Checking what took effect.** Settings → **Directory settings** lists your recent directories
+and expands each one to the values in force, with a swatch per color and the path of the file
+they came from. It also names the keys it **dropped** (a color that isn't `#rrggbb`, a size out
+of range) and the keys it doesn't read at all (`badgeColour`, a global-only setting) — which is
+what tells "I never set that" apart from "I set it and it didn't take".
+
 ---
 
 ## Running
@@ -768,6 +784,10 @@ shrinks the pane rather than reflowing xterm into garbage. It works in both zoom
 (cockpit roster and thumbnail filmstrip), the pane re-roots as you walk the zoom between
 terminals, and whether it's open plus how wide it is are remembered per browser.
 
+The toggle is not the only way in: while a cell is enlarged, **clicking a file path the agent
+printed** opens it here too, rather than in a new tab or full-screen — see
+[Clicking a file path](#clicking-a-file-path).
+
 All reads and writes go through `GET/PUT /api/files/browse/*?cwd=&path=`, and every
 `path` is **contained within the project root** (server-side) — `..`/absolute escapes
 are rejected for reads and writes alike, so editing can't reach outside the directory
@@ -920,6 +940,11 @@ Favorited collections get their own toolbar buttons.
   session.
 - **Notifications** (🔔) — a toolbar bell with an unread badge and a dropdown of active
   notifications; click a row to jump to its session.
+- **Star MulmoTerminal** — a star button in the grid toolbar that stars the project on GitHub
+  through your own `gh` login, in one click. It is a one-time ask: once the repo is starred the
+  button is gone for good and stops calling the server at all. It shows **only when `gh` can
+  answer** — with no `gh`, no login, or no network, one click couldn't star anything, so nothing
+  is shown and nothing is recorded. Set `gh` up later and the button appears by itself.
 - **Voice input** — dictate a prompt via on-device Whisper (`POST /api/transcribe`, macOS
   only; the model downloads on first use). Settings picks **the language you dictate in**
   (per browser): your browser's, whisper's own per-clip detection, or a fixed one. Worth
@@ -1102,6 +1127,7 @@ same-origin-guarded.
 | `GET /api/worktrees?cwd=` · `GET /api/worktrees/diff?cwd=` | List managed worktrees / diff one vs its base. |
 | `POST /api/worktrees/create` · `/remove` · `/push` · `/pr` | Create on `agent/<slug>`, remove (managed root only), push, open a PR (`gh`, else compare URL). |
 | `GET /api/prs` · `GET /api/issues` | Open PRs / issues across the configured `prRepos` (via `gh`). |
+| `GET /api/github/star` · `POST /api/github/star` | Whether you have starred MulmoTerminal, and star it (via `gh`). `starred: null` means `gh` could not answer, and hides the button. |
 
 **Workspace views**
 
@@ -1128,6 +1154,7 @@ same-origin-guarded.
 | -------- | ------- |
 | `GET\|POST /api/config` | User UI config (`cwdPresets`, `soundFile`, `soundKinds`, `sounds`, `prRepos`, `launchers`, `quickCommands`, `userMcpServers`, `providers`). |
 | `GET /api/sound?kind=` · `/api/dir-sound?cwd=&kind=` · `/api/sound-preset/:id` · `/api/dir-config?cwd=` | Custom / per-directory / preset attention sound + per-dir config. `kind` selects a config entry, never a path. |
+| `GET /api/dir-config-detail?cwd=` | The same per-dir config, **plus** the settings a running terminal doesn't need (`provider`, `model`, `skills`, `addDirs`, header button/chip **labels**), **plus** which keys the file set and how each fared (applied / dropped in validation / not a setting at all). Read-only; backs the Settings modal's **Directory settings** preview. Unlike the other `?cwd=` routes this one does **not** fall back to the default workspace — it reports on the directory it was asked about, so a path that no longer exists comes back as `exists:false`. Sound paths and button commands stay server-side. |
 | `GET /api/launch-options` | The Anthropic-compatible backends this server can reach, each with its models and — when it can't — the reason. Reports the **name** of the env var a key is read from, never the key. |
 | `GET /api/notifications`(`/history`) · `POST /api/notifications/:id/clear` | Notification feed. |
 | `POST /api/transcribe`(`/model`…) | Voice-input transcription (Whisper, macOS). |
@@ -1328,6 +1355,30 @@ knows the live session's id — even before the session's `.jsonl` file exists.
 
 ---
 
+## Closing summary
+
+Every Claude session is spawned with `claude --append-system-prompt '<text>'`, asking the
+agent to end a reply with a short summary **when it hands control back** — the work is
+finished, or it is stopping to ask a question. Coming back to a grid cell after a while, the
+standing request and what came of it are otherwise only recoverable by scrolling the whole
+session.
+
+The summary states three things: the request **for the conversation as a whole** (not the
+last message — several turns of refinement do not replace what was asked first), what was
+achieved, and what was not and why. It is written in the language of the conversation, and
+placed last with nothing after it.
+
+It is deliberately **not** written on every turn: mid-work replies and short factual answers
+carry no standing request, and a summary that always appears stops being read. The wording
+lives in `server/agents/session-summary-prompt.ts`; there is no setting to turn it off.
+
+Passed inline rather than as `--append-system-prompt-file` for the same reason `--settings`
+is: the sandbox spawn runs in a container that cannot read a host path.
+
+Codex sessions are unaffected — the CLI has no equivalent flag.
+
+---
+
 ## Session discovery & titles
 
 Claude stores each project's sessions as JSONL files under

package/bin/cli-args.d.ts

@@ -10,3 +10,4 @@ export declare const SECOND_INSTANCE_NOTE: string;
 export declare const MIN_NODE_LABEL: string;
 export declare function nodeMeetsMinimum(version: string): boolean;
 export declare function serverNodeArgs(serverEntry: string, launchDir: string): string[];
+export declare function serverSpawnEnv(env: Record<string, string | undefined>, port: number, cwd: string): Record<string, string | undefined>;

package/bin/cli-args.js

@@ -150,3 +150,19 @@ export function nodeMeetsMinimum(version) {
 export function serverNodeArgs(serverEntry, launchDir) {
   return ["--import", "tsx", `--env-file-if-exists=${join(launchDir, ".env")}`, serverEntry];
 }
+
+/**
+ * The environment the launcher spawns the server with.
+ *
+ * Only the two values the server cannot work out for itself. In particular NOT `NODE_ENV`:
+ * the server hands its own environment to every PTY it spawns, so a `NODE_ENV=production`
+ * set here reaches every terminal in every cell — where yarn v1 reads it and installs
+ * WITHOUT devDependencies while still reporting success (#955). Express is pinned to
+ * production separately, in the server, so nothing here needs to say it.
+ *
+ * A `NODE_ENV` the user exported themselves passes through untouched: this decides what the
+ * launcher adds, not what it removes.
+ */
+export function serverSpawnEnv(env, port, cwd) {
+  return { ...env, PORT: String(port), CLAUDE_CWD: cwd };
+}

package/bin/mulmoterminal.js

@@ -26,6 +26,7 @@ import {
   nodeMeetsMinimum,
   MIN_NODE_LABEL,
   serverNodeArgs,
+  serverSpawnEnv,
 } from "./cli-args.js";
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
@@ -272,7 +273,7 @@ function runServer(port, noOpen, cwd, onChild) {
     // package directory, so serverNodeArgs makes that path absolute (#795).
     const server = spawn(process.execPath, serverNodeArgs(SERVER_ENTRY, process.cwd()), {
       cwd: PKG_DIR,
-      env: { ...process.env, NODE_ENV: "production", PORT: String(port), CLAUDE_CWD: cwd },
+      env: serverSpawnEnv(process.env, port, cwd),
       stdio: ["inherit", "inherit", "pipe"],
     });
     let stderrTail = "";

package/common/dirConfigSource.ts

@@ -0,0 +1,101 @@
+// Where a directory's settings actually came from, for the settings modal's preview.
+//
+// The rest of the app only ever needs the RESOLVED config — the values a terminal adopts.
+// That is exactly what makes a mistyped or rejected setting invisible: the file is on disk,
+// the app reads it, and the value silently isn't there. This says which keys the file set,
+// which of them survived validation, and which the app doesn't read at all.
+
+// Every key `loadDirConfig` reads. A key outside this list is a typo as far as the app is
+// concerned, however sensible it looks — `server/config/dir-config.spec.ts` pins the two
+// together so a field added to the loader can't quietly go missing here.
+export const DIR_CONFIG_KEYS = [
+  "name",
+  "badgeColor",
+  "headerColor",
+  "headerTextColor",
+  "cellColor",
+  "cellBorderColor",
+  "dotColor",
+  "buttonColor",
+  "fontSize",
+  "fontFamily",
+  "orderPriority",
+  "theme",
+  "colors",
+  "sound",
+  "sounds",
+  "buttons",
+  "chips",
+  "skills",
+  "provider",
+  "model",
+  "addDirs",
+] as const;
+
+export interface DirConfigSource {
+  // Keys the file set that the app is using.
+  applied: string[];
+  // Keys the file set that the app read and then dropped — an out-of-range number, a colour
+  // that isn't #rrggbb, a sound outside the directory. The value is gone; the key is the clue.
+  ignored: string[];
+  // Keys the app doesn't read at all: a misspelling (`badgeColour`), or a setting that belongs
+  // in the global config instead of a directory's file.
+  unknown: string[];
+}
+
+export const EMPTY_DIR_CONFIG_SOURCE: DirConfigSource = { applied: [], ignored: [], unknown: [] };
+
+// The settings a directory can make that the per-cell config deliberately doesn't carry — the
+// browser has no use for them while running a terminal, but the preview has to SHOW them or a
+// key like `provider` can only be named as applied, never read back.
+//
+// Labels only for buttons and chips: their `cmd` / `text` is what the button would type into
+// the session, which is not what "did my config take effect" needs, and is the sort of thing a
+// screenshot of the settings screen shouldn't carry. Sound paths stay out for the same reason
+// they stay out of the per-cell config — that path is the confinement boundary.
+export interface DirConfigExtras {
+  provider: string | null;
+  model: string | null;
+  skills: string[] | null;
+  addDirs: string[] | null;
+  buttonLabels: string[];
+  chipLabels: string[];
+}
+
+export const EMPTY_DIR_CONFIG_EXTRAS: DirConfigExtras = {
+  provider: null,
+  model: null,
+  skills: null,
+  addDirs: null,
+  buttonLabels: [],
+  chipLabels: [],
+};
+
+// "The loader kept nothing for this key" — null/undefined, but also the empty collections the
+// loader returns when every entry in a map or list was rejected.
+function isEmptyValue(value: unknown): boolean {
+  if (value === null || value === undefined) return true;
+  if (Array.isArray(value)) return value.length === 0;
+  if (typeof value === "object") return Object.keys(value as Record<string, unknown>).length === 0;
+  return false;
+}
+
+// The keys a resolved config actually holds a value for. Taken as a SET by describeDirConfig
+// rather than the config object itself: the loader's type is an interface with named fields,
+// which no amount of `Record<string, unknown>` will accept without a cast.
+export function keysWithValue(resolved: object): Set<string> {
+  const entries: [string, unknown][] = Object.entries(resolved);
+  return new Set(entries.filter(([, value]) => !isEmptyValue(value)).map(([key]) => key));
+}
+
+// `raw` is the file as parsed; `kept` is which keys survived the loader (see keysWithValue).
+export function describeDirConfig(raw: Record<string, unknown>, kept: ReadonlySet<string>): DirConfigSource {
+  const known = new Set<string>(DIR_CONFIG_KEYS);
+  const source: DirConfigSource = { applied: [], ignored: [], unknown: [] };
+  for (const key of Object.keys(raw)) {
+    if (!known.has(key)) source.unknown.push(key);
+    else if (kept.has(key)) source.applied.push(key);
+    else source.ignored.push(key);
+  }
+  return source;
+}

package/common/githubRepo.ts

@@ -0,0 +1,22 @@
+// This project's own GitHub repository, and the wire shape of the star endpoints that act on it.
+// Both sides decide from these — the server builds the `gh api` path and types its response,
+// the client parses that response — so they live here rather than as two copies.
+import { isRecord } from "./isRecord.js";
+
+export const GITHUB_REPO = "receptron/mulmoterminal";
+
+// GitHub's "star a repository" endpoint for this repo: GET reports the current state, PUT stars it.
+export const STAR_API_PATH = `/user/starred/${GITHUB_REPO}`;
+
+// GET/POST /api/github/star. `null` means the server could not tell — no `gh`, not logged in,
+// offline. That is a DIFFERENT answer from "not starred": one click cannot star anything in that
+// state, so the client hides the button rather than offering one that would do nothing.
+export interface GithubStarState {
+  starred: boolean | null;
+}
+
+// Read a star state off an untrusted response body. Anything unparseable reads as "cannot tell",
+// which is the same degradation as a failed request, so the caller needs no separate error path.
+export function parseStarState(value: unknown): boolean | null {
+  return isRecord(value) && typeof value.starred === "boolean" ? value.starred : null;
+}

package/common/terminalClipboard.ts

@@ -50,3 +50,22 @@ export function clipboardActionFor(keymap: Keymap, e: ClipboardKeyEvent, hasSele
   // to be reversed.
   return action === "copy" && !hasSelection ? null : action;
 }
+
+// Copy-on-select: the text a settled selection should put on the clipboard, or null to leave the
+// clipboard alone. `lastCopied` is what this terminal wrote last, so an unchanged selection is not
+// written twice.
+//
+// Unlike the rules above, this one ends in a clipboard WRITE — no keystroke happened, so nothing in
+// the browser was ever going to copy anything by itself. That is what makes the two skips here
+// matter more than they look:
+//
+//   - Whitespace only. Dragging across empty terminal space selects spaces, and silently replacing
+//     the user's clipboard with a run of them is this feature's worst failure. Anyone who really
+//     wants indentation still has the `copy` keymap action.
+//   - Unchanged text. A second identical write buys nothing and costs a duplicate entry in the OS
+//     clipboard history (Win+V), which is the same reason the caller waits for the selection to
+//     settle instead of writing on every onSelectionChange.
+export function selectionToCopy(enabled: boolean, selection: string, lastCopied: string | null): string | null {
+  if (!enabled || selection.trim() === "" || selection === lastCopied) return null;
+  return selection;
+}

package/dist/assets/{abnfDiagram-VRR7QNED-RjiYivmv-6R9yRbUC.js → abnfDiagram-VRR7QNED-RjiYivmv-CqId6L6N.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-CO4zEYzL.js";import"./chunk-WYO6CB5R-BlzOfotS-DmR4SyZL.js";import"./chunk-VAUOI2AC-CS9QJ4yz-Cng5RPUP.js";import{n as i,r as a,t as o}from"./chunk-MOJQB5TN-CMZRaeqt-BsIB0ZXR.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-BzSqp82H.js";import"./chunk-WYO6CB5R-BlzOfotS-CTDodsnw.js";import"./chunk-VAUOI2AC-CS9QJ4yz-DFjCsQc6.js";import{n as i,r as a,t as o}from"./chunk-MOJQB5TN-CMZRaeqt-CYfpp2V7.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-CfmmIOkB.js

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