agent_status_bulb 0.1.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c2183f0e22cc7a3779f409df20894bdf032f6e2c2a342c72553f838e00d27c76
-  data.tar.gz: 0def7cc82dcf2aaa15b031c9275c01da270d939841b59e7ca0a6c3b018544f94
+  metadata.gz: 926cc8a1e54b9ad64f98b0be63c4340343c8c560af83c3c1b1e1cfdd6975296b
+  data.tar.gz: 2700ad1bc86a4a5bef790c1821a3748056741bc6e3fe3e2ba76869873fdf5c86
 SHA512:
-  metadata.gz: 517a923a5b5011fd1dccce4c9e53aec9610ce278f485b148000cf99660162867ff8e53868d89e792ac7ff1b7d5feaadf709ab929bdb2000d1f0f6354124a33c4
-  data.tar.gz: 6c7cd63f36191263c4ef9b7f0527e76b672bd8cf3ce2757e8f4fbfef21f584d461161698bdf8cf95239210acbbe0f8081d9f7f70732e826b787b7ab360526d62
+  metadata.gz: fb0a80dc7f123fd52525dd0787e1273271c9ef192c3d09f96cdda78c841d7605f20c1ddbe155385a20bec2ff59410822195803b3de71ecc69a51ed7b16930d33
+  data.tar.gz: 36b57bd8fb3a83191af52dde0107a0f142da019f51eab8ba16cc0f662623f9f56aa2947ed4a6b1f4148ab8d5147d9ec3e058d2803dff09fe0af6be1f9b5a1c8a

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,18 @@
 ## [Unreleased]
 
+## [0.3.0] - 2025-11-29
+
+- Added `version` CLI command to print the current version
+- CLI commands now ignore unknown options instead of exiting with an error
+
+## [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,65 @@
 # 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
+  class Cli < Thor
+    map 'run' => :run_command
+    map 'wait' => :wait_command
+    map 'idle' => :idle_command
+    map 'off' => :off_command
 
-    def initialize(argv, io: $stdin, out: $stdout, err: $stderr)
-      @argv = argv.dup
-      @io = io
-      @out = out
-      @err = err
+    desc 'configure', 'Save Token/Secret/Device ID'
+    def configure(*_args)
+      handle_error { configurator.configure! }
     end
 
-    def run
-      command = @argv.shift
-      handle_command(command)
-    rescue StandardError => e
-      @err.puts(e.message)
-      1
+    desc 'version', 'Print the current version'
+    def version(*_args)
+      puts AgentStatusBulb::VERSION
     end
 
-    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 'run', 'Set color to running (blue)'
+    def run_command(*_args)
+      handle_error { bulb.blue }
     end
 
-    def configure!
-      configurator.configure!
-      0
+    desc 'wait', 'Set color to waiting (orange)'
+    def wait_command(*_args)
+      handle_error { bulb.orange }
     end
 
-    def apply!(command)
-      bulb = build_bulb(configurator.load!)
-      apply_status(bulb, command)
-      0
+    desc 'idle', 'Set color to idle (green)'
+    def idle_command(*_args)
+      handle_error { bulb.green }
     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 'off', 'Turn off the bulb'
+    def off_command(*_args)
+      handle_error { bulb.off }
     end
 
-    def apply_status(bulb, command)
-      return bulb.off if command == 'off'
+    no_commands do
+      def handle_error
+        yield
+      rescue StandardError => e
+        raise Thor::Error, e.message
+      end
 
-      bulb.color(status_colors.fetch(command))
-    end
-
-    def usage
-      "Usage: #{File.basename($PROGRAM_NAME)} [run|wait|idle|off|configure]"
-    end
+      def bulb
+        @bulb ||= AgentStatusBulb::Bulb.from_config(configurator.load!)
+      end
 
-    def configurator
-      @configurator ||= AgentStatusBulb::Configure.new(io: @io, out: @out)
+      def configurator
+        @configurator ||= AgentStatusBulb::Configure.new
+      end
     end
 
-    def status_colors
-      {
-        'idle' => '0:255:0',
-        'run' => '0:0:255',
-        'wait' => '255:125:0'
-      }
+    def self.exit_on_failure?
+      true
     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.3.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.3.0
 platform: ruby
 authors:
 - Yoshiki Takagi
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-11-18 00:00:00.000000000 Z
+date: 2025-11-29 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