@herbertgao/pi-extensions 2026.9.1 → 2026.9.2

package/node_modules/pi-lens/dist/clients/test-runner-delivery.js

@@ -2,17 +2,14 @@
  * Automatic test-runner delivery for the post-agent idle window (#2366).
  *
  * Test results remain in `test-runner-findings` for pull diagnostics and the
- * commit guard. This module only stages an owner-qualified pointer and
- * appends a non-context custom entry after the host proves that the agent is
+ * commit guard. This module stages an owner-qualified pointer and marks it
+ * eligible for the next context build after the host proves that the agent is
  * idle. Provenance validation stays in `peekTestFindings`, the one shared
  * delivery gate for this cache.
  */
 import { emitBounded } from "./bounded-telemetry.js";
-import { peekTestFindings } from "./runtime-context.js";
-import { fitLines } from "./tui-fit.js";
-export const TEST_RUNNER_ENTRY_TYPE = "pilens:test-runner-findings";
+import { consumeTestFindings, peekTestFindings } from "./runtime-context.js";
 const MAX_PENDING_DELIVERIES = 32;
-const MAX_ENTRY_CONTENT = 12_000;
 const pending = new Map();
 function key(cwd, sessionId, ownerId) {
     // `cwd` comes from the same host session field at turn_end and settle. Keep
@@ -20,14 +17,6 @@ function key(cwd, sessionId, ownerId) {
     // identity; dispatch contexts already normalize their own path fields.
     return `${ownerId ?? "direct"}\u0000${cwd}\u0000${sessionId}`;
 }
-function boundedContent(content) {
-    if (content.length <= MAX_ENTRY_CONTENT)
-        return { content, droppedDetailCount: 0 };
-    return {
-        content: `${content.slice(0, MAX_ENTRY_CONTENT)}\n[…details truncated]`,
-        droppedDetailCount: content.length - MAX_ENTRY_CONTENT,
-    };
-}
 function record(identity, outcome, delivery, metadata = {}) {
     emitBounded("test_runner_delivery", identity, {
         filePath: delivery.cwd,
@@ -44,9 +33,7 @@ function record(identity, outcome, delivery, metadata = {}) {
         },
     }, {
         capPerTurn: { limit: 8, turnIndex: delivery.generation },
-        ledgerKind: outcome === "delivery-failed" || outcome === "capability-unavailable"
-            ? "test-runner-delivery"
-            : undefined,
+        ledgerKind: outcome === "delivery-failed" ? "test-runner-delivery" : undefined,
         reason: `test runner delivery ${outcome}`,
     });
 }
@@ -70,6 +57,7 @@ export function stageTestRunnerDelivery(args) {
             generation: args.generation,
             targetCount: args.targetCount,
             createdAt: Date.now(),
+            eligible: false,
             owner: args.owner,
         };
         record(deliveryKey, "superseded", superseded, {
@@ -95,13 +83,14 @@ export function stageTestRunnerDelivery(args) {
         generation: args.generation,
         targetCount: args.targetCount,
         createdAt: Date.now(),
+        eligible: false,
         owner: args.owner,
     };
     pending.set(deliveryKey, delivery);
     prunePending();
     record(deliveryKey, "staged", delivery);
 }
-/** Deliver the latest staged result during a host-confirmed idle window. */
+/** Mark the latest staged result eligible during a host-confirmed idle window. */
 export function deliverTestRunnerFindings(args) {
     const cwd = args.ctx.cwd ?? process.cwd();
     const deliveryKey = key(cwd, args.sessionId, args.ownerId);
@@ -112,16 +101,11 @@ export function deliverTestRunnerFindings(args) {
     if (currentGeneration !== undefined &&
         currentGeneration > delivery.generation) {
         pending.delete(deliveryKey);
-        record(deliveryKey, "superseded", delivery, {
-            currentGeneration,
-        });
+        record(deliveryKey, "superseded", delivery, { currentGeneration });
         return;
     }
     if (typeof args.ctx.isIdle !== "function") {
         pending.delete(deliveryKey);
-        record(deliveryKey, "capability-unavailable", delivery, {
-            reason: "host does not expose ctx.isIdle",
-        });
         return;
     }
     try {
@@ -136,14 +120,13 @@ export function deliverTestRunnerFindings(args) {
         });
         return;
     }
-    const findings = peekTestFindings(args.cacheManager, cwd, args.runtime, true);
-    if (!findings) {
+    if (!peekTestFindings(args.cacheManager, cwd, args.runtime, true)) {
         pending.delete(deliveryKey);
         record(deliveryKey, "superseded", delivery);
         return;
     }
-    // Recheck immediately before append. The host may accept a prompt between
-    // the first check and this synchronous call.
+    // Recheck immediately before marking eligible. The host may accept a prompt
+    // between the first check and this synchronous call.
     try {
         if (!args.ctx.isIdle()) {
             record(deliveryKey, "carried", delivery);
@@ -156,32 +139,23 @@ export function deliverTestRunnerFindings(args) {
         });
         return;
     }
-    // SAFETY: appendEntry is an optional host capability absent from older Pi SDKs.
-    const appendEntry = args.pi.appendEntry;
-    if (typeof appendEntry !== "function") {
+    delivery.eligible = true;
+    const current = args.cacheManager.readCache("test-runner-findings", cwd)?.data;
+    if (!current?.content || current.testRunGeneration !== delivery.generation) {
         pending.delete(deliveryKey);
-        record(deliveryKey, "capability-unavailable", delivery);
         return;
     }
-    const bounded = boundedContent(findings.messages[0]?.content ?? "");
-    try {
-        appendEntry.call(args.pi, TEST_RUNNER_ENTRY_TYPE, {
-            ...bounded,
+    args.cacheManager.writeCache("test-runner-findings", {
+        ...current,
+        deliveryEligible: {
             sessionId: delivery.sessionId,
             generation: delivery.generation,
-            targetCount: delivery.targetCount,
-        });
-        pending.delete(deliveryKey);
-        record(deliveryKey, "delivered", delivery, {
-            droppedDetailCount: bounded.droppedDetailCount,
-            ageMs: Math.max(0, Date.now() - delivery.createdAt),
-        });
-    }
-    catch (error) {
-        record(deliveryKey, "delivery-failed", delivery, {
-            error: String(error).slice(0, 500),
-        });
-    }
+            eligibleAt: Date.now(),
+        },
+    }, cwd);
+    record(deliveryKey, "eligible", delivery, {
+        ageMs: Math.max(0, Date.now() - delivery.createdAt),
+    });
 }
 /** Deliver only the result staged by this activation and settled session. */
 export function deliverStagedTestRunnerFindings(args) {
@@ -195,7 +169,6 @@ export function deliverStagedTestRunnerFindings(args) {
     if (!delivery?.owner)
         return;
     deliverTestRunnerFindings({
-        pi: delivery.owner.pi,
         ctx: delivery.owner.getCtx(),
         cacheManager: delivery.owner.cacheManager,
         runtime: delivery.owner.runtime,
@@ -203,31 +176,63 @@ export function deliverStagedTestRunnerFindings(args) {
         ownerId: delivery.ownerId,
     });
 }
-export function registerTestRunnerEntryRenderer(pi) {
-    // SAFETY: registerEntryRenderer is an optional host capability absent from older Pi SDKs.
-    const register = pi.registerEntryRenderer;
-    if (typeof register !== "function")
-        return false;
-    try {
-        register.call(pi, TEST_RUNNER_ENTRY_TYPE, renderTestRunnerEntry);
-        return true;
+/** Consume the result made eligible by the settled idle boundary. */
+export function consumeStagedTestRunnerFindings(args) {
+    const deliveryKey = key(args.cwd, args.sessionId, args.ownerId);
+    let delivery = pending.get(deliveryKey);
+    const persisted = args.cacheManager.readCache("test-runner-findings", args.cwd)?.data;
+    const eligible = persisted?.deliveryEligible;
+    if (!delivery && persisted?.content && eligible) {
+        const sameSession = eligible.sessionId === args.sessionId;
+        if (!sameSession) {
+            const foreignDelivery = {
+                cwd: args.cwd,
+                sessionId: eligible.sessionId,
+                generation: eligible.generation,
+                targetCount: 0,
+                createdAt: eligible.eligibleAt,
+                eligible: true,
+                rehydrated: true,
+            };
+            record(deliveryKey, "foreign-session", foreignDelivery, {
+                currentSessionId: args.sessionId,
+                currentOwnerId: args.ownerId,
+                reason: "session-mismatch",
+            });
+            return undefined;
+        }
+        delivery = {
+            cwd: args.cwd,
+            sessionId: args.sessionId,
+            ownerId: args.ownerId,
+            generation: eligible.generation,
+            targetCount: 0,
+            createdAt: eligible.eligibleAt,
+            eligible: true,
+            rehydrated: true,
+        };
+        pending.set(deliveryKey, delivery);
     }
-    catch {
-        return false;
+    if (!delivery?.eligible)
+        return undefined;
+    const currentGeneration = persisted?.testRunGeneration;
+    if (currentGeneration !== delivery.generation) {
+        pending.delete(deliveryKey);
+        record(deliveryKey, "superseded", delivery, { currentGeneration });
+        return undefined;
     }
-}
-function renderTestRunnerEntry(entry, _options, theme) {
-    const content = entry.data?.content;
-    if (typeof content !== "string")
+    const findings = consumeTestFindings(args.cacheManager, args.cwd, args.runtime);
+    if (!findings) {
+        pending.delete(deliveryKey);
+        record(deliveryKey, "superseded", delivery);
         return undefined;
-    const lines = [
-        theme.fg("error", "pi-lens test failures"),
-        ...content.split("\n"),
-    ];
-    return {
-        render: (width) => fitLines(lines, width),
-        invalidate: () => { },
-    };
+    }
+    pending.delete(deliveryKey);
+    record(deliveryKey, "delivered", delivery, {
+        ageMs: Math.max(0, Date.now() - delivery.createdAt),
+        rehydrated: delivery.rehydrated === true,
+    });
+    return findings;
 }
 /** Test-only reset; session ownership prevents cross-session consumption. */
 export function _resetTestRunnerDeliveryForTests() {

package/node_modules/pi-lens/dist/clients/tool-policy.js

@@ -756,7 +756,7 @@ const FORMATTER_POLICY_BY_FILENAME = new Map(TERRAGRUNT_FILENAMES.map((name) =>
 // lines don't fall at the head of the pre-existing lookup-table duplication and
 // get counted as new duplicated code. Bindings are live, so position is safe.
 export { AUTO_INSTALLABLE_DEFAULT_FORMATTERS, FORMATTER_POLICY_BY_EXTENSION, FORMATTER_POLICY_BY_FILENAME, };
-export function getFormatterPolicyForExtension(ext) {
+function getFormatterPolicyForExtension(ext) {
     return FORMATTER_POLICY_BY_EXTENSION.get(ext.toLowerCase());
 }
 export function getFormatterPolicyForFile(filePath) {

package/node_modules/pi-lens/dist/clients/tree-sitter-client.js

@@ -80,7 +80,7 @@ function grammarFileStamp(filePath) {
         return undefined;
     }
 }
-export function isTreeSitterWasmAbortError(error) {
+function isTreeSitterWasmAbortError(error) {
     const message = error instanceof Error ? error.message : String(error);
     return message.includes("Aborted") || message.includes("abort()");
 }
@@ -4012,54 +4012,3 @@ export class TreeSitterClient {
     }
 }
 // --- Simplified Pattern Search (regex fallback) ---
-/**
- * Fallback structural search using regex when tree-sitter unavailable
- * Less accurate but works without WASM dependencies
- */
-export function regexStructuralSearch(pattern, files, options = {}) {
-    const matches = [];
-    const maxResults = options.maxResults ?? 50;
-    // Extract pattern structure for regex
-    // "console.log($MSG)" -> /console\.log\(([^)]+)\)/
-    const regexPattern = pattern
-        .replace(/\\/g, "\\\\")
-        .replace(/\./g, "\\.")
-        .replace(/\$\$\$[A-Z_][A-Z0-9_]*/g, "(.*?)") // variadic - non-greedy
-        .replace(/\$[A-Z_][A-Z0-9_]*/g, "([^,)]+)"); // single - capture group
-    try {
-        const regex = new RegExp(regexPattern, "g");
-        for (const file of files) {
-            if (matches.length >= maxResults)
-                break;
-            try {
-                const content = fs.readFileSync(file, "utf-8");
-                const lines = content.split("\n");
-                for (let i = 0; i < lines.length; i++) {
-                    regex.lastIndex = 0;
-                    const match = regex.exec(lines[i]);
-                    if (match) {
-                        const captures = {};
-                        // Extract captures
-                        for (let j = 1; j < match.length; j++) {
-                            captures[`$${j}`] = match[j];
-                        }
-                        matches.push({
-                            file,
-                            line: i + 1,
-                            column: match.index + 1,
-                            matchedText: match[0],
-                            captures,
-                        });
-                        if (matches.length >= maxResults)
-                            break;
-                    }
-                }
-            }
-            catch { }
-        }
-    }
-    catch {
-        // Invalid regex
-    }
-    return matches;
-}

package/node_modules/pi-lens/dist/clients/tui-fit.js

@@ -18,10 +18,6 @@
 import { truncateToWidth } from "./deps/pi-tui.js";
 const ELLIPSIS_KIND = { "…": 0, "...": 1, "": 2 };
 let truncateIsNative = null;
-/** Test-only: reset the signature probe between test cases. */
-export function _resetTruncateProbeForTests() {
-    truncateIsNative = null;
-}
 export function fitLine(s, maxWidth, ellipsis = "...") {
     const w = Math.max(0, maxWidth);
     const fn = truncateToWidth;

package/node_modules/pi-lens/dist/clients/user-notify.js

@@ -32,10 +32,6 @@ export function wireUserNotifier(portsOrGetter) {
 export function resetUserNotifier() {
     notifierGetter = undefined;
 }
-/** True when a host render path is available (for callers that want to branch). */
-export function hasUserNotifier() {
-    return notifierGetter !== undefined;
-}
 /**
  * Best-effort delivery of a user-facing degradation. Never throws — a stale
  * `ctx` after a session replacement must degrade to a no-op, not break the

package/node_modules/pi-lens/dist/clients/word-index-logger.js

@@ -47,10 +47,3 @@ export function logWordIndex(entry) {
         cwd: normalizeFilePath(entry.cwd),
     });
 }
-export function getWordIndexLogPath() {
-    return WORD_INDEX_LOG_FILE;
-}
-/** Resolve once all enqueued word-index writes are on disk (tests/shutdown). */
-export function flushWordIndexLog() {
-    return writer.flush();
-}

package/node_modules/pi-lens/dist/clients/word-index-store.js

@@ -30,8 +30,8 @@ import { createDeadline, yieldIfOverBudget, } from "./cooperative-budget.js";
 /**
  * Bytes of backing store per posting entry: one `fileId` lane and one `line`
  * lane, both `Int32`. This is the constant #2069's acceptance criterion is
- * written against, and {@link estimatePostingListBytes} is the arithmetic the
- * memory-sample record and the regression test both read.
+ * written against, and the arithmetic the memory-sample record and the
+ * regression test both read.
  */
 export const WORD_POSTING_ENTRY_BYTES = 8;
 /**
@@ -189,10 +189,6 @@ export class WordPostingList {
         return list;
     }
 }
-/** Estimated bytes one posting list holds, backing store plus fixed headers. */
-export function estimatePostingListBytes(list) {
-    return list.byteLength + WORD_POSTING_LIST_OVERHEAD_BYTES;
-}
 /**
  * Re-home every posting list into ONE shared `ArrayBuffer`, exactly sized.
  *
@@ -405,7 +401,7 @@ export class WordForwardEntry {
  * Bytes per forward entry: one pointer-compressed slot for the shared token
  * string and one `Int32` count lane.
  */
-export const WORD_FORWARD_ENTRY_BYTES = 8;
+const WORD_FORWARD_ENTRY_BYTES = 8;
 /**
  * The word index's dense file-id space: canonical path key → integer id →
  * display path.

package/node_modules/pi-lens/dist/clients/word-index.js

@@ -725,17 +725,6 @@ async function updateWordIndexDocumentAsyncUnsafe(index, doc, shouldContinue) {
     commitWordIndexDocumentReplacement(index, doc, perTokenHits, docLength, startedAt);
     return true;
 }
-/** Bounds shared by every word-index build path — keep the walk off the
- * critical path on large repos: cap the file count, and skip files too large
- * to be hand-written source (generated/bundled output the source filter
- * didn't already exclude).
- *
- * Deprecated (#776): `collectWordIndexDocs` below no longer reads this
- * constant directly — it derives its cap from `getWordIndexMaxFilesDerived`
- * (project-scale.ts's `maxProjectFiles` knob), which reproduces this same
- * 6,000 default at the default base. Kept exported for tests/callers that
- * still reference the literal. */
-export const WORD_INDEX_MAX_FILES = 6000;
 export const WORD_INDEX_MAX_BYTES = 512 * 1024;
 /**
  * Collect the bounded `{path, content}` doc set `buildWordIndex` consumes —
@@ -748,9 +737,9 @@ export const WORD_INDEX_MAX_BYTES = 512 * 1024;
 export async function collectWordIndexDocs(root, shouldContinue = () => true, preflightFiles) {
     const { collectSourceFilesAsync } = await import("./source-filter.js");
     // #747 hardening: pass the cap INTO the walk — without it,
-    // `collectSourceFilesAsync` defaults to an unbounded traversal and the
-    // `WORD_INDEX_MAX_FILES` slice below only trims the result AFTER the whole
-    // tree (all of $HOME, on a misrooted cwd) has already been enumerated.
+    // `collectSourceFilesAsync` defaults to an unbounded traversal and any
+    // slice below would only trim the result AFTER the whole tree (all of
+    // $HOME, on a misrooted cwd) has already been enumerated.
     // #760: the walk is additionally bounded by source-filter's default
     // visited-entry budget (DEFAULT_MAX_SCAN_ENTRIES), so a mixed tree with few
     // source files among a huge pile of non-source files can't force a
@@ -811,7 +800,7 @@ export async function collectWordIndexDocs(root, shouldContinue = () => true, pr
 // Node's fs.promises.stat runs on libuv's threadpool (default 4 slots), so
 // real parallelism tops out there; the surplus workers are queue depth that
 // keeps the pool saturated without starving other threadpool consumers.
-export const WORD_INDEX_STAT_CONCURRENCY = 8;
+const WORD_INDEX_STAT_CONCURRENCY = 8;
 const WORD_INDEX_INCREMENTAL_CHURN_THRESHOLD = 0.3;
 // A ratio alone misclassifies one stale file in a three-file project as dense.
 // Below this absolute floor, per-document replacement is bounded and cheaper
@@ -1092,7 +1081,11 @@ export async function refreshWordIndexIncrementally(index, root, shouldContinue
         timings,
     };
 }
-export const WORD_INDEX_QUERY_FILTER_KEYS = ["lang", "file", "ext"];
+const WORD_INDEX_QUERY_FILTER_KEYS = [
+    "lang",
+    "file",
+    "ext",
+];
 /** Thrown by {@link parseWordIndexQuery}/{@link buildWordIndexQueryFilter} for
  * an unrecognized `key:` prefix or an unrecognized `lang:` value — a query
  * typo fails loudly with the supported list instead of silently degrading

package/node_modules/pi-lens/dist/clients/zizmor-config.js

@@ -15,7 +15,7 @@ import { classifyProbeFailure, createAvailabilityLatch, logAvailabilityDecision,
 // presence of one is the repo's deliberate opt-in: it carries the author's
 // chosen severities/ignores, so we let zizmor findings BLOCK in that workspace
 // (advisory-only otherwise, like Opengrep's local-rules gate).
-export const LOCAL_ZIZMOR_CONFIG_NAMES = [
+const LOCAL_ZIZMOR_CONFIG_NAMES = [
     path.join(".github", "zizmor.yml"),
     path.join(".github", "zizmor.yaml"),
     "zizmor.yml",