procon_bypass_man 0.1.13 → 0.1.16.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 62fab79bffee3adf43af06325d7ef6bce2060f844d8caaa824968ff77219e5de
-  data.tar.gz: c15ab8b50da926169727b9af5ff097585cd35c7e24fdf2ce75d2ca7d4fe5f07e
+  metadata.gz: 5af6fb77bbbd2aca55c93c8af1bb9c72ede43a043f069001b30201bf8352e7de
+  data.tar.gz: 5607eb2f41d4a09de6e7e04cb8afcf998c09d790931ed105dbe5136c7160698a
 SHA512:
-  metadata.gz: c94bc5e37b89c54a31fe165e53f84ac32c1b4d10fc617cc1397e0347baf1a39206fd50aff3a65626b8546545d00f33ac0e44a90da5ff25c02778d27e60aafe0f
-  data.tar.gz: 7ad56ef9e17bb8a26d840003faf8fa7d065ba878182dcd23d66118b43053c2827221814f37bb4098174cdfbcc449486d895bf6431520f60ce295a4e7382c39e7
+  metadata.gz: f21c116b2d69b79a2c896cd8c1298a1e80d82c2c879e0e0db328b64f9abb19901da93edd0a88d64314413092bfafead55a9ab359f9e09a2546794812178701e1
+  data.tar.gz: 89d32cbe122d38ad19d55481539a05cfad959695d46c29d2dc6db8ac34eb251a2f2f49d7ab36f5d514e191751ba19f747dd13e5a21fd4cde4ae435a93ce117de

data/.github/workflows/release.yml

@@ -0,0 +1,33 @@
+name: Publish to RubyGems
+
+on:
+  [workflow_dispatch]
+
+jobs:
+  release:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: write
+
+    steps:
+    - uses: actions/checkout@v2
+      with:
+        token: ${{ secrets.GITHUB_TOKEN }}
+    - name: Set up Ruby 2.6
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: 2.6
+        bundler-cache: true
+    - name: Publish to RubyGems
+      run: |
+        git config --local user.email "action@github.com"
+        git config --local user.name "GitHub Action"
+
+        mkdir -p $HOME/.gem
+        touch $HOME/.gem/credentials
+        chmod 0600 $HOME/.gem/credentials
+        printf -- "---\n:rubygems_api_key: ${RUBYGEMS_API_TOKEN}\n" > $HOME/.gem/credentials
+        bundle exec rake release
+      env:
+        RUBYGEMS_API_TOKEN: "${{secrets.RUBYGEMS_API_TOKEN}}"
+        GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}

data/CHANGELOG.md

@@ -1,3 +1,16 @@
+## [0.1.16.1] - 2022-01-22
+- 初期状態ではinternal api(pbm-web)の通信を無効にしました
+
+## [0.1.16] - 2021-12-26
+- websocket経由で外部サーバからの命令を受け取れるようになりました
+
+## [0.1.15] - 2021-12-08
+- 外部サーバから設定ファイルのリストアができるようになりました
+- splatoon2 pluginのネームスペースを変更しました
+
+## [0.1.14] - 2021-12-02
+- 外部サーバからOSの再起動やprocon_bypass_manのバージョンアップができるようになりました
+
 ## [0.1.13] - 2021-11-24
 - 起動を早くするために、procon_bypass_man-splatoon2を同梱しました
 

data/Gemfile.lock

@@ -1,11 +1,16 @@
 PATH
   remote: .
   specs:
-    procon_bypass_man (0.1.13)
+    procon_bypass_man (0.1.16.1)
+      action_cable_client
+      pbmenv
+      sorted_set
 
 GEM
   remote: https://rubygems.org/
   specs:
+    action_cable_client (3.1.0)
+      websocket-eventmachine-client (>= 1.2.0)
     activesupport (6.1.4.1)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
@@ -16,6 +21,7 @@ GEM
     coderay (1.1.3)
     concurrent-ruby (1.1.9)
     diff-lcs (1.4.4)
+    eventmachine (1.2.7)
     ffi (1.15.4)
     i18n (1.8.11)
       concurrent-ruby (~> 1.0)
@@ -28,8 +34,9 @@ GEM
     mustermann (1.1.1)
       ruby2_keywords (~> 0.0.1)
     parallel (1.21.0)
-    parser (3.0.2.0)
+    parser (3.0.3.2)
       ast (~> 2.4.1)
+    pbmenv (0.1.7)
     pry (0.14.1)
       coderay (~> 1.1)
       method_source (~> 1.0)
@@ -42,7 +49,8 @@ GEM
     rb-inotify (0.10.1)
       ffi (~> 1.0)
     rbs (1.7.1)
-    regexp_parser (2.1.1)
+    rbtree (0.4.4)
+    regexp_parser (2.2.0)
     rexml (3.2.5)
     rspec (3.10.0)
       rspec-core (~> 3.10.0)
@@ -66,23 +74,27 @@ GEM
       rubocop-ast (>= 1.12.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 3.0)
-    rubocop-ast (1.13.0)
+    rubocop-ast (1.14.0)
       parser (>= 3.0.1.1)
     ruby-progressbar (1.11.0)
     ruby2_keywords (0.0.5)
+    set (1.0.2)
     sinatra (2.1.0)
       mustermann (~> 1.0)
       rack (~> 2.2)
       rack-protection (= 2.1.0)
       tilt (~> 2.0)
-    steep (0.46.0)
+    sorted_set (1.0.3)
+      rbtree
+      set (~> 1.0)
+    steep (0.47.0)
       activesupport (>= 5.1)
       language_server-protocol (>= 3.15, < 4.0)
       listen (~> 3.0)
       parallel (>= 1.0.0)
       parser (>= 3.0)
       rainbow (>= 2.2.2, < 4.0)
-      rbs (>= 1.2.0)
+      rbs (~> 1.7.0)
       terminal-table (>= 2, < 4)
     terminal-table (3.0.2)
       unicode-display_width (>= 1.1.1, < 3)
@@ -92,6 +104,14 @@ GEM
       concurrent-ruby (~> 1.0)
     unicode-display_width (2.1.0)
     webrick (1.7.0)
+    websocket (1.2.9)
+    websocket-eventmachine-base (1.2.0)
+      eventmachine (~> 1.0)
+      websocket (~> 1.0)
+      websocket-native (~> 1.0)
+    websocket-eventmachine-client (1.3.0)
+      websocket-eventmachine-base (~> 1.0)
+    websocket-native (1.0.0)
     zeitwerk (2.5.1)
 
 PLATFORMS

data/README.md

@@ -28,12 +28,30 @@ Switch <-- (PBM): ZR連打
 ## Usage
 * USBガジェットモードで起動するRaspberry Pi4を用意する
   * https://github.com/splaplapla/procon_bypass_man/blob/master/docs/setup_raspi.md
-* Raspberry Pi4 で sudo pbmenv install latest && cd /usr/share/pbm/current && sudo ruby app.rb
+* Raspberry Pi4 でprocon_bypass_manを実行するための準備
+  * rubyのインストール
+    * sudo apt-get install rbenv
+    * git clone https://github.com/rbenv/ruby-build.git "$(rbenv root)"/plugins/ruby-build
+    * rbenv install 3.0.1
+  * sudo gem install pbmenv
+  * sudo [pbmenv](https://github.com/splaplapla/pbmenv) install latest
+* Raspberry Pi4 でprocon_bypass_manを実行する
+  * cd /usr/share/pbm/current
+  * sudo /home/pi/.rbenv/versions/3.0.1/bin/ruby app.rb
   * 動いたのを確認したらserviceとして登録にするなどしてください
+    * [serviceとして登録する方法](https://github.com/splaplapla/procon_bypass_man/tree/master/project_template#systemd%E3%82%92%E4%BD%BF%E3%81%A3%E3%81%A6%E3%82%B5%E3%83%BC%E3%83%93%E3%82%B9%E3%81%AB%E7%99%BB%E9%8C%B2%E3%81%99%E3%82%8B%E6%96%B9%E6%B3%95)
 
 ## Plugins
 * https://github.com/splaplapla/procon_bypass_man-splatoon2
 
+## 関連ソフトウェア
+* サーバソフトウェア(WIP)
+  * https://github.com/splaplapla/procon_bypass_man_cloud
+  * このサーバからOS自体の再起動、設定ファイルの変更、PBMのバージョンアップができます
+  * 自分でホストティングしてください
+* pbmenv
+  * バージョンマネージャー
+
 ## FAQ
 * どうやって動かすの?
     * このツールはRaspberry Pi4をUSBガジェットモードで起動して有線でプロコンとSwitchに接続して使います
@@ -46,6 +64,8 @@ Switch <-- (PBM): ZR連打
     * マクロは特定のキーを順番に入れていく機能。キーの入力が終わったらマクロは終了する
 * レイヤーとは？
     * 自作キーボードみたいな感じでレイヤー毎に設定内容を記述して切り替えれる
+* レイヤーを切り替える方法は？
+    * 設定ファイルに記述している `prefix_keys_for_changing_layer`の後ろにあるキーを同時押しながら、十字キーのどれかを押すことで任意のレイヤーに切り替わります
 * このツールでできることは？
     * キーリマップ, 連射, マクロ, 特定の同じ操作の繰り返し(mode)
         * リマップは1つのキーを別のキーに割り当てます
@@ -54,6 +74,10 @@ Switch <-- (PBM): ZR連打
     * 操作するdeviceファイルの所有者がrootだから
 * 市販されているサードパーティ製連射機との違いは？
     * サードパーティ製のコントローラーは、設定方法や形状が特殊で買い換えるたびに学習・設定コストが発生します。本ツールを使えば、設定内容はテキストで管理することができ、使い慣れたプロコンで同等のことができます。
+* sshなしで運用は可能ですか？
+    * 後述するWEBアプリケーションを使えば、sshを使わずに運用が可能です
+        * https://github.com/splaplapla/procon_bypass_man_cloud
+    * 機能開発中ですが、使ってくれる人を探しているのでdiscordからお問合せください
 
 ## TODO
 * レコーディング機能(プロコンの入力をマクロとして登録ができる)
@@ -67,7 +91,7 @@ sudo kill -USR2 `cat ./pbm_pid`
 ```
 
 ### 起動ログをサーバに送信する
-* `ProconBypassMan.api_server = "http://.."` を設定すると、 `POST /api/events` に対して起動ログなどを送信するようになります
+* `ProconBypassMan.api_servers = "http://.."` を設定すると、 `POST /api/events` に対して起動ログなどを送信するようになります
 
 ### 開発環境でログの送信を確認する方法
 * `bundle exec bin/dev_api_server.rb`
@@ -83,6 +107,6 @@ sudo kill -USR2 `cat ./pbm_pid`
 
 The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).
 
-
 ## Links
 * https://discord.gg/bEcRNKf4ep
+  * 質問などご意見をdiscordでも受け付けています

data/docs/setup_raspi.md

@@ -1,3 +1,5 @@
+https://github.com/splaplapla/procon_bypass_man/blob/master/docs/setup_raspi_by_mitamae.md に半自動化した手順があります
+
 # Raspberry Pi4のセットアップ手順
 * SDカードにRaspberry Pi OS (32-bit)を焼く
 * SDカードをRaspberry Pi4本体に挿して起動する

data/docs/setup_raspi.mitamae.rb

@@ -1,6 +1,6 @@
 run_command "apt-get update"
 
-package 'ruby' do
+package 'rbenv' do
   action :install
 end
 
@@ -41,16 +41,15 @@ execute "Initialize PBM" do
  SHELL
 end
 
-# rbenv
-execute "Install rbenv" do
-  not_if "which rbenv"
-  command "git clone https://github.com/rbenv/ruby-build.git "$(rbenv root)"/plugins/ruby-build"
-end
-
 # ruby
 execute "Install ruby" do
+  user "pi"
   not_if "rbenv versions | grep 3.0.1"
-  command "rbenv install 3.0.1"
+  command <<~EOH
+    mkdir -p "$(rbenv root)"/plugins
+    git clone https://github.com/rbenv/ruby-build.git --depth 1 "$(rbenv root)"/plugins/ruby-build
+    rbenv install 3.0.1
+  EOH
 end
 
 run_command 'sudo systemctl disable triggerhappy.socket'

data/docs/setup_raspi_by_mitamae.md

@@ -1,9 +1,11 @@
+セットアップ手順を半自動化にしました
+
 # Raspberry Pi4のセットアップ手順 With mitamae
 * SDカードにRaspberry Pi OS lite (32-bit)を焼く
     * sshをできる状態で焼いておく
 * SDカードをRaspberry Pi4本体に挿して起動する
 * sshする
-* curl -L https://github.com/itamae-kitchen/mitamae/releases/latest/download/mitamae-armhf-linux.tar.gz | tar xvz
+* wget -O - "https://github.com/itamae-kitchen/mitamae/releases/latest/download/mitamae-armhf-linux.tar.gz" | tar xvz
 * wget https://raw.githubusercontent.com/splaplapla/procon_bypass_man/master/docs/setup_raspi.mitamae.rb -O setup_raspi.mitamae.rb
 * sudo ./mitamae-armhf-linux local setup_raspi.mitamae.rb -l debug
 * sudo reboot
@@ -12,3 +14,37 @@
 * PCとRaspberry Pi4を接続し、プロコンとして認識していることを確認する
 * sudo gem i pbmenv
 * sudo pbmenv install latest
+
+## 上記手順の動作確認方法
+* 使えそうなイメージ
+  * navikey/raspbian-bullseye
+  * balenalib/raspberry-pi
+
+```shell
+docker run -it --rm --name my-running-app2 navikey/raspbian-bullseye bash
+```
+
+### 準備
+* docker runするとrootなのでpiでログインする
+
+```shell
+useradd -m --uid 1000 --groups sudo pi
+echo pi:pi | chpasswd
+su pi
+cd ~ && sudo ls
+```
+
+### mitamaeスクリプトを実行する
+
+```
+wget -O - "https://github.com/itamae-kitchen/mitamae/releases/latest/download/mitamae-armhf-linux.tar.gz" | tar xvz
+wget https://raw.githubusercontent.com/splaplapla/procon_bypass_man/master/docs/setup_raspi.mitamae.rb -O setup_raspi.mitamae.rb
+sudo ./mitamae-armhf-linux local setup_raspi.mitamae.rb -l debug
+```
+
+実行に成功したら以下を確認する
+
+* /home/pi/.rbenv/verions に ruby3.0.1がインストールしていること
+* /etc/modules に指定の文字列があること
+* /boot/config.txt に指定の文字列があること
+* /usr/share/pbm/shared/add_procon_gadget.sh が存在していること

data/lib/ext/em_pure_ruby.rb

@@ -0,0 +1,25 @@
+# https://github.com/eventmachine/eventmachine/pull/929 is not released yet.
+# i will delete this patch if released.
+module EventMachine
+  # @private
+  class Reactor
+    def run_timers
+      timers_to_delete = []
+      @timers.each {|t|
+        if t.first <= @current_loop_time
+          #@timers.delete t
+          timers_to_delete << t
+          EventMachine::event_callback "", TimerFired, t.last
+        else
+          break
+        end
+      }
+      timers_to_delete.map{|c| @timers.delete c}
+      timers_to_delete = nil
+      #while @timers.length > 0 and @timers.first.first <= now
+      #  t = @timers.shift
+      #  EventMachine::event_callback "", TimerFired, t.last
+      #end
+    end
+  end
+end

data/lib/procon_bypass_man/background/jobs/base_job.rb

@@ -1,12 +1,7 @@
 class ProconBypassMan::BaseJob
-  extend ProconBypassMan::Background::HasServerPool
   extend ProconBypassMan::Background::JobRunnable
 
   def self.servers
-    ProconBypassMan.config.api_servers
-  end
-
-  def self.path
-    "/api/events"
+    raise NotImplementedError
   end
 end

data/lib/procon_bypass_man/background/jobs/concerns/has_external_api_setting.rb

@@ -0,0 +1,5 @@
+module ProconBypassMan::HasExternalApiSetting
+  def server_pool
+    ProconBypassMan.config.server_pool
+  end
+end

data/lib/procon_bypass_man/background/jobs/concerns/has_internal_api_setting.rb

@@ -0,0 +1,5 @@
+module ProconBypassMan::HasInternalApiSetting
+  def server_pool
+    ProconBypassMan.config.internal_server_pool
+  end
+end

data/lib/procon_bypass_man/background/jobs/report_boot_job.rb

@@ -1,9 +1,11 @@
-class ProconBypassMan::ReportBootJob <  ProconBypassMan::BaseJob
+class ProconBypassMan::ReportBootJob < ProconBypassMan::ReportEventBaseJob
+  extend ProconBypassMan::HasExternalApiSetting
+
   # @param [String] body
   def self.perform(body)
-    ProconBypassMan::Background::HttpClient.new(
+    ProconBypassMan::ReportHttpClient.new(
       path: path,
-      pool_server: pool_server,
+      server_pool: server_pool,
       retry_on_connection_error: true,
     ).post(body: body, event_type: :boot)
   end

data/lib/procon_bypass_man/background/jobs/report_error_job.rb

@@ -1,10 +1,11 @@
-class ProconBypassMan::ReportErrorJob < ProconBypassMan::BaseJob
+class ProconBypassMan::ReportErrorJob < ProconBypassMan::ReportEventBaseJob
+  extend ProconBypassMan::HasExternalApiSetting
+
   # @param [String] body
   def self.perform(body)
-    ProconBypassMan::Background::HttpClient.new(
+    ProconBypassMan::ReportHttpClient.new(
       path: path,
-      pool_server: pool_server,
-      retry_on_connection_error: false,
-    ).post(body: body, event_type: :error,)
+      server_pool: server_pool,
+    ).post(body: body, event_type: :error)
   end
 end

data/lib/procon_bypass_man/background/jobs/report_event_base_job.rb

@@ -0,0 +1,5 @@
+class ProconBypassMan::ReportEventBaseJob < ProconBypassMan::BaseJob
+  def self.path
+    "/api/events".freeze
+  end
+end

data/lib/procon_bypass_man/background/jobs/report_load_config_job.rb

@@ -1,10 +1,11 @@
-class ProconBypassMan::ReportLoadConfigJob <  ProconBypassMan::BaseJob
+class ProconBypassMan::ReportLoadConfigJob < ProconBypassMan::ReportEventBaseJob
+  extend ProconBypassMan::HasExternalApiSetting
+
   # @param [String] body
   def self.perform(body)
-    ProconBypassMan::Background::HttpClient.new(
+    ProconBypassMan::ReportHttpClient.new(
       path: path,
-      pool_server: pool_server,
-      retry_on_connection_error: false,
+      server_pool: server_pool,
     ).post(body: body, event_type: :load_config)
   end
 end

data/lib/procon_bypass_man/background/jobs/report_pressed_buttons_job.rb

@@ -1,18 +1,15 @@
-class ProconBypassMan::ReportPressedButtonsJob
-  extend ProconBypassMan::Background::HasServerPool
-  extend ProconBypassMan::Background::JobRunnable
-
-  PATH = "/api/pressed_buttons"
+class ProconBypassMan::ReportPressedButtonsJob < ProconBypassMan::BaseJob
+  extend ProconBypassMan::HasInternalApiSetting
 
   # @param [String] body
   def self.perform(body)
-    ProconBypassMan::Background::HttpClient.new(
-      path: PATH,
-      pool_server: pool_server,
+    ProconBypassMan::ReportHttpClient.new(
+      path: path,
+      server_pool: server_pool,
     ).post(body: body, event_type: :internal)
   end
 
-  def self.servers
-    ProconBypassMan.config.internal_api_servers
+  def self.path
+    "/api/pressed_buttons"
   end
 end

data/lib/procon_bypass_man/background/jobs/report_reload_config_job.rb

@@ -1,10 +1,11 @@
-class ProconBypassMan::ReportReloadConfigJob <  ProconBypassMan::BaseJob
+class ProconBypassMan::ReportReloadConfigJob < ProconBypassMan::ReportEventBaseJob
+  extend ProconBypassMan::HasExternalApiSetting
+
   # @param [String] body
   def self.perform(body)
-    ProconBypassMan::Background::HttpClient.new(
+    ProconBypassMan::ReportHttpClient.new(
       path: path,
-      pool_server: pool_server,
-      retry_on_connection_error: false,
+      server_pool: server_pool,
     ).post(body: body, event_type: :reload_config)
   end
 end

data/lib/procon_bypass_man/background/jobs/sync_device_stats_job.rb

@@ -0,0 +1,16 @@
+class ProconBypassMan::SyncDeviceStatsJob < ProconBypassMan::BaseJob
+  extend ProconBypassMan::HasExternalApiSetting
+
+  # @param [Symbol] status
+  def self.perform(status)
+    ProconBypassMan::SendDeviceStatsHttpClient.new(
+      path: path,
+      server_pool: server_pool,
+    ).post(status: status, pbm_session_id: ProconBypassMan.session_id)
+  end
+
+  def self.path
+    device_id = ProconBypassMan.device_id
+    "/api/devices/#{ProconBypassMan.device_id}/device_statuses"
+  end
+end

data/lib/procon_bypass_man/background.rb

@@ -1,12 +1,13 @@
-require "procon_bypass_man/background/job_runnable"
+require "procon_bypass_man/background/jobs/concerns/has_internal_api_setting"
+require "procon_bypass_man/background/jobs/concerns/has_external_api_setting"
+require "procon_bypass_man/background/jobs/concerns/job_runnable"
 require "procon_bypass_man/background/job_runner"
 require "procon_bypass_man/background/job_performer"
-require "procon_bypass_man/background/has_server_pool"
-require "procon_bypass_man/background/http_client"
 require "procon_bypass_man/background/jobs/base_job"
+require "procon_bypass_man/background/jobs/report_event_base_job"
 require "procon_bypass_man/background/jobs/report_boot_job"
 require "procon_bypass_man/background/jobs/report_reload_config_job"
 require "procon_bypass_man/background/jobs/report_load_config_job"
 require "procon_bypass_man/background/jobs/report_error_job"
-require "procon_bypass_man/background/jobs/report_heartbeat_job"
 require "procon_bypass_man/background/jobs/report_pressed_buttons_job"
+require "procon_bypass_man/background/jobs/sync_device_stats_job"

data/lib/procon_bypass_man/buttons_setting_configuration/layer.rb

@@ -10,7 +10,6 @@ module ProconBypassMan
         self.remaps = {}
         self.left_analog_stick_caps = {}
         self.disables = []
-        instance_eval(&block) if block_given?
       end
 
       # @param [Symbol] button
@@ -50,11 +49,8 @@ module ProconBypassMan
           end
           hash[:flip_interval] = interval
         end
-        if self.flips[button]
-          raise "#{button}への設定をすでに割り当て済みです"
-        else
-          self.flips[button] = hash
-        end
+
+        self.flips[button] = hash
       end
 
       def macro(name, if_pressed: )

data/lib/procon_bypass_man/bypass/usb_hid_logger.rb

@@ -28,14 +28,12 @@ class ProconBypassMan::Bypass
         end
       end
 
-      ProconBypassMan.cache.fetch key: 'pressed_buttons_reporter', expires_in: 5 do
-        ProconBypassMan::ReportPressedButtonsJob.perform_async(
-          ProconBypassMan::ProconReader.new(binary: bypass_value.binary).to_hash
-        )
-      end
-
-      ProconBypassMan.cache.fetch key: 'heartbeat_reporter', expires_in: 60 do
-        ProconBypassMan::ReportHeartbeatJob.perform_async(ProconBypassMan::BootMessage.new.to_hash)
+      if ProconBypassMan.config.enable_reporting_pressed_buttons
+        ProconBypassMan.cache.fetch key: 'pressed_buttons_reporter', expires_in: 5 do
+          ProconBypassMan::ReportPressedButtonsJob.perform_async(
+            bypass_value.binary.to_procon_reader.to_hash
+          )
+        end
       end
     end
   end

data/lib/procon_bypass_man/bypass.rb

@@ -5,7 +5,7 @@ class ProconBypassMan::Bypass
 
   class BypassValue < Struct.new(:binary, :sent)
     def to_text
-      "#{binary.unpack("H*").first} #{'x' unless sent}"
+      "#{binary.unpack.first} #{'x' unless sent}"
     end
   end
 
@@ -21,14 +21,14 @@ class ProconBypassMan::Bypass
   def send_gadget_to_procon!
     monitor.record(:start_function)
     input = nil
-    self.bypass_value = BypassValue.new(input, sent = false)
+    self.bypass_value = BypassValue.new(nil, sent = false)
 
     run_callbacks(:send_gadget_to_procon) do
       begin
         break if $will_terminate_token
         # TODO blocking readにしたいが、接続時のフェーズによって長さが違うので厳しい
         input = self.gadget.read_nonblock(64)
-        self.bypass_value.binary = input
+        self.bypass_value.binary = ProconBypassMan::Domains::InboundProconBinary.new(binary: input)
       rescue IO::EAGAINWaitReadable
         monitor.record(:eagain_wait_readable_on_read)
         sleep(0.001)
@@ -50,14 +50,14 @@ class ProconBypassMan::Bypass
   def send_procon_to_gadget!
     monitor.record(:start_function)
     output = nil
-    self.bypass_value = BypassValue.new(output, sent = false)
+    self.bypass_value = BypassValue.new(nil, sent = false)
 
     run_callbacks(:send_procon_to_gadget) do
       begin
         break if $will_terminate_token
         Timeout.timeout(1) do
           output = self.procon.read(64)
-          self.bypass_value.binary = output
+          self.bypass_value.binary = ProconBypassMan::Domains::InboundProconBinary.new(binary: output)
         end
       rescue Timeout::Error
         ProconBypassMan.logger.debug { "read timeout! do sleep. by send_procon_to_gadget!" }
@@ -72,8 +72,15 @@ class ProconBypassMan::Bypass
         retry
       end
 
+      # blocking readをしているのでnilが入ることはないが、雑なテストでnilが通るので分岐を入れる。できれば消したい
+      break if output.nil?
+
       begin
-        self.gadget.write_nonblock(ProconBypassMan::Processor.new(output).process)
+        self.gadget.write_nonblock(
+          ProconBypassMan::Processor.new(
+            ProconBypassMan::Domains::InboundProconBinary.new(binary: output)
+          ).process
+        )
         self.bypass_value.sent = true
       rescue IO::EAGAINWaitReadable
         monitor.record(:eagain_wait_readable_on_write)

data/lib/procon_bypass_man/commands/connect_device_command.rb

@@ -2,8 +2,13 @@ class ProconBypassMan::ConnectDeviceCommand
   # @return [void]
   def self.execute!
     gadget, procon = ProconBypassMan::DeviceConnector.connect
+  rescue ProconBypassMan::DeviceConnector::NotFoundProconError => e
+    ProconBypassMan.logger.error e
+    gadget&.close
+    procon&.close
+    raise ProconBypassMan::NotFoundProconError
   rescue ProconBypassMan::SafeTimeout::Timeout
-    ::ProconBypassMan.logger.error "デバイスとの通信でタイムアウトが起きて接続ができませんでした。"
+    ProconBypassMan.logger.error "デバイスとの通信でタイムアウトが起きて接続ができませんでした。"
     gadget&.close
     procon&.close
     raise ::ProconBypassMan::EternalConnectionError