@mjasnikovs/pi-task 0.40.31 → 0.40.32

package/dist/workers/docs-core.js

@@ -550,7 +550,11 @@ function docsRawCached(cache, pkg, profile, query, ensureIndexed, retrieveChunks
 function docsRawUncached(pkg, profile, cacheError, autoInstalled) {
     const parts = [];
     const surfaceFiles = collectFiles(pkg, profile).surface;
-    const entryFirst = pkg.entry ? [pkg.entry, ...surfaceFiles.filter(f => f !== pkg.entry)] : surfaceFiles;
+    // Only an entry the rules KEPT goes first. A CJS-first package names its
+    // `.d.cts` in `types`, so the entry is exactly the twin `collectFiles` drops,
+    // and prepending it blind put it back at the head of the truncated blob.
+    const entry = pkg.entry !== null && surfaceFiles.includes(pkg.entry) ? pkg.entry : null;
+    const entryFirst = entry === null ? surfaceFiles : [entry, ...surfaceFiles.filter(f => f !== entry)];
     for (const abs of entryFirst) {
         let raw;
         try {

package/dist/workers/docs-index.js

@@ -93,6 +93,9 @@ function computeContentHash(pkg, profile, supplements = []) {
 function walkSurface(root, profile) {
     const out = [];
     const stack = [root];
+    // A directory symlink is followed by its resolved target, so one pointing at
+    // an ancestor inside `root` walks the same subtree forever without this.
+    const walked = new Set([root]);
     while (stack.length) {
         const dir = stack.pop();
         let entries;
@@ -107,9 +110,11 @@ function walkSurface(root, profile) {
                 continue;
             const full = path.join(dir, entry.name);
             if (entry.isSymbolicLink()) {
+                let stat;
                 let realPath;
                 try {
                     realPath = fs.realpathSync(full);
+                    stat = fs.statSync(realPath);
                 }
                 catch {
                     continue;
@@ -117,15 +122,22 @@ function walkSurface(root, profile) {
                 const relReal = path.relative(root, realPath);
                 if (relReal.startsWith('..'))
                     continue;
-                const stat = fs.statSync(realPath);
-                if (stat.isDirectory())
+                if (stat.isDirectory()) {
+                    if (walked.has(realPath))
+                        continue;
+                    walked.add(realPath);
                     stack.push(realPath);
+                }
                 else if (stat.isFile() && profile.isSurfaceFile(realPath))
                     out.push(realPath);
                 continue;
             }
-            if (entry.isDirectory())
+            if (entry.isDirectory()) {
+                if (walked.has(full))
+                    continue;
+                walked.add(full);
                 stack.push(full);
+            }
             else if (entry.isFile() && profile.isSurfaceFile(entry.name))
                 out.push(full);
         }

package/dist/workers/eco-go.js

@@ -25,7 +25,7 @@ import * as os from 'node:os';
 import * as path from 'node:path';
 import { ResolveError } from './docs-resolve.js';
 import { findAtOrAbove } from './eco-cargo.js';
-import { buildConstraint } from './go-surface.js';
+import { buildConstraint, splitGoItems } from './go-surface.js';
 import { readZip, readEntry, isUnsafeEntryName } from '../shared/zip.js';
 import { acquireStdlibPackage, findInGoroot, findSliced } from './go-stdlib.js';
 const PROXY = 'https://proxy.golang.org';
@@ -653,17 +653,21 @@ function holdsByDefault(src) {
 /**
  * The import paths a Go file's import block names.
  *
- * Scoped to the import declaration, not the file: a sibling's import path also
+ * Scoped to the import DECLARATION, not the file: a sibling's import path also
  * occurs as a plain string constant, in an error message and in a `go:generate`
  * line, and reading those keeps the very subpackage `selectOwnPackage` exists to
- * drop. An `ImportPath` is a `string_lit`, so the backtick form is legal Go and
+ * drop. A regex cannot draw that line — a column-0 `import (` also sits inside a
+ * generator template's raw string and inside a doc comment's example — so the
+ * split is the surface scanner's, which already skips comments and literals.
+ * An `ImportPath` is a `string_lit`, so the backtick form is legal Go and
  * dropping it silently drops everything reachable only through it.
  */
 function importsOf(src) {
     const out = [];
-    for (const decl of src.matchAll(/^import\s*(?:\(([\s\S]*?)^\)|(.*))$/gm)) {
-        const body = decl[1] ?? decl[2] ?? '';
-        for (const m of body.matchAll(/"([^"\n]+)"|`([^`]+)`/g))
+    for (const item of splitGoItems(src)) {
+        if (!/^import\b/.test(item.text))
+            continue;
+        for (const m of item.text.matchAll(/"([^"\n]+)"|`([^`]+)`/g))
             out.push(m[1] ?? m[2]);
     }
     return out;
@@ -683,9 +687,16 @@ function goMajor(name, version) {
     const inVersion = /^(?:v|go)?(\d+)\./.exec(version);
     return inVersion ? Number(inVersion[1]) : 1;
 }
-/** Does this file declare anything a caller of the package could be handed? */
+/**
+ * Does this file declare anything a caller of the package could be handed?
+ *
+ * Read from the scanned declarations, not the raw text. `cloud.google.com/go`'s
+ * doc.go is one block comment, and a wrapped sentence beginning `type ` sits at
+ * column 0 — enough for a multiline regex to call the root the API and drop the
+ * subdirectories that hold all of it.
+ */
 function declaresApi(src) {
-    return /^(?:func|type|var|const)\b/m.test(src);
+    return splitGoItems(src).some(item => /^(?:func|type|var|const)\b/.test(item.text));
 }
 /**
  * Keep the package the caller asked for, and only the subpackages it can reach.
@@ -729,7 +740,16 @@ export function selectOwnPackage(files, root, name, version) {
     const byDir = new Map();
     for (const f of files)
         byDir.set(dirOf(f), [...(byDir.get(dirOf(f)) ?? []), f]);
-    if (!(byDir.get('') ?? []).some(f => declaresApi(safeRead(f) ?? '')))
+    const sources = new Map();
+    const sourceOf = (f) => {
+        const seen = sources.get(f);
+        if (seen !== undefined)
+            return seen;
+        const text = safeRead(f) ?? '';
+        sources.set(f, text);
+        return text;
+    };
+    if (!(byDir.get('') ?? []).some(f => declaresApi(sourceOf(f))))
         return [...files];
     const major = goMajor(name, version);
     const reachable = new Set(['']);
@@ -737,7 +757,7 @@ export function selectOwnPackage(files, root, name, version) {
     while (queue.length > 0) {
         const dir = queue.shift();
         for (const f of byDir.get(dir) ?? []) {
-            for (const imp of importsOf(safeRead(f) ?? '')) {
+            for (const imp of importsOf(sourceOf(f))) {
                 if (!imp.startsWith(`${name}/`))
                     continue;
                 const sub = imp.slice(name.length + 1);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@mjasnikovs/pi-task",
-    "version": "0.40.31",
+    "version": "0.40.32",
     "description": "Deterministic task planning and spec-orchestration for local models — crash-safe /task pipelines with verify/enforce gates, a real-time remote web view, and web/docs/fetch/worker subagent tools.",
     "type": "module",
     "main": "./dist/index.js",