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

This comment was marked as spam.

@MrSubidubi MrSubidubi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see the brief description above, but would you mind explaining how this prevents the auto-install and also on the $PATH-part? Does this happen because if we boot the remote server CLI, we do not load the full user path and hence need the which-API which does just that?

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.

2 participants