hyperlist 1.1.4 → 1.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ceec87c634faff3f44aea9faab6b7e2c5fbb6299827371c148b01ac833a6653e
-  data.tar.gz: d2aee8287a2af42238f8d55ac47e5ac4f38738d55c4672f5f7835c561653246a
+  metadata.gz: c3394f0a19dc7d9641036a62d294be9dcb4d07627e8f28abbf52be6db409b0ce
+  data.tar.gz: 226af1d1c88dc25128f9fb881e6195bb57f4ee9ec8248320d9c12834b1275aff
 SHA512:
-  metadata.gz: a6bb97db751579efeb55724dd0280edf506a9c547c6809ae61e4fe2ed7ae8b505fb1fc85f138a6ae0c5aea47abc74ed391b6e35f347bbb0dc8e09c562eb2ea77
-  data.tar.gz: efdac57b3bff33a8cefa53f26782ff99dfee21536d54bd185497d89bd010669cc543b2e5c167bdb2d0fe250de73a82a4922dc6c927af33d6f07dbb74429cc2f4
+  metadata.gz: e636855dff0f0179dcb0f7eb56dc67c577f3775cd1fbf93d9c49b1812ce896e34160331146c4a6e8cea5e7fa92cafba197244210bb48b24f5abfa86e1e7c266b
+  data.tar.gz: b5793788afef9b63fe54471687ed3f03f6f100616f2aa89eb13a5fa5e36201cbfaabac24b7683f612084bfacf3ffb827bac1bf7e363958ece2de0f32fecedc7a

data/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 All notable changes to the HyperList Ruby TUI will be documented in this file.
 
+## [1.1.6] - 2025-08-14
+
+### Fixed
+- **Ruby 3.4+ Compatibility**
+  - Added explicit rcurses initialization for Ruby 3.4.0 and later
+  - Improved error handling during terminal initialization
+  - Added proper cleanup on exit for newer Ruby versions
+  - Fixed silent crashes on Arch Linux with Ruby 3.4.5
+
 ## [1.1.0] - 2025-08-13
 
 ### Added

data/diagnose_ruby34.rb

@@ -0,0 +1,145 @@
+#!/usr/bin/env ruby
+# Diagnostic script for Ruby 3.4.5 compatibility issues
+
+puts "Ruby Compatibility Diagnostic for HyperList"
+puts "=" * 50
+puts "Ruby version: #{RUBY_VERSION}"
+puts "Ruby platform: #{RUBY_PLATFORM}"
+puts
+
+# Test 1: Basic rcurses functionality
+puts "Test 1: Basic rcurses require and init..."
+begin
+  require 'rcurses'
+  puts "  ✓ rcurses loaded successfully"
+  puts "  Rcurses version: #{Rcurses::VERSION}" if defined?(Rcurses::VERSION)
+rescue LoadError => e
+  puts "  ✗ Failed to load rcurses: #{e.message}"
+  exit 1
+end
+
+# Test 2: Rcurses initialization with error capture
+puts "\nTest 2: Rcurses initialization..."
+begin
+  # Capture any output during init
+  original_stdout = $stdout
+  original_stderr = $stderr
+  
+  # Create StringIO to capture output
+  require 'stringio'
+  captured_out = StringIO.new
+  captured_err = StringIO.new
+  
+  $stdout = captured_out
+  $stderr = captured_err
+  
+  # Try to initialize
+  Rcurses.init!
+  
+  # Restore output
+  $stdout = original_stdout
+  $stderr = original_stderr
+  
+  # Check if anything was captured
+  out_content = captured_out.string
+  err_content = captured_err.string
+  
+  if !out_content.empty?
+    puts "  Captured stdout during init: #{out_content.inspect}"
+  end
+  if !err_content.empty?
+    puts "  Captured stderr during init: #{err_content.inspect}"
+  end
+  
+  puts "  ✓ Rcurses initialized"
+  
+  # Test basic functionality
+  puts "  Testing basic screen operations..."
+  print "\e[2J\e[H"  # Clear screen
+  print "Test"
+  sleep 0.1
+  print "\e[2J\e[H"  # Clear again
+  
+  puts "  ✓ Basic operations work"
+  
+rescue => e
+  puts "  ✗ Failed during initialization: #{e.message}"
+  puts "  Backtrace:"
+  e.backtrace.first(5).each { |line| puts "    #{line}" }
+ensure
+  # Try to restore terminal
+  begin
+    Rcurses.done! if defined?(Rcurses.done!)
+  rescue
+    # Fallback terminal restore
+    print "\e[?25h"  # Show cursor
+    system("stty sane 2>/dev/null")
+  end
+end
+
+# Test 3: Check for method availability issues in Ruby 3.4
+puts "\nTest 3: Ruby 3.4 specific checks..."
+
+# Check IO methods that might have changed
+if IO.respond_to?(:console)
+  puts "  ✓ IO.console available"
+else
+  puts "  ✗ IO.console not available"
+end
+
+begin
+  require 'io/console'
+  $stdin.raw { }
+  puts "  ✓ IO#raw method works"
+rescue => e
+  puts "  ✗ IO#raw failed: #{e.message}"
+end
+
+# Test 4: Check encoding
+puts "\nTest 4: Encoding checks..."
+puts "  Default external: #{Encoding.default_external}"
+puts "  Default internal: #{Encoding.default_internal}"
+puts "  Console encoding: #{$stdout.external_encoding}" if $stdout.respond_to?(:external_encoding)
+
+# Test 5: Simple hyperlist initialization test
+puts "\nTest 5: HyperList class initialization..."
+begin
+  # Load just the class definition part
+  hyperlist_code = File.read(File.join(File.dirname(__FILE__), 'hyperlist'))
+  
+  # Extract just the class without running main
+  class_only = hyperlist_code.split(/^if __FILE__ == \$0/)[0]
+  
+  # Try to evaluate it
+  eval(class_only)
+  
+  puts "  ✓ HyperList class loaded"
+  
+  # Try to create instance (without running)
+  app = HyperListApp.new
+  puts "  ✓ HyperListApp instance created"
+  
+rescue => e
+  puts "  ✗ Failed to load HyperList: #{e.message}"
+  puts "  Error at: #{e.backtrace.first}"
+end
+
+# Test 6: Terminal capability check
+puts "\nTest 6: Terminal capabilities..."
+puts "  TERM: #{ENV['TERM']}"
+puts "  Columns: #{`tput cols`.strip}" rescue nil
+puts "  Lines: #{`tput lines`.strip}" rescue nil
+
+# Test 7: Check for signal handling changes
+puts "\nTest 7: Signal handling..."
+begin
+  old_handler = Signal.trap("WINCH") { }
+  Signal.trap("WINCH", old_handler)
+  puts "  ✓ SIGWINCH trap works"
+rescue => e
+  puts "  ✗ SIGWINCH trap failed: #{e.message}"
+end
+
+puts "\n" + "=" * 50
+puts "Diagnostic complete!"
+puts "\nPlease share this output to help identify the Ruby 3.4.5 compatibility issue."

data/hyperlist

@@ -7,7 +7,7 @@
 # Check for help/version BEFORE loading any libraries
 if ARGV[0] == '-h' || ARGV[0] == '--help'
   puts <<~HELP
-    HyperList v1.1.4 - Terminal User Interface for HyperList files
+    HyperList v1.1.6 - Terminal User Interface for HyperList files
     
     USAGE
       hyperlist [OPTIONS] [FILE]
@@ -52,7 +52,7 @@ if ARGV[0] == '-h' || ARGV[0] == '--help'
   HELP
   exit 0
 elsif ARGV[0] == '-v' || ARGV[0] == '--version'
-  puts "HyperList v1.1.4"
+  puts "HyperList v1.1.6"
   exit 0
 end
 
@@ -70,7 +70,7 @@ class HyperListApp
   include Rcurses::Input
   include Rcurses::Cursor
   
-  VERSION = "1.1.4"
+  VERSION = "1.1.6"
   
   def initialize(filename = nil)
     @filename = filename ? File.expand_path(filename) : nil
@@ -706,7 +706,7 @@ class HyperListApp
       line = "    " * item["level"]  # 4 spaces per level
       
       # Add fold indicator
-      real_idx = @items.index(item)
+      real_idx = get_real_index(item)
       if real_idx && has_children?(real_idx, @items) && item["fold"]
         color = (@presentation_mode && !is_item_in_presentation_focus?(item)) ? "240" : "245"
         line += "▶".fg(color) + " "
@@ -1111,7 +1111,10 @@ class HyperListApp
         next
       end
       
-      visible << item
+      # Store the item with its real index to avoid lookup issues
+      item_with_index = item.dup
+      item_with_index["_real_index"] = idx
+      visible << item_with_index
       
       if item["fold"] && has_children?(idx, @items)
         # Skip children if folded
@@ -1129,6 +1132,12 @@ class HyperListApp
     visible
   end
   
+  # Helper method to get the real index from a visible item
+  def get_real_index(visible_item)
+    return nil unless visible_item
+    visible_item["_real_index"] || @items.index(visible_item)
+  end
+  
   def has_children?(idx, items)
     return false if idx >= items.length - 1
     return false if idx < 0
@@ -1219,7 +1228,7 @@ class HyperListApp
       return if @current >= visible.length
       
       item = visible[@current]
-      real_idx = @items.index(item)
+      real_idx = get_real_index(item)
       
       if real_idx && has_children?(real_idx, @items)
         @items[real_idx]["fold"] = !@items[real_idx]["fold"]
@@ -1385,7 +1394,7 @@ class HyperListApp
     # Remember which item we're focused on
     current_item = visible_items[@current]
     current_level = current_item["level"]
-    current_real_idx = @items.index(current_item)
+    current_real_idx = get_real_index(current_item)
     
     # First, fold everything
     @items.each_with_index do |item, idx|
@@ -1602,7 +1611,7 @@ class HyperListApp
     visible = get_visible_items
     if @current < visible.length
       level = visible[@current]["level"]
-      real_idx = @items.index(visible[@current])
+      real_idx = get_real_index(visible[@current])
       @items.insert(real_idx + 1, {"text" => text, "level" => level, "fold" => false})
     else
       @items << {"text" => text, "level" => 0, "fold" => false}
@@ -1633,7 +1642,7 @@ class HyperListApp
     visible = get_visible_items
     if @current < visible.length
       level = visible[@current]["level"] + 1
-      real_idx = @items.index(visible[@current])
+      real_idx = get_real_index(visible[@current])
       
       # Unfold parent if needed
       @items[real_idx]["fold"] = false
@@ -1651,7 +1660,7 @@ class HyperListApp
     return if @current >= visible.length
     
     item = visible[@current]
-    real_idx = @items.index(item)
+    real_idx = get_real_index(item)
     
     @mode = :insert
     
@@ -1672,7 +1681,7 @@ class HyperListApp
     return if @current >= visible.length
     
     item = visible[@current]
-    real_idx = @items.index(item)
+    real_idx = get_real_index(item)
     
     save_undo_state  # Save state before modification
     @items[real_idx]["text"] = text
@@ -1688,7 +1697,7 @@ class HyperListApp
     save_undo_state  # Save state before modification
     
     item = visible[@current]
-    real_idx = @items.index(item)
+    real_idx = get_real_index(item)
     
     # First, yank the item(s) to clipboard
     @clipboard = []
@@ -1742,7 +1751,7 @@ class HyperListApp
     return if @current >= visible.length
     
     item = visible[@current]
-    real_idx = @items.index(item)
+    real_idx = get_real_index(item)
     
     @clipboard = []
     @clipboard << item.dup
@@ -1771,7 +1780,7 @@ class HyperListApp
     
     visible = get_visible_items
     if @current < visible.length
-      real_idx = @items.index(visible[@current])
+      real_idx = get_real_index(visible[@current])
       
       if @clipboard_is_tree
         # For tree paste (C-D or Y), maintain original indentation structure
@@ -1810,7 +1819,7 @@ class HyperListApp
     save_undo_state  # Save state before modification
     
     item = visible[@current]
-    real_idx = @items.index(item)
+    real_idx = get_real_index(item)
     
     # Can't move if already at the beginning
     return if real_idx == 0
@@ -1852,7 +1861,7 @@ class HyperListApp
     save_undo_state  # Save state before modification
     
     item = visible[@current]
-    real_idx = @items.index(item)
+    real_idx = get_real_index(item)
     
     # Collect item(s) to move
     items_to_move = [item]
@@ -1896,7 +1905,7 @@ class HyperListApp
     return if @current >= visible.length
     
     item = visible[@current]
-    real_idx = @items.index(item)
+    real_idx = get_real_index(item)
     
     # Can only indent if there's a previous item at same or higher level
     if real_idx > 0
@@ -1925,7 +1934,7 @@ class HyperListApp
     return if @current >= visible.length
     
     item = visible[@current]
-    real_idx = @items.index(item)
+    real_idx = get_real_index(item)
     
     if @items[real_idx]["level"] > 0
       save_undo_state  # Save state before modification
@@ -1955,7 +1964,7 @@ class HyperListApp
     save_undo_state  # Save state before modification
     
     item = visible[@current]
-    real_idx = @items.index(item)
+    real_idx = get_real_index(item)
     text = @items[real_idx]["text"]
     
     if text =~ /^\[.\]/
@@ -1984,7 +1993,7 @@ class HyperListApp
     save_undo_state  # Save state before modification
     
     item = visible[@current]
-    real_idx = @items.index(item)
+    real_idx = get_real_index(item)
     text = @items[real_idx]["text"]
     
     # First, handle existing timestamp in the hyperlist.vim format
@@ -4197,7 +4206,7 @@ class HyperListApp
     @marks ||= {}
     visible = get_visible_items
     if @current < visible.length
-      real_idx = @items.index(visible[@current])
+      real_idx = get_real_index(visible[@current])
       @marks[mark] = real_idx
       @message = "Mark '#{mark}' set"
     end
@@ -4212,7 +4221,7 @@ class HyperListApp
       # Find the marked item in visible items
       target_idx = @marks[mark]
       visible = get_visible_items
-      visible_idx = visible.find_index { |item| @items.index(item) == target_idx }
+      visible_idx = visible.find_index { |item| get_real_index(item) == target_idx }
       
       if visible_idx
         @current = visible_idx
@@ -4348,13 +4357,13 @@ class HyperListApp
           when "o"
             visible = get_visible_items
             if @current < visible.length
-              real_idx = @items.index(visible[@current])
+              real_idx = get_real_index(visible[@current])
               @items[real_idx]["fold"] = false if real_idx
             end
           when "c"
             visible = get_visible_items
             if @current < visible.length
-              real_idx = @items.index(visible[@current])
+              real_idx = get_real_index(visible[@current])
               if real_idx && has_children?(real_idx, @items)
                 @items[real_idx]["fold"] = true
               end
@@ -4546,8 +4555,35 @@ class HyperListApp
 end
 
 # Main
-if __FILE__ == $0
+if __FILE__ == $0 && !defined?($hyperlist_wrapper_mode)
   # Normal operation - help/version already handled at top of file
+  
+  # For Ruby 3.4+, explicitly initialize rcurses if needed
+  if RUBY_VERSION >= "3.4.0"
+    begin
+      Rcurses.init! unless Rcurses.respond_to?(:initialized?) && Rcurses.initialized?
+    rescue => e
+      # If initialization fails, try to provide helpful error
+      puts "Failed to initialize terminal interface: #{e.message}"
+      puts "Try running with: ruby #{__FILE__} #{ARGV.join(' ')}"
+      exit 1
+    end
+  end
+  
   app = HyperListApp.new(ARGV[0])
-  app.run
+  
+  begin
+    app.run
+  ensure
+    # Ensure proper cleanup for Ruby 3.4+
+    if RUBY_VERSION >= "3.4.0"
+      begin
+        Rcurses.done! if defined?(Rcurses.done!)
+      rescue
+        # Fallback cleanup
+        print "\e[?25h"  # Show cursor
+        system("stty sane 2>/dev/null")
+      end
+    end
+  end
 end

data/hyperlist.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "hyperlist"
-  spec.version       = "1.1.4"
+  spec.version       = "1.1.6"
   spec.authors       = ["Geir Isene"]
   spec.email         = ["g@isene.com"]
 

data/hyperlist_ruby34_wrapper.rb

@@ -0,0 +1,57 @@
+#!/usr/bin/env ruby
+# encoding: utf-8
+
+# Wrapper for Ruby 3.4+ compatibility
+# This ensures proper initialization of rcurses
+
+# Handle help/version before loading libraries
+if ARGV[0] == '-h' || ARGV[0] == '--help' || ARGV[0] == '-v' || ARGV[0] == '--version'
+  # Pass through to main hyperlist
+  load File.join(File.dirname(__FILE__), 'hyperlist')
+  exit 0
+end
+
+# Explicitly initialize rcurses before loading the main app
+require 'rcurses'
+
+begin
+  # Initialize rcurses with proper error handling
+  Rcurses.init!
+  
+  # Now load and run the main hyperlist app
+  # We need to prevent the main file from running automatically
+  $hyperlist_wrapper_mode = true
+  
+  # Load the hyperlist file
+  load File.join(File.dirname(__FILE__), 'hyperlist')
+  
+  # Create and run the app
+  app = HyperListApp.new(ARGV[0])
+  app.run
+  
+rescue Interrupt
+  # Handle Ctrl+C gracefully
+  Rcurses.done! if defined?(Rcurses.done!)
+  puts "\nInterrupted"
+  exit 0
+rescue => e
+  # Ensure terminal is restored on error
+  Rcurses.done! if defined?(Rcurses.done!)
+  
+  # Fallback terminal restoration
+  print "\e[?25h"  # Show cursor
+  system("stty sane 2>/dev/null")
+  
+  puts "Error: #{e.message}"
+  puts "Backtrace:" if ENV['DEBUG']
+  e.backtrace.first(10).each { |line| puts "  #{line}" } if ENV['DEBUG']
+  exit 1
+ensure
+  # Always try to restore terminal
+  begin
+    Rcurses.done! if defined?(Rcurses.done!)
+  rescue
+    print "\e[?25h"
+    system("stty sane 2>/dev/null")
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hyperlist
 version: !ruby/object:Gem::Version
-  version: 1.1.4
+  version: 1.1.6
 platform: ruby
 authors:
 - Geir Isene
@@ -75,11 +75,13 @@ files:
 - README.md
 - debug_instructions.md
 - diagnose.rb
+- diagnose_ruby34.rb
 - diagnose_safe.rb
 - fix_terminal.sh
 - hyperlist
 - hyperlist.gemspec
 - hyperlist_logo.svg
+- hyperlist_ruby34_wrapper.rb
 - sample.hl
 - screenshot_help.png
 - screenshot_sample.png