rufio 0.83.0 → 0.83.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b75328867a09d6dfa3b4c759e242d239ab8d087540a43af19471473cc0515c4c
-  data.tar.gz: 798358451d71a8b74ee70ce2d362b2f24fb887e6de9b574f700c0f25a449e389
+  metadata.gz: c96ef480cab0968de6e595b56af26baeb59c0fd0542a85e2ccc0e4d1970bd696
+  data.tar.gz: 50366b86a8ef40df39f00a58e7a4e4d86d8314c919855acd96cdc0a03f39a156
 SHA512:
-  metadata.gz: 8072bc03c642bbc3043caca4cfa4899c72b6622ab7a74517ef7964010a9257e938005191de9a8e8aaed444749108ef1ef983b6b4dc8483aa8b4d2a6a8cb075a6
-  data.tar.gz: d6fb6e10e070e94120518d5a883bd241b46a957e7938b375b09e7dc3e28e73e9652aabe7b6a191e2575872764919581ac6d4b163d48fbb2d5dcd50e7740e2e2f
+  metadata.gz: dd1d2a5f82340c12caf57184b90cf3ad0773ef905caf07214376eab6b41d2af5c99fec088bd6e4dc51c20b928c3640d62c8b1d3681d08724fdb32e719e4205cd
+  data.tar.gz: d38b805a6592cae4b6cc1c8e08ea587c3f88ebdadd605561b1cef67a3700009f5d656df61548a2f755db659c89b995dfabdd38a29e14c8e728280d22f12fe337

data/CHANGELOG.md

@@ -5,6 +5,16 @@ All notable changes to rufio 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).
 
+## [0.83.1] - 2026-03-20
+
+### Changed
+- **Windows console input detection rewritten using Win32 API**: Replaced the background thread + Queue workaround (introduced in v0.82.1 to handle ESC key detection on Windows) with `GetNumberOfConsoleInputEvents` via the Win32 API (`fiddle` stdlib)
+  - The previous approach had a race condition where the background `STDIN.read(1)` thread competed with `STDIN.getch` calls in dialogs, potentially dropping dialog input (including multi-byte/Japanese characters)
+  - Background thread, Queue, and `windows_read_next_byte` helper are removed
+  - `read_next_input_byte` is now a single unified implementation for both Windows and Unix (`read_nonblock`)
+  - Cygwin is excluded from the Windows path since it provides POSIX-compatible `IO.select`
+  - Falls back to `read_nonblock` if `Fiddle::DLError` occurs (no extra gem required — `fiddle` is a Ruby stdlib)
+
 ## [0.83.0] - 2026-03-14
 
 ### Added

data/lib/rufio/terminal_ui.rb

@@ -189,23 +189,6 @@ module Rufio
       print "\e[?1003h\e[?1006h"
       STDOUT.flush
 
-      # Windows: IO.select + read_nonblockはコンソールハンドルのESCキーを
-      # 検出できない場合がある。バックグラウンドスレッドでSTDINを読み取り
-      # Queueに格納することで信頼性のある入力検出を実現する。
-      if windows?
-        @windows_input_queue = Queue.new
-        @windows_input_thread = Thread.new do
-          loop do
-            begin
-              byte = STDIN.read(1)
-              @windows_input_queue << byte if byte
-            rescue
-              break
-            end
-          end
-        end
-      end
-
       # re-acquire terminal size (just in case)
       update_screen_size
     end
@@ -217,31 +200,37 @@ module Rufio
       @screen_width, @screen_height = console.winsize.reverse
     end
 
+    # Cygwinは POSIX互換のIO.selectが使えるため除外
     def windows?
-      RUBY_PLATFORM =~ /mswin|mingw|cygwin/ ? true : false
+      RUBY_PLATFORM =~ /mswin|mingw/ ? true : false
     end
 
-    # Windows: エスケープシーケンスの後続バイトをQueueから短いタイムアウトで読み取る
-    # VT対応ターミナルでの矢印キー（\e[A 等）の後続バイト読み取りに使用
-    def windows_read_next_byte
-      deadline = Process.clock_gettime(Process::CLOCK_MONOTONIC) + 0.005 # 5ms
-      loop do
-        begin
-          return @windows_input_queue.pop(true)
-        rescue ThreadError
-          return nil if Process.clock_gettime(Process::CLOCK_MONOTONIC) >= deadline
-          sleep 0.001
-        end
-      end
+    # Windows: GetNumberOfConsoleInputEvents でコンソール入力バッファを確認する。
+    # IO.select はWindowsコンソールハンドルでESCキーを取りこぼすため使用しない。
+    # fiddle はRuby標準ライブラリなので追加gemは不要。
+    def windows_console_input_available?
+      require 'fiddle'
+      @win32_kernel32 ||= Fiddle.dlopen('kernel32')
+      @win32_get_std_handle ||= Fiddle::Function.new(
+        @win32_kernel32['GetStdHandle'],
+        [Fiddle::TYPE_INT], Fiddle::TYPE_VOIDP
+      )
+      @win32_get_num_events ||= Fiddle::Function.new(
+        @win32_kernel32['GetNumberOfConsoleInputEvents'],
+        [Fiddle::TYPE_VOIDP, Fiddle::TYPE_VOIDP], Fiddle::TYPE_INT
+      )
+      handle = @win32_get_std_handle.call(0xFFFFFFF6) # STD_INPUT_HANDLE = (DWORD)(-10)
+      count_ptr = Fiddle::Pointer.malloc(4)
+      @win32_get_num_events.call(handle, count_ptr)
+      count_ptr[0, 4].unpack1('L') > 0
+    rescue Fiddle::DLError
+      # fiddle が使えない場合は常に入力ありとみなし read_nonblock に任せる
+      true
     end
 
     # エスケープシーケンスの後続バイトを読み取る（Windows/Unix共通ヘルパー）
     def read_next_input_byte
-      if @windows_input_queue
-        windows_read_next_byte
-      else
-        STDIN.read_nonblock(1) rescue nil
-      end
+      STDIN.read_nonblock(1) rescue nil
     end
 
     def cleanup_terminal
@@ -250,13 +239,6 @@ module Rufio
         STDIN.cooked!
       end
 
-      # Windowsバックグラウンド入力スレッドを停止
-      if @windows_input_thread
-        @windows_input_thread.kill rescue nil
-        @windows_input_thread = nil
-        @windows_input_queue = nil
-      end
-
       # マウスレポートを無効化
       print "\e[?1003l\e[?1006l"
       STDOUT.flush
@@ -442,31 +424,24 @@ module Rufio
     private
 
     # ノンブロッキング入力処理（ゲームループ用）
-    # Windows: Queueベース、Unix: IO.select + read_nonblock
+    # Windows: GetNumberOfConsoleInputEvents で入力確認後 read_nonblock
+    # Unix:    IO.select(timeout=0) で入力確認後 read_nonblock
     def handle_input_nonblocking
       # 入力バイトを1つ読み取る
-      if @windows_input_queue
-        # Windows: バックグラウンドスレッドのQueueからノンブロッキングで読み取る
-        begin
-          input = @windows_input_queue.pop(true)
-        rescue ThreadError
-          return false
-        end
+      if windows?
+        # Windows: IO.selectはESCキーを取りこぼすため Win32 API で入力確認
+        return false unless windows_console_input_available?
       else
         # Unix: 0msタイムアウトで即座にチェック（30FPS = 33.33ms/frame）
-        ready = IO.select([STDIN], nil, nil, 0)
-        return false unless ready
-
-        begin
-          # read_nonblockを使ってノンブロッキングで1文字読み取る
-          input = STDIN.read_nonblock(1)
-        rescue IO::WaitReadable, IO::EAGAINWaitReadable
-          # 入力が利用できない
-          return false
-        rescue Errno::ENOTTY, Errno::ENODEV
-          # ターミナルでない環境
-          return false
-        end
+        return false unless IO.select([STDIN], nil, nil, 0)
+      end
+
+      begin
+        input = STDIN.read_nonblock(1)
+      rescue IO::WaitReadable, IO::EAGAINWaitReadable
+        return false
+      rescue Errno::ENOTTY, Errno::ENODEV
+        return false
       end
 
       # コマンドモードがアクティブな場合は、エスケープシーケンス処理をスキップ

data/lib/rufio/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Rufio
-  VERSION = '0.83.0'
+  VERSION = '0.83.1'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rufio
 version: !ruby/object:Gem::Version
-  version: 0.83.0
+  version: 0.83.1
 platform: ruby
 authors:
 - masisz
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-03-14 00:00:00.000000000 Z
+date: 2026-03-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: io-console