@240xu/dsh-websearch 2.8.1 → 2.8.3

package/README.md

@@ -253,3 +253,25 @@ v2.7.0 缓存/熔断/历史三项的依据与刻意取舍（含查证过的反
 ## License | 许可
 
 MIT © 2026 240xu
+
+## 2.8.2 · Bug 猎场修复（P1×3 + 回归测试）
+
+- **[P1] 零结果伪造失败**：所有后端成功但 0 命中时，原实现落进 `all enabled
+  backends failed (0) - `（计数自相矛盾、detail 空）并丢弃 mergedContent（Tavily
+  直答）。成功即空 → 正常返回 `{sources:[]}` + history 记录。
+- **[P1] limiter 死锁**：`Promise.resolve(fn())` 先执行 fn——同步抛出（如 baseURL
+  校验在建 timeout 前）逃出后 `running` 永不递减，泄漏 ≥ concurrency 后所有任务
+  永久排队且超时机制未武装（实测 HUNG exit=13）。改 `Promise.resolve().then(fn)`。
+- **[P1] exa category 漏门控**：tavily/searxng 的 deepCoverage 增强都门控，唯独
+  exa 漏了——默认关 deepCoverage 也注入 category 收窄结果，与 README 承诺相反。
+- 回归测试 3 条（零结果/limiter 不死锁带 HUNG 超时护栏/exa 双态门控）；
+  `createLimiter` 导出供测试。89/89。
+
+## 2.8.3 · Bug 猎场修复第二批（P2×3 + 测试）
+
+- **[P2] 整形空查询早拒**：寒暄独词（"搜索"/"search"/"查询"）被剥空后不再扇出、
+  不入缓存（原来空串共享缓存键 + 落入 all-backends-failed(0) 误导错误）；
+  ZH 寒暄表补裸「搜索」（原 `搜索一下?` 要求带“一”漏掉裸词）。
+- **[P2] CJK 分隔符词内保护**：和/比较/对比 须**任一侧**有空白/标点边界才切——
+  词内裸和（柔和光线/和平精英）不再切断实体；单侧空格的真分隔（"sftp 和面板"）照切。
+- 回归测试 ×4（零结果/limiter 死锁/exa 门控/CJK 保护/空查询拒），91/91。

package/lib/backends/exa.js

@@ -42,10 +42,14 @@ export const exaBackend = {
       query: request.query,
       numResults: request.maxResults ?? 8,
     };
-    // v2.8.0: category inference (feature-detect — only sent when confident;
-    // if the endpoint rejects it the fan-out demotes softly).
-    const category = inferExaCategory(request.query);
-    if (category) args.category = category;
+    // v2.8.0: category inference — deepCoverage-gated like tavily (searchDepth
+    // advanced) and searxng (category sweep): 2.8.1 审计发现此处漏了门控，
+    // 默认关着 deepCoverage 也注入 category，把结果悄悄收窄到单类目，与 README
+    // 「deepCoverage 深度覆盖」承诺相反。
+    if (request.deepCoverage === true) {
+      const category = inferExaCategory(request.query);
+      if (category) args.category = category;
+    }
     hooks.recordRequest?.({
       endpoint: EXA_ENDPOINT,
       toolName: EXA_TOOL,

package/lib/multiquery.js

@@ -9,7 +9,14 @@
 // Separators per the v2.8 spec: "vs", "and", 比较, 对比, 和 — plus sentence
 // punctuation for the long-query case.
 
-const SEPARATOR_SPLIT_RE = /\s+(?:vs\.?|and)\s+|\s*(?:比较|对比|和)\s*|\s*[、；;。！？!?|]+\s*/i;
+// 2.8.2（P2）：CJK 分隔符必须两侧有空白才切——裸 "和"（无空格）常在词内
+// （柔和/和平/和尚），原无边界写法会把实体拦腰切断
+// （"柔和光线和自然光" → 变体丢 "柔和"；"和平精英" → 切出 "平精英"）。
+// 显式空格分隔的 "A 和 B" 与顿号/标点分隔仍正常切。
+// 2.8.2（P2 定稿）：CJK 分隔符须**任一侧**有空白/标点边界才切——词内裸和
+// （柔和/和平/和尚，两侧都无空白）不切，保住实体；"sftp 和面板"（单侧空格）
+// 这类真分隔照切。
+const SEPARATOR_SPLIT_RE = /\s+(?:vs\.?|and)\s+|\s+(?:比较|对比|和)|(?:比较|对比|和)\s+|\s*[、；;。！？!?|]+\s*/i;
 const COMPLEX_MARK_RE = /(?:\bvs\.?\b|\band\b|比较|对比|和)/i;
 
 /** Sub-query hard cap (original + 2 variants max). */

package/lib/prompting.js

@@ -30,7 +30,7 @@ export const QUERY_MAX_CHARS = 1500;
 const FILLER_PREFIX_RE =
   /^(?:please|pls|kindly|could you(?: please)?|can you(?: please)?|help me(?: to)?|search for|search|look up|google|find me|find|i(?:'d| would)? like to know|tell me|what is|who is|when is|where is|"|「|『)\s*/i;
 const FILLER_PREFIX_RE_ZH =
-  /^(?:请|麻烦|帮我?|帮忙|帮我查一?下|查一?下|查询|搜索一下?|搜一下?|查一下?|帮我找|找一下?|我想知道|请问)/;
+  /^(?:请|麻烦|帮我?|帮忙|帮我查一?下|查一?下|查询|搜索一?下?|搜一下?|查一下?|帮我找|找一下?|我想知道|请问)/;
 
 /**
  * 确定性查询整形：剥离口语前缀（循环多次以处理叠加寒暄）、压缩空白、

package/lib/provider.js

@@ -82,7 +82,8 @@ export function getBackendBreaker(id) {
  * Simple concurrency limiter using a semaphore pattern.
  * No external dependency needed.
  */
-function createLimiter(concurrency) {
+// 2.8.2: exported for P1 regression tests (limiter sync-throw deadlock)
+export function createLimiter(concurrency) {
   const queue = [];
   let running = 0;
   
@@ -95,7 +96,12 @@ function createLimiter(concurrency) {
         while (running < concurrency && queue.length > 0) {
           const { fn, resolve, reject } = queue.shift();
           running++;
-          Promise.resolve(fn()).then(resolve).catch(reject).finally(() => {
+          // 2.8.2（P1）：必须 .then(fn) 而非 Promise.resolve(fn())——后者先执行
+          // fn()，同步抛出（如 runBackend 的 baseURL 校验在创建 timeout 之前 throw）
+          // 会逃出本函数：running 已 ++ 却永不 --（finally 没挂上），leaked 计数
+          // ≥ concurrency 后所有后续任务永久排队，且 timeoutId 未创建 → 超时也救不了
+          // （实测全挂起 exit=13）。.then(fn) 把同步抛出转为 rejection 落进 finally。
+          Promise.resolve().then(fn).then(resolve).catch(reject).finally(() => {
             running--;
             process();
           });
@@ -190,6 +196,15 @@ export function createUnifiedSearchProvider({ ctx, resolveOptions, backends, cac
       // 确定性归一 —— 剥离寒暄前缀、压缩空白、截断到 1500 字符；
       // 专名/版本号/日期原样保留，不改写语义。
       effReq.query = shapeQuery(effReq.query);
+      // 2.8.2（P2）：整形后为空（"搜索"/"search" 等寒暄独词被剥光）→ 早拒。
+      // 不拒会：①空串扇出全部后端 ②不同原始查询共享 "" 缓存键（别名污染）
+      // ③落入 all-backends-failed(0) 的误导错误。
+      if (!effReq.query) {
+        throw new WebError(
+          "unified search: empty query after shaping (the request contained only fillers like \"search\")",
+          "WEB_PROVIDER_ERROR",
+        );
+      }
       if (Object.keys(filters).length > 0) effReq = { ...effReq, filters };
       const maxResults = effReq.maxResults;
       const concurrency = opts.concurrency ?? 6;
@@ -435,6 +450,31 @@ export function createUnifiedSearchProvider({ ctx, resolveOptions, backends, cac
         };
       }
 
+      // 2.8.2（P1）：零结果 ≠ 全部失败。所有后端成功但确实 0 条命中（冷僻查询/
+      // 过滤后为空）时，原实现无条件落入下面的 throw，模型端拿到
+      // `all enabled backends failed (0) - `（detail 空、计数自相矛盾），且丢弃已
+      // 拿到的 mergedContent（如 Tavily 直答）。成功即空 → 正常返回空结果。
+      if (allFailures.length === 0) {
+        let emptyContent = perVariant.find((v) => v.mergedContent !== undefined)?.mergedContent;
+        if (opts.resultHeader !== false && emptyContent !== undefined) {
+          const header = resultHeader(typeof request?.query === "string" && /[\u4e00-\u9fff]/.test(request.query) ? "zh" : "en");
+          emptyContent = header + "\n\n" + emptyContent;
+        }
+        history?.record({
+          query: request?.query,
+          time: Date.now(),
+          resultCount: 0,
+          backendsOk: allTelemetry.filter((t) => t.ok).length,
+          backendsTotal: allTelemetry.length,
+        });
+        if (cacheKey && cache) cache.set(cacheKey, { sources: [], content: emptyContent, truncated: false });
+        return {
+          sources: [],
+          ...(emptyContent !== undefined ? { content: emptyContent } : {}),
+          truncated: false,
+        };
+      }
+
       // All enabled backends failed - synthesize one meaningful error that
       // names each backend's own reason so the caller (model or user) can act.
       const detail = allFailures.slice(0, 4)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@240xu/dsh-websearch",
-  "version": "2.8.1",
+  "version": "2.8.3",
   "description": "Unified web search provider for DeepSeek Harness (ctx.web): 11 backends — Exa + Parallel + DDG + SearXNG (keyless), DeepSeek + Anthropic + OpenAI + Brave + Tavily + Serper + Mojeek (key-gated via Settings credentials). Registers provider id \"unified\" for zero-config web_search. 统一网页搜索聚合：11 后端并发扇出、去重合并，零依赖。See CHANGELOG.md for version history.",
   "type": "module",
   "main": "lib/index.js",