@heyhuynhgiabuu/pi-task 0.1.2 → 0.1.4

package/CHANGELOG.md

@@ -4,6 +4,51 @@ All notable changes to `@heyhuynhgiabuu/pi-task` are documented here.
 The format is based on [Keep a Changelog](https://keepachangelog.com/),
 and this project adheres to [Semantic Versioning](https://semver.org/).
 
+## [0.1.4] — 2026-06-21
+
+### Fixed
+
+- Detect the current tmux pane size before launching a task pane and choose
+  the split direction based on available space: side-by-side for wide panes,
+  stacked for narrow panes.
+- Target the exact pane that was measured when running `tmux split-window`,
+  avoiding focus races where a different pane could be split.
+- Apply the same pane-size-aware split logic to the subagent tmux helper.
+
+### Verified
+
+- `npm test` passes
+- `npm run typecheck` passes
+- `npm run build` passes
+- `npm run smoke` passes
+- `npm pack --dry-run` succeeds
+- Real tmux integration check passed for narrow `120x40` and wide `200x40`
+  sessions.
+
+[0.1.4]: https://github.com/heyhuynhgiabuu/pi-task/releases/tag/v0.1.4
+
+## [0.1.3] — 2026-06-21
+
+### Fixed
+
+- Replaced tmux task startup via `send-keys` with direct
+  `split-window <command>` execution so long, quoted `pi ...` launch
+  commands are not truncated or interrupted by terminal input buffering.
+- Hardened tmux steering/follow-up text injection by using tmux buffers
+  (`load-buffer` + `paste-buffer`) instead of typing long text via
+  `send-keys`.
+
+### Verified
+
+- `npm test` passes
+- `npm run typecheck` passes
+- `npm run build` passes
+- `npm run smoke` passes
+- Long task-tool tmux launch stress test passed with quotes, backticks,
+  shell expansions, redirects, newlines, unicode, and long prompt text.
+
+[0.1.3]: https://github.com/heyhuynhgiabuu/pi-task/releases/tag/v0.1.3
+
 ## [0.1.2] — 2025
 
 ### Fixed

package/dist/helpers.d.ts

@@ -50,7 +50,9 @@ export declare function parseResultXml(raw: string): ParsedResult;
 export declare function formatMs(ms: number): string;
 export declare function parseIdTimestamp(id: string): number;
 export declare function shellQuote(value: string): string;
-export declare function buildTmuxSendKeysArgs(paneId: string, command: string): string[];
+export type TmuxSplitDirection = "-h" | "-v";
+export declare function chooseTmuxSplitDirection(paneWidth: number, paneHeight: number): TmuxSplitDirection;
+export declare function buildTmuxSplitWindowArgs(cwd: string, command: string, direction?: TmuxSplitDirection, targetPane?: string | null): string[];
 export interface BackgroundReceiptInput {
     taskId: string;
     agentType: string;

package/dist/helpers.js

@@ -128,8 +128,29 @@ export function parseIdTimestamp(id) {
 export function shellQuote(value) {
     return `'${value.replace(/'/g, `'"'"'`)}'`;
 }
-export function buildTmuxSendKeysArgs(paneId, command) {
-    return ["send-keys", "-t", paneId, command, "Enter"];
+export function chooseTmuxSplitDirection(paneWidth, paneHeight) {
+    const minSideBySideWidth = 160;
+    const minStackedHeight = 24;
+    if (Number.isFinite(paneWidth) && paneWidth >= minSideBySideWidth) {
+        return "-h";
+    }
+    if (Number.isFinite(paneHeight) && paneHeight >= minStackedHeight) {
+        return "-v";
+    }
+    return "-h";
+}
+export function buildTmuxSplitWindowArgs(cwd, command, direction = "-h", targetPane) {
+    const args = [
+        "split-window",
+        direction,
+        "-P",
+        "-F",
+        "#{pane_id}",
+    ];
+    if (targetPane)
+        args.push("-t", targetPane);
+    args.push("-c", cwd, command);
+    return args;
 }
 export function formatBackgroundReceipt(input) {
     return [

package/dist/index.js

@@ -22,7 +22,7 @@ import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 import { Type } from "@sinclair/typebox";
 import { Text, truncateToWidth } from "@earendil-works/pi-tui";
-import { TASK_BACKGROUND_DEFAULT, TASK_RESULT_XML_INSTRUCTIONS, TASK_TOOL_DESCRIPTION, buildTmuxSendKeysArgs, formatBackgroundReceipt, buildPiArgs, parseResultXml, formatMs, shellQuote, discoverAgents, formatAgentList, countToolUses, readRecentToolCalls, } from "./helpers.js";
+import { TASK_BACKGROUND_DEFAULT, TASK_RESULT_XML_INSTRUCTIONS, TASK_TOOL_DESCRIPTION, buildTmuxSplitWindowArgs, chooseTmuxSplitDirection, formatBackgroundReceipt, buildPiArgs, parseResultXml, formatMs, shellQuote, discoverAgents, formatAgentList, countToolUses, readRecentToolCalls, } from "./helpers.js";
 import { runSdkSubagent } from "./subagent/runSdk.js";
 import { checkTaskCompletion, waitForTaskCompletion as waitForSessionTaskCompletion, } from "./subagent/waitCompletion.js";
 import { buildAgentToolSelection } from "./agent-tools.js";
@@ -79,20 +79,28 @@ function getCurrentPaneId() {
         return null;
     }
 }
+function getCurrentPaneSize(targetPane) {
+    try {
+        const args = ["display-message", "-p", "#{pane_width} #{pane_height}"];
+        if (targetPane)
+            args.splice(1, 0, "-t", targetPane);
+        const raw = tmuxCmd(args);
+        const [widthRaw, heightRaw] = raw.trim().split(/\s+/, 2);
+        const width = Number(widthRaw);
+        const height = Number(heightRaw);
+        if (!Number.isFinite(width) || !Number.isFinite(height))
+            return null;
+        return { width, height };
+    }
+    catch {
+        return null;
+    }
+}
 function splitWindowPane(cwd, command) {
     const originalPane = getCurrentPaneId();
-    const paneId = tmuxCmd([
-        "split-window",
-        "-h",
-        "-P",
-        "-F",
-        "#{pane_id}",
-        "-c",
-        cwd,
-    ]);
-    execFileSync("tmux", buildTmuxSendKeysArgs(paneId, command), {
-        stdio: "ignore",
-    });
+    const paneSize = getCurrentPaneSize(originalPane);
+    const direction = chooseTmuxSplitDirection(paneSize?.width ?? 0, paneSize?.height ?? 0);
+    const paneId = tmuxCmd(buildTmuxSplitWindowArgs(cwd, command, direction, originalPane));
     return { paneId, originalPane };
 }
 function killAgentPane(paneId, originalPane) {

package/dist/subagent/tmux.d.ts

@@ -5,10 +5,14 @@ export declare function tmuxCmd(args: string[]): string;
 export declare function hasTmux(): boolean;
 export declare function paneExists(paneId: string): boolean;
 export declare function getCurrentPaneId(): string | null;
+export declare function getCurrentPaneSize(targetPane?: string | null): {
+    width: number;
+    height: number;
+} | null;
 export declare function splitWindowPane(cwd: string, command: string): {
     paneId: string;
     originalPane: string | null;
 };
 export declare function killAgentPane(paneId: string, originalPane: string | null): void;
-/** Inject keys into a running subagent pane (steer / follow-up). */
+/** Inject text into a running subagent pane (steer / follow-up). */
 export declare function tmuxSteerPane(paneId: string, message: string): void;

package/dist/subagent/tmux.js

@@ -2,6 +2,7 @@
  * Tmux helpers for subagent panes (shared by task extension).
  */
 import { execFileSync } from "node:child_process";
+import { buildTmuxSplitWindowArgs, chooseTmuxSplitDirection } from "../helpers.js";
 export function tmuxCmd(args) {
     return execFileSync("tmux", args, {
         encoding: "utf-8",
@@ -34,18 +35,28 @@ export function getCurrentPaneId() {
         return null;
     }
 }
+export function getCurrentPaneSize(targetPane) {
+    try {
+        const args = ["display-message", "-p", "#{pane_width} #{pane_height}"];
+        if (targetPane)
+            args.splice(1, 0, "-t", targetPane);
+        const raw = tmuxCmd(args);
+        const [widthRaw, heightRaw] = raw.trim().split(/\s+/, 2);
+        const width = Number(widthRaw);
+        const height = Number(heightRaw);
+        if (!Number.isFinite(width) || !Number.isFinite(height))
+            return null;
+        return { width, height };
+    }
+    catch {
+        return null;
+    }
+}
 export function splitWindowPane(cwd, command) {
     const originalPane = getCurrentPaneId();
-    const paneId = tmuxCmd([
-        "split-window",
-        "-h",
-        "-P",
-        "-F",
-        "#{pane_id}",
-        "-c",
-        cwd,
-        command,
-    ]);
+    const paneSize = getCurrentPaneSize(originalPane);
+    const direction = chooseTmuxSplitDirection(paneSize?.width ?? 0, paneSize?.height ?? 0);
+    const paneId = tmuxCmd(buildTmuxSplitWindowArgs(cwd, command, direction, originalPane));
     return { paneId, originalPane };
 }
 export function killAgentPane(paneId, originalPane) {
@@ -64,9 +75,23 @@ export function killAgentPane(paneId, originalPane) {
         }
     }
 }
-/** Inject keys into a running subagent pane (steer / follow-up). */
+/** Inject text into a running subagent pane (steer / follow-up). */
 export function tmuxSteerPane(paneId, message) {
-    const escaped = message.replace(/'/g, `'\"'\"'`);
-    tmuxCmd(["send-keys", "-t", paneId, "-l", escaped]);
+    const bufferName = `pi-task-steer-${process.pid}-${Date.now()}`;
+    try {
+        execFileSync("tmux", ["load-buffer", "-b", bufferName, "-"], {
+            input: message,
+            stdio: ["pipe", "pipe", "pipe"],
+        });
+        tmuxCmd(["paste-buffer", "-b", bufferName, "-t", paneId]);
+    }
+    finally {
+        try {
+            tmuxCmd(["delete-buffer", "-b", bufferName]);
+        }
+        catch {
+            /* ignore */
+        }
+    }
     tmuxCmd(["send-keys", "-t", paneId, "Enter"]);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@heyhuynhgiabuu/pi-task",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "description": "Delegating task/subagent extension for Pi: foreground/background subagents, widgets, tmux observability, SDK fallback.",
   "type": "module",
   "main": "./dist/index.js",