Skip to content

fix(session): 修复会话清理内存泄漏并补充回归测试#123

Merged
qorzj merged 5 commits into
lessweb:mainfrom
Lellansin:dev/memory-leak-fixes-origin-main
May 26, 2026
Merged

fix(session): 修复会话清理内存泄漏并补充回归测试#123
qorzj merged 5 commits into
lessweb:mainfrom
Lellansin:dev/memory-leak-fixes-origin-main

Conversation

@Lellansin

@Lellansin Lellansin commented May 26, 2026

Copy link
Copy Markdown
Contributor

变更说明

  • 修复会话删除与自动裁剪场景下的会话级内存泄漏,统一清理 file state/snippet/snippet counter/file version 缓存
  • 修复会话删除后的 bash 工作目录残留问题,确保 session 复用时 cwd 回落到项目根目录
  • 增强 SessionManager.dispose 清理逻辑,补齐 active prompt controllersession controllerprocess timeout controls 的释放
  • 增加会话资源统一清理路径,覆盖消息文件、控制器与受当前运行期跟踪的进程超时控制项
  • 避免清理阶段误杀陈旧 PID(仅处理当前运行期可确认的进程控制项),消除潜在主机风险
  • 新增 src/tests/memory-leak.test.ts 回归测试,覆盖删除、自动裁剪、dispose、cwd 重置及 stale pid 防护场景
  • 修复 Windows CI 下 pwd 路径断言兼容性(Git Bash 输出 POSIX 路径)

验证

  • npm run typecheck
  • npm run test:single -- src/tests/memory-leak.test.ts
  • codex review --base origin/main

@qorzj qorzj merged commit b660dbf into lessweb:main May 26, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants