termpix 0.1.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 321b92b52842de9fc8f7cd06688662f235423478f947ce9d5c96cac2083f582c
-  data.tar.gz: 76d16a71354bed1f1cc8e3196d01d48a09800c3f9a1eb9f330605f720ef3d06a
+  metadata.gz: 34ec8fff70b1018305deaa94931516564e573406a713fac62bbf6691dfe120b8
+  data.tar.gz: fcf1056c196bf42680fc1f78b8067db71a64ee342651c6684cb78f34d27f6d1d
 SHA512:
-  metadata.gz: 5d510b46a5ad6de11c7ae89d43c7d0a07192db3a04590e5b684f9516113a18408089effc2048be2be0fe35cd9cf1ae9f4de0704d0a7e009773bb18b44beeb5b6
-  data.tar.gz: a52a9e6bace28d65a4b70db051c277979866ce2c00f6276f471d51c1b6f53b86c36a44b7c38a5977ff406de1bf0130e7484404a6aa589362db52fd123a1c478f
+  metadata.gz: f706b1ff612ec9f6f08596833fe66b5fc1b50946c61aa76bb9c8a88ba327d6e595c5607fb439024a94e29d857b603801f577a05876864c8f8493a9bbdbc8a906
+  data.tar.gz: 4610351de9645c8dd6855d25f70b800e74b088633a6cf8d3bc02e785f104ac62230e637923bf4c7aa9251973c434e4917aaa6c0c60d15a1d1a27bad7eb4760c6

data/README.md

@@ -6,6 +6,8 @@
 
 Display images in the terminal using the best available protocol.
 
+<br clear="left"/>
+
 ## Features
 
 - Auto-detects terminal capabilities

data/lib/termpix/protocols.rb

@@ -4,21 +4,19 @@ require 'base64'
 module Termpix
   module Protocols
     # Kitty Graphics Protocol
+    # After extensive testing: Kitty protocol fundamentally incompatible with curses
+    # The protocol requires precise cursor control that conflicts with curses' buffer
+    # management. Recommend disabling Kitty protocol for curses-based apps.
     module Kitty
       def self.display(image_path, x:, y:, max_width:, max_height:)
-        # Read image and encode to base64
-        image_data = Base64.strict_encode64(File.read(image_path))
-
-        # Use virtual placement (no cursor positioning - avoids curses conflicts)
-        # Transmit image without positioning, let it flow inline
-        $stdout.write "\e_Ga=T,f=100,q=2;#{image_data}\e\\"
-        $stdout.flush
+        # Kitty graphics protocol does not work reliably with curses applications
+        # The fundamental issue is that both Kitty and curses need exclusive control
+        # of terminal state, causing unavoidable conflicts
+        false
       end
 
       def self.clear
-        # Delete all Kitty images
-        $stdout.write "\e_Ga=d,d=A\e\\"
-        $stdout.flush
+        true
       end
 
       private
@@ -52,8 +50,9 @@ module Termpix
 
         # Position cursor at the specified character position
         print "\e[#{y};#{x}H"
-        # Convert to sixel and display with proper pixel dimensions
-        system("convert #{escaped} -resize #{pixel_width}x#{pixel_height} sixel:- 2>/dev/null")
+        # Use > to only shrink, never enlarge, preserving aspect ratio
+        # ImageMagick will fit image within box while maintaining aspect ratio
+        system("convert #{escaped} -resize #{pixel_width}x#{pixel_height}\\> sixel:- 2>/dev/null")
       end
 
       def self.clear
@@ -117,14 +116,35 @@ module Termpix
         img_max_w = char_w * max_width
         img_max_h = char_h * max_height
 
-        # Get image dimensions
+        # Check if image has EXIF orientation data before auto-orienting
+        # This avoids unnecessary temp file creation for most images
         escaped = Shellwords.escape(image_path)
-        dimensions = `identify -format "%wx%h" #{escaped}[0] 2>/dev/null`.strip
+
+        # Quick check: only auto-orient if image has EXIF orientation tag
+        has_orientation = `identify -format "%[EXIF:Orientation]" #{escaped}[0] 2>/dev/null`.strip
+
+        if has_orientation && !has_orientation.empty? && has_orientation != "1"
+          # Image needs rotation - create cached temp file
+          require 'digest'
+          file_hash = Digest::MD5.hexdigest(image_path)
+          temp_file = "/tmp/termpix_#{file_hash}.jpg"
+
+          unless File.exist?(temp_file)
+            system("convert #{escaped}[0] -auto-orient #{temp_file} 2>/dev/null")
+          end
+
+          display_path = File.exist?(temp_file) ? temp_file : image_path
+        else
+          # No rotation needed - use original image
+          display_path = image_path
+        end
+
+        dimensions = `identify -format "%wx%h" #{Shellwords.escape(display_path)} 2>/dev/null`.strip
         return if dimensions.empty?
 
         img_w, img_h = dimensions.split('x').map(&:to_i)
 
-        # Scale if needed
+        # Scale if needed - preserve aspect ratio
         if img_w > img_max_w || img_h > img_max_h
           scale_w = img_max_w.to_f / img_w
           scale_h = img_max_h.to_f / img_h
@@ -134,9 +154,11 @@ module Termpix
         end
 
         # Display using w3mimgdisplay protocol
-        `echo '0;1;#{img_x};#{img_y};#{img_w};#{img_h};;;;;#{image_path}
+        `echo '0;1;#{img_x};#{img_y};#{img_w};#{img_h};;;;;#{display_path}
 4;
 3;' | #{@imgdisplay} 2>/dev/null`
+
+        # Don't delete temp file - keep it cached for performance
       end
 
       def self.clear(x:, y:, width:, height:, term_width:, term_height:)

data/lib/termpix/version.rb

@@ -1,3 +1,3 @@
 module Termpix
-  VERSION = "0.1.0"
+  VERSION = "0.2.1"
 end

data/lib/termpix.rb

@@ -80,11 +80,9 @@ module Termpix
         return :sixel if check_dependency('convert')
       end
 
-      # Kitty graphics protocol disabled - incompatible with curses apps
-      # Kitty protocol needs full terminal control, conflicts with curses rendering
-      # Users can choose between:
-      # 1. Enable w3m for Kitty (has brief flash) - set TERMPIX_KITTY_USE_W3M=1
-      # 2. No images in Kitty (clean UI)
+      # Kitty graphics protocol disabled - fundamentally incompatible with curses
+      # After extensive testing: Kitty protocol requires exclusive terminal control
+      # that conflicts with curses' buffer management. Use w3m fallback instead.
 
       # Überzug++ - disabled for now (implementation incomplete)
       # TODO: Implement proper Überzug++ JSON-RPC communication

metadata

@@ -1,18 +1,19 @@
 --- !ruby/object:Gem::Specification
 name: termpix
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Geir Isene
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-10-26 00:00:00.000000000 Z
+date: 2025-11-03 00:00:00.000000000 Z
 dependencies: []
-description: Termpix provides a clean API for displaying images in the terminal using
-  the best available protocol (Kitty, Sixel, Überzug++, or w3m). Auto-detects terminal
-  capabilities and falls back gracefully.
+description: 'Termpix v0.2.1: Performance optimization - only auto-orients images
+  with EXIF rotation data. Provides clean API for displaying images in terminal using
+  best available protocol (Sixel or w3m). Auto-detects terminal capabilities and falls
+  back gracefully.'
 email: g@isene.com
 executables: []
 extensions: []