xtrm-tools 0.5.45 → 0.5.47

package/cli/package.json

@@ -1,6 +1,6 @@
 {
   "name": "xtrm-cli",
-  "version": "0.5.45",
+  "version": "0.5.47",
   "description": "Claude Code tools installer (skills, hooks, MCP servers)",
   "main": "./dist/index.js",
   "type": "module",

package/config/instructions/agents-top.md

@@ -20,8 +20,6 @@
 | **Stop** | Session end with unclosed claim | `bd close <id>` |
 | **Memory** | Auto-fires at session end if issue closed | `bd remember "<insight>"` then run the `bd kv set` command shown in the gate message |
 
-> `bd close` auto-commits via `git commit -am`. Do not double-commit after closing.
-
 ## bd Command Reference
 
 ```bash
@@ -45,7 +43,7 @@ bd create --title="..." --description="..." --type=task --priority=2
 # types: task | bug | feature | epic | chore | decision
 
 # Closing
-bd close <id>                          # Close + auto-commit
+bd close <id>                          # Close issue
 bd close <id> --reason="Done: ..."     # Close with context
 bd close <id1> <id2> <id3>            # Batch close
 
@@ -74,7 +72,7 @@ bd doctor                              # Diagnose installation issues
 git checkout -b feature/<issue-id>-<slug>   # or fix/... chore/...
 bd update <id> --claim                       # claim before any edit
 # ... write code ...
-bd close <id> --reason="..."                 # closes issue + auto-commits
+bd close <id> --reason="..."                 # closes issue
 xt end                                       # push, PR, merge, worktree cleanup
 ```
 

package/config/instructions/claude-top.md

@@ -20,8 +20,6 @@
 | **Stop** | Session end with unclosed claim | `bd close <id>` |
 | **Memory** | Auto-fires at Stop if issue closed this session | `bd remember "<insight>"` then run the `bd kv set` command shown in the gate message |
 
-> `bd close` auto-commits via `git commit -am`. Do not double-commit after closing.
-
 ## bd Command Reference
 
 ```bash
@@ -45,7 +43,7 @@ bd create --title="..." --description="..." --type=task --priority=2
 # types: task | bug | feature | epic | chore | decision
 
 # Closing
-bd close <id>                          # Close + auto-commit
+bd close <id>                          # Close issue
 bd close <id> --reason="Done: ..."     # Close with context
 bd close <id1> <id2> <id3>            # Batch close
 
@@ -74,7 +72,7 @@ bd doctor                              # Diagnose installation issues
 git checkout -b feature/<issue-id>-<slug>   # or fix/... chore/...
 bd update <id> --claim                       # claim before any edit
 # ... write code ...
-bd close <id> --reason="..."                 # closes issue + auto-commits
+bd close <id> --reason="..."                 # closes issue
 xt end                                       # push, PR, merge, worktree cleanup
 ```
 

package/config/pi/extensions/beads/index.ts

@@ -2,61 +2,6 @@ import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
 import { isToolCallEventType, isBashToolResult } from "@mariozechner/pi-coding-agent";
 import { SubprocessRunner, EventAdapter } from "../core/lib";
 
-// ─── Autocommit helpers (mirrors hooks/beads-claim-sync.mjs) ─────────────────
-
-async function hasGitChanges(cwd: string): Promise<boolean> {
-	const result = await SubprocessRunner.run("git", ["status", "--porcelain"], { cwd });
-	if (result.code !== 0) return false;
-	return result.stdout.trim().length > 0;
-}
-
-async function stageUntracked(cwd: string): Promise<void> {
-	const result = await SubprocessRunner.run("git", ["ls-files", "--others", "--exclude-standard"], { cwd });
-	if (result.code !== 0 || !result.stdout.trim()) return;
-	const untracked = result.stdout.trim().split("\n").filter(Boolean);
-	if (untracked.length > 0) {
-		await SubprocessRunner.run("git", ["add", "--", ...untracked], { cwd });
-	}
-}
-
-async function getCloseReason(cwd: string, issueId: string, command: string): Promise<string> {
-	// 1. Parse --reason "..." from the command itself
-	const reasonMatch = command.match(/--reason[=\s]+["']([^"']+)["']/);
-	if (reasonMatch) return reasonMatch[1].trim();
-
-	// 2. Fall back to bd show <id> --json
-	const show = await SubprocessRunner.run("bd", ["show", issueId, "--json"], { cwd });
-	if (show.code === 0 && show.stdout.trim()) {
-		try {
-			const parsed = JSON.parse(show.stdout);
-			const issue = Array.isArray(parsed) ? parsed[0] : parsed;
-			const reason = issue?.close_reason;
-			if (typeof reason === "string" && reason.trim()) return reason.trim();
-		} catch { /* fall through */ }
-	}
-
-	return `Close ${issueId}`;
-}
-
-async function autoCommit(cwd: string, issueId: string, command: string): Promise<{ ok: boolean; message: string }> {
-	if (!await hasGitChanges(cwd)) {
-		return { ok: true, message: "No changes detected — auto-commit skipped." };
-	}
-
-	await stageUntracked(cwd);
-
-	const reason = await getCloseReason(cwd, issueId, command);
-	const commitMessage = `${reason} (${issueId})`;
-	const result = await SubprocessRunner.run("git", ["commit", "--no-verify", "-am", commitMessage], { cwd });
-
-	if (result.code !== 0) {
-		const err = (result.stderr || result.stdout || "").trim();
-		return { ok: false, message: `Auto-commit failed: ${err || "unknown error"}` };
-	}
-
-	return { ok: true, message: `Auto-committed: \`${commitMessage}\`` };
-}
-
 export default function (pi: ExtensionAPI) {
 	const getCwd = (ctx: any) => ctx.cwd || process.cwd();
 
@@ -195,26 +140,33 @@ export default function (pi: ExtensionAPI) {
 			const closeMatch = command.match(/\bbd\s+close\s+(\S+)/);
 			const closedIssueId = closeMatch?.[1] ?? null;
 
-			// Auto-commit staged changes (mirrors hooks/beads-claim-sync.mjs)
-			const commit = closedIssueId ? await autoCommit(cwd, closedIssueId, command) : null;
-
 			if (closedIssueId) {
 				await SubprocessRunner.run("bd", ["kv", "set", `closed-this-session:${sessionId}`, closedIssueId], { cwd });
 				memoryGateFired = false;
 			}
 
-			const commitLine = commit
-				? `\n${commit.ok ? "✅" : "⚠️"} **Session Flow**: ${commit.message}`
-				: "";
-			const reminder = `\n\n**Beads Insight**: Work completed. Consider if this session produced insights worth persisting via \`bd remember\`.${commitLine}`;
-			return { content: [...event.content, { type: "text", text: reminder }] };
+			// Inject memory gate as agent-visible context only — parity with Claude Stop hook {additionalContext}.
+			// No UI notification; the agent sees this silently in its tool result context.
+			const memoryGateText = closedIssueId
+				? `\n\n**Beads Memory Gate**: Issue \`${closedIssueId}\` closed this session.\n` +
+				  `For each candidate insight, check ALL 4:\n` +
+				  `  1. Hard to rediscover from code/docs?\n` +
+				  `  2. Not obvious from the current implementation?\n` +
+				  `  3. Will affect a future decision?\n` +
+				  `  4. Still relevant in ~14 days?\n` +
+				  `KEEP (all 4 yes) → \`bd remember "<insight>"\`\n` +
+				  `SKIP examples: file maps, flag inventories, per-issue summaries, wording tweaks, facts obvious from reading the source.\n` +
+				  `When done: \`bd kv set "memory-gate-done:${sessionId}" "saved: <key>"\` (or \`"nothing novel — <reason>"\`)`
+				: `\n\n**Beads**: Work completed. Consider if this session produced insights worth persisting via \`bd remember\`.`;
+			return { content: [...event.content, { type: "text", text: memoryGateText }] };
 		}
 
 		return undefined;
 	});
 
-	// Memory gate: if this session closed an issue, prompt for insight persistence.
-	// Uses sendUserMessage to trigger a new turn in Pi (non-blocking alternative to Claude Stop hook).
+	// Memory gate: clean up session markers and check ack at agent_end/session_shutdown.
+	// Memory gate prompt was already injected into bd close tool_result context (silent, agent-visible only).
+	// No UI notification — parity with Claude Stop hook {additionalContext} pattern.
 	const triggerMemoryGateIfNeeded = async (ctx: any) => {
 		const cwd = getCwd(ctx);
 		if (!EventAdapter.isBeadsProject(cwd)) return;
@@ -234,19 +186,7 @@ export default function (pi: ExtensionAPI) {
 		if (!closedIssueId) return;
 
 		memoryGateFired = true;
-		pi.sendUserMessage(
-			`🧠 Memory gate: claim \`${closedIssueId}\` was closed this session.\n` +
-			`For each candidate insight, check ALL 4:\n` +
-			`  1. Hard to rediscover from code/docs?\n` +
-			`  2. Not obvious from the current implementation?\n` +
-			`  3. Will affect a future decision?\n` +
-			`  4. Still relevant in ~14 days?\n` +
-			`KEEP (all 4 yes) → \`bd remember "<insight>"\`\n` +
-			`SKIP examples: file maps, flag inventories, per-issue summaries,\n` +
-			`  wording tweaks, facts obvious from reading the source.\n` +
-			`KEEP: \`bd kv set "memory-gate-done:${sessionId}" "saved: <key>"\`\n` +
-			`SKIP: \`bd kv set "memory-gate-done:${sessionId}" "nothing novel — <one-line reason>"\``,
-		);
+		// No notify — memory gate was injected into bd close tool_result content (silent, agent-visible only).
 	};
 
 	pi.on("agent_end", async (_event, ctx) => {

package/config/pi/extensions/custom-footer/index.ts

@@ -264,12 +264,11 @@ export default function (pi: ExtensionAPI) {
 
 					const BOLD = "\x1b[1m";
 					const BOLD_OFF = "\x1b[22m";
-					const brand = `${BOLD}${theme.fg("accent", "XTRM")}${BOLD_OFF}`;
+					const brand = `${BOLD}${theme.fg("dim", "XTRM")}${BOLD_OFF}`;
 
 					const usage = ctx.getContextUsage();
 					const pct = usage?.percent ?? 0;
-					const pctColor = pct > 75 ? "error" : pct > 50 ? "warning" : "success";
-					const usageStr = theme.fg(pctColor, `[${pct.toFixed(0)}%]`);
+					const usageStr = theme.fg("dim", `[${pct.toFixed(0)}%]`);
 
 					const modelId = ctx.model?.id || "no-model";
 					const modelStr = `${modelId} ${usageStr}`;

package/config/pi/extensions/xtrm-ui/format.ts

@@ -0,0 +1,93 @@
+export interface DiffStats {
+  additions: number;
+  removals: number;
+}
+
+export function shortenHome(path: string): string {
+  const home = process.env.HOME;
+  if (home && path.startsWith(home)) return `~${path.slice(home.length)}`;
+  return path;
+}
+
+export function shortenPath(path: string, max = 56): string {
+  const normalized = shortenHome(path);
+  if (normalized.length <= max) return normalized;
+  const parts = normalized.split("/").filter(Boolean);
+  if (parts.length <= 2) return `…${normalized.slice(-(max - 1))}`;
+  const tail = parts.slice(-2).join("/");
+  const head = parts[0]?.startsWith("~") ? "~/" : "…/";
+  const candidate = `${head}${tail}`;
+  if (candidate.length <= max) return candidate;
+  return `…${candidate.slice(-(max - 1))}`;
+}
+
+export function shortenCommand(command: string, max = 72): string {
+  const singleLine = command.replace(/\s+/g, " ").trim();
+  if (singleLine.length <= max) return singleLine;
+  return `${singleLine.slice(0, Math.max(0, max - 1))}…`;
+}
+
+export function lineCount(text: string): number {
+  if (!text) return 0;
+  return text.split("\n").length;
+}
+
+export function previewLines(text: string, count: number): string[] {
+  return text.split("\n").slice(0, count);
+}
+
+export function cleanOutputLines(text: string): string[] {
+  return text
+    .split("\n")
+    .filter((line) => line.trim().length > 0)
+    .filter((line) => !/^exit code:\s*-?\d+$/i.test(line.trim()));
+}
+
+export function countPrefixedItems(text: string, prefixes: string[]): number {
+  return text.split("\n").filter((line) => prefixes.some((prefix) => line.startsWith(prefix))).length;
+}
+
+export function diffStats(diff: string): DiffStats {
+  let additions = 0;
+  let removals = 0;
+  for (const line of diff.split("\n")) {
+    if (line.startsWith("+") && !line.startsWith("+++")) additions++;
+    if (line.startsWith("-") && !line.startsWith("---")) removals++;
+  }
+  return { additions, removals };
+}
+
+export function formatDuration(durationMs: number | undefined): string | undefined {
+  if (!durationMs || durationMs < 0) return undefined;
+  if (durationMs < 1000) return `${durationMs}ms`;
+  const seconds = durationMs / 1000;
+  if (seconds < 10) return `${seconds.toFixed(1)}s`;
+  return `${Math.round(seconds)}s`;
+}
+
+export function formatLineLabel(count: number, noun: string): string {
+  return `${count} ${noun}${count === 1 ? "" : "s"}`;
+}
+
+export function renderToolSummary(
+  theme: { fg(color: string, text: string): string; bold(text: string): string },
+  status: "pending" | "success" | "error" | "muted",
+  label: string,
+  subject?: string,
+  meta?: string,
+): string {
+  const color =
+    status === "pending" ? "accent"
+    : status === "error" ? "error"
+    : status === "success" ? "success"
+    : "muted";
+  let text = `${theme.fg(color, "•")} ${theme.fg("toolTitle", theme.bold(label))}`;
+  if (subject) text += ` ${theme.fg("accent", subject)}`;
+  if (meta) text += theme.fg("muted", ` · ${meta}`);
+  return text;
+}
+
+export function joinMeta(parts: Array<string | undefined | false>): string | undefined {
+  const filtered = parts.filter((part): part is string => typeof part === "string" && part.length > 0);
+  return filtered.length > 0 ? filtered.join(" · ") : undefined;
+}