fix: correct variable quoting in rm command to match tests

Fixed failing unit tests for variable quoting:

1. Test 9 - Variables are quoted in critical sections:
   - Changed `rm -rfv "${dir}"` to `rm -rfv "$dir"`
   - Test expects `"$dir"` format (quotes without braces)
   - Both forms are secure but test was written for standard format

2. Test 26 - Removes cloned repository after installation:
   - Updated test to search for quoted variable: `rm -rfv "\$dir"`
   - Previously searched for unquoted variable which contradicts security
   - Now both tests verify proper quoting consistently

Changes ensure:
- Variables are properly quoted (security requirement)
- Tests verify the correct quoting format
- No word splitting or injection vulnerabilities

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: 4ndymcfly

setup.sh

@@ -390,7 +390,7 @@ sleep 2
 
 {
 	rm -rfv ~/tools
-	rm -rfv "${dir}"
+	rm -rfv "$dir"
 } > /dev/null 2>&1
 
 echo -e "\n${GREEN}[+] Done\n${NOCOLOR}"

tests/test_setup.bats

@@ -183,7 +183,7 @@ teardown() {
 }
 
 @test "Removes cloned repository after installation" {
-    grep -q 'rm -rfv \$dir' "$SETUP_FILE"
+    grep -q 'rm -rfv "\$dir"' "$SETUP_FILE"
 }
 
 #==============================================================================