Clean up trailing spaces in JavaScript files and use Open3 for git operations

- Remove trailing spaces from all JavaScript files
- Replace Shellwords escaping with Open3 array form in sessions_controller.rb
- Update test files to use Open3.capture3 instead of system() calls
- Improve safety by using Open3's automatic escaping instead of manual Shellwords

Author: TravisTheTechie

app/controllers/sessions_controller.rb

@@ -243,11 +243,8 @@ def diff_file_contents
       ""
     end
 
-    # Escape file path for shell
-    escaped_path = Shellwords.escape(file_path)
-
     # Get git status for this specific file
-    git_status, _, _ = Open3.capture3("git status --porcelain #{escaped_path}", chdir: safe_dir)
+    git_status, _, _ = Open3.capture3("git", "status", "--porcelain", file_path, chdir: safe_dir)
     git_status = git_status.strip
 
     # Determine file status and get appropriate content
@@ -260,13 +257,13 @@ def diff_file_contents
     elsif git_status.match?(/^[AM]M/)
       # File has both staged and unstaged changes
       # Show HEAD vs working directory (to see all changes)
-      original_content, _, _ = Open3.capture3("git show HEAD:#{escaped_path}", chdir: safe_dir)
+      original_content, _, _ = Open3.capture3("git", "show", "HEAD:#{file_path}", chdir: safe_dir)
       # modified_content already has the working directory content
     elsif git_status.match?(/^[AM]\s/)
       # File is only staged (not modified in working directory)
       # This means working directory matches staged version
       # Show HEAD vs working directory (which equals staged)
-      original_content, _, status = Open3.capture3("git show HEAD:#{escaped_path}", chdir: safe_dir)
+      original_content, _, status = Open3.capture3("git", "show", "HEAD:#{file_path}", chdir: safe_dir)
       unless status.success?
         # New file, show empty vs working directory
         original_content = ""
@@ -275,11 +272,11 @@ def diff_file_contents
     elsif git_status.match?(/^\sM/)
       # File is only modified (not staged)
       # Show HEAD vs working directory
-      original_content, _, _ = Open3.capture3("git show HEAD:#{escaped_path}", chdir: safe_dir)
+      original_content, _, _ = Open3.capture3("git", "show", "HEAD:#{file_path}", chdir: safe_dir)
       # modified_content already has the working directory content
     else
       # Default: show HEAD vs working directory
-      original_content, _, _ = Open3.capture3("git show HEAD:#{escaped_path}", chdir: safe_dir)
+      original_content, _, _ = Open3.capture3("git", "show", "HEAD:#{file_path}", chdir: safe_dir)
       # modified_content already has the working directory content
     end
 

app/javascript/controllers/auto_refresh_controller.js

@@ -1,7 +1,7 @@
 import { Controller } from "@hotwired/stimulus"
 
 export default class extends Controller {
-  static values = { 
+  static values = {
     interval: { type: Number, default: 3000 }
   }
 
@@ -16,7 +16,7 @@ export default class extends Controller {
   checkAndRefresh() {
     // Check if there are any importing projects
     const hasImportingProjects = !!this.element.querySelector('[data-import-status="importing"]')
-    
+
     if (hasImportingProjects) {
       this.startRefreshing()
     } else {