Skip to content

fix: 移动端侧边栏无法展开 + 网页断开任务中断#442

Open
HNGM-HP wants to merge 18 commits intoop7418:mainfrom
HNGM-HP:fix/mobile-sidebar-and-task-interrupt
Open

fix: 移动端侧边栏无法展开 + 网页断开任务中断#442
HNGM-HP wants to merge 18 commits intoop7418:mainfrom
HNGM-HP:fix/mobile-sidebar-and-task-interrupt

Conversation

@HNGM-HP
Copy link
Copy Markdown

@HNGM-HP HNGM-HP commented Apr 6, 2026

Summary

  • 移动端侧边栏无法展开:NavRail 组件移除后切换按钮丢失,导致移动端/小屏幕下无法打开左侧会话列表。本次在 UnifiedTopBar 添加 SidebarSimple hamburger 按钮,修复 ChatListPanel 残留的 left-14 偏移,并将 chatListOpen / setChatListOpen 通过 PanelContext 串联。
  • 网页断开连接后任务中断/api/chat/route.tsrequest.signal abort 直接转发给 SDK 子进程,导致浏览器关闭/断网时正在执行的任务被杀。移除该转发后,任务在服务端独立运行至完成;用户主动停止仍走 /api/chat/interrupt 正常中断。

Changes

文件 改动
src/components/layout/UnifiedTopBar.tsx 新增 SidebarSimple 切换按钮(侧边栏关闭时显示)
src/components/layout/ChatListPanel.tsx left-14left-0(移除已删 NavRail 的 56px 偏移)
src/hooks/usePanel.ts 接口新增 chatListOpen + setChatListOpen
src/components/layout/AppShell.tsx 通过 PanelContext 传递 chatListOpen / setChatListOpen
src/app/api/chat/route.ts 移除 request.signalabortController.abort() 转发

Test

  • npm run typecheck ✅ 0 errors
  • npm run lint ✅ 0 errors (161 pre-existing warnings)

AI Briefing and others added 18 commits March 16, 2026 18:55
Handle permission prompts in CodePilot instead of relying on Claude Code bypass mode so auto-approve still works in environments where bypassPermissions is rejected.
…n browser disconnect

1. Mobile sidebar: Add SidebarSimple toggle button in UnifiedTopBar
   (visible when sidebar is closed on all viewports). Remove orphaned
   left-14 offset from ChatListPanel (was for the removed NavRail).
   Wire chatListOpen/setChatListOpen through PanelContext.

2. Task interrupt: Remove request.signal → abortController forwarding
   in /api/chat/route.ts so browser disconnect no longer kills the
   SDK subprocess. Intentional stops already use /api/chat/interrupt.
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 6, 2026

@HNGM-HP is attempting to deploy a commit to the op7418's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

1 participant