fractor 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b9d15481939349c5d4ad4f3b09368d0221b690cf06a737007b5f247a20cda6e6
-  data.tar.gz: 66aca66a7c4b1ac1559a77fa97d20917bd52b3a9cf3c4da04906f2a6295ded75
+  metadata.gz: a3fc9be0fb1d8993186e56bbef30b33396314e878f8a9ea3d31d79efd3c46f06
+  data.tar.gz: 8edb3ff38f5c054d9c1e477bfb799605fd4827eb679dc10db224ad1eb45e8d64
 SHA512:
-  metadata.gz: 205fbcb518ea078314f5964e3e60a61212c95e3f2959bb3420e6a060bb17f8bbcef0fbd7a74e27ec5304718ead2c8defd41591f4707757f7ce25bcb22374d0c0
-  data.tar.gz: c66a1af867247746dfd5d63bd30348ae0cf18bb13186b549ccd1380eb3829dd4aa662f4dc359a9479d7666f418cb0e586e6d0b0caf0b74c45f17f29282048326
+  metadata.gz: 17fbad9c566525c9b01ccd8f613a6fe2eb2be62298894fc4ae636736ad678f3d1c46fbf49cce5e84d6e3584cc0a5e31ce06f92a994f2a3b92e58cceedabafcbc
+  data.tar.gz: a33d27eb70ffbd8769c6db1f5484788d0b187b5c1feb33ba65602579a2ecc17c3c14be6654ad74c414a63cf9b5d19a818bd462293eca0ca13bfd1f92272760f1

data/.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2025-05-06 11:12:48 UTC using RuboCop version 1.75.5.
+# on 2025-05-08 09:25:37 UTC using RuboCop version 1.75.5.
 # 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
@@ -14,6 +14,11 @@ Lint/ConstantDefinitionInBlock:
     - 'spec/fractor/integration_spec.rb'
     - 'spec/fractor/work_spec.rb'
 
+# Offense count: 1
+Lint/DuplicateMethods:
+  Exclude:
+    - 'lib/fractor/supervisor.rb'
+
 # Offense count: 1
 Lint/HashCompareByIdentity:
   Exclude:
@@ -30,36 +35,41 @@ Lint/RescueException:
   Exclude:
     - 'lib/fractor/wrapped_ractor.rb'
 
-# Offense count: 15
+# Offense count: 18
 # Configuration parameters: AllowedMethods, AllowedPatterns, CountRepeatedAttributes.
 Metrics/AbcSize:
-  Max: 83
+  Max: 98
 
-# Offense count: 8
+# Offense count: 9
 # Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
 # AllowedMethods: refine
 Metrics/BlockLength:
-  Max: 78
+  Max: 101
 
 # Offense count: 2
 # Configuration parameters: CountComments, CountAsOne.
 Metrics/ClassLength:
-  Max: 155
+  Max: 159
 
 # Offense count: 3
 # Configuration parameters: AllowedMethods, AllowedPatterns.
 Metrics/CyclomaticComplexity:
-  Max: 25
+  Max: 37
 
-# Offense count: 32
+# Offense count: 33
 # Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
 Metrics/MethodLength:
-  Max: 60
+  Max: 67
+
+# Offense count: 1
+# Configuration parameters: Max, CountKeywordArgs.
+Metrics/ParameterLists:
+  MaxOptionalParameters: 4
 
 # Offense count: 2
 # Configuration parameters: AllowedMethods, AllowedPatterns.
 Metrics/PerceivedComplexity:
-  Max: 25
+  Max: 37
 
 # Offense count: 1
 Security/Eval:
@@ -74,7 +84,13 @@ Style/Documentation:
     - 'test/**/*'
     - 'examples/hierarchical_hasher/hierarchical_hasher.rb'
 
-# Offense count: 8
+# Offense count: 13
+# This cop supports safe autocorrection (--autocorrect).
+Style/IfUnlessModifier:
+  Exclude:
+    - 'lib/fractor/supervisor.rb'
+
+# Offense count: 6
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, AllowedPatterns, SplitStrings.
 # URISchemes: http, https

data/lib/fractor/supervisor.rb

@@ -37,9 +37,7 @@ module Fractor
     # Adds a single work item to the queue.
     # The item must be an instance of Fractor::Work or a subclass.
     def add_work_item(work)
-      unless work.is_a?(Fractor::Work)
-        raise ArgumentError, "#{work.class} must be an instance of Fractor::Work"
-      end
+      raise ArgumentError, "#{work.class} must be an instance of Fractor::Work" unless work.is_a?(Fractor::Work)
 
       @work_queue << work
       @total_work_count += 1
@@ -139,9 +137,7 @@ module Fractor
 
         # Break if no active workers and queue is empty, but work remains (indicates potential issue)
         if active_ractors.empty? && @work_queue.empty? && !@continuous_mode && processed_count < @total_work_count
-          if ENV["FRACTOR_DEBUG"]
-            puts "Warning: No active workers and queue is empty, but not all work is processed. Exiting loop."
-          end
+          puts "Warning: No active workers and queue is empty, but not all work is processed. Exiting loop." if ENV["FRACTOR_DEBUG"]
           break
         end
 
@@ -160,30 +156,22 @@ module Fractor
         # Find the corresponding WrappedRactor instance
         wrapped_ractor = @ractors_map[ready_ractor_obj]
         unless wrapped_ractor
-          if ENV["FRACTOR_DEBUG"]
-            puts "Warning: Received message from unknown Ractor: #{ready_ractor_obj}. Ignoring."
-          end
+          puts "Warning: Received message from unknown Ractor: #{ready_ractor_obj}. Ignoring." if ENV["FRACTOR_DEBUG"]
           next
         end
 
-        if ENV["FRACTOR_DEBUG"]
-          puts "Selected Ractor: #{wrapped_ractor.name}, Message Type: #{message[:type]}"
-        end
+        puts "Selected Ractor: #{wrapped_ractor.name}, Message Type: #{message[:type]}" if ENV["FRACTOR_DEBUG"]
 
         # Process the received message
         case message[:type]
         when :initialize
-          if ENV["FRACTOR_DEBUG"]
-            puts "Ractor initialized: #{message[:processor]}"
-          end
+          puts "Ractor initialized: #{message[:processor]}" if ENV["FRACTOR_DEBUG"]
           # Send work immediately upon initialization if available
           send_next_work_if_available(wrapped_ractor)
         when :result
           # The message[:result] should be a WorkResult object
           work_result = message[:result]
-          if ENV["FRACTOR_DEBUG"]
-            puts "Completed work: #{work_result.inspect} in Ractor: #{message[:processor]}"
-          end
+          puts "Completed work: #{work_result.inspect} in Ractor: #{message[:processor]}" if ENV["FRACTOR_DEBUG"]
           @results.add_result(work_result)
           if ENV["FRACTOR_DEBUG"]
             puts "Result processed. Total processed: #{@results.results.size + @results.errors.size}"
@@ -194,9 +182,7 @@ module Fractor
         when :error
           # The message[:result] should be a WorkResult object containing the error
           error_result = message[:result]
-          if ENV["FRACTOR_DEBUG"]
-            puts "Error processing work #{error_result.work&.inspect} in Ractor: #{message[:processor]}: #{error_result.error}"
-          end
+          puts "Error processing work #{error_result.work&.inspect} in Ractor: #{message[:processor]}: #{error_result.error}" if ENV["FRACTOR_DEBUG"]
           @results.add_result(error_result) # Add error to aggregator
           if ENV["FRACTOR_DEBUG"]
             puts "Error handled. Total processed: #{@results.results.size + @results.errors.size}"
@@ -205,17 +191,13 @@ module Fractor
           # Send next piece of work even after an error
           send_next_work_if_available(wrapped_ractor)
         else
-          if ENV["FRACTOR_DEBUG"]
-            puts "Unknown message type received: #{message[:type]} from #{wrapped_ractor.name}"
-          end
+          puts "Unknown message type received: #{message[:type]} from #{wrapped_ractor.name}" if ENV["FRACTOR_DEBUG"]
         end
         # Update processed count for the loop condition
         processed_count = @results.results.size + @results.errors.size
       end
 
-      if ENV["FRACTOR_DEBUG"]
-        puts "Main loop finished."
-      end
+      puts "Main loop finished." if ENV["FRACTOR_DEBUG"]
       return if @continuous_mode
 
       return unless ENV["FRACTOR_DEBUG"]
@@ -238,17 +220,11 @@ module Fractor
         if !@work_queue.empty?
           work_item = @work_queue.pop # Now directly a Work object
 
-          if ENV["FRACTOR_DEBUG"]
-            puts "Sending next work #{work_item.inspect} to Ractor: #{wrapped_ractor.name}"
-          end
+          puts "Sending next work #{work_item.inspect} to Ractor: #{wrapped_ractor.name}" if ENV["FRACTOR_DEBUG"]
           wrapped_ractor.send(work_item) # Send the Work object
-          if ENV["FRACTOR_DEBUG"]
-            puts "Work sent to #{wrapped_ractor.name}."
-          end
+          puts "Work sent to #{wrapped_ractor.name}." if ENV["FRACTOR_DEBUG"]
         else
-          if ENV["FRACTOR_DEBUG"]
-            puts "Work queue empty. Not sending new work to Ractor #{wrapped_ractor.name}."
-          end
+          puts "Work queue empty. Not sending new work to Ractor #{wrapped_ractor.name}." if ENV["FRACTOR_DEBUG"]
           # In continuous mode, don't close workers as more work may come
           unless @continuous_mode
             # Consider closing the Ractor if the queue is empty and no more work is expected.
@@ -260,9 +236,7 @@ module Fractor
           end
         end
       else
-        if ENV["FRACTOR_DEBUG"]
-          puts "Attempted to send work to an invalid or closed Ractor: #{wrapped_ractor&.name || "unknown"}."
-        end
+        puts "Attempted to send work to an invalid or closed Ractor: #{wrapped_ractor&.name || "unknown"}." if ENV["FRACTOR_DEBUG"]
         # Remove from map if found but closed
         @ractors_map.delete(wrapped_ractor.ractor) if wrapped_ractor && @ractors_map.key?(wrapped_ractor.ractor)
       end

data/lib/fractor/version.rb

@@ -2,5 +2,5 @@
 
 module Fractor
   # Fractor version
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

data/lib/fractor/worker.rb

@@ -4,6 +4,11 @@ module Fractor
   # Base class for defining work processors.
   # Subclasses must implement the `process` method.
   class Worker
+    def initialize(name: nil, **options)
+      @name = name
+      @options = options
+    end
+
     def process(work)
       raise NotImplementedError, "Subclasses must implement the 'process' method."
     end

data/lib/fractor/wrapped_ractor.rb

@@ -25,7 +25,7 @@ module Fractor
         Ractor.yield({ type: :initialize, processor: name })
 
         # Instantiate the specific worker inside the Ractor
-        worker = worker_cls.new
+        worker = worker_cls.new(name: name)
 
         loop do
           # Ractor.receive will block until a message is received

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fractor
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Ronald Tse
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-05-07 00:00:00.000000000 Z
+date: 2025-05-08 00:00:00.000000000 Z
 dependencies: []
 description: Fractor is a lightweight Ruby framework designed to simplify the process
   of distributing computational work across multiple Ractors.