Repository navigation
fix: Replace console.error with structured logging in emitEventToSinks #140
Description
问题描述
session_runtime_helpers.ts:120-128 中的 emitEventToSinks 函数在生产代码中使用 console.error：
export async function emitEventToSinks(event: HistoryEvent, sinks: HistorySink[]) {
for (const sink of sinks) {
try {
await sink.append(event)
} catch (err) {
console.error(`Failed to write history event: ${(err as Error).message}`)
}
}
}问题：
- 生产环境错误日志：在关键路径（历史事件写入）失败时只记录到 console，可能导致重要数据丢失不被注意
- 错误信息泄露：
console.error可能输出敏感信息到日志 - 缺乏重试机制：写入失败后没有重试逻辑
建议修复
- 使用结构化日志系统（如 pino）
- 添加错误回调或通知机制
- 考虑实现重试逻辑
- 在关键错误时提供更详细的上下文
Activity
minorcell commentedon Feb 11, 2026
Fixed in commit f7e4056.
Changes:
console.errorinemitEventToSinkswith structured JSON logging tostderr.level,event,sink, andmessage.Validation:
pnpm test packages/core/src/runtime/session_runtime_helpers.test.tsminorcell commentedon Feb 11, 2026
Closing as fixed by f7e4056.
fix(core): use structured stderr logging for sink append failures (#140)