factorix 0.5.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 266f397cb572fd4db54fe30671c9ba4e8ba3f60a4b17593193a829ae69b526c0
-  data.tar.gz: aeecbbf8447e8823d2d374ad13899581f51060eda1ec95959f672b883e60c8a4
+  metadata.gz: 1eb581275b20274cebfa09669eb5f0111ed00f358d5a68947333b5990eb04d56
+  data.tar.gz: 2a57a400c94476e860c2647af1e8fb3f845b231b1e377e265bfcc1d1de11d7ef
 SHA512:
-  metadata.gz: 2c28e713c148c4ebb43f03094713c9b02a8c038dd64a8d6da6d69c5a09f43f1dcee7430e67a9b92be5d784d26f3986c25bf7e9d9cfd5ba3402469b0d0649723b
-  data.tar.gz: cec4b9103df79b6f2e94bb15599d4306346378030a18a3ea9480c27a7cf6004a62f7c2d2d6fa7c3c0d7416af0e4ebd5adeee5df2d281b2d68d09f9ce5d52c675
+  metadata.gz: 9134480ff3cd2675216f6fc9407bb0719467c2488a43449c727294f5871289cb955b6b5a20257314e11d562a60e2eea8edf81bf72c8df739055675142dff8ad5
+  data.tar.gz: 15630f358ac2dbe3ae85b3a0798fdea532b7d9cba43352ffc982c74cf6e5eea1a264329f9543c26100618c7d3e45c2b6cc8a69bf5aca203333a2c77157b68571

data/CHANGELOG.md

@@ -1,5 +1,30 @@
 ## [Unreleased]
 
+## [0.6.0] - 2026-01-18
+
+### Changed
+
+- Reorganize configuration and DI container interfaces (#7, #9)
+  - `Factorix::Application` renamed to `Factorix::Container` (DI container only)
+  - Configuration interface (`config`, `configure`, `load_config`) moved to `Factorix` module
+  - Use `Factorix.configure { |c| ... }` instead of `Factorix::Container.configure { |c| ... }`
+
+### Fixed
+
+- Fix integer CLI options being parsed as strings after dry-cli 1.4.0 update (#12)
+
+### Deprecated
+
+- `Factorix::Application` still works but emits deprecation warnings; will be removed in v1.0
+  - DI methods (`[]`, `resolve`, `register`) delegate to `Factorix::Container`
+  - Configuration methods (`config`, `configure`) delegate to `Factorix`
+
+## [0.5.1] - 2026-01-13
+
+### Fixed
+
+- Fix `completion` command failing without arguments after dry-cli 1.4.0 update (#4)
+
 ## [0.5.0] - 2025-12-26
 
 ### Added

data/README.md

@@ -65,7 +65,7 @@ $EDITOR ~/.config/factorix/config.rb
 
 **Example configuration:**
 ```ruby
-Factorix::Application.configure do |config|
+Factorix.configure do |config|
   config.runtime.executable_path = "/Applications/Factorio.app/Contents/MacOS/factorio"
   config.runtime.user_dir = "#{Dir.home}/Library/Application Support/factorio"
   config.runtime.data_dir = "/Applications/Factorio.app/Contents/data"

data/lib/factorix/api/mod_download_api.rb

@@ -51,7 +51,7 @@ module Factorix
         return @service_credential if defined?(@service_credential)
 
         @service_credential_mutex.synchronize do
-          @service_credential ||= Application[:service_credential]
+          @service_credential ||= Container[:service_credential]
         end
       end
 

data/lib/factorix/api/mod_info.rb

@@ -114,7 +114,7 @@ module Factorix
 
           URI(value)
         rescue URI::InvalidURIError => e
-          Application[:logger].warn("Skipping invalid URI '#{value}': #{e.message}")
+          Container[:logger].warn("Skipping invalid URI '#{value}': #{e.message}")
           nil
         end
       end
@@ -145,7 +145,7 @@ module Factorix
             Release[**r]
           rescue RangeError => e
             # Skip releases with invalid version numbers
-            Application[:logger].warn("Skipping release #{name}@#{r[:version]}: #{e.message}")
+            Container[:logger].warn("Skipping release #{name}@#{r[:version]}: #{e.message}")
             nil
           end
         }

data/lib/factorix/api/mod_management_api.rb

@@ -208,7 +208,7 @@ module Factorix
         return @api_credential if defined?(@api_credential)
 
         @api_credential_mutex.synchronize do
-          @api_credential ||= Application[:api_credential]
+          @api_credential ||= Container[:api_credential]
         end
       end
 

data/lib/factorix/api_credential.rb

@@ -22,7 +22,7 @@ module Factorix
     # @return [APICredential] new instance with API key from environment
     # @raise [CredentialError] if API key is not set in environment
     def self.load
-      logger = Application["logger"]
+      logger = Container["logger"]
       logger.debug "Loading API credentials from environment"
 
       api_key = ENV.fetch(ENV_API_KEY, nil)

data/lib/factorix/cli/commands/backup_support.rb

@@ -15,7 +15,7 @@ module Factorix
         # @param base [Class] the class prepending this module
         def self.prepended(base)
           base.class_eval do
-            option :backup_extension, type: :string, default: ".bak", desc: "Backup file extension"
+            option :backup_extension, default: ".bak", desc: "Backup file extension"
           end
         end
 

data/lib/factorix/cli/commands/base.rb

@@ -68,8 +68,8 @@ module Factorix
         def self.backup_support! = prepend BackupSupport
 
         # Common options available to all commands
-        option :config_path, type: :string, aliases: ["-c"], desc: "Path to configuration file"
-        option :log_level, type: :string, values: %w[debug info warn error fatal], desc: "Set log level"
+        option :config_path, aliases: ["-c"], desc: "Path to configuration file"
+        option :log_level, values: %w[debug info warn error fatal], desc: "Set log level"
         option :quiet, type: :flag, default: false, aliases: ["-q"], desc: "Suppress non-essential output"
 
         private def say(message, prefix: "")
@@ -78,7 +78,7 @@ module Factorix
           resolved_prefix = EMOJI_PREFIXES.fetch(prefix) { prefix.to_s }
           output = resolved_prefix.empty? ? message : "#{resolved_prefix} #{message}"
           style = STYLES.fetch(prefix, PLAIN)
-          puts style[output]
+          out.puts style[output]
         end
 
         private def quiet?

data/lib/factorix/cli/commands/cache/evict.rb

@@ -36,7 +36,7 @@ module Factorix
 
           option :all, type: :flag, default: false, desc: "Remove all entries"
           option :expired, type: :flag, default: false, desc: "Remove expired entries only"
-          option :older_than, type: :string, default: nil, desc: "Remove entries older than AGE (e.g., 30s, 5m, 2h, 7d)"
+          option :older_than, default: nil, desc: "Remove entries older than AGE (e.g., 30s, 5m, 2h, 7d)"
 
           # Execute the cache evict command
           #
@@ -95,7 +95,7 @@ module Factorix
           # @return [Array<Symbol>] resolved cache names
           # @raise [InvalidArgumentError] if unknown cache name specified
           private def resolve_cache_names(caches)
-            all_caches = Application.config.cache.values.keys
+            all_caches = Factorix.config.cache.values.keys
 
             return all_caches if caches.nil? || caches.empty?
 
@@ -114,7 +114,7 @@ module Factorix
           # @param expired [Boolean] remove expired entries only
           # @return [Hash] eviction result with :count and :size
           private def evict_cache(name, all:, expired:)
-            config = Application.config.cache.public_send(name)
+            config = Factorix.config.cache.public_send(name)
             cache_dir = config.dir
             ttl = config.ttl
 

data/lib/factorix/cli/commands/cache/stat.rb

@@ -8,7 +8,7 @@ module Factorix
       module Cache
         # Display cache statistics
         #
-        # This command outputs statistics for all cache stores
+        # This command outout.puts statistics for all cache stores
         # in a human-readable or JSON format.
         #
         # @example
@@ -42,18 +42,18 @@ module Factorix
             logger.debug("Collecting cache statistics")
 
             @now = Time.now
-            cache_names = Application.config.cache.values.keys
+            cache_names = Factorix.config.cache.values.keys
             stats = cache_names.to_h {|name| [name, collect_stats(name)] }
 
             if json
-              puts JSON.pretty_generate(stats)
+              out.puts JSON.pretty_generate(stats)
             else
               output_text(stats)
             end
           end
 
           private def collect_stats(name)
-            config = Application.config.cache.public_send(name)
+            config = Factorix.config.cache.public_send(name)
             cache_dir = config.dir
 
             entries = scan_entries(cache_dir, config.ttl)
@@ -142,8 +142,8 @@ module Factorix
           # @return [void]
           private def output_text(stats)
             stats.each_with_index do |(name, data), index|
-              puts if index > 0
-              puts "#{name}:"
+              out.puts if index > 0
+              out.puts "#{name}:"
               output_cache_stats(data)
             end
           end
@@ -153,26 +153,26 @@ module Factorix
           # @param data [Hash] cache statistics
           # @return [void]
           private def output_cache_stats(data)
-            puts "  Directory:      #{data[:directory]}"
-            puts "  TTL:            #{format_ttl(data[:ttl])}"
-            puts "  Max file size:  #{format_size(data[:max_file_size])}"
-            puts "  Compression:    #{format_compression(data[:compression_threshold])}"
+            out.puts "  Directory:      #{data[:directory]}"
+            out.puts "  TTL:            #{format_ttl(data[:ttl])}"
+            out.puts "  Max file size:  #{format_size(data[:max_file_size])}"
+            out.puts "  Compression:    #{format_compression(data[:compression_threshold])}"
 
             entries = data[:entries]
             valid_pct = entries[:total] > 0 ? (Float(entries[:valid]) / entries[:total] * 100) : 0.0
-            puts "  Entries:        #{entries[:valid]} / #{entries[:total]} (#{"%.1f" % valid_pct}% valid)"
+            out.puts "  Entries:        #{entries[:valid]} / #{entries[:total]} (#{"%.1f" % valid_pct}% valid)"
 
             size = data[:size]
-            puts "  Size:           #{format_size(size[:total])} (avg #{format_size(size[:avg])})"
+            out.puts "  Size:           #{format_size(size[:total])} (avg #{format_size(size[:avg])})"
 
             age = data[:age]
             if age[:oldest]
-              puts "  Age:            #{format_duration(age[:newest])} - #{format_duration(age[:oldest])} (avg #{format_duration(age[:avg])})"
+              out.puts "  Age:            #{format_duration(age[:newest])} - #{format_duration(age[:oldest])} (avg #{format_duration(age[:avg])})"
             else
-              puts "  Age:            -"
+              out.puts "  Age:            -"
             end
 
-            puts "  Stale locks:    #{data[:stale_locks]}"
+            out.puts "  Stale locks:    #{data[:stale_locks]}"
           end
 
           # Format TTL value for display

data/lib/factorix/cli/commands/command_wrapper.rb

@@ -23,14 +23,14 @@ module Factorix
           super
         rescue Error => e
           # Expected errors (validation failures, missing dependencies, etc.)
-          log = Application[:logger]
+          log = Container[:logger]
           log.warn(e.message)
           log.debug(e)
           say "Error: #{e.message}", prefix: :error unless @quiet
           raise # Re-raise for exe/factorix to handle exit code
         rescue => e
           # Unexpected errors (bugs, system failures, etc.)
-          log = Application[:logger]
+          log = Container[:logger]
           log.error(e)
           say "Unexpected error: #{e.message}", prefix: :error unless @quiet
           raise # Re-raise for exe/factorix to handle exit code
@@ -40,7 +40,7 @@ module Factorix
           path = resolve_config_path(explicit_path)
           return unless path
 
-          Application.load_config(path)
+          Factorix.load_config(path)
         end
 
         # Resolves which config path to use
@@ -50,14 +50,14 @@ module Factorix
           return Pathname(explicit_path) if explicit_path
           return Pathname(ENV.fetch("FACTORIX_CONFIG")) if ENV["FACTORIX_CONFIG"]
 
-          default_path = Application[:runtime].factorix_config_path
+          default_path = Container[:runtime].factorix_config_path
           default_path.exist? ? default_path : nil
         end
 
         # Sets the application logger's level
         # @param level [String] log level (debug, info, warn, error, fatal)
         private def log_level!(level)
-          logger = Application[:logger]
+          logger = Container[:logger]
           level_constant = Logger.const_get(level.upcase)
 
           # Change only the File backend (first backend) level

data/lib/factorix/cli/commands/completion.rb

@@ -28,9 +28,8 @@ module Factorix
         desc "Generate shell completion script"
 
         argument :shell,
-          type: :string,
           required: false,
-          values: SUPPORTED_SHELLS.keys,
+          values: [nil] + SUPPORTED_SHELLS.keys,
           desc: "Shell type. Defaults to current shell from $SHELL"
 
         example [
@@ -53,7 +52,7 @@ module Factorix
           script_path = COMPLETION_DIR / SUPPORTED_SHELLS[shell_type]
           raise ConfigurationError, "#{shell_type.capitalize} completion script not found" unless script_path.exist?
 
-          puts script_path.read
+          out.puts script_path.read
         end
 
         # Detect shell type from SHELL environment variable

data/lib/factorix/cli/commands/confirmable.rb

@@ -41,7 +41,7 @@ module Factorix
             raise InvalidOperationError, "Cannot prompt for confirmation in quiet mode. Use --yes to proceed automatically."
           end
 
-          print "#{message} [y/N] "
+          out.print "#{message} [y/N] "
           response = $stdin.gets&.strip&.downcase
 
           # Only explicit y or yes means yes (default is no for safety)

data/lib/factorix/cli/commands/download_support.rb

@@ -91,13 +91,13 @@ module Factorix
         # @param jobs [Integer] Number of parallel downloads
         # @return [void]
         private def download_mods(targets, jobs)
-          multi_presenter = Progress::MultiPresenter.new(title: "\u{1F4E5}\u{FE0E} Downloads")
+          multi_presenter = Progress::MultiPresenter.new(title: "\u{1F4E5}\u{FE0E} Downloads", output: err)
 
           pool = Concurrent::FixedThreadPool.new(jobs)
 
           futures = targets.map {|target|
             Concurrent::Future.execute(executor: pool) do
-              thread_portal = Application[:portal]
+              thread_portal = Container[:portal]
               thread_downloader = thread_portal.mod_download_api.downloader
 
               presenter = multi_presenter.register(

data/lib/factorix/cli/commands/mod/check.rb

@@ -24,7 +24,7 @@ module Factorix
           # @return [void]
           def call(**)
             mod_list = MODList.load
-            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Scanning MOD(s)", output: $stderr)
+            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Scanning MOD(s)", output: err)
             handler = Progress::ScanHandler.new(presenter)
             installed_mods = InstalledMOD.all(handler:)
             graph = Dependency::Graph::Builder.build(installed_mods:, mod_list:)

data/lib/factorix/cli/commands/mod/disable.rb

@@ -39,7 +39,7 @@ module Factorix
 
             # Without validation to allow fixing issues
             mod_list = MODList.load
-            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Scanning MOD(s)", output: $stderr)
+            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Scanning MOD(s)", output: err)
             handler = Progress::ScanHandler.new(presenter)
             installed_mods = InstalledMOD.all(handler:)
             graph = Dependency::Graph::Builder.build(installed_mods:, mod_list:)

data/lib/factorix/cli/commands/mod/download.rb

@@ -29,8 +29,8 @@ module Factorix
           ]
 
           argument :mod_specs, type: :array, required: true, desc: "MOD specifications (name@version or name@latest or name)"
-          option :directory, type: :string, aliases: ["-d"], default: ".", desc: "Download directory"
-          option :jobs, type: :integer, aliases: ["-j"], default: 4, desc: "Number of parallel downloads"
+          option :directory, aliases: ["-d"], default: ".", desc: "Download directory"
+          option :jobs, aliases: ["-j"], default: "4", desc: "Number of parallel downloads"
           option :recursive, type: :flag, aliases: ["-r"], default: false, desc: "Include required dependencies recursively"
 
           # Execute the download command
@@ -40,7 +40,8 @@ module Factorix
           # @param jobs [Integer] Number of parallel downloads
           # @param recursive [Boolean] Include required dependencies recursively
           # @return [void]
-          def call(mod_specs:, directory: ".", jobs: 4, recursive: false, **)
+          def call(mod_specs:, directory: ".", jobs: "4", recursive: false, **)
+            jobs = Integer(jobs)
             download_dir = Pathname(directory).expand_path
 
             raise DirectoryNotFoundError, "Download directory does not exist: #{download_dir}" unless download_dir.exist?
@@ -69,7 +70,7 @@ module Factorix
           # @param recursive [Boolean] Include dependencies
           # @return [Array<Hash>] Download targets with MOD info and releases
           private def plan_download(mod_specs, download_dir, jobs, recursive)
-            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Fetching MOD info", output: $stderr)
+            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Fetching MOD info", output: err)
 
             target_infos = fetch_target_mod_info(mod_specs, jobs, presenter)
 

data/lib/factorix/cli/commands/mod/edit.rb

@@ -19,16 +19,16 @@ module Factorix
             "some-mod --deprecated                # Mark as deprecated"
           ]
 
-          argument :mod_name, type: :string, required: true, desc: "MOD name"
-          option :description, type: :string, desc: "Markdown description"
-          option :summary, type: :string, desc: "Brief description"
-          option :title, type: :string, desc: "MOD title"
-          option :category, type: :string, desc: "MOD category"
+          argument :mod_name, required: true, desc: "MOD name"
+          option :description, desc: "Markdown description"
+          option :summary, desc: "Brief description"
+          option :title, desc: "MOD title"
+          option :category, desc: "MOD category"
           option :tags, type: :array, desc: "Array of tags"
-          option :license, type: :string, desc: "License identifier"
-          option :homepage, type: :string, desc: "Homepage URL"
-          option :source_url, type: :string, desc: "Repository URL"
-          option :faq, type: :string, desc: "FAQ text"
+          option :license, desc: "License identifier"
+          option :homepage, desc: "Homepage URL"
+          option :source_url, desc: "Repository URL"
+          option :faq, desc: "FAQ text"
           option :deprecated, type: :boolean, desc: "Deprecation flag"
 
           # Execute the edit command

data/lib/factorix/cli/commands/mod/enable.rb

@@ -33,7 +33,7 @@ module Factorix
           def call(mod_names:, **)
             # Load current state (without validation to allow fixing issues)
             mod_list = MODList.load
-            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Scanning MOD(s)", output: $stderr)
+            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Scanning MOD(s)", output: err)
             handler = Progress::ScanHandler.new(presenter)
             installed_mods = InstalledMOD.all(handler:)
             graph = Dependency::Graph::Builder.build(installed_mods:, mod_list:)

data/lib/factorix/cli/commands/mod/image/add.rb

@@ -18,8 +18,8 @@ module Factorix
               "some-mod screenshot.png   # Add image to MOD"
             ]
 
-            argument :mod_name, type: :string, required: true, desc: "MOD name"
-            argument :image_file, type: :string, required: true, desc: "Path to image file"
+            argument :mod_name, required: true, desc: "MOD name"
+            argument :image_file, required: true, desc: "Path to image file"
 
             # Execute the add command
             #

data/lib/factorix/cli/commands/mod/image/edit.rb

@@ -18,7 +18,7 @@ module Factorix
               "some-mod abc123 def456   # Set image order (IDs from 'image list')"
             ]
 
-            argument :mod_name, type: :string, required: true, desc: "MOD name"
+            argument :mod_name, required: true, desc: "MOD name"
             argument :image_ids, type: :array, required: true, desc: "Image IDs in desired order"
 
             # Execute the edit command

data/lib/factorix/cli/commands/mod/image/list.rb

@@ -19,7 +19,7 @@ module Factorix
               "some-mod --json   # List images in JSON format"
             ]
 
-            argument :mod_name, type: :string, required: true, desc: "MOD name"
+            argument :mod_name, required: true, desc: "MOD name"
 
             option :json, type: :flag, default: false, desc: "Output in JSON format"
 
@@ -45,7 +45,7 @@ module Factorix
                        end
 
               if json
-                puts JSON.pretty_generate(images)
+                out.puts JSON.pretty_generate(images)
               else
                 output_table(images)
               end
@@ -60,10 +60,10 @@ module Factorix
               id_width = [images.map {|i| i[:id].length }.max, 2].max
               thumb_width = [images.map {|i| i[:thumbnail].length }.max, 9].max
 
-              puts "%-#{id_width}s  %-#{thumb_width}s  %s" % %w[ID THUMBNAIL URL]
+              out.puts "%-#{id_width}s  %-#{thumb_width}s  %s" % %w[ID THUMBNAIL URL]
 
               images.each do |image|
-                puts "%-#{id_width}s  %-#{thumb_width}s  %s" % [image[:id], image[:thumbnail], image[:url]]
+                out.puts "%-#{id_width}s  %-#{thumb_width}s  %s" % [image[:id], image[:thumbnail], image[:url]]
               end
             end
           end

data/lib/factorix/cli/commands/mod/install.rb

@@ -34,17 +34,18 @@ module Factorix
           ]
 
           argument :mod_specs, type: :array, required: true, desc: "MOD specifications (name@version or name@latest or name)"
-          option :jobs, type: :integer, aliases: ["-j"], default: 4, desc: "Number of parallel downloads"
+          option :jobs, aliases: ["-j"], default: "4", desc: "Number of parallel downloads"
 
           # Execute the install command
           #
           # @param mod_specs [Array<String>] MOD specifications
           # @param jobs [Integer] Number of parallel downloads
           # @return [void]
-          def call(mod_specs:, jobs: 4, **)
+          def call(mod_specs:, jobs: "4", **)
+            jobs = Integer(jobs)
             # Load current state (without validation to allow fixing issues)
             mod_list = MODList.load
-            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Scanning MOD(s)", output: $stderr)
+            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Scanning MOD(s)", output: err)
             handler = Progress::ScanHandler.new(presenter)
             installed_mods = InstalledMOD.all(handler:)
             graph = Dependency::Graph::Builder.build(installed_mods:, mod_list:)
@@ -128,7 +129,7 @@ module Factorix
           # @return [Array<Hash>] Installation targets with MOD info and releases
           private def plan_installation(mod_specs, graph, jobs)
             # Create progress presenter for info fetching
-            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Fetching MOD info", output: $stderr)
+            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Fetching MOD info", output: err)
 
             # Phase 1: Fetch info for target MODs
             target_infos = fetch_target_mod_info(mod_specs, jobs, presenter)

data/lib/factorix/cli/commands/mod/list.rb

@@ -80,7 +80,7 @@ module Factorix
             validate_filter_options!(enabled:, disabled:, errors:, outdated:)
 
             mod_list = MODList.load
-            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Scanning MOD(s)", output: $stderr)
+            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Scanning MOD(s)", output: err)
             handler = Progress::ScanHandler.new(presenter)
             installed_mods = InstalledMOD.all(handler:)
             graph = Dependency::Graph::Builder.build(installed_mods:, mod_list:)
@@ -226,7 +226,7 @@ module Factorix
             }
 
             # Only show progress for MOD(s) that need API calls
-            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Checking for updates", output: $stderr)
+            presenter = Progress::Presenter.new(title: "\u{1F50D}\u{FE0E} Checking for updates", output: err)
             presenter.start(total: regular_mods.size)
 
             pool = Concurrent::FixedThreadPool.new(DEFAULT_JOBS)
@@ -307,19 +307,19 @@ module Factorix
               latest_width = [mod_infos.map {|m| m.latest_version&.to_s&.length || 0 }.max, 6].max
 
               # Header with LATEST column
-              puts "%-#{name_width}s  %-#{version_width}s  %-#{latest_width}s  %s" % %w[NAME VERSION LATEST STATUS]
+              out.puts "%-#{name_width}s  %-#{version_width}s  %-#{latest_width}s  %s" % %w[NAME VERSION LATEST STATUS]
 
               # Rows with LATEST column
               mod_infos.each do |info|
-                puts "%-#{name_width}s  %-#{version_width}s  %-#{latest_width}s  %s" % [info.name, info.version, info.latest_version, info.status]
+                out.puts "%-#{name_width}s  %-#{version_width}s  %-#{latest_width}s  %s" % [info.name, info.version, info.latest_version, info.status]
               end
             else
               # Header
-              puts "%-#{name_width}s  %-#{version_width}s  %s" % %w[NAME VERSION STATUS]
+              out.puts "%-#{name_width}s  %-#{version_width}s  %s" % %w[NAME VERSION STATUS]
 
               # Rows
               mod_infos.each do |info|
-                puts "%-#{name_width}s  %-#{version_width}s  %s" % [info.name, info.version, info.status]
+                out.puts "%-#{name_width}s  %-#{version_width}s  %s" % [info.name, info.version, info.status]
               end
             end
 
@@ -363,7 +363,7 @@ module Factorix
               end
             }
 
-            puts JSON.pretty_generate(data)
+            out.puts JSON.pretty_generate(data)
           end
         end
       end

data/lib/factorix/cli/commands/mod/search.rb

@@ -29,11 +29,11 @@ module Factorix
           argument :mod_names, type: :array, required: false, default: [], desc: "MOD names to search"
 
           option :hide_deprecated, type: :boolean, default: true, desc: "Hide deprecated MOD(s)"
-          option :page, type: :integer, default: 1, desc: "Page number"
-          option :page_size, type: :integer, default: 25, desc: "Results per page (max 500)"
-          option :sort, type: :string, values: %w[name created_at updated_at], desc: "Sort field"
-          option :sort_order, type: :string, values: %w[asc desc], desc: "Sort order"
-          option :version, type: :string, desc: "Filter by Factorio version (default: installed version)"
+          option :page, default: "1", desc: "Page number"
+          option :page_size, default: "25", desc: "Results per page (max 500)"
+          option :sort, values: %w[name created_at updated_at], desc: "Sort field"
+          option :sort_order, values: %w[asc desc], desc: "Sort order"
+          option :version, desc: "Filter by Factorio version (default: installed version)"
           option :json, type: :flag, default: false, desc: "Output in JSON format"
 
           # Execute the search command
@@ -47,7 +47,9 @@ module Factorix
           # @param version [String, nil] Factorio version filter
           # @param json [Boolean] Output in JSON format
           # @return [void]
-          def call(mod_names: [], hide_deprecated: true, page: 1, page_size: 25, sort: nil, sort_order: nil, version: nil, json: false, **)
+          def call(mod_names: [], hide_deprecated: true, page: "1", page_size: "25", sort: nil, sort_order: nil, version: nil, json: false, **)
+            page = Integer(page)
+            page_size = Integer(page_size)
             version ||= default_factorio_version
 
             mods = portal.list_mods(*mod_names, hide_deprecated: hide_deprecated || nil, page:, page_size:, sort:, sort_order:, version:)
@@ -60,7 +62,7 @@ module Factorix
           end
 
           private def output_json(mods)
-            puts JSON.pretty_generate(mods.map {|mod| mod_to_hash(mod) })
+            out.puts JSON.pretty_generate(mods.map {|mod| mod_to_hash(mod) })
           end
 
           private def mod_to_hash(mod)
@@ -99,10 +101,10 @@ module Factorix
             headers = %w[NAME TITLE CATEGORY OWNER LATEST]
             widths = headers.map.with_index {|h, i| [h.length, *rows.map {|r| r[i].to_s.length }].max }
 
-            puts format_table_row(headers, widths)
+            out.puts format_table_row(headers, widths)
 
             rows.each do |row|
-              puts format_table_row(row, widths)
+              out.puts format_table_row(row, widths)
             end
 
             say "#{mods.size} MOD(s) found", prefix: :info

data/lib/factorix/cli/commands/mod/settings/dump.rb

@@ -22,8 +22,8 @@ module Factorix
               "/path/to/mod-settings.dat -o out.json   # Dump specific file"
             ]
 
-            argument :settings_file, type: :string, required: false, desc: "Path to mod-settings.dat file"
-            option :output, type: :string, aliases: ["-o"], desc: "Output file path"
+            argument :settings_file, required: false, desc: "Path to mod-settings.dat file"
+            option :output, aliases: ["-o"], desc: "Output file path"
 
             # Execute the dump command
             #
@@ -43,7 +43,7 @@ module Factorix
               if output
                 Pathname(output).write(output_string)
               else
-                puts output_string
+                out.puts output_string
               end
             end
 

data/lib/factorix/cli/commands/mod/settings/restore.rb

@@ -24,8 +24,8 @@ module Factorix
               "                        # Restore from stdin"
             ]
 
-            argument :settings_file, type: :string, required: false, desc: "Path to mod-settings.dat file to write"
-            option :input, type: :string, aliases: ["-i"], desc: "Input file path"
+            argument :settings_file, required: false, desc: "Path to mod-settings.dat file to write"
+            option :input, aliases: ["-i"], desc: "Input file path"
 
             # Execute the restore command
             #