@yagni-app/code-staging 0.3.4-staging.1155.1 → 0.3.4-staging.1156.1

package/dist/extension/cmux/index.d.ts

@@ -1,3 +1,19 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-export declare function registerCmuxBridge(pi: ExtensionAPI): void;
+import { type SessionState } from "./state.js";
+import { type SummarizeTitleResult } from "../sessionTitle/summarize.js";
+/** Injectable seams so the bridge's state machine is testable without a child. */
+export interface CmuxBridgeDeps {
+    summarize?: (prompt: string, deps?: {
+        cwd?: string;
+    }) => Promise<SummarizeTitleResult>;
+}
+export declare function registerCmuxBridge(pi: ExtensionAPI, deps?: CmuxBridgeDeps): void;
+/**
+ * The naming half of `publishCompletion`, extracted so the placeholder→final
+ * guard is directly testable without a child process. Once a session's title
+ * has been upgraded (or the upgrade was attempted), a later settle does nothing.
+ */
+export declare function upgradeTitleOnce(state: SessionState, summarize: (prompt: string, deps?: {
+    cwd?: string;
+}) => Promise<SummarizeTitleResult>, setTitle: ((title: string) => void) | undefined, onFail: (reason: string) => void, cwd: string): Promise<void>;
 //# sourceMappingURL=index.d.ts.map

package/dist/extension/cmux/index.js

@@ -3,7 +3,9 @@ import { join } from "node:path";
 import { CmuxDispatcher } from "./dispatcher.js";
 import { stateFor, snapshotContext, beginTurn, currentTurnId, finishTurn, settleTurn, lastAssistantMessage, firstString, objectValue, warn, } from "./state.js";
 import { sendHook, ensureResumeBinding, clearResumeBinding, releaseSessionRuntime, } from "./hooks.js";
-import { renameWorkspaceFromPrompt } from "./naming.js";
+import { isDriverCaller } from "../config.js";
+import { sessionTitle } from "../sessionTitle/title.js";
+import { summarizeTitle } from "../sessionTitle/summarize.js";
 /** Remove a stale cmux-installed extension so hooks don't double-fire. */
 function removeStaleManagedExtension() {
     const agentDir = process.env.PI_CODING_AGENT_DIR || process.env.YAGNI_CODING_AGENT_DIR;
@@ -24,11 +26,17 @@ function removeStaleManagedExtension() {
     }
     catch { }
 }
-export function registerCmuxBridge(pi) {
+export function registerCmuxBridge(pi, deps = {}) {
     if (!process.env.CMUX_SURFACE_ID)
         return;
     if (process.env.CMUX_PI_HOOKS_DISABLED === "1")
         return;
+    // A child process (guardian consult, advisor, /go stage, subagent) has no
+    // direct user — reporting its internals to cmux would overwrite the driver
+    // session's card with e.g. the Guardian consult prompt/verdict.
+    if (!isDriverCaller())
+        return;
+    const summarize = deps.summarize ?? summarizeTitle;
     const dispatcher = new CmuxDispatcher();
     const sessionStates = new Map();
     const lifecycleTails = new Map();
@@ -74,6 +82,18 @@ export function registerCmuxBridge(pi) {
         if (typeof event.prompt === "string" && event.prompt.trim()) {
             st.lastPrompt = event.prompt.trim();
         }
+        // Set the terminal title to the heuristic session title immediately at the
+        // first prompt (mirroring Claude's derive-then-upgrade: the tab is named
+        // from the prompt, not the model reply). cmux freezes auto-titling on its
+        // own once the user sets a manual name (customTitle), so we never clobber it.
+        // The optional LLM upgrade happens later on agent_settled.
+        if (!st.nameState && typeof event.prompt === "string" && event.prompt.trim()) {
+            const placeholder = sessionTitle(event.prompt.trim());
+            if (placeholder) {
+                st.nameState = "placeholder";
+                context.setTitle?.(placeholder);
+            }
+        }
         enqueueLifecycleTask(sessionId, context, () => sendHook(dispatcher, "prompt-submit", context, { prompt: event.prompt, turn_id: turnId }));
     });
     pi.on("agent_end", (event, ctx) => {
@@ -104,7 +124,7 @@ export function registerCmuxBridge(pi) {
             return;
         const completion = settleTurn(sessionStates, sessionId);
         if (completion) {
-            enqueueLifecycleTask(sessionId, context, () => publishCompletion(dispatcher, sessionStates, context, sessionId, completion));
+            enqueueLifecycleTask(sessionId, context, () => publishCompletion(dispatcher, sessionStates, context, sessionId, completion, summarize));
         }
     });
     pi.on("session_shutdown", async (event, ctx) => {
@@ -133,7 +153,27 @@ export function registerCmuxBridge(pi) {
         });
     });
 }
-async function publishCompletion(dispatcher, sessionStates, context, sessionId, completion) {
+/**
+ * The naming half of `publishCompletion`, extracted so the placeholder→final
+ * guard is directly testable without a child process. Once a session's title
+ * has been upgraded (or the upgrade was attempted), a later settle does nothing.
+ */
+export async function upgradeTitleOnce(state, summarize, setTitle, onFail, cwd) {
+    if (state.nameState !== "placeholder")
+        return;
+    state.nameState = "final";
+    const prompt = state.lastPrompt;
+    if (!prompt)
+        return;
+    const result = await summarize(prompt, { cwd });
+    if (result.ok) {
+        setTitle?.(result.title);
+    }
+    else {
+        onFail(result.reason);
+    }
+}
+async function publishCompletion(dispatcher, sessionStates, context, sessionId, completion, summarize) {
     const stopPayload = {
         last_assistant_message: completion.lastAssistantMessage,
         turn_id: completion.turnId,
@@ -147,9 +187,8 @@ async function publishCompletion(dispatcher, sessionStates, context, sessionId,
         stopPayload.cmux_notification_routed = true;
     await sendHook(dispatcher, "stop", context, stopPayload);
     const state = stateFor(sessionStates, sessionId);
-    if (!state.hasNamed) {
-        renameWorkspaceFromPrompt(dispatcher, sessionStates, context, sessionId);
-        state.hasNamed = true;
-    }
+    // Upgrade the heuristic placeholder to an LLM-summarized title, once (see
+    // upgradeTitleOnce). A failed summary keeps the placeholder in place.
+    await upgradeTitleOnce(state, summarize, context.setTitle, (reason) => warn(context, "title summary failed; keeping heuristic title", { session_id: sessionId, reason }), context.cwd);
 }
 //# sourceMappingURL=index.js.map

package/dist/extension/cmux/state.d.ts

@@ -4,18 +4,22 @@ export interface PendingCompletion {
     notificationType: string;
     turnId: string;
 }
+/** The naming lifecycle of a session's cmux workspace title. */
+export type NameState = "unnamed" | "placeholder" | "final";
 export interface SessionState {
     nextTurn: number;
     activeTurnId?: string;
     pendingCompletion?: PendingCompletion;
     stopped: boolean;
     lastPrompt?: string;
-    hasNamed?: boolean;
+    nameState?: NameState;
 }
 export interface PiExtensionContextSnapshot {
     readonly sessionId: string | null;
     readonly cwd: string;
     readonly notifyWarning?: () => void;
+    /** Set the terminal tab/window title (pi's OSC 0), mirroring Claude's title channel. */
+    readonly setTitle?: (title: string) => void;
 }
 export declare function firstString(...values: unknown[]): string | null;
 export declare function objectValue(value: unknown, keys: string[]): unknown;

package/dist/extension/cmux/state.js

@@ -1,3 +1,6 @@
+import { appendFileSync, mkdirSync } from "node:fs";
+import { dirname, join } from "node:path";
+import { codeStateHome } from "../stateHome.js";
 export function firstString(...values) {
     for (const value of values) {
         if (typeof value === "string" && value.trim().length > 0)
@@ -67,14 +70,18 @@ export function snapshotContext(ctx) {
         return { sessionId: null, cwd: process.cwd() };
     }
     let notifyWarning;
+    let setTitle;
     try {
         const ui = ctx.ui;
         if (typeof ui?.notify === "function") {
             notifyWarning = () => ui.notify?.("cmux integration warning — check the terminal for details", "warning");
         }
+        if (typeof ui?.setTitle === "function") {
+            setTitle = (title) => ui.setTitle?.(title);
+        }
     }
     catch { }
-    return { sessionId: sessionIdFrom(ctx), cwd: cwdFrom(ctx), notifyWarning };
+    return { sessionId: sessionIdFrom(ctx), cwd: cwdFrom(ctx), notifyWarning, setTitle };
 }
 export function stateFor(sessionStates, sessionId) {
     let state = sessionStates.get(sessionId);
@@ -126,11 +133,18 @@ export function settleTurn(sessionStates, sessionId) {
 }
 export function warn(ctx, message, details = {}, notifyUser = false) {
     const payload = { source: "yagni-cmux-bridge", level: "warning", message, ...details };
+    // The TUI is in raw mode: writing to stdout/stderr corrupts the terminal (the
+    // JSON was landing at the prompt cursor). Route warnings/errors to a rotating
+    // local log instead — never to the terminal.
     try {
-        console.warn(JSON.stringify(payload));
+        if (!process.env.NODE_TEST_CONTEXT) {
+            const path = join(codeStateHome(null), "logs", "cmux-bridge.log");
+            mkdirSync(dirname(path), { recursive: true });
+            appendFileSync(path, JSON.stringify({ ts: new Date().toISOString(), ...payload }) + "\n", "utf8");
+        }
     }
     catch {
-        console.warn(`[yagni-cmux-bridge] ${message}`);
+        /* logging must never throw into the editor */
     }
     if (notifyUser) {
         try {

package/dist/extension/pipeline/personas.js

@@ -184,6 +184,14 @@ Output ONLY a JSON object with this exact shape:
 {"outcome":"allow"|"ask"|"deny","riskLevel":"low"|"medium"|"high"|"critical","rationale":"see rationale rules"}
 
 Do not output anything else after the JSON. No markdown fences, only the JSON object.`;
+const TITLE_BODY = `You produce a session title from a user's prompt. Output ONLY a concise, sentence-case title of 3-7 words that captures the main topic or goal. Capitalize only the first word and proper nouns. Do not include a ticket code in the title text itself (the caller prepends it). No markdown, no prose, no quotes — just the title on one line.
+
+Good:
+Fix the login page
+Add OAuth authentication
+
+Bad (too vague): Code changes
+Bad (too long): Investigate and fix the login button not responding on mobile devices`;
 /** Persona body keyed by the agent name referenced in `stages.ts`. */
 export const PERSONA_BODIES = {
     scout: SCOUT_BODY,
@@ -192,6 +200,7 @@ export const PERSONA_BODIES = {
     reviewer: REVIEWER_BODY,
     advisor: ADVISOR_BODY,
     guardian: GUARDIAN_BODY,
+    title: TITLE_BODY,
     orchestrator: [ORCHESTRATOR_BODY, PARTITION_CONTRACT].join("\n\n"),
     synthesizer: SYNTHESIZER_BODY,
 };

package/dist/extension/sessionTitle/summarize.d.ts

@@ -0,0 +1,40 @@
+/**
+ * The I/O half of session titling: summarize a prompt into a title via a
+ * locked-down efficient-tier child (same `runStage` seam the Guardian and
+ * advisor use). Fully optional — any failure returns undefined so the caller
+ * falls back to the heuristic title without ever blocking a turn.
+ */
+import { runStage as defaultRunStage } from "../pipeline/runner.js";
+import type { PipelineStage } from "../pipeline/types.js";
+/** The model tier the title child runs on (read-only, like the Guardian). */
+export declare const TITLE_MODEL_TIER = "efficient";
+/** The read-only tools the title child may use (only `read`, for context). */
+export declare const TITLE_TOOLS: string[];
+/** Default wall-clock timeout for the title consult (matches Guardian). */
+export declare const DEFAULT_TITLE_TIMEOUT_MS = 15000;
+/**
+ * The synthetic stage a title consult runs as. Borrows the `plan` StageId (same
+ * pattern as the advisor/guardian) so it doesn't ripple into /go feed/reducers.
+ * The agent name selects the `title` persona from PERSONA_BODIES.
+ */
+export declare function titleStage(modelTier?: string): PipelineStage;
+export interface SummarizeTitleDeps {
+    runStage?: typeof defaultRunStage;
+    timeoutMs?: number;
+    cwd?: string;
+}
+/** The outcome of a title consult: a prefixed title, or a failure reason. */
+export type SummarizeTitleResult = {
+    ok: true;
+    title: string;
+} | {
+    ok: false;
+    reason: string;
+};
+/**
+ * Summarize a prompt into a title. Returns `{ title }` on success, or
+ * `{ reason }` on any failure (timeout, abort, non-zero exit, empty or unusable
+ * output) — the caller falls back to the heuristic and logs the reason.
+ */
+export declare function summarizeTitle(prompt: string, deps?: SummarizeTitleDeps): Promise<SummarizeTitleResult>;
+//# sourceMappingURL=summarize.d.ts.map

package/dist/extension/sessionTitle/summarize.js

@@ -0,0 +1,63 @@
+/**
+ * The I/O half of session titling: summarize a prompt into a title via a
+ * locked-down efficient-tier child (same `runStage` seam the Guardian and
+ * advisor use). Fully optional — any failure returns undefined so the caller
+ * falls back to the heuristic title without ever blocking a turn.
+ */
+import { runStage as defaultRunStage } from "../pipeline/runner.js";
+import { parseSummarizedTitle } from "./title.js";
+/** The model tier the title child runs on (read-only, like the Guardian). */
+export const TITLE_MODEL_TIER = "efficient";
+/** The read-only tools the title child may use (only `read`, for context). */
+export const TITLE_TOOLS = ["read"];
+/** Default wall-clock timeout for the title consult (matches Guardian). */
+export const DEFAULT_TITLE_TIMEOUT_MS = 15_000;
+/**
+ * The synthetic stage a title consult runs as. Borrows the `plan` StageId (same
+ * pattern as the advisor/guardian) so it doesn't ripple into /go feed/reducers.
+ * The agent name selects the `title` persona from PERSONA_BODIES.
+ */
+export function titleStage(modelTier = TITLE_MODEL_TIER) {
+    return {
+        id: "plan",
+        agent: "title",
+        model: modelTier,
+        tools: TITLE_TOOLS,
+        taskTemplate: "{ticket}",
+    };
+}
+/**
+ * Summarize a prompt into a title. Returns `{ title }` on success, or
+ * `{ reason }` on any failure (timeout, abort, non-zero exit, empty or unusable
+ * output) — the caller falls back to the heuristic and logs the reason.
+ */
+export async function summarizeTitle(prompt, deps = {}) {
+    const runStage = deps.runStage ?? defaultRunStage;
+    const timeoutMs = deps.timeoutMs ?? DEFAULT_TITLE_TIMEOUT_MS;
+    const controller = new AbortController();
+    const timer = setTimeout(() => controller.abort(), timeoutMs);
+    timer.unref?.();
+    try {
+        const result = await runStage(titleStage(), { ticket: `Summarize this session into a short title:\n\n${prompt}` }, {
+            cwd: deps.cwd ?? process.cwd(),
+            signal: controller.signal,
+            callerLabel: "title",
+        });
+        if (result.exitCode !== 0) {
+            const stderr = result.stderr?.trim().slice(0, 200);
+            return { ok: false, reason: stderr ? `exit=${result.exitCode} ${stderr}` : `exit=${result.exitCode}` };
+        }
+        const title = parseSummarizedTitle(prompt, result.finalOutput);
+        if (title)
+            return { ok: true, title };
+        return { ok: false, reason: `empty_output output=${JSON.stringify(result.finalOutput.slice(0, 200))}` };
+    }
+    catch (error) {
+        const msg = error instanceof Error ? error.message : String(error);
+        return { ok: false, reason: `threw ${msg.slice(0, 200)}` };
+    }
+    finally {
+        clearTimeout(timer);
+    }
+}
+//# sourceMappingURL=summarize.js.map

package/dist/extension/sessionTitle/title.d.ts

@@ -0,0 +1,27 @@
+/**
+ * Pure session-title derivation — the reusable half of naming an agent session.
+ * No cmux, no workspace, no I/O: just "given a prompt, what's a good title".
+ *
+ * cmux is the current (and only) consumer, but this module is deliberately
+ * cmux-free so a future consumer (terminal title, a resume hint, the Feed) can
+ * reuse it without importing cmux vocabulary.
+ */
+/** Extract a ticket code from a prompt, or null when none is present. */
+export declare function ticketCode(prompt: string): string | null;
+/** The heuristic fallback title: the first 8 words, capped at 60 chars. */
+export declare function titleFromPrompt(prompt: string): string | undefined;
+/**
+ * Compose a session title from a prompt, prepending the ticket code when one is
+ * present. The code is stripped from the derived body so it doesn't appear
+ * twice ("YAG-532: work on YAG-532 …").
+ */
+export declare function sessionTitle(prompt: string, body?: string): string | undefined;
+/**
+ * Normalize a model-produced title into the same shape `sessionTitle` emits.
+ * Returns undefined when the model's output is empty, whitespace, or otherwise
+ * unusable — the caller then falls back to the heuristic.
+ */
+export declare function parseSummarizedTitle(prompt: string, raw: string): string | undefined;
+/** The maximum chars `parseSummarizedTitle` may return (the same cap). */
+export declare const SESSION_TITLE_MAX_LEN = 60;
+//# sourceMappingURL=title.d.ts.map

package/dist/extension/sessionTitle/title.js

@@ -0,0 +1,57 @@
+/**
+ * Pure session-title derivation — the reusable half of naming an agent session.
+ * No cmux, no workspace, no I/O: just "given a prompt, what's a good title".
+ *
+ * cmux is the current (and only) consumer, but this module is deliberately
+ * cmux-free so a future consumer (terminal title, a resume hint, the Feed) can
+ * reuse it without importing cmux vocabulary.
+ */
+/** The ticket-code shape ("YAG-532", "PROJ-123"), matching the backend's IDENTIFIER_RE. */
+const TICKET_ID_RE = /\b[A-Z][A-Z0-9]*-\d+\b/i;
+const MAX_TITLE_LEN = 60;
+/** Extract a ticket code from a prompt, or null when none is present. */
+export function ticketCode(prompt) {
+    return prompt.match(TICKET_ID_RE)?.[0]?.toUpperCase() ?? null;
+}
+/** The heuristic fallback title: the first 8 words, capped at 60 chars. */
+export function titleFromPrompt(prompt) {
+    const words = prompt.trim().split(/\s+/).filter(Boolean);
+    if (words.length === 0)
+        return undefined;
+    return words.slice(0, 8).join(" ").slice(0, MAX_TITLE_LEN) || undefined;
+}
+/**
+ * Compose a session title from a prompt, prepending the ticket code when one is
+ * present. The code is stripped from the derived body so it doesn't appear
+ * twice ("YAG-532: work on YAG-532 …").
+ */
+export function sessionTitle(prompt, body) {
+    const code = ticketCode(prompt);
+    // Derive the body from the prompt with the ticket code removed (unless the
+    // caller supplied an already-summarized body, in which case prefer it).
+    const effectiveBody = body ?? titleFromPrompt(code ? prompt.replace(TICKET_ID_RE, "") : prompt);
+    if (!code)
+        return effectiveBody;
+    if (!effectiveBody)
+        return code;
+    return `${code}: ${effectiveBody}`.slice(0, MAX_TITLE_LEN);
+}
+/**
+ * Normalize a model-produced title into the same shape `sessionTitle` emits.
+ * Returns undefined when the model's output is empty, whitespace, or otherwise
+ * unusable — the caller then falls back to the heuristic.
+ */
+export function parseSummarizedTitle(prompt, raw) {
+    const cleaned = raw.trim().replace(/\s+/g, " ").replace(/^["']|["']$/g, "");
+    if (!cleaned)
+        return undefined;
+    // Strip any ticket-code token the model echoed back into its summary (the
+    // prompt carries the code, and models repeat it) so the composed title never
+    // doubles the prefix ("YAG-485: YAG-485 …"). Deterministic, not prompt hygiene.
+    const code = ticketCode(prompt);
+    const body = code ? cleaned.replace(TICKET_ID_RE, "").trim().replace(/\s+/g, " ") : cleaned;
+    return sessionTitle(prompt, body);
+}
+/** The maximum chars `parseSummarizedTitle` may return (the same cap). */
+export const SESSION_TITLE_MAX_LEN = MAX_TITLE_LEN;
+//# sourceMappingURL=title.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yagni-app/code-staging",
-  "version": "0.3.4-staging.1155.1",
+  "version": "0.3.4-staging.1156.1",
   "description": "YAGNI Code: a terminal coding agent that already knows your company. One YAGNI login routes the model and grounds the agent in your team's context.",
   "license": "SEE LICENSE IN LICENSE.md",
   "author": "YAGNI, Inc. <jack@yagni.app> (https://yagni.app)",
@@ -39,5 +39,5 @@
     "smol-toml": "^1.8.0",
     "typebox": "^1.3.15"
   },
-  "yagniSourceSha": "17bd689a1ff1bc6e8d9fc0d420b2d4e580a781dd"
+  "yagniSourceSha": "28861a8fed4d500c1391588cc71597d6ff0fa43a"
 }

package/dist/extension/cmux/naming.d.ts

@@ -1,5 +0,0 @@
-import type { CmuxDispatcher } from "./dispatcher.js";
-import { type PiExtensionContextSnapshot, type SessionState } from "./state.js";
-export declare function titleFromPrompt(prompt: string): string | undefined;
-export declare function renameWorkspaceFromPrompt(dispatcher: CmuxDispatcher, sessionStates: Map<string, SessionState>, context: PiExtensionContextSnapshot, sessionId: string): void;
-//# sourceMappingURL=naming.d.ts.map

package/dist/extension/cmux/naming.js

@@ -1,23 +0,0 @@
-import { firstString } from "./state.js";
-export function titleFromPrompt(prompt) {
-    const words = prompt.trim().split(/\s+/).filter(Boolean);
-    if (words.length === 0)
-        return undefined;
-    return words.slice(0, 8).join(" ").slice(0, 60) || undefined;
-}
-export function renameWorkspaceFromPrompt(dispatcher, sessionStates, context, sessionId) {
-    const prompt = sessionStates.get(sessionId)?.lastPrompt;
-    if (!prompt)
-        return;
-    const title = titleFromPrompt(prompt);
-    if (!title)
-        return;
-    // Use the workspace-scoped rename form: cmux workspace rename <ws> --title <title>
-    // (NOT rename-workspace --workspace ... --surface ... which misinterprets --surface
-    // as part of the title).
-    const workspaceId = firstString(process.env.CMUX_WORKSPACE_ID);
-    if (!workspaceId)
-        return;
-    void dispatcher.run(["workspace", "rename", workspaceId, "--title", title], context.cwd, undefined, context);
-}
-//# sourceMappingURL=naming.js.map