Skip to content

Fix DAP binary resolution for remote worktrees#94

Open
TateWalker wants to merge 2 commits into
zed-extensions:mainfrom
TateWalker:main
Open

Fix DAP binary resolution for remote worktrees#94
TateWalker wants to merge 2 commits into
zed-extensions:mainfrom
TateWalker:main

Conversation

@TateWalker

@TateWalker TateWalker commented Jun 12, 2026

Copy link
Copy Markdown

Problem

get_dap_binary passed raw binary names ("flutter", "dart", "fvm")
as commands without resolving them against the host's PATH. WASM extensions
cannot read $PATH directly, so the debug adapter failed to launch on remote
(SSH) worktrees.

Fix

Resolve binary paths using worktree.which(), which works correctly on both
local and remote worktrees. This is consistent with how language_server_binary
already resolves the dart binary.

Related Issues

@cla-bot

cla-bot Bot commented Jun 12, 2026

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement, and we don't have @TateWalker on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@TateWalker

Copy link
Copy Markdown
Author

@cla-bot check

@cla-bot cla-bot Bot added the cla-signed label Jun 12, 2026
@cla-bot

cla-bot Bot commented Jun 12, 2026

Copy link
Copy Markdown

The cla-bot has been summoned, and re-checked this pull request!

@TateWalker

Copy link
Copy Markdown
Author

@MrSubidubi checking to see if anything on this needs changing

@TateWalker

Copy link
Copy Markdown
Author

@maxdeviant could I get some eyes on this? Since it hasn't been merged yet, Zed auto-installs the official version every time I open it up, slowing down my development workflow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant