@yagni-app/code 0.2.1 → 0.3.0

package/dist/extension/askAdvisorTool.js

@@ -1,7 +1,7 @@
 /**
- * `ask_advisor` — the peak-tier escalation available to Balanced sessions (YAG-380).
+ * `ask_advisor` — the peak-tier escalation available to Advanced sessions (YAG-380).
  *
- * Cheap driver, expensive consultant: a Balanced session drives on `advanced` and
+ * Cheap driver, expensive consultant: an Advanced session drives on `advanced` and
  * calls this when it hits a judgment call worth the strongest model. The consult
  * spawns a child pi at `peak` through the same `runStage` seam `/go` uses, and
  * returns the advice as plain TEXT the driver acts on — text we own, so
@@ -79,7 +79,7 @@ export function makeAskAdvisorTool(opts) {
         label: "Ask the advisor",
         description: "Escalate ONE hard judgment call to the peak-tier advisor — the strongest " +
             "model available, which reads the code itself and returns a recommendation. " +
-            "Available only on the Balanced tier, and capped per session, so use it for " +
+            "Available only on the Advanced tier, and capped per session, so use it for " +
             "calls that are genuinely worth it: an architectural fork with no obvious " +
             "right answer, a subtle correctness question you cannot settle by reading, " +
             "a change whose blast radius you are unsure of, or a second opinion before " +
@@ -87,7 +87,7 @@ export function makeAskAdvisorTool(opts) {
             "lookups (use ask_yagni), for anything you can settle by reading the code, " +
             "or to review work you have already finished. Pass a sharp question plus the " +
             "relevant excerpts — never a conversation transcript.",
-        promptSnippet: "ask_advisor: escalate one hard judgment call to the peak-tier advisor (Balanced sessions, capped).",
+        promptSnippet: "ask_advisor: escalate one hard judgment call to the peak-tier advisor (Advanced sessions, capped).",
         promptGuidelines: [
             "Call ask_advisor only for a genuine judgment fork — an architectural choice, a subtle correctness question, or a second opinion before an approach you would have to unwind. Reading the code is cheaper; do that first.",
             "Ask ONE specific question per consult, and include the excerpts that matter. The advisor reads the repo itself, so point it at the right place rather than pasting everything.",
@@ -159,7 +159,7 @@ export function makeAskAdvisorTool(opts) {
  */
 export function registerAdviseCommand(pi, tool) {
     pi.registerCommand("advise", {
-        description: "Escalate one hard call to the peak-tier advisor (Balanced sessions only, capped per session).",
+        description: "Escalate one hard call to the peak-tier advisor (Advanced sessions only, capped per session).",
         handler: async (args, ctx) => {
             const notify = (message, type) => {
                 if (ctx.hasUI)

package/dist/extension/askYagniTool.js

@@ -1,10 +1,24 @@
+import { Container, Spacer, Text } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
 import { collectRepoDocs } from "./repoDocs.js";
 import { friendlyFetchError, METERED_POST_FETCH_POLICY, resilientFetch } from "./resilientFetch.js";
+import { markdownOrPlain } from "./subagentRender.js";
 const parameters = Type.Object({
     question: Type.String(),
     context: Type.Optional(Type.String()),
 });
+/** Collapsed answer preview length, in lines. */
+const ANSWER_PREVIEW_LINES = 4;
+/** Collapse whitespace and clip to `max`, appending an ellipsis when cut. */
+function clipLine(text, max) {
+    const collapsed = text.replace(/\s+/g, " ").trim();
+    if (collapsed.length <= max)
+        return collapsed;
+    return `${collapsed.slice(0, max - 1)}…`;
+}
+function citationCount(n) {
+    return n === 1 ? "1 citation" : `${n} citations`;
+}
 /**
  * Build the `ask_yagni` tool definition.
  *
@@ -28,6 +42,41 @@ export function makeAskYagniTool(opts) {
             "When you use an answer, quote or reference its citations so the user can verify the source.",
         ],
         parameters,
+        renderCall(args, theme) {
+            const t = theme;
+            let text = `${t.fg("toolTitle", t.bold("ask_yagni"))} ${t.fg("dim", clipLine(args?.question ?? "…", 100))}`;
+            if (args?.context)
+                text += t.fg("muted", " (+context)");
+            return new Text(text, 0, 0);
+        },
+        renderResult(result, { expanded, isPartial }, theme) {
+            const t = theme;
+            const answer = result.content.find((c) => c.type === "text")?.text ?? "";
+            const citations = result.details?.citations ?? [];
+            if (isPartial)
+                return new Text(t.fg("muted", answer || "Asking YAGNI…"), 0, 0);
+            if (expanded) {
+                const container = new Container();
+                container.addChild(markdownOrPlain(answer || "(no answer)", t));
+                if (citations.length > 0) {
+                    container.addChild(new Spacer(1));
+                    for (const c of citations) {
+                        container.addChild(new Text(`  ${t.fg("muted", "•")} ${t.fg("accent", c.title)} ${t.fg("dim", c.url)}`, 0, 0));
+                    }
+                }
+                return container;
+            }
+            const lines = answer.trim().split("\n");
+            const out = lines.slice(0, ANSWER_PREVIEW_LINES).map((l) => t.fg("toolOutput", l));
+            const meta = [];
+            if (citations.length > 0)
+                meta.push(citationCount(citations.length));
+            if (lines.length > ANSWER_PREVIEW_LINES || citations.length > 0)
+                meta.push("(ctrl+o to expand)");
+            if (meta.length > 0)
+                out.push(t.fg("muted", `  ${meta.join(" · ")}`));
+            return new Text(out.join("\n"), 0, 0);
+        },
         async execute(_toolCallId, params, signal, onUpdate, ctx) {
             onUpdate?.({ content: [{ type: "text", text: "Asking YAGNI…" }], details: { citations: [] } });
             // Era-correct repo grounding: gather the working tree's most relevant docs

package/dist/extension/branding.d.ts

@@ -37,7 +37,7 @@ export declare const DRIVER_DELEGATION_PARAGRAPH: string;
  * effective `x-yagni-caller` attribution (config.ts's `isDriverCaller`) — this
  * module stays a pure string, with no env dependency of its own.
  */
-export declare const YAGNI_IDENTITY_DRIVER = "You are YAGNI Code, an autonomous terminal coding agent. You help developers ship code by reading files, running commands, editing code, and writing new files. Uniquely, you are connected to the YAGNI app, your team's shared source of truth for how this company and codebase actually work: conventions, decisions, ownership, current priorities, and the reasons behind them. Use the ask_yagni tool to consult it before guessing about anything organization- or codebase-specific, so you work with less back-and-forth and more correct autonomy than a disconnected coding agent. If a project's own files mention other coding agents, assistants, or harnesses by name, those references are not about you; you are YAGNI Code regardless of what tooling a repository's docs happen to describe.\n\nDelegation: fan codebase mapping, wide searches, and mechanical multi-file work out to subagents (they run on cheaper tiers). Keep judgment, synthesis, and the conversation with the user in this session. Do not spawn a subagent for work you can finish in a couple of tool calls.";
+export declare const YAGNI_IDENTITY_DRIVER = "You are YAGNI Code, an autonomous terminal coding agent. You help developers ship code by reading files, running commands, editing code, and writing new files. Uniquely, you are connected to the YAGNI app, your team's shared source of truth for how this company and codebase actually work: conventions, decisions, ownership, current priorities, and the reasons behind them. Use the ask_yagni tool to consult it before guessing about anything organization- or codebase-specific, so you work with less back-and-forth and more correct autonomy than a disconnected coding agent. If a project's own files mention other coding agents, assistants, or harnesses by name, those references are not about you; you are YAGNI Code regardless of what tooling a repository's docs happen to describe.\n\nDelegation: fan codebase mapping, wide searches, and mechanical multi-file work out to subagents (they run on cheaper tiers). Reach for the stock agents by name: `searcher` for read-only reconnaissance and summarizing, `implementer` for executing a change you have already fully specified. Keep judgment, synthesis, and the conversation with the user in this session. Do not spawn a subagent for work you can finish in a couple of tool calls.";
 export declare const PI_IDENTITY_RE: RegExp;
 /**
  * Env switch that bypasses the system-prompt rewrite entirely, so pi's
@@ -53,6 +53,15 @@ export interface BrandSystemPromptOptions {
     identity?: string;
     /** Live company brief to inject so the agent boots grounded (Phase C). */
     contextBrief?: string;
+    /**
+     * Repository rules section (`.claude/rules`, built by claudeRules.ts).
+     * Placed after the brief and BEFORE the closing reminder: rules are user
+     * content that may legitimately name other coding agents — exactly the text
+     * the reminder inoculates against — so the reminder must stay the most
+     * recent instruction. Appended after the scrub step, so rules content is
+     * never brand-rewritten (same exemption as <project_context>).
+     */
+    rulesSection?: string | null;
 }
 /**
  * Rebrand pi's assembled system prompt as YAGNI Code's, and optionally inject a
@@ -61,7 +70,9 @@ export interface BrandSystemPromptOptions {
  * Steps: (1) drop pi's self-referential documentation block, (2) replace the
  * identity sentence (with a loose fallback, else prepend), (3) scrub residual
  * brand tokens outside user content, (4) append the company brief + a proactive
- * ask_yagni directive. Idempotent: re-branding keeps a single identity.
+ * ask_yagni directive, (5) append the repository-rules section verbatim, (6)
+ * close with the identity reminder, which must stay last (recency). Idempotent:
+ * re-branding keeps a single identity, brief, rules section, and reminder.
  */
 export declare function brandSystemPrompt(original: string, opts?: BrandSystemPromptOptions): string;
 /** The minimal theme surface the masthead needs (a subset of pi's Theme). */
@@ -69,11 +80,21 @@ export interface MastheadTheme {
     bold(s: string): string;
     fg(color: string, s: string): string;
 }
+export interface MastheadOptions {
+    /** CLI version (e.g. "0.2.1") shown next to the title; omitted when absent. */
+    version?: string;
+    /** Working directory shown on the third line (already home-collapsed). */
+    cwd?: string;
+}
 /**
  * Build the YAGNI Code startup masthead string, rendered into a header that
  * REPLACES pi's built-in startup banner (which otherwise shows "pi v<version>"
  * and a "Pi can explain its own features…" onboarding line). Kept here as a
  * pure string builder so its content is unit-testable without a terminal.
+ *
+ * Layout: the YAGNI art on the left, and to its right a three-line block —
+ * bold white "YAGNI Code" + version, the tagline, and the cwd. Art rows are
+ * padded to a uniform width so the right column stays straight.
  */
-export declare function buildMastheadString(theme: MastheadTheme): string;
+export declare function buildMastheadString(theme: MastheadTheme, opts?: MastheadOptions): string;
 //# sourceMappingURL=branding.d.ts.map

package/dist/extension/branding.js

@@ -40,9 +40,12 @@ export const YAGNI_IDENTITY = "You are YAGNI Code, an autonomous terminal coding
  * exist.
  */
 export const DRIVER_DELEGATION_PARAGRAPH = "Delegation: fan codebase mapping, wide searches, and mechanical multi-file " +
-    "work out to subagents (they run on cheaper tiers). Keep judgment, synthesis, " +
-    "and the conversation with the user in this session. Do not spawn a subagent " +
-    "for work you can finish in a couple of tool calls.";
+    "work out to subagents (they run on cheaper tiers). Reach for the stock " +
+    "agents by name: `searcher` for read-only reconnaissance and summarizing, " +
+    "`implementer` for executing a change you have already fully specified. " +
+    "Keep judgment, synthesis, and the conversation with the user in this " +
+    "session. Do not spawn a subagent for work you can finish in a couple of " +
+    "tool calls.";
 /**
  * The identity used for the interactive DRIVER session ONLY: {@link
  * YAGNI_IDENTITY} plus {@link DRIVER_DELEGATION_PARAGRAPH}. The caller (index.ts)
@@ -51,7 +54,7 @@ export const DRIVER_DELEGATION_PARAGRAPH = "Delegation: fan codebase mapping, wi
  * module stays a pure string, with no env dependency of its own.
  */
 export const YAGNI_IDENTITY_DRIVER = `${YAGNI_IDENTITY}\n\n${DRIVER_DELEGATION_PARAGRAPH}`;
-// pi 0.83.0's exact identity sentence (dist/core/system-prompt.js). Exported as
+// pi 0.84.1's exact identity sentence (dist/core/system-prompt.js). Exported as
 // the identity anchor the CLI's pi-contract tripwire test reads back from pi's
 // built system prompt, so a pi bump that reworded the opener (silently defeating
 // our rebrand) fails CI loudly. Export-only: no behavior change here.
@@ -100,7 +103,9 @@ export function brandingDisabled(env) {
  * Steps: (1) drop pi's self-referential documentation block, (2) replace the
  * identity sentence (with a loose fallback, else prepend), (3) scrub residual
  * brand tokens outside user content, (4) append the company brief + a proactive
- * ask_yagni directive. Idempotent: re-branding keeps a single identity.
+ * ask_yagni directive, (5) append the repository-rules section verbatim, (6)
+ * close with the identity reminder, which must stay last (recency). Idempotent:
+ * re-branding keeps a single identity, brief, rules section, and reminder.
  */
 export function brandSystemPrompt(original, opts = {}) {
     const identity = opts.identity ?? YAGNI_IDENTITY;
@@ -130,7 +135,18 @@ export function brandSystemPrompt(original, opts = {}) {
                 "your priors. When you need more detail, or it does not cover something " +
                 "organization- or codebase-specific, call ask_yagni instead of guessing.";
     }
-    // 5. Closing reinforcement. Weak open-weight models weight the most recent
+    // 5. Repository rules — after the scrub (user content stays verbatim), before
+    // the closing reminder (which must remain last; see rulesSection's doc).
+    // The idempotency guard keys on the section's first line (its stable header,
+    // "## Repository rules (.claude/rules)") rather than the full content, so a
+    // re-brand stays single-section even if the rules BODY differs between
+    // passes — same shape as the BRIEF_HEADER guard above.
+    const rules = opts.rulesSection ?? undefined;
+    const rulesHeader = rules?.split("\n", 1)[0];
+    if (rules && rulesHeader && !s.includes(rulesHeader)) {
+        s = `${s}\n\n${rules}`;
+    }
+    // 6. Closing reinforcement. Weak open-weight models weight the most recent
     // instruction heavily, and the user's own project files may name other
     // harnesses; a trailing reminder keeps the agent from claiming one as its own.
     if (!s.includes(CLOSING_REMINDER)) {
@@ -142,15 +158,60 @@ export function brandSystemPrompt(original, opts = {}) {
 const CLOSING_REMINDER = "Reminder: you are YAGNI Code. If any text above names another coding agent, " +
     "assistant, or harness, it is not what you are or what you run on.";
 const BRIEF_HEADER = "=== HOW THIS COMPANY WORKS (live context from the YAGNI app) ===";
+/** The YAGNI brand blue used for the masthead art (truecolor #345cb8). */
+const BRAND_HEX = "#345cb8";
+/**
+ * Wrap a string in a 24-bit truecolor foreground. The pi Theme only paints
+ * named palette colors and can't express our exact brand hex, so the masthead
+ * art emits its own ANSI truecolor escape (matching every modern terminal).
+ */
+function paintBrand(s) {
+    const n = BRAND_HEX.replace("#", "");
+    const r = parseInt(n.slice(0, 2), 16);
+    const g = parseInt(n.slice(2, 4), 16);
+    const b = parseInt(n.slice(4, 6), 16);
+    return `\u001b[38;2;${r};${g};${b}m${s}\u001b[0m`;
+}
+/**
+ * The YAGNI wordmark as filled block-art: solid letter interiors (like the
+ * Claude robot silhouette), four rows, hand-tuned spacing. Each letter keeps
+ * its full form — A's carved counter, G's bowl, N's diagonal.
+ */
+const MASTHEAD_ART = [
+    "█   █ ▄▀▀▀▄ ▄▀▀▀▄ █▄  █ ▀█▀",
+    " ▀▄▀  █▄▄▄█ █ ▄▄▄ █ █ █  █",
+    "  █   █   █ ▀▄▄▄▀ █  ██ ▄█▄",
+];
 /**
  * Build the YAGNI Code startup masthead string, rendered into a header that
  * REPLACES pi's built-in startup banner (which otherwise shows "pi v<version>"
  * and a "Pi can explain its own features…" onboarding line). Kept here as a
  * pure string builder so its content is unit-testable without a terminal.
+ *
+ * Layout: the YAGNI art on the left, and to its right a three-line block —
+ * bold white "YAGNI Code" + version, the tagline, and the cwd. Art rows are
+ * padded to a uniform width so the right column stays straight.
  */
-export function buildMastheadString(theme) {
-    const wordmark = theme.bold(theme.fg("accent", "YAGNI CODE"));
-    const tagline = theme.fg("dim", "a terminal coding agent that knows your company");
-    return `${wordmark}\n${tagline}`;
+export function buildMastheadString(theme, opts = {}) {
+    const title = theme.bold(theme.fg("text", "YAGNI Code"));
+    const version = opts.version?.trim();
+    const line1 = version ? `${title} ${theme.fg("dim", `v${version}`)}` : title;
+    const right = [
+        line1,
+        theme.fg("dim", "a terminal coding agent that knows your company"),
+        ...(opts.cwd ? [theme.fg("dim", opts.cwd)] : []),
+    ];
+    const artW = Math.max(...MASTHEAD_ART.map((l) => [...l].length));
+    const gap = "  ";
+    const rows = Math.max(MASTHEAD_ART.length, right.length);
+    const lines = [];
+    for (let i = 0; i < rows; i++) {
+        const rawArt = MASTHEAD_ART[i] ?? "";
+        const pad = " ".repeat(Math.max(0, artW - [...rawArt].length));
+        const left = rawArt ? paintBrand(rawArt) + pad : " ".repeat(artW);
+        const r = right[i] ?? "";
+        lines.push(r ? `${left}${gap}${r}` : left);
+    }
+    return lines.join("\n");
 }
 //# sourceMappingURL=branding.js.map

package/dist/extension/chipEditor.d.ts

@@ -17,7 +17,7 @@
  * literal text is the source of truth, so undo/cursor/history and the
  * chip→model mapping all share one string and can't desync.
  *
- * Validated against pi 0.83.0 (spike, 5/5): the subclass owns `onPasteImage`
+ * Validated against pi 0.84.1 (spike, 5/5): the subclass owns `onPasteImage`
  * because `setCustomEditorComponent` only fills it when unset, and jiti aliases
  * the extension's pi imports to the same module instance pi uses.
  */
@@ -48,21 +48,33 @@ export interface ImageAttachment {
  */
 export declare function unwrapBracketedPaste(data: string): string | null;
 /**
- * Recognize a pasted temp-image path written by a Ghostty-based terminal
- * (cmux, Ghostty). Those terminals name the file `clipboard-<ts>-<id>.<ext>`
- * in the system temp dir. We match strictly — clipboard- prefix, image
- * extension, real file, and genuine image magic bytes — so a path the user
- * typed by hand (e.g. /Users/me/photo.png) is never silently converted.
- * Returns the decoded image, or null when the pasted text is not one such path.
+ * Recognize a pasted payload that is entirely image-file paths and decode
+ * them. This covers Finder/desktop drag-drop and Ghostty/cmux's Cmd+V
+ * (which writes the clipboard image to a `clipboard-*.png` temp file and
+ * pastes its path) — Claude Code parity: dragging a screenshot in becomes
+ * `[Image #N]`, not a literal path.
+ *
+ * Guardrails against converting something the user meant as text: the WHOLE
+ * paste must be path tokens, every token must be an absolute path to a real
+ * file with an image extension AND genuine image magic bytes, and the
+ * conversion is visible — each image becomes a chip in the prompt, so nothing
+ * is ever attached silently. Returns null when the paste is not such a list
+ * (caller passes it through as ordinary text).
  */
-export declare function readPastedImagePath(pastedText: string): ClipboardImage | null;
+export declare function readPastedImagePaths(pastedText: string): ClipboardImage[] | null;
 /**
  * Read an image from the system clipboard without forking pi. Best-effort and
- * cross-platform: native module if present, else osascript/pngpaste (macOS),
+ * cross-platform using ONLY tools the OS ships with: osascript (macOS),
  * wl-paste (Wayland), xclip (X11), PowerShell (Windows/WSL). Returns null when
  * the clipboard holds no image (caller then pastes text instead).
  */
 export declare const defaultClipboardImageReader: ClipboardImageReader;
+/**
+ * Parse osascript's clipboard dump — `«data PNGf89504E47…»` — into raw bytes.
+ * The 4-char tag after `«data ` is the pasteboard flavor; the hex that follows
+ * is the image. Returns null when the output holds no such dump.
+ */
+export declare function parseOsascriptImageHex(out: string): Uint8Array | null;
 /**
  * A CustomEditor that turns clipboard image paste into numbered `[Image #N]`
  * chips. The image bytes live here, keyed by chip number; the editor text holds
@@ -72,6 +84,15 @@ export declare class ChipEditor extends CustomEditor {
     private stashed;
     private chipCounter;
     private readImage;
+    /**
+     * Fired when Ctrl+C lands on an EMPTY prompt. pi's own binding is
+     * clear-editor (Esc is the interrupt key), but with nothing typed that is a
+     * no-op, and every peer harness (Claude Code, Codex, Gemini, Aider) treats a
+     * single Ctrl+C as "stop what's happening". The hook lets the extension wire
+     * exactly that; the key still falls through to pi afterwards, so clear and
+     * the 500ms double-press exit window are untouched.
+     */
+    onEmptyCtrlC?: () => void;
     constructor(tui: TUI, theme: EditorTheme, keybindings: ConstructorParameters<typeof CustomEditor>[2], readImage?: ClipboardImageReader, options?: EditorOptions);
     /**
      * Paste-image must fire on Cmd+V too (Claude Code parity on macOS). Claude