mulmoterminal 2.9.0 → 3.0.0

package/README.md

@@ -585,6 +585,7 @@ The Settings modal (⚙) persists per-user UI choices to `~/.mulmoterminal/confi
 | `soundKinds` | Which moments beep — see [Notification sounds](#notification-sounds). Defaults to `["finished","waiting"]`; the other kinds are opt-in. |
 | `sounds`     | Per-kind sound: `{ "waiting": "preset:coin" }`. A `preset:<id>` reference or an absolute path; a kind with no entry falls back to `soundFile`. |
 | `prRepos`    | `owner/repo` entries whose open PRs/issues the cross-repo **PRs & Issues** view aggregates (via your `gh` login). |
+| `repoDirs`   | `{ "owner/repo": "/abs/path" }` — which local clone work on a repo starts in, when you keep several side by side. Only the *choice* is stored; which clones exist is re-derived from `cwdPresets` on every read, and an entry that no longer names a clone of that repo is ignored. |
 | `launchers`  | `{ label, command }` entries offered in a grid cell's launcher besides the agents — any interactive command. A plain shell needs no entry: the launch form's **Shell** toggle opens `$SHELL` unconfigured. |
 | `quickCommands` | `{ label, text, agents? }` phrases the **phone** offers as chips on a session's terminal view. Tapping one puts `text` in the input box; it is not sent until you press send. `agents` (`"claude"` / `"codex"` / `"shell"`) scopes a chip to session kinds — omit it to offer the chip everywhere. Empty by default. |
 | `userMcpServers` | `{ id, url }` HTTP MCP servers merged into the **single-view** Claude session's `--mcp-config` (a `localhost` URL is reached over `host.docker.internal` in the Docker sandbox). Takes effect on the next session. |
@@ -958,6 +959,18 @@ separate working tree that shares the repo's `.git`, so several agents can work
 repo without colliding. Worktrees live under `~/.mulmoterminal/worktrees/` (override with
 `MULMOTERMINAL_HOME`), and existing ones are listed for reuse.
 
+A worktree started **from an issue** gets an `issue/<N>-<slug>` branch instead. The number
+in the name is what later tells the app which issue the work belongs to: the ⧉ Open PR
+button puts `Fixes #<N>` in the PR body, and the branch chip, the issue work comment and
+the merge-time auto-close all read the same number rather than guessing at it.
+
+That path also **fetches first and forks from `origin/<base>`**, because several clones of
+one repo often run side by side and only the one being worked in gets pulled — forking from
+the local branch would start the work on however old that clone happens to be. A local base
+that already contains the remote wins anyway (it is a superset, so nothing is lost), and
+with no remote reachable the local branch is used and the worktree is still created.
+Typing a task name yourself keeps the local base it has always used, with no fetch.
+
 ![An empty cell's launch form — choose the agent, working directory, or a worktree](https://raw.githubusercontent.com/receptron/mulmoterminal/main/docs/guide/images/grid-launch-form.png)
 
 *Every empty grid cell shows this launch form: toggle **Claude / Codex / Antigravity / Shell**, type a **working directory** (frequent ones autocomplete from your presets), or — in a git repo — name a task under **OR ISOLATE IN A WORKTREE** and hit **＋ New worktree** to start the agent on its own isolated branch. **Shell** runs your OS default shell there instead of an agent; **OR LAUNCH** runs one of your configured launch commands.*
@@ -980,6 +993,24 @@ PRs show a CI-rollup / review-decision / draft badge; each repo lists its latest
 issues. Rows are real links, per-repo errors don't sink the view, and the two lists load
 independently. Backed by `GET /api/prs` and `GET /api/issues`.
 
+**Starting work from an issue row.** Each issue row carries a **▶** button that does the setup in
+one click: read the issue, cut an `issue/<number>-<slug>` worktree in your clone of that repo, and
+open Claude there as a grid cell with the issue **typed into its input box but not sent**. The
+prompt is seeded server-side as a *draft* (`server/session/draft-injection.ts`), which waits for
+claude's input box to be ready — text pushed in before that lands in the scrollback instead. A repo
+with several clones asks which one the first time and remembers the answer; a repo with no clone
+here disables the button and says why. Backed by `POST /api/issues/start`.
+
+**Which clone a repo's work happens in.** `GET /api/repo-dirs` answers the reverse of the
+GitHub link a cell already shows: given `owner/repo`, which of your saved directories are
+clones of it. The candidates are derived from your directory presets by reading each one's
+`origin` — there is no second list to keep in step — and are ordered by each directory's
+`orderPriority`, then by path. Several clones of one repo commonly run side by side, so the
+answer is a choice rather than a lookup; once you make it, `repoDirs` in the config records
+`owner/repo` → the chosen path and it is used from then on. A recording is dropped if the
+directory is no longer a saved clone of that repo, and a repo with no clone here is simply
+absent from the answer — which is how a caller learns work cannot start on it.
+
 ---
 
 ## Cost & token usage
@@ -1063,6 +1094,17 @@ Favorited collections get their own toolbar buttons.
 
 ![Zoom — one agent enlarged, the others as a filmstrip along the bottom](https://raw.githubusercontent.com/receptron/mulmoterminal/main/docs/guide/images/grid-zoom.png)
 
+- **Set a terminal aside** — the moon button in a cell's header **sinks** it: the tile, its
+  filmstrip thumbnail and its cockpit-roster row all fade, and the working dot stops pulsing.
+  The session stays **connected and keeps its whole history** — this is what to reach for
+  instead of `/clear`-ing a cell you are done with for now, which resets the conversation just
+  to change how the cell looks. The setting survives a reload. **Enlarging it keeps it faded** —
+  that is how you read a set-aside session without waking it, and its roster row keeps the blue
+  "you are here" edge either way — while **typing into it wakes it**, so nothing has to be undone
+  by hand. Clicking or scrolling to read it does *not* wake it, even though a mouse-tracking agent
+  receives those as input. A cell that **stops for a permission prompt comes back to full strength
+  on its own**, so setting one aside can never hide a session that is waiting on you; a merely *finished* turn
+  does not, since that is the expected outcome of setting a running agent aside.
 - **Timeline** (🕘) — a read-only per-session activity timeline (tools run, newest first),
   from `GET /api/transcript/timeline`.
 - **Bring another cell's turn here** (💬) — pick another terminal in the grid and its
@@ -1286,8 +1328,10 @@ same-origin-guarded.
 | `GET /api/git-status?cwd=` | `{ repo, branch, detached, dirty, ahead, behind, upstream }`. |
 | `POST /api/git-remote` | The dir's GitHub repo URL (for the header GitHub menu). |
 | `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). |
+| `POST /api/worktrees/create` · `/remove` · `/push` · `/pr` | Create on `agent/<slug>` — or, with `issue: <N>`, on `issue/<N>-<slug>` forked from a freshly fetched `origin/<base>`; 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/repo-dirs` | Which saved directories clone which GitHub repo, ordered, with the recorded choice per repo. |
+| `POST /api/issues/start` | Cut an issue's worktree in one of that repo's known clones and spawn a session there, seeded with the issue as a draft. |
 | `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**
@@ -1341,6 +1385,10 @@ connection (or reattach to an existing background PTY).
   background PTY exists for `<id>`, the socket reattaches to it (and its recent
   output buffer is replayed); otherwise the server spawns
   `claude --resume <id> --settings <hooks>`.
+- `&cols=<n>&rows=<n>` — the terminal's geometry, on every endpoint that starts a PTY. The
+  PTY is created at it instead of the 120x30 default, so nothing is ever drawn at a size the
+  browser didn't ask for. Out-of-range values are ignored (same bounds as a `resize` frame),
+  and a connection that sends none keeps the default until its first `resize`.
 
 **Server → client** (JSON text frames):
 

package/common/dirPriorityOrder.ts

@@ -0,0 +1,24 @@
+// Ordering BY directory, for the two places that do it: the grid's "priority" sort and the
+// launcher's directory chips. Shared because those two are read side by side — a launcher that
+// answered "what does an unset rank mean?" differently would put a project in one place on the
+// chips and another in the grid.
+
+// A directory that sets no `orderPriority` sorts after every directory that does, so adding the
+// key to ONE project doesn't displace all the others. Infinity rather than a big sentinel: no
+// integer a user could write should be able to outrank "unset".
+export const UNSET_PRIORITY = Number.POSITIVE_INFINITY;
+
+export const dirPriority = (cwd: string | null, priorityByCwd: Record<string, number>): number =>
+  cwd ? (priorityByCwd[cwd] ?? UNSET_PRIORITY) : UNSET_PRIORITY;
+
+// Stable: equal ranks keep their incoming order, so the directories that declare none stay in
+// whatever order the caller already holds them in (for the chips, most-recently-used).
+//
+// Two unset directories subtract to `Infinity - Infinity` = NaN, which is falsy — so that pair
+// falls through to the index comparison and keeps its order, which is what the tie is meant to do.
+export function orderByDirPriority<T>(items: readonly T[], cwdOf: (item: T) => string | null, priorityByCwd: Record<string, number>): T[] {
+  return items
+    .map((item, i) => ({ item, i }))
+    .sort((a, b) => dirPriority(cwdOf(a.item), priorityByCwd) - dirPriority(cwdOf(b.item), priorityByCwd) || a.i - b.i)
+    .map((x) => x.item);
+}

package/common/prPhase.ts

@@ -42,14 +42,23 @@ export function workItemHeadline(item: WorkItem): string | null {
 //
 // `[1-9]\d*` rather than `\d+`: there is no issue #0, and `#0123` is not issue 123 — both are
 // typos, and a typo that renders as a link to somebody else's issue is worse than no chip.
-const CLOSING_KEYWORD = /\b(?:close[sd]?|fix(?:e[sd])?|resolve[sd]?)[:\s]+#([1-9]\d*)/i;
+const CLOSING_VERB = "(?:close[sd]?|fix(?:e[sd])?|resolve[sd]?)";
+const CLOSING_KEYWORD = new RegExp(`\\b${CLOSING_VERB}[:\\s]+#([1-9]\\d*)`, "i");
+
+// The other form GitHub accepts for the same statement: the issue's full URL. Each path segment
+// is a run of non-slash characters rather than `\S+`, so there is nothing for the engine to
+// backtrack over — same reason the separator above is one character class.
+const CLOSING_URL = new RegExp(`\\b${CLOSING_VERB}[:\\s]+https?://[^\\s/]+(?:/[^\\s/]+){2}/issues/[1-9]\\d*`, "i");
+
+// An issue or PR number as anything in this app will accept it. Shared because every surface that
+// handles one applies the same rule and a divergence would show as a link to nothing: digits from
+// a body or a branch name are unbounded, and `Number("9".repeat(20))` is 1e20 — which renders in
+// the chip as "#1e+20" (found by Codex review). There is also no issue #0 and no negative one.
+export const isIssueNumber = (v: unknown): v is number => typeof v === "number" && Number.isSafeInteger(v) && v > 0;
 
-// Digits from a body or a branch name are unbounded, and `Number("9".repeat(20))` is 1e20 — which
-// would render in the chip as "#1e+20" and link nowhere. Anything past the safe-integer range is
-// not an issue number (found by Codex review).
 function toIssueNumber(digits: string): number | null {
   const n = Number(digits);
-  return Number.isSafeInteger(n) && n > 0 ? n : null;
+  return isIssueNumber(n) ? n : null;
 }
 
 // The issue a PR body says it closes, or null. Deliberately blind to the full-URL form
@@ -60,6 +69,14 @@ export function issueRefFromPrBody(body: string | null | undefined): number | nu
   return found ? toIssueNumber(found[1]) : null;
 }
 
+// Whether the body ALREADY states what merging it closes, in either form GitHub honours. Broader
+// than issueRefFromPrBody on purpose, and the difference matters: that one answers "which issue in
+// THIS repo" and so must ignore a URL that may name another repository, while this one answers
+// "has the author already declared a closure" — where the URL form counts just as much, because
+// adding a second keyword on top of it closes BOTH issues on merge (found by Codex review).
+export const declaresClosingReference = (body: string | null | undefined): boolean =>
+  typeof body === "string" && (CLOSING_KEYWORD.test(body) || CLOSING_URL.test(body));
+
 // A branch named after its issue the way this repo names them: `fix/966-preserve-unknown-keys`.
 // Requires a type prefix and a hyphen after the digits, so `chore/dep-updates-20260728` is not
 // read as issue #20260728.
@@ -73,3 +90,19 @@ export function issueCandidateFromBranch(branch: string | null | undefined): num
   const found = typeof branch === "string" ? BRANCH_ISSUE.exec(branch) : null;
   return found ? toIssueNumber(found[1]) : null;
 }
+
+// The prefix this app gives a branch it creates FOR an issue (#1171). Shared because the two
+// sides must agree on the same string: worktree creation writes it, and the reader below is only
+// sound because nothing else in the app produces it.
+export const ISSUE_BRANCH_PREFIX = "issue/";
+
+const ANCHORED_ISSUE = new RegExp(`^${ISSUE_BRANCH_PREFIX}([1-9]\\d*)-`);
+
+// The issue a branch was CREATED for. Unlike the candidate above this is not a guess — the prefix
+// is one this app writes, so a match is a statement it made about its own branch. That is what
+// makes it safe to derive a PR body's `Fixes #N` from: a wrong number there closes somebody
+// else's issue the moment the PR merges, which no amount of confirming afterwards undoes.
+export function issueFromAnchoredBranch(branch: string | null | undefined): number | null {
+  const found = typeof branch === "string" ? ANCHORED_ISSUE.exec(branch) : null;
+  return found ? toIssueNumber(found[1]) : null;
+}

package/common/repoDirs.ts

@@ -0,0 +1,57 @@
+// Which local clones a GitHub repository has on this machine — what `GET /api/repo-dirs` answers.
+//
+// Shared because both sides decide from it: the server resolves each saved directory's `origin`
+// and groups them, and the UI picks which clone to start work in (and offers the rest). Several
+// clones of one repo commonly run side by side here, so "the directory for owner/repo" is a
+// CHOICE rather than a lookup — which is why a candidate carries what it takes to choose.
+
+export interface RepoDirCandidate {
+  /** Absolute path of the clone (the canonical spelling `cwdPresets` stores). */
+  path: string;
+  /** The preset's label, which is what the user named this clone in Settings. */
+  label: string;
+  /** The directory's own `orderPriority` from `.mulmoterminal.json`, or null when it sets none. */
+  orderPriority: number | null;
+}
+
+export interface RepoDirs {
+  /** `owner/repo`. */
+  repo: string;
+  /** Candidates, already ordered: by `orderPriority`, then by path for the ones that set none. */
+  dirs: RepoDirCandidate[];
+  /** The recorded choice for this repo, or null when there is none to honour. Always one of
+   *  `dirs` — a recording whose directory has since gone or now points at a different repo is
+   *  dropped rather than returned, because the caller would otherwise start work in it. */
+  primary: string | null;
+}
+
+export interface RepoDirsResponse {
+  repos: RepoDirs[];
+}
+
+// The server is the only writer of this shape, but it arrives as a network response: an older
+// build, a proxy returning HTML, or a half-deployed server would otherwise put `undefined` where
+// a WORKING DIRECTORY is expected — and this value becomes the cwd a session is started in.
+// Anything malformed is dropped rather than repaired, so a bad entry offers no clone at all
+// instead of a wrong one.
+const isCandidate = (v: unknown): v is RepoDirCandidate =>
+  isObject(v) &&
+  typeof v.path === "string" &&
+  v.path !== "" &&
+  typeof v.label === "string" &&
+  (v.orderPriority === null || typeof v.orderPriority === "number");
+
+const isObject = (v: unknown): v is Record<string, unknown> => typeof v === "object" && v !== null && !Array.isArray(v);
+
+export function parseRepoDirsResponse(data: unknown): RepoDirs[] {
+  if (!isObject(data) || !Array.isArray(data.repos)) return [];
+  return data.repos.filter(isObject).flatMap((entry) => {
+    if (typeof entry.repo !== "string" || entry.repo === "" || !Array.isArray(entry.dirs)) return [];
+    const dirs = entry.dirs.filter(isCandidate);
+    // `primary` must name one of the candidates it came with. A recording the server kept but a
+    // candidate list that no longer holds it would otherwise start work in a directory this side
+    // never offered.
+    const primary = typeof entry.primary === "string" && dirs.some((d) => d.path === entry.primary) ? entry.primary : null;
+    return [{ repo: entry.repo, dirs, primary }];
+  });
+}

package/common/terminalSize.ts

@@ -0,0 +1,30 @@
+// A terminal's geometry, and what counts as a usable one.
+//
+// Shared because BOTH sides decide from it: the browser puts the size it has already measured on
+// the connect URL so the pty is born with it (#1178), and the server validates that number before
+// it reaches node-pty — a crafted or buggy client must not be able to ask for a 0-column or
+// absurdly large terminal. Two copies of these bounds would let the two sides disagree about which
+// sizes exist at all.
+export interface TerminalSize {
+  cols: number;
+  rows: number;
+}
+
+export const MIN_TERM_COLS = 2;
+export const MAX_TERM_COLS = 500;
+export const MIN_TERM_ROWS = 1;
+export const MAX_TERM_ROWS = 200;
+
+export function isUsableTerminalSize({ cols, rows }: TerminalSize): boolean {
+  if (!Number.isInteger(cols) || !Number.isInteger(rows)) return false;
+  return cols >= MIN_TERM_COLS && cols <= MAX_TERM_COLS && rows >= MIN_TERM_ROWS && rows <= MAX_TERM_ROWS;
+}
+
+/** The geometry a connect URL carries, or null when it carries none it can stand behind — an
+ *  absent, unparsable or out-of-range pair. Null means "spawn at the default and wait for the
+ *  browser's resize", which is what every client did before this was sent. */
+export function parseTerminalSize(cols: string | null, rows: string | null): TerminalSize | null {
+  if (cols === null || rows === null) return null;
+  const size = { cols: Number(cols), rows: Number(rows) };
+  return isUsableTerminalSize(size) ? size : null;
+}

package/dist/assets/{abnfDiagram-VRR7QNED-RjiYivmv-FRF8kLeT.js → abnfDiagram-VRR7QNED-RjiYivmv-D3Xi4MgH.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-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};
+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-Dit0KGYk.js";import"./chunk-WYO6CB5R-BlzOfotS-BJCy4_YQ.js";import"./chunk-VAUOI2AC-CS9QJ4yz-Oly0QbKc.js";import{n as i,r as a,t as o}from"./chunk-MOJQB5TN-CMZRaeqt-EDD8-kXb.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-C59c2mZk.js

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