sergeant 1.0.0 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 80f83fe4916418417a546134baeff0c43e0daf5e383b39cdd49c77bdea88e3a3
-  data.tar.gz: 6105effe9798842cf02bdb5b56e32b1b58e5838da3c66004b24da31365982758
+  metadata.gz: 6f8e44aa17efd4cddd56d88d910edc0082e94488cd3d25cf6def3b832a579c3d
+  data.tar.gz: c66899a4748192ded5481c0497a81ebb81edad956a932d6253beab34ab834a67
 SHA512:
-  metadata.gz: a90c6c72ab39338aa0d27a603051a2fea99b97b9f9c0764f2cc3f59bde980c3310b0aa97015028543453e03b11d4fff62f162fc2ae716fa991a91090628bb680
-  data.tar.gz: 845f119f78cfb9d08d4afa1abd6816195b630a16e2276cc59a7caab318a34e7fa15cb1984e6aa9a82c5f43e758894238740ba99a8490a8d676c9577b0560e78f
+  metadata.gz: 1590f36daa589471edc9f65f150b24b9764ef192b77d46cc51ffc1884e01dafa2fd9a4832a5fef635dfef8bca12f3d16f49b305a295a4fd8e6159e50529d9127
+  data.tar.gz: 8ee3a3f6eacd45f3a91c42b7a96e94ccc0961c12cbd3b363ee8a92553efe4db4db73be8499d5304d54702b82333c50fa29b27ad0f87a6f2e141a0a9ca4dad3f2

data/CHANGELOG.md

@@ -5,6 +5,17 @@ All notable changes to Sergeant will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.0.1] - 2024-12-24
+
+### Fixed
+- **Major performance improvement**: Fixed severe input lag with large directories
+  - Directory contents now only refresh when necessary (directory changes, file operations)
+  - Previously refreshed on every keystroke, causing thousands of unnecessary file system calls
+  - Navigation (arrow keys, marking) is now instant regardless of directory size
+- **Better error handling**: Added error messages to help diagnose installation issues
+  - Shows load path and helpful reinstall instructions if gem fails to load
+  - Displays full error details instead of silently failing
+
 ## [1.0.0] - 2024-12-23
 
 ### Added

data/README.md

@@ -90,6 +90,39 @@ gem install ./sergeant-1.0.0.gem
 
 That's it! The `sgt` command will automatically be added to your PATH.
 
+### Troubleshooting Installation
+
+**If sgt doesn't display anything (shows blank screen):**
+
+This can happen on Arch Linux or other systems using Ruby version managers (mise, rbenv, asdf).
+
+**Recommended fix - Add an alias (simplest):**
+```bash
+# Add to your ~/.bashrc or ~/.zshrc:
+echo 'alias sgt='"'"'ruby "$(which sgt)"'"'"'' >> ~/.bashrc
+
+# Reload your shell:
+source ~/.bashrc  # or: source ~/.zshrc
+
+# Now sgt works!
+sgt
+```
+
+**Alternative - Quick test:**
+```bash
+# Run with explicit ruby (temporary fix)
+ruby $(which sgt)
+```
+
+**Alternative - Automated fix script:**
+```bash
+# Creates a wrapper script (requires cloning repo)
+cd Sergeant
+bash arch_fix.sh
+```
+
+**For detailed troubleshooting**, see [INSTALL_TROUBLESHOOTING.md](./INSTALL_TROUBLESHOOTING.md)
+
 ### Development Installation
 
 If you want to work on the gem:

data/bin/sgt

@@ -4,10 +4,20 @@
 # Sergeant (sgt) - Interactive TUI directory navigator
 # Version: 1.0.0
 
-# Add lib directory to load path for development
-$LOAD_PATH.unshift(File.expand_path('../lib', __dir__))
-
-require 'sergeant'
+begin
+  require 'sergeant'
+rescue LoadError => e
+  warn "Error loading sergeant: #{e.message}"
+  warn "Load path: #{$LOAD_PATH.join(', ')}"
+  warn "\nTry reinstalling: gem uninstall sergeant && gem install sergeant"
+  exit 1
+end
 
 # Run the navigator
-SergeantApp.new.run
+begin
+  SergeantApp.new.run
+rescue StandardError => e
+  warn "\nSergeant error: #{e.message}"
+  warn e.backtrace.join("\n")
+  exit 1
+end

data/lib/sergeant/modals/file_operations.rb

@@ -49,11 +49,13 @@ module Sergeant
 
         # Restore curses screen
         init_screen
-        start_color
+        if has_colors?
+          start_color
+          apply_color_theme
+        end
         curs_set(0)
         noecho
         stdscr.keypad(true)
-        apply_color_theme
       end
 
       def preview_file
@@ -102,11 +104,13 @@ module Sergeant
 
         # Restore curses screen
         init_screen
-        start_color
+        if has_colors?
+          start_color
+          apply_color_theme
+        end
         curs_set(0)
         noecho
         stdscr.keypad(true)
-        apply_color_theme
       end
 
       def paste_with_modal
@@ -163,6 +167,9 @@ module Sergeant
         else
           show_info_modal("Successfully pasted #{success_count} item(s)")
         end
+
+        # Force refresh to show new files
+        force_refresh
       end
 
       def get_unique_filename(path)
@@ -208,6 +215,9 @@ module Sergeant
         else
           show_info_modal("Successfully deleted #{success_count} item(s)")
         end
+
+        # Force refresh to show changes
+        force_refresh
       end
 
       def rename_with_modal(item)
@@ -358,6 +368,9 @@ module Sergeant
               @copied_items.delete(old_path)
               @copied_items << new_path
             end
+
+            # Force refresh to show renamed item
+            force_refresh
           rescue StandardError => e
             show_error_modal("Error: #{e.message}")
           end
@@ -496,6 +509,9 @@ module Sergeant
               FileUtils.mkdir_p(new_path)
               show_info_modal('Directory created successfully!')
             end
+
+            # Force refresh to show new item
+            force_refresh
           rescue StandardError => e
             show_error_modal("Error: #{e.message}")
           end
@@ -624,11 +640,16 @@ module Sergeant
 
         # Restore curses
         init_screen
-        start_color
+        if has_colors?
+          start_color
+          apply_color_theme
+        end
         curs_set(0)
         noecho
         stdscr.keypad(true)
-        apply_color_theme
+
+        # Force refresh to show any changes from the command
+        force_refresh
       end
     end
   end

data/lib/sergeant/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Sergeant
-  VERSION = '1.0.0'
+  VERSION = '1.0.2'
 end

data/lib/sergeant.rb

@@ -31,20 +31,27 @@ class SergeantApp
     @marked_items = []
     @copied_items = []
     @cut_mode = false
+    @last_refreshed_dir = nil
+    @items = []
   end
 
   def run
     init_screen
-    start_color
+
+    # Only initialize colors if terminal supports them
+    if has_colors?
+      start_color
+      apply_color_theme
+    end
+
     curs_set(0)
     noecho
     stdscr.keypad(true)
 
-    apply_color_theme
-
     begin
       loop do
-        refresh_items
+        # Only refresh items when directory changes, not on every keystroke
+        refresh_items_if_needed
         draw_screen
 
         key = getch
@@ -165,11 +172,13 @@ class SergeantApp
     end
 
     init_screen
-    start_color
+    if has_colors?
+      start_color
+      apply_color_theme
+    end
     curs_set(0)
     noecho
     stdscr.keypad(true)
-    apply_color_theme
 
     return unless selected && !selected.empty?
 
@@ -182,6 +191,20 @@ class SergeantApp
     @scroll_offset = 0
   end
 
+  def refresh_items_if_needed
+    # Only refresh if directory has changed, or if showing ownership toggle changed
+    # This prevents expensive file system operations on every keystroke
+    if @current_dir != @last_refreshed_dir
+      refresh_items
+      @last_refreshed_dir = @current_dir
+    end
+  end
+
+  def force_refresh
+    # Force a refresh even if directory hasn't changed (e.g., after file operations)
+    @last_refreshed_dir = nil
+  end
+
   def refresh_items
     entries = Dir.entries(@current_dir).reject { |e| e == '.' }
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sergeant
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.2
 platform: ruby
 authors:
 - Mateusz Grotha