pi-condense 2.4.1 → 2.4.2

package/CHANGELOG.md

@@ -7,6 +7,11 @@ Published to npm as [`pi-condense`](https://www.npmjs.com/package/pi-condense) (
 Pushing a `vX.Y.Z` tag triggers `.github/workflows/release.yml`, which runs the tests and
 publishes via OIDC trusted publishing. See `.agents/skills/release/SKILL.md`.
 
+## [2.4.2] - 2026-08-04
+
+- **No-op renders skip context serialization.** `pruneMessages` (`src/pruner.ts`) computed `sizeMessages` (a full `JSON.stringify` over the entire message array) unconditionally on every `context` render, including no-ops where the result is never read (`index.ts` consumes `beforeChars`/`afterChars` only under `if (result.pruned)`). It now computes both sizes lazily in the pruned branch and returns a `{ beforeChars: 0, afterChars: 0 }` sentinel on a no-op, so a render that prunes nothing does zero `JSON.stringify` over the array. CPU/GC only - zero token cost, no wire or return-shape change. Also corrects a stale fast-path comment in the `context` handler (`index.ts`): index/registry emptiness alone does not imply a no-op, because error-purge and thinking-strip prune independently.
+- **Bumped `engines.node` to `>=22.19.0`** to match the host pi runtime (`@earendil-works/pi-coding-agent@0.83.0`); drops advertised support for Node 20/21, which cannot run current pi. Bumped the `@sinclair/typebox` devDependency floor to `^0.34.52` (freshness only; the caret already resolved there).
+
 ## [2.4.1] - 2026-07-30
 
 - **Fix 421 Misdirected Request for GitHub Copilot business/enterprise seats.** The summarizer called pi-ai's `stream()` with the static model definition, whose shipped `baseUrl` pins the individual Copilot host (`api.individual.githubcopilot.com`); business/enterprise tokens are rejected there with `421 Misdirected Request`, so any `github-copilot/*` `summarizerModel` failed on those seats. `runOnce` now mirrors the main agent loop (pi's `model-runtime`): it resolves provider auth via `ctx.modelRegistry.getProviderAuth(model.provider)` and, when the resolved auth carries a seat-specific `baseUrl`, rebases the model onto it before streaming. Also bumps `@earendil-works/*` devDependencies to `^0.83.0` (needed for `getProviderAuth` on the extension-facing `ModelRegistry`) and imports `stream` from `@earendil-works/pi-ai/compat` (its export moved off the root in current pi-ai).

package/index.ts

@@ -818,9 +818,11 @@ export default function (pi: ExtensionAPI) {
     let changed = false;
 
     // pruneMessages is the single source of truth for "is there work to do".
-    // It fast-paths (returns the original array reference) when both the
-    // tool-call index and chain registry are empty, so calling it
-    // unconditionally is safe and avoids a split gate here.
+    // It returns the original array reference (pruned: false) only when none of
+    // the four phases changed anything; index/registry emptiness alone does not
+    // imply a no-op, since error-purge (phase 2) and thinking-strip (phase 4)
+    // prune independently of them. Calling it unconditionally is safe and avoids
+    // a split gate here.
     const result = pruneMessages(
       messages,
       indexer,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-condense",
-  "version": "2.4.1",
+  "version": "2.4.2",
   "description": "Pi coding-agent extension that summarizes completed tool-call batches, replaces raw outputs with short stubs, compresses closed tool-call chains, and recovers any original on demand via context_tree_query.",
   "author": "Jacek Juraszek",
   "license": "MIT",
@@ -27,7 +27,7 @@
     "ai"
   ],
   "engines": {
-    "node": ">=20.3.0"
+    "node": ">=22.19.0"
   },
   "files": [
     "index.ts",
@@ -60,6 +60,6 @@
     "@earendil-works/pi-ai": "^0.83.0",
     "@earendil-works/pi-coding-agent": "^0.83.0",
     "@earendil-works/pi-tui": "^0.83.0",
-    "@sinclair/typebox": "^0.34.49"
+    "@sinclair/typebox": "^0.34.52"
   }
 }

package/src/pruner.test.ts

@@ -591,14 +591,17 @@ describe("sizeMessages", () => {
 });
 
 describe("pruneMessages beforeChars/afterChars", () => {
-  it("no-op fast path: beforeChars === afterChars === sizeMessages(input) and pruned false", () => {
+  it("no-op returns {0,0} sentinel and pruned false (no serialization on no-op path)", () => {
     const indexer = makeMockIndexer();
+    // Non-empty input: a reverted fix that recomputes sizeMessages(messages)
+    // on the unconditional path would yield a nonzero beforeChars here and fail.
     const messages = [{ role: "user", content: "hello", timestamp: 1 }];
     const result = pruneMessages(messages, indexer);
     expect(result.pruned).toBe(false);
-    const expected = sizeMessages(messages);
-    expect(result.beforeChars).toBe(expected);
-    expect(result.afterChars).toBe(expected);
+    expect(result.beforeChars).toBe(0);
+    expect(result.afterChars).toBe(0);
+    // No-op returns the original array reference unchanged.
+    expect(result.messages).toBe(messages);
   });
 
   it("pruning path: beforeChars > afterChars when stubs shrink content", () => {

package/src/pruner.ts

@@ -55,6 +55,12 @@ export function sizeMessages(messages: any[]): number {
  *   - `pruned: false` — nothing matched; the returned `messages` is the
  *     **original input array reference** so the caller can cheaply skip
  *     the reconstruction path.
+ *   - `beforeChars` / `afterChars` — serialized context size (`sizeMessages`)
+ *     before and after pruning when `pruned` is true. When `pruned` is false
+ *     both are `0`: a no-op sentinel, not a measurement — the size is never
+ *     computed on the no-op path (zero `JSON.stringify` over the array), and
+ *     the only consumer (`index.ts` live-reclaim) reads them solely under
+ *     `if (result.pruned)`.
  *
  * AssistantMessage tool-call blocks (which carry the IDs) are kept
  * unchanged so the model can still reference them by id when calling
@@ -69,7 +75,6 @@ export function pruneMessages(
   protection?: ProtectionConfig,
   recoveryGraceTurns: number = 0,
 ): { messages: any[]; pruned: boolean; beforeChars: number; afterChars: number } {
-  const beforeChars = sizeMessages(messages);
   // Phase 1: stub-replace summarized tool results
   let pruned = false;
   const inGrace = inGraceRecoveryToolCallIds(messages, recoveryGraceTurns);
@@ -158,5 +163,7 @@ export function pruneMessages(
     }
   }
 
-  return { messages: current, pruned, beforeChars, afterChars: pruned ? sizeMessages(current) : beforeChars };
+  return pruned
+    ? { messages: current, pruned, beforeChars: sizeMessages(messages), afterChars: sizeMessages(current) }
+    : { messages, pruned, beforeChars: 0, afterChars: 0 };
 }