mulmoterminal 2.3.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.
 
@@ -620,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
@@ -771,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
@@ -1137,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). |

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/dist/assets/{abnfDiagram-VRR7QNED-RjiYivmv-B8nu2pUR.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-BILJo_fK.js";import"./chunk-WYO6CB5R-BlzOfotS-04XZ1Jfd.js";import"./chunk-VAUOI2AC-CS9QJ4yz-Dd_VpRsu.js";import{n as i,r as a,t as o}from"./chunk-MOJQB5TN-CMZRaeqt-Dy6S_sS_.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};