@zhushanwen/pi-system-prompt 1.1.0 → 1.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zhushanwen/pi-system-prompt",
-  "version": "1.1.0",
+  "version": "1.1.2",
   "description": "System prompt injection extension for Pi — reads config and appends to system prompt",
   "type": "module",
   "main": "src/index.ts",
@@ -24,13 +24,16 @@
     "index.ts"
   ],
   "peerDependencies": {
-    "@earendil-works/pi-coding-agent": "^0.84.1"
+    "@earendil-works/pi-coding-agent": "^0.84.4"
   },
   "devDependencies": {
     "@types/node": "^24.0.0",
     "@vitest/coverage-v8": "^4.1.9",
     "vitest": "^4.1.8"
   },
+  "dependencies": {
+    "@zhushanwen/pi-extension-logger": "0.3.1"
+  },
   "scripts": {
     "typecheck": "npx tsc --noEmit",
     "test": "vitest run"

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

@@ -8,7 +8,7 @@
  *   swap 注入顺序两行的 mutant 会被顺序用例 kill）
  * - -nc / --no-context-files 守卫：global 注入跳过、append 不受影响
  * - global 候选选择：候选序优先、空白内容跳过继续找、目录缺失降级
- * - fail-safe：handler 全程 throw → return undefined + stderr 可观测；stderr 写失败的
+ * - fail-safe：handler 全程 throw → return undefined + logger.error 可观测；logger 自身抛错的
  *   终极兜底 console.debug；systemPrompt 非法类型的旧 quirk 锚定
  *
  * mock 策略（参照 msg-id-mapper 测试模式）：pi SDK import type 零运行时解析，
@@ -17,6 +17,14 @@
  * 运行：cd extensions/taiji/system-prompt && npx vitest run
  */
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
+const { loggerMock } = vi.hoisted(() => ({
+  loggerMock: { debug: vi.fn(), warn: vi.fn(), error: vi.fn() },
+}))
+vi.mock('@zhushanwen/pi-extension-logger', () => ({
+  getLogger: () => loggerMock,
+  createLogger: () => loggerMock,
+}))
+
 import { readFileSync, readdirSync, statSync } from 'node:fs'
 import path from 'node:path'
 import createExtension from '../index'
@@ -271,8 +279,7 @@ describe('global 候选文件选择（readGlobalAgentsFile）', () => {
 })
 
 describe('fail-safe（外层 catch return undefined，永不阻断 agent loop）', () => {
-  it('handler 全程 throw（systemPrompt getter 抛错）→ return undefined + stderr 落盘可观测', () => {
-    const stderrSpy = vi.spyOn(process.stderr, 'write').mockImplementation(() => true)
+  it('handler 全程 throw（systemPrompt getter 抛错）→ return undefined + logger.error 落盘可观测', () => {
     const h = createHarness()
     const event = { type: 'before_agent_start', prompt: 'hi' } as unknown as BeforeAgentStartEvent
     Object.defineProperty(event, 'systemPrompt', {
@@ -281,16 +288,16 @@ describe('fail-safe（外层 catch return undefined，永不阻断 agent loop）
       },
     })
     expect(h.beforeAgentStart(event)).toBeUndefined()
-    expect(stderrSpy).toHaveBeenCalledWith(
-      expect.stringContaining('[xyz-system-prompt-extension] before_agent_start hook failed: Error: getter boom'),
+    expect(loggerMock.error).toHaveBeenCalledWith(
+      'before_agent_start hook failed: Error: getter boom',
     )
   })
 
-  it('stderr 写失败的终极兜底 → console.debug 吞掉，仍 return undefined', () => {
-    const stderrSpy = vi.spyOn(process.stderr, 'write').mockImplementation(() => {
-      throw new Error('stderr gone')
+  it('logger 自身抛错的终极兜底 → stderr 兜底，仍 return undefined', () => {
+    const stderrSpy = vi.spyOn(process.stderr, 'write').mockImplementation(() => true)
+    loggerMock.error.mockImplementation(() => {
+      throw new Error('logger gone')
     })
-    const debugSpy = vi.spyOn(console, 'debug').mockImplementation(() => {})
     const h = createHarness()
     const event = { type: 'before_agent_start', prompt: 'hi' } as unknown as BeforeAgentStartEvent
     Object.defineProperty(event, 'systemPrompt', {
@@ -299,9 +306,8 @@ describe('fail-safe（外层 catch return undefined，永不阻断 agent loop）
       },
     })
     expect(h.beforeAgentStart(event)).toBeUndefined()
-    expect(debugSpy).toHaveBeenCalledWith(
-      expect.stringContaining('[xyz-system-prompt-extension] stderr write also failed:'),
-      expect.any(Error),
+    expect(stderrSpy).toHaveBeenCalledWith(
+      expect.stringContaining('logHookFailure also failed'),
     )
   })
 

package/src/index.ts

@@ -25,6 +25,9 @@ import path from 'node:path'
 import { homedir } from 'node:os'
 import { readFileSync, readdirSync, statSync } from 'node:fs'
 import type { ExtensionAPI, BeforeAgentStartEvent } from '@earendil-works/pi-coding-agent'
+import { getLogger } from '@zhushanwen/pi-extension-logger'
+
+const logger = getLogger('xyz-system-prompt-extension')
 
 const CONFIG_FILE = 'system-prompt.json'
 
@@ -121,7 +124,7 @@ function readGlobalAgentsFile(): { path: string; content: string } | null {
       }
     } catch (err) {
       // best-effort：候选文件 stat/read 失败（如权限）→ 试下一个候选，never throw into the agent loop。
-      console.debug('[xyz-system-prompt-extension] candidate file read failed, trying next:', err)
+      logger.debug('candidate file read failed, trying next', { detail: String(err) })
     }
   }
   return null
@@ -220,10 +223,15 @@ function buildSystemPrompt(event: BeforeAgentStartEvent): { systemPrompt: string
 function logHookFailure(err: unknown): void {
   try {
     const msg = err instanceof Error ? `${err.name}: ${err.message}` : String(err)
-    process.stderr.write(`[xyz-system-prompt-extension] before_agent_start hook failed: ${msg}\n`)
+    logger.error(`before_agent_start hook failed: ${msg}`)
   } catch (nestedErr) {
     // best-effort：stderr 写失败的终极兜底——console 内部吞错不会抛，仍不外泄到 agent loop。
-    console.debug('[xyz-system-prompt-extension] stderr write also failed:', nestedErr)
+    try {
+      process.stderr.write(`[xyz-system-prompt-extension] logHookFailure also failed: ${String(nestedErr)}\n`)
+    } catch (finalErr) {
+      /* 完全静默：两层兜底都失败时无处可写 */
+      void finalErr
+    }
   }
 }