refactor: remove unused functions and improve command clarity

- Removed the unused helper function `__fish_git_gtr_config_is_write` from `completions/git-gtr.fish` and `scripts/generate-completions.sh` to streamline the code.
- Deleted the `copy_file` function from `lib/copy.sh` as it was redundant and not utilized in the current codebase.
- Eliminated the `list_worktrees` function from `lib/core.sh` to enhance clarity and maintainability of the code.

Author: helizaga

completions/git-gtr.fish

@@ -117,17 +117,6 @@ function __fish_git_gtr_config_is_read
   return 1
 end
 
-# Helper to check if config action is a write operation (set, add, unset)
-function __fish_git_gtr_config_is_write
-  set -l cmd (commandline -opc)
-  for i in $cmd
-    if test "$i" = "set" -o "$i" = "add" -o "$i" = "unset"
-      return 0
-    end
-  end
-  return 1
-end
-
 # Scope flags for config command
 # --local and --global available for all operations
 complete -f -c git -n '__fish_git_gtr_using_command config' -l local -d 'Use local git config'

lib/commands/ai.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
-# shellcheck disable=SC2154
 
 # AI command
+# shellcheck disable=SC2154  # _arg_* _pa_* set by parse_args, _ctx_* set by resolve_*
 cmd_ai() {
   parse_args "--ai: value" "$@"
 

lib/commands/clean.sh

@@ -1,5 +1,4 @@
 #!/usr/bin/env bash
-# shellcheck disable=SC2154
 
 # Clean command (remove prunable worktrees)
 # Remove worktrees whose PRs/MRs are merged (handles squash merges)
@@ -83,17 +82,28 @@ _clean_merged() {
         removed=$((removed + 1))
       elif [ "$yes_mode" -eq 1 ] || prompt_yes_no "Remove worktree and delete branch '$branch'?"; then
         log_step "Removing worktree: $branch"
-        local remove_output
-        if remove_output=$(git worktree remove "$dir" 2>&1); then
-          # Also delete the local branch
+
+        # Run pre-remove hooks (skip worktree on failure)
+        if ! run_hooks_in preRemove "$dir" \
+          REPO_ROOT="$repo_root" \
+          WORKTREE_PATH="$dir" \
+          BRANCH="$branch"; then
+          log_warn "Pre-remove hook failed for $branch, skipping"
+          skipped=$((skipped + 1))
+          continue
+        fi
+
+        if remove_worktree "$dir" 0; then
+          # Delete the local branch (safe: already merged)
           git branch -d "$branch" 2>/dev/null || git branch -D "$branch" 2>/dev/null || true
-          log_info "Removed: $branch"
           removed=$((removed + 1))
-        else
-          if [ -n "$remove_output" ]; then
-            log_error "Failed to remove worktree: $remove_output"
-          else
-            log_error "Failed to remove worktree: $branch"
+
+          # Run post-remove hooks (worktree already removed, don't abort)
+          if ! run_hooks postRemove \
+            REPO_ROOT="$repo_root" \
+            WORKTREE_PATH="$dir" \
+            BRANCH="$branch"; then
+            log_warn "Post-remove hook failed for $branch"
           fi
         fi
       else
@@ -112,6 +122,7 @@ _clean_merged() {
   fi
 }
 
+# shellcheck disable=SC2154  # _arg_* set by parse_args, _ctx_* set by resolve_*
 cmd_clean() {
   local _spec
   _spec="--merged

lib/commands/copy.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
-# shellcheck disable=SC2154
 
 # Copy command (copy files between worktrees)
+# shellcheck disable=SC2154  # _arg_* _pa_* set by parse_args, _ctx_* set by resolve_*/merge_copy_patterns
 cmd_copy() {
   local _spec
   _spec="--from: value

lib/commands/create.sh

@@ -1,9 +1,9 @@
 #!/usr/bin/env bash
-# shellcheck disable=SC2154
 
 # Create command
 # Copy files and directories to newly created worktree
 # Usage: _post_create_copy repo_root worktree_path
+# shellcheck disable=SC2154  # _ctx_copy_* set by merge_copy_patterns
 _post_create_copy() {
   local repo_root="$1"
   local worktree_path="$2"
@@ -30,6 +30,7 @@ _post_create_copy() {
 
 # Show next steps after worktree creation (resolves collision for --folder overrides)
 # Usage: _post_create_next_steps branch_name folder_name folder_override repo_root base_dir prefix
+# shellcheck disable=SC2154  # _ctx_is_main set by resolve_target/unpack_target
 _post_create_next_steps() {
   local branch_name="$1" folder_name="$2" folder_override="$3"
   local repo_root="$4" base_dir="$5" prefix="$6"
@@ -83,6 +84,7 @@ _create_resolve_from_ref() {
 
   printf "%s" "$from_ref"
 }
+# shellcheck disable=SC2154  # _arg_* _pa_* set by parse_args, _ctx_* set by resolve_*
 cmd_create() {
   local _spec
   _spec="--from: value

lib/commands/editor.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
-# shellcheck disable=SC2154
 
 # Editor command
+# shellcheck disable=SC2154  # _arg_* _pa_* set by parse_args, _ctx_* set by resolve_*
 cmd_editor() {
   parse_args "--editor: value" "$@"
 

lib/commands/go.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
-# shellcheck disable=SC2154
 
 # Go command (navigate to worktree - prints path for shell integration)
+# shellcheck disable=SC2154  # _pa_* set by parse_args, _ctx_* set by resolve_*
 cmd_go() {
   parse_args "" "$@"
 

lib/commands/list.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
-# shellcheck disable=SC2154
 
 # List command
+# shellcheck disable=SC2154  # _arg_* set by parse_args, _ctx_* set by resolve_*
 cmd_list() {
   parse_args "--porcelain" "$@"
 

lib/commands/remove.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
-# shellcheck disable=SC2154
 
 # Remove command
+# shellcheck disable=SC2154  # _arg_* _pa_* set by parse_args, _ctx_* set by resolve_*
 cmd_remove() {
   local _spec
   _spec="--delete-branch

lib/commands/rename.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
-# shellcheck disable=SC2154
 
 # Rename command (rename worktree and branch)
+# shellcheck disable=SC2154  # _arg_* _pa_* set by parse_args, _ctx_* set by resolve_*
 cmd_rename() {
   local _spec
   _spec="--force