@ramarivera/coding-buddy 0.9.2 → 0.9.4

package/.claude-plugin/marketplace.json

@@ -5,14 +5,14 @@
     },
     "metadata": {
         "description": "Persistent coding companion for Claude Code, Pi, and Oh My Pi (requires bun: https://bun.sh/install)",
-        "version": "0.8.0"
+        "version": "0.9.4"
     },
     "plugins": [
         {
             "name": "coding-buddy",
             "source": "./",
             "description": "Persistent coding companion for Claude Code, Pi, and Oh My Pi. MCP-based terminal pet with ASCII art, stats, reactions, and personality. Requires bun runtime (https://bun.sh/install).",
-            "version": "0.8.0",
+            "version": "0.9.4",
             "author": {
                 "name": "ramarivera"
             },

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
     "name": "coding-buddy",
-    "version": "0.8.0",
+    "version": "0.9.4",
     "description": "Persistent coding companion for Claude Code, Pi, and Oh My Pi. MCP-based terminal pet with ASCII art, stats, reactions, and personality.",
     "author": {
         "name": "ramarivera"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ramarivera/coding-buddy",
-  "version": "0.9.2",
+  "version": "0.9.4",
   "description": "Persistent coding companion for Claude Code, Pi, and Oh My Pi",
   "type": "module",
   "omp": {
@@ -36,6 +36,8 @@
     "test": "bun test",
     "typecheck": "tsc --noEmit",
     "gen:emoji-widths": "bun run scripts/gen-emoji-widths.ts",
+    "sync-version": "bun run scripts/sync-version.ts",
+    "version": "bun run sync-version && git add .claude-plugin",
     "ci:statusline": "bash scripts/ci/check-statusline.sh",
     "ci:vhs:render": "bash scripts/ci/render-vhs-goldens.sh",
     "ci:vhs:update": "bash scripts/ci/render-vhs-goldens.sh --update",

package/scripts/sync-version.ts

@@ -0,0 +1,122 @@
+/**
+ * Sync every version field in the repo to package.json.
+ *
+ *   bun run sync-version           # rewrite drifted fields
+ *   bun run sync-version --check   # exit 1 on drift, change nothing (CI)
+ *
+ * Wired into the npm `version` lifecycle so `npm version <bump>` updates the
+ * manifests in the same commit as package.json, and into CI via --check so
+ * drift cannot land.
+ *
+ * Design and the original implementation come from #110 by @grlee. That PR
+ * conflicted with main and lives on a fork, so it is reimplemented here rather
+ * than rebased.
+ *
+ * server/index.ts is deliberately NOT a target — it imports pkg.version
+ * directly, which is strictly better than syncing a literal. Anything that can
+ * import package.json should do that instead of being added below.
+ */
+
+import { readFileSync, writeFileSync } from "fs";
+import { join } from "path";
+
+const ROOT = join(import.meta.dir, "..");
+const CHECK_ONLY = process.argv.includes("--check");
+
+const version: unknown = JSON.parse(
+  readFileSync(join(ROOT, "package.json"), "utf8"),
+).version;
+
+if (typeof version !== "string" || version === "") {
+  console.error("sync-version: package.json is missing a version field");
+  process.exit(2);
+}
+
+interface Target {
+  /** Repo-relative path. */
+  path: string;
+  /** Every version field in the document, for reporting and for rewriting. */
+  fields: (doc: any) => { label: string; get: () => string; set: () => void }[];
+}
+
+const TARGETS: Target[] = [
+  {
+    path: ".claude-plugin/plugin.json",
+    fields: (doc) => [
+      { label: "version", get: () => doc.version, set: () => { doc.version = version; } },
+    ],
+  },
+  {
+    path: ".claude-plugin/marketplace.json",
+    fields: (doc) => [
+      {
+        label: "metadata.version",
+        get: () => doc.metadata?.version,
+        // Creates metadata if absent. Guarding the write instead would let
+        // --check report drift that the fixer could never resolve.
+        set: () => {
+          doc.metadata ??= {};
+          doc.metadata.version = version;
+        },
+      },
+      ...(doc.plugins ?? []).map((plugin: any, i: number) => ({
+        label: `plugins[${i}].version`,
+        get: () => plugin.version,
+        set: () => { plugin.version = version; },
+      })),
+    ],
+  },
+];
+
+export interface Drift {
+  path: string;
+  label: string;
+  current: string;
+}
+
+/** Returns every field that disagrees with package.json. Never writes. */
+export function findDrift(): Drift[] {
+  const drift: Drift[] = [];
+  for (const target of TARGETS) {
+    const doc = JSON.parse(readFileSync(join(ROOT, target.path), "utf8"));
+    for (const field of target.fields(doc)) {
+      const current = field.get();
+      if (current !== version) drift.push({ path: target.path, label: field.label, current });
+    }
+  }
+  return drift;
+}
+
+function main(): void {
+  const drift = findDrift();
+
+  for (const d of drift) {
+    console.log(
+      `${CHECK_ONLY ? "DRIFT" : "sync"}: ${d.path} :: ${d.label} ${d.current} -> ${version}`,
+    );
+  }
+
+  if (drift.length === 0) {
+    console.log(`sync-version: all version fields already at ${version}`);
+    return;
+  }
+
+  if (CHECK_ONLY) {
+    console.error(
+      "sync-version: version fields are out of sync with package.json. Run `bun run sync-version`.",
+    );
+    process.exit(1);
+  }
+
+  // Rewrite only the files that actually drifted, preserving 4-space style.
+  const drifted = new Set(drift.map((d) => d.path));
+  for (const target of TARGETS) {
+    if (!drifted.has(target.path)) continue;
+    const full = join(ROOT, target.path);
+    const doc = JSON.parse(readFileSync(full, "utf8"));
+    for (const field of target.fields(doc)) field.set();
+    writeFileSync(full, `${JSON.stringify(doc, null, 4)}\n`);
+  }
+}
+
+if (import.meta.main) main();

package/server/index.ts

@@ -11,6 +11,9 @@ import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js"
 import { z } from "zod";
 import { existsSync } from "fs";
 import { join, resolve, dirname } from "path";
+// The version advertised in the MCP handshake. Imported rather than written
+// as a literal so it cannot drift from the published package (it had, to 0.6.0).
+import pkg from "../package.json" with { type: "json" };
 
 import { generateBones,
 generatePersonality,
@@ -125,7 +128,7 @@ function getInstructions(): string {
 const server = new McpServer(
   {
     name: "coding-buddy",
-    version: "0.6.0",
+    version: pkg.version,
   },
   {
     instructions: getInstructions(),

package/server/state.ts

@@ -454,6 +454,10 @@ export interface StatusState {
   reaction: string;
   muted: boolean;
   achievement: string;
+  /** Epoch ms the achievement was awarded; 0 when none. The statusline expires
+   *  the trophy banner on this, otherwise it latches into the shared
+   *  status.json and pins every session's bubble to the last unlock. */
+  achievementAt: number;
   frames: string[];
   compactFrames: string[];
   minimalFrames: string[];
@@ -543,6 +547,7 @@ export function writeStatusState(
     reaction: "",
     muted: muted ?? false,
     achievement: achievement ?? "",
+    achievementAt: achievement ? Date.now() : 0,
     frames,
     compactFrames,
     minimalFrames,

package/statusline/buddy-status.sh

@@ -58,6 +58,9 @@ STARS=$(jq -r '.stars // ""' "$STATE" 2>/dev/null)
 SHINY=$(jq -r '.shiny // false' "$STATE" 2>/dev/null)
 REACTION_FILE="$BUDDY_STATE_DIR/reaction.$SID.json"
 ACHIEVEMENT=$(jq -r '.achievement // ""' "$STATE" 2>/dev/null)
+# "absent" distinguishes a legacy status.json (no field at all) from an explicit
+# 0, which means "no achievement pending" and must not render.
+ACHIEVEMENT_AT=$(jq -r 'if has("achievementAt") then (.achievementAt // 0) else "absent" end' "$STATE" 2>/dev/null)
 LEVEL=$(jq -r '.level // 1' "$STATE" 2>/dev/null)
 MOOD=$(jq -r '.mood // "focused"' "$STATE" 2>/dev/null)
 
@@ -229,10 +232,10 @@ DETECTED_COLS="$COLS"
 DETECTED_ROWS="$ROWS"
 
 # ─── Reaction bubble (with TTL check) ────────────────────────────────────────
+# The achievement banner is resolved further down, once REACTION_TTL is known —
+# it expires on the same clock as a reaction. Without that it latches into the
+# shared status.json and pins every session's bubble to the last trophy.
 BUBBLE=""
-if [ -n "$ACHIEVEMENT" ] && [ "$ACHIEVEMENT" != "null" ] && [ "$ACHIEVEMENT" != "" ]; then
-    BUBBLE=$'\xf0\x9f\x8f\x86'" $ACHIEVEMENT"
-fi
 REACTION_TTL=900
 INNER_W=44
 MARGIN=8
@@ -307,6 +310,31 @@ _sweep_expired_reactions() {
 }
 
 _sweep_expired_reactions
+
+# Achievement banner: shown only while fresh. ACHIEVEMENT_AT is epoch ms, written
+# alongside the name by writeStatusState. A legacy status.json without the field
+# (pre-upgrade, or a snapshot fixture) is treated as fresh — the next write from
+# the server backfills it.
+#
+# Validity and age are separate gates on purpose. A zeroed or malformed
+# achievementAt means "nothing pending" and must never render, including under
+# reactionTTL=0 — that opt-out disables *expiry*, not the field's meaning.
+if [ -n "$ACHIEVEMENT" ] && [ "$ACHIEVEMENT" != "null" ]; then
+    ACH_FRESH=1
+    if [ "$ACHIEVEMENT_AT" != "absent" ]; then
+        case "$ACHIEVEMENT_AT" in
+            ''|0|*[!0-9]*) ACH_FRESH=0 ;;
+            *)
+                if [ "$REACTION_TTL" -gt 0 ] 2>/dev/null; then
+                    ACH_AGE=$(( ($(date +%s) * 1000 - ACHIEVEMENT_AT) / 1000 ))
+                    [ "$ACH_AGE" -ge "$REACTION_TTL" ] && ACH_FRESH=0
+                fi
+                ;;
+        esac
+    fi
+    [ "$ACH_FRESH" -eq 1 ] && BUBBLE=$'\xf0\x9f\x8f\x86'" $ACHIEVEMENT"
+fi
+
 REACTION=$(jq -r '.reaction // ""' "$REACTION_FILE" 2>/dev/null)
 if [ -n "$REACTION" ] && [ "$REACTION" != "null" ] && [ "$REACTION" != "" ]; then
     FRESH=0