Skip to content

fix: improve Web Remote mobile scrolling and connection recovery - #97

Merged
chenbstack merged 4 commits into
chenbstack:mainfrom
defia:fix/web-remote-mobile-scroll
Aug 10, 2026
Merged

fix: improve Web Remote mobile scrolling and connection recovery#97
chenbstack merged 4 commits into
chenbstack:mainfrom
defia:fix/web-remote-mobile-scroll

Conversation

@defia

@defia defia commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

变更内容 / Summary

  • 为 Web Remote 终端增加双指触摸滚动,并修复快速切换 pane 时旧选择覆盖新选择的问题。
  • 浏览器每 3 秒探测连接;连续 12 秒没有服务端消息,或网络/页面恢复时发现连接失效,会自动重连。
  • 服务端接收失败时主动取消底层 NWConnection,避免断开的浏览器遗留 socket 描述符。
  • 保留旧 socket 的事件隔离,自动恢复不会重新引入手动重连循环。

验证方式 / Testing

  • Web Remote protocol and integration tests: 52 tests, 0 failures.
  • node --check Glint/Resources/WebRemote/web-remote.js.
  • Deterministic Node reconnect harness passed.
  • Real abrupt-disconnect probe left no residual CLOSE_WAIT or CLOSED client socket.
  • git diff --check.
  • macOS Debug build succeeded.
  • 手机浏览器实测 Codex 和 Claude Code 双指滚动正常。

截图或录屏 / Screenshots or recordings

无。

关联议题 / Related issues

无。

@defia defia changed the title fix: improve Web Remote mobile scrolling and pane switching fix: improve Web Remote mobile scrolling and connection recovery Aug 9, 2026
@chenbstack
chenbstack merged commit 2e67a27 into chenbstack:main Aug 10, 2026
1 check passed
CtriXin pushed a commit to CtriXin/glint that referenced this pull request Aug 10, 2026
…nbstack#97)

* fix(web-remote): improve mobile scrolling and pane switching

* fix(web-remote): recover stale browser connections

* fix(web-remote): isolate stale async callbacks

* test(web-remote): avoid random token assertion flake
CtriXin pushed a commit to CtriXin/glint that referenced this pull request Aug 10, 2026
Tracks upstream chenbstack/glint v0.1.27-beta.2: two-finger touch
scrolling and stale-connection recovery in Web Remote (chenbstack#97), and the
white-flash fix when closing a split (chenbstack#99). Both cherry-picked cleanly
onto the distribution branch.
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