opencode-harness-kit 0.5.3 → 0.5.5

package/src/health.js

@@ -15,12 +15,96 @@
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs"
 import { join } from "node:path"
 
+/**
+ * @typedef {object} SpentEntry
+ * @property {string} at when the route was recorded as spent
+ * @property {string} reason the pattern that matched
+ */
+
 /**
  * @typedef {object} Health
- * @property {Record<string, { at: string, reason: string }>} unhealthy
+ * @property {Record<string, SpentEntry>} unhealthy
  * @property {string} [active]
  */
 
+/**
+ * How long a recorded spent route is honoured for, by default.
+ *
+ * The provider's free window resets at 00:00 UTC, so a route recorded as
+ * spent is expected to come back within a day. Without an expiry the file is
+ * a one-way ratchet: a fleet that hit one rate limit never uses that model
+ * again, its lanes drain onto the last rung of their chains, and the only
+ * way back is a human editing a JSON file by hand.
+ *
+ * 24h is the default because the provider's window is a day. The window
+ * belongs to the provider rather than to the kit, so it is configurable:
+ * HARNESS_SPENT_WINDOW_HOURS overrides it.
+ */
+const DEFAULT_WINDOW_MS = 24 * 60 * 60 * 1000
+
+/**
+ * The window, in milliseconds, a recorded spent route is honoured for.
+ *
+ * Read at call time rather than at import time, so a caller that changes it
+ * does not have to reload the module to see the effect.
+ *
+ * @returns {number}
+ */
+export function spentWindowMs() {
+  const raw = process.env.HARNESS_SPENT_WINDOW_HOURS
+  if (!raw) return DEFAULT_WINDOW_MS
+  const hours = Number(raw)
+  // A window of zero would expire every route the moment it was written, and
+  // a negative one would expire routes that were never spent. Neither is a
+  // setting anybody meant, so the default stands.
+  if (!Number.isFinite(hours) || hours <= 0) return DEFAULT_WINDOW_MS
+  return hours * 60 * 60 * 1000
+}
+
+/**
+ * Whether a recorded route is still spent.
+ *
+ * An entry with no usable timestamp is KEPT. The router always stamps one,
+ * so an entry without one was written by a hand, and honouring a standing
+ * statement is safer than putting a route that may be dead back into a
+ * fleet. An entry that cannot be shown to be stale is not dropped.
+ *
+ * @param {unknown} entry
+ * @param {number} [now]
+ * @returns {boolean}
+ */
+export function isSpent(entry, now = Date.now()) {
+  if (!entry || typeof entry !== "object" || Array.isArray(entry)) return false
+  const at = /** @type {any} */ (entry).at
+  if (typeof at !== "string") return true
+  const when = Date.parse(at)
+  if (Number.isNaN(when)) return true
+  return now - when <= spentWindowMs()
+}
+
+/**
+ * The spent routes that are still spent.
+ *
+ * Every reader of the file goes through here, so the router and the effort
+ * hook cannot disagree about which routes are down. A reader that looked at
+ * the raw file would see a route that came back an hour ago as spent, and
+ * would fail the fleet over onto a model that works.
+ *
+ * @param {Health} health
+ * @param {number} [now]
+ * @returns {Record<string, SpentEntry>}
+ */
+export function liveUnhealthy(health, now = Date.now()) {
+  /** @type {Record<string, SpentEntry>} */
+  const out = {}
+  const source = health?.unhealthy
+  if (!source || typeof source !== "object" || Array.isArray(source)) return out
+  for (const [key, entry] of Object.entries(source)) {
+    if (isSpent(entry, now)) out[key] = entry
+  }
+  return out
+}
+
 /**
  * @param {string} dir the project root
  * @returns {string}
@@ -30,31 +114,69 @@ export function healthPath(dir) {
 }
 
 /**
+ * The health file, or an empty one.
+ *
+ * A file that is not an object - null, an array, a number, a string - is not
+ * a health file, and reading it as one made `unhealthyModels` throw on a
+ * null. That call sits inside a bare catch in src/index.ts, so the throw
+ * silently skipped the effort plan AND the write of `.space/active` that the
+ * pruner depends on: a corrupt file switched the pruner off without a word.
+ *
  * @param {string} dir the project root
  * @returns {Health}
  */
 export function readHealth(dir) {
+  /** @type {any} */
+  let parsed
   try {
-    return JSON.parse(readFileSync(healthPath(dir), "utf8"))
+    parsed = JSON.parse(readFileSync(healthPath(dir), "utf8"))
   } catch {
     return { unhealthy: {} }
   }
+  if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) {
+    return { unhealthy: {} }
+  }
+  const unhealthy = parsed.unhealthy
+  if (!unhealthy || typeof unhealthy !== "object" || Array.isArray(unhealthy)) {
+    // The other keys are kept: `active` names the profile the fleet is
+    // running, and dropping it because the routes are malformed would lose
+    // the pruner's target as well.
+    return { ...parsed, unhealthy: {} }
+  }
+  return parsed
 }
 
 /**
+ * Write the health file, without the routes whose window has passed.
+ *
+ * The expiry is applied on the way out as well as on the way in, so a file
+ * that is read and written again - which the router does on every event -
+ * does not carry its dead routes forever.
+ *
  * @param {string} dir the project root
  * @param {Health} health
  */
 export function writeHealth(dir, health) {
   try {
     if (!existsSync(join(dir, ".space"))) mkdirSync(join(dir, ".space"), { recursive: true })
-    writeFileSync(healthPath(dir), JSON.stringify(health, null, 2))
+    const pruned = { ...(health ?? {}), unhealthy: liveUnhealthy(health) }
+    writeFileSync(healthPath(dir), JSON.stringify(pruned, null, 2))
   } catch {
     // Best effort. A health file that cannot be written means no failover,
     // and failing to write it must not break the session.
   }
 }
 
+/**
+ * The routes on disk that are still spent.
+ *
+ * @param {string} dir the project root
+ * @returns {Record<string, SpentEntry>}
+ */
+export function unhealthyRoutes(dir) {
+  return liveUnhealthy(readHealth(dir))
+}
+
 /**
  * The models a previous session found spent.
  *
@@ -67,13 +189,16 @@ export function writeHealth(dir, health) {
  * lane chains list bare model ids, so the prefix is dropped here. Comparing
  * the two directly would never match, and a spent route would look healthy.
  *
+ * A route whose window has passed is not spent any more, and is not returned
+ * here: the provider's quota comes back at 00:00 UTC, and a lane that stays
+ * on its fallback forever is a lane that never comes back either.
+ *
  * @param {string} dir the project root
  * @returns {Set<string>}
  */
 export function unhealthyModels(dir) {
-  const health = readHealth(dir)
   const out = new Set()
-  for (const key of Object.keys(health.unhealthy ?? {})) {
+  for (const key of Object.keys(unhealthyRoutes(dir))) {
     out.add(key.includes("/") ? key.slice(key.indexOf("/") + 1) : key)
   }
   return out

package/src/lane-usage.js

@@ -34,10 +34,11 @@ import { dirname, join, resolve } from "node:path"
  * The telemetry files for the project containing `startDir`.
  *
  * ALL of them, oldest first. The plugin rotates the file when it passes 4 MB
- * and renames the old one to `telemetry.jsonl.1` - and the reader used to
- * look at `telemetry.jsonl` alone, so a rotation silently threw away the
- * history and every lane reported "never used". The number that decides
- * whether the fleet stays at ten lanes was disappearing on a schedule.
+ * and keeps THREE generations - `telemetry.jsonl`, `telemetry.jsonl.1` and
+ * `telemetry.jsonl.2` - and the reader used to look at `telemetry.jsonl`
+ * alone, so a rotation silently threw away the history and every lane
+ * reported "never used". The number that decides whether the fleet stays at
+ * ten lanes was disappearing on a schedule.
  *
  * @param {string} startDir
  * @returns {string[]} the paths, or an empty array when the project has none
@@ -62,13 +63,19 @@ export function findTelemetryFiles(startDir) {
 }
 
 /**
- * The telemetry file for the project containing `startDir`.
+ * The CURRENT telemetry file for the project containing `startDir`.
+ *
+ * The current one, not the oldest one. The files are enumerated oldest
+ * first, so the current file is the LAST of them - and taking the first
+ * handed back the file that had been rotated away twice, which is the
+ * history rather than the present.
  *
  * @param {string} startDir
  * @returns {string | null} the path, or null when the project has no telemetry
  */
 export function findTelemetryFile(startDir) {
-  return findTelemetryFiles(startDir)[0] ?? null
+  const files = findTelemetryFiles(startDir)
+  return files.length ? files[files.length - 1] : null
 }
 
 /**

package/src/profile.js

@@ -180,6 +180,16 @@ export function loadProfile(id) {
     const msg = err instanceof Error ? err.message : String(err)
     throw new ProfileError(`cannot read ${path}: ${msg}`)
   }
+  // The file name IS the id. The generated header tells the user to edit
+  // `profiles/<id>.json` and to re-run `oc-harness use <id>`, and both
+  // come from this field, so a profile whose id disagrees with its file
+  // name sends them to a file that does not exist and to a command that
+  // cannot resolve. Nothing else would say so.
+  if (raw?.id && raw.id !== id) {
+    throw new ProfileError(
+      `${path}: profile id is "${raw.id}", which does not match the file name "${id}"`,
+    )
+  }
   validate(raw, path)
   return normalize(raw)
 }
@@ -463,25 +473,43 @@ export function resolveLaneChain(p, role) {
  * @returns {{ model: string, levels: string[] }}
  */
 export function resolveLaneModel(p, role, opts = {}) {
+  const model = laneModelId(p, role, opts)
+  return { model, levels: levelsOfModel(model) }
+}
+
+/**
+ * The model a lane runs on, decided without loading any profile.
+ *
+ * The split is what makes the answer safe to ask from inside `validate()`.
+ * Resolving a lane used to mean loading that model's profile to read its
+ * effort levels, and loading a profile validates it, and validating it
+ * resolves its lanes: a loop. The levels are not part of this question, so
+ * they are left out of it.
+ *
+ * @param {Profile} p
+ * @param {string} role
+ * @param {{ unhealthy?: Set<string> }} [opts]
+ * @returns {string}
+ */
+function laneModelId(p, role, opts = {}) {
   const unhealthy = opts.unhealthy ?? new Set()
   const ceiling = /** @type {Retention} */ (p.maxRetention ?? p.retention)
   for (const candidate of resolveLaneChain(p, role)) {
     if (unhealthy.has(candidate)) continue
     if (!retentionAllowed(candidate, ceiling)) continue
-    return { model: candidate, levels: levelsOfModel(candidate) }
+    return candidate
   }
   // Nothing in the chain qualifies. The base model is the next thing to try:
   // it is the profile's own route, and a lane pinned to a rung the router has
   // already recorded as spent is worse than a lane on a route that was never
   // preferred for this job but is alive.
   if (p.model && !unhealthy.has(p.model) && retentionAllowed(p.model, ceiling)) {
-    return { model: p.model, levels: levelsOfModel(p.model) }
+    return p.model
   }
   // Last resort: the first rung anyway rather than leaving the lane without a
   // model, and let the config header say what the effective retention turned
   // out to be.
-  const fallback = resolveLaneChain(p, role)[0] ?? p.model
-  return { model: fallback, levels: levelsOfModel(fallback) }
+  return resolveLaneChain(p, role)[0] ?? p.model
 }
 
 /**
@@ -578,29 +606,42 @@ export function resolveEffort(levels, requested) {
   // no variant" and let the model use its own default.
   if (requested === "none") return null
 
+  // `none` is not a candidate either, for the same reason: a lane that asked
+  // for `minimal` and was handed `none` had its reasoning switched off, which
+  // is the opposite of what it asked for. The distance search used to count
+  // `none` as the rung below `minimal`, so it won every near miss.
+  const rungs = levels.filter(
+    (level) => level !== "none" && CANON.indexOf(level) >= 0,
+  )
+
   const want = CANON.indexOf(requested)
   if (want < 0) {
-    // Unknown label. Snap to the top level, which is the safe reading of a
+    // Unknown label. Snap to the top rung, which is the safe reading of a
     // role that asked for something strong.
-    return levels[levels.length - 1] ?? null
+    return rungs[rungs.length - 1] ?? null
   }
   let best = null
   let bestDist = Infinity
-  for (const level of levels) {
+  for (const level of rungs) {
     const have = CANON.indexOf(level)
-    if (have < 0) continue
     const d = Math.abs(have - want)
     if (d < bestDist) {
       bestDist = d
       best = level
     }
   }
-  return best ?? levels[levels.length - 1] ?? null
+  return best ?? rungs[rungs.length - 1] ?? null
 }
 
 // --- validation -------------------------------------------------------------
 
-const VALID_RETENTION = ["zero-retention", "retained", "may-train", "unknown"]
+// The retention levels a profile may DECLARE. `unknown` is not one of them:
+  // it is the absence of a level, and a profile that says "nobody has stated
+  // the terms of this provider" is not a profile. It used to be accepted, and
+  // `normalize()` then derived a `maxRetention: "unknown"` from it - a value
+  // `validate()` refuses when it is written out by hand, and one the profile
+  // schema does not allow for `maxRetention` either.
+const DECLARABLE_RETENTION = ["zero-retention", "retained", "may-train"]
 
 /** @param {Profile} p @param {string} [path] */
 export function validate(p, path = "<memory>") {
@@ -608,25 +649,49 @@ export function validate(p, path = "<memory>") {
   if (!p?.id) throw new ProfileError(`${path}: missing "id"`)
   if (!p.provider) throw new ProfileError(`${where}: missing "provider"`)
   if (!p.model) throw new ProfileError(`${where}: missing "model"`)
-  if (!VALID_RETENTION.includes(p.retention)) {
-    throw new ProfileError(
-      `${where}: retention must be one of ${VALID_RETENTION.join(", ")}`,
-    )
+  if (p.retention === "unknown") {
+    throw new ProfileError(`${where}: retention cannot be unknown`)
   }
-  if (p.maxRetention && !VALID_RETENTION.includes(p.maxRetention)) {
+  if (!DECLARABLE_RETENTION.includes(p.retention)) {
     throw new ProfileError(
-      `${where}: maxRetention must be one of ${VALID_RETENTION.join(", ")}`,
+      `${where}: retention must be one of ${DECLARABLE_RETENTION.join(", ")}`,
     )
   }
   if (p.maxRetention === "unknown") {
     throw new ProfileError(`${where}: maxRetention cannot be unknown`)
   }
-  if (!p.limits?.context || p.limits.context <= 0) {
+  if (p.maxRetention && !DECLARABLE_RETENTION.includes(p.maxRetention)) {
+    throw new ProfileError(
+      `${where}: maxRetention must be one of ${DECLARABLE_RETENTION.join(", ")}`,
+    )
+  }
+  // The message said "must be a positive integer" and the check said "greater
+  // than zero", so 1.5 and "200000" were accepted. A context of 1.5 makes
+  // the prune threshold NEGATIVE, because the reserve has a floor of 8,000
+  // tokens and the cap is one and a half.
+  if (!Number.isInteger(p.limits?.context) || p.limits.context <= 0) {
     throw new ProfileError(`${where}: limits.context must be a positive integer`)
   }
-  if (!p.limits?.output || p.limits.output <= 0) {
+  if (!Number.isInteger(p.limits?.output) || p.limits.output <= 0) {
     throw new ProfileError(`${where}: limits.output must be a positive integer`)
   }
+  // The levels are written into the config as each lane's `variant`, and
+  // opencode cannot resolve a variant the model does not publish. The scale
+  // is the canonical one: `none` is reasoning off, and the rest are the
+  // reasoning levels.
+  for (const level of p.effort?.levels ?? []) {
+    if (!CANON.includes(level)) {
+      throw new ProfileError(
+        `${where}: effort.levels contains "${level}", which is not one of ${CANON.join(", ")}`,
+      )
+    }
+  }
+  const wanted = p.effort?.default
+  if (wanted !== undefined && wanted !== null && !CANON.includes(wanted)) {
+    throw new ProfileError(
+      `${where}: effort.default is "${wanted}", which is not one of ${CANON.join(", ")}`,
+    )
+  }
   const share = p.effort?.reserveShare
   if (share !== undefined && (share < 0 || share > 0.5)) {
     throw new ProfileError(
@@ -743,14 +808,25 @@ export function validate(p, path = "<memory>") {
       )
     }
   }
+  // The base model is the model the config names as the session model, and
+  // the one every lane falls back to. It was never checked against
+  // retention.yml, so a profile could name a model nobody had stated the
+  // terms of and the header would print a level for a model that does not
+  // have one.
+  if (!retentionFor(p.model)) {
+    throw new ProfileError(
+      `${where}: model ${p.model} has no retention.yml entry`,
+    )
+  }
   // The utility route has to be a model this profile can actually run, or a
   // failover leaves opencode's short auxiliary calls pointing at a model that
   // is not in the fleet.
   //
   // The check is STATIC - the union of the profile's own model and every
-  // model its chains name. Asking `modelsInPlay` would be the obvious way,
-  // and it is a loop: resolving a lane reads the levels of a model, which
-  // loads that model's profile, which validates it, which resolves a lane.
+  // model its chains name - and it has to be: the route has to be reachable
+  // at all, not reachable today. A chain rung that is not resolving right
+  // now is still a route the profile runs when the rung above it is spent,
+  // and the utility route is written into the config once and kept.
   if (p.utility) {
     if (typeof p.utility.model !== "string" || !p.utility.model) {
       throw new ProfileError(`${where}: utility.model must be a model id`)
@@ -765,6 +841,15 @@ export function validate(p, path = "<memory>") {
         `${where}: utility.model is ${p.utility.model}, which this profile does not resolve to (it has ${[...reachable].join(", ")})`,
       )
     }
+    // The route is written into the config and serves real requests, so its
+    // terms have to be on record. A utility route with no retention.yml
+    // entry is an `unknown` in the header and a refusal at install time;
+    // saying so here names the field instead of leaving a surprise.
+    if (!retentionFor(p.utility.model)) {
+      throw new ProfileError(
+        `${where}: utility.model ${p.utility.model} has no retention.yml entry`,
+      )
+    }
   }
 
   // The fallback queue is walked at startup, and an id that does not resolve
@@ -777,6 +862,37 @@ export function validate(p, path = "<memory>") {
       )
     }
   }
+
+  // The retention the profile DECLARES may be equal to the worst level
+  // among the models its own base and lanes resolve to, or WORSE than that
+  // level. It may not be BETTER.
+  //
+  // The claim was never checked against retention.yml, so a profile could
+  // say "zero-retention" while a lane ran on a provider that may train -
+  // and the config header printed the claim as fact.
+  //
+  // The check is by SEVERITY and not by equality. Equality refused the
+  // conservative declaration: a profile that says "retained" while every
+  // model it runs on is zero-retention was rejected as though it had lied.
+  // It had not - it had promised less than the models give, and a promise
+  // that is stricter than the truth is the safe direction to err in. Only
+  // one direction is a lie, and it is the one the check now refuses.
+  //
+  // `retentionSeverity` is the same table the ceiling check uses, so the
+  // two cannot disagree about which of two levels is the worse one.
+  //
+  // The utility route is NOT part of this comparison. It is an extra route
+  // the profile adds on top of its own models, and it is reported where it
+  // belongs: the header names it as the reason the file is worse than the
+  // profile claims, and `oc-harness use` warns about it. A profile that
+  // names a may-train utility route is therefore installable, and the file
+  // it writes says may-train.
+  const actual = worstRetention(laneModelsInPlay(p))
+  if (retentionSeverity(p.retention) < retentionSeverity(actual.retention)) {
+    throw new ProfileError(
+      `${where}: retention is ${p.retention}, but ${actual.worst} is ${actual.retention} - a profile cannot be more private than the models it runs on`,
+    )
+  }
 }
 
 /**
@@ -840,17 +956,53 @@ export function pruneThreshold(p, opts = {}) {
 }
 
 /**
- * Every model a profile can put behind the user: the base model plus the model
- * each lane actually resolves to.
+ * Every model a profile can put behind the user: the base model, the model
+ * each lane actually resolves to, and the utility route.
+ *
+ * The utility route belongs in this list because it is WRITTEN into the
+ * config as `small_model` and opencode sends real requests down it - a
+ * title, a one-line summary, a rewritten question. It used to be a chain
+ * rung only, so it never appeared here, and a profile could name a
+ * may-train utility route under a zero-retention claim: the file said
+ * "zero-retention" while a training provider served the calls.
+ *
+ * Every caller of this list wants the same thing - the models that can
+ * serve a request - so the list is widened rather than a second one
+ * invented beside it. The one caller that needs the lanes alone is
+ * `validate()`, which cross-checks the profile's DECLARED retention against
+ * the base model and the lanes; that is `laneModelsInPlay`.
  *
  * @param {Profile} p
  * @param {{ unhealthy?: Set<string> }} [opts]
  * @returns {string[]}
  */
 export function modelsInPlay(p, opts = {}) {
+  const out = laneModelsInPlay(p, opts)
+  const utility = p.utility?.model
+  if (typeof utility === "string" && utility && !out.includes(utility)) {
+    out.push(utility)
+  }
+  return out
+}
+
+/**
+ * The models the profile's own base and lanes put behind the user, without
+ * the utility route.
+ *
+ * This is the set the profile's own `retention` field is about: the base
+ * model and the lanes it resolves to. The utility route is an extra route
+ * the profile adds on top of them, and it is reported where it belongs - the
+ * config header names it, and `oc-harness use` warns about it - so it does
+ * not decide what the profile may declare about itself.
+ *
+ * @param {Profile} p
+ * @param {{ unhealthy?: Set<string> }} [opts]
+ * @returns {string[]}
+ */
+export function laneModelsInPlay(p, opts = {}) {
   const out = [p.model]
   for (const role of Object.keys(p.roleModels ?? {})) {
-    const model = resolveLaneModel(p, role, opts).model
+    const model = laneModelId(p, role, opts)
     if (!out.includes(model)) out.push(model)
   }
   return out
@@ -894,10 +1046,20 @@ export function roleSplitModels(p, opts = {}) {
  * @returns {{ retention: string, worst: string | null }}
  */
 export function effectiveRetention(p, opts = {}) {
+  return worstRetention(modelsInPlay(p, opts))
+}
+
+/**
+ * The worst retention among a list of models, and the model responsible.
+ *
+ * @param {string[]} ids
+ * @returns {{ retention: string, worst: string | null }}
+ */
+function worstRetention(ids) {
   /** @type {string | null} */
   let worst = null
   let retention = "zero-retention"
-  for (const id of modelsInPlay(p, opts)) {
+  for (const id of ids) {
     const declared = modelRetention(id)
     if (
       !worst ||

package/src/prune.ts

@@ -66,7 +66,6 @@ interface StubStats {
   bytesAfter: number
 }
 
-/** Rough size of a part, used only for the stats line. */
 /**
  * The identity of a tool call's output: the tool and what it was asked for.
  *
@@ -88,14 +87,37 @@ function toolSignature(part: any): string | null {
   return `${tool} ${input}`
 }
 
-function sizeOf(part: any): number {
-  // CHARACTERS, not bytes. A UTF-8 character is more than one byte, and a
-  // report that says "bytes" and counts characters is off by a factor on
-  // any text that is not ASCII - which is most of the code in a repository.
-  // The field is named for what it measures.
-  if (typeof part?.text === "string") return part.text.length
-  if (typeof part?.state?.output === "string") return part.state.output.length
-  return 0
+/**
+ * The size of a PART, as it sits in the request.
+ *
+ * The whole part, not the text inside it. The saving this plugin reports is
+ * a claim about the REQUEST, and the request is built out of parts - so
+ * measuring `part.text` measured a piece of a part and called the difference
+ * a saving. The stub replaces a tool result with a new state object that
+ * carries its own metadata, and that metadata was invisible to the measure,
+ * so a turn whose request GREW was reported as a saving of the difference
+ * between two texts. That is the worst bug this plugin can have: a number
+ * that says "I saved you 322 characters" for a request that grew by 406.
+ *
+ * The metadata the stub adds is counted as part of the stub, deliberately:
+ * it is bytes the request really does carry, so a stub that costs more than
+ * the output it replaced has to show up as a cost rather than as a saving.
+ *
+ * CHARACTERS, not bytes. A UTF-8 character is more than one byte, and a
+ * report that says "bytes" and counts characters is off by a factor on
+ * any text that is not ASCII - which is most of the code in a repository.
+ * The field is named for what it measures.
+ */
+function partSize(part: any): number {
+  try {
+    const json = JSON.stringify(part)
+    return typeof json === "string" ? json.length : 0
+  } catch {
+    // A part that cannot be serialised cannot be measured either. It counts
+    // as zero rather than throwing: a prune that cannot measure itself must
+    // still prune.
+    return 0
+  }
 }
 
 /**
@@ -261,7 +283,7 @@ export const PrunePlugin: Plugin = async ({ worktree, directory }) => {
           if (!msg?.parts) continue
           const isUser = msg.info?.role === "user"
           for (const part of msg.parts) {
-            const size = sizeOf(part)
+            const size = partSize(part)
             stats.bytesBefore += size
 
             if (isUser && keepUsers) {
@@ -295,8 +317,14 @@ export const PrunePlugin: Plugin = async ({ worktree, directory }) => {
                 // part schema has no signature, so replacing its text is
                 // safe. A part that grew one would have to be removed rather
                 // than rewritten.
+                //
+                // There is NO floor under which a reasoning part is left
+                // alone. A short one is replaced too, and the stub is about
+                // seventy-eight characters - so a reasoning part shorter
+                // than that makes the request BIGGER. The log says so,
+                // because the log is a measurement and not a claim.
                 part.text = REASONING_STUB
-                stats.bytesAfter += sizeOf(part)
+                stats.bytesAfter += partSize(part)
                 stats.stubbed++
                 break
               }
@@ -310,6 +338,15 @@ export const PrunePlugin: Plugin = async ({ worktree, directory }) => {
                   stats.kept++
                   break
                 }
+                // The stub is a NEW state object, and the new state carries
+                // its own metadata: a flag and a timestamp, about fifty
+                // characters per stubbed part.
+                //
+                // Those characters are part of the stub, and they are
+                // measured as part of it. They used to be invisible to the
+                // measurement, so the log reported the difference between
+                // two TEXTS while the request carried the metadata as well -
+                // and a request that grew was reported as a saving.
                 part.state = {
                   status: "completed",
                   input: state.input,
@@ -318,7 +355,7 @@ export const PrunePlugin: Plugin = async ({ worktree, directory }) => {
                   time: state.time,
                   metadata: { ...(state.metadata ?? {}), pruned: true, prunedAt: new Date().toISOString() },
                 }
-                stats.bytesAfter += sizeOf(part)
+                stats.bytesAfter += partSize(part)
                 stats.stubbed++
                 break
               }
@@ -336,7 +373,15 @@ export const PrunePlugin: Plugin = async ({ worktree, directory }) => {
         }
 
         if (stats.stubbed > 0) {
-          const saved = Math.max(0, stats.bytesBefore - stats.bytesAfter)
+          // The delta of the REQUEST, exactly, and not floored at zero.
+          //
+          // The saving is a claim about the request, and a request that grew
+          // has to say so. A zero would be read as "nothing was saved" and
+          // would hide the fact that the pruner made the request BIGGER -
+          // which is the failure this number exists to prevent. The line is
+          // still logged either way, because a part was stubbed, and that is
+          // the event worth recording.
+          const saved = stats.bytesBefore - stats.bytesAfter
           await logPrune(project, profile.id, {
             ...stats,
             saved,