pi-better-openai 0.1.12 → 0.1.14

package/README.md

@@ -36,7 +36,7 @@ pi install npm:pi-better-openai
 Codex pets are an OpenAI Codex app feature, so the floating overlay and pet picker are still controlled by Codex (`Settings → Appearance → Pets` or `/pet`). This extension can also render compatible custom pet spritesheets directly in pi's Better OpenAI footer.
 
 ```bash
-/pets wake          # render the selected pet, or the first ready pet
+/pets wake          # render the selected pet, or pick one if none is selected
 /pets wake <slug>   # render a specific ready pet
 /pets select <slug> # select a ready pet without changing visibility
 /pets tuck          # hide it

package/index.ts

@@ -66,7 +66,7 @@ import {
   listCodexPets,
   nextAnimationFrameDelayMs,
   PET_ANIMATION_ROWS,
-  deleteCodexPetKittyPlacement,
+  deleteCodexPetKittyImage,
   registerOpenAIPets,
   renderCodexPetFrame,
   resetCodexPetKittyCache,
@@ -79,7 +79,7 @@ const OPENAI_SETTINGS_COMMAND = "openai-settings";
 const FLAG = "fast";
 const PET_RESIZE_FREEZE_MS = 120;
 const PET_RENDER_CACHE_LIMIT = 48;
-const PET_AUTO_VALUE = "first ready pet";
+const PET_EMPTY_VALUE = "not selected";
 const SERVICE_TIER = "priority";
 type SettingsPickerItem = {
   id: string;
@@ -146,15 +146,15 @@ function petSizeCellsForPlacement(placement: PetPlacement, sizeCells: number): n
 
 function readyPetPickerValues(pets: CodexPetPackage[]): string[] | undefined {
   const readyPets = pets.filter((pet) => pet.hasSpritesheet);
-  return readyPets.length > 0 ? [PET_AUTO_VALUE, ...readyPets.map((pet) => pet.slug)] : undefined;
+  return readyPets.length > 0 ? readyPets.map((pet) => pet.slug) : undefined;
 }
 
 function petConfigPickerValue(cfg: ResolvedConfig): string {
-  return cfg.pets.slug || PET_AUTO_VALUE;
+  return cfg.pets.slug || PET_EMPTY_VALUE;
 }
 
 function petSlugFromPickerValue(value: string): string {
-  return value === PET_AUTO_VALUE ? "" : value;
+  return value === PET_EMPTY_VALUE ? "" : value;
 }
 
 function petPickerLookupKey(value: string): string {
@@ -177,9 +177,14 @@ function petPickerDescription(cfg: ResolvedConfig, pets: CodexPetPackage[]): str
   const readyPets = pets.filter((pet) => pet.hasSpritesheet);
   if (readyPets.length === 0)
     return "No ready custom pets found. Use /pets help to create one, then return here.";
-  const selectedPet = cfg.pets.slug ? findPickerPet(cfg.pets.slug, readyPets) : readyPets[0];
+  if (!cfg.pets.slug) {
+    const firstPet = readyPets[0];
+    const lastPet = readyPets[readyPets.length - 1];
+    return `No pet selected. Enter/Space/→ selects ${firstPet.name}; ← selects ${lastPet.name}.`;
+  }
+  const selectedPet = findPickerPet(cfg.pets.slug, readyPets);
   const selected = selectedPet
-    ? `${cfg.pets.slug ? "Selected" : "Auto"}: ${selectedPet.name} (${selectedPet.slug})`
+    ? `Selected: ${selectedPet.name} (${selectedPet.slug})`
     : `Selected: ${cfg.pets.slug}`;
   return `${selected}. Enter/Space/→ cycles pets; ← cycles back. Preview appears in the footer while this menu is open.`;
 }
@@ -462,7 +467,7 @@ export default function betterOpenAI(pi: ExtensionAPI): void {
   function takePetKittyCleanupSequence(): string {
     if (pendingPetKittyCleanupImageIds.size === 0) return "";
     const sequence = Array.from(pendingPetKittyCleanupImageIds)
-      .map((imageId) => deleteCodexPetKittyPlacement(imageId))
+      .map((imageId) => deleteCodexPetKittyImage(imageId))
       .join("");
     pendingPetKittyCleanupImageIds.clear();
     return sequence;
@@ -885,7 +890,7 @@ export default function betterOpenAI(pi: ExtensionAPI): void {
       `Image enabled: ${cfg.image.enabled}`,
       `Image default save: ${cfg.image.defaultSave}`,
       `Pet enabled: ${cfg.pets.enabled}`,
-      `Pet slug: ${cfg.pets.slug || "first ready pet"}`,
+      `Pet slug: ${cfg.pets.slug || PET_EMPTY_VALUE}`,
       `Pet placement: ${cfg.pets.placement}`,
       `Pet failed tool state: ${cfg.pets.failedToolState}`,
       `Pet idle emotes: ${cfg.pets.idleEmotes} (${cfg.pets.idleEmoteIntervalMs}ms)`,
@@ -1477,7 +1482,7 @@ export default function betterOpenAI(pi: ExtensionAPI): void {
 
       const next = writePetConfig(ctx, {
         enabled: true,
-        ...(slug ? { slug: selectedPet.slug } : {}),
+        slug: selectedPet.slug,
       });
       updateFooter(ctx);
       await refreshPet(ctx, next, true);
@@ -1957,7 +1962,7 @@ export const _test = {
   readRawConfig,
   supportsFast,
   combineInlinePetFooter,
-  PET_AUTO_VALUE,
+  PET_EMPTY_VALUE,
   readyPetPickerValues,
   petConfigPickerValue,
   petSlugFromPickerValue,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-better-openai",
-  "version": "0.1.12",
+  "version": "0.1.14",
   "description": "Personal pi extension that improves OpenAI with fast mode, usage stats, and footer polish.",
   "keywords": [
     "fast",

package/src/pets.ts

@@ -415,6 +415,10 @@ export function deleteCodexPetKittyPlacement(imageId: number): string {
   return `\x1b_Ga=d,d=i,i=${imageId},p=1,q=2\x1b\\`;
 }
 
+export function deleteCodexPetKittyImage(imageId: number): string {
+  return `\x1b_Ga=d,d=I,i=${imageId},q=2\x1b\\`;
+}
+
 function placeKittyImage(imageId: number, columns: number, rows: number): string {
   return `\x1b_Ga=p,i=${imageId},p=1,c=${columns},r=${rows},q=2,C=1\x1b\\`;
 }
@@ -467,7 +471,12 @@ function renderKittyPetFrame(
   const upload = frame.kittyUploaded ? "" : encodeKittyRawRgba(frame, frameImageId);
   frame.kittyUploaded = true;
   previousKittyFrameByPlacement.set(options.imageId, frameImageId);
-  const sequence = `${deletePrevious}${deleteCurrent}${upload}${placeKittyImage(frameImageId, columns, rows)}`;
+  // Recent pi-tui versions automatically free Kitty image IDs they own on changed lines.
+  // Pet frames are managed here and intentionally reused across animation loops, so keep
+  // the first Kitty command pointed at a harmless footer-placement ID, not a frame ID.
+  const hostCleanupSentinel =
+    frameImageId !== options.imageId ? deleteCodexPetKittyPlacement(options.imageId) : "";
+  const sequence = `${hostCleanupSentinel}${deletePrevious}${deleteCurrent}${upload}${placeKittyImage(frameImageId, columns, rows)}`;
   const lines: string[] = [];
   for (let i = 0; i < rows - 1; i++) lines.push("");
   const moveUp = rows > 1 ? `\x1b[${rows - 1}A` : "";