@zhushanwen/pi-system-prompt 1.4.0 → 1.4.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zhushanwen/pi-system-prompt",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "description": "System prompt injection extension for Pi — reads config and appends to system prompt",
   "type": "module",
   "main": "src/index.ts",
@@ -36,6 +36,6 @@
   },
   "scripts": {
     "typecheck": "npx tsc --noEmit",
-    "test": "vitest run"
+    "test": "vitest run --pool=threads"
   }
 }

package/src/__tests__/system-prompt.test.ts

@@ -17,7 +17,7 @@
  * 在 config 里显式 "capability":{"enabled":false} 隔离关注点；无法塞字段的用例（畸形 JSON /
  * 文件缺失）期望值如实更新为含 capability 段。capability 自身行为由专项 describe 锚定。
  *
- * mock 策略（参照 msg-id-mapper 测试模式）：pi SDK import type 零运行时解析，
+ * mock 策略（参照 host-internals msg-id-mapper 测试模式）：pi SDK import type 零运行时解析，
  * ExtensionAPI 用结构化桩；node:fs mock 后按路径分流（env 指向假目录，不碰真实文件系统）。
  *
  * 运行：cd extensions/taiji/system-prompt && npx vitest run
@@ -58,7 +58,7 @@ const CAP_HEADER = '# TaiJi capabilities'
 /** 既有用例隔离关注点用的「capability 显式关闭」config 片段 */
 const CAP_OFF = '"capability": {"enabled": false}'
 
-/** hook 注册表桩（参照 msg-id-mapper harness 模式）——handler 第二参 ctx 透传（路径注入测试用） */
+/** hook 注册表桩（参照 host-internals msg-id-mapper harness 模式）——handler 第二参 ctx 透传（路径注入测试用） */
 function createHarness(): { beforeAgentStart: (event: BeforeAgentStartEvent, ctx?: unknown) => unknown } {
   const handlers = new Map<string, (...args: unknown[]) => unknown>()
   const pi = {
@@ -386,24 +386,6 @@ describe('fail-safe（外层 catch return undefined，永不阻断 agent loop）
     )
   })
 
-  it('logger 自身抛错的终极兜底 → stderr 兜底，仍 return undefined', () => {
-    const stderrSpy = vi.spyOn(process.stderr, 'write').mockImplementation(() => true)
-    loggerMock.error.mockImplementation(() => {
-      throw new Error('logger gone')
-    })
-    const h = createHarness()
-    const event = { type: 'before_agent_start', prompt: 'hi' } as unknown as BeforeAgentStartEvent
-    Object.defineProperty(event, 'systemPrompt', {
-      get() {
-        throw new Error('getter boom')
-      },
-    })
-    expect(h.beforeAgentStart(event)).toBeUndefined()
-    expect(stderrSpy).toHaveBeenCalledWith(
-      expect.stringContaining('logHookFailure also failed'),
-    )
-  })
-
   it('systemPrompt 非法类型（undefined）且无 append → 旧 quirk 锚定：返回仅含 capability 段的结果而非 undefined', () => {
     // R3 复核锚定的返回值检查 quirk（index.ts newPrompt === event.systemPrompt 比较）：
     // base 收敛 ''，capability 默认开 → newPrompt 非空且 !== undefined → 必然返回对象。

package/src/index.ts

@@ -452,22 +452,15 @@ function buildSystemPrompt(event: BeforeAgentStartEvent, ctx?: ExtensionContext)
  * 落盘诊断，不泄露配置内容。
  *
  * 通道：extension-logger 的 error → appendEntry 写入 session JSONL（需 setPiHandle
- * 注入 pi handle 后生效）；TAIJI_AGENT_DEBUG=1 时另落文件日志。仅当 logger 自身抛错
- * 时才兜底 process.stderr.write（下方 catch），不外泄到 agent loop。
+ * 注入 pi handle 后生效）；TAIJI_AGENT_DEBUG=1 时另落文件日志。logger 全程吞错
+ * （appendEntry 与 fileLog 通道内部各自整体 try/catch，实装见 extension-logger
+ * error 方法），不会抛错到本调用方——旧的两层嵌套兜底（stderr 终极兜底 + 静默
+ * catch）已删，恒不命中。若未来 logger 改为可抛实现，异常由外层 handler 的
+ * try/catch 兜住（同样不外泄到 agent loop）。
  */
 function logHookFailure(err: unknown): void {
-  try {
-    const msg = err instanceof Error ? `${err.name}: ${err.message}` : String(err)
-    logger.error(`before_agent_start hook failed: ${msg}`)
-  } catch (nestedErr) {
-    // best-effort：logger 抛错时的终极兜底 process.stderr.write——其内部吞错不会抛，仍不外泄到 agent loop。
-    try {
-      process.stderr.write(`[taiji-system-prompt-extension] logHookFailure also failed: ${String(nestedErr)}\n`)
-    } catch (finalErr) {
-      /* 完全静默：两层兜底都失败时无处可写 */
-      void finalErr
-    }
-  }
+  const msg = err instanceof Error ? `${err.name}: ${err.message}` : String(err)
+  logger.error(`before_agent_start hook failed: ${msg}`)
 }
 
 export default function (pi: ExtensionAPI): void {