@herbertgao/pi-extensions 2026.9.0 → 2026.9.1

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/edits.js

@@ -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;
 }