@sayknow-cli/coding-agent 0.4.5 → 0.4.7

package/CHANGELOG.md

@@ -5,6 +5,35 @@ This file tracks the **fork's own releases**; upstream's full feature history li
 in that project. Each release notes the upstream version it is built on.
 
 
+## [0.4.7] — 2026-07-24
+
+### Fixed (Sayknow Pet scroll under tmux)
+
+- **Wrap the pet's sixel draw (save-cursor + position + sixel + restore-cursor)
+  in a single tmux DCS passthrough unit.** The pet uses `DECSC`/`DECRC`
+  (`\x1b7`/`\x1b8`) to stay cursor-neutral, but those bytes were emitted outside
+  the passthrough envelope, so only tmux saw them — the outer terminal received
+  just the cursor-advancing sixel via passthrough, never restored its cursor,
+  and scrolled the whole viewport up on every animation frame (leaving the pet
+  pinned in place). The save/restore now travels with the sixel through
+  passthrough, so the outer cursor returns and nothing scrolls. The footprint
+  clears stay tmux-level so tmux still repaints the vacated cells. Only affected
+  tmux + a sixel terminal.
+
+## [0.4.6] — 2026-07-23
+
+### Fixed (Sayknow Pet sizing/animation under tmux)
+
+- **Wrap the terminal cell-pixel-size query (`CSI 16 t`) in tmux's DCS
+  passthrough envelope.** Under tmux the query was answered by tmux itself with
+  a wrong cell size, so the sixel pet was encoded far too large — it overflowed
+  the composer and its animation froze (the oversized sprite resolved to an
+  out-of-bounds position, so each animation frame's overlay payload became
+  `null` and never redrew). The outer terminal now reports its real cell size,
+  so the pet renders at its intended 2-row height and animates. Only affected
+  tmux + a sixel terminal (the 0.4.5 pet-in-tmux path); non-multiplexed
+  rendering was already correct.
+
 ## [0.4.5] — 2026-07-23
 
 ### Added (Sayknow Pet in tmux)

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@sayknow-cli/coding-agent",
-	"version": "0.4.5",
+	"version": "0.4.7",
 	"description": "Sayknow-CLI CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://sayknow-cli.com",
 	"author": "jaybeyond",
@@ -52,12 +52,12 @@
 		"@agentclientprotocol/sdk": "1.2.1",
 		"@babel/parser": "^7.29.3",
 		"@mozilla/readability": "^0.6.0",
-		"@sayknow-cli/stats": "0.4.5",
-		"@sayknow-cli/agent-core": "0.4.5",
-		"@sayknow-cli/ai": "0.4.5",
-		"@sayknow-cli/natives": "0.4.5",
-		"@sayknow-cli/tui": "0.4.5",
-		"@sayknow-cli/utils": "0.4.5",
+		"@sayknow-cli/stats": "0.4.7",
+		"@sayknow-cli/agent-core": "0.4.7",
+		"@sayknow-cli/ai": "0.4.7",
+		"@sayknow-cli/natives": "0.4.7",
+		"@sayknow-cli/tui": "0.4.7",
+		"@sayknow-cli/utils": "0.4.7",
 		"@puppeteer/browsers": "^2.13.0",
 		"@types/turndown": "5.0.6",
 		"@xterm/headless": "^6.0.0",

package/src/modes/components/sayknow-pet-widget.ts

@@ -4,6 +4,7 @@ import {
 	type Component,
 	type Container,
 	getCellDimensions,
+	isUnderTmux,
 	PARA_PARA_STEPS,
 	PET_SKINS,
 	type PetMode,
@@ -281,9 +282,6 @@ export class SayknowPetWidget {
 			sixelTopPaddingPx: protocol === "sixel" ? PET_SIXEL_DROP_PX : 0,
 			kittyCellYOffsetPx: protocol === "kitty" ? petKittyDropPx(cell.heightPx) : 0,
 			kittyImageId: protocol === "kitty" ? this.#kittyImageId : undefined,
-			// Forward the pet's sixel raster through tmux's DCS passthrough envelope
-			// when under tmux (no-op otherwise), so the outer terminal renders it.
-			wrapSixel: wrapTmuxPassthrough,
 		});
 		this.#framedEditor.setReserve(this.#pixel.columns + PET_SIDE_MARGIN);
 	}
@@ -493,6 +491,7 @@ export class SayknowPetWidget {
 		this.#frameCleanupAwaitingAck = false;
 		const { x, y } = pos;
 		let out = "";
+		const positioned = `\x1b[${y + 1};${x + 1}H${pixel.frames[this.#frame]}`;
 
 		if (pixel.protocol === "sixel") {
 			const footprint = { x, y, columns: pixel.columns, rows: pixel.rasterRows };
@@ -501,13 +500,21 @@ export class SayknowPetWidget {
 			}
 			if (clearPet) out += this.#clearSixelFootprint(footprint);
 			this.#lastSixelFootprint = footprint;
-		} else {
-			// A kitty frame emitted below (re)places the image, so cleanup is
-			// pending again even if a narrow-terminal pass consumed it earlier.
-			this.#kittyCleanupPending = true;
+			// Under tmux the sixel is forwarded to the OUTER terminal via passthrough
+			// and advances its cursor; the frame's surrounding \x1b7/\x1b8 only reach
+			// tmux, so the outer terminal never restores and scrolls the viewport on
+			// every animation frame. Wrap save-cursor + position + sixel + restore-
+			// cursor as one passthrough unit so the outer cursor returns and nothing
+			// scrolls. The footprint clears above stay tmux-level so tmux repaints the
+			// vacated cells.
+			out += isUnderTmux() ? wrapTmuxPassthrough(`\x1b7${positioned}\x1b8`) : positioned;
+			return out;
 		}
 
-		out += `\x1b[${y + 1};${x + 1}H${pixel.frames[this.#frame]}`;
+		// A kitty frame (re)places the image cursor-neutrally, so cleanup is pending
+		// again even if a narrow-terminal pass consumed it earlier.
+		this.#kittyCleanupPending = true;
+		out += positioned;
 		return out;
 	}
 }