enhance: makes Create New Branch option as the default selected mode for adding new worktree

Author: love-linger

src/Commands/Worktree.cs

@@ -62,15 +62,20 @@ public Worktree(string repo)
             return worktrees;
         }
 
-        public bool Add(string fullpath, string name, string tracking, Action<string> outputHandler)
+        public bool Add(string fullpath, string name, bool createNew, string tracking, Action<string> outputHandler)
         {
             Args = "worktree add ";
 
             if (!string.IsNullOrEmpty(tracking))
                 Args += "--track ";
 
             if (!string.IsNullOrEmpty(name))
-                Args += $"-B {name} ";
+            {
+                if (createNew)
+                    Args += $"-b {name} ";
+                else
+                    Args += $"-B {name} ";
+            }
 
             Args += $"\"{fullpath}\" ";
 

src/ViewModels/AddWorktree.cs

@@ -15,17 +15,17 @@ public string Path
             set => SetProperty(ref _path, value);
         }
 
-        public bool UseExistingBranch
+        public bool CreateNewBranch
         {
-            get => _useExistingBranch;
+            get => _createNewBranch;
             set
             {
-                if (SetProperty(ref _useExistingBranch, value, true))
+                if (SetProperty(ref _createNewBranch, value, true))
                 {
                     if (value)
-                        SelectedBranch = LocalBranches.Count > 0 ? LocalBranches[0] : string.Empty;
-                    else
                         SelectedBranch = string.Empty;
+                    else
+                        SelectedBranch = LocalBranches.Count > 0 ? LocalBranches[0] : string.Empty;
                 }
             }
         }
@@ -74,11 +74,6 @@ public AddWorktree(Repository repo)
                     RemoteBranches.Add($"{branch.Remote}/{branch.Name}");
             }
 
-            if (LocalBranches.Count > 0)
-                SelectedBranch = LocalBranches[0];
-            else
-                SelectedBranch = string.Empty;
-
             if (RemoteBranches.Count > 0)
                 SelectedTrackingBranch = RemoteBranches[0];
             else
@@ -114,19 +109,20 @@ public override Task<bool> Sure()
             _repo.SetWatcherEnabled(false);
             ProgressDescription = "Adding worktree ...";
 
+            var branchName = _selectedBranch;
             var tracking = _setTrackingBranch ? SelectedTrackingBranch : string.Empty;
 
             return Task.Run(() =>
             {
-                var succ = new Commands.Worktree(_repo.FullPath).Add(_path, _selectedBranch, tracking, SetProgressDescription);
+                var succ = new Commands.Worktree(_repo.FullPath).Add(_path, branchName, _createNewBranch, tracking, SetProgressDescription);
                 CallUIThread(() => _repo.SetWatcherEnabled(true));
                 return succ;
             });
         }
 
         private Repository _repo = null;
         private string _path = string.Empty;
-        private bool _useExistingBranch = true;
+        private bool _createNewBranch = true;
         private string _selectedBranch = string.Empty;
         private bool _setTrackingBranch = false;
     }

src/Views/AddWorktree.axaml

@@ -36,25 +36,32 @@
                  Margin="0,0,8,0"
                  Text="{DynamicResource Text.AddWorktree.WhatToCheckout}"/>
       <StackPanel Grid.Row="1" Grid.Column="1" Height="32" Orientation="Horizontal">
-        <RadioButton Content="{DynamicResource Text.AddWorktree.WhatToCheckout.Existing}"
-                     GroupName="LocalChanges"
-                     IsChecked="{Binding UseExistingBranch, Mode=TwoWay}" />
         <RadioButton Content="{DynamicResource Text.AddWorktree.WhatToCheckout.CreateNew}"
-                     GroupName="LocalChanges"
-                     Margin="8,0,0,0" />
+                     GroupName="WhatToCheckout"
+                     IsChecked="{Binding CreateNewBranch, Mode=TwoWay}"/>
+        <RadioButton Content="{DynamicResource Text.AddWorktree.WhatToCheckout.Existing}"
+                     GroupName="WhatToCheckout"
+                     Margin="8,0,0,0"/>
       </StackPanel>
 
       <TextBlock Grid.Row="2" Grid.Column="0"
                  HorizontalAlignment="Right" VerticalAlignment="Center"
                  Margin="0,0,8,0"
                  Text="{DynamicResource Text.AddWorktree.Name}"/>
+      <TextBox Grid.Row="2" Grid.Column="1"
+               Height="28"
+               CornerRadius="3"
+               Text="{Binding SelectedBranch, Mode=TwoWay}"
+               Watermark="{DynamicResource Text.AddWorktree.Name.Placeholder}"
+               IsEnabled="{Binding CreateNewBranch, Mode=OneWay}"
+               IsVisible="{Binding CreateNewBranch, Mode=OneWay}"/>
       <ComboBox Grid.Row="2" Grid.Column="1"
                 Height="28" Padding="8,0"
                 VerticalAlignment="Center" HorizontalAlignment="Stretch"
                 ItemsSource="{Binding LocalBranches}"
                 SelectedItem="{Binding SelectedBranch, Mode=TwoWay}"
-                IsEnabled="{Binding UseExistingBranch, Mode=OneWay}"
-                IsVisible="{Binding UseExistingBranch, Mode=OneWay}">
+                IsEnabled="{Binding !CreateNewBranch, Mode=OneWay}"
+                IsVisible="{Binding !CreateNewBranch, Mode=OneWay}">
         <ComboBox.ItemTemplate>
           <DataTemplate>
             <StackPanel Orientation="Horizontal" Height="20" VerticalAlignment="Center">
@@ -63,14 +70,7 @@
             </StackPanel>
           </DataTemplate>
         </ComboBox.ItemTemplate>
-      </ComboBox>
-      <TextBox Grid.Row="2" Grid.Column="1"
-               Height="28"
-               CornerRadius="3"
-               Text="{Binding SelectedBranch, Mode=TwoWay}"
-               Watermark="{DynamicResource Text.AddWorktree.Name.Placeholder}"
-               IsEnabled="{Binding !UseExistingBranch, Mode=OneWay}"
-               IsVisible="{Binding !UseExistingBranch, Mode=OneWay}"/>  
+      </ComboBox> 
 
       <Border Grid.Row="3" Grid.Column="0" 
               Height="32"