@yagni-app/code 0.3.5 → 1.0.0

package/dist/extension/crashReport.js

@@ -27,6 +27,7 @@
 import { spawn } from "node:child_process";
 import { scrubSecrets } from "./pipeline/scrubSecrets.js";
 import { isDesktopSurface } from "./surface.js";
+import { logEvent } from "./errorSink.js";
 const defaultSpawn = (command, args, options) => spawn(command, args, options);
 export const CRASH_REPORT_DISABLE_ENV = "YAGNI_DISABLE_CRASH_REPORTS";
 export const CRASH_REPORT_TIMEOUT_MS = 1_500;
@@ -267,6 +268,17 @@ export function reportFatalCrash(error, opts, context) {
 export function installUncaughtExceptionMonitor(opts, proc = process) {
     proc.on("uncaughtExceptionMonitor", (err) => {
         reportFatalCrash(err, opts, "uncaught-exception");
+        // Also seed the local error trail (best-effort): the crash report is a
+        // sanitized POST, but the ON-DISK trail is what /feedback binds for a
+        // report someone files next session.
+        logEvent({
+            source: "tool",
+            level: "error",
+            event: "uncaught_exception",
+            sessionId: process.env.YAGNI_SESSION_ID ?? undefined,
+            flush: "sync",
+            fields: { errorClass: err instanceof Error ? err.name || "Error" : "Error" },
+        });
     });
 }
 //# sourceMappingURL=crashReport.js.map

package/dist/extension/decisionCapture.js

@@ -17,6 +17,7 @@
  * never affects the tool call that triggered it.
  */
 import { bankDecision } from "./decisions.js";
+import { logEvent } from "./errorSink.js";
 /** At most one capture prompt per this window (spec: 10 minutes). */
 export const CAPTURE_DEBOUNCE_MS = 10 * 60 * 1000;
 /** Build a session-scoped decision capture (holds the debounce timestamp). */
@@ -54,10 +55,12 @@ export function makeDecisionCapture(deps) {
                     ctx.ui.notify("Saved the decision locally; it will sync automatically.", "info");
                 }
                 else {
+                    logEvent({ source: "decision-capture", level: "error", event: "capture_failed", fields: { kind: outcome.kind } });
                     ctx.ui.notify(outcome.message, "error");
                 }
             }
             catch {
+                logEvent({ source: "decision-capture", level: "error", event: "capture_failed", fields: { kind: "threw" } });
                 /* fail-soft: a capture failure never affects the tool call */
             }
         },

package/dist/extension/decisions.js

@@ -20,6 +20,7 @@
 import { randomUUID } from "node:crypto";
 import { friendlyFetchError, METERED_POST_FETCH_POLICY, resilientFetch } from "./resilientFetch.js";
 import { sendOrSpool } from "./spool.js";
+import { logEvent } from "./errorSink.js";
 /**
  * Bank a `cli_correction` decision durably. Every logical write carries a
  * generated idempotencyKey; the backend dedups on it, so a spool replay or a
@@ -151,6 +152,7 @@ export function registerDecisionCommands(pi, opts) {
                 notify(notice.message, notice.type);
             }
             catch (err) {
+                logEvent({ source: "decisions", level: "error", event: "decide_failed", fields: { op: "decide" } });
                 notify(`Could not record the decision: ${err instanceof Error ? err.message : String(err)}`, "error");
             }
         },
@@ -183,6 +185,7 @@ export function registerDecisionCommands(pi, opts) {
                     notify(`Superseded decision ${shortId(id)}.`, "info");
                 }
                 catch (err) {
+                    logEvent({ source: "decisions", level: "error", event: "supersede_failed", fields: { op: "supersede" } });
                     notify(`Could not supersede decision: ${err instanceof Error ? err.message : String(err)}`, "error");
                 }
                 return;
@@ -192,6 +195,7 @@ export function registerDecisionCommands(pi, opts) {
                 await pi.sendUserMessage(formatDecisionsList(items));
             }
             catch (err) {
+                logEvent({ source: "decisions", level: "error", event: "list_failed", fields: { op: "list" } });
                 notify(`Could not list decisions: ${err instanceof Error ? err.message : String(err)}`, "error");
             }
         },

package/dist/extension/diagnostics.d.ts

@@ -12,7 +12,9 @@
  * A support flow can tail this file and POST it with the user's consent;
  * nothing is uploaded automatically.
  */
+import { readSessionTrail } from "./errorSink.js";
 export declare function _setDiagnosticsHomeForTest(dir: string | null): void;
+/** Active unified-sink path (one rotating per-day JSONL for all sources). */
 export declare function diagnosticsLogPath(): string;
 /** Layer A: verbose mode. `YAGNI_DEBUG=1` (or "true") turns on extra detail. */
 export declare function isDebug(env?: NodeJS.ProcessEnv): boolean;
@@ -34,8 +36,37 @@ export interface ImagePasteEvent {
  * prompt. `detail` is included only when YAGNI_DEBUG is on.
  */
 export declare function logImagePaste(ev: ImagePasteEvent): void;
+export interface AskQuestionEvent {
+    /** Stable event name: "key" | "finish" | "render" | "navigate" | "other_key". */
+    event: string;
+    /** Human-readable description; NEVER user-typed text or question content. */
+    detail?: string;
+    selectedIndex?: number;
+    otherIndex?: number;
+    otherLen?: number;
+    /** Raw key bytes, escaped (only for debug). */
+    key?: string;
+    /** Resolution status when the tool finishes: selected | other | cancelled. */
+    status?: string;
+    /** Whether the question is multiSelect at the time of the event. */
+    multi?: boolean;
+    /** 0-based index of the question within this tool call (multi-question runs). */
+    qIndex?: number;
+    /** Whether the "Other" row auto-check is on (multi-select). */
+    checked?: boolean;
+}
+/**
+ * Append one sanitized ask-user-question event to the unified sink
+ * (source:"ask-question").
+ * Fail-soft and hermetically gated under `node --test` exactly like
+ * `logImagePaste`. Never logs user-typed text, question text, or option labels
+ * — only indices, lengths, key bytes (escaped), and resolution status.
+ */
+export declare function logAskQuestion(ev: AskQuestionEvent): void;
 /** Read the most recent log content (for a user-triggered report). */
 export declare function readRecentDiagnostics(maxBytes?: number): string;
 /** List existing diagnostic log files (active + rotations), for a report. */
 export declare function listDiagnosticFiles(): string[];
+/** Re-export the session-scoped trail reader for the /feedback flow. */
+export { readSessionTrail };
 //# sourceMappingURL=diagnostics.d.ts.map

package/dist/extension/diagnostics.js

@@ -12,22 +12,19 @@
  * A support flow can tail this file and POST it with the user's consent;
  * nothing is uploaded automatically.
  */
-import { appendFileSync, mkdirSync, readFileSync, readdirSync, renameSync, statSync, } from "node:fs";
+import { readFileSync, readdirSync } from "node:fs";
 import { dirname, join, basename } from "node:path";
-import { codeStateHome } from "./stateHome.js";
+import { _setErrorSinkHomeForTest, logEvent, errorSinkPath, readSessionTrail, } from "./errorSink.js";
 /** Test seam (mirrors `_setYagniCodeHomeForTest`): point the log at a tmpdir. */
 let homeOverride = null;
 export function _setDiagnosticsHomeForTest(dir) {
     homeOverride = dir;
+    _setErrorSinkHomeForTest(dir);
 }
-function yagniCodeHome() {
-    return codeStateHome(homeOverride);
-}
+/** Active unified-sink path (one rotating per-day JSONL for all sources). */
 export function diagnosticsLogPath() {
-    return join(yagniCodeHome(), "logs", "image-paste.log");
+    return errorSinkPath();
 }
-const MAX_LOG_BYTES = 256 * 1024; // rotate the active file past this
-const KEEP_ROTATIONS = 2; // keep image-paste.log.1 and .2 alongside the active file
 /** Layer A: verbose mode. `YAGNI_DEBUG=1` (or "true") turns on extra detail. */
 export function isDebug(env = process.env) {
     const v = env.YAGNI_DEBUG;
@@ -38,56 +35,55 @@ export function isDebug(env = process.env) {
  * prompt. `detail` is included only when YAGNI_DEBUG is on.
  */
 export function logImagePaste(ev) {
-    try {
-        // Hermetic under `node --test`: never touch the real home dir from a test
-        // unless the test explicitly stubbed it. Writing to the runner's $HOME made
-        // the suite's exit depend on the CI filesystem (a read-only or slow $HOME
-        // could stall the write), which is the leading suspect for the CI hang.
-        if (process.env.NODE_TEST_CONTEXT && homeOverride === null)
-            return;
-        const line = {
-            ts: new Date().toISOString(),
-            event: ev.event,
-            ...(ev.outcome !== undefined ? { outcome: ev.outcome } : {}),
-            ...(ev.mimeType !== undefined ? { mimeType: ev.mimeType } : {}),
-            ...(ev.bytes !== undefined ? { bytes: ev.bytes } : {}),
-            ...(ev.imageCount !== undefined ? { imageCount: ev.imageCount } : {}),
-            ...(ev.file !== undefined ? { file: basename(ev.file) } : {}),
-            ...(ev.detail !== undefined && isDebug() ? { detail: ev.detail } : {}),
-        };
-        const path = diagnosticsLogPath();
-        mkdirSync(dirname(path), { recursive: true });
-        rotateIfNeeded(path);
-        appendFileSync(path, JSON.stringify(line) + "\n", "utf8");
-    }
-    catch {
-        /* logging must never throw into the editor */
-    }
+    // The unified sink: content-free fields are always-on, the debug-only
+    // `detail` rides level:debug (never bound by /feedback).
+    const fields = {
+        ...(ev.outcome !== undefined ? { outcome: ev.outcome } : {}),
+        ...(ev.mimeType !== undefined ? { mimeType: ev.mimeType } : {}),
+        ...(ev.bytes !== undefined ? { bytes: ev.bytes } : {}),
+        ...(ev.imageCount !== undefined ? { imageCount: ev.imageCount } : {}),
+        ...(ev.file !== undefined ? { file: basename(ev.file) } : {}),
+        ...(ev.detail !== undefined && isDebug() ? { detail: ev.detail } : {}),
+    };
+    logEvent({
+        source: "image-paste",
+        level: ev.detail !== undefined && isDebug() ? "debug" : "info",
+        event: ev.event,
+        fields,
+    });
 }
-/** Shift image-paste.log -> .1 -> .2 once the active file passes the size cap. */
-function rotateIfNeeded(path) {
-    try {
-        if (statSync(path).size < MAX_LOG_BYTES)
-            return;
-        for (let i = KEEP_ROTATIONS; i >= 1; i--) {
-            const from = i === 1 ? path : `${path}.${i - 1}`;
-            const to = `${path}.${i}`;
-            try {
-                renameSync(from, to);
-            }
-            catch {
-                /* absent source — fine */
-            }
-        }
-    }
-    catch {
-        /* rotation is best-effort */
-    }
+/**
+ * Append one sanitized ask-user-question event to the unified sink
+ * (source:"ask-question").
+ * Fail-soft and hermetically gated under `node --test` exactly like
+ * `logImagePaste`. Never logs user-typed text, question text, or option labels
+ * — only indices, lengths, key bytes (escaped), and resolution status.
+ */
+export function logAskQuestion(ev) {
+    // Raw key bytes (`key`) and `detail` are content-ish: gate them behind DEBUG.
+    const debug = isDebug();
+    const fields = {
+        ...(ev.selectedIndex !== undefined ? { selectedIndex: ev.selectedIndex } : {}),
+        ...(ev.otherIndex !== undefined ? { otherIndex: ev.otherIndex } : {}),
+        ...(ev.otherLen !== undefined ? { otherLen: ev.otherLen } : {}),
+        ...(ev.status !== undefined ? { status: ev.status } : {}),
+        ...(ev.multi !== undefined ? { multi: ev.multi } : {}),
+        ...(ev.qIndex !== undefined ? { qIndex: ev.qIndex } : {}),
+        ...(ev.checked !== undefined ? { checked: ev.checked } : {}),
+        ...(ev.key !== undefined && debug ? { key: ev.key } : {}),
+        ...(ev.detail !== undefined && debug ? { detail: ev.detail } : {}),
+    };
+    logEvent({
+        source: "ask-question",
+        level: debug ? "debug" : "info",
+        event: ev.event,
+        fields,
+    });
 }
 /** Read the most recent log content (for a user-triggered report). */
 export function readRecentDiagnostics(maxBytes = 64 * 1024) {
     try {
-        const data = readFileSync(diagnosticsLogPath(), "utf8");
+        const data = readFileSync(errorSinkPath(), "utf8");
         return data.length > maxBytes ? data.slice(data.length - maxBytes) : data;
     }
     catch {
@@ -97,9 +93,9 @@ export function readRecentDiagnostics(maxBytes = 64 * 1024) {
 /** List existing diagnostic log files (active + rotations), for a report. */
 export function listDiagnosticFiles() {
     try {
-        const dir = dirname(diagnosticsLogPath());
+        const dir = dirname(errorSinkPath());
         return readdirSync(dir)
-            .filter((f) => f.startsWith("image-paste.log"))
+            .filter((f) => f.startsWith("errors-"))
             .sort()
             .map((f) => join(dir, f));
     }
@@ -107,4 +103,6 @@ export function listDiagnosticFiles() {
         return [];
     }
 }
+/** Re-export the session-scoped trail reader for the /feedback flow. */
+export { readSessionTrail };
 //# sourceMappingURL=diagnostics.js.map

package/dist/extension/errorSink.d.ts

@@ -0,0 +1,64 @@
+/**
+ * The unified local error/log sink for YAGNI Code (YAG-580).
+ *
+ * Replaces the previous ~9 hand-rolled JSONL writers (image-paste.log,
+ * ask-question.log, turn-lifecycle.log, guardian.log, cost-divergence.log,
+ * auth-events.log, cmux-bridge.log, hooks.log, *.stream.log) with ONE sink.
+ *
+ * Two storage layers, purpose-named so their roles stay clear:
+ *
+ *  1. The DURABLE TRAIL — one rotating per-day JSONL under
+ *     `~/.yagni-code/logs/errors-<date>.jsonl` (size-capped + 2 rotations).
+ *     This is the crash-survivable WAL: `turn_start` without a matching
+ *     `turn_end` still leaves a record even if the process is killed. Critical
+ *     events append SYNCHRONOUSLY for exactly that reason.
+ *
+ *  2. The IN-MEMORY CAPTURE — a byte-budgeted ring buffer (not line-counted).
+ *     This is the `/feedback` binding convenience, NOT durability (an in-memory
+ *     ring does not survive a crash). Mirrors Codex's CodexFeedback ring and
+ *     Claude's inMemoryErrorLog.
+ *
+ * Every line carries a REQUIRED `sessionId` and a `source`/`level`/`event`
+ * triple so a shared file stays filterable: `jq 'select(.source=="guardian")'`
+ * reproduces today's per-file tail exactly, and /feedback reads the trail
+ * filtered by sessionId (never the raw file) so one session's report never
+ * leaks another session's errors.
+ *
+ * Default-on vs DEBUG invariant (the thing that makes "log everything by
+ * default" safe): default-on == scrub-safe == upload-safe. Any field carrying
+ * raw content (tool arguments, partial/result bodies, provider payloads, raw
+ * key bytes) must be gated behind YAGNI_DEBUG, and the /feedback reader refuses
+ * to bind any `level: debug` line. DEBUG == may-contain-content == never-uploads.
+ */
+export type SinkLevel = "error" | "warn" | "info" | "debug";
+export interface SinkEvent {
+    /** Former filename / subsystem: tool, turn, guardian, image-paste, ask-question, cost, auth, cmux, hooks. */
+    source: string;
+    level: SinkLevel;
+    /** Stable machine name (e.g. "turn_start", "bash.exit_1", "denied"). */
+    event: string;
+    /** Additional structured fields. NEVER raw content on a non-debug line. */
+    fields?: Record<string, unknown>;
+    /** Session id so the trail is filterable and scoped per feedback. Defaults to YAGNI_SESSION_ID. */
+    sessionId?: string;
+    /** "sync" flushes immediately (critical events); "buffered" is fine for high-volume debug. */
+    flush?: "sync" | "buffered";
+}
+export declare function _setErrorSinkHomeForTest(dir: string | null): void;
+export declare function errorSinkPath(now?: Date): string;
+export declare function _clearErrorSinkRingForTest(): void;
+export declare function errorSinkInMemory(): string;
+/**
+ * Append one event to both the ring and the durable trail. Fail-soft: a logging
+ * failure must never break the session. `flush: "sync"` (default for
+ * error-level events and lifecycle turns) bypasses any future buffering so a
+ * turn that starts but never ends still leaves a durable `turn_start`.
+ */
+export declare function logEvent(ev: SinkEvent): void;
+/**
+ * Read recent trail lines for ONE session, filtered by `sessionId`, up to
+ * `maxBytes`. Never returns `level: debug` lines — the default-on tier is the
+ * upload-safe tier, and DEBUG may contain content that must not leave the machine.
+ */
+export declare function readSessionTrail(sessionId: string, maxBytes?: number): string;
+//# sourceMappingURL=errorSink.d.ts.map

package/dist/extension/errorSink.js

@@ -0,0 +1,180 @@
+/**
+ * The unified local error/log sink for YAGNI Code (YAG-580).
+ *
+ * Replaces the previous ~9 hand-rolled JSONL writers (image-paste.log,
+ * ask-question.log, turn-lifecycle.log, guardian.log, cost-divergence.log,
+ * auth-events.log, cmux-bridge.log, hooks.log, *.stream.log) with ONE sink.
+ *
+ * Two storage layers, purpose-named so their roles stay clear:
+ *
+ *  1. The DURABLE TRAIL — one rotating per-day JSONL under
+ *     `~/.yagni-code/logs/errors-<date>.jsonl` (size-capped + 2 rotations).
+ *     This is the crash-survivable WAL: `turn_start` without a matching
+ *     `turn_end` still leaves a record even if the process is killed. Critical
+ *     events append SYNCHRONOUSLY for exactly that reason.
+ *
+ *  2. The IN-MEMORY CAPTURE — a byte-budgeted ring buffer (not line-counted).
+ *     This is the `/feedback` binding convenience, NOT durability (an in-memory
+ *     ring does not survive a crash). Mirrors Codex's CodexFeedback ring and
+ *     Claude's inMemoryErrorLog.
+ *
+ * Every line carries a REQUIRED `sessionId` and a `source`/`level`/`event`
+ * triple so a shared file stays filterable: `jq 'select(.source=="guardian")'`
+ * reproduces today's per-file tail exactly, and /feedback reads the trail
+ * filtered by sessionId (never the raw file) so one session's report never
+ * leaks another session's errors.
+ *
+ * Default-on vs DEBUG invariant (the thing that makes "log everything by
+ * default" safe): default-on == scrub-safe == upload-safe. Any field carrying
+ * raw content (tool arguments, partial/result bodies, provider payloads, raw
+ * key bytes) must be gated behind YAGNI_DEBUG, and the /feedback reader refuses
+ * to bind any `level: debug` line. DEBUG == may-contain-content == never-uploads.
+ */
+import { appendFileSync, mkdirSync, readFileSync, renameSync, statSync } from "node:fs";
+import { dirname, join } from "node:path";
+import { codeStateHome } from "./stateHome.js";
+import { scrubSecrets } from "./pipeline/scrubSecrets.js";
+const MAX_LOG_BYTES = 256 * 1024;
+const KEEP_ROTATIONS = 2;
+const RING_MAX_BYTES = 256 * 1024;
+/** Test seam: point the log at a tmpdir (mirrors _setDiagnosticsHomeForTest). */
+let homeOverride = null;
+export function _setErrorSinkHomeForTest(dir) {
+    homeOverride = dir;
+}
+function logDir() {
+    return join(codeStateHome(homeOverride), "logs");
+}
+function dayStamp(now = new Date()) {
+    return now.toISOString().slice(0, 10); // YYYY-MM-DD
+}
+export function errorSinkPath(now = new Date()) {
+    return join(logDir(), `errors-${dayStamp(now)}.jsonl`);
+}
+/** In-memory ring buffer, byte-budgeted (trailing bytes kept when over cap). */
+class RingBuffer {
+    maxBytes;
+    chunks = [];
+    bytes = 0;
+    constructor(maxBytes) {
+        this.maxBytes = maxBytes;
+    }
+    push(line) {
+        const b = Buffer.byteLength(line, "utf8");
+        if (b >= this.maxBytes) {
+            this.chunks = [line];
+            this.bytes = b;
+            return;
+        }
+        while (this.bytes + b > this.maxBytes && this.chunks.length > 0) {
+            const dropped = this.chunks.shift();
+            this.bytes -= Buffer.byteLength(dropped, "utf8");
+        }
+        this.chunks.push(line);
+        this.bytes += b;
+    }
+    snapshot() {
+        return this.chunks.join("");
+    }
+    clear() {
+        this.chunks = [];
+        this.bytes = 0;
+    }
+}
+const ring = new RingBuffer(RING_MAX_BYTES);
+export function _clearErrorSinkRingForTest() {
+    ring.clear();
+}
+export function errorSinkInMemory() {
+    return ring.snapshot();
+}
+function rotateIfNeeded(path) {
+    try {
+        if (!statSync(path).isFile() || statSync(path).size < MAX_LOG_BYTES)
+            return;
+        for (let i = KEEP_ROTATIONS; i >= 1; i--) {
+            const from = i === 1 ? path : `${path}.${i - 1}`;
+            const to = `${path}.${i}`;
+            try {
+                renameSync(from, to);
+            }
+            catch {
+                /* absent source — fine */
+            }
+        }
+    }
+    catch {
+        /* rotation is best-effort */
+    }
+}
+function isDebug(env = process.env) {
+    const v = env.YAGNI_DEBUG;
+    return v === "1" || v === "true";
+}
+function sessionIdFor(ev) {
+    return ev.sessionId ?? process.env.YAGNI_SESSION_ID ?? "";
+}
+function serialize(ev) {
+    const line = {
+        ts: new Date().toISOString(),
+        source: ev.source,
+        level: ev.level,
+        event: ev.event,
+        sessionId: sessionIdFor(ev),
+        ...(ev.fields ?? {}),
+    };
+    return JSON.stringify(line) + "\n";
+}
+/**
+ * Append one event to both the ring and the durable trail. Fail-soft: a logging
+ * failure must never break the session. `flush: "sync"` (default for
+ * error-level events and lifecycle turns) bypasses any future buffering so a
+ * turn that starts but never ends still leaves a durable `turn_start`.
+ */
+export function logEvent(ev) {
+    try {
+        if (process.env.NODE_TEST_CONTEXT && homeOverride === null)
+            return;
+        const line = serialize(ev);
+        ring.push(line);
+        const path = errorSinkPath();
+        mkdirSync(dirname(path), { recursive: true });
+        rotateIfNeeded(path);
+        appendFileSync(path, line, "utf8");
+    }
+    catch {
+        /* logging must never throw into the editor */
+    }
+}
+/**
+ * Read recent trail lines for ONE session, filtered by `sessionId`, up to
+ * `maxBytes`. Never returns `level: debug` lines — the default-on tier is the
+ * upload-safe tier, and DEBUG may contain content that must not leave the machine.
+ */
+export function readSessionTrail(sessionId, maxBytes = 64 * 1024) {
+    try {
+        const data = readFileSync(errorSinkPath(), "utf8");
+        // Defense-in-depth: scrub each kept line so /diagnostics and /feedback
+        // never surface a secret or local path, even if a future caller slipped a
+        // content-bearing value onto an always-on line.
+        const lines = data
+            .split("\n")
+            .filter((l) => l.length > 0)
+            .filter((l) => {
+            try {
+                const obj = JSON.parse(l);
+                return obj.sessionId === sessionId && obj.level !== "debug";
+            }
+            catch {
+                return false;
+            }
+        })
+            .map((l) => scrubSecrets(l))
+            .join("\n");
+        return lines.length > maxBytes ? lines.slice(lines.length - maxBytes) : lines;
+    }
+    catch {
+        return "";
+    }
+}
+//# sourceMappingURL=errorSink.js.map

package/dist/extension/feedbackCommand.d.ts

@@ -0,0 +1,38 @@
+/**
+ * The `/feedback` (alias `/bug`) command + the `/diagnostics` companion
+ * (YAG-580).
+ *
+ * `/feedback` captures the session transcript (pi's own append-only JSONL, read
+ * via `ctx.sessionManager` — never reconstructed from YAGNI_SESSION_ID, which is
+ * the proxy-attribution id), the session-scoped error trail (from the unified
+ * sink), any child `/go` run transcripts, a sanitized `yagni doctor` report, and
+ * git metadata — sanitizes the whole bundle with the shared scrub contract, and
+ * POSTs it to the YAGNI backend (opt-in, gated, named-human).
+ *
+ * `/diagnostics` is the read-only companion: it prints the last N sink lines for
+ * THIS session so the user can see what failed before deciding to attach it.
+ *
+ * Both treat the transcript as best-effort enrichment: pi's flush-to-file timing
+ * at command-invoke is not guaranteed, so a report never claims the reporting
+ * turn is captured unless it verifiably is.
+ */
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+export interface FeedbackDeps {
+    baseUrl: string;
+    getToken: () => string | undefined;
+    fetchImpl?: typeof fetch;
+    env?: NodeJS.ProcessEnv;
+    /** Sanitized `yagni doctor` output (string), or undefined to omit. */
+    getDoctorReport?: () => Promise<string | undefined>;
+    /** Git facts for the report; undefined fields are omitted. */
+    getGitState?: (cwd: string) => Promise<{
+        branch?: string;
+        commit?: string;
+        remote?: string;
+        dirty?: boolean;
+    }>;
+    /** Child `/go` run transcripts keyed by run id, for the current run tree. */
+    getChildTranscripts?: (cwd: string, sessionFile: string | undefined) => Promise<Record<string, string>>;
+}
+export declare function registerFeedbackCommands(pi: ExtensionAPI, deps: FeedbackDeps): void;
+//# sourceMappingURL=feedbackCommand.d.ts.map