@andares/pi-toolkits 0.7.1 → 0.7.3

package/src/features/compact/index.ts

@@ -11,8 +11,11 @@
  *    before running its own compaction
  *  - prompts, message assembly, split-turn handling, output budgets, and the
  *    summary file-list tail are verbatim pi built-in behavior (copied from
- *    0.87.1, verified unchanged through 1.0.2) — only the executing model
- *    differs
+ *    0.87.1, byte-identical across every published version of the supported
+ *    window, 0.99.0 → 1.2.0) — only the executing model differs. The one
+ *    exception is `serializeConversation`, which is imported from the host
+ *    instead of mirrored, so it tracks whatever the running host does
+ *    (1.2.0 stopped feeding assistant thinking into the request).
  *  - lossless fallback: unset config, unavailable model, empty/failed
  *    summary, or user abort → return undefined and pi's default compaction
  *    (session model) runs untouched. The extension can never make
@@ -116,7 +119,7 @@ async function completeSummary(
 }
 
 /**
- * pi's output-budget formula (compaction.js, 0.87.1 → 1.0.2 unchanged): a
+ * pi's output-budget formula (compaction.js, 0.87.1 → 1.2.0 unchanged): a
  * fraction of reserveTokens clamped by the model's own output cap.
  */
 function summarizationBudget(
@@ -132,7 +135,8 @@ function summarizationBudget(
 
 /**
  * Run the full compaction summarization with the configured model,
- * replicating pi's compact() (0.87.1, verified unchanged through 1.0.2):
+ * replicating pi's compact() (0.87.1, byte-identical across the supported
+ * 0.99.0 → 1.2.0 window):
  * history summary (first-connection or iterative-update prompt, custom
  * focus appended), plus a separate turn-prefix checkpoint when the cut point
  * splits a turn, merged into one summary with the file-list tail. Returns

package/src/features/compact/prompt.ts

@@ -5,9 +5,15 @@
  * compaction (dist/core/compaction/compaction.js and utils.js) so that
  * summaries produced by the configured compact model are formatted exactly
  * like pi's own compaction output. Only the executing model differs.
- * Verified byte-identical against pi 1.0.2 (0.88–0.98 were never published;
- * the only 0.87.1→1.0.2 change there is a combineUsage refactor into a
- * shared module, no prompt or budget change).
+ * Verified byte-identical against every published version of the supported
+ * window (0.99.0 → 1.2.0): dist/core/compaction/compaction.js and
+ * core/usage-totals.js are unchanged files across that span, and the only
+ * utils.js change (1.2.0 dropping assistant thinking from
+ * serializeConversation) does not touch the strings mirrored here — that
+ * serializer is imported from the host, not mirrored. 0.87.0 shipped a
+ * different split-turn prefix prompt; the text mirrored here is 0.87.1's
+ * fix, and the mirror is the only variant any supported host gets —
+ * intentional.
  *
  * Update these strings whenever pi changes its built-in prompts (diff
  * dist/core/compaction/compaction.js between versions).
@@ -196,7 +202,9 @@ export function formatFileOperations(
 
 /**
  * Combine two provider usage records field-wise (pi 0.87.1 compaction.js
- * combineUsage): used when a split turn produces two summarization calls.
+ * combineUsage, moved verbatim to core/usage-totals.js in 0.99.0 and
+ * unchanged through 1.2.0): used when
+ * a split turn produces two summarization calls.
  */
 export function combineUsage<T extends UsageLike>(first: T, second: T): T {
 	return {

package/src/features/favorites/constants.ts

@@ -8,7 +8,8 @@
  * bindings — only the cycle semantics change (patch-cycle.ts), so no
  * keybinding conflicts arise.
  *
- * Verified against pi 0.86.0 / 0.87.0 / 0.99.0 / 1.0.2 / 1.0.4 keybinding
+ * Verified against every published version of the supported window
+ * (0.99.0 → 1.2.0) keybinding
  * tables + pi-tui input dispatch on WSL2 + Windows Terminal; re-run
  * `pnpm check:keybindings` after every host upgrade before trusting this.
  */
@@ -25,15 +26,15 @@ export const FAVORITES_FILE = "pi-toolkits-favorites.json";
  * missed — audit trail below; re-run `pnpm check:keybindings` on upgrades.
  *
  * Why ctrl+f is dead: fullscreen became pi's default `tuiMode` in 1.0
- * (0.86–0.99 defaulted to `regular`, where the binding is inert). On
+ * (0.87–0.99 defaulted to `regular`, where the binding is inert). On
  * Windows/WSL `tui.altScreen.search` resolves to ctrl+f, and
  * `TuiAltScreen.handleViewportInput()` checks it before
  * `shouldDeferViewportInputToOverlay()`, so the keystroke opens transcript
- * search and the selector never sees it (verified in pi-tui 1.0.4
+ * search and the selector never sees it (verified in pi-tui 1.2.0
  * `dist/tui-alt-screen.js`: the search check is the first key branch and
  * returns `{ consume: true }` unconditionally).
  *
- * Why ctrl+alt+f was discarded: clean at the pi layer across 0.86–1.0 (only
+ * Why ctrl+alt+f was discarded: clean at the pi layer across 0.99–1.2 (only
  * ctrl+alt+] is taken from that family) and byte-distinguishable without
  * kitty — but grabbed at the OS/app layer on the author's machine by a
  * Windows screenshot tool, so it never reached the terminal. Lesson: a key

package/src/features/favorites/patch-cycle.ts

@@ -14,7 +14,8 @@
  *  - with exactly one favorite, already on it → undefined (stock "only one
  *    model available" feedback); otherwise jump to it
  *  - the apply logic mirrors pi's _cycleScopedModel/_cycleAvailableModel
- *    (byte-identical across every published version in 0.86.0 → 1.0.2:
+ *    (byte-identical across every published version in the supported window
+ *    0.99.0 → 1.2.0:
  *    state swap, session persistence, default-model persistence only when
  *    `options.persist` is set, thinking-level re-clamp, model_select emit
  *    with source "cycle")
@@ -30,9 +31,9 @@ import { getFavoritesStore, modelKey } from "./store.js";
 
 /**
  * pi's `Model<any>` — the exact type of `ModelCycleResult.model`
- * (pi-coding-agent 0.86–1.0; the field is unchanged across that span).
+ * (pi-coding-agent 0.99–1.2; the field is unchanged across that span).
  * Derived here instead of importing `Model` from @earendil-works/pi-ai,
- * whose index.d.ts (0.86 through 1.0.2 alike) re-exports it via an
+ * whose index.d.ts (0.99 through 1.2.0 alike) re-exports it via an
  * extensioned `"./types.ts"` specifier that not every TS resolver substitutes
  * to `types.d.ts` (tsc 5.9 bundler does; other tsserver builds don't).
  */
@@ -56,7 +57,7 @@ interface CycleSessionInternals {
 		setDefaultModelAndProvider(provider: string, id: string): void;
 	};
 	/**
-	 * pi 0.87: keeps a persisted default usable when cycling inside a
+	 * pi 0.99–1.2: keeps a persisted default usable when cycling inside a
 	 * non-empty --models scope.
 	 */
 	_addPersistedDefaultToNonEmptyScope(model: unknown): void;
@@ -97,7 +98,7 @@ function nextCycleIndex(
 		: (currentIndex - 1 + len) % len;
 }
 
-/** Options accepted by pi 0.87's cycleModel (persist gates default-model persistence). */
+/** Options accepted by pi's cycleModel (persist gates default-model persistence). */
 interface CycleOptions {
 	persist?: boolean;
 }
@@ -139,7 +140,7 @@ export function applyCyclePatch(): boolean {
 			return original.call(this, direction, options);
 		}
 
-		// Same source and availability filtering as pi 0.87's stock cycling.
+		// Same source and availability filtering as pi's stock cycling (0.99–1.2).
 		const scoped = this._scopedModels.length > 0;
 		let candidates: CycleCandidate[];
 		if (scoped) {
@@ -184,7 +185,7 @@ export function applyCyclePatch(): boolean {
 				: this._getThinkingLevelForModelSwitch();
 		this.agent.state.model = next.model;
 		this.sessionManager.appendModelChange(next.model.provider, next.model.id);
-		// pi 0.87: only persist the default model when the caller asked for it.
+		// pi 0.99–1.2: only persist the default model when the caller asked for it.
 		if (options.persist) {
 			this.settingsManager.setDefaultModelAndProvider(
 				next.model.provider,

package/src/features/favorites/patch-selector.ts

@@ -22,7 +22,7 @@
  * keeps loading and ctrl+p falls back to stock behavior. The prototype
  * shape touched here (handleInput/updateList/filterModels and the instance
  * fields below) is byte-identical across every published version in
- * pi 0.86.0 → 1.0.4.
+ * pi 0.99.0 → 1.2.0 (and 0.87.0 → 1.1.0 before the floor moved).
  */
 import {
 	ModelSelectorComponent,
@@ -61,7 +61,7 @@ interface SelectorPatchTarget {
 	handleInput(keyData: string): void;
 	updateList(): void;
 	filterModels(query: string): void;
-	/** pi 0.87: default-model badge source. */
+	/** pi 0.99–1.2: default-model badge source. */
 	isDefaultModel(model: ModelRef): boolean;
 	listContainer: ListContainerLike;
 	searchInput: SearchInputLike;
@@ -116,7 +116,7 @@ let patched = false;
 export function applyModelSelectorPatches(): boolean {
 	if (patched) return true;
 	// SAFETY: ModelSelectorComponent's members are TS-private (compile-time
-	// only); the structural view matches pi 0.86–1.0's prototype shape
+	// only); the structural view matches pi 0.99–1.2's prototype shape
 	// (byte-identical across that span) and every touched method is
 	// existence-checked below before any patching happens.
 	const proto =
@@ -155,7 +155,7 @@ export function applyModelSelectorPatches(): boolean {
 	};
 
 	// ── 2. List rendering: hint row + bold bright yellow favorites ────────
-	// Replicates the built-in updateList layout (pi 0.86–1.0: fixed-width cursor
+	// Replicates the built-in updateList layout (pi 0.99–1.2: fixed-width cursor
 	// and ✓ current-marker columns, provider badge, `· default` badge, scroll
 	// indicator, empty/error states, refresh status, Model Name footer) with
 	// one change: favorited model ids render in the favorite style.

package/src/features/stash/stash.ts

@@ -21,7 +21,7 @@ import type { KeyId } from "@earendil-works/pi-tui";
 /**
  * Stash hotkey.
  *
- * Compatibility rationale (verified against pi 0.84 keybindings, live
+ * Compatibility rationale (verified against pi 0.99.0–1.2.0 keybindings, live
  * matchesKey probes and terminal/OS behavior):
  *  - pi: no default binding for ctrl+alt+<letter> (only ctrl+alt+] is taken);
  *    extension shortcuts dispatch first while the editor is focused

package/src/lib/model-config.ts

@@ -4,8 +4,8 @@
  * (review, compact-model — favorites keeps its own store-local key logic).
  *
  * Types are derived from pi-coding-agent's own exports (ModelRegistry)
- * rather than imported from @earendil-works/pi-ai: pi-ai's index.d.ts (0.87
- * through 1.0.2 alike) re-exports `Model`/`Api` via extensioned
+ * rather than imported from @earendil-works/pi-ai: pi-ai's index.d.ts (0.99
+ * through 1.2.0 alike) re-exports `Model`/`Api` via extensioned
  * `./types.ts` specifiers, which tsc 5.9 bundler resolution substitutes to
  * `.d.ts` but some tsserver builds do not. Deriving from ModelRegistry keeps
  * every resolver in agreement with the exact type

package/src/lib/model-picker.ts

@@ -9,7 +9,7 @@
  *
  * The component is driven through a small ModelRuntime facade backed by
  * ctx.modelRegistry (pi's extension-facing wrapper). SAFETY: verified
- * against pi 0.86–1.0 dist (modes/interactive/components/model-selector.js +
+ * against pi 0.99.0–1.2.0 dist (modes/interactive/components/model-selector.js +
  * model-catalog-refresh.js — byte-identical across that span) — the
  * component only calls getAvailableSnapshot, getModel, getError and refresh
  * on modelRuntime, and ModelRegistry provides exactly those
@@ -42,7 +42,7 @@ function modelRuntimeFacade(ctx: ExtensionContext): ModelRuntime {
 		refresh: (options?: { signal?: AbortSignal }) =>
 			ctx.modelRegistry.refresh(options),
 	};
-	// SAFETY: the selector only touches the four members above (pi 0.86–1.0
+	// SAFETY: the selector only touches the four members above (pi 0.99–1.2
 	// model-selector.js); everything else on ModelRuntime is never reached
 	// from this component.
 	return facade as unknown as ModelRuntime;

package/src/lib/tools.ts

@@ -8,9 +8,11 @@ export function snapshotActiveTools(pi: ExtensionAPI): string[] {
 	return [...pi.getActiveTools()];
 }
 
-/** Restore a previously snapshotted tool set. No-op when the snapshot is empty. */
-export function restoreActiveTools(pi: ExtensionAPI, snapshot: string[]): void {
-	if (snapshot.length === 0) return;
+/** Restore a previously snapshotted tool set (an empty snapshot = no tools). */
+export function restoreActiveTools(
+	pi: ExtensionAPI,
+	snapshot: readonly string[],
+): void {
 	pi.setActiveTools([...snapshot]);
 }