fix: disable worktree mode when repo has no local path#124
Merged
Conversation
Worktree mode requires a parent repo clone to create linked worktrees. Previously only "Existing repo" mode was disabled when no local path was configured, letting the user select "Git worktree" which would then fail at launch time. Now both "Existing repo" and "Git worktree" are disabled in the UI when no local path is set, with a hint directing users to Settings. The error message in executeLaunch is also improved to name the specific mode that failed. Closes #92
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
executeLaunchto name the specific mode that failed and suggest "Fresh clone" as an alternativeaddRepoaccepts optionallocalPath, the UI labels it optional,ClonePromptModalhandles clone-on-demand at launchTest plan
pnpm turbo typecheckpassespnpm turbo test— 407 tests pass