feat(app): Mobile Touch Optimization by noahbentusi · Pull Request #18767 · anomalyco/opencode

noahbentusi · 2026-03-23T11:05:41Z

Issue for this PR

Closes #

Type of change

New feature
Bug fix
Refactor / code improvement
Documentation

What does this PR do?

This PR optimizes the OpenCode App for mobile/touch devices while preserving the existing desktop experience. It addresses 5 key pain points for mobile users:

Toolbar Overflow - Hidden toolbar buttons are now accessible via an overflow menu on narrow screens
Mobile Panel Fullscreen - Side panels (file tree, review) display fullscreen on mobile instead of split view
Selection Comment - Touch and mouse users can select code and add comments via a floating "+" button
Long-Press File Menu - Touch users can long-press files to add them to the conversation context
Markdown Render Toggle - Markdown files can switch between rendered and source views

The changes include:

Toolbar overflow menu with "⋯" button for hidden options on screens < 768px
Mobile fullscreen panel layout for file tree and review panels
Floating selection toolbar with "+" button for adding comments to selected code
Long-press context menu for file items to add to conversation
Markdown render/source toggle functionality
Back to chat navigation button on mobile toolbar

These improvements enhance mobile usability by making touch interactions more intuitive and preserving the desktop experience for larger screens.

How did you verify your code works?

All features tested on mobile viewport (720×1920):

Overflow menu shows hidden toolbar options
Back to chat button returns to chat view
File tree panel opens fullscreen
Clicking file shows content in panel
Selecting code shows "+" button
Clicking "+" opens comment dialog
Long-press/right-click on file shows context menu
Desktop mode (>768px) remains unaffected

Screenshots / recordings

Mobile toolbar with overflow menu and back button visible. File tree in fullscreen mode with file content displayed.

Checklist

I have tested my changes locally
I have not included unrelated changes in this PR

…t and message rendering When the client clock is ahead of the server, user message IDs (generated client-side) sort after assistant message IDs (generated server-side). This broke the prompt loop exit check and the UI message pairing logic. - Extract shouldExitLoop() into a pure function that uses parentID matching instead of relying on ID ordering - Extract findAssistantMessages() with forward+backward scan to handle messages sorted out of expected order due to clock skew - Remove debug console.log statements added during investigation - Add tests for both extracted functions Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

…t-loop-exit # Conflicts: # packages/ui/src/components/session-turn.tsx

…lers Use Node's createServer for MCP OAuth callback and Codex plugin OAuth servers instead of Bun.serve, making them work under Node.js.

Use npm-install:${pkg} instead of a global npm-install lock so concurrent installs of different packages can run in parallel.

When the lockfile exists but .bin is empty or absent, add() would read the lockfile via loadVirtual() and return early without calling reify(). Delete the lockfile before calling add() so it proceeds with a full install.

Replace tsx dependency with explicit 'node' invocation for the compiled ESLint server. Remove all BUN_BE_BUN env var references which are no longer needed after the Bun-to-Node migration.

Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>

Npm.which now returns undefined instead of throwing when binary not found. Update all callers to check for undefined and return early. Avoid explicit union types by using a resolved variable pattern.

This restores the SIGHUP handler that was previously reverted. The handler ensures the process exits on terminal hangup (eg. closing the terminal tab), preventing long-running commands like `serve` from becoming orphaned processes.

- Replace manual subscription Map with PubSub.unbounded per instance - Per-type PubSubs + wildcard PubSub, cleaned up via addFinalizer - InstanceDisposed published before PubSub shutdown so subscribers see it - Add makeRuntime to run-service.ts (single runtime with runPromise + runFork) - Legacy facade preserved: publish/subscribe/subscribeAll same signatures - subscribe/subscribeAll fork stream consumer fibers, return interrupt function - Extract Format.file() for explicit formatting, remove event-driven subscription - Inline Format.file() calls in write/edit/apply_patch tools - Drop Bus.once (zero callers) - Keep makeRunPromise as deprecated wrapper for existing services

- Replace manual subscription Map with PubSub.unbounded per instance - Per-type PubSubs + wildcard PubSub, cleaned up via addFinalizer - InstanceDisposed published before PubSub shutdown so subscribers see it - Replace makeRunPromise with makeRuntime (single runtime with runPromise + runFork) - Update all 19 services to use makeRuntime destructuring - Legacy facade preserved: publish/subscribe/subscribeAll same signatures - subscribe/subscribeAll fork stream consumer fibers, return interrupt function - Extract Format.file() for explicit formatting, remove event-driven subscription - Inline Format.file() calls in write/edit/apply_patch tools - Drop Bus.once (zero callers)

… docs

- Add runCallback to makeRuntime (returns interrupt function directly) - Simplify forkStream to use runCallback instead of runFork + Fiber.interrupt - Add 6 bus tests covering publish/subscribe, unsubscribe, subscribeAll, and InstanceDisposed delivery on instance disposal

- Plugin: use bus.subscribeAll() Stream + forkScoped instead of callback facade - Vcs: use bus.subscribe() Stream + forkScoped instead of acquireRelease callback - Status: use bus.publish() Effect directly instead of Effect.promise wrapper - Rewrite Vcs test to use facade functions (shared memoMap ensures singleton Bus) - Add Bus integration tests for subscribe cleanup and instance disposal

MakonnenMak and others added 30 commits February 20, 2026 13:20

fix: remove as any type cast in processor exit logic

fc258ea

Merge remote-tracking branch 'upstream/dev' into fix/clock-skew-promp…

e993ace

…t-loop-exit # Conflicts: # packages/ui/src/components/session-turn.tsx

Merge remote-tracking branch 'upstream/dev' into fix/clock-skew-promp…

1392d86

…t-loop-exit # Conflicts: # packages/ui/src/components/session-turn.tsx

fix(opencode): image paste on Windows Terminal 1.25+ with kitty keyboard

342719d

fix(opencode): gate kitty keyboard events to windows

01c0902

refactor: replace BunProc with Npm module using @npmcli/arborist

ca2099e

refactor: replace Bun.serve with Node http.createServer in OAuth hand…

816a5f7

…lers Use Node's createServer for MCP OAuth callback and Codex plugin OAuth servers instead of Bun.serve, making them work under Node.js.

fix: scope Npm.add() lock to per-package key

34c676b

Use npm-install:${pkg} instead of a global npm-install lock so concurrent installs of different packages can run in parallel.

fix: force reinstall in which() when .bin is missing

c075a5d

When the lockfile exists but .bin is empty or absent, add() would read the lockfile via loadVirtual() and return early without calling reify(). Delete the lockfile before calling add() so it proceeds with a full install.

fix: remove BUN_BE_BUN workaround, use node for ESLint server

7409ce8

Replace tsx dependency with explicit 'node' invocation for the compiled ESLint server. Remove all BUN_BE_BUN env var references which are no longer needed after the Bun-to-Node migration.

fix: type error in mcp env and add missing @npmcli/arborist dependency

205affa

Apply suggestion from @greptile-apps[bot]

20249bb

Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>

Update packages/opencode/src/npm/index.ts

e48da96

Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>

fix type

3b669e7

sync

bc43bf3

Merge branch 'dev' into refactor/oauth-node-http

9f5713e

Merge branch 'dev' into refactor/npm-over-bunproc

b1d537b

fix: handle undefined from Npm.which in all callers

df44b41

Npm.which now returns undefined instead of throwing when binary not found. Update all callers to check for undefined and return early. Avoid explicit union types by using a resolved variable pattern.

fix(core): move SIGHUP handler into exit context for proper cleanup

9f1f625

simplify: extract forkStream, fix dead event test subscribers, update…

83916ca

… docs

test(bus): add instance isolation and multiple subscriber tests

0f5bd32

fix(bus): prevent race in getOrCreate with double-check after yield

8c559c2

remove unnecessary double-check in getOrCreate (Effect.sync is atomic)

fcec621

cleanup: remove unused memoMap import, fix extra blank line

ea31863

opencode-agent bot force-pushed the beta branch 16 times, most recently from 66c43da to dd0ae7f Compare March 26, 2026 01:02

github-actions bot mentioned this pull request Mar 26, 2026

📊 AI CLI 工具社区动态日报 2026-03-26 gsscsd/big_model_radar#96

Open

opencode-agent bot force-pushed the beta branch 6 times, most recently from 7a46b9a to c551d41 Compare March 26, 2026 06:55

This was referenced Mar 27, 2026

📊 AI CLI 工具社区动态日报 2026-03-27 gsscsd/big_model_radar#101

Open

📊 AI CLI 工具社区动态日报 2026-03-28 gsscsd/big_model_radar#107

Open

📊 AI CLI 工具社区动态日报 2026-03-29 gsscsd/big_model_radar#113

Open

JackLuguibin mentioned this pull request Apr 5, 2026

📊 AI CLI 工具社区动态日报 2026-04-05 JackLuguibin/big_model_radar#1

Open

This was referenced Apr 6, 2026

📊 AI CLI 工具社区动态日报 2026-04-06 gsscsd/big_model_radar#141

Open

📊 AI CLI 工具社区动态日报 2026-04-07 gsscsd/big_model_radar#147

Open

📊 AI CLI 工具社区动态日报 2026-04-08 gsscsd/big_model_radar#152

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(app): Mobile Touch Optimization#18767

feat(app): Mobile Touch Optimization#18767
noahbentusi wants to merge 80 commits intoanomalyco:betafrom
noahbentusi:yjh/dev

noahbentusi commented Mar 23, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Conversation

noahbentusi commented Mar 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Issue for this PR

Type of change

What does this PR do?

How did you verify your code works?

Screenshots / recordings

Checklist

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

noahbentusi commented Mar 23, 2026 •

edited

Loading