@herbertgao/pi-extensions 2026.9.0 → 2026.9.2

package/node_modules/pi-lens/dist/clients/lsp/diagnostic-binding.js

@@ -135,7 +135,7 @@ export function hashDiagnosticContent(content) {
 export function composeBoundToCurrentDisk(values) {
     if (values.some((v) => v === false))
         return false;
-    if (values.length === 0 || values.every((v) => v === "unknown")) {
+    if (values.every((v) => v === "unknown")) {
         return "unknown";
     }
     return true;

package/node_modules/pi-lens/dist/clients/lsp/document-drift.js

@@ -88,7 +88,7 @@ const SKIPPED = {
     tracked: 0,
 };
 /** Documents stat'd per pass. Bounds the per-pass cost on a large workspace. */
-export const DRIFT_CHECK_BATCH = 64;
+const DRIFT_CHECK_BATCH = 64;
 /** Resyncs issued per pass. Paces the heal so a bulk edit cannot burst. */
 export const DRIFT_RESYNC_BATCH = 4;
 /** Tracked-document ceiling. A workspace sweep can open far more than this. */

package/node_modules/pi-lens/dist/clients/lsp/edits.js

@@ -131,7 +131,7 @@ function parseWorkspaceEdit(edit) {
 function formatRange(range) {
     return `${range.start.line + 1}:${range.start.character + 1}-${range.end.line + 1}:${range.end.character + 1}`;
 }
-export function rangesOverlap(a, b) {
+function rangesOverlap(a, b) {
     return (comparePosition(a.start, b.end) < 0 && comparePosition(b.start, a.end) < 0);
 }
 function positionsEqual(a, b) {
@@ -351,7 +351,7 @@ export async function normalizeWorkspaceEditToUtf16(edit, positionEncoding, cwd)
         ...(edit.documentChanges !== undefined ? { documentChanges } : {}),
     };
 }
-export function flattenWorkspaceTextEdits(edit) {
+function flattenWorkspaceTextEdits(edit) {
     parseWorkspaceEdit(edit);
     const buckets = new Map();
     const push = (uri, edits) => {
@@ -568,7 +568,17 @@ function planWorkspaceEdit(edit, trackOrigins = false) {
     };
     const flushSubtree = (uri) => {
         const key = indexKey(uri);
-        for (const candidate of [...(descendants.get(key) ?? [])]) {
+        // flushUri -> removeIndex deletes `candidate` from this very Set (`key`
+        // is one of `candidate`'s ancestors by construction). No defensive copy
+        // needed: Set iterators reflect deletions safely — deleting the current
+        // or a not-yet-visited entry during iteration removes it from the
+        // remaining traversal without skipping or duplicating any other member
+        // (verified: https://tc39.es/ecma262/#sec-set.prototype.values, same
+        // guarantee already relied on for the Map iteration below and at
+        // clients/lsp/client.ts / clients/persist-debounce.ts /
+        // clients/review-graph/builder.ts). Nothing adds to `descendants`
+        // during this loop (only `addIndex` does, and it isn't called here).
+        for (const candidate of descendants.get(key) ?? []) {
             const item = pending.get(candidate);
             if (item)
                 flushUri(item.uri);
@@ -607,7 +617,10 @@ function planWorkspaceEdit(edit, trackOrigins = false) {
             flushSubtree(resource.uri);
         ops.push(resource);
     }
-    for (const item of [...pending.values()])
+    // flushUri deletes from `pending` — the same Map being iterated here — but
+    // Map iterators reflect deletions safely (see flushSubtree above), so no
+    // defensive copy is needed.
+    for (const item of pending.values())
         flushUri(item.uri);
     return ops;
 }
@@ -773,7 +786,7 @@ async function createWorkspaceUriConfiner(cwd) {
             filePath = uriToPath(uri);
         }
         catch (err) {
-            throw new Error(`invalid workspace edit URI ${uri}: ${err instanceof Error ? err.message : String(err)}`);
+            throw new Error(`invalid workspace edit URI ${uri}: ${err instanceof Error ? err.message : String(err)}`, { cause: err });
         }
         if (!path.isAbsolute(filePath))
             throw new Error(`workspace edit path escapes workspace: ${uri}`);