agent_status_bulb 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c2183f0e22cc7a3779f409df20894bdf032f6e2c2a342c72553f838e00d27c76
-  data.tar.gz: 0def7cc82dcf2aaa15b031c9275c01da270d939841b59e7ca0a6c3b018544f94
+  metadata.gz: 1687e4b5c95d5ab157aa65b015cc8ea7ca7f57790271819d4ff279ca3a550e3e
+  data.tar.gz: 4d0125c57a0f6689f297bb7ba460040afe0455e6ead33d33628d34f4ca03e8f7
 SHA512:
-  metadata.gz: 517a923a5b5011fd1dccce4c9e53aec9610ce278f485b148000cf99660162867ff8e53868d89e792ac7ff1b7d5feaadf709ab929bdb2000d1f0f6354124a33c4
-  data.tar.gz: 6c7cd63f36191263c4ef9b7f0527e76b672bd8cf3ce2757e8f4fbfef21f584d461161698bdf8cf95239210acbbe0f8081d9f7f70732e826b787b7ab360526d62
+  metadata.gz: 894c08da441f4af27c49630952bd7219c771edd363a797a836738d45d7a2f01ea69edb2d21e49407db0248493a7703156a4f36c90457d22a43e612cabd49b4c5
+  data.tar.gz: 82a12d8f0f460f8295fe2adb3611eb09167eb061948a4acd5af377b6d5429ab5ee298930c4d8ae767ff0d53e9ae5916ba6d6f90f33f548c1894faa245d57e7e0

data/.rubocop.yml

@@ -8,3 +8,9 @@ plugins:
 
 Style/Documentation:
   Enabled: false
+
+RSpec/ExampleLength:
+  Max: 9
+
+RSpec/MultipleExpectations:
+  Max: 2

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 ## [Unreleased]
 
+## [0.2.0] - 2025-11-19
+
+- Introduced `AgentStatusBulb::Bulb` to wrap SwitchBot integration and provide run/wait/idle/off color APIs with automatic power control
+- Rebuilt the CLI on top of Thor and routed every command through `handle_error` to normalize error handling
+- Simplified configuration input/save/load flows by driving them from a single field definition list
+- Added RSpec coverage for CLI, Configure, and Bulb to document expected behaviour
+- Added RuboCop configuration to keep coding style consistent
+
 ## [0.1.0] - 2025-11-18
 
 - Initial release

data/exe/asb

@@ -3,4 +3,10 @@
 
 require 'agent_status_bulb/cli'
 
-exit AgentStatusBulb::Cli.start(ARGV)
+begin
+  AgentStatusBulb::Cli.start(ARGV)
+  exit 0
+rescue StandardError => e
+  warn e.message
+  exit 1
+end

data/lib/agent_status_bulb/bulb.rb

@@ -0,0 +1,43 @@
+# frozen_string_literal: true
+
+require 'switchbot'
+
+module AgentStatusBulb
+  class Bulb
+    def self.from_config(config)
+      client = Switchbot::Client.new(config.fetch(:token), config.fetch(:secret))
+      new(client.color_bulb(config.fetch(:device_id)))
+    end
+
+    def initialize(device)
+      @device = device
+    end
+
+    def blue
+      apply_color('0:0:255')
+    end
+
+    def orange
+      apply_color('255:125:0')
+    end
+
+    def green
+      apply_color('0:255:0')
+    end
+
+    def off
+      @device.off
+    end
+
+    private
+
+    def apply_color(value)
+      turn_on_if_needed
+      @device.color(value)
+    end
+
+    def turn_on_if_needed
+      @device.on if @device.off?
+    end
+  end
+end

data/lib/agent_status_bulb/cli.rb

@@ -1,78 +1,56 @@
 # frozen_string_literal: true
 
+require 'thor'
 require 'agent_status_bulb'
-require 'switchbot'
 require 'agent_status_bulb/configure'
+require 'agent_status_bulb/bulb'
 
 module AgentStatusBulb
-  class Cli
-    def self.start(argv)
-      new(argv).run
-    end
-
-    def initialize(argv, io: $stdin, out: $stdout, err: $stderr)
-      @argv = argv.dup
-      @io = io
-      @out = out
-      @err = err
-    end
-
-    def run
-      command = @argv.shift
-      handle_command(command)
-    rescue StandardError => e
-      @err.puts(e.message)
-      1
-    end
+  class Cli < Thor
+    map 'run' => :run_command
+    map 'wait' => :wait_command
+    map 'idle' => :idle_command
+    map 'off' => :off_command
 
-    private
-
-    def handle_command(command)
-      return configure! if command == 'configure'
-      return apply!(command) if status_colors.key?(command) || command == 'off'
-
-      @err.puts usage
-      1
+    desc 'configure', 'Save Token/Secret/Device ID'
+    def configure
+      handle_error { configurator.configure! }
     end
 
-    def configure!
-      configurator.configure!
-      0
+    desc 'run', 'Set color to running (blue)'
+    def run_command
+      handle_error { bulb.blue }
     end
 
-    def apply!(command)
-      bulb = build_bulb(configurator.load!)
-      apply_status(bulb, command)
-      0
+    desc 'wait', 'Set color to waiting (orange)'
+    def wait_command
+      handle_error { bulb.orange }
     end
 
-    def build_bulb(config)
-      client = Switchbot::Client.new(config[:token], config[:secret])
-      bulb = client.color_bulb(config[:device_id])
-      bulb.on if bulb.off?
-      bulb
+    desc 'idle', 'Set color to idle (green)'
+    def idle_command
+      handle_error { bulb.green }
     end
 
-    def apply_status(bulb, command)
-      return bulb.off if command == 'off'
-
-      bulb.color(status_colors.fetch(command))
+    desc 'off', 'Turn off the bulb'
+    def off_command
+      handle_error { bulb.off }
     end
 
-    def usage
-      "Usage: #{File.basename($PROGRAM_NAME)} [run|wait|idle|off|configure]"
-    end
+    no_commands do
+      def handle_error
+        yield
+      rescue StandardError => e
+        raise Thor::Error, e.message
+      end
 
-    def configurator
-      @configurator ||= AgentStatusBulb::Configure.new(io: @io, out: @out)
-    end
+      def bulb
+        @bulb ||= AgentStatusBulb::Bulb.from_config(configurator.load!)
+      end
 
-    def status_colors
-      {
-        'idle' => '0:255:0',
-        'run' => '0:0:255',
-        'wait' => '255:125:0'
-      }
+      def configurator
+        @configurator ||= AgentStatusBulb::Configure.new
+      end
     end
   end
 end

data/lib/agent_status_bulb/configure.rb

@@ -7,44 +7,59 @@ require 'io/console'
 module AgentStatusBulb
   class Configure
     DEFAULT_PATH = File.expand_path('~/.config/agent_status_bulb.yml').freeze
+    CREDENTIAL_FIELDS = [
+      { key: :token, label: 'Token', conceal: true },
+      { key: :secret, label: 'Secret', conceal: true },
+      { key: :device_id, label: 'Device ID', conceal: false }
+    ].freeze
 
-    def initialize(path: DEFAULT_PATH, io: $stdin, out: $stdout)
+    def initialize(path = DEFAULT_PATH)
       @path = path
-      @io = io
-      @out = out
     end
 
     def configure!
-      token = prompt('Token', conceal: true)
-      secret = prompt('Secret', conceal: true)
-      device_id = prompt('Device ID')
-
-      FileUtils.mkdir_p(File.dirname(@path))
-      File.write(@path, { 'token' => token, 'secret' => secret, 'device_id' => device_id }.to_yaml)
-      @out.puts "Saved config to: #{@path}"
+      credentials = gather_credentials
+      write_config(credentials)
+      $stdout.puts "Saved config to: #{@path}"
     end
 
     def load!
-      raise "Config file not found: #{@path}" unless File.file?(@path)
+      data = read_config
+      CREDENTIAL_FIELDS.each_with_object({}) do |field, result|
+        value = data.fetch(field[:key].to_s, '').to_s.strip
+        raise missing_config_error if value.empty?
+
+        result[field[:key]] = value
+      end
+    end
 
-      data = YAML.safe_load_file(@path, permitted_classes: [], aliases: false) || {}
-      token = data['token'].to_s
-      secret = data['secret'].to_s
-      device_id = data['device_id'].to_s
+    private
 
-      if token.empty? || secret.empty? || device_id.empty?
-        raise "Config must contain token, secret, and device_id: #{@path}"
+    def gather_credentials
+      CREDENTIAL_FIELDS.each_with_object({}) do |field, result|
+        result[field[:key]] = prompt(field[:label], conceal: field[:conceal])
       end
+    end
 
-      { token: token, secret: secret, device_id: device_id }
+    def write_config(credentials)
+      FileUtils.mkdir_p(File.dirname(@path))
+      File.write(@path, credentials.transform_keys(&:to_s).to_yaml)
     end
 
-    private
+    def read_config
+      raise "Config file not found: #{@path}" unless File.file?(@path)
+
+      YAML.safe_load_file(@path, permitted_classes: [], aliases: false) || {}
+    end
+
+    def missing_config_error
+      "Config must contain token, secret, and device_id: #{@path}"
+    end
 
     def prompt(label, conceal: false)
-      @out.print "#{label}: "
-      input = conceal ? @io.noecho(&:gets) : @io.gets
-      @out.puts if conceal
+      $stdout.print "#{label}: "
+      input = conceal ? $stdin.noecho(&:gets) : $stdin.gets
+      $stdout.puts if conceal
       input.to_s.strip
     end
   end

data/lib/agent_status_bulb/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module AgentStatusBulb
-  VERSION = '0.1.0'
+  VERSION = '0.2.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: agent_status_bulb
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Yoshiki Takagi
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-11-18 00:00:00.000000000 Z
+date: 2025-11-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: switchbot
@@ -24,6 +24,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: thor
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Provides simple commands (asb run, asb wait, asb idle) to visualize the
   state of any agent or task through color changes.
 email:
@@ -42,6 +56,7 @@ files:
 - Rakefile
 - exe/asb
 - lib/agent_status_bulb.rb
+- lib/agent_status_bulb/bulb.rb
 - lib/agent_status_bulb/cli.rb
 - lib/agent_status_bulb/configure.rb
 - lib/agent_status_bulb/version.rb