omnizip 0.3.52 → 0.3.53

data/lib/omnizip/models/profile_report.rb

@@ -1,26 +1,65 @@
 # frozen_string_literal: true
 
+require "lutaml/model"
+
 module Omnizip
   module Models
-    # Represents an aggregated profiling report with hot path analysis
-    class ProfileReport
-      attr_reader :profile_name, :results, :hot_paths, :bottlenecks,
-                  :timestamp, :metadata
+    # Aggregated profiling report with hot path analysis.
+    #
+    # Serialized via lutaml-model — no hand-rolled +to_h+ / +to_json+.
+    # The +summary+ key serializes a nested Summary model built from
+    # the report's aggregates; hot paths and bottlenecks are the
+    # analyzer-supplied hashes.
+    class ProfileReport < Lutaml::Model::Serializable
+      # Aggregate block of {ProfileReport#to_hash}.
+      class Summary < Lutaml::Model::Serializable
+        attribute :total_execution_time, :double, default: 0.0
+        attribute :total_memory_allocated, :integer, default: 0
+        attribute :total_gc_runs, :integer, default: 0
+        attribute :operation_count, :integer, default: 0
+
+        key_value do
+          map "total_execution_time", to: :total_execution_time,
+                                      render_default: true, render_nil: true
+          map "total_memory_allocated", to: :total_memory_allocated,
+                                        render_default: true, render_nil: true
+          map "total_gc_runs", to: :total_gc_runs,
+                               render_default: true, render_nil: true
+          map "operation_count", to: :operation_count,
+                                 render_default: true, render_nil: true
+        end
+      end
+
+      attribute :profile_name, :string
+      attribute :results, PerformanceResult, collection: true, default: []
+      attribute :hot_paths, :hash, collection: true, default: []
+      attribute :bottlenecks, :hash, collection: true, default: []
+      attribute :timestamp, :date_time, default: -> { Time.now }
+      attribute :metadata, :hash, default: {}
+      attribute :summary, Summary, method: :build_summary
+
+      key_value do
+        map "profile_name", to: :profile_name,
+                            render_default: true, render_nil: true
+        map "timestamp", to: :timestamp,
+                         render_default: true, render_nil: true
+        map "summary", to: :summary,
+                       render_default: true, render_nil: true
+        map "results", to: :results,
+                       render_default: true, render_nil: true
+        map "hot_paths", to: :hot_paths,
+                         render_default: true, render_nil: true
+        map "bottlenecks", to: :bottlenecks,
+                           render_default: true, render_nil: true
+        map "metadata", to: :metadata,
+                        render_default: true, render_nil: true
+      end
 
-      def initialize(
-        profile_name:,
-        results: [],
-        hot_paths: [],
-        bottlenecks: [],
-        timestamp: Time.now,
-        metadata: {}
-      )
-        @profile_name = profile_name
-        @results = results
-        @hot_paths = hot_paths
-        @bottlenecks = bottlenecks
-        @timestamp = timestamp
-        @metadata = metadata
+      # lutaml-model 0.8.x does not render empty-collection defaults,
+      # so they are assigned explicitly to keep every key present.
+      def initialize(attrs = {})
+        super({ results: [], hot_paths: [], bottlenecks: [],
+                metadata: {} }.merge(attrs))
       end
 
       def total_execution_time
@@ -50,32 +89,27 @@ module Omnizip
       end
 
       def add_result(result)
-        @results << result
+        results << result
       end
 
       def add_hot_path(hot_path)
-        @hot_paths << hot_path
+        hot_paths << hot_path
       end
 
       def add_bottleneck(bottleneck)
-        @bottlenecks << bottleneck
+        bottlenecks << bottleneck
       end
 
-      def to_h
-        {
-          profile_name: profile_name,
-          timestamp: timestamp.iso8601,
-          summary: {
-            total_execution_time: total_execution_time,
-            total_memory_allocated: total_memory_allocated,
-            total_gc_runs: total_gc_runs,
-            operation_count: results.size,
-          },
-          results: results.map(&:to_h),
-          hot_paths: hot_paths,
-          bottlenecks: bottlenecks,
-          metadata: metadata,
-        }
+      # Serialized aggregate block.
+      #
+      # @return [Summary]
+      def build_summary
+        Summary.new(
+          total_execution_time: total_execution_time,
+          total_memory_allocated: total_memory_allocated,
+          total_gc_runs: total_gc_runs,
+          operation_count: results.size,
+        )
       end
     end
   end

data/lib/omnizip/parallel/parallel_extractor.rb

@@ -87,7 +87,7 @@ module Omnizip
           ::File.open(archive_path, "rb") do |io|
             # Find the entry in reader
             reader_entry = reader.entries.find { |e| e.filename == entry.name }
-            raise "Entry not found in archive: #{entry.name}" unless reader_entry
+            raise Errno::ENOENT, "Entry not found in archive: #{entry.name}" unless reader_entry
 
             # Seek to entry data
             io.seek(reader_entry.local_header_offset, ::IO::SEEK_SET)
@@ -288,7 +288,7 @@ module Omnizip
           @write_mutex.synchronize do
             # Check if file exists
             if ::File.exist?(dest_path) && !overwrite
-              raise "File exists: #{dest_path}"
+              raise Errno::EEXIST, "File exists: #{dest_path}"
             end
 
             # Write file or create directory

data/lib/omnizip/parallel/worker_pool.rb

@@ -93,7 +93,7 @@ module Omnizip
       # @param work [Fractor::Work] work item to process
       # @return [void]
       def submit(work)
-        raise "Worker pool not started" unless @running
+        raise Omnizip::IOError, "Worker pool not started" unless @running
 
         if @continuous
           # In continuous mode, add to work queue
@@ -116,8 +116,8 @@ module Omnizip
       #
       # @return [void]
       def run
-        raise "Can only run in batch mode" if @continuous
-        raise "Worker pool not started" unless @running
+        raise Omnizip::IOError, "Can only run in batch mode" if @continuous
+        raise Omnizip::IOError, "Worker pool not started" unless @running
 
         @supervisor.run
 

data/lib/omnizip/profiler/report_generator.rb

@@ -33,7 +33,7 @@ module Omnizip
 
       # Generate a JSON report
       def generate_json_report
-        JSON.pretty_generate(report.to_h)
+        JSON.pretty_generate(report.to_hash)
       end
 
       # Save report to file

data/lib/omnizip/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Omnizip
-  VERSION = "0.3.52"
+  VERSION = "0.3.53"
 end

data/lib/omnizip/zip/file.rb

@@ -402,7 +402,7 @@ module Omnizip
         ::File.open(@name, "rb") do |io|
           # Find the entry in reader
           reader_entry = @reader.entries.find { |e| e.filename == entry.name }
-          raise "Entry not found in archive: #{entry.name}" unless reader_entry
+          raise Errno::ENOENT, "Entry not found in archive: #{entry.name}" unless reader_entry
 
           # Extract just the data
           io.seek(reader_entry.local_header_offset, ::IO::SEEK_SET)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: omnizip
 version: !ruby/object:Gem::Version
-  version: 0.3.52
+  version: 0.3.53
 platform: ruby
 authors:
 - Ribose Inc.