refactor: enhance command context handling with shellcheck directives

- Added shellcheck disable directives in various command scripts to suppress specific warnings related to variable usage and sourcing.
- Improved clarity and maintainability of command context management by ensuring proper variable handling across multiple command files.
- Streamlined the sourcing of command handlers in `bin/gtr` for better organization and execution flow.

Author: helizaga

bin/gtr

@@ -32,6 +32,7 @@ resolve_script_dir() {
 
 # Source command handlers
 for _cmd_file in "$GTR_DIR"/lib/commands/*.sh; do
+  # shellcheck disable=SC1090
   . "$_cmd_file"
 done
 unset _cmd_file

lib/adapters.sh

@@ -133,6 +133,7 @@ editor_open() {
 
   # Use workspace file if provided and exists
   if [ -n "$workspace" ] && [ -f "$workspace" ]; then
+    # shellcheck disable=SC2034
     target="$workspace"
   fi
 

lib/commands/ai.sh

@@ -52,11 +52,13 @@ cmd_ai() {
   load_ai_adapter "$ai_tool" || exit 1
 
   resolve_repo_context || exit 1
+  # shellcheck disable=SC2154
   local repo_root="$_ctx_repo_root" base_dir="$_ctx_base_dir" prefix="$_ctx_prefix"
 
   # Resolve target branch
   local worktree_path branch
   resolve_worktree "$identifier" "$repo_root" "$base_dir" "$prefix" || exit 1
+  # shellcheck disable=SC2154
   worktree_path="$_ctx_worktree_path" branch="$_ctx_branch"
 
   log_step "Starting $ai_tool for: $branch"

lib/commands/clean.sh

@@ -149,6 +149,7 @@ cmd_clean() {
   fi
 
   resolve_repo_context || exit 1
+  # shellcheck disable=SC2154
   local repo_root="$_ctx_repo_root" base_dir="$_ctx_base_dir" prefix="$_ctx_prefix"
 
   if [ ! -d "$base_dir" ]; then

lib/commands/copy.sh

@@ -55,11 +55,13 @@ cmd_copy() {
 
   # Get repo context
   resolve_repo_context || exit 1
+  # shellcheck disable=SC2154
   local repo_root="$_ctx_repo_root" base_dir="$_ctx_base_dir" prefix="$_ctx_prefix"
 
   # Resolve source path
   local src_path
   resolve_worktree "$source" "$repo_root" "$base_dir" "$prefix" || exit 1
+  # shellcheck disable=SC2154
   src_path="$_ctx_worktree_path"
 
   # Get patterns (flag > config)
@@ -101,6 +103,7 @@ cmd_copy() {
   for target_id in $targets; do
     local dst_path dst_branch
     resolve_worktree "$target_id" "$repo_root" "$base_dir" "$prefix" || continue
+    # shellcheck disable=SC2154
     dst_path="$_ctx_worktree_path" dst_branch="$_ctx_branch"
 
     # Skip if source == destination

lib/commands/create.sh

@@ -51,6 +51,7 @@ _post_create_next_steps() {
     local resolve_result
     if resolve_result=$(resolve_target "$folder_name" "$repo_root" "$base_dir" "$prefix" 2>/dev/null); then
       unpack_target "$resolve_result"
+      # shellcheck disable=SC2154
       if [ "$_ctx_is_main" = "1" ]; then
         # Collision: folder name matches current branch, use branch name instead
         next_steps_id="$branch_name"
@@ -242,6 +243,7 @@ cmd_create() {
 
   # Get repo info
   resolve_repo_context || exit 1
+  # shellcheck disable=SC2154
   local repo_root="$_ctx_repo_root" base_dir="$_ctx_base_dir" prefix="$_ctx_prefix"
 
   # Get branch name if not provided

lib/commands/editor.sh

@@ -36,12 +36,14 @@ cmd_editor() {
   fi
 
   resolve_repo_context || exit 1
+  # shellcheck disable=SC2154
   local repo_root="$_ctx_repo_root" base_dir="$_ctx_base_dir" prefix="$_ctx_prefix"
 
   # Resolve target branch
-  local worktree_path branch
+  local worktree_path
   resolve_worktree "$identifier" "$repo_root" "$base_dir" "$prefix" || exit 1
-  worktree_path="$_ctx_worktree_path" branch="$_ctx_branch"
+  # shellcheck disable=SC2154
+  worktree_path="$_ctx_worktree_path"
 
   if [ "$editor" = "none" ]; then
     open_in_gui "$worktree_path"

lib/commands/go.sh

@@ -9,11 +9,13 @@ cmd_go() {
 
   local identifier="$1"
   resolve_repo_context || exit 1
+  # shellcheck disable=SC2154
   local repo_root="$_ctx_repo_root" base_dir="$_ctx_base_dir" prefix="$_ctx_prefix"
 
   # Resolve target branch
   local is_main worktree_path branch
   resolve_worktree "$identifier" "$repo_root" "$base_dir" "$prefix" || exit 1
+  # shellcheck disable=SC2154
   is_main="$_ctx_is_main" worktree_path="$_ctx_worktree_path" branch="$_ctx_branch"
 
   # Human messages to stderr so stdout can be used in command substitution

lib/commands/remove.sh

@@ -39,12 +39,14 @@ cmd_remove() {
   fi
 
   resolve_repo_context || exit 1
+  # shellcheck disable=SC2154
   local repo_root="$_ctx_repo_root" base_dir="$_ctx_base_dir" prefix="$_ctx_prefix"
 
   for identifier in $identifiers; do
     # Resolve target branch
     local is_main worktree_path branch_name
     resolve_worktree "$identifier" "$repo_root" "$base_dir" "$prefix" || continue
+    # shellcheck disable=SC2154
     is_main="$_ctx_is_main" worktree_path="$_ctx_worktree_path" branch_name="$_ctx_branch"
 
     # Cannot remove main repository

lib/commands/rename.sh

@@ -40,11 +40,13 @@ cmd_rename() {
   fi
 
   resolve_repo_context || exit 1
+  # shellcheck disable=SC2154
   local repo_root="$_ctx_repo_root" base_dir="$_ctx_base_dir" prefix="$_ctx_prefix"
 
   # Resolve old worktree
   local is_main old_path old_branch
   resolve_worktree "$old_identifier" "$repo_root" "$base_dir" "$prefix" || exit 1
+  # shellcheck disable=SC2154
   is_main="$_ctx_is_main" old_path="$_ctx_worktree_path" old_branch="$_ctx_branch"
 
   # Cannot rename main repository