serialbench 0.1.2 → 0.1.3

data/.github/workflows/windows-debug.yml

@@ -0,0 +1,171 @@
+name: Windows Debug
+
+on:
+  push:
+    branches: [ master, main ]
+    paths:
+      - '.github/workflows/windows-debug.yml'
+  pull_request:
+    paths:
+      - '.github/workflows/windows-debug.yml'
+  workflow_dispatch:
+
+jobs:
+  test-libxml2-install:
+    runs-on: windows-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby: ['3.1', '3.4']
+        method: ['vcpkg', 'msys2', 'direct']
+
+    name: Test ${{ matrix.method }} on Ruby ${{ matrix.ruby }}
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+
+      - name: Install via vcpkg
+        if: matrix.method == 'vcpkg'
+        shell: powershell
+        run: |
+          Write-Host "Installing libxml2 via vcpkg..."
+          vcpkg install libxml2:x64-windows
+          vcpkg integrate install
+
+          # Find the libxml2 installation
+          $vcpkgRoot = "C:\vcpkg"
+          $libxml2Path = Join-Path $vcpkgRoot "installed\x64-windows"
+
+          Write-Host "libxml2 installed at: $libxml2Path"
+
+          # Set environment variables
+          echo "LIBXML2_INCLUDE=$libxml2Path\include" >> $env:GITHUB_ENV
+          echo "LIBXML2_LIB=$libxml2Path\lib" >> $env:GITHUB_ENV
+          echo "PKG_CONFIG_PATH=$libxml2Path\lib\pkgconfig" >> $env:GITHUB_ENV
+
+          # Add to PATH
+          $currentPath = [Environment]::GetEnvironmentVariable("PATH", "Process")
+          $newPath = "$libxml2Path\bin;$currentPath"
+          echo "$newPath" >> $env:GITHUB_PATH
+
+          # Verify installation
+          Get-ChildItem -Path $libxml2Path -Recurse | Where-Object { $_.Name -like "*libxml*" } | Select-Object FullName
+
+      - name: Install via msys2
+        if: matrix.method == 'msys2'
+        shell: powershell
+        run: |
+          Write-Host "Installing libxml2 via msys2..."
+          choco install -y msys2
+
+          # Initialize msys2 and install libxml2
+          C:\tools\msys64\usr\bin\bash -lc "pacman -Syu --noconfirm"
+          C:\tools\msys64\usr\bin\bash -lc "pacman -S --noconfirm mingw-w64-x86_64-libxml2 mingw-w64-x86_64-pkg-config"
+
+          # Set environment variables
+          $msys2Root = "C:\tools\msys64\mingw64"
+          echo "LIBXML2_INCLUDE=$msys2Root\include\libxml2" >> $env:GITHUB_ENV
+          echo "LIBXML2_LIB=$msys2Root\lib" >> $env:GITHUB_ENV
+          echo "PKG_CONFIG_PATH=$msys2Root\lib\pkgconfig" >> $env:GITHUB_ENV
+
+          # Add to PATH
+          echo "$msys2Root\bin" >> $env:GITHUB_PATH
+
+          # Verify installation
+          Get-ChildItem -Path $msys2Root -Recurse | Where-Object { $_.Name -like "*libxml*" } | Select-Object FullName
+
+      - name: Install via direct download
+        if: matrix.method == 'direct'
+        shell: powershell
+        run: |
+          Write-Host "Installing libxml2 via direct download..."
+
+          # Download pre-built binaries from gnome.org
+          $downloadDir = "$env:TEMP\libxml2"
+          New-Item -ItemType Directory -Force -Path $downloadDir
+
+          # Download libxml2 and dependencies
+          $libxml2Url = "https://download.gnome.org/binaries/win64/libxml2/2.9/libxml2-2.9.14-x64_dll.zip"
+          $zlibUrl = "https://download.gnome.org/binaries/win64/dependencies/zlib-1.2.13-x64_dll.zip"
+          $iconvUrl = "https://download.gnome.org/binaries/win64/dependencies/libiconv-1.17-x64_dll.zip"
+
+          Invoke-WebRequest -Uri $libxml2Url -OutFile "$downloadDir\libxml2.zip"
+          Invoke-WebRequest -Uri $zlibUrl -OutFile "$downloadDir\zlib.zip"
+          Invoke-WebRequest -Uri $iconvUrl -OutFile "$downloadDir\iconv.zip"
+
+          # Extract archives
+          Expand-Archive -Path "$downloadDir\libxml2.zip" -DestinationPath "C:\libxml2" -Force
+          Expand-Archive -Path "$downloadDir\zlib.zip" -DestinationPath "C:\libxml2" -Force
+          Expand-Archive -Path "$downloadDir\iconv.zip" -DestinationPath "C:\libxml2" -Force
+
+          # Set environment variables
+          echo "LIBXML2_INCLUDE=C:\libxml2\include" >> $env:GITHUB_ENV
+          echo "LIBXML2_LIB=C:\libxml2\lib" >> $env:GITHUB_ENV
+
+          # Add to PATH
+          echo "C:\libxml2\bin" >> $env:GITHUB_PATH
+
+          # Verify installation
+          Get-ChildItem -Path "C:\libxml2" -Recurse | Where-Object { $_.Name -like "*libxml*" } | Select-Object FullName
+
+      - name: Verify pkg-config
+        shell: powershell
+        run: |
+          Write-Host "Checking for pkg-config..."
+          try {
+            $pkgConfigPath = Get-Command pkg-config -ErrorAction Stop
+            Write-Host "pkg-config found at: $($pkgConfigPath.Source)"
+            pkg-config --version
+          } catch {
+            Write-Host "pkg-config not found, installing pkgconfiglite..."
+            choco install -y pkgconfiglite
+            pkg-config --version
+          }
+
+          # Try to find libxml2 via pkg-config
+          Write-Host "`nSearching for libxml-2.0.pc..."
+          if ($env:PKG_CONFIG_PATH) {
+            Write-Host "PKG_CONFIG_PATH: $env:PKG_CONFIG_PATH"
+            pkg-config --list-all | Select-String libxml
+          }
+
+      - name: Install bundler
+        run: gem install bundler
+
+      - name: Try to install libxml-ruby gem
+        shell: powershell
+        continue-on-error: true
+        run: |
+          Write-Host "Attempting to install libxml-ruby..."
+
+          # Set additional environment variables for gem installation
+          if ($env:LIBXML2_INCLUDE) {
+            $env:CFLAGS = "-I$env:LIBXML2_INCLUDE"
+            $env:CPPFLAGS = "-I$env:LIBXML2_INCLUDE"
+          }
+          if ($env:LIBXML2_LIB) {
+            $env:LDFLAGS = "-L$env:LIBXML2_LIB"
+          }
+
+          gem install libxml-ruby --verbose
+
+      - name: Report results
+        if: always()
+        shell: powershell
+        run: |
+          Write-Host "`n=== Installation Method: ${{ matrix.method }} ==="
+          Write-Host "Ruby Version: ${{ matrix.ruby }}"
+          Write-Host "`nEnvironment Variables:"
+          Write-Host "LIBXML2_INCLUDE: $env:LIBXML2_INCLUDE"
+          Write-Host "LIBXML2_LIB: $env:LIBXML2_LIB"
+          Write-Host "PKG_CONFIG_PATH: $env:PKG_CONFIG_PATH"
+          Write-Host "`nPATH:"
+          $env:PATH -split ';' | ForEach-Object { Write-Host "  $_" }
+
+          Write-Host "`nChecking for libxml2 files..."
+          Get-ChildItem C:\ -Recurse -ErrorAction SilentlyContinue -Include libxml2.dll,libxml2.lib,xml2-config.h 2>$null | Select-Object FullName

data/.gitignore

@@ -0,0 +1,32 @@
+# RSpec
+.rspec_status
+
+# ASDF
+.tool-versions
+
+# Bundler
+Gemfile.lock
+
+# Generated sites
+_site/
+_site.bak/
+
+# Gem builds
+pkg/
+
+# Benchmark results
+results/
+
+# macOS
+.DS_Store
+
+# Ruby LSP
+.ruby-lsp/
+
+# Editor files
+*.swp
+*.tmp
+*.log
+
+# Ruby version manager files
+.ruby-version

data/.rubocop.yml

@@ -0,0 +1 @@
+inherit_from: .rubocop_todo.yml

data/.rubocop_todo.yml

@@ -0,0 +1,274 @@
+# This configuration was generated by
+# `rubocop --auto-gen-config`
+# on 2025-10-18 07:01:07 UTC using RuboCop version 1.76.0.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: EnforcedStyle, IndentationWidth.
+# SupportedStyles: with_first_argument, with_fixed_indentation
+Layout/ArgumentAlignment:
+  Exclude:
+    - 'lib/serialbench/config_manager.rb'
+
+# Offense count: 4
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: EnforcedStyle, IndentOneStep, IndentationWidth.
+# SupportedStyles: case, end
+Layout/CaseIndentation:
+  Exclude:
+    - 'lib/serialbench/serializers/xml/libxml_serializer.rb'
+    - 'lib/serialbench/serializers/xml/oga_serializer.rb'
+
+# Offense count: 2
+# This cop supports safe autocorrection (--autocorrect).
+Layout/ElseAlignment:
+  Exclude:
+    - 'lib/serialbench/serializers/xml/libxml_serializer.rb'
+    - 'lib/serialbench/serializers/xml/oga_serializer.rb'
+
+# Offense count: 2
+# This cop supports safe autocorrection (--autocorrect).
+Layout/EmptyLineAfterGuardClause:
+  Exclude:
+    - 'lib/serialbench/config_manager.rb'
+
+# Offense count: 3
+# This cop supports safe autocorrection (--autocorrect).
+Layout/EmptyLineAfterMagicComment:
+  Exclude:
+    - 'Gemfile'
+    - 'lib/serialbench/models/benchmark_config.rb'
+    - 'lib/serialbench/models/environment_config.rb'
+
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: EmptyLineBetweenMethodDefs, EmptyLineBetweenClassDefs, EmptyLineBetweenModuleDefs, DefLikeMacros, AllowAdjacentOneLineDefs, NumberOfEmptyLines.
+Layout/EmptyLineBetweenDefs:
+  Exclude:
+    - 'lib/serialbench/serializers/toml/base_toml_serializer.rb'
+
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+Layout/EmptyLines:
+  Exclude:
+    - 'lib/serialbench/serializers/toml/base_toml_serializer.rb'
+
+# Offense count: 5
+# This cop supports safe autocorrection (--autocorrect).
+Layout/EmptyLinesAroundExceptionHandlingKeywords:
+  Exclude:
+    - 'lib/serialbench/cli/ruby_build_cli.rb'
+
+# Offense count: 10
+# This cop supports safe autocorrection (--autocorrect).
+Layout/EmptyLinesAroundMethodBody:
+  Exclude:
+    - 'lib/serialbench/cli/ruby_build_cli.rb'
+    - 'lib/serialbench/config_manager.rb'
+
+# Offense count: 2
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: EnforcedStyleAlignWith, Severity.
+# SupportedStylesAlignWith: keyword, variable, start_of_line
+Layout/EndAlignment:
+  Exclude:
+    - 'lib/serialbench/serializers/xml/libxml_serializer.rb'
+    - 'lib/serialbench/serializers/xml/oga_serializer.rb'
+
+# Offense count: 7
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: EnforcedStyle.
+# SupportedStyles: normal, indented_internal_methods
+Layout/IndentationConsistency:
+  Exclude:
+    - 'lib/serialbench/config_manager.rb'
+    - 'lib/serialbench/serializers/xml/libxml_serializer.rb'
+    - 'lib/serialbench/serializers/xml/oga_serializer.rb'
+
+# Offense count: 10
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: Width, AllowedPatterns.
+Layout/IndentationWidth:
+  Exclude:
+    - 'lib/serialbench/cli/ruby_build_cli.rb'
+    - 'lib/serialbench/serializers/xml/libxml_serializer.rb'
+    - 'lib/serialbench/serializers/xml/oga_serializer.rb'
+
+# Offense count: 4
+# This cop supports safe autocorrection (--autocorrect).
+Layout/RescueEnsureAlignment:
+  Exclude:
+    - 'lib/serialbench/cli/ruby_build_cli.rb'
+
+# Offense count: 11
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: AllowInHeredoc.
+Layout/TrailingWhitespace:
+  Exclude:
+    - 'lib/serialbench/cli/ruby_build_cli.rb'
+    - 'lib/serialbench/config_manager.rb'
+
+# Offense count: 1
+# Configuration parameters: AllowedParentClasses.
+Lint/MissingSuper:
+  Exclude:
+    - 'lib/serialbench/serializers/xml/ox_serializer.rb'
+
+# Offense count: 22
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: AutoCorrect, AllowUnusedKeywordArguments, IgnoreEmptyMethods, IgnoreNotImplementedMethods, NotImplementedExceptions.
+# NotImplementedExceptions: NotImplementedError
+Lint/UnusedMethodArgument:
+  Enabled: false
+
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: AutoCorrect, ContextCreatingMethods, MethodCreatingMethods.
+Lint/UselessAccessModifier:
+  Exclude:
+    - 'lib/serialbench/serializers/toml/base_toml_serializer.rb'
+
+# Offense count: 2
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: AutoCorrect.
+Lint/UselessAssignment:
+  Exclude:
+    - 'lib/serialbench/cli/resultset_cli.rb'
+
+# Offense count: 32
+# Configuration parameters: AllowedMethods, AllowedPatterns, CountRepeatedAttributes.
+Metrics/AbcSize:
+  Max: 50
+
+# Offense count: 13
+# Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
+# AllowedMethods: refine
+Metrics/BlockLength:
+  Max: 449
+
+# Offense count: 6
+# Configuration parameters: CountComments, CountAsOne.
+Metrics/ClassLength:
+  Max: 367
+
+# Offense count: 4
+# Configuration parameters: AllowedMethods, AllowedPatterns.
+Metrics/CyclomaticComplexity:
+  Max: 12
+
+# Offense count: 60
+# Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
+Metrics/MethodLength:
+  Max: 63
+
+# Offense count: 1
+# Configuration parameters: AllowedMethods, AllowedPatterns.
+Metrics/PerceivedComplexity:
+  Max: 12
+
+# Offense count: 1
+Naming/AccessorMethodName:
+  Exclude:
+    - 'lib/serialbench/runners/asdf_runner.rb'
+
+# Offense count: 37
+# Configuration parameters: AllowedConstants.
+Style/Documentation:
+  Enabled: false
+
+# Offense count: 3
+# This cop supports unsafe autocorrection (--autocorrect-all).
+# Configuration parameters: EnforcedStyle.
+# SupportedStyles: always, always_true, never
+Style/FrozenStringLiteralComment:
+  Exclude:
+    - '**/*.arb'
+    - 'Gemfile'
+    - 'lib/serialbench/models/benchmark_config.rb'
+    - 'lib/serialbench/models/environment_config.rb'
+
+# Offense count: 2
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: MinBodyLength, AllowConsecutiveConditionals.
+Style/GuardClause:
+  Exclude:
+    - 'lib/serialbench/config_manager.rb'
+
+# Offense count: 12
+# This cop supports unsafe autocorrection (--autocorrect-all).
+Style/IdenticalConditionalBranches:
+  Exclude:
+    - 'lib/serialbench/serializers/xml/libxml_serializer.rb'
+    - 'lib/serialbench/serializers/xml/oga_serializer.rb'
+
+# Offense count: 10
+# This cop supports safe autocorrection (--autocorrect).
+Style/IfUnlessModifier:
+  Exclude:
+    - 'lib/serialbench/config_manager.rb'
+    - 'lib/serialbench/ruby_build_manager.rb'
+    - 'lib/serialbench/runners/docker_runner.rb'
+
+# Offense count: 5
+# This cop supports unsafe autocorrection (--autocorrect-all).
+# Configuration parameters: EnforcedStyle, AllowedMethods, AllowedPatterns.
+# SupportedStyles: predicate, comparison
+Style/NumericPredicate:
+  Exclude:
+    - 'spec/**/*'
+    - 'lib/serialbench/benchmark_runner.rb'
+    - 'lib/serialbench/cli/ruby_build_cli.rb'
+    - 'lib/serialbench/serializers/xml/rexml_serializer.rb'
+
+# Offense count: 4
+# This cop supports safe autocorrection (--autocorrect).
+Style/RedundantBegin:
+  Exclude:
+    - 'lib/serialbench/cli/ruby_build_cli.rb'
+
+# Offense count: 5
+# This cop supports unsafe autocorrection (--autocorrect-all).
+# Configuration parameters: ConvertCodeThatCanStartToReturnNil, AllowedMethods, MaxChainLength.
+# AllowedMethods: present?, blank?, presence, try, try!
+Style/SafeNavigation:
+  Exclude:
+    - 'lib/serialbench/serializers/toml/tomlib_serializer.rb'
+    - 'lib/serialbench/serializers/xml/libxml_serializer.rb'
+    - 'lib/serialbench/serializers/xml/nokogiri_serializer.rb'
+    - 'lib/serialbench/serializers/xml/oga_serializer.rb'
+    - 'lib/serialbench/serializers/xml/ox_serializer.rb'
+
+# Offense count: 1
+# This cop supports unsafe autocorrection (--autocorrect-all).
+# Configuration parameters: Mode.
+Style/StringConcatenation:
+  Exclude:
+    - 'lib/serialbench/ruby_build_manager.rb'
+
+# Offense count: 12
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: EnforcedStyle, ConsistentQuotesInMultiline.
+# SupportedStyles: single_quotes, double_quotes
+Style/StringLiterals:
+  Exclude:
+    - 'lib/serialbench/cli/ruby_build_cli.rb'
+    - 'lib/serialbench/config_manager.rb'
+
+# Offense count: 2
+# This cop supports unsafe autocorrection (--autocorrect-all).
+# Configuration parameters: AllowMethodsWithArguments, AllowedMethods, AllowedPatterns, AllowComments.
+# AllowedMethods: define_method
+Style/SymbolProc:
+  Exclude:
+    - 'lib/serialbench/serializers.rb'
+
+# Offense count: 4
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, AllowedPatterns, SplitStrings.
+# URISchemes: http, https
+Layout/LineLength:
+  Max: 163

data/Gemfile

@@ -1,8 +1,20 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 gemspec
 
-gem 'lutaml-model', git: 'https://github.com/lutaml/lutaml-model.git'
+# Platform-specific dependencies
+# nokogiri and libxml-ruby fail to compile on Windows ARM due to libxml2
+# compilation issues, but work on other platforms
+unless Gem.win_platform? && RUBY_PLATFORM.include?('aarch64')
+  gem 'nokogiri'
+  gem 'libxml-ruby'
+end
+
+gem 'base64'  # Required for Ruby 3.4+
+gem 'lutaml-model', '~> 0.7'
+gem 'octokit'
 gem 'rake'
 gem 'rspec'
 gem 'rubocop'

data/README.adoc

@@ -125,6 +125,42 @@ $ sudo apt-get install libxml2-dev libxslt1-dev build-essential
 $ sudo yum install libxml2-devel libxslt-devel gcc gcc-c++
 ----
 
+=== GitHub API token (optional but recommended)
+
+Serialbench fetches Ruby build definitions from the GitHub API. While this works
+without authentication, GitHub imposes strict rate limits on unauthenticated
+requests (60 requests per hour). For better reliability, especially in CI/CD
+environments or when running multiple benchmarks, set the `GITHUB_TOKEN`
+environment variable.
+
+**Rate limits:**
+
+* **Without token**: 60 requests per hour
+* **With token**: 1,000 requests per hour
+
+**Setting the token:**
+
+[source,bash]
+----
+# For local usage
+export GITHUB_TOKEN=ghp_your_token_here
+
+# For GitHub Actions (automatic)
+# GitHub Actions automatically provides ${{ secrets.GITHUB_TOKEN }}
+# No manual configuration needed in workflows
+----
+
+**Creating a token:**
+
+. Go to https://github.com/settings/tokens
+. Click "Generate new token (classic)"
+. Select scope: `public_repo` (for accessing public repositories)
+. Copy the generated token
+. Store it securely in your environment
+
+NOTE: The token is optional. Serialbench will work without it but may hit rate
+limits when updating Ruby build definitions frequently.
+
 == Installation
 
 Add this line to your application's Gemfile:

data/data/schemas/result.yml

@@ -0,0 +1,29 @@
+---
+type: object
+title: Serialbench Result
+description: Schema for serialbench benchmark result files (basic structure validation)
+properties:
+  platform:
+    type: object
+    properties:
+      platform_string:
+        type: string
+      os:
+        type: string
+      arch:
+        type: string
+      ruby_version:
+        type: string
+  metadata:
+    type: object
+  environment_config:
+    type: object
+    properties:
+      name:
+        type: string
+      kind:
+        type: string
+  benchmark_config:
+    type: object
+  benchmark_result:
+    type: object

data/docs/PLATFORM_VALIDATION_FIX.md

@@ -0,0 +1,79 @@
+# Platform Validation Fix
+
+## Problem
+
+GitHub Actions workflow run [18624230866](https://github.com/metanorma/serialbench/actions/runs/18624230866/job/53099708431) failed with the error:
+
+```
+Error adding run to resultset: undefined method `platform_string' for nil
+```
+
+This occurred when trying to add benchmark results from `artifacts/benchmark-results-macos-latest-ruby-3.2` to the weekly resultset.
+
+## Root Cause
+
+The error occurred in `lib/serialbench/models/result_set.rb` line 56, where the code attempted to access `result.platform.platform_string` to check for duplicate results. When `result.platform` is `nil`, this causes a NoMethodError.
+
+The underlying issue is that some result files were missing the `platform` section in their YAML data, causing `Result.load()` to return a Result object with a nil platform.
+
+## Solution Implemented
+
+Added validation in the `ResultSet#add_result` method to check for required fields before attempting to access their properties:
+
+```ruby
+# Validate that the result has required fields
+raise ArgumentError, "Result from #{result_path} is missing platform information" if result.platform.nil?
+raise ArgumentError, "Result from #{result_path} is missing environment_config" if result.environment_config.nil?
+raise ArgumentError, "Result from #{result_path} is missing benchmark_config" if result.benchmark_config.nil?
+```
+
+This provides clear, actionable error messages that indicate:
+1. Which result file has the problem
+2. Which specific field is missing
+
+## Testing
+
+Created comprehensive RSpec tests in `spec/serialbench/models/result_set_spec.rb` that verify:
+- Missing platform information raises appropriate error
+- Missing environment_config raises appropriate error
+- Missing benchmark_config raises appropriate error
+- Complete results are added successfully
+
+All tests pass.
+
+## Next Steps
+
+While this fix improves error reporting, the underlying issue of why platform data is missing needs investigation:
+
+1. **Investigate Result Generation**: Check why some results are being created without platform data
+   - Review `LocalRunner#run_benchmark` in `lib/serialbench/runners/local_runner.rb`
+   - Review how platform data is serialized in `Platform.to_yaml`
+   - Check if there's a Lutaml::Model serialization issue
+
+2. **GitHub Actions Debugging**:
+   - Add debugging output to show the contents of result files before adding them to resultsets
+   - Verify that all benchmark runs are creating complete result files with platform data
+
+3. **Platform Serialization**: Ensure `Platform` model properly serializes/deserializes:
+   - All attributes are included in YAML output
+   - Default values are properly set
+   - No Lutaml::Model configuration issues
+
+## Commit
+
+```
+fix(resultset): add validation for missing platform data in results
+
+Previously, when adding a result to a resultset, if the result's YAML file
+was missing platform information (or environment_config/benchmark_config),
+the code would fail with 'undefined method platform_string for nil' when
+trying to check for duplicates.
+
+This commit adds proper validation to check for nil values before accessing
+their properties, providing clear error messages that indicate which field
+is missing and which result path has the issue.
+
+Also adds comprehensive RSpec tests to verify the validation works correctly
+for all three required fields: platform, environment_config, and benchmark_config.
+
+Fixes the error seen in GitHub Actions workflow run 18624230866.