pi-better-background-tasks 0.2.3 → 0.2.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-better-background-tasks",
-  "version": "0.2.3",
+  "version": "0.2.5",
   "description": "Pi extension for durable background shell tasks, watchers, logs, and status inspection.",
   "license": "MIT",
   "type": "module",

package/src/remote-task-preset.ts

@@ -113,8 +113,14 @@ const REQUIRED_SSH_OPTIONS = new Set(["batchmode", "connecttimeout", "requesttty
 const DIRECT_STOP_WARNING = "Direct SSH mode has weak stop semantics: stopping the local SSH client may leave the remote process running.";
 const TMUX_STATUS_PREFIX = "__PI_BG_STATUS__=";
 const TMUX_SIZE_PREFIX = "__PI_BG_SIZE__=";
+const TMUX_PATH_PREFIX = "__PI_BG_TMUX_PATH__=";
+const TMUX_VERSION_PREFIX = "__PI_BG_TMUX_VERSION__=";
 const TMUX_CAPTURE_CHUNK_BYTES = 256 * 1024;
-const TMUX_PROBE_COMMAND = "tmux_path=$(command -v tmux) || exit 127; printf '%s\\n' \"$tmux_path\" && \"$tmux_path\" -V";
+const TMUX_PROBE_COMMAND = [
+  "tmux_path=$(command -v tmux) || exit 127",
+  `printf '${TMUX_PATH_PREFIX}%s\\n' "$tmux_path"`,
+  `printf '${TMUX_VERSION_PREFIX}%s\\n' "$("$tmux_path" -V)"`,
+].join("; ");
 const TMUX_PACKAGE_MANAGERS: TmuxPackageManager[] = ["apt-get", "dnf", "yum", "apk", "pacman", "zypper", "brew"];
 const TMUX_INSTALL_COMMANDS: Record<TmuxPackageManager, string> = {
   "apt-get": "apt-get update && apt-get install -y tmux",
@@ -523,8 +529,16 @@ function resolveBootstrapTimeoutMs(timeoutMs: number | undefined): number {
 }
 
 function parseTmuxCapability(result: CommandResult): { tmuxPath: string; tmuxVersion: string } | undefined {
-  const [tmuxPath, tmuxVersion] = result.stdout.trim().split("\n");
-  if (result.exitCode !== 0 || !tmuxPath || !tmuxVersion) return undefined;
+  if (result.exitCode !== 0) return undefined;
+  let tmuxPath: string | undefined;
+  let tmuxVersion: string | undefined;
+  for (const line of result.stdout.split("\n")) {
+    if (line.startsWith(TMUX_PATH_PREFIX)) tmuxPath = line.slice(TMUX_PATH_PREFIX.length);
+    if (line.startsWith(TMUX_VERSION_PREFIX)) tmuxVersion = line.slice(TMUX_VERSION_PREFIX.length);
+  }
+  if (!tmuxPath || !tmuxVersion) return undefined;
+  if (!tmuxPath.startsWith("/") || /\s/.test(tmuxPath)) return undefined;
+  if (!/^tmux\b/i.test(tmuxVersion)) return undefined;
   return { tmuxPath, tmuxVersion };
 }
 

package/src/tools.ts

@@ -68,7 +68,7 @@ const ListParams = Type.Object({
 });
 const LogParams = Type.Object({
   id: Type.String({ description: "Background task id." }),
-  tail_lines: Type.Optional(Type.Number({ description: "Number of trailing lines. Default 20 for compact model ingestion. Set <=0 only when the full log is explicitly required." })),
+  tail_lines: Type.Optional(Type.Number({ description: "Number of trailing lines. Default 5 for compact model ingestion. Set <=0 only when the full log is explicitly required." })),
 });
 
 const ActionParams = Type.Object({
@@ -168,7 +168,7 @@ export function registerTools(pi: ExtensionAPI): void {
   pi.registerTool({
     name: "bg_task_log",
     label: "BG Log",
-    description: "Read a background task log. Default output is a compact 20-line terminal-aware tail for model ingestion. Pass tail_lines for a bounded tail; tail_lines:0 returns the retained raw log, capped at 512 KiB for safe recovery. Nonblocking.",
+    description: "Read a background task log. Default output is a compact 5-line terminal-aware tail for model ingestion. Pass tail_lines for a bounded tail; tail_lines:0 returns the retained raw log, capped at 512 KiB for safe recovery. Nonblocking.",
     parameters: LogParams,
     renderResult(result: unknown, options: unknown, theme: unknown) {
       return renderBackgroundTaskLogDisplay(result, options, theme);
@@ -338,7 +338,7 @@ function formatCompactStatus(meta: BackgroundTaskMeta): string {
   if (meta.lastState !== undefined) lines.push(`last state: ${oneLine(meta.lastState, 800)}`);
   if (meta.logDiscardedBytes) lines.push(`log retention: ${meta.logDiscardedBytes} bytes discarded in ${meta.logRetentionEvents ?? 1} compaction(s).`);
   lines.push(`log: ${meta.logPath}`);
-  lines.push(`For full metadata use bg_task_status id=${meta.id} verbose=true. For logs use bg_task_log id=${meta.id} tail_lines=20, or tail_lines=0 for the retained raw log.`);
+  lines.push(`For full metadata use bg_task_status id=${meta.id} verbose=true. For logs use bg_task_log id=${meta.id} tail_lines=5, or tail_lines=0 for the retained raw log.`);
   return lines.join("\n");
 }
 
@@ -360,7 +360,7 @@ function oneLine(value: unknown, maxLength: number): string {
 function formatLog(id: string, tailLines?: number): string {
   const meta = readMeta(id);
   if (!meta) return `No background task found for id ${id}.`;
-  const log = readLog(meta.logPath, tailLines ?? 20);
+  const log = readLog(meta.logPath, tailLines ?? 5);
   const prefix = log.truncated ? `[showing tail of ${meta.logPath}]\n` : `[${meta.logPath}]\n`;
   return prefix + (log.text || "(log is empty)");
 }