xmi 0.3.21 → 0.5.1

data/lib/tasks/performance_helpers.rb

@@ -0,0 +1,238 @@
+# frozen_string_literal: true
+
+require "json"
+require "open3"
+require "tmpdir"
+require "fileutils"
+
+module PerformanceHelpers
+  # ANSI color codes for terminal output
+  CLEAR   = "\e[0m"
+  BOLD    = "\e[1m"
+  DIM     = "\e[2m"
+  CYAN    = "\e[36m"
+  GREEN   = "\e[32m"
+  YELLOW  = "\e[33m"
+  RED     = "\e[31m"
+  GRAY    = "\e[90m"
+  MAGENTA = "\e[35m"
+
+  # Terminal formatting helpers
+  module Term
+    extend self
+
+    HL = "─"
+    VL = "│"
+    TL = "┌"
+    TR = "┐"
+    BL = "└"
+    BR = "┘"
+
+    def header(title, color: PerformanceHelpers::CYAN)
+      width = 78
+      line = HL * width
+      puts
+      puts "#{color}#{TL}#{line}#{TR}#{CLEAR}"
+      puts "#{color}#{VL}#{CLEAR}  #{BOLD}#{color}#{title}#{CLEAR}#{' ' * (width - title.length - 4)}#{color}#{VL}#{CLEAR}"
+      puts "#{color}#{BL}#{line}#{BR}#{CLEAR}"
+    end
+
+    def sep(char: HL, width: 78)
+      puts "#{DIM}#{char * width}#{CLEAR}"
+    end
+  end
+
+  module Base
+  end
+
+  module Current
+  end
+
+  class << self
+    def load_into_namespace(module_obj, file_path)
+      content = File.read(file_path)
+      module_obj.module_eval(content, file_path)
+    end
+
+    def ruby_exec(cmd, env: {})
+      Open3.capture3(env, cmd)
+    end
+
+    def current_branch
+      stdout, = ruby_exec("git rev-parse --abbrev-ref HEAD")
+      stdout.strip
+    end
+
+    # Clone base branch into a temp dir and return its path
+    def clone_base_repo(base, performance_dir, script)
+      puts "#{DIM}Cloning base #{base}...#{CLEAR}"
+      safe_ref = base.gsub(/[^0-9A-Za-z._-]/, "-")
+      clone_dir = File.join(performance_dir, "base-#{safe_ref}")
+      FileUtils.rm_rf(clone_dir)
+
+      repo_url, = ruby_exec("git config --get remote.origin.url")
+      repo_url = repo_url.strip
+
+      stdout, stderr, status = ruby_exec("git clone --branch #{safe_ref} --single-branch #{repo_url} #{clone_dir}")
+      raise "git clone failed: #{stderr}\n#{stdout}" unless status.success?
+
+      Dir.chdir(clone_dir) do
+        stdout, stderr, status = ruby_exec("bundle install --quiet")
+        raise "bundle install failed: #{stderr}\n#{stdout}" unless status.success?
+
+        bench_copy_dir = File.join(clone_dir, "lib", "tasks")
+        FileUtils.mkdir_p(bench_copy_dir)
+        bench_copy = File.join(bench_copy_dir, "benchmark_runner.rb")
+        File.write(bench_copy, File.read(script))
+        load_into_namespace(Base, bench_copy)
+      end
+    end
+
+    def run_benchmarks(base_runner, current_runner, threshold, all_base,
+                       all_current)
+      base_results = base_runner.run_benchmarks
+      curr_results = current_runner.run_benchmarks
+
+      all_base.merge!(base_results)
+      all_current.merge!(curr_results)
+
+      # Collect comparison results
+      comparison_rows = []
+
+      curr_results.each do |label, result|
+        base_result = base_results[label]
+        cmp = compare_metrics(label, result, base_result, threshold)
+        comparison_rows << cmp
+      end
+
+      print_comparison_table(comparison_rows, threshold)
+    end
+
+    def print_comparison_table(comparison_rows, threshold)
+      rows = comparison_rows.map do |cmp|
+        {
+          benchmark: cmp[:label],
+          base_ips: cmp[:base_ips]&.round(1),
+          curr_ips: cmp[:curr_ips]&.round(1),
+          change: cmp[:change] ? "#{(cmp[:change] * 100).round(1)}%" : "N/A",
+          status: if cmp[:base_ips].nil?
+                    "NEW"
+                  elsif cmp[:change] < -threshold
+                    "REGRESSED"
+                  else
+                    "OK"
+                  end,
+        }
+      end
+
+      return if rows.empty?
+
+      puts "  #{'Benchmark'.ljust(40)} #{'Base IPS'.rjust(12)} #{'Curr IPS'.rjust(12)} #{'Change'.rjust(10)} #{'Status'.rjust(10)}"
+      puts "  #{DIM}#{'─' * 86}#{CLEAR}"
+
+      rows.each do |row|
+        status_color = case row[:status]
+                       when "REGRESSED" then RED
+                       when "NEW" then YELLOW
+                       else GREEN
+                       end
+        row[:status] == "REGRESSED" ? RED : DIM
+
+        puts "  #{row[:benchmark].ljust(40)} #{format('%-12.1f',
+                                                      row[:base_ips] || 0)} #{format('%-12.1f',
+                                                                                     row[:curr_ips] || 0)} #{format('%-10s', row[:change]).gsub('%',
+                                                                                                                                                '%%')} #{status_color}#{row[:status].rjust(10)}#{CLEAR}"
+      end
+
+      puts
+    end
+
+    def compare_metrics(label, curr, base, threshold)
+      unless base
+        return { label: label, base_ips: nil, curr_ips: nil, change: nil,
+                 regressed: false }
+      end
+
+      base_ips = base.fetch(:lower)
+      curr_ips = curr.fetch(:upper)
+      change = (curr_ips - base_ips) / base_ips.to_f
+
+      {
+        label: label,
+        base_ips: base_ips,
+        curr_ips: curr_ips,
+        change: change,
+        regressed: change < -threshold,
+      }
+    end
+
+    def summary_report(current_results, base_results, base, run_time, threshold)
+      summary = {
+        run_time: run_time,
+        threshold: threshold,
+        branch: current_branch,
+        base: base,
+        regressions: [],
+        new_benchmarks: [],
+      }
+
+      current_results.each do |label, metrics|
+        base_result = base_results[label]
+        cmp = compare_metrics(label, metrics, base_result, threshold)
+
+        # Track new benchmarks that don't exist in base
+        if base_result.nil?
+          summary[:new_benchmarks] << label
+          next
+        end
+
+        next unless cmp[:regressed]
+
+        summary[:regressions] << {
+          label: label,
+          base_ips: cmp[:base_ips],
+          curr_ips: cmp[:curr_ips],
+          delta_fraction: cmp[:change],
+        }
+      end
+
+      log_regressions(summary[:regressions], threshold)
+      log_new_benchmarks(summary[:new_benchmarks])
+      summary
+    end
+
+    def log_new_benchmarks(new_benchmarks)
+      return if new_benchmarks.empty?
+
+      puts
+      puts "#{YELLOW}🆕 New benchmarks (not in base branch):#{CLEAR}"
+      new_benchmarks.each do |label|
+        puts "  • #{label}"
+      end
+    end
+
+    def log_regressions(regressions, threshold)
+      return if regressions.empty?
+
+      puts
+      puts "#{RED}⚠️  Performance Regressions Detected#{CLEAR}"
+      puts "#{RED}   (< -#{(threshold * 100).round(2)}% IPS)#{CLEAR}"
+      puts
+      regressions.each do |regression|
+        delta = regression[:delta_fraction]
+        base_ips = regression[:base_ips]
+        curr_ips = regression[:curr_ips]
+
+        delta_str = delta ? format("%+0.2f%%", delta * 100) : "N/A"
+        base_str = base_ips ? format("%.2f", base_ips) : "N/A"
+        curr_str = curr_ips ? format("%.2f", curr_ips) : "N/A"
+
+        puts "  #{BOLD}#{regression[:label]}#{CLEAR}"
+        puts "    #{GRAY}base: #{base_str} IPS#{CLEAR}"
+        puts "    #{RED}curr: #{curr_str} IPS#{CLEAR}"
+        puts "    #{RED}change: #{delta_str}#{CLEAR}"
+        puts
+      end
+    end
+  end
+end

data/lib/xmi/add.rb

@@ -5,12 +5,12 @@ require_relative "extension"
 
 module Xmi
   class Add < Lutaml::Model::Serializable
-    attribute :id, :string
-    attribute :label, :string
-    attribute :uuid, :string
+    attribute :id, ::Xmi::Type::XmiId
+    attribute :label, ::Xmi::Type::XmiLabel
+    attribute :uuid, ::Xmi::Type::XmiUuid
     attribute :href, :string
-    attribute :idref, :string
-    attribute :type, :string
+    attribute :idref, ::Xmi::Type::XmiIdRef
+    attribute :type, ::Xmi::Type::XmiType
     attribute :target, :string
     attribute :container, :string
     attribute :position, :integer
@@ -18,46 +18,22 @@ module Xmi
     attribute :difference, Difference, collection: true
     attribute :extension, Extension, collection: true
 
-    xml do # rubocop:disable Metrics/BlockLength
+    xml do
       root "Add"
-      namespace "http://www.omg.org/spec/XMI/20131001", "xmlns"
+      namespace ::Xmi::Namespace::Omg::Xmi
 
-      map_attribute "id", to: :id, prefix: "xmlns", namespace: "http://www.omg.org/spec/XMI/20131001"
-      map_attribute "label", to: :label, prefix: "xmlns", namespace: "http://www.omg.org/spec/XMI/20131001"
-      map_attribute "uuid", to: :uuid, prefix: "xmlns", namespace: "http://www.omg.org/spec/XMI/20131001"
+      map_attribute "id", to: :id
+      map_attribute "label", to: :label
+      map_attribute "uuid", to: :uuid
       map_attribute "href", to: :href
-      map_attribute "idref", to: :idref, prefix: "xmlns", namespace: "http://www.omg.org/spec/XMI/20131001"
-      map_attribute "type", to: :type, prefix: "xmlns", namespace: "http://www.omg.org/spec/XMI/20131001"
+      map_attribute "idref", to: :idref
+      map_attribute "type", to: :type
       map_attribute "target", to: :target
       map_attribute "container", to: :container
       map_attribute "position", to: :position
       map_attribute "addition", to: :addition
-      map_element "difference", to: :difference, prefix: nil, namespace: nil,
-                                value_map: {
-                                  from: {
-                                    nil: :empty,
-                                    empty: :empty,
-                                    omitted: :empty
-                                  },
-                                  to: {
-                                    nil: :empty,
-                                    empty: :empty,
-                                    omitted: :empty
-                                  }
-                                }
-      map_element "Extension", to: :extension,
-                               value_map: {
-                                 from: {
-                                   nil: :empty,
-                                   empty: :empty,
-                                   omitted: :empty
-                                 },
-                                 to: {
-                                   nil: :empty,
-                                   empty: :empty,
-                                   omitted: :empty
-                                 }
-                               }
+      map_element "difference", to: :difference, value_map: VALUE_MAP
+      map_element "Extension", to: :extension, value_map: VALUE_MAP
     end
   end
 end

data/lib/xmi/{the_custom_profile.rb → custom_profile.rb}

@@ -1,13 +1,13 @@
 # frozen_string_literal: true
 
 module Xmi
-  module TheCustomProfile
+  module CustomProfile
     class Bibliography < Lutaml::Model::Serializable
       attribute :base_class, :string
 
       xml do
-        root "Bibliography"
-        namespace "http://www.sparxsystems.com/profiles/thecustomprofile/1.0", "thecustomprofile"
+        element "Bibliography"
+        namespace ::Xmi::Namespace::Sparx::CustomProfile
 
         map_attribute "base_Class", to: :base_class
       end
@@ -17,8 +17,8 @@ module Xmi
       attribute :base_class, :string
 
       xml do
-        root "BasicDoc"
-        namespace "http://www.sparxsystems.com/profiles/thecustomprofile/1.0", "thecustomprofile"
+        element "BasicDoc"
+        namespace ::Xmi::Namespace::Sparx::CustomProfile
 
         map_attribute "base_Class", to: :base_class
       end
@@ -28,8 +28,8 @@ module Xmi
       attribute :base_enumeration, :string
 
       xml do
-        root "enumeration"
-        namespace "http://www.sparxsystems.com/profiles/thecustomprofile/1.0", "thecustomprofile"
+        element "enumeration"
+        namespace ::Xmi::Namespace::Sparx::CustomProfile
 
         map_attribute "base_Enumeration", to: :base_enumeration
       end
@@ -39,8 +39,8 @@ module Xmi
       attribute :base_constraint, :string
 
       xml do
-        root "OCL"
-        namespace "http://www.sparxsystems.com/profiles/thecustomprofile/1.0", "thecustomprofile"
+        element "OCL"
+        namespace ::Xmi::Namespace::Sparx::CustomProfile
 
         map_attribute "base_Constraint", to: :base_constraint
       end
@@ -50,8 +50,8 @@ module Xmi
       attribute :base_constraint, :string
 
       xml do
-        root "invariant"
-        namespace "http://www.sparxsystems.com/profiles/thecustomprofile/1.0", "thecustomprofile"
+        element "invariant"
+        namespace ::Xmi::Namespace::Sparx::CustomProfile
 
         map_attribute "base_Constraint", to: :base_constraint
       end
@@ -62,8 +62,8 @@ module Xmi
       attribute :publication_date, :string
 
       xml do
-        root "publicationDate"
-        namespace "http://www.sparxsystems.com/profiles/thecustomprofile/1.0", "thecustomprofile"
+        element "publicationDate"
+        namespace ::Xmi::Namespace::Sparx::CustomProfile
 
         map_attribute "base_Package", to: :base_package
         map_attribute "publicationDate", to: :publication_date
@@ -75,8 +75,8 @@ module Xmi
       attribute :edition, :string
 
       xml do
-        root "edition"
-        namespace "http://www.sparxsystems.com/profiles/thecustomprofile/1.0", "thecustomprofile"
+        element "edition"
+        namespace ::Xmi::Namespace::Sparx::CustomProfile
 
         map_attribute "base_Package", to: :base_package
         map_attribute "edition", to: :edition
@@ -88,8 +88,8 @@ module Xmi
       attribute :number, :string
 
       xml do
-        root "number"
-        namespace "http://www.sparxsystems.com/profiles/thecustomprofile/1.0", "thecustomprofile"
+        element "number"
+        namespace ::Xmi::Namespace::Sparx::CustomProfile
 
         map_attribute "base_Package", to: :base_package
         map_attribute "number", to: :number
@@ -101,8 +101,8 @@ module Xmi
       attribute :year_version, :string
 
       xml do
-        root "yearVersion"
-        namespace "http://www.sparxsystems.com/profiles/thecustomprofile/1.0", "thecustomprofile"
+        element "yearVersion"
+        namespace ::Xmi::Namespace::Sparx::CustomProfile
 
         map_attribute "base_Package", to: :base_package
         map_attribute "yearVersion", to: :year_version
@@ -113,8 +113,8 @@ module Xmi
       attribute :base_package, :string
 
       xml do
-        root "informative"
-        namespace "http://www.sparxsystems.com/profiles/thecustomprofile/1.0", "thecustomprofile"
+        element "informative"
+        namespace ::Xmi::Namespace::Sparx::CustomProfile
 
         map_attribute "base_Package", to: :base_package
       end
@@ -126,8 +126,8 @@ module Xmi
       attribute :persistence, :string
 
       xml do
-        root "persistence"
-        namespace "http://www.sparxsystems.com/profiles/thecustomprofile/1.0", "thecustomprofile"
+        element "persistence"
+        namespace ::Xmi::Namespace::Sparx::CustomProfile
 
         map_attribute "base_Class", to: :base_class
         map_attribute "base_Enumeration", to: :base_enumeration
@@ -139,8 +139,8 @@ module Xmi
       attribute :base_class, :string
 
       xml do
-        root "Abstract"
-        namespace "http://www.sparxsystems.com/profiles/thecustomprofile/1.0", "thecustomprofile"
+        element "Abstract"
+        namespace ::Xmi::Namespace::Sparx::CustomProfile
 
         map_attribute "base_Class", to: :base_class
       end

data/lib/xmi/delete.rb

@@ -5,55 +5,31 @@ require_relative "extension"
 
 module Xmi
   class Delete < Lutaml::Model::Serializable
-    attribute :id, :string
-    attribute :label, :string
-    attribute :uuid, :string
+    attribute :id, ::Xmi::Type::XmiId
+    attribute :label, ::Xmi::Type::XmiLabel
+    attribute :uuid, ::Xmi::Type::XmiUuid
     attribute :href, :string
-    attribute :idref, :string
-    attribute :type, :string
+    attribute :idref, ::Xmi::Type::XmiIdRef
+    attribute :type, ::Xmi::Type::XmiType
     attribute :target, :string
     attribute :container, :string
     attribute :difference, Difference, collection: true
     attribute :extension, Extension, collection: true
 
-    xml do # rubocop:disable Metrics/BlockLength
+    xml do
       root "Delete"
-      namespace "http://www.omg.org/spec/XMI/20131001", "xmlns"
+      namespace ::Xmi::Namespace::Omg::Xmi
 
-      map_attribute "id", to: :id, prefix: "xmlns", namespace: "http://www.omg.org/spec/XMI/20131001"
-      map_attribute "label", to: :label, prefix: "xmlns", namespace: "http://www.omg.org/spec/XMI/20131001"
-      map_attribute "uuid", to: :uuid, prefix: "xmlns", namespace: "http://www.omg.org/spec/XMI/20131001"
+      map_attribute "id", to: :id
+      map_attribute "label", to: :label
+      map_attribute "uuid", to: :uuid
       map_attribute "href", to: :href
-      map_attribute "idref", to: :idref, prefix: "xmlns", namespace: "http://www.omg.org/spec/XMI/20131001"
-      map_attribute "type", to: :type, prefix: "xmlns", namespace: "http://www.omg.org/spec/XMI/20131001"
+      map_attribute "idref", to: :idref
+      map_attribute "type", to: :type
       map_attribute "target", to: :target
       map_attribute "container", to: :container
-      map_element "difference", to: :difference, prefix: nil, namespace: nil,
-                                value_map: {
-                                  from: {
-                                    nil: :empty,
-                                    empty: :empty,
-                                    omitted: :empty
-                                  },
-                                  to: {
-                                    nil: :empty,
-                                    empty: :empty,
-                                    omitted: :empty
-                                  }
-                                }
-      map_element "Extension", to: :extension,
-                               value_map: {
-                                 from: {
-                                   nil: :empty,
-                                   empty: :empty,
-                                   omitted: :empty
-                                 },
-                                 to: {
-                                   nil: :empty,
-                                   empty: :empty,
-                                   omitted: :empty
-                                 }
-                               }
+      map_element "difference", to: :difference, value_map: VALUE_MAP
+      map_element "Extension", to: :extension, value_map: VALUE_MAP
     end
   end
 end

data/lib/xmi/difference.rb

@@ -4,55 +4,31 @@ require_relative "extension"
 
 module Xmi
   class Difference < Lutaml::Model::Serializable
-    attribute :id, :string
-    attribute :label, :string
-    attribute :uuid, :string
+    attribute :id, ::Xmi::Type::XmiId
+    attribute :label, ::Xmi::Type::XmiLabel
+    attribute :uuid, ::Xmi::Type::XmiUuid
     attribute :href, :string
-    attribute :idref, :string
-    attribute :type, :string
+    attribute :idref, ::Xmi::Type::XmiIdRef
+    attribute :type, ::Xmi::Type::XmiType
     attribute :target, :string
     attribute :container, :string
     attribute :difference, Difference, collection: true
     attribute :extension, Extension, collection: true
 
-    xml do # rubocop:disable Metrics/BlockLength
+    xml do
       root "Difference"
-      namespace "http://www.omg.org/spec/XMI/20131001", "xmlns"
+      namespace ::Xmi::Namespace::Omg::Xmi
 
-      map_attribute "id", to: :id, prefix: "xmlns", namespace: "http://www.omg.org/spec/XMI/20131001"
-      map_attribute "label", to: :label, prefix: "xmlns", namespace: "http://www.omg.org/spec/XMI/20131001"
-      map_attribute "uuid", to: :uuid, prefix: "xmlns", namespace: "http://www.omg.org/spec/XMI/20131001"
+      map_attribute "id", to: :id
+      map_attribute "label", to: :label
+      map_attribute "uuid", to: :uuid
       map_attribute "href", to: :href
-      map_attribute "idref", to: :idref, prefix: "xmlns", namespace: "http://www.omg.org/spec/XMI/20131001"
-      map_attribute "type", to: :type, prefix: "xmlns", namespace: "http://www.omg.org/spec/XMI/20131001"
+      map_attribute "idref", to: :idref
+      map_attribute "type", to: :type
       map_attribute "target", to: :target
       map_attribute "container", to: :container
-      map_element "difference", to: :difference, prefix: nil, namespace: nil,
-                                value_map: {
-                                  from: {
-                                    nil: :empty,
-                                    empty: :empty,
-                                    omitted: :empty
-                                  },
-                                  to: {
-                                    nil: :empty,
-                                    empty: :empty,
-                                    omitted: :empty
-                                  }
-                                }
-      map_element "Extension", to: :extension,
-                               value_map: {
-                                 from: {
-                                   nil: :empty,
-                                   empty: :empty,
-                                   omitted: :empty
-                                 },
-                                 to: {
-                                   nil: :empty,
-                                   empty: :empty,
-                                   omitted: :empty
-                                 }
-                               }
+      map_element "difference", to: :difference, value_map: VALUE_MAP
+      map_element "Extension", to: :extension, value_map: VALUE_MAP
     end
   end
 end