mirror of
https://github.com/openclaw/openclaw.git
synced 2026-06-07 09:42:55 +00:00
2f449285b9
Refactor ACP close-session ownership by extracting the runtime close/recovery lifecycle into `manager.close-session.ts`. Verification: - `pnpm test src/acp/control-plane/manager.test.ts src/acp/control-plane/manager.runtime-config.test.ts src/acp/control-plane/manager.runtime-handles.test.ts` - `pnpm tsgo:prod` - `pnpm check:test-types` - `node scripts/run-oxlint.mjs src/acp/control-plane/manager.core.ts src/acp/control-plane/manager.close-session.ts` - `pnpm format:check src/acp/control-plane/manager.core.ts src/acp/control-plane/manager.close-session.ts` - `git diff --check` - `.agents/skills/autoreview/scripts/autoreview --mode local` - GitHub PR checks for #88744 passed Real behavior proof: Behavior addressed: ACP close-session ownership moved out of `AcpSessionManager` without changing close/recovery behavior. Real environment tested: Local OpenClaw checkout, Node/pnpm repo toolchain, GitHub Actions PR CI. Exact steps or command run after this patch: Focused ACP manager tests covering close-session behavior, runtime config, and runtime handles, plus prod/test type checks, lint, format, diff check, autoreview, and PR CI. Evidence after fix: All listed local commands passed, autoreview reported no accepted/actionable findings, and GitHub PR checks passed. Observed result after fix: `manager.core.ts` dropped from 1149 LOC to 1038 LOC while close-session runtime lifecycle handling lives in `manager.close-session.ts`. What was not tested: Live ACP backend close/recovery against a real external ACP provider.