procon_bypass_man 0.1.10 → 0.1.14

data/lib/procon_bypass_man/device_connector.rb

@@ -1,5 +1,6 @@
 class ProconBypassMan::DeviceConnector
   class BytesMismatchError < StandardError; end
+  class NotFoundProconError < StandardError; end
 
   class Value
     attr_accessor :read_from, :values
@@ -48,6 +49,7 @@ class ProconBypassMan::DeviceConnector
   end
 
   def drain_all
+    debug_log_buffer = []
     unless @initialized_devices
       init_devices
     end
@@ -55,11 +57,13 @@ class ProconBypassMan::DeviceConnector
     while(item = @stack.shift)
       item.values.each do |value|
         data = nil
-        timer = ProconBypassMan::Timer.new
+        timer = ProconBypassMan::SafeTimeout.new
         begin
           timer.throw_if_timeout!
           data = from_device(item).read_nonblock(64)
+          debug_log_buffer << "read_from(#{item.read_from}): #{data}"
         rescue IO::EAGAINWaitReadable
+          debug_log_buffer << "read_from(#{item.read_from}): IO::EAGAINWaitReadable"
           retry
         end
 
@@ -74,15 +78,19 @@ class ProconBypassMan::DeviceConnector
           end
         if result
           ProconBypassMan.logger.info "OK(expected: #{value}, got: #{data.unpack("H*")})"
+          debug_log_buffer << "OK(expected: #{value}, got: #{data.unpack("H*")})"
         else
           ProconBypassMan.logger.info "NG(expected: #{value}, got: #{data.unpack("H*")})"
+          debug_log_buffer << "NG(expected: #{value}, got: #{data.unpack("H*")})"
           raise BytesMismatchError if @throw_error_if_mismatch
         end
         to_device(item).write_nonblock(data)
       end
     end
-  rescue ProconBypassMan::Timer::Timeout
+  rescue ProconBypassMan::SafeTimeout::Timeout
     ProconBypassMan.logger.error "timeoutになりました"
+    copressed_buffer_text = ProconBypassMan::CompressArray.new(debug_log_buffer).compress.join("\n")
+    ProconBypassMan::SendErrorCommand.execute(error: copressed_buffer_text)
     raise if @throw_error_if_timeout
   end
 
@@ -95,31 +103,31 @@ class ProconBypassMan::DeviceConnector
       init_devices
     end
 
-    timer = ProconBypassMan::Timer.new
+    timer = ProconBypassMan::SafeTimeout.new
     data = nil
     begin
       timer.throw_if_timeout!
       switch.write_nonblock(data)
     rescue IO::EAGAINWaitReadable
       retry
-    rescue ProconBypassMan::Timer::Timeout
+    rescue ProconBypassMan::SafeTimeout::Timeout
       ProconBypassMan.logger.error "writeでtimeoutになりました"
       raise
     end
     return(data.unpack("H*")) if only_write
 
-    timer = ProconBypassMan::Timer.new
+    timer = ProconBypassMan::SafeTimeout.new
     begin
       timer.throw_if_timeout!
       data = switch.read_nonblock(64)
       ProconBypassMan.logger.debug { " >>> #{data.unpack("H*")})" }
     rescue IO::EAGAINWaitReadable
       retry
-    rescue ProconBypassMan::Timer::Timeout
+    rescue ProconBypassMan::SafeTimeout::Timeout
       ProconBypassMan.logger.error "readでtimeoutになりました"
       raise
     end
-  rescue ProconBypassMan::Timer::Timeout
+  rescue ProconBypassMan::SafeTimeout::Timeout
     raise if @throw_error_if_timeout
   end
 
@@ -131,30 +139,30 @@ class ProconBypassMan::DeviceConnector
       init_devices
     end
 
-    timer = ProconBypassMan::Timer.new
+    timer = ProconBypassMan::SafeTimeout.new
     begin
       timer.throw_if_timeout!
       procon.write_nonblock(data)
     rescue IO::EAGAINWaitReadable
       retry
-    rescue ProconBypassMan::Timer::Timeout
+    rescue ProconBypassMan::SafeTimeout::Timeout
       ProconBypassMan.logger.error "writeでtimeoutになりました"
       raise
     end
     return(data.unpack("H*")) if only_write
 
-    timer = ProconBypassMan::Timer.new
+    timer = ProconBypassMan::SafeTimeout.new
     begin
       timer.throw_if_timeout!
       data = procon.read_nonblock(64)
       ProconBypassMan.logger.error " <<< #{data.unpack("H*")})"
     rescue IO::EAGAINWaitReadable
       retry
-    rescue ProconBypassMan::Timer::Timeout
+    rescue ProconBypassMan::SafeTimeout::Timeout
       ProconBypassMan.logger.error "readでtimeoutになりました"
       raise
     end
-  rescue ProconBypassMan::Timer::Timeout
+  rescue ProconBypassMan::SafeTimeout::Timeout
     raise if @throw_error_if_timeout
   end
 
@@ -164,30 +172,30 @@ class ProconBypassMan::DeviceConnector
     end
 
     data = nil
-    timer = ProconBypassMan::Timer.new
+    timer = ProconBypassMan::SafeTimeout.new
     begin
       timer.throw_if_timeout!
       data = procon.read_nonblock(64)
       ProconBypassMan.logger.debug { " <<< #{data.unpack("H*")})" }
     rescue IO::EAGAINWaitReadable
       retry
-    rescue ProconBypassMan::Timer::Timeout
+    rescue ProconBypassMan::SafeTimeout::Timeout
       ProconBypassMan.logger.error "readでtimeoutになりました"
       raise
     end
     return(data.unpack("H*")) if only_read
 
-    timer = ProconBypassMan::Timer.new
+    timer = ProconBypassMan::SafeTimeout.new
     begin
       timer.throw_if_timeout!
       switch.write_nonblock(data)
     rescue IO::EAGAINWaitReadable
       retry
-    rescue ProconBypassMan::Timer::Timeout
+    rescue ProconBypassMan::SafeTimeout::Timeout
       ProconBypassMan.logger.error "writeでtimeoutになりました"
       raise
     end
-  rescue ProconBypassMan::Timer::Timeout
+  rescue ProconBypassMan::SafeTimeout::Timeout
     raise if @throw_error_if_timeout
   end
 
@@ -197,30 +205,30 @@ class ProconBypassMan::DeviceConnector
     end
 
     data = nil
-    timer = ProconBypassMan::Timer.new
+    timer = ProconBypassMan::SafeTimeout.new
     begin
       timer.throw_if_timeout!
       data = switch.read_nonblock(64)
       ProconBypassMan.logger.debug { " >>> #{data.unpack("H*")})" }
     rescue IO::EAGAINWaitReadable
       retry
-    rescue ProconBypassMan::Timer::Timeout
+    rescue ProconBypassMan::SafeTimeout::Timeout
       ProconBypassMan.logger.error "readでtimeoutになりました"
       raise
     end
     return(data.unpack("H*")) if only_read
 
-    timer = ProconBypassMan::Timer.new
+    timer = ProconBypassMan::SafeTimeout.new
     begin
       timer.throw_if_timeout!
       procon.write_nonblock(data)
     rescue IO::EAGAINWaitReadable
       retry
-    rescue ProconBypassMan::Timer::Timeout
+    rescue ProconBypassMan::SafeTimeout::Timeout
       ProconBypassMan.logger.error "writeでtimeoutになりました"
       raise
     end
-  rescue ProconBypassMan::Timer::Timeout
+  rescue ProconBypassMan::SafeTimeout::Timeout
     raise if @throw_error_if_timeout
   end
 
@@ -293,7 +301,7 @@ class ProconBypassMan::DeviceConnector
       @procon = File.open(PROCON2_PATH, "w+b")
       @gadget = File.open('/dev/hidg0', "w+b")
     else
-      raise "/dev/hidraw0, /dev/hidraw1の両方見つかりませんでした"
+      raise NotFoundProconError, "/dev/hidraw0, /dev/hidraw1の両方見つかりませんでした"
     end
     system('echo > /sys/kernel/config/usb_gadget/procon/UDC')
     system('ls /sys/class/udc > /sys/kernel/config/usb_gadget/procon/UDC')
@@ -309,11 +317,10 @@ class ProconBypassMan::DeviceConnector
     end
   rescue Errno::ENXIO => e
     # /dev/hidg0 をopenできないときがある
-    ProconBypassMan.logger.error "Errno::ENXIO (No such device or address @ rb_sysopen - /dev/hidg0)が起きました。resetします"
-    ProconBypassMan.logger.error e
+    ProconBypassMan::SendErrorCommand.execute(error: "Errno::ENXIO (No such device or address @ rb_sysopen - /dev/hidg0)が起きました。resetします. #{e.full_message}")
     system('echo > /sys/kernel/config/usb_gadget/procon/UDC')
     system('ls /sys/class/udc > /sys/kernel/config/usb_gadget/procon/UDC')
-    sleep 2
+    sleep 0.5
     retry
   end
 end

data/lib/procon_bypass_man/device_status.rb

@@ -0,0 +1,44 @@
+class ProconBypassMan::DeviceStatus
+  INITIALIZED = :initialized
+  RUNNING = :running
+  CONNECTED_BUT_SLEEPING = :connected_but_sleeping # コードはつながっているが、switchがsleepしているとき
+  PROCON_NOT_FOUND_ERROR = :procon_not_found_error # 繋がっていないとか、デバイスが使えない時
+  CONNECTED_BUT_ERROR = :connected_but_error # 実行時エラーあたり
+  SETTING_SYNTAX_ERROR_AND_SHUTDOWN = :setting_syntax_error_and_shutdown
+
+  @@status = nil
+
+  def self.current
+    @@status || INITIALIZED
+  end
+
+  def self.change_to_running!
+    @@status = RUNNING
+    ProconBypassMan::SyncDeviceStatsJob.perform_async(ProconBypassMan::DeviceStatus.current)
+  end
+
+  def self.change_to_connected_but_sleeping!
+    @@status = CONNECTED_BUT_SLEEPING
+    ProconBypassMan::SyncDeviceStatsJob.perform_async(ProconBypassMan::DeviceStatus.current)
+  end
+
+  def self.change_to_procon_not_found_error!
+    @@status = PROCON_NOT_FOUND_ERROR
+    ProconBypassMan::SyncDeviceStatsJob.perform_async(ProconBypassMan::DeviceStatus.current)
+  end
+
+  def self.change_to_device_error!
+    @@status = DEVICE_ERROR
+    ProconBypassMan::SyncDeviceStatsJob.perform_async(ProconBypassMan::DeviceStatus.current)
+  end
+
+  def self.change_to_connected_but_error!
+    @@status = CONNECTED_BUT_ERROR
+    ProconBypassMan::SyncDeviceStatsJob.perform_async(ProconBypassMan::DeviceStatus.current)
+  end
+
+  def self.change_to_setting_syntax_error_and_shutdown!
+    @@status = SETTING_SYNTAX_ERROR_AND_SHUTDOWN
+    ProconBypassMan::SyncDeviceStatsJob.perform_async(ProconBypassMan::DeviceStatus.current)
+  end
+end

data/lib/procon_bypass_man/io_monitor.rb

@@ -1,11 +1,12 @@
 module ProconBypassMan
   class Counter
-    attr_accessor :label, :table, :previous_table
+    attr_accessor :label, :table, :previous_table, :active
 
     def initialize(label: )
       self.label = label
       self.table = {}
       self.previous_table = {}
+      self.active = true
     end
 
     # アクティブなバケットは1つだけ
@@ -24,7 +25,7 @@ module ProconBypassMan
       self
     end
 
-    def formated_previous_table
+    def formatted_previous_table
       t = previous_table.dup
       start_function = t[:start_function] || 0
       end_function = t[:end_function] || 0
@@ -32,6 +33,10 @@ module ProconBypassMan
       eagain_wait_readable_on_write = t[:eagain_wait_readable_on_write] || 0
       "(#{(end_function / start_function.to_f * 100).floor(1)}%(#{end_function}/#{start_function}), loss: #{eagain_wait_readable_on_read}, #{eagain_wait_readable_on_write})"
     end
+
+    def shutdown
+      self.active = false
+    end
   end
 
   module IOMonitor
@@ -51,14 +56,14 @@ module ProconBypassMan
       Thread.start do
         max_output_length = 0
         loop do
-          list = @@list.dup
+          list = @@list.select(&:active).dup
           unless list.all? { |x| x&.previous_table.is_a?(Hash) }
             sleep 0.5
             next
           end
 
           line = list.map { |counter|
-            "#{counter.label}(#{counter.formated_previous_table})"
+            "#{counter.label}(#{counter.formatted_previous_table})"
           }.join(", ")
           max_output_length = line.length
           sleep 0.7

data/lib/procon_bypass_man/procon/analog_stick.rb

@@ -0,0 +1,31 @@
+class ProconBypassMan::Procon::AnalogStick
+  attr_accessor :neutral_position
+  attr_accessor :bin_x, :bin_y
+
+  def initialize(binary: )
+    @neutral_position = ProconBypassMan::ButtonsSettingConfiguration.instance.neutral_position
+
+    byte6 = binary[6].unpack("H*").first.to_i(16).to_s(2).rjust(8, "0")
+    byte7 = binary[7].unpack("H*").first.to_i(16).to_s(2).rjust(8, "0")
+    byte8 = binary[8].unpack("H*").first.to_i(16).to_s(2).rjust(8, "0")
+
+    self.bin_x = "#{byte7[4..7]}#{byte6}"
+    self.bin_y = "#{byte8}#{byte7[0..3]}"
+  end
+
+  def abs_x
+    bin_x.to_i(2)
+  end
+
+  def abs_y
+    bin_y.to_i(2)
+  end
+
+  def relative_x
+    bin_x.to_i(2) - neutral_position.x
+  end
+
+  def relative_y
+    bin_y.to_i(2) - neutral_position.y
+  end
+end

data/lib/procon_bypass_man/procon/analog_stick_cap.rb

@@ -18,19 +18,9 @@ class ProconBypassMan::Procon::AnalogStickCap
     end
   end
 
-  attr_accessor :bin_x, :bin_y
-  attr_accessor :neutral_position
-
   def initialize(binary)
-    @neutral_position = ProconBypassMan::ButtonsSettingConfiguration.instance.neutral_position
     @binary = binary
-
-    byte6 = binary[6].unpack("H*").first.to_i(16).to_s(2).rjust(8, "0")
-    byte7 = binary[7].unpack("H*").first.to_i(16).to_s(2).rjust(8, "0")
-    byte8 = binary[8].unpack("H*").first.to_i(16).to_s(2).rjust(8, "0")
-
-    self.bin_x = "#{byte7[4..7]}#{byte6}"
-    self.bin_y = "#{byte8}#{byte7[0..3]}"
+    @analog_stick = ProconBypassMan::Procon::AnalogStick.new(binary: binary)
   end
 
   # @return [ProconBypassMan::Procon::AnalogStickCap::Position]
@@ -41,8 +31,8 @@ class ProconBypassMan::Procon::AnalogStickCap
       relative_capped_x = -(relative_capped_x.abs) if relative_x.negative?
       relative_capped_y = -(relative_capped_y.abs) if relative_y.negative?
       return Position.new(
-        x: relative_capped_x + neutral_position.x,
-        y: relative_capped_y + neutral_position.y,
+        x: relative_capped_x + @analog_stick.neutral_position.x,
+        y: relative_capped_y + @analog_stick.neutral_position.y,
       )
     else
       return position
@@ -54,23 +44,10 @@ class ProconBypassMan::Procon::AnalogStickCap
     Position.new(x: abs_x, y: abs_y)
   end
 
-  # 0, 0からのx
-  def abs_x
-    bin_x.to_i(2)
-  end
-
-  # 0, 0からのy
-  def abs_y
-    bin_y.to_i(2)
-  end
-
-  def relative_x
-    bin_x.to_i(2) - neutral_position.x
-  end
-
-  def relative_y
-    bin_y.to_i(2) - neutral_position.y
-  end
+  def abs_x; @analog_stick.abs_x; end # 0, 0からのx
+  def abs_y; @analog_stick.abs_y; end # 0, 0からのy
+  def relative_x; @analog_stick.relative_x; end
+  def relative_y; @analog_stick.relative_y; end
 
   # @deprecated
   def x; relative_x; end
@@ -78,11 +55,11 @@ class ProconBypassMan::Procon::AnalogStickCap
 
   def rad
     (
-      Math.atan(y / x.to_f) * 180 / Math::PI
+      Math.atan(relative_y / relative_x.to_f) * 180 / Math::PI
     ).floor(6)
   end
 
   def hypotenuse
-    Math.sqrt(x**2 + y**2).floor(6)
+    Math.sqrt((relative_x**2) + (relative_y**2)).floor(6)
   end
 end

data/lib/procon_bypass_man/procon/button_collection.rb

@@ -37,6 +37,7 @@ class ProconBypassMan::Procon::ButtonCollection
   BUTTONS_MAP = BYTES_MAP.reduce({}) { |acc, value|
     next acc if value[1].nil?
     value[1].reverse.each.with_index do |button, index|
+      next(acc) if button == :grip || button == :_undefined_key
       acc[button] = { byte_position: value[0], bit_position: index }
     end
     acc

data/lib/procon_bypass_man/procon/{data.rb → consts.rb}

@@ -1,5 +1,5 @@
 class ProconBypassMan::Procon
-  module Data
+  module Consts
     NO_ACTION = "30f28100800078c77448287509550274ff131029001b0022005a0271ff191028001e00210064027cff1410280020002100000000000000000000000000000000".freeze
   end
 end

data/lib/procon_bypass_man/procon/layer_changer.rb

@@ -0,0 +1,40 @@
+class ProconBypassMan::Procon::LayerChanger
+  def initialize(binary: )
+    @procon_reader = ProconBypassMan::ProconReader.new(binary: binary).freeze
+  end
+
+  # @return [Symbol]
+  def next_layer_key
+    case
+    when pressed?(button: :up)
+      :up
+    when pressed?(button: :right)
+      :right
+    when pressed?(button: :left)
+      :left
+    when pressed?(button: :down)
+      :down
+    else
+      ProconBypassMan.logger.warn("next_layer_key is unknown")
+      :up
+    end
+  end
+
+  # @return [Boolean]
+  def change_layer?
+    if ProconBypassMan::ButtonsSettingConfiguration.instance.prefix_keys.empty?
+      raise "prefix_keysが未設定です"
+    end
+    ProconBypassMan::ButtonsSettingConfiguration.instance.prefix_keys.map { |b| pressed?(button: b) }.all?
+  end
+
+  # @return [Boolean]
+  def pressed_next_layer?
+    change_layer? && (pressed?(button: :up) || pressed?(button: :right) || pressed?(button: :left) || pressed?(button: :down))
+  end
+
+  # @return [Boolean]
+  def pressed?(button: )
+    @procon_reader.pressed.include?(button)
+  end
+end

data/lib/procon_bypass_man/procon/user_operation.rb

@@ -1,12 +1,14 @@
 class ProconBypassMan::Procon
   class UserOperation
-    include LayerChangeable
-    extend PushedButtonHelper::Dynamic
-
     attr_reader :binary
 
+    ::ProconBypassMan::Procon::ButtonCollection::BUTTONS_MAP.each do |button, _value|
+      define_method "pressed_#{button}?" do
+        pressed_button?(button)
+      end
+    end
+
     def initialize(binary)
-      self.class.compile_if_not_compile_yet!
       unless binary.encoding.name == ASCII_ENCODING
         raise "おかしいです"
       end
@@ -16,20 +18,16 @@ class ProconBypassMan::Procon
     ZERO_BIT = ["0"].pack("H*").freeze
     ASCII_ENCODING = "ASCII-8BIT"
 
-    # @depilicate
-    def binary=(binary)
-      unless binary.encoding.name == ASCII_ENCODING
-        raise "おかしいです"
-      end
-      @binary = binary
-    end
-
     def set_no_action!
       binary[3] = ZERO_BIT
       binary[4] = ZERO_BIT
       binary[5] = ZERO_BIT
     end
 
+    def apply_left_analog_stick_cap(cap: )
+      binary[6..8] = ProconBypassMan::Procon::AnalogStickCap.new(binary).capped_position(cap_hypotenuse: cap).to_binary
+    end
+
     def unpress_button(button)
       return if not pressed_button?(button)
 
@@ -38,10 +36,6 @@ class ProconBypassMan::Procon
       binary[byte_position] = ["%02X" % value.to_s].pack("H*")
     end
 
-    def apply_left_analog_stick_cap(cap: )
-      binary[6..8] = ProconBypassMan::Procon::AnalogStickCap.new(binary).capped_position(cap_hypotenuse: cap).to_binary
-    end
-
     def press_button(button)
       return if pressed_button?(button)
 
@@ -51,7 +45,7 @@ class ProconBypassMan::Procon
     end
 
     def press_button_only(button)
-      [ProconBypassMan::Procon::Data::NO_ACTION.dup].pack("H*").tap do |no_action_binary|
+      [ProconBypassMan::Procon::Consts::NO_ACTION.dup].pack("H*").tap do |no_action_binary|
         ButtonCollection.load(button).byte_position
         byte_position = ButtonCollection.load(button).byte_position
         value = 2**ButtonCollection.load(button).bit_position

data/lib/procon_bypass_man/procon.rb

@@ -1,10 +1,9 @@
 class ProconBypassMan::Procon
-  require "procon_bypass_man/procon/data"
+  require "procon_bypass_man/procon/consts"
   require "procon_bypass_man/procon/mode_registry"
   require "procon_bypass_man/procon/macro_registry"
-  require "procon_bypass_man/procon/layer_changeable"
+  require "procon_bypass_man/procon/layer_changer"
   require "procon_bypass_man/procon/button_collection"
-  require "procon_bypass_man/procon/pressed_button_helper"
   require "procon_bypass_man/procon/user_operation"
   require "procon_bypass_man/procon/flip_cache"
   require "procon_bypass_man/procon/press_button_aware"
@@ -35,8 +34,9 @@ class ProconBypassMan::Procon
   end
 
   def apply!
-    if user_operation.change_layer?
-      @@status[:current_layer_key] = user_operation.next_layer_key if user_operation.pressed_next_layer?
+    layer_changer = ProconBypassMan::Procon::LayerChanger.new(binary: user_operation.binary)
+    if layer_changer.change_layer?
+      @@status[:current_layer_key] = layer_changer.next_layer_key if layer_changer.pressed_next_layer?
       user_operation.set_no_action!
       return
     end
@@ -137,11 +137,7 @@ class ProconBypassMan::Procon
       end
     end
 
-    b = user_operation.binary
-    ProconBypassMan.cache.fetch key: 'user_operation.binary', expires_in: 60 do
-      ProconBypassMan::Procon::DebugDumper.new(binary: b).dump_analog_sticks
-    end
-    b
+    user_operation.binary
   end
 
   private

data/lib/procon_bypass_man/procon_reader.rb

@@ -0,0 +1,31 @@
+class ProconBypassMan::ProconReader
+  def initialize(binary: )
+    @binary = binary
+    @analog_stick = ProconBypassMan::Procon::AnalogStick.new(binary: binary)
+  end
+
+  # @return [Array<Symbol>]
+  def pressed
+    aware = ProconBypassMan::PpressButtonAware.new(@binary)
+    pressed_table = ::ProconBypassMan::Procon::ButtonCollection::BUTTONS.reduce({}) do |acc, button|
+      acc[button] = aware.pressed_button?(button)
+      acc
+    end
+    pressed_table.select { |_key, value| value }.keys
+  end
+
+  def left_analog_stick
+    { x: @analog_stick.relative_x, y: @analog_stick.relative_y }
+  end
+
+  def left_analog_stick_by_abs
+    { x: @analog_stick.abs_x, y: @analog_stick.abs_y }
+  end
+
+  def to_hash
+    { left_analog_stick: left_analog_stick,
+      left_analog_stick_by_abs: left_analog_stick_by_abs,
+      buttons: pressed,
+    }
+  end
+end

data/lib/procon_bypass_man/remote_pbm_action/base_action.rb

@@ -0,0 +1,53 @@
+module ProconBypassMan
+  module RemotePbmAction
+    class ActionUnexpectedError < StandardError; end
+    class NeedPbmVersionError < ActionUnexpectedError; end
+
+    class BaseAction
+      attr_accessor :pbm_job_uuid
+
+      # @param [String] pbm_job_uuid
+      def initialize(pbm_job_uuid: )
+        self.pbm_job_uuid = pbm_job_uuid
+      end
+
+      # @return [void]
+      def action_content(_args)
+        raise NotImplementedError, nil
+      end
+
+      # @param [Hash] args
+      # @return [void]
+      def run!(job_args: )
+        before_action_callback
+        action_content(args: job_args)
+        after_action_callback
+      rescue => e
+        be_failed
+        ProconBypassMan::SendErrorCommand.execute(error: e)
+      end
+
+      private
+
+      # @return [void]
+      def before_action_callback; end
+      # @return [void]
+      def after_action_callback; end
+
+      # @return [void]
+      def be_failed
+        ProconBypassMan::UpdateRemotePbmActionStatusCommand.new(pbm_job_uuid: pbm_job_uuid).execute(to_status: :failed)
+      end
+
+      # @return [void]
+      def be_in_progress
+        ProconBypassMan::UpdateRemotePbmActionStatusCommand.new(pbm_job_uuid: pbm_job_uuid).execute(to_status: :in_progress)
+      end
+
+      # @return [void]
+      def be_processed
+        ProconBypassMan::UpdateRemotePbmActionStatusCommand.new(pbm_job_uuid: pbm_job_uuid).execute(to_status: :processed)
+      end
+    end
+  end
+end

data/lib/procon_bypass_man/remote_pbm_action/change_pbm_version_action.rb

@@ -0,0 +1,21 @@
+module ProconBypassMan
+  module RemotePbmAction
+    class ChangePbmVersionAction < BaseAction
+
+      def action_content(args: )
+        require "pbmenv"
+        ProconBypassMan.logger.info "execute ChangePbmVersionAction!"
+        pbm_version = args["pbm_version"] or raise(ProconBypassMan::RemotePbmAction::NeedPbmVersionError, "pbm_versionが必要です, #{args.inspect}")
+        Pbmenv.install(pbm_version)
+        Pbmenv.use(pbm_version)
+        `reboot` # symlinkの参照先が変わるのでrebootする必要がある
+      end
+
+      private
+
+      def before_action_callback
+        be_processed
+      end
+    end
+  end
+end