@code-yeongyu/senpi-tui 2026.10.19 → 2026.10.20-1

package/dist/autocomplete.js

@@ -168,7 +168,8 @@ async function walkDirectoryWithFd(baseDir, fdPath, query, maxResults, signal, m
             resolve(results);
         };
         const onAbort = () => {
-            if (child.exitCode === null) {
+            // A child that never started has no pid; on Node < 24.22 / 26.10 its kill() signals our own process group.
+            if (child.pid !== undefined && child.exitCode === null) {
                 child.kill("SIGKILL");
             }
         };

package/dist/kitty-graphics-probe.d.ts

@@ -0,0 +1,21 @@
+/**
+ * Kitty graphics support probe for multiplexer panes whose environment says
+ * nothing about image support (herdr; senpi #3698).
+ *
+ * A 1x1 RGBA direct-transmission query (`a=q`) is written to the terminal; a
+ * terminal that implements the Kitty graphics protocol replies with an APC
+ * response naming the query's image id. Silence, an error payload, or a reply
+ * for another id all mean "no graphics" — the caller must then keep the text
+ * fallback instead of emitting graphics sequences blindly.
+ */
+export declare const KITTY_GRAPHICS_PROBE_TIMEOUT_MS = 500;
+/** `AAAA` is base64 for four zero bytes: one 1x1 RGBA pixel. */
+export declare function buildKittyGraphicsQuery(imageId: number): string;
+export type KittyGraphicsProbeOutcome = "ok" | "err";
+/**
+ * Classify one APC chunk as the answer to our query. Returns null when the
+ * chunk is not a Kitty graphics response for `imageId`, so unrelated APC
+ * traffic (graphics commands from other writers) is left untouched.
+ */
+export declare function parseKittyGraphicsProbeResponse(data: string, imageId: number): KittyGraphicsProbeOutcome | null;
+//# sourceMappingURL=kitty-graphics-probe.d.ts.map

package/dist/kitty-graphics-probe.js

@@ -0,0 +1,37 @@
+/**
+ * Kitty graphics support probe for multiplexer panes whose environment says
+ * nothing about image support (herdr; senpi #3698).
+ *
+ * A 1x1 RGBA direct-transmission query (`a=q`) is written to the terminal; a
+ * terminal that implements the Kitty graphics protocol replies with an APC
+ * response naming the query's image id. Silence, an error payload, or a reply
+ * for another id all mean "no graphics" — the caller must then keep the text
+ * fallback instead of emitting graphics sequences blindly.
+ */
+export const KITTY_GRAPHICS_PROBE_TIMEOUT_MS = 500;
+/** `AAAA` is base64 for four zero bytes: one 1x1 RGBA pixel. */
+export function buildKittyGraphicsQuery(imageId) {
+    return `\x1b_Gi=${imageId},s=1,v=1,a=q,t=d,f=24;AAAA\x1b\\`;
+}
+/**
+ * Classify one APC chunk as the answer to our query. Returns null when the
+ * chunk is not a Kitty graphics response for `imageId`, so unrelated APC
+ * traffic (graphics commands from other writers) is left untouched.
+ */
+export function parseKittyGraphicsProbeResponse(data, imageId) {
+    const start = "\x1b_G";
+    const end = "\x1b\\";
+    if (!data.startsWith(start) || !data.endsWith(end))
+        return null;
+    const body = data.slice(start.length, -end.length);
+    const separator = body.indexOf(";");
+    if (separator === -1)
+        return null;
+    const keys = body.slice(0, separator);
+    const message = body.slice(separator + 1);
+    const idMatch = /(?:^|,)i=(\d+)(?:,|$)/.exec(keys);
+    if (!idMatch || Number.parseInt(idMatch[1] ?? "", 10) !== imageId)
+        return null;
+    return message.startsWith("OK") ? "ok" : "err";
+}
+//# sourceMappingURL=kitty-graphics-probe.js.map

package/dist/terminal-capabilities.d.ts

@@ -8,6 +8,8 @@ export interface TerminalCapabilities {
     kittyUnicodePlaceholders?: boolean;
     /** Shown next to the image placeholder when the terminal could show images after a user action. */
     imageHint?: string;
+    /** Set when image support is still unknown from the environment: the TUI must query the terminal and record the answer via `setProbedImageProtocol`. */
+    imageProbe?: "kitty";
 }
 export interface DetectedTerminalCapabilities extends TerminalCapabilities {
     cellDimensions?: {

package/dist/terminal-image.d.ts

@@ -1,5 +1,5 @@
 import type { TerminalColorMode } from "./colors.ts";
-import { type DetectedTerminalCapabilities, type TerminalCapabilities } from "./terminal-capabilities.ts";
+import { type DetectedTerminalCapabilities, type ImageProtocol, type TerminalCapabilities } from "./terminal-capabilities.ts";
 export type { ImageProtocol, TerminalCapabilities } from "./terminal-capabilities.ts";
 export { outerKittyGraphicsMode } from "./terminal-capabilities.ts";
 export interface TmuxPassthroughState {
@@ -27,6 +27,8 @@ export interface ImageRenderOptions {
 }
 export declare function getCellDimensions(): CellDimensions;
 export declare function setCellDimensions(dims: CellDimensions): void;
+/** Record the TUI probe's answer for probe-requesting terminals. `undefined` re-arms the probe. */
+export declare function setProbedImageProtocol(protocol: ImageProtocol | undefined): void;
 export declare function detectCapabilities(tmuxForwardsHyperlink?: () => boolean, tmuxPassthroughState?: () => TmuxPassthroughState): DetectedTerminalCapabilities;
 export declare function getCapabilities(): TerminalCapabilities;
 export declare function getTerminalColorMode(capabilities?: TerminalCapabilities): TerminalColorMode;

package/dist/terminal-image.js

@@ -7,6 +7,10 @@ import { isOldWindowsWezTerm, WEZTERM_WINDOWS_IMAGES_HINT } from "./wezterm-wind
 export { outerKittyGraphicsMode } from "./terminal-capabilities.js";
 let cachedCapabilities = null;
 let capabilityOverrides = {};
+// Answer of the TUI's Kitty graphics probe, recorded once per process. It
+// outlives cachedCapabilities because focus events reset that cache and
+// re-detect from the environment (senpi #3698).
+let probedImageProtocol;
 // Default cell dimensions - updated by TUI when terminal responds to query
 let cellDimensions = { widthPx: 9, heightPx: 18 };
 export function getCellDimensions() {
@@ -74,6 +78,17 @@ function detectCapabilitiesFromEnvironment(tmuxForwardsHyperlink) {
     if (terminalEmulator === "jetbrains-jediterm") {
         return { images: null, trueColor: true, hyperlinks: false };
     }
+    // herdr panes: the environment says nothing about the outer terminal's graphics support, and herdr
+    // itself leaks graphics sequences when the outer terminal cannot render them. Detection defers to
+    // the TUI's Kitty graphics probe and latches its answer (senpi #3698).
+    if (termProgram === "herdr" || process.env.HERDR_ENV === "1") {
+        return {
+            images: probedImageProtocol ?? null,
+            trueColor: hasTrueColorHint,
+            hyperlinks: false,
+            ...(probedImageProtocol === undefined ? { imageProbe: "kitty" } : {}),
+        };
+    }
     // Windows Terminal does not always set WT_SESSION, for example when it hosts
     // a cmd.exe launched directly from Win+R. Modern Windows consoles support
     // truecolor; keep hyperlinks off unless we positively detected support above.
@@ -89,9 +104,17 @@ function detectCapabilitiesFromEnvironment(tmuxForwardsHyperlink) {
 function parseBooleanCapabilityOverride(value) {
     return value === "1" ? true : value === "0" ? false : undefined;
 }
-// The hint explains why detection turned images off; an explicit image choice makes it wrong.
+/** Record the TUI probe's answer for probe-requesting terminals. `undefined` re-arms the probe. */
+export function setProbedImageProtocol(protocol) {
+    if (probedImageProtocol === protocol)
+        return;
+    probedImageProtocol = protocol;
+    cachedCapabilities = null;
+}
+// The hint explains why detection turned images off; an explicit image choice makes it wrong. The
+// probe flag is stripped for the same reason: an explicit choice settles the question, no probe needed.
 function withoutImageHint(caps) {
-    const { imageHint: _imageHint, ...rest } = caps;
+    const { imageHint: _imageHint, imageProbe: _imageProbe, ...rest } = caps;
     return rest;
 }
 export function detectCapabilities(tmuxForwardsHyperlink, tmuxPassthroughState) {

package/dist/tui.d.ts

@@ -363,6 +363,7 @@ export declare abstract class TuiBase extends Container {
      * replies belong to the oldest one. Queries stay here after a timeout to collect late replies.
      */
     private pendingTerminalColorQueries;
+    private pendingKittyGraphicsProbe?;
     private terminalColorSchemeListeners;
     private terminalColorSchemeNotificationsEnabled;
     /** Directory for debug/crash logs. The fork keeps a concrete default (`~/.senpi/agent`) so PI_DEBUG_REDRAW and crash dumps stay in the agent directory. */
@@ -470,6 +471,9 @@ export declare abstract class TuiBase extends Container {
     onTerminalColorSchemeChange(listener: (scheme: TerminalColorScheme) => void): () => void;
     setTerminalColorSchemeNotifications(enabled: boolean): void;
     private queryCellSize;
+    private probeKittyGraphics;
+    private consumeKittyGraphicsProbeResponse;
+    private resolveKittyGraphicsProbe;
     stop(options?: TuiStopOptions): void;
     /**
      * While a reply streams, a frame that changes rows above the viewport would replay the whole scrollback

package/dist/tui.js

@@ -7,9 +7,10 @@ import * as os from "node:os";
 import * as path from "node:path";
 import { performance } from "node:perf_hooks";
 import { isKeyRelease, matchesKey } from "./keys.js";
+import { buildKittyGraphicsQuery, KITTY_GRAPHICS_PROBE_TIMEOUT_MS, parseKittyGraphicsProbeResponse, } from "./kitty-graphics-probe.js";
 import { isMultiplexerSession, useLegacyMuxRender, viewportRenderEnabled } from "./mux.js";
 import { parseOscColorResponse, parseTerminalColorSchemeReport, } from "./terminal-colors.js";
-import { deleteKittyImage, getCapabilities, isImageLine, resetCapabilitiesCache, setCellDimensions, } from "./terminal-image.js";
+import { allocateImageId, deleteKittyImage, getCapabilities, isImageLine, resetCapabilitiesCache, setCellDimensions, setProbedImageProtocol, } from "./terminal-image.js";
 import { consumeTmuxFocusEvent, DISABLE_FOCUS_REPORTING, ENABLE_FOCUS_REPORTING } from "./tmux-focus.js";
 import { extractSegments, normalizeTerminalOutput, sliceByColumn, sliceWithWidth, visibleWidth } from "./utils.js";
 import { planViewportInsertScroll } from "./viewport-insert-scroll.js";
@@ -1242,6 +1243,7 @@ export class TuiBase extends Container {
             this.terminal.write("\x1b[?2031h");
         }
         this.queryCellSize();
+        this.probeKittyGraphics();
         this.requestRender();
     }
     addInputListener(listener) {
@@ -1277,6 +1279,40 @@ export class TuiBase extends Container {
         // Response format: CSI 6 ; height ; width t
         this.terminal.write("\x1b[16t");
     }
+    probeKittyGraphics() {
+        if (this.pendingKittyGraphicsProbe)
+            return;
+        if (getCapabilities().imageProbe !== "kitty")
+            return;
+        const imageId = allocateImageId();
+        const timer = setTimeout(() => this.resolveKittyGraphicsProbe(false), KITTY_GRAPHICS_PROBE_TIMEOUT_MS);
+        this.pendingKittyGraphicsProbe = { imageId, timer };
+        this.terminal.write(buildKittyGraphicsQuery(imageId));
+    }
+    consumeKittyGraphicsProbeResponse(data) {
+        const probe = this.pendingKittyGraphicsProbe;
+        if (!probe)
+            return false;
+        const outcome = parseKittyGraphicsProbeResponse(data, probe.imageId);
+        if (outcome === null)
+            return false;
+        this.resolveKittyGraphicsProbe(outcome === "ok");
+        return true;
+    }
+    resolveKittyGraphicsProbe(supported) {
+        const probe = this.pendingKittyGraphicsProbe;
+        if (!probe)
+            return;
+        this.pendingKittyGraphicsProbe = undefined;
+        clearTimeout(probe.timer);
+        setProbedImageProtocol(supported ? "kitty" : null);
+        if (this.stopped)
+            return;
+        if (supported)
+            this.queryCellSize();
+        this.invalidate();
+        this.requestRender();
+    }
     stop(options = {}) {
         this.stopped = true;
         // The hold itself belongs to the turn (interactive mode sets and releases it); a stop()/start() handover
@@ -1287,6 +1323,10 @@ export class TuiBase extends Container {
         this.renderRequested = false;
         this.inputRenderPending = false;
         this.cancelRenderTimer();
+        if (this.pendingKittyGraphicsProbe) {
+            clearTimeout(this.pendingKittyGraphicsProbe.timer);
+            this.pendingKittyGraphicsProbe = undefined;
+        }
         if (this.terminalColorSchemeNotificationsEnabled) {
             this.terminal.write("\x1b[?2031l");
         }
@@ -1493,6 +1533,9 @@ export class TuiBase extends Container {
         if (this.consumeTerminalColorSchemeReport(data)) {
             return;
         }
+        if (this.consumeKittyGraphicsProbeResponse(data)) {
+            return;
+        }
         // A key press means the user is at the bottom again. Terminal reports (mouse/wheel, OSC/DCS replies,
         // DEC private reports, window and cell-size reports) are not the user and must not trigger it.
         if (!isTerminalReport(data)) {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@code-yeongyu/senpi-tui",
-	"version": "2026.10.19",
+	"version": "2026.10.20-1",
 	"description": "Terminal User Interface library with differential rendering for efficient text-based applications",
 	"type": "module",
 	"main": "dist/index.js",