fix(sidebar): drag-release bounce, session-delete terminal cleanup, silent resize

[chalei1122-cell]

PR 描述（复制到 GitHub Pull Request 使用）

标题

fix(sidebar): eliminate drag-release bounce, clean up terminals on session delete, silent resize drag

描述（中文版）

一、修复拖动松手时的布局弹跳（3 个根因，src/client/Sidebar.tsx）

1. 聊天区弹回全宽再回归

• 现象：拖动右栏拉宽/拉窄松手后，聊天区瞬间弹回全宽再缩回。
• 根因：--dsh-sidebar-width/-height 的维护 effect 是 useEffect（paint 后异步执行），cleanup 的 removeProperty 与 setup 的 setProperty 在 passive-effect 分片 flush 时可能跨一次 paint——变量缺失帧里 #root 的 margin-right: var(...,0px) 回退到 0，聊天区被画成全宽。
• 修复：改为 useLayoutEffect（paint 前同步，cleanup+setup 同帧），卸载时的 removeProperty（issue Error-boundary swap leaves --dsh-sidebar-width/--dsh-sidebar-height behind: sidebar "cannot be hidden" after a render crash #31 语义）原样保留。

2. 底部面板右侧跳位

• 现象：开着底部面板拖右栏宽度，松手后底部面板右侧跳位。
• 根因：right: window.innerWidth - centerRect.right 依赖最近一次测量，松手提交那一帧 centerRect 还是拖动前旧值。
• 修复：宽度/角部拖动 onPointerUp 提交时同步更新 centerRect.right（用最终宽度 + 保留 details 列偏移推导），与 applyDrag 拖动中的直写一致。

3. 右栏关闭时聊天区被误挤 + 松手微跳

• 根因 A：applyDrag 无条件写 --dsh-sidebar-width，拖底部高度时传入的是 state.width 历史值（右栏已关闭仍非 0），聊天区被挤左。
• 根因 B：onPointerUp 先 stopDragScheduling() 丢弃挂起的 rAF，DOM 停留在倒数第二帧，与提交值差几像素。
• 修复：applyDrag 宽度只在 panelOpen 时写（effWidth，保留 details 偏移语义）；三个拖动（宽度/高度/角部）松手时先 flush 最终帧再提交。

二、会话删除时立即关闭该会话终端（src/index.ts + src/pty-manager.ts）

• 背景：UI 终端断开后由 reconnectGraceMs（默认 30s）宽限回收，用于"切换会话/刷新后重连同一 shell"。但删除会话后终端仍要等宽限到期才被回收。
• 修复：新增 PtyManager.closeSession(sessionId)，订阅 DSH 的 session/disposed 事件，会话销毁时立即关闭该会话全部 UI 终端（agent 终端由 agent 生命周期管理，不受影响）。

三、拖动时不高亮可拖动范围（src/client/sidebar.module.css）

• 本地偏好：右栏/底部/角部拖动手柄在拖动中不再显示高亮背景（panelResizeActive/bottomResizeActive/cornerHandle[data-dragging] 的 background 移除）；角部 hover 提示保留。如与项目风格冲突可舍弃此条。

测试

本机 DSH 0.1.0-rc.6 + 0.12.2（Windows）实测：

• 右栏/底部/角部拖动松手无弹跳、底部面板无跳位、右栏关闭时拖底部不挤聊天区
• 删除会话后该会话终端立即释放
• 切换会话后终端保留并在宽限期内重连同一条 shell

补丁文件

better-sidebar-contribution.patch（4 文件，93 增 22 删，git apply 干净通过）。

---

Description (English)

1. Fix drag-release layout bounce (src/client/Sidebar.tsx)

• Conversation flashes full-width: the --dsh-sidebar-width/-height effect ran as a passive useEffect; during chunked passive-effect flushing, the cleanup's removeProperty and setup's setProperty can straddle a paint, so #root's margin-right falls back to 0px for one painted frame. Switching to useLayoutEffect keeps cleanup+setup in the same pre-paint frame; the unmount removeProperty (issue Error-boundary swap leaves --dsh-sidebar-width/--dsh-sidebar-height behind: sidebar "cannot be hidden" after a render crash #31 semantics) is preserved.
• Bottom panel jumps on width drag release: right: window.innerWidth - centerRect.right used the pre-drag measurement for one frame. The width/corner onPointerUp now updates centerRect.right synchronously with the final width (details-column offset preserved).
• Closed right panel still squeezed the conversation while dragging the bottom height (applyDrag wrote the stale state.width), and release snapped a few px because stopDragScheduling() dropped the pending rAF. applyDrag now writes the width only when panelOpen (effWidth), and all three drags flush the final frame before committing.

2. Close a session's terminals when the conversation is deleted (src/index.ts + src/pty-manager.ts)

UI terminals survive disconnects for reconnectGraceMs (default 30 s) so a tab switch/refresh reattaches the same shell — but deleting a session left them lingering until the grace expired. Added PtyManager.closeSession(sessionId) and subscribed to DSH's session/disposed to release them immediately. Agent terminals are agent-lifecycle-owned and untouched.

3. No highlight while dragging the resize handles (src/client/sidebar.module.css)

Local preference: removed the drag-state background on the width/height/corner handles; the corner hover hint stays. Drop this commit if it conflicts with the project style.

Tests

Verified on DSH 0.1.0-rc.6 + 0.12.2 (Windows): no bounce on any drag release, bottom panel stays put, closed right panel no longer squeezes the conversation, session deletion releases its terminals immediately, and tab switches keep the same shell within the grace window.

[KaramachiA217]

独立排查了同款问题，结论一致。补两点实测证据 + 一个可选替代修法，供参考。

实测证据：松手瞬间逐帧采样 #root 的 margin-right（500ms）：

[503, 0, 3.2, 15.5, 41, 85, 150, 225, 293, 346, 388, 419, 443, 462, 476, 486, 494, 499, 502, 503, 503...]

松手瞬间 503→0，再用 300ms 缓动爬回 503，正是聊天区弹回全宽再缩回的量化。而且 delta=0（停下再松手、指针没动）同样掉 0，说明它与尾差无关，纯粹是 removeProperty 回退 0px 兜底 + 过渡重放。

触发点细化：除 passive effect 跨 paint 外，更精确的触发是 measureCenter() 的 getBoundingClientRect() 在 cleanup（removeProperty）与 setup（setProperty）之间运行，强制一次同步重排，把 0px 兜底定格成过渡起点。所以 useLayoutEffect 能消（cleanup+setup 同帧），但先移除再重设这步在每次 width/height 依赖变化（松手、开合、切会话）时仍会发生。

替代修法（可选）：把释放拆成独立的「仅卸载时」effect，依赖变化时只 setProperty、不 remove——0px 瞬态从机制上不存在，也不依赖 layout-effect 时序；卸载/HMR/崩溃边界交换时的释放语义（issue #31）原样保留：

useEffect(() => {
  // 只 setProperty --dsh-sidebar-width / --dsh-sidebar-height
}, [narrow, panelOpen, width, bottomOpen, bottomHeight])

useEffect(() => () => {
  document.documentElement.style.removeProperty('--dsh-sidebar-width')
  document.documentElement.style.removeProperty('--dsh-sidebar-height')
}, [])

尾差 flush（onPointerUp 先 flush 最终帧再提交）这部分结论一致。两种修法都成立，供取舍。