ben
fe5508228f
fix: 修复多 backend 并行日志 PID 混乱并移除包装格式 ( #74 ) ( #76 )
...
* fix(logger): 修复多 backend 并行日志 PID 混乱并移除包装格式
**问题：**
- logger.go:288 使用 os.Getpid() 导致并行任务日志 PID 混乱
- 日志文件添加时间戳/PID/级别前缀包装，应输出 backend 原始内容
**修复：**
1. Logger 结构体添加 pid 字段，创建时捕获 PID
2. 日志写入使用固定 l.pid 替代 os.Getpid()
3. 移除日志输出格式包装，直接写入原始消息
4. 添加内存缓存 ERROR/WARN 条目，ExtractRecentErrors 从缓存读取
5. 优化 executor.go context 初始化顺序，避免重复创建 logger
**测试：**
- 所有测试通过（23.7s）
- 更新相关测试用例匹配新格式
Closes #74
* fix(logger): 增强并发日志隔离和 task ID 清理
## 核心修复
### 1. Task ID Sanitization (logger.go)
- 新增 sanitizeLogSuffix(): 清理非法字符 (/, \, :, 等)
- 新增 fallbackLogSuffix(): 为空/非法 ID 生成唯一后备名
- 新增 isSafeLogRune(): 仅允许 [A-Za-z0-9._-]
- 路径穿越防护: ../../../etc/passwd → etc-passwd-{hash}.log
- 超长 ID 处理: 截断到 64 字符 + hash 确保唯一性
- 自动创建 TMPDIR (MkdirAll)
### 2. 共享日志标识 (executor.go)
- 新增 taskLoggerHandle 结构: 封装 logger、路径、共享标志
- 新增 newTaskLoggerHandle(): 统一处理 logger 创建和回退
- printTaskStart(): 显示 "Log (shared)" 标识
- generateFinalOutput(): 在 summary 中标记共享日志
- 并发失败时明确标识所有任务使用共享主日志
### 3. 内部标志 (config.go)
- TaskResult.sharedLog: 非导出字段，标识共享日志状态
### 4. Race Detector 修复 (logger.go:209-219)
- Close() 在关闭 channel 前先等待 pendingWG
- 消除 Logger.Close() 与 Logger.log() 之间的竞态条件
## 测试覆盖
### 新增测试 (logger_suffix_test.go)
- TestLoggerWithSuffixSanitizesUnsafeSuffix: 非法字符清理
- TestLoggerWithSuffixReturnsErrorWhenTempDirNotWritable: 只读目录处理
### 新增测试 (executor_concurrent_test.go)
- TestConcurrentTaskLoggerFailure: 多任务失败时共享日志标识
- TestSanitizeTaskID: 并发场景下 task ID 清理验证
## 验证结果
✅ 所有单元测试通过
✅ Race detector 无竞态 (65.4s)
✅ 路径穿越攻击防护
✅ 并发日志完全隔离
✅ 边界情况正确处理
Resolves: PR #76 review feedback
Co-Authored-By: Codex Review <codex@anthropic.ai >
Generated with swe-agent-bot
Co-Authored-By: swe-agent-bot <agent@swe-agent.ai >
* fix(logger): 修复关键 bug 并优化日志系统 (v5.2.5)
修复 P0 级别问题：
- sanitizeLogSuffix 的 trim 碰撞（防止多 task 日志文件名冲突）
- ExtractRecentErrors 边界检查（防止 slice 越界）
- Logger.Close 阻塞风险（新增可配置超时机制）
代码质量改进：
- 删除无用字段 Logger.pid 和 logEntry.level
- 优化 sharedLog 标记绑定到最终 LogPath
- 移除日志前缀，直接输出 backend 原始内容
测试覆盖增强：
- 新增 4 个测试用例（碰撞防护、边界检查、缓存上限、shared 判定）
- 优化测试注释和逻辑
版本更新：5.2.4 → 5.2.5
Generated with swe-agent-bot
Co-Authored-By: swe-agent-bot <agent@swe-agent.ai >
---------
Co-authored-by: swe-agent-bot <agent@swe-agent.ai >
2025-12-17 10:33:38 +08:00
cexll
d215c33549
fix(executor): isolate log files per task in parallel mode
...
Previously, all parallel tasks shared the same log file path, making it
difficult to debug individual task execution. This change creates a
separate log file for each task using the naming convention:
codeagent-wrapper-{pid}-{taskName}.log
Changes:
- Add withTaskLogger/taskLoggerFromContext for per-task logger injection
- Modify executeConcurrentWithContext to create independent Logger per task
- Update printTaskStart to display task-specific log paths
- Extract defaultRunCodexTaskFn for proper test hook reset
- Add runCodexTaskFn reset to resetTestHooks()
Test coverage: 93.7%
Generated with swe-agent-bot
Co-Authored-By: swe-agent-bot <agent@swe-agent.ai >
2025-12-16 10:05:54 +08:00
