@diegopetrucci/pi-extensions 0.1.78 → 0.1.80

package/extensions/dynamic-context-pruning/index.ts

@@ -140,12 +140,6 @@ export function normalizeSessionProjection(value: unknown): MinimalSessionProjec
   return { entries, messages: entries.flatMap((entry) => entry.messages) };
 }
 
-export function sessionProjectionToMessages(
-  projection: MinimalSessionProjection,
-): MinimalMessage[] {
-  return projection.entries.flatMap((entry) => entry.messages);
-}
-
 const EXTENSION_ID = "dynamic-context-pruning";
 const CONFIG_FILE_NAME = "dynamic-context-pruning.json";
 const DECISION_ENTRY_TYPE = "dynamic-context-pruning:decision";
@@ -164,37 +158,10 @@ const DEFAULT_RECENT_TURNS = 4;
 /** Cache cost model default: fraction of full (non-cached) price still paid for cached tokens. */
 const DEFAULT_CACHED_PRICE_RATIO = 0.1;
 
-/**
- * Default net-benefit gate threshold (max amortization calls). Recalibrated
- * from the representative-corpus benchmark (~/.the-last-harness/
- * agent/sessions, 1,390+ session files, 556 candidates): this is the
- * hindsight-optimal break-even T at cachedPriceRatio r=0.1 (aggressive
- * prompt caching, the common Anthropic case). IMPORTANT caveats:
- *  - Provider/ratio-dependent: this optimum shifts higher as caching gets
- *    weaker (T=29 at r=0.25, T=54 at r=0.5, T=58 at r=0.9 on the same
- *    corpus) — see docs/v2-design.md section 4 and the extension README.
- *  - At r=0.1 the realized deterministic savings this threshold buys are
- *    economically marginal (~20.6k token-units total across the whole
- *    corpus, i.e. pennies); savings only become material at r>=0.25.
- *  - Re-derive via `scripts/benchmark.mjs <sessions-dir> --ratio 0.1` if
- *    the representative corpus changes meaningfully.
- */
+/** Default idle net-benefit gate threshold; see README.md and docs/v2-design.md §4. */
 const DEFAULT_BREAK_EVEN_THRESHOLD = 22;
 
-/**
- * Default mid_loop break-even threshold. The package's own
- * representative-corpus benchmark (turn-START state definition — the same one
- * the runtime `context` handler uses) favored rejecting mid_loop candidates
- * at cachedPriceRatio r=0.1 (mid_loop-optimal T=1, total realized
- * net benefit ~0.0). This supports a conservative default for the measured
- * corpus, not a universal claim that mid_loop pruning cannot pay off.
- * The reported 2026-09-08 byte-level capture (a mid_loop prune at T=22
- * invalidating 81% of a warm request prefix) illustrates cache invalidation,
- * not an inability to amortize it: enough subsequent calls can recoup a
- * one-time bust. Idle keeps T=22; turn-start classification does not imply
- * the provider cache is cold. Users who want the old behavior can override
- * via gate.breakEvenThresholdByState.mid_loop.
- */
+/** Default mid_loop threshold; see README.md for the runtime-aligned benchmark rationale. */
 const DEFAULT_MID_LOOP_BREAK_EVEN_THRESHOLD = 1;
 
 /** Tool names never pruned by default: orchestration/subagent-style tools and todo-like state. */
@@ -243,18 +210,7 @@ export interface PruneProtections {
 }
 
 export interface PruneThresholds {
-  /**
-   * Minimum raw text characters a fresh AUTOMATIC prune proposal must remove
-   * (original text chars minus placeholder chars) to be considered at all
-   * Enforced by `runDynamicContextPruningPipeline` BEFORE the
-   * net-benefit gate (see `PruneGateConfig`/`gate`, which uses break-even token
-   * math and is a separate, later check): text-only candidates below this floor
-   * are dropped up front and never reach gate/batch consideration. Image-bearing
-   * tool-result candidates bypass this raw-text floor (images receive no
-   * synthetic character count) and proceed to the token-based gate. Manual
-   * prunes and already-persisted/replayed decisions always bypass this floor,
-   * same as the gate. A value of 0 disables the floor (no filtering).
-   */
+  /** Minimum automatic-prune text saving; see README.md Configuration for bypass rules. */
   minCharsSaved: number;
 }
 
@@ -287,13 +243,15 @@ export interface PruneGateConfig {
   /** Fraction of full price still paid for cached ( r ). Default 0.1. */
   cachedPriceRatio: number;
   /**
-   * Default break-even threshold (max future calls to amortize a cache
-   * bust) used when no state-specific override applies. Recalibrated from
-   * the representative-corpus benchmark; see DEFAULT_BREAK_EVEN_THRESHOLD
-   * for the full rationale and ratio-sensitivity caveats.
+   * @deprecated Legacy global fallback retained for direct callers that pass
+   * an unnormalized gate object to resolveBreakEvenThreshold. normalizeConfig
+   * always supplies per-state values, so loaded config does not use this field
+   * to fill missing or invalid state values. Recalibrated from the
+   * representative-corpus benchmark; see DEFAULT_BREAK_EVEN_THRESHOLD for the
+   * full rationale and ratio-sensitivity caveats.
    */
   breakEvenThreshold: number;
-  /** Per-state overrides. Defaults: idle 22, mid_loop 1. */
+  /** Per-state overrides. normalizeConfig defaults to idle 22 and mid_loop 1. */
   breakEvenThresholdByState: { idle: number; mid_loop: number };
 }
 
@@ -493,29 +451,8 @@ export function estimateTokensForContent(
 export type TokenEstimator = (text: string) => number;
 
 /**
- * Classify the agent's current state from the message payload alone:
- * this is the runtime-observable definition used both by the live `context`
- * event handler and (via re-export) by the offline benchmark, so the two can
- * never drift apart.
- *
- * Walk backwards from the end of `messages`, skipping any message that is
- * neither `user`, `assistant`, nor `toolResult` (e.g. system-role bookkeeping
- * entries). The first such message found determines the state:
- *  - `user` (a plain user turn message) => "idle": this call is the FIRST LLM
- *    call of a turn, i.e. the decision is being made from a turn-start state.
- *  - `assistant` or `toolResult` (mid-chain) => "mid_loop": the agent is
- *    iterating (already made at least one call this turn, or is reacting to a
- *    tool result).
- *  - No such message found (empty history, or system-only) => "idle" (safe
- *    default: nothing suggests an in-progress loop).
- *
- * The benchmark (scripts/benchmark.mjs) reclassifies its candidate labels to
- * this SAME definition, reusing this function directly, so the
- * defaults below rest on evidence measured with the exact definition the
- * runtime observes. (An earlier turn-end-based benchmark definition -- "idle"
- * meant "this IS the turn's last assistant message", only knowable in
- * hindsight -- has been retired: the runtime cannot know in advance whether a
- * call is a turn's last one, so that definition was never runtime-observable.)
+ * Shared runtime/benchmark state classifier; see README.md Configuration and
+ * docs/v2-design.md §4 for the turn-start definition and retired turn-end analysis.
  */
 export function classifyAgentStateFromMessages(messages: MinimalMessage[]): AgentState {
   for (let i = messages.length - 1; i >= 0; i--) {
@@ -677,7 +614,7 @@ export function isWithinRecencyWindow(index: number, boundaryIndex: number): boo
  * Used by the error-input purge strategy's "older than N turns" rule; a
  * message at the very end of the conversation has 0 turns elapsed.
  */
-export function computeTurnsElapsedSince(messages: MinimalMessage[], index: number): number {
+function computeTurnsElapsedSince(messages: MinimalMessage[], index: number): number {
   let count = 0;
   for (let i = index + 1; i < messages.length; i++) {
     if (messages[i].role === "user") count++;
@@ -974,7 +911,7 @@ export function resolvePruneTombstoneState(entries: MinimalSessionEntry[]): Prun
   return { activeDecisions, activeIdempotencyKeys, restoredKeys, lastDecisionByKey };
 }
 
-export function buildPlaceholderText(reason: string, charsRemoved: number): string {
+function buildPlaceholderText(reason: string, charsRemoved: number): string {
   return `[pruned by ${EXTENSION_ID}: ${reason} (${charsRemoved} chars removed)]`;
 }
 
@@ -1581,7 +1518,7 @@ export function collectCompletedToolCallOccurrences(
 }
 
 /** Placeholder text for a duplicate tool result whose content was replaced. */
-export function buildDedupePlaceholder(toolName: string, newestToolCallId: string): string {
+function buildDedupePlaceholder(toolName: string, newestToolCallId: string): string {
   return `[pruned by ${EXTENSION_ID}: duplicate ${toolName} call result; see newest occurrence (call ${newestToolCallId})]`;
 }
 
@@ -1642,46 +1579,6 @@ export const errorPurgeStrategy: PruneStrategy = {
   },
 };
 
-// When the same file path is read and/or written multiple times in a
-// session, older read/write/edit tool *outputs* for that path can become
-// stale relative to a later operation. This strategy proposes replacing
-// those older outputs with a placeholder, per the following CONSERVATIVE,
-// binding rules (do not loosen without an architect decision):
-//
-//   1. A full-file read (no offset/limit args) is superseded by a LATER
-//      full-file read of the same normalized path.
-//   2. A partial read (offset/limit args present) is NOT superseded by a
-//      later partial read UNLESS the later read's range fully covers the
-//      earlier one's range. If range coverage can't be reliably determined
-//      from the args (e.g. earlier read has no limit — reads to EOF — but
-//      the later read is bounded), skip: do not propose supersession.
-//      Full-file reads are represented as the range [1, EOF); comparing
-//      that representation via the same coverage check naturally satisfies
-//      rule 1 as a special case of rule 2 (see `readRangeCovers`).
-//   3. ANY read of a path (full or partial) is superseded once a LATER
-//      *successful* (non-error) write/edit to that same path occurs — the
-//      old read output is now known-stale regardless of read-vs-read range
-//      comparisons. The placeholder explicitly notes the file has since
-//      changed. An ERRORED later write/edit does NOT count (the file may be
-//      unchanged; conservative).
-//   4. Older write/edit outputs for a path are superseded by a LATER
-//      *successful* write/edit to that same path (again, a later errored
-//      write/edit never supersedes). Write/edit tool outputs are usually
-//      small, so proposals are only ever emitted when there is a non-empty
-//      result to prune; the shared net-benefit gate additionally decides
-//      whether the estimated savings are worth the cache-bust cost.
-//
-// The NEWEST operation for a given path (by message order) is never
-// superseded/pruned, regardless of kind — there is nothing later to point
-// the placeholder at.
-//
-// Path extraction is defensive: pi's built-in `read`/`write`/`edit` tools
-// all accept a `path` argument (see packages/coding-agent/src/core/tools/
-// {read,write,edit}.ts); some tool-call renderers also fall back to a
-// legacy `file_path` name, and `filePath` is accepted as a further
-// defensive fallback. `bash` is explicitly OUT of scope: this strategy does
-// not attempt to parse shell commands for file paths.
-
 const SUPERSEDED_FILE_OPS_STRATEGY_ID = "superseded-file-ops";
 
 type FileOpKind = "read" | "write" | "edit";
@@ -1736,11 +1633,6 @@ export function computeFileReadRange(args: Record<string, unknown> | undefined):
   return { start, end };
 }
 
-/** True when a range represents a full-file read (starts at line 1, no limit). */
-export function isFullFileReadRange(range: FileReadRange): boolean {
-  return range.start === 1 && range.end === undefined;
-}
-
 /**
  * Conservative range-coverage check: does `later` fully cover `earlier`?
  * Implements rules 1+2 above via one unified representation (a full-file
@@ -2234,7 +2126,7 @@ export function sessionEntriesToMessages(
   entries: MinimalSessionEntry[],
   projection?: MinimalSessionProjection,
 ): MinimalMessage[] {
-  if (projection) return sessionProjectionToMessages(projection);
+  if (projection) return projection.entries.flatMap((entry) => entry.messages);
   const messages: MinimalMessage[] = [];
   for (const entry of entries) {
     if (entry.type === "message" && entry.message) messages.push(entry.message);
@@ -2373,11 +2265,9 @@ const STRATEGY_ALIASES: Record<string, StrategyKey> = {
   "error-purge": "errorPurge",
   errorpurge: "errorPurge",
   error_purge: "errorPurge",
-  errorPurge: "errorPurge",
   "superseded-file-ops": "supersededFileOps",
   supersededfileops: "supersededFileOps",
   superseded_file_ops: "supersededFileOps",
-  supersededFileOps: "supersededFileOps",
 };
 
 const STRATEGY_DISPLAY_NAMES: Record<StrategyKey, string> = {
@@ -2420,9 +2310,11 @@ export function parseContextPruningArgs(rawArgs: string): ContextPruningSubcomma
       return { kind: "toggle" };
     case "strategy": {
       const [nameRaw, valueRaw] = rest;
-      const key = nameRaw
-        ? (STRATEGY_ALIASES[nameRaw.toLowerCase()] ?? STRATEGY_ALIASES[nameRaw])
-        : undefined;
+      const normalizedName = nameRaw?.toLowerCase();
+      const key =
+        normalizedName && Object.hasOwn(STRATEGY_ALIASES, normalizedName)
+          ? STRATEGY_ALIASES[normalizedName]
+          : undefined;
       const value = valueRaw?.toLowerCase();
       if (!key || (value !== "on" && value !== "off")) return { kind: "unknown", raw: rawArgs };
       return { kind: "strategy", strategy: key, value: value === "on" };
@@ -2568,7 +2460,6 @@ function notify(
 }
 
 export const __testing = {
-  globToRegExp,
   getConfigPath,
   readConfig,
   writeConfig,

package/extensions/dynamic-context-pruning/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-dynamic-context-pruning",
-  "version": "0.1.13",
+  "version": "0.1.15",
   "type": "module",
   "description": "A pi extension that dynamically prunes stale context to keep sessions lean.",
   "keywords": [

package/extensions/fast/README.md

@@ -2,7 +2,7 @@
 
 A pi extension that provides `/fast` and `/ultrafast` toggles for supported direct OpenAI API, OpenAI Codex, and Anthropic Claude models. It selects the provider-specific Fast-mode request shape from the currently selected model.
 
-For eligible direct OpenAI API models it injects:
+For eligible direct OpenAI API-key models it injects:
 
 ```json
 {
@@ -10,6 +10,14 @@ For eligible direct OpenAI API models it injects:
 }
 ```
 
+For eligible direct OpenAI ChatGPT OAuth models it injects:
+
+```json
+{
+  "service_tier": "priority"
+}
+```
+
 For eligible OpenAI Codex OAuth models it injects:
 
 ```json
@@ -39,16 +47,15 @@ Direct OpenAI API Fast mode requires all of the following:
 - Provider `openai`.
 - API `openai-responses` or `openai-completions`.
 - Model `gpt-6.1-sol`, `gpt-6-astra`, `gpt-6-sol`, or `gpt-6-luna`.
-- API-key auth or Pi's `openai` ChatGPT OAuth (`Sign in with ChatGPT`).
+- API-key auth sets `service_tier: "fast"`; Pi's `openai` ChatGPT OAuth sets `priority`.
 - No existing `service_tier` field in the request payload.
 
-The direct OpenAI route stays on OpenAI's direct API contract for both API-key
-auth and Pi's `openai` ChatGPT OAuth. Acceptance of `service_tier: "fast"` by
-Pi's `openai` ChatGPT OAuth sign-in endpoint has not been live-verified.
-This direct route is never forwarded to legacy `openai-codex` or its
-subscription-usage endpoint. The documented request shape is support evidence,
-not live certification; this change was validated only with mocked eligibility
-and request payloads, not live or paid provider calls.
+The direct OpenAI route stays on OpenAI's direct API contract but selects the tier by auth
+type. OAuth uses `priority` because its endpoint rejected `fast`; acceptance or application
+of `priority` on Pi's OAuth endpoint has not been live-verified. This direct route is never
+forwarded to legacy `openai-codex` or its subscription-usage endpoint. The request shape is
+support evidence, not certification; tests use mocked eligibility and payloads only, with
+no live or paid provider calls.
 
 ### Ultrafast support (initial, fail-closed)
 

package/extensions/fast/index.ts

@@ -155,7 +155,7 @@ function describeMode(state: SessionState): string {
   return state.config.enabled ? "on (config default)" : "off (config default)";
 }
 
-function getEligibility(ctx: ExtensionContext): Eligibility {
+function getEligibility(ctx: ExtensionContext, includeServiceTier = true): Eligibility {
   const model = ctx.model;
   if (!model) {
     return { eligible: false, modelKey: "no-model", reason: "no model is selected" };
@@ -237,7 +237,11 @@ function getEligibility(ctx: ExtensionContext): Eligibility {
       eligible: true,
       modelKey: key,
       provider: "openai",
-      serviceTier: OPENAI_API_FAST_SERVICE_TIER,
+      serviceTier: includeServiceTier
+        ? ctx.modelRegistry.isUsingOAuth(model)
+          ? OPENAI_FAST_SERVICE_TIER
+          : OPENAI_API_FAST_SERVICE_TIER
+        : undefined,
     };
   }
 
@@ -328,7 +332,7 @@ function updateStatus(ctx: ExtensionContext, state: SessionState): void {
     return;
   }
 
-  const eligibility = getEligibility(ctx);
+  const eligibility = getEligibility(ctx, false);
   ctx.ui.setStatus(EXTENSION_ID, isFastEnabled(state) && eligibility.eligible ? "fast" : undefined);
 }
 
@@ -346,7 +350,7 @@ function getStatusMessage(ctx: ExtensionContext, state: SessionState): string {
   }
 
   const enabled = isFastEnabled(state);
-  const eligibility = getEligibility(ctx);
+  const eligibility = getEligibility(ctx, enabled);
   const active = enabled && eligibility.eligible;
 
   if (active) {
@@ -405,7 +409,7 @@ async function injectAnthropicFastHeader(
   hasActiveTools: boolean,
 ): Promise<void> {
   if (state.ultrafast || !isFastEnabled(state)) return;
-  const eligibility = getEligibility(ctx);
+  const eligibility = getEligibility(ctx, false);
   if (!eligibility.eligible || eligibility.provider !== "anthropic") return;
 
   const matchingKeys = Object.keys(headers).filter(

package/extensions/fast/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diegopetrucci/pi-fast",
-  "version": "0.1.9",
+  "version": "0.1.10",
   "description": "A pi extension that enables Fast mode for supported direct OpenAI API, OpenAI Codex, and Anthropic Claude models based on the currently selected model.",
   "keywords": [
     "pi-package",

package/extensions/git-footer/index.ts

@@ -13,10 +13,6 @@ type GitStatusSnapshot = {
 
 type PullRequestSnapshot = {
   number?: number | string;
-  state?: string;
-  isDraft?: boolean;
-  url?: string;
-  title?: string;
 };
 
 type CommandResult = {
@@ -74,7 +70,7 @@ const GIT_STATUS_ARGS = [
   "--porcelain=v2",
   "--branch",
 ] as const;
-const GH_PR_VIEW_ARGS = ["pr", "view", "--json", "number,state,isDraft,url,title"] as const;
+const GH_PR_VIEW_ARGS = ["pr", "view", "--json", "number"] as const;
 
 function createEmptyGitStatus(): GitStatusSnapshot {
   return {
@@ -202,10 +198,6 @@ function parsePullRequestJson(stdout: string): PullRequestSnapshot | undefined {
   if (typeof record.number === "number" || typeof record.number === "string") {
     snapshot.number = record.number;
   }
-  if (typeof record.state === "string") snapshot.state = record.state;
-  if (typeof record.isDraft === "boolean") snapshot.isDraft = record.isDraft;
-  if (typeof record.url === "string") snapshot.url = record.url;
-  if (typeof record.title === "string") snapshot.title = record.title;
   return snapshot;
 }
 
@@ -228,13 +220,7 @@ function pullRequestSnapshotsEqual(
   left: PullRequestSnapshot | undefined,
   right: PullRequestSnapshot | undefined,
 ): boolean {
-  return (
-    left?.number === right?.number &&
-    left?.state === right?.state &&
-    left?.isDraft === right?.isDraft &&
-    left?.url === right?.url &&
-    left?.title === right?.title
-  );
+  return left?.number === right?.number;
 }
 
 function defaultRunner(

package/extensions/git-footer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diegopetrucci/pi-git-footer",
-  "version": "0.1.12",
+  "version": "0.1.13",
   "description": "A TLH-style git status add-on for pi's built-in footer.",
   "keywords": [
     "pi-package",

package/extensions/librarian/index.ts

@@ -766,11 +766,6 @@ async function writeLibrarianPreferences(preferences: LibrarianPreferences): Pro
   await fs.writeFile(configPath, `${JSON.stringify(config, null, 2)}\n`, "utf8");
 }
 
-async function writeCachePreference(preference: CacheMode): Promise<void> {
-  const preferences = await readLibrarianPreferences();
-  await writeLibrarianPreferences({ ...preferences, cacheMode: preference });
-}
-
 function resolveCacheDecision(preference: CacheMode): { enabled: boolean; reason: string } {
   if (preference === "enabled") {
     return { enabled: true, reason: "cache preference enabled; using cached local checkouts" };
@@ -1045,7 +1040,7 @@ function tryCurlyQuotePathVariant(filePath: string): string {
 }
 /**
  * Returns all read path candidates including Pi's resolveReadPathAsync fallback
- * variants. Used only for the read tool guard; grep/find/ls use resolveToCwd only.
+ * variants.
  */
 function getReadPathCandidates(resolved: string): string[] {
   const nfd = tryNFDPathVariant(resolved);
@@ -1058,11 +1053,6 @@ function getReadPathCandidates(resolved: string): string[] {
   ];
 }
 
-function resolveToolPath(cwd: string, rawPath: string): string {
-  const normalized = rawPath.startsWith("@") ? rawPath.slice(1) : rawPath;
-  return path.isAbsolute(normalized) ? path.resolve(normalized) : path.resolve(cwd, normalized);
-}
-
 function getBlockedBashReason(
   command: string,
   options: { workspace: string; cacheRoot: string; cacheEnabled: boolean },
@@ -1165,17 +1155,9 @@ function createLibrarianRuntimeGuardExtension(options: {
             (options.cacheEnabled && (isInside(options.cacheRoot, p) || isInside(realCacheRoot, p)))
           );
         }
-        const resolved = resolveToolPath(options.workspace, input.path);
-        const realPath = await fs.realpath(resolved).catch(() => resolved);
-        if (!isAllowedPath(realPath))
-          return {
-            block: true,
-            reason: `Librarian read is limited to its workspace/cache: ${realPath}`,
-          };
-        // Pi-effective path check (monotonic: block if either resolution is
-        // outside). For the read tool, also check Pi's resolveReadPathAsync
-        // fallback variants so a missing file whose variant resolves outside is
-        // not silently allowed.
+        // For the read tool, also check Pi's resolveReadPathAsync fallback
+        // variants so a missing file whose variant resolves outside is not
+        // silently allowed.
         try {
           const piResolved = resolveToCwd(input.path, options.workspace);
           const candidates = getReadPathCandidates(piResolved);
@@ -1293,12 +1275,6 @@ function addAssistantMessageUsage(total: Usage, message: unknown): void {
   addUsage(total, assistant.usage as Usage);
 }
 
-function aggregateAssistantUsage(messages: unknown[]): Usage {
-  const total = createEmptyUsage();
-  for (const message of messages) addAssistantMessageUsage(total, message);
-  return total;
-}
-
 function addSessionEventUsage(total: Usage, event: unknown): void {
   const sessionEvent = event as { type?: string; message?: unknown; result?: { usage?: Usage } };
   if (sessionEvent?.type === "message_end")
@@ -1481,8 +1457,7 @@ export const __test__ = {
   isModelAvailabilityError,
   parseModelPreference,
   resolveThinkingLevel,
-  aggregateAssistantUsage,
-  addAssistantMessageUsage,
+  createEmptyUsage,
   addSessionEventUsage,
   inspectFinalAssistant,
   classifyRunFailure,
@@ -1490,14 +1465,12 @@ export const __test__ = {
   createLibrarianAbortController,
   cleanupExpiredCache,
   cleanupExpiredRepos,
-  cleanupLockOwnershipError,
   tryAcquireCleanupLock,
   heartbeatCleanupLock,
   releaseCleanupLock,
   CLEANUP_LOCK_FILE,
   CLEANUP_LOCK_STALE_MS,
   CLEANUP_LOCK_MAX_LEASE_MS,
-  isCleanupLockLeaseExpired,
 };
 
 export default function librarianExtension(pi: ExtensionAPI) {
@@ -1668,17 +1641,6 @@ export default function librarianExtension(pi: ExtensionAPI) {
       const cacheRoot = getUserCacheReposRoot();
       const configPath = getCacheConfigPath();
 
-      const setPreference = async (mode: CacheMode): Promise<string | undefined> => {
-        cachePreference = mode;
-        try {
-          await writeCachePreference(mode);
-          return undefined;
-        } catch (error) {
-          const message = error instanceof Error ? error.message : String(error);
-          return `Preference changed for this process, but could not save ${configPath}: ${message}`;
-        }
-      };
-
       const formatSetMessage = (mode: CacheMode, warning?: string): string => {
         const main =
           mode === "enabled"
@@ -1688,20 +1650,20 @@ export default function librarianExtension(pi: ExtensionAPI) {
       };
 
       if (action === "on" || action === "enable") {
-        const warning = await setPreference("enabled");
+        const warning = await savePreferences({ ...currentPreferences(), cacheMode: "enabled" });
         notifyCommand(ctx, formatSetMessage("enabled", warning), warning ? "warning" : "info");
         return;
       }
 
       if (action === "off" || action === "disable") {
-        const warning = await setPreference("disabled");
+        const warning = await savePreferences({ ...currentPreferences(), cacheMode: "disabled" });
         notifyCommand(ctx, formatSetMessage("disabled", warning), warning ? "warning" : "info");
         return;
       }
 
       if (action === "toggle") {
         const next = cachePreference === "enabled" ? "disabled" : "enabled";
-        const warning = await setPreference(next);
+        const warning = await savePreferences({ ...currentPreferences(), cacheMode: next });
         notifyCommand(ctx, formatSetMessage(next, warning), warning ? "warning" : "info");
         return;
       }

package/extensions/librarian/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diegopetrucci/pi-librarian",
-  "version": "0.1.19",
+  "version": "0.1.21",
   "description": "A pi GitHub research scout with a toggleable local repo checkout cache under the user's OS cache directory.",
   "keywords": [
     "pi-package",

package/extensions/minimal-footer/README.md

@@ -10,7 +10,7 @@ It replaces pi's built-in footer with a cleaner two-line layout that focuses on
 - TLH-style git dirty counts, ahead/behind counts, and optional PR number
 - current repo name
 - current context percentage
-- red `DUMB ZONE` indicator when context usage is above 300k tokens
+- red `DUMB ZONE` indicator when context usage is above 350k tokens
 - current model and thinking level
 - OpenAI Codex usage windows, labeled from the durations reported by OpenAI
 - `xp` marker when Pi experimental features are enabled
@@ -39,7 +39,7 @@ main · +1 ~2 ?3 ↑1 • PR #42
 
 The markers are `!` conflicts, `+` staged, `~` unstaged, `?` untracked, `↑` ahead, and `↓` behind. PR numbers are fetched best-effort with `gh pr view`. Git polling is skipped until Pi reports that the project is trusted, and `git status` is run with fsmonitor disabled.
 
-When context usage is above 300k tokens, the bottom-left line includes a red warning:
+When context usage is above 350k tokens, the bottom-left line includes a red warning:
 
 ```text
 44.1% · DUMB ZONE
@@ -115,7 +115,7 @@ Example:
     "showPercent": true,
     "dumbZone": {
       "enabled": true,
-      "thresholdTokens": 300000,
+      "thresholdTokens": 350000,
       "label": "DUMB ZONE",
       "color": "error"
     }
@@ -231,7 +231,7 @@ Disable git dirty/ahead/PR status:
 
 - **Top left:** current git branch plus dirty/ahead/PR status when available
 - **Top right:** current repo directory name
-- **Bottom left:** current context usage percentage, plus red `DUMB ZONE` above 300k context tokens
+- **Bottom left:** current context usage percentage, plus red `DUMB ZONE` above 350k context tokens
 - **Bottom left on `openai-codex`:** current context usage percentage plus the Codex usage windows reported for the account
 - **Bottom left on `openai` ChatGPT OAuth:** current context usage percentage plus `usage unsupported`; no legacy WHAM request is attempted
 - **Bottom left with `PI_EXPERIMENTAL=1`:** current context usage percentage plus `xp`
@@ -246,7 +246,7 @@ This extension also lives inside the broader [`pi-extensions`](../../README.md)
 - Replaces pi's built-in footer entirely.
 - Uses pi footer data for git branch updates, plus background cached git/gh checks for dirty counts and PR number after project trust is granted.
 - Shows only context percentage, not context window size.
-- Shows `DUMB ZONE` only while context usage is above 300k tokens.
+- Shows `DUMB ZONE` only while context usage is above 350k tokens.
 - Shows the model id rather than a provider-specific display label.
 - Shows `xp` when `PI_EXPERIMENTAL=1`.
 - For `openai-codex`, reads pi's stored OAuth login and fetches usage from ChatGPT's backend usage endpoint. Window labels are inferred from each window's reported duration; missing or unrecognized durations use neutral `usage` labels rather than guessing from window position.

package/extensions/minimal-footer/index.ts

@@ -24,7 +24,7 @@ const DEFAULT_CONFIG: MinimalFooterConfig = {
     showPercent: true,
     dumbZone: {
       enabled: true,
-      thresholdTokens: 300_000,
+      thresholdTokens: 350_000,
       label: "DUMB ZONE",
       color: "error",
     },
@@ -148,10 +148,6 @@ type GitStatusSnapshot = {
 
 type PullRequestSnapshot = {
   number?: number | string;
-  state?: string;
-  isDraft?: boolean;
-  url?: string;
-  title?: string;
 };
 
 type CommandResult = {
@@ -200,7 +196,7 @@ const GIT_STATUS_ARGS = [
   "--porcelain=v2",
   "--branch",
 ] as const;
-const GH_PR_VIEW_ARGS = ["pr", "view", "--json", "number,state,isDraft,url,title"] as const;
+const GH_PR_VIEW_ARGS = ["pr", "view", "--json", "number"] as const;
 
 function readConfigFile(path: string): RecursivePartial<MinimalFooterConfig> {
   if (!existsSync(path)) return {};
@@ -468,10 +464,6 @@ function parsePullRequestJson(stdout: string): PullRequestSnapshot | undefined {
   if (typeof record.number === "number" || typeof record.number === "string") {
     snapshot.number = record.number;
   }
-  if (typeof record.state === "string") snapshot.state = record.state;
-  if (typeof record.isDraft === "boolean") snapshot.isDraft = record.isDraft;
-  if (typeof record.url === "string") snapshot.url = record.url;
-  if (typeof record.title === "string") snapshot.title = record.title;
   return snapshot;
 }
 
@@ -494,13 +486,7 @@ function pullRequestSnapshotsEqual(
   left: PullRequestSnapshot | undefined,
   right: PullRequestSnapshot | undefined,
 ): boolean {
-  return (
-    left?.number === right?.number &&
-    left?.state === right?.state &&
-    left?.isDraft === right?.isDraft &&
-    left?.url === right?.url &&
-    left?.title === right?.title
-  );
+  return left?.number === right?.number;
 }
 
 function defaultRunner(