@lincoln504/pi-research 1.0.1 → 1.0.2

package/dist/cli.mjs

@@ -20166,7 +20166,8 @@ ${prompt}`);
           const msg2 = `Researcher ${id} (${researcherConfig.name}) timed out after ${config.RESEARCHER_TIMEOUT_MS}ms`;
           session.abort().catch((err) => {
             logger.warn("[ResearcherExecutor] Failed to abort timed-out researcher session:", err);
-          }).finally(() => reject(new Error(msg2)));
+          });
+          reject(new Error(msg2));
         }, config.RESEARCHER_TIMEOUT_MS);
       });
       let abortCleanup;
@@ -20248,9 +20249,18 @@ ${responseText}`);
       }
     } finally {
       subscription();
-      await session.abort().catch((err) => {
-        logger.warn(`[ResearcherExecutor] Failed to abort researcher session ${id}:`, err);
-      });
+      await Promise.race([
+        session.abort().catch((err) => {
+          logger.warn(`[ResearcherExecutor] Failed to abort researcher session ${id}:`, err);
+        }),
+        new Promise((resolve8) => {
+          const t = setTimeout(() => {
+            logger.warn(`[ResearcherExecutor] Researcher ${id} session abort did not settle within 10s; continuing cleanup`);
+            resolve8();
+          }, 1e4);
+          t.unref?.();
+        })
+      ]);
       sessionService?.unregisterSession(researchId, id);
       if (ctx.mode !== "tui" && ctx.hasUI && typeof ctx.ui.setHiddenThinkingLabel === "function") {
         ctx.ui.setHiddenThinkingLabel();
@@ -20526,7 +20536,8 @@ ${prompt}`);
             const msg2 = `Quick research timed out after ${this.config.RESEARCHER_TIMEOUT_MS}ms`;
             session.abort().catch((err) => {
               logger.warn("[QuickOrchestrator] Failed to abort timed-out session:", err);
-            }).finally(() => reject(new Error(msg2)));
+            });
+            reject(new Error(msg2));
           }, this.config.RESEARCHER_TIMEOUT_MS);
         });
         let abortCleanup;

package/docs/ARCHITECTURE.md

@@ -301,7 +301,7 @@ Browser & scraping
 
 - [Camoufox](https://camoufox.com) — stealth Firefox (driven via [Playwright](https://playwright.dev)) for undetected search and scraping
 - [poolifier](https://github.com/poolifier/poolifier) — the worker-process pool behind the browser workers
-- [html-to-markdown](https://github.com/kreuzberg-dev/html-to-markdown) & [node-html-markdown](https://github.com/crosstype/node-html-markdown) — convert scraped HTML to Markdown
+- [html-to-markdown](https://github.com/kreuzberg-dev/html-to-markdown) — converts scraped HTML to Markdown (node-html-markdown serves as the pure-JS fallback)
 - `pdf-oxide-wasm` — PDF text extraction (Rust/WASM)
 
 Knowledge store & embeddings

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lincoln504/pi-research",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "A set of web research tools (search, scrape, and others) and research agent system that uses them safely and efficiently.",
   "type": "module",
   "private": false,

package/skills/pi-research/SKILL.md

@@ -2,7 +2,7 @@
 name: pi-research
 description: "Multi-agent web research for any coding agent. Use for anything requiring current information from the internet — news, trends, product/service/company facts, statistics, people, places, events, APIs, library behavior, or 'what is X / how does X work' questions. Also use to search a local knowledge store of previously researched findings before doing live research. Powered by the pi-research engine (search, stealth scraping, security databases, Stack Exchange). Not for reading local files, running project commands, or analyzing this repository's own source."
 license: MIT
-metadata: { "author": "Lincoln Deen", "version": "1.0.1", "package": "@lincoln504/pi-research", "engine": "pi-research-sdk", "openclaw": { "requires": { "bins": ["node"] } } }
+metadata: { "author": "Lincoln Deen", "version": "1.0.2", "package": "@lincoln504/pi-research", "engine": "pi-research-sdk", "openclaw": { "requires": { "bins": ["node"] } } }
 allowed-tools: Bash(node:*)
 ---
 

package/src/orchestration/quick-research-orchestrator.ts

@@ -269,9 +269,13 @@ export class QuickResearchOrchestrator {
           const timeoutPromise = new Promise<void>((_, reject) => {
               timeoutId = setTimeout(() => {
                   const msg = `Quick research timed out after ${this.config.RESEARCHER_TIMEOUT_MS}ms`;
+                  // Reject FIRST, abort as fire-and-forget cleanup — see researcher-executor:
+                  // abort() awaits the stuck in-flight request, so gating the rejection on it
+                  // settling disables the timeout exactly when it is needed.
                   session.abort().catch((err) => {
                       logger.warn('[QuickOrchestrator] Failed to abort timed-out session:', err);
-                  }).finally(() => reject(new Error(msg)));
+                  });
+                  reject(new Error(msg));
               }, this.config.RESEARCHER_TIMEOUT_MS);
           });
 

package/src/orchestration/researcher-executor.ts

@@ -264,9 +264,14 @@ export async function runResearcher(options: RunResearcherOptions): Promise<void
       const timeoutPromise = new Promise<void>((_, reject) => {
         timeoutId = setTimeout(() => {
           const msg = `Researcher ${id} (${researcherConfig.name}) timed out after ${config.RESEARCHER_TIMEOUT_MS}ms`;
+          // Reject FIRST, abort as fire-and-forget cleanup: session.abort() awaits the
+          // in-flight request, which is the very thing that is stuck (e.g. a provider
+          // retry loop against an exhausted quota) — gating the rejection on it settling
+          // makes the timeout ineffective exactly when it is needed.
           session.abort().catch((err) => {
             logger.warn('[ResearcherExecutor] Failed to abort timed-out researcher session:', err);
-          }).finally(() => reject(new Error(msg)));
+          });
+          reject(new Error(msg));
         }, config.RESEARCHER_TIMEOUT_MS);
       });
 
@@ -375,9 +380,23 @@ export async function runResearcher(options: RunResearcherOptions): Promise<void
       }
     } finally {
       subscription();
-      await session.abort().catch((err) => {
-        logger.warn(`[ResearcherExecutor] Failed to abort researcher session ${id}:`, err);
-      });
+      // Bounded: after a timeout, session.abort() awaits the very in-flight request
+      // that is stuck (e.g. a provider retry loop against an exhausted quota) and may
+      // never settle — never let per-attempt cleanup hang the retry loop on it. The
+      // retry builds a fresh session; the abandoned abort keeps draining in the
+      // background and its session dies with the run's teardown.
+      await Promise.race([
+        session.abort().catch((err) => {
+          logger.warn(`[ResearcherExecutor] Failed to abort researcher session ${id}:`, err);
+        }),
+        new Promise<void>((resolve) => {
+          const t = setTimeout(() => {
+            logger.warn(`[ResearcherExecutor] Researcher ${id} session abort did not settle within 10s; continuing cleanup`);
+            resolve();
+          }, 10_000);
+          t.unref?.();
+        }),
+      ]);
       sessionService?.unregisterSession(researchId, id);
 
       // Restore the default thinking label now that the researcher is done.