@ramarivera/coding-buddy 0.9.0 → 0.9.2

package/adapters/omp/ui.ts

@@ -1,19 +1,13 @@
 import type { OmpBuddyUiContext } from "./context.ts";
 import type { Achievement } from "../../core/achievements.ts";
 import type { Companion, ReactionState } from "../../core/model.ts";
-import { renderAchievementsSummary, renderBuddyWidget } from "./renderers.ts";
+import { renderAchievementsSummary } from "./renderers.ts";
 import { OmpBuddyStorage } from "./storage.ts";
-import { getWidgetWidth, subscribeToWidgetResize } from "../shared/widget-layout.ts";
+import { BuddyWidget } from "../shared/widget-layout.ts";
 
 export class OmpBuddyUI {
   private readonly timers: Array<{ timer: Timer; clear: (timer: Timer) => void }> = [];
-  private resizeUnsubscribe: (() => void) | null = null;
-  private rendered: {
-    ctx: OmpBuddyUiContext;
-    companion: Companion;
-    reaction: ReactionState | null;
-    achievements: Achievement[];
-  } | null = null;
+  private readonly buddy = new BuddyWidget();
 
   constructor(private readonly storage: OmpBuddyStorage) {}
 
@@ -29,9 +23,7 @@ export class OmpBuddyUI {
 
   dispose(): void {
     this.cancelTimers();
-    this.resizeUnsubscribe?.();
-    this.resizeUnsubscribe = null;
-    this.rendered = null;
+    this.buddy.dispose();
   }
 
   refresh(
@@ -42,12 +34,15 @@ export class OmpBuddyUI {
   ): void {
     const currentReaction = reaction ?? null;
     const muted = this.storage.isMuted();
-    this.rendered = { ctx, companion, reaction: currentReaction, achievements: [...achievements] };
-    this.subscribeToResize();
+    const scheduler = getScheduler(ctx);
+
     ctx.ui.setStatus("buddy", undefined);
-    ctx.ui.setWidget(
-      "buddy",
-      renderBuddyWidget(companion, muted ? null : currentReaction, achievements, getWidgetWidth()),
+    this.buddy.refresh(
+      (lines) => ctx.ui.setWidget("buddy", lines),
+      companion,
+      muted ? null : currentReaction,
+      [...achievements],
+      scheduler ?? undefined,
     );
 
     this.cancelTimers();
@@ -55,8 +50,6 @@ export class OmpBuddyUI {
 
     const ttl = this.storage.loadConfig().reactionTTL;
     if (ttl <= 0) return;
-
-    const scheduler = getScheduler(ctx);
     if (!scheduler) return;
 
     const reactionTimestamp = currentReaction.timestamp;
@@ -83,21 +76,10 @@ export class OmpBuddyUI {
     unlocked: Array<{ achievement: Achievement; unlockedAt: number; slot?: string }>,
     remaining: Achievement[],
   ): void {
-    this.rendered = null;
+    this.buddy.clear();
     ctx.ui.setWidget("buddy", renderAchievementsSummary(unlocked, remaining));
     ctx.ui.notify(`Achievements: ${unlocked.length} unlocked`, "info");
   }
-
-  private subscribeToResize(): void {
-    if (this.resizeUnsubscribe) return;
-    const resizeHandler = () => {
-      if (!this.rendered) return;
-      const { ctx, companion, reaction, achievements } = this.rendered;
-      const muted = this.storage.isMuted();
-      ctx.ui.setWidget("buddy", renderBuddyWidget(companion, muted ? null : reaction, achievements, getWidgetWidth()));
-    };
-    this.resizeUnsubscribe = subscribeToWidgetResize(resizeHandler);
-  }
 }
 
 function getScheduler(

package/adapters/pi/ui.ts

@@ -1,9 +1,9 @@
 import type { ExtensionContext } from "@mariozechner/pi-coding-agent";
 import type { Achievement } from "../../core/achievements.ts";
 import type { Companion, ReactionState } from "../../core/model.ts";
-import { renderAchievementsSummary, renderBuddyWidget } from "./renderers.ts";
+import { renderAchievementsSummary } from "./renderers.ts";
 import { PiBuddyStorage } from "./storage.ts";
-import { getWidgetWidth, subscribeToWidgetResize } from "../shared/widget-layout.ts";
+import { BuddyWidget } from "../shared/widget-layout.ts";
 
 type PiBuddyUiContext = ExtensionContext & {
   setTimeout?: (callback: () => void, ms?: number) => Timer;
@@ -12,13 +12,7 @@ type PiBuddyUiContext = ExtensionContext & {
 
 export class PiBuddyUI {
   private readonly timers: Array<{ timer: Timer; clear: (timer: Timer) => void }> = [];
-  private resizeUnsubscribe: (() => void) | null = null;
-  private rendered: {
-    ctx: PiBuddyUiContext;
-    companion: Companion;
-    reaction: ReactionState | null;
-    achievements: Achievement[];
-  } | null = null;
+  private readonly buddy = new BuddyWidget();
 
   constructor(private readonly storage: PiBuddyStorage) {}
 
@@ -34,9 +28,7 @@ export class PiBuddyUI {
 
   dispose(): void {
     this.cancelTimers();
-    this.resizeUnsubscribe?.();
-    this.resizeUnsubscribe = null;
-    this.rendered = null;
+    this.buddy.dispose();
   }
 
   refresh(
@@ -47,12 +39,15 @@ export class PiBuddyUI {
   ): void {
     const currentReaction = reaction ?? null;
     const muted = this.storage.isMuted();
-    this.rendered = { ctx, companion, reaction: currentReaction, achievements: [...achievements] };
-    this.subscribeToResize();
+    const scheduler = getScheduler(ctx);
+
     ctx.ui.setStatus("buddy", undefined);
-    ctx.ui.setWidget(
-      "buddy",
-      renderBuddyWidget(companion, muted ? null : currentReaction, achievements, getWidgetWidth()),
+    this.buddy.refresh(
+      (lines) => ctx.ui.setWidget("buddy", lines),
+      companion,
+      muted ? null : currentReaction,
+      [...achievements],
+      scheduler ?? undefined,
     );
 
     this.cancelTimers();
@@ -60,8 +55,6 @@ export class PiBuddyUI {
 
     const ttl = this.storage.loadConfig().reactionTTL;
     if (ttl <= 0) return;
-
-    const scheduler = getScheduler(ctx);
     if (!scheduler) return;
 
     const reactionTimestamp = currentReaction.timestamp;
@@ -88,21 +81,10 @@ export class PiBuddyUI {
     unlocked: Array<{ achievement: Achievement; unlockedAt: number; slot?: string }>,
     remaining: Achievement[],
   ): void {
-    this.rendered = null;
+    this.buddy.clear();
     ctx.ui.setWidget("buddy", renderAchievementsSummary(unlocked, remaining));
     ctx.ui.notify(`Achievements: ${unlocked.length} unlocked`, "info");
   }
-
-  private subscribeToResize(): void {
-    if (this.resizeUnsubscribe) return;
-    const resizeHandler = () => {
-      if (!this.rendered) return;
-      const { ctx, companion, reaction, achievements } = this.rendered;
-      const muted = this.storage.isMuted();
-      ctx.ui.setWidget("buddy", renderBuddyWidget(companion, muted ? null : reaction, achievements, getWidgetWidth()));
-    };
-    this.resizeUnsubscribe = subscribeToWidgetResize(resizeHandler);
-  }
 }
 
 function getScheduler(

package/adapters/shared/file-storage.ts

@@ -56,6 +56,7 @@ export const DEFAULT_FILE_BUDDY_CONFIG: FileBuddyConfig = {
   bubblePosition: "top",
   showRarity: true,
   statusLineEnabled: true,
+  statuslineDensity: "auto",
   turnCommentModel: undefined,
   turnCommentTimeoutMs: undefined,
   muted: false,

package/adapters/shared/widget-layout.ts

@@ -164,16 +164,20 @@ function padLine(line: string, width: number): string {
 }
 
 function frameReaction(reaction: string, achievements: Achievement[], innerWidth: number): string[] {
-  const reactionLines = wrapReaction(reaction, innerWidth, 2);
-  const achievementLine = achievements[0] ? `🏆 ${achievements[0].name}` : "";
-  const content = [...reactionLines, ...(achievementLine ? [achievementLine] : [])]
-    .slice(0, 3)
-    .map((line) => `${DIM_ITALIC}${line}${RESET}`);
+  const maxContentLines = WIDGET_MAX_LINES - 2;
+  const trimmedReaction = reaction.trim();
+  const achievementLines = achievements.map((achievement) => `🏆 ${achievement.name}`);
+  const reactionMax = Math.max(0, maxContentLines - achievementLines.length);
+  const reactionLines = trimmedReaction ? wrapReaction(trimmedReaction, innerWidth, reactionMax) : [];
+  const content = reactionLines.concat(achievementLines).slice(0, maxContentLines);
+  const paddedContent = content.length === 1 ? ["", ...content] : content;
+
   const top = `.${"-".repeat(innerWidth + 2)}.`;
+  const bottom = `\`${"-".repeat(innerWidth + 2)}'`;
   return [
     top,
-    ...content.map((line) => `| ${padLine(line, innerWidth)} |`),
-    `\`${"-".repeat(innerWidth + 2)}'`,
+    ...paddedContent.map((line) => `| ${DIM_ITALIC}${padLine(line, innerWidth)}${RESET} |`),
+    bottom,
   ];
 }
 
@@ -205,16 +209,21 @@ export function renderCompanionWidget(
   const maxBubbleInner = safeWidth - spriteWidth - tailWidth - 4;
   const hasContent = Boolean(reaction?.reaction?.trim()) || achievements.length > 0;
   const canFrame = hasContent && maxBubbleInner >= minBubbleInner;
-  const bubbleInner = canFrame ? Math.min(44, maxBubbleInner) : 0;
+  const bubbleInner = canFrame ? maxBubbleInner : 0;
   const bubble = canFrame ? frameReaction(reaction?.reaction ?? "", achievements, bubbleInner) : [];
   const bubbleWidth = bubbleInner + 4;
   const cardWidth = canFrame ? bubbleWidth + tailWidth + spriteWidth : spriteWidth;
-  const height = Math.max(clippedArt.length + 1, bubble.length);
+
+  const achievementSprites = canFrame
+    ? []
+    : achievements.map((achievement) => `${DIM_ITALIC}${padLine(`🏆 ${achievement.name}`, spriteWidth)}${RESET}`);
+  const spriteBody = [...clippedArt, clippedLabel, ...achievementSprites];
+  const height = Math.max(spriteBody.length, bubble.length);
   const connectorRow = canFrame ? Math.min(bubble.length - 2, Math.max(1, Math.floor(bubble.length / 2))) : -1;
 
   return Array.from({ length: Math.min(WIDGET_MAX_LINES, height) }, (_, index) => {
-    const artLine = index < clippedArt.length ? clippedArt[index]! : index === clippedArt.length ? clippedLabel : "";
-    const sprite = padLine(artLine, spriteWidth);
+    const spriteLine = spriteBody[index] ?? "";
+    const sprite = padLine(spriteLine, spriteWidth);
     let body: string;
     if (canFrame) {
       const bubbleLine = bubble[index] ?? " ".repeat(bubbleWidth);
@@ -226,3 +235,108 @@ export function renderCompanionWidget(
     return `${" ".repeat(Math.max(0, safeWidth - cardWidth))}${body}`;
   });
 }
+
+export interface BuddyWidgetScheduler {
+  setTimeout(callback: () => void, ms?: number): Timer;
+  clearTimer(timer: Timer): void;
+}
+
+/**
+ * Width-aware buddy widget manager.
+ *
+ * Renders once when state arrives, then re-renders with the current terminal
+ * width after coalescing rapid process.stdout resize events. Callers supply
+ * only their harness `setWidget` and the current state; this class owns the
+ * resize subscription, debounce, and re-render logic.
+ */
+export class BuddyWidget {
+  private readonly getWidth: () => number;
+  private scheduler: BuddyWidgetScheduler;
+  private readonly coalesceMs: number;
+  private setWidget: ((lines: string[]) => void) | null = null;
+  private state: { companion: Companion; reaction: ReactionState | null; achievements: Achievement[] } | null = null;
+  private resizeUnsubscribe: (() => void) | null = null;
+  private timer: Timer | null = null;
+
+  constructor(options?: {
+    getWidth?: () => number;
+    scheduler?: BuddyWidgetScheduler;
+    coalesceMs?: number;
+  }) {
+    this.getWidth = options?.getWidth ?? getWidgetWidth;
+    this.scheduler = options?.scheduler ?? {
+      setTimeout: globalThis.setTimeout.bind(globalThis),
+      clearTimer: globalThis.clearTimeout.bind(globalThis),
+    };
+    this.coalesceMs = options?.coalesceMs ?? 50;
+  }
+
+  /**
+   * Render immediately with the current state and start (or renew) the shared
+   * resize subscription. The `setWidget` callback is the adapter's harness
+   * setter bound to the current UI context.
+   */
+  refresh(
+    setWidget: (lines: string[]) => void,
+    companion: Companion,
+    reaction: ReactionState | null,
+    achievements: Achievement[] = [],
+    scheduler?: BuddyWidgetScheduler,
+  ): void {
+    // Cancel any pending resize timer before swapping the scheduler or state
+    // so clearTimer is always paired with the scheduler that created the timer.
+    this.cancelTimer();
+    this.setWidget = setWidget;
+    if (scheduler) this.scheduler = scheduler;
+    this.state = { companion, reaction, achievements };
+    this.renderAndSet();
+    this.subscribe();
+  }
+
+  /** Stop reacting to resizes without unsubscribing from the shared listener. */
+  clear(): void {
+    this.cancelTimer();
+    this.state = null;
+  }
+
+  /** Tear down timers and the shared resize listener. */
+  dispose(): void {
+    this.cancelTimer();
+    this.resizeUnsubscribe?.();
+    this.resizeUnsubscribe = null;
+    this.state = null;
+    this.setWidget = null;
+  }
+
+  private subscribe(): void {
+    if (this.resizeUnsubscribe) return;
+    this.resizeUnsubscribe = subscribeToWidgetResize(() => this.onResize());
+  }
+
+  private onResize(): void {
+    // A cleared widget has no state/setter to render; don't schedule work.
+    if (!this.state || !this.setWidget) return;
+    this.cancelTimer();
+    this.timer = this.scheduler.setTimeout(() => {
+      this.timer = null;
+      try {
+        this.renderAndSet();
+      } catch (error) {
+        console.error("BuddyWidget render failed:", error);
+      }
+    }, this.coalesceMs);
+  }
+  private renderAndSet(): void {
+    if (!this.state || !this.setWidget) return;
+    const width = this.getWidth();
+    const lines = renderCompanionWidget(this.state.companion, this.state.reaction, this.state.achievements, width);
+    this.setWidget(lines);
+  }
+
+  private cancelTimer(): void {
+    if (this.timer !== null) {
+      this.scheduler.clearTimer(this.timer);
+      this.timer = null;
+    }
+  }
+}

package/cli/install.ts

@@ -24,6 +24,17 @@ import { loadCompanion, saveCompanion, resolveUserId, writeStatusState } from ".
 import { generateFallbackName } from "../core/reactions.ts"
 import { copyRuntimeApp, stableRuntimePaths } from "./runtime-app.ts";
 
+/** Recognise any buddy-owned hook command, however it is pathed. */
+function isBuddyHookCommand(command: unknown): boolean {
+  if (typeof command !== "string") return false;
+  return (
+    command.includes("coding-buddy") ||
+    command.includes("claude-buddy") ||
+    command.includes("buddy-state") ||
+    /\/(buddy-comment|suggest|react|mood-react|name-react|file-type-react)\.(sh|ts)\b/.test(command)
+  );
+}
+
 const CYAN = "\x1b[36m";
 const GREEN = "\x1b[32m";
 const YELLOW = "\x1b[33m";
@@ -203,7 +214,7 @@ function installHooks(settings: Record<string, any>, appDir: string) {
   // plus file-type specific reactions on Write/Edit.
   if (!settings.hooks.PostToolUse) settings.hooks.PostToolUse = [];
   settings.hooks.PostToolUse = settings.hooks.PostToolUse.filter(
-    (h: any) => !h.hooks?.some((hh: any) => hh.command?.includes("coding-buddy") || hh.command?.includes("claude-buddy")),
+    (h: any) => !h.hooks?.some((hh: any) => isBuddyHookCommand(hh.command)),
   );
   settings.hooks.PostToolUse.push({
     matcher: "Bash",
@@ -216,8 +227,15 @@ function installHooks(settings: Record<string, any>, appDir: string) {
 
   // Stop: extract <!-- buddy: --> comment from Claude's response
   if (!settings.hooks.Stop) settings.hooks.Stop = [];
+  // Match on the hook script filenames, not the package name: the installed
+  // path is <state>/app/hooks/buddy-comment.sh, which contains neither
+  // "coding-buddy" nor "claude-buddy". The old filter therefore never matched
+  // and every install appended another pair — observed at 8 Stop entries on a
+  // dogfooding machine. Duplicates are not harmless: the first invocation
+  // adopts a buddy_react reaction and stamps the stop marker, then the rest
+  // see it as stale and overwrite the bubble with a canned pool line.
   settings.hooks.Stop = settings.hooks.Stop.filter(
-    (h: any) => !h.hooks?.some((hh: any) => hh.command?.includes("coding-buddy") || hh.command?.includes("claude-buddy")),
+    (h: any) => !h.hooks?.some((hh: any) => isBuddyHookCommand(hh.command)),
   );
   settings.hooks.Stop.push({
     hooks: [commandHook(commentHook)],
@@ -230,7 +248,7 @@ function installHooks(settings: Record<string, any>, appDir: string) {
   // reaction, plus mood-react based on prompt content.
   if (!settings.hooks.UserPromptSubmit) settings.hooks.UserPromptSubmit = [];
   settings.hooks.UserPromptSubmit = settings.hooks.UserPromptSubmit.filter(
-    (h: any) => !h.hooks?.some((hh: any) => hh.command?.includes("coding-buddy") || hh.command?.includes("claude-buddy")),
+    (h: any) => !h.hooks?.some((hh: any) => isBuddyHookCommand(hh.command)),
   );
   settings.hooks.UserPromptSubmit.push({
     hooks: [commandHook(nameHook)],

package/core/model.ts

@@ -38,6 +38,7 @@ export interface BuddyConfig {
   bubblePosition: "top" | "left";
   showRarity: boolean;
   statusLineEnabled: boolean;
+  statuslineDensity?: "auto" | "full" | "compact" | "minimal";
   turnCommentModel?: BuddyTurnCommentModelConfig;
   /** Optional per-host bound for end-of-turn reaction LLM calls (ms). */
   turnCommentTimeoutMs?: number;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ramarivera/coding-buddy",
-  "version": "0.9.0",
+  "version": "0.9.2",
   "description": "Persistent coding companion for Claude Code, Pi, and Oh My Pi",
   "type": "module",
   "omp": {
@@ -17,7 +17,6 @@
     "coding-buddy": "cli/index.ts"
   },
   "scripts": {
-    "postinstall": "node scripts/postinstall.cjs",
     "server": "bun run server/index.ts",
     "pick": "bun run cli/pick.ts",
     "hunt": "bun run cli/hunt.ts",