@mjasnikovs/pi-task 0.40.11 → 0.40.13

package/dist/shared/child-output.js

@@ -44,9 +44,18 @@ export function isExcerptInContent(excerpt, content) {
         return false;
     return normaliseWhitespace(content).includes(ne);
 }
-/** The child writes these to mark its own join, so they are not source text and
- *  not evidence of invention either. */
+/**
+ * Tokens that are not source text and not evidence of invention either: the
+ * child's own elision marks, and the provenance tag the PROMPT gave it.
+ *
+ * `buildExtractionPrompt` wraps the identity as `<package>aeson@2.2.5.1</package>`,
+ * and re-run 5 caught a child quoting that back inside an otherwise verbatim
+ * excerpt — the only absent word in 24 unverified excerpts across three runs. A
+ * complete lowercase element is the whole rule; `Vec<String>` does not start with
+ * `<` and `<T>` has no closing tag, so neither is excused.
+ */
 const ELISION = /^(?:\.{3}|\u2026|\/\/\s*\.{3})$/;
+const PROMPT_TAG = /^<([a-z_]+)>[^<]*<\/\1>$/;
 /**
  * Cover the excerpt with the longest runs the source actually contains, greedily.
  *
@@ -67,7 +76,7 @@ function coverBySource(normalisedExcerpt, normalisedContent) {
             run++;
         }
         if (run === 0) {
-            if (!ELISION.test(words[i]))
+            if (!ELISION.test(words[i]) && !PROMPT_TAG.test(words[i]))
                 absent.push(words[i]);
             i++;
         }

package/dist/workers/docs-chunk.d.ts

@@ -24,6 +24,14 @@ export declare const MAX_CHUNK_BYTES: number;
 /**
  * Where a declaration starts. Splitting here keeps a signature and its body in
  * one chunk, which is what makes a retrieved chunk quotable as evidence.
+ *
+ * The modifiers are SEQUENTIAL, not alternatives. Written as `export|declare`
+ * they excluded `export declare function` — which is what `tsc --declaration`
+ * emits for every exported function in a module: 6,935 such lines in a
+ * 4,000-file sample, 3,479 of them functions, and not one started a chunk.
+ * `undici-types/fetch.d.ts` put `export declare function fetch(` inside a chunk
+ * headed `export type RequestInfo`, so nothing could retrieve the chunk that
+ * defines `fetch`.
  */
 export declare const DECL_SPLIT_RE: RegExp;
 /** Where a README section starts. */

package/dist/workers/docs-chunk.js

@@ -24,8 +24,16 @@ export const MAX_CHUNK_BYTES = 8 * 1024;
 /**
  * Where a declaration starts. Splitting here keeps a signature and its body in
  * one chunk, which is what makes a retrieved chunk quotable as evidence.
+ *
+ * The modifiers are SEQUENTIAL, not alternatives. Written as `export|declare`
+ * they excluded `export declare function` — which is what `tsc --declaration`
+ * emits for every exported function in a module: 6,935 such lines in a
+ * 4,000-file sample, 3,479 of them functions, and not one started a chunk.
+ * `undici-types/fetch.d.ts` put `export declare function fetch(` inside a chunk
+ * headed `export type RequestInfo`, so nothing could retrieve the chunk that
+ * defines `fetch`.
  */
-export const DECL_SPLIT_RE = /^(?:export\s+|declare\s+)?(?:default\s+)?(?:async\s+)?(?:function|class|interface|type|namespace|module|const|let|var|enum)\s+/m;
+export const DECL_SPLIT_RE = /^(?:export\s+)?(?:declare\s+)?(?:default\s+)?(?:abstract\s+)?(?:async\s+)?(?:function|class|interface|type|namespace|module|const|let|var|enum)\s+/m;
 /** Where a README section starts. */
 export const README_SPLIT_RE = /^#{1,2} /m;
 /**

package/dist/workers/docs-retrieve.js

@@ -210,6 +210,16 @@ function definitionChunk(cache, opts, name) {
  * declaration per chunk, so the short chunk carrying the name IS its declaration and
  * the long ones are the prose that merely mentions it.
  */
+/** The chunk without the `// path` / `-- path` line the indexer prepends. */
+function chunkBody(content) {
+    return content.replace(/^(?:\/\/|--)[^\n]*\n/, '');
+}
+/**
+ * A chunk that only declares a MODULE. `pub mod serve;` is shorter than the
+ * function it contains and wins smallest-first, while being neither the value nor
+ * the type this hop exists to find.
+ */
+const MODULE_DECL_RE = /^(?:#\[[^\n]*\]\s*)*(?:pub(?:\s*\([^)]*\))?\s+|export\s+|declare\s+)*(?:mod|module|namespace)\s+[A-Za-z_][\w']*\s*;?\s*$/;
 function valueChunk(cache, opts, name) {
     const rows = cache.db
         .prepare(`SELECT file_path, kind, content, 0 AS rank FROM chunks
@@ -219,7 +229,14 @@ function valueChunk(cache, opts, name) {
     // LIKE has no word boundary, so `decodeFile` matches `decodeFileStrict` — the
     // wrong declaration, and the exact confusion these runs keep producing.
     const whole = new RegExp(`(?<![A-Za-z0-9_])${escapeRe(name)}(?![A-Za-z0-9_])`);
-    const row = rows.find(r => whole.test(r.content));
+    // Over the BODY, never the `// path` line the indexer prepends. Every chunk
+    // from `src/serve/mod.rs` carries "serve" in that line, so smallest-first
+    // handed axum's `serve` a 50-byte `pub struct TapIo` from a neighbouring file
+    // — the name was in the header and nowhere else.
+    const row = rows.find(r => {
+        const text = chunkBody(r.content);
+        return whole.test(text) && !MODULE_DECL_RE.test(text);
+    });
     if (!row)
         return null;
     return { filePath: row.file_path, kind: row.kind, content: row.content, rank: row.rank };

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@mjasnikovs/pi-task",
-    "version": "0.40.11",
+    "version": "0.40.13",
     "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",