@verneuil/dsh-ocg-session-head 0.1.3 → 0.1.4

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## 0.1.4
+
+### 修复
+- 修复请求头监听器的异常边界：下游贡献者抛错时会被本插件二次调用 `next()`，把真实错误改写成 `next() called multiple times`。现在 `next()` 只在 `try` 外调用一次，下游错误原样上浮，自身错误仍 fail-open。
+
+---
+### Fixes
+- Fixed the listener's error boundary: when a downstream contributor threw, the plugin could call `next()` a second time and replace the real error with `next() called multiple times`. `next()` is now called exactly once outside `try`, downstream errors surface untouched, and the plugin's own failures still fail open.
+
 ## 0.1.3
 
 ### 修复

package/index.js

@@ -243,24 +243,28 @@ export function apply(ctx, config) {
   const uuidBySession = new Map()
 
   ctx.on('llm-pi-ai/request-headers', (request, headers, next) => {
-    // 宿主契约要求监听者必须调用 next()；任何异常都不允许把整次推理打断，
-    // 因此在本地逻辑上失败即「放弃注入、原样放行」。
+    // 契约要求每个监听者**恰好调用一次** next()。
+    // try 只包住「自己的判断与写头」：这里出错就地放弃注入（fail-open），
+    // 但不能把下游监听者的异常一起吞掉——那会把别人的真实错误改写成 cordis 的
+    // "next() called multiple times"，排查时完全失真。
+    if (typeof next !== 'function') return headers
     try {
-      if (typeof next !== 'function') return headers
-      if (request === null || typeof request !== 'object') return next()
-      if (headers === null || typeof headers !== 'object' || typeof headers.set !== 'function') return next()
-      if (!isOpenCodeRoute(options, request)) return next()
-      // 已有同名头一律让路：与官方实现、其它插件、静态 headers 配置共存。
-      if (typeof headers.has === 'function' && headers.has(options.header)) return next()
-      const value = headerValueFor(options, uuidBySession, request.sessionId)
-      if (value === undefined) return next()
-      headers.set(options.header, value)
-      if (options.debug) {
-        ctx.logger.info(
-          `[dsh-ocg-session-head] ${options.header}=${value} `
-            + `provider=${String(request.provider)} model=${String(request.model)} `
-            + `purpose=${String(request.purpose ?? 'chat')}`,
-        )
+      if (request !== null && typeof request === 'object'
+        && headers !== null && typeof headers === 'object' && typeof headers.set === 'function'
+        && isOpenCodeRoute(options, request)
+        // 已有同名头一律让路：与官方实现、其它插件、静态 headers 配置共存。
+        && !(typeof headers.has === 'function' && headers.has(options.header))) {
+        const value = headerValueFor(options, uuidBySession, request.sessionId)
+        if (value !== undefined) {
+          headers.set(options.header, value)
+          if (options.debug) {
+            ctx.logger.info(
+              `[dsh-ocg-session-head] ${options.header}=${value} `
+                + `provider=${String(request.provider)} model=${String(request.model)} `
+                + `purpose=${String(request.purpose ?? 'chat')}`,
+            )
+          }
+        }
       }
     } catch (error) {
       ctx.logger.warn(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@verneuil/dsh-ocg-session-head",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "OpenCode Go 会话头：监听宿主 llm-pi-ai/request-headers 瀑布，按会话为 ocg 路由注入 x-opencode-session（不猴补丁 fetch、不修改宿主）。",
   "main": "index.js",
   "type": "module",