fix: improve directory handling and test execution in clean command

- Updated the `clean.sh` script to enhance the logic for finding empty directories by using `grep -Fxv` for better accuracy.
- Modified tests in `cmd_clean.bats` and `hooks.bats` to use `run` for executing commands, improving consistency in test status checks and output handling.
- Ensured that hooks in `hooks.bats` reference the correct variable for repository paths, enhancing portability and reliability of hook execution.

Author: helizaga

lib/commands/clean.sh

@@ -159,7 +159,7 @@ cmd_clean() {
   # Find and remove empty directories
   local cleaned=0
   local empty_dirs
-  empty_dirs=$(find "$base_dir" -maxdepth 1 -type d -empty 2>/dev/null | grep -v "^${base_dir}$" || true)
+  empty_dirs=$(find "$base_dir" -maxdepth 1 -type d -empty 2>/dev/null | grep -Fxv "$base_dir" || true)
 
   if [ -n "$empty_dirs" ]; then
     while IFS= read -r dir; do

tests/cmd_clean.bats

@@ -47,28 +47,28 @@ teardown() {
 # ── _clean_should_skip ──────────────────────────────────────────────────────
 
 @test "_clean_should_skip skips detached HEAD" {
-  _clean_should_skip "/some/dir" "(detached)"
-  # Returns 0 = should skip
+  run _clean_should_skip "/some/dir" "(detached)"
+  [ "$status" -eq 0 ]
 }
 
 @test "_clean_should_skip skips empty branch" {
-  _clean_should_skip "/some/dir" ""
-  # Returns 0 = should skip
+  run _clean_should_skip "/some/dir" ""
+  [ "$status" -eq 0 ]
 }
 
 @test "_clean_should_skip skips dirty worktree" {
   create_test_worktree "dirty-test"
   echo "dirty" > "$TEST_WORKTREES_DIR/dirty-test/untracked.txt"
   git -C "$TEST_WORKTREES_DIR/dirty-test" add untracked.txt
-  _clean_should_skip "$TEST_WORKTREES_DIR/dirty-test" "dirty-test"
-  # Returns 0 = should skip (staged changes)
+  run _clean_should_skip "$TEST_WORKTREES_DIR/dirty-test" "dirty-test"
+  [ "$status" -eq 0 ]
 }
 
 @test "_clean_should_skip skips worktree with untracked files" {
   create_test_worktree "untracked-test"
   echo "new" > "$TEST_WORKTREES_DIR/untracked-test/newfile.txt"
-  _clean_should_skip "$TEST_WORKTREES_DIR/untracked-test" "untracked-test"
-  # Returns 0 = should skip
+  run _clean_should_skip "$TEST_WORKTREES_DIR/untracked-test" "untracked-test"
+  [ "$status" -eq 0 ]
 }
 
 @test "_clean_should_skip does not skip clean worktree" {

tests/hooks.bats

@@ -18,7 +18,7 @@ teardown() {
 }
 
 @test "run_hooks executes single hook" {
-  git config --add gtr.hook.postCreate "touch $TEST_REPO/hook-ran"
+  git config --add gtr.hook.postCreate 'touch "$REPO_ROOT/hook-ran"'
   run_hooks postCreate REPO_ROOT="$TEST_REPO"
   [ -f "$TEST_REPO/hook-ran" ]
 }
@@ -36,8 +36,8 @@ teardown() {
 }
 
 @test "run_hooks executes multiple hooks in order" {
-  git config --add gtr.hook.postCreate "echo first >> $TEST_REPO/order"
-  git config --add gtr.hook.postCreate "echo second >> $TEST_REPO/order"
+  git config --add gtr.hook.postCreate 'echo first >> "$REPO_ROOT/order"'
+  git config --add gtr.hook.postCreate 'echo second >> "$REPO_ROOT/order"'
   run_hooks postCreate REPO_ROOT="$TEST_REPO"
   [ "$(head -1 "$TEST_REPO/order")" = "first" ]
   [ "$(tail -1 "$TEST_REPO/order")" = "second" ]