serialbench 0.1.2 → 0.2.0

data/lib/serialbench/cli/ruby_build_cli.rb

@@ -32,7 +32,6 @@ module Serialbench
             say "\n📋 Recent Ruby versions available:", :white
             recent_versions.each { |version| say "  #{version}", :cyan }
           end
-
         rescue StandardError => e
           say "❌ Failed to update Ruby-Build definitions: #{e.message}", :red
           exit 1
@@ -52,42 +51,39 @@ module Serialbench
       DESC
       option :limit, type: :numeric, default: 50, desc: 'Maximum number of definitions to show'
       def list(filter = nil)
-        begin
-          definitions = RubyBuildManager.list_definitions(filter: filter)
-
-          if definitions.empty?
-            if filter
-              say "No Ruby-Build definitions found matching '#{filter}'", :yellow
-            else
-              say 'No Ruby-Build definitions found in cache', :yellow
-              say 'Update the cache first: serialbench ruby-build update', :white
-            end
-            return
-          end
-
-          # Limit results if there are many
-          limited_definitions = definitions.first(options[:limit])
+        definitions = RubyBuildManager.list_definitions(filter: filter)
 
-          say "Ruby-Build Definitions#{filter ? " (filtered by '#{filter}')" : ''}:", :green
-          say '=' * 60, :green
-
-          limited_definitions.each do |definition|
-            say "  #{definition}", :cyan
+        if definitions.empty?
+          if filter
+            say "No Ruby-Build definitions found matching '#{filter}'", :yellow
+          else
+            say 'No Ruby-Build definitions found in cache', :yellow
+            say 'Update the cache first: serialbench ruby-build update', :white
           end
+          return
+        end
 
-          if definitions.length > options[:limit]
-            remaining = definitions.length - options[:limit]
-            say "\n... and #{remaining} more definitions", :yellow
-            say "Use --limit to show more results", :white
-          end
+        # Limit results if there are many
+        limited_definitions = definitions.first(options[:limit])
 
-          say "\nTotal: #{definitions.length} definitions", :white
+        say "Ruby-Build Definitions#{filter ? " (filtered by '#{filter}')" : ''}:", :green
+        say '=' * 60, :green
 
-        rescue StandardError => e
-          say "❌ Failed to list Ruby-Build definitions: #{e.message}", :red
-          say 'Try updating the cache: serialbench ruby-build update', :white
-          exit 1
+        limited_definitions.each do |definition|
+          say "  #{definition}", :cyan
+        end
+
+        if definitions.length > options[:limit]
+          remaining = definitions.length - options[:limit]
+          say "\n... and #{remaining} more definitions", :yellow
+          say 'Use --limit to show more results', :white
         end
+
+        say "\nTotal: #{definitions.length} definitions", :white
+      rescue StandardError => e
+        say "❌ Failed to list Ruby-Build definitions: #{e.message}", :red
+        say 'Try updating the cache: serialbench ruby-build update', :white
+        exit 1
       end
 
       desc 'show TAG', 'Show details for a specific Ruby-Build definition'
@@ -99,21 +95,18 @@ module Serialbench
           serialbench ruby-build show 3.2.4
       DESC
       def show(tag)
-        begin
-          definition = RubyBuildManager.show_definition(tag)
-
-          say "Ruby-Build Definition: #{tag}", :green
-          say '=' * 40, :green
-          say "Tag: #{definition[:tag]}", :cyan
-          say "Available: #{definition[:available] ? '✅ Yes' : '❌ No'}", :cyan
-          say "Source: #{definition[:source]}", :cyan
-          say "Cache file: #{definition[:cache_file]}", :white
-
-        rescue StandardError => e
-          say "❌ #{e.message}", :red
-          say 'Available definitions: serialbench ruby-build list', :white
-          exit 1
-        end
+        definition = RubyBuildManager.show_definition(tag)
+
+        say "Ruby-Build Definition: #{tag}", :green
+        say '=' * 40, :green
+        say "Tag: #{definition[:tag]}", :cyan
+        say "Available: #{definition[:available] ? '✅ Yes' : '❌ No'}", :cyan
+        say "Source: #{definition[:source]}", :cyan
+        say "Cache file: #{definition[:cache_file]}", :white
+      rescue StandardError => e
+        say "❌ #{e.message}", :red
+        say 'Available definitions: serialbench ruby-build list', :white
+        exit 1
       end
 
       desc 'validate TAG', 'Validate a Ruby-Build tag'
@@ -125,31 +118,28 @@ module Serialbench
           serialbench ruby-build validate 3.2.4
       DESC
       def validate(tag)
-        begin
-          valid = RubyBuildManager.validate_tag(tag)
-
-          if valid
-            say "✅ Ruby-Build tag '#{tag}' is valid", :green
-          else
-            say "❌ Ruby-Build tag '#{tag}' is not valid", :red
+        valid = RubyBuildManager.validate_tag(tag)
 
-            # Suggest similar tags
-            definitions = RubyBuildManager.list_definitions
-            similar = definitions.select { |d| d.include?(tag.split('.').first(2).join('.')) }.first(5)
+        if valid
+          say "✅ Ruby-Build tag '#{tag}' is valid", :green
+        else
+          say "❌ Ruby-Build tag '#{tag}' is not valid", :red
 
-            if similar.any?
-              say "\n💡 Similar available tags:", :yellow
-              similar.each { |s| say "  #{s}", :cyan }
-            end
+          # Suggest similar tags
+          definitions = RubyBuildManager.list_definitions
+          similar = definitions.select { |d| d.include?(tag.split('.').first(2).join('.')) }.first(5)
 
-            exit 1
+          if similar.any?
+            say "\n💡 Similar available tags:", :yellow
+            similar.each { |s| say "  #{s}", :cyan }
           end
 
-        rescue StandardError => e
-          say "❌ Failed to validate tag: #{e.message}", :red
-          say 'Try updating the cache: serialbench ruby-build update', :white
           exit 1
         end
+      rescue StandardError => e
+        say "❌ Failed to validate tag: #{e.message}", :red
+        say 'Try updating the cache: serialbench ruby-build update', :white
+        exit 1
       end
 
       desc 'suggest', 'Suggest Ruby-Build tag for current Ruby version'
@@ -163,26 +153,23 @@ module Serialbench
           serialbench ruby-build suggest
       DESC
       def suggest
-        begin
-          current_ruby = RUBY_VERSION
-          suggested_tag = RubyBuildManager.suggest_current_ruby_tag
+        current_ruby = RUBY_VERSION
+        suggested_tag = RubyBuildManager.suggest_current_ruby_tag
 
-          say "Current Ruby version: #{current_ruby}", :cyan
-          say "Suggested ruby_build_tag: #{suggested_tag}", :green
-
-          # Validate the suggestion
-          if RubyBuildManager.validate_tag(suggested_tag)
-            say "✅ Suggested tag is valid", :green
-          else
-            say "⚠️  Suggested tag not found in ruby-build definitions", :yellow
-            say "You may need to update the cache or use a different tag", :white
-          end
+        say "Current Ruby version: #{current_ruby}", :cyan
+        say "Suggested ruby_build_tag: #{suggested_tag}", :green
 
-        rescue StandardError => e
-          say "❌ Failed to suggest tag: #{e.message}", :red
-          say 'Try updating the cache: serialbench ruby-build update', :white
-          exit 1
+        # Validate the suggestion
+        if RubyBuildManager.validate_tag(suggested_tag)
+          say '✅ Suggested tag is valid', :green
+        else
+          say '⚠️  Suggested tag not found in ruby-build definitions', :yellow
+          say 'You may need to update the cache or use a different tag', :white
         end
+      rescue StandardError => e
+        say "❌ Failed to suggest tag: #{e.message}", :red
+        say 'Try updating the cache: serialbench ruby-build update', :white
+        exit 1
       end
 
       desc 'cache-info', 'Show information about the Ruby-Build definitions cache'
@@ -203,19 +190,19 @@ module Serialbench
           say "Location: #{RubyBuildManager::CACHE_FILE}", :cyan
           say "Definitions: #{definitions_count}", :cyan
           say "Age: #{format_cache_age(cache_age)}", :cyan
-          say "Status: ✅ Available", :green
+          say 'Status: ✅ Available', :green
 
           if cache_age > 7 * 24 * 60 * 60 # 7 days
             say "\n💡 Cache is older than 7 days, consider updating:", :yellow
-            say "  serialbench ruby-build update", :white
+            say '  serialbench ruby-build update', :white
           end
         else
           say 'Ruby-Build Cache Information:', :green
           say '=' * 40, :green
           say "Location: #{RubyBuildManager::CACHE_FILE}", :cyan
-          say "Status: ❌ Not found", :red
+          say 'Status: ❌ Not found', :red
           say "\n📥 Update the cache first:", :yellow
-          say "  serialbench ruby-build update", :white
+          say '  serialbench ruby-build update', :white
         end
       end
 
@@ -225,12 +212,12 @@ module Serialbench
         days = (seconds / (24 * 60 * 60)).to_i
         hours = ((seconds % (24 * 60 * 60)) / (60 * 60)).to_i
 
-        if days > 0
+        if days.positive?
           "#{days} day#{'s' if days != 1}, #{hours} hour#{'s' if hours != 1}"
-        elsif hours > 0
+        elsif hours.positive?
           "#{hours} hour#{'s' if hours != 1}"
         else
-          "less than 1 hour"
+          'less than 1 hour'
         end
       end
     end

data/lib/serialbench/cli/validate_cli.rb

@@ -0,0 +1,88 @@
+# frozen_string_literal: true
+
+require_relative 'base_cli'
+require_relative '../yaml_validator'
+
+module Serialbench
+  module Cli
+    # CLI for validating YAML files against schemas
+    class ValidateCli < BaseCli
+      desc 'result RESULT_FILE', 'Validate a benchmark result YAML file'
+      long_desc <<~DESC
+        Validate a benchmark result YAML file against its schema.
+
+        RESULT_FILE should be the path to a results.yaml file
+
+        Examples:
+          serialbench validate result results/runs/my-run/results.yaml
+          serialbench validate result test-artifacts/benchmark-results-ubuntu-latest-ruby-3.4/results.yaml
+      DESC
+      def result(file_path)
+        validate_file(file_path, 'result', 'Result')
+      end
+
+      desc 'config BENCHMARK_CONFIG', 'Validate a benchmark configuration file'
+      long_desc <<~DESC
+        Validate a benchmark configuration file against its schema.
+
+        BENCHMARK_CONFIG should be the path to a benchmark config YAML file
+
+        Examples:
+          serialbench validate config config/benchmarks/short.yml
+          serialbench validate config config/benchmarks/full.yml
+      DESC
+      def config(file_path)
+        validate_file(file_path, 'benchmark_config', 'Benchmark config')
+      end
+
+      desc 'environment ENV_CONFIG', 'Validate an environment configuration file'
+      long_desc <<~DESC
+        Validate an environment configuration file against its schema.
+
+        ENV_CONFIG should be the path to an environment config YAML file
+
+        Examples:
+          serialbench validate environment config/environments/local-dev.yml
+          serialbench validate environment config/environments/docker-ruby-3.4.yml
+      DESC
+      def environment(file_path)
+        validate_file(file_path, 'environment_config', 'Environment config')
+      end
+
+      desc 'resultset RESULTSET_FILE', 'Validate a resultset YAML file'
+      long_desc <<~DESC
+        Validate a resultset YAML file against its schema.
+
+        RESULTSET_FILE should be the path to a resultset.yaml file
+
+        Examples:
+          serialbench validate resultset results/sets/weekly-benchmark/resultset.yaml
+          serialbench validate resultset results/sets/comparison/resultset.yaml
+      DESC
+      def resultset(file_path)
+        validate_file(file_path, 'resultset', 'Resultset')
+      end
+
+      private
+
+      def validate_file(file_path, schema_name, friendly_name)
+        unless File.exist?(file_path)
+          say "❌ File not found: #{file_path}", :red
+          exit 1
+        end
+
+        say "📝 Validating #{friendly_name}: #{file_path}", :cyan
+
+        if Serialbench::YamlValidator.validate(file_path, schema_name)
+          say "✅ #{friendly_name} is valid", :green
+        else
+          say "❌ #{friendly_name} validation failed", :red
+          exit 1
+        end
+      rescue StandardError => e
+        say "❌ Error: #{e.message}", :red
+        exit 1
+      end
+    end
+  end
+end

data/lib/serialbench/cli.rb

@@ -6,6 +6,7 @@ require_relative 'cli/environment_cli'
 require_relative 'cli/benchmark_cli'
 require_relative 'cli/resultset_cli'
 require_relative 'cli/ruby_build_cli'
+require_relative 'cli/validate_cli'
 
 module Serialbench
   # Main CLI entry point for the new object-oriented command structure
@@ -22,6 +23,9 @@ module Serialbench
     desc 'ruby_build SUBCOMMAND', 'Manage Ruby-Build definitions for validation'
     subcommand 'ruby_build', Serialbench::Cli::RubyBuildCli
 
+    desc 'validate SUBCOMMAND', 'Validate YAML files against schemas'
+    subcommand 'validate', Serialbench::Cli::ValidateCli
+
     desc 'version', 'Show Serialbench version'
     def self.version
       puts "Serialbench version #{Serialbench::VERSION}"
@@ -73,14 +77,14 @@ module Serialbench
     end
 
     # Handle unknown commands gracefully
-    def method_missing(method_name, *args)
+    def method_missing(method_name, *_args)
       puts "Unknown command: #{method_name}"
       puts ''
       help
       exit 1
     end
 
-    def respond_to_missing?(method_name, include_private = false)
+    def respond_to_missing?(_method_name, _include_private = false)
       false
     end
   end

data/lib/serialbench/config_manager.rb

@@ -39,16 +39,14 @@ module Serialbench
     private
 
     # Validate configuration against schema
-    def validate_config(config_data, config_path)
+    def validate_config(config_data, _config_path)
       # For now, perform basic validation since we don't have a specific config schema validator
       validate_basic_config(config_data)
     end
 
     # Load the configuration schema
     def load_schema
-      unless File.exist?(SCHEMA_PATH)
-        raise ConfigurationError, "Configuration schema not found: #{SCHEMA_PATH}"
-      end
+      raise ConfigurationError, "Configuration schema not found: #{SCHEMA_PATH}" unless File.exist?(SCHEMA_PATH)
 
       begin
         YAML.load_file(SCHEMA_PATH)
@@ -81,28 +79,22 @@ module Serialbench
 
     # Validate Docker-specific configuration
     def validate_docker_config(config)
-      unless config.image_variants && !config.image_variants.empty?
-        raise ConfigurationError, "Docker runtime requires 'image_variants' to be specified"
-      end
+      raise ConfigurationError, "Docker runtime requires 'image_variants' to be specified" unless config.image_variants && !config.image_variants.empty?
 
       invalid_variants = config.image_variants - %w[slim alpine]
-      unless invalid_variants.empty?
-        raise ConfigurationError, "Invalid image variants: #{invalid_variants.join(', ')}. Valid variants: slim, alpine"
-      end
+      return if invalid_variants.empty?
+
+      raise ConfigurationError, "Invalid image variants: #{invalid_variants.join(', ')}. Valid variants: slim, alpine"
     end
 
     # Validate ASDF-specific configuration
     def validate_asdf_config(config)
       # Check if ASDF is available
-      unless command_available?('asdf')
-        raise ConfigurationError, "ASDF is not installed or not in PATH. Please install ASDF to use asdf runtime."
-      end
+      raise ConfigurationError, 'ASDF is not installed or not in PATH. Please install ASDF to use asdf runtime.' unless command_available?('asdf')
 
       # Validate Ruby version format for ASDF (should include patch version)
       config.ruby_versions.each do |version|
-        unless version.match?(/^\d+\.\d+\.\d+$/)
-          raise ConfigurationError, "ASDF runtime requires full version numbers (e.g., '3.2.8'), got: #{version}"
-        end
+        raise ConfigurationError, "ASDF runtime requires full version numbers (e.g., '3.2.8'), got: #{version}" unless version.match?(/^\d+\.\d+\.\d+$/)
       end
     end
 
@@ -111,25 +103,22 @@ module Serialbench
       # Check required fields
       required_fields = %w[runtime ruby_versions output_dir benchmark_config]
       required_fields.each do |field|
-        unless config_data.key?(field)
-          raise ConfigurationError, "Missing required field: #{field}"
-        end
+        raise ConfigurationError, "Missing required field: #{field}" unless config_data.key?(field)
       end
 
       # Validate runtime
       valid_runtimes = %w[docker asdf]
       unless valid_runtimes.include?(config_data['runtime'])
-        raise ConfigurationError, "Invalid runtime: #{config_data['runtime']}. Valid runtimes: #{valid_runtimes.join(', ')}"
+        raise ConfigurationError,
+              "Invalid runtime: #{config_data['runtime']}. Valid runtimes: #{valid_runtimes.join(', ')}"
       end
 
       # Validate ruby_versions is an array
-      unless config_data['ruby_versions'].is_a?(Array)
-        raise ConfigurationError, "ruby_versions must be an array"
-      end
+      raise ConfigurationError, 'ruby_versions must be an array' unless config_data['ruby_versions'].is_a?(Array)
 
-      if config_data['ruby_versions'].empty?
-        raise ConfigurationError, "ruby_versions cannot be empty"
-      end
+      return unless config_data['ruby_versions'].empty?
+
+      raise ConfigurationError, 'ruby_versions cannot be empty'
     end
 
     # Check if a command is available in PATH

data/lib/serialbench/models/benchmark_config.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'lutaml/model'
 
 module Serialbench
@@ -51,6 +53,16 @@ module Serialbench
       attribute :warmup, :integer, default: -> { 1 }
       attribute :operations, :string, collection: true, values: %w[parse generate memory streaming]
 
+      key_value do
+        map 'name', to: :name
+        map 'description', to: :description
+        map 'data_sizes', to: :data_sizes
+        map 'formats', to: :formats
+        map 'iterations', to: :iterations
+        map 'warmup', to: :warmup
+        map 'operations', to: :operations
+      end
+
       def to_file(file_path)
         File.write(file_path, to_yaml)
       end

data/lib/serialbench/models/benchmark_result.rb

@@ -8,16 +8,24 @@ module Serialbench
       attribute :format, :string, values: %w[xml json yaml toml]
       attribute :name, :string
       attribute :version, :string
-    end
 
-    class SerializerInformationCollection < Lutaml::Model::Collection
-      instances :items, SerializerInformation
+      key_value do
+        map 'format', to: :format
+        map 'name', to: :name
+        map 'version', to: :version
+      end
     end
 
     class AdapterPerformance < Lutaml::Model::Serializable
       attribute :adapter, :string
       attribute :format, :string, values: %w[xml json yaml toml]
       attribute :data_size, :string, values: %w[small medium large]
+
+      key_value do
+        map 'adapter', to: :adapter
+        map 'format', to: :format
+        map 'data_size', to: :data_size
+      end
     end
 
     class IterationPerformance < AdapterPerformance
@@ -25,6 +33,16 @@ module Serialbench
       attribute :time_per_iteration, :float
       attribute :iterations_per_second, :float
       attribute :iterations_count, :integer
+
+      key_value do
+        map 'adapter', to: :adapter
+        map 'format', to: :format
+        map 'data_size', to: :data_size
+        map 'time_per_iterations', to: :time_per_iterations
+        map 'time_per_iteration', to: :time_per_iteration
+        map 'iterations_per_second', to: :iterations_per_second
+        map 'iterations_count', to: :iterations_count
+      end
     end
 
     class MemoryPerformance < AdapterPerformance
@@ -32,6 +50,16 @@ module Serialbench
       attribute :total_retained, :integer
       attribute :allocated_memory, :integer
       attribute :retained_memory, :integer
+
+      key_value do
+        map 'adapter', to: :adapter
+        map 'format', to: :format
+        map 'data_size', to: :data_size
+        map 'total_allocated', to: :total_allocated
+        map 'total_retained', to: :total_retained
+        map 'allocated_memory', to: :allocated_memory
+        map 'retained_memory', to: :retained_memory
+      end
     end
 
     class BenchmarkResult < Lutaml::Model::Serializable
@@ -40,6 +68,14 @@ module Serialbench
       attribute :generation, IterationPerformance, collection: true
       attribute :memory, MemoryPerformance, collection: true
       attribute :streaming, IterationPerformance, collection: true
+
+      key_value do
+        map 'serializers', to: :serializers
+        map 'parsing', to: :parsing
+        map 'generation', to: :generation
+        map 'memory', to: :memory
+        map 'streaming', to: :streaming
+      end
     end
   end
 end

data/lib/serialbench/models/environment_config.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'lutaml/model'
 require_relative '../ruby_build_manager'
 
@@ -43,8 +45,7 @@ module Serialbench
       attribute :name, :string
       attribute :kind, :string
       attribute :created_at, :string, default: -> { Time.now.utc.iso8601 }
-      attribute :ruby_build_tag, :string, values:
-        RubyBuildManager.list_definitions
+      attribute :ruby_build_tag, :string
       attribute :description, :string
       attribute :docker, DockerEnvConfig
       attribute :asdf, AsdfEnvConfig

data/lib/serialbench/models/platform.rb

@@ -15,17 +15,46 @@ module Serialbench
     class Platform < Lutaml::Model::Serializable
       attribute :platform_string, :string
       attribute :kind, :string, default: -> { 'local' }
-      attribute :os, :string, default: -> { detect_os }
-      attribute :arch, :string, default: -> { detect_arch }
+      # Fully qualified: lutaml-model 0.8+ instance-evals default
+      # lambdas, so bare detect_os/detect_arch (class methods) no
+      # longer resolve there.
+      attribute :os, :string, default: -> { Serialbench::Models::Platform.detect_os }
+      attribute :arch, :string, default: -> { Serialbench::Models::Platform.detect_arch }
       attribute :ruby_version, :string, default: -> { RUBY_VERSION }
       attribute :ruby_platform, :string, default: -> { RUBY_PLATFORM }
-      attribute :ruby_build_tag, :string, values: RubyBuildManager.list_definitions
+      attribute :ruby_build_tag, :string
+
+      key_value do
+        map 'platform_string', to: :platform_string
+        map 'kind', to: :kind
+        map 'os', to: :os
+        map 'arch', to: :arch
+        map 'ruby_version', to: :ruby_version
+        map 'ruby_platform', to: :ruby_platform
+        map 'ruby_build_tag', to: :ruby_build_tag
+      end
+
+      def self.current_local(ruby_version: nil)
+        version = ruby_version || RUBY_VERSION
+
+        # Check for GitHub Actions runner platform
+        github_platform = ENV['GITHUB_RUNNER_PLATFORM']
+        if github_platform
+          os, arch = parse_github_platform(github_platform)
+          platform_string = "#{github_platform}-ruby-#{version}"
+        else
+          os = detect_os
+          arch = detect_arch
+          platform_string = "local-#{version}"
+        end
 
-      def self.current_local
         new(
-          platform_string: "local-#{RUBY_VERSION}",
+          platform_string: platform_string,
           kind: 'local',
-          ruby_build_tag: RUBY_VERSION
+          os: os,
+          arch: arch,
+          ruby_version: version,
+          ruby_build_tag: version
         )
       end
 
@@ -54,6 +83,32 @@ module Serialbench
           'unknown'
         end
       end
+
+      def self.parse_github_platform(platform_name)
+        # Parse GitHub Actions runner platform names
+        # Examples: ubuntu-24.04, ubuntu-24.04-arm, macos-13, macos-15-intel,
+        #           macos-14, macos-15, macos-26, windows-2022, windows-2025, windows-11-arm
+
+        case platform_name
+        when /^ubuntu.*-arm$/
+          ['linux', 'arm64']
+        when /^ubuntu/
+          ['linux', 'x86_64']
+        when /^macos-.*-intel$/
+          ['macos', 'x86_64']
+        when /^macos-(13)$/
+          ['macos', 'x86_64']
+        when /^macos-(14|15|26)$/
+          ['macos', 'arm64']
+        when /^windows.*-arm$/
+          ['windows', 'arm64']
+        when /^windows/
+          ['windows', 'x86_64']
+        else
+          # Fallback to automatic detection
+          [detect_os, detect_arch]
+        end
+      end
     end
   end
 end