ractorize 0.0.9 → 0.0.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 72a53fba452511ad30a39dc46714c285570a2904f167c960e97d519e63ee1829
-  data.tar.gz: 77c170a01ff7ec50bfb63c8c70b905be6d999bdaa2b8cd0e3826afd7bc5bb9d1
+  metadata.gz: 4600c739ac97a907068b19020d82d72c695df5dba8e47c56620cada3418de052
+  data.tar.gz: ea91548851760e76582f06e9dc6e49bc4626a648442ce7503b59b40df6c47ccf
 SHA512:
-  metadata.gz: 3655da0fc231e1d6b5c9bd063cf8e676ab6a9ce7f3b09421e48d085978257574bbbf988d538acb190dd0512d3838b5fe49a9436fa85e65f67a09aebfe137ff8b
-  data.tar.gz: 0dad22a3bf13eedff038063711e18e473b25f5e6e760b3b400de4603996240f71d3635788c8f06e2705da2a8103df236c5c76f94303ff44900122baedd368a9d
+  metadata.gz: 198181e33f7fdab8ca7a98713d39b0dadffe240273353e6d9f28de6fbda3be139d9cd4f92df99e678231a05bcb41a276547f16d2553bb6e1a09d56e632cb615f
+  data.tar.gz: adfd5a6ab472f0d1b3479ed51cc401b0112bc563798594863067c7f6c4cd3284210a1e7a148e2ffc9645607dd3bbab5e977ffcc60201ae62dea0edc71f40b30f

data/CHANGELOG.md

@@ -1,3 +1,11 @@
+## [0.0.11] - 2026-07-31
+
+- Log if a RactorizedRactor is taken down via ClosedError
+
+## [0.0.10] - 2026-06-30
+
+- Handle thunks being cloned
+
 ## [0.0.9] - 2026-06-29
 
 - Fix bug that results from moving Time, which is neither shareable nor movable!

data/README.md

@@ -274,6 +274,15 @@ Note that if you call either `#__close__` or `#__join__` on the object, then the
 
 An easy way to avoid the deadlock is just don't make any use of such an object after closing it.
 
+### Objects returned from a method called on a ractorized object might be cloned!
+
+You might have a "void" method where you don't care about the return value but it also doesn't
+return `nil` instead it just returns the last expression arbitrarily.
+
+This return value (or parts of it) will be cloned by CRuby/Ractor if it's not shareable!
+
+Confusing stuff can happen with certain types of objects when cloned, such as open files.
+
 ## Fine print
 
 Ractors are still experimental and so this gem is also still experimental.

data/src/ractorize/garbage_collection/tracker.rb

@@ -2,11 +2,13 @@ module Ractorize
   module GarbageCollection
     class Tracker
       attr_accessor :ractorized_object_id_to_ractor,
-                    :thunk_id_to_ractor
+                    :thunk_id_to_ractor,
+                    :ractor_to_thunk_ids
 
       def initialize
         self.ractorized_object_id_to_ractor = ObjectSpace::WeakMap.new
         self.thunk_id_to_ractor = ObjectSpace::WeakMap.new
+        self.ractor_to_thunk_ids = ObjectSpace::WeakKeyMap.new
       end
 
       def track_ractorized_object(ractorized_object)
@@ -24,9 +26,32 @@ module Ractorize
         thunk_id_to_ractor[thunk_id] = ractor
       end
 
+      def thunk_cloned(old_thunk_id, new_thunk_id, thunk_ractor)
+        thunk_ids = ractor_to_thunk_ids[thunk_ractor]
+
+        if thunk_ids
+          thunk_ids << new_thunk_id
+        else
+          ractor_to_thunk_ids[thunk_ractor] = [old_thunk_id, new_thunk_id]
+        end
+
+        thunk_id_to_ractor[new_thunk_id] = thunk_ractor
+      end
+
       def cleanup_after_thunk(thunk_id)
         ractor = thunk_id_to_ractor.delete(thunk_id)
-        ractor&.<<(:__close__)
+
+        return unless ractor
+
+        thunk_ids = ractor_to_thunk_ids[ractor]
+
+        if thunk_ids
+          thunk_ids.delete(thunk_id)
+
+          return unless thunk_ids.empty?
+        end
+
+        ractor << :__close__
       rescue Ractor::ClosedError
         # do nothing
       end

data/src/ractorize/garbage_collection/tracking_ractor.rb

@@ -10,20 +10,7 @@ module Ractorize
             tracker = Tracker.new
 
             loop do
-              # SimpleCov branch coverage doesn't like that we aren't testing not matching anything
-              # but this does result in an error unlike case/when so no point in checking that.
-              # :nocov:
-              case receive
-                # :nocov:
-              in :track_ractorized_object, ractorized_object
-                tracker.track_ractorized_object(ractorized_object)
-              in :cleanup_after_ractorized_object, ractorized_object_id
-                tracker.cleanup_after_ractorized_object(ractorized_object_id)
-              in :track_thunk, thunk_id, thunk_ractor
-                tracker.track_thunk(thunk_id, thunk_ractor)
-              in :cleanup_after_thunk, thunk_id
-                tracker.cleanup_after_thunk(thunk_id)
-              end
+              tracker.send(*receive)
             rescue TrackingRactor::ClosedError
               # do nothing
             end

data/src/ractorize/garbage_collection.rb

@@ -14,7 +14,7 @@ module Ractorize
       end
 
       def track_thunk(thunk)
-        # We have to define the finalizer here, not in the tracker, because it's not frozen yet
+        # We have to define the finalizer here, not in the tracker, because it's not shareable
         ObjectSpace.define_finalizer(thunk, &finalize_thunk_proc)
 
         begin
@@ -24,6 +24,24 @@ module Ractorize
         end
       end
 
+      def thunk_cloned(old_thunk, new_thunk)
+        ractor = old_thunk.__thunk_ractor__
+
+        # We have to define the finalizer here, not in the tracker, because it's not shareable
+        # ObjectSpace.define_finalizer(new_thunk, &finalize_thunk_proc)
+
+        begin
+          TRACKING_RACTOR << [
+            :thunk_cloned,
+            old_thunk.__object_id__,
+            new_thunk.__object_id__,
+            ractor
+          ].freeze
+        rescue TrackingRactor::ClosedError
+          # do nothing
+        end
+      end
+
       def cleanup_after_ractorized_object(ractorized_object_id)
         TRACKING_RACTOR << [:cleanup_after_ractorized_object, ractorized_object_id].freeze
       rescue Ractor::ClosedError

data/src/ractorize/ractorized_object/ractorized_ractor.rb

@@ -3,6 +3,8 @@ require_relative "../base_ractor"
 module Ractorize
   class RactorizedObject < BasicObject
     class RactorizedRactor < ::BaseRactor
+      class UnexpectedClosedError < StandardError; end
+
       class << self
         def new(name: nil)
           super do
@@ -25,15 +27,13 @@ module Ractorize
 
                        klass.new(*args.freeze, **opts.freeze, &block)
                      else
-                       # :nocov:
+                       # simplecov:disable
                        raise "Invalid mode #{mode}"
-                       # :nocov:
+                       # simplecov:enable
                      end
 
             loop do
-              # rubocop:disable Lint/UselessAssignment
               value = method_name = method_args = opts = return_port = thunk_ractor = block_given = nil
-              # rubocop:enable Lint/UselessAssignment
               method_name, method_args, opts, return_port, thunk_ractor, block_given = receive
 
               case method_name
@@ -80,9 +80,9 @@ module Ractorize
                       # TODO: handle error situation
                       break
                     else
-                      # :nocov:
+                      # simplecov:disable
                       raise "Not sure how to handle outcome_type #{outcome_type}"
-                      # :nocov:
+                      # simplecov:enable
                     end
                   end
 
@@ -100,10 +100,10 @@ module Ractorize
                   rescue IOError => e
                     # Unclear why this sometimes manifests as this error instead of ClosedError but
                     # need to handle them both.
-                    # :nocov:
+                    # simplecov:disable
                     raise unless e.message == "closed stream"
-                    # :nocov:
-                  rescue Ractor::ClosedError
+                    # simplecov:enable
+                  rescue RactorizedRactor::ClosedError
                     # Whoa... this error inherits from StopIteration and will kill the loop!!!
                     # Nothing really to do here but keep the loop going and handle other
                     # method calls to the ractorized object from other ractors.
@@ -112,19 +112,28 @@ module Ractorize
               end
 
               nil
+            rescue RactorizedRactor::ClosedError => e
+              # simplecov:disable
+              puts "unexpected closed error!"
+              puts e.backtrace
+              error = UnexpectedClosedError.new
+              error.set_backtrace(e.backtrace)
+
+              raise error
+              # simplecov:enable
             end
 
             nil
           # object
           rescue => e
-            # :nocov:
+            # simplecov:disable
             puts
             puts "an unhandled error!!! #{e.class} #{e.message} #{e}"
             puts e.backtrace
             puts
 
             raise
-            # :nocov:
+            # simplecov:enable
           end
         end
       end

data/src/ractorize/ractorized_object.rb

@@ -67,9 +67,9 @@ module Ractorize
         @__ractor__ << [:__target_object_id__, return_port].freeze
         @__target_object_id__ = return_port.receive
       else
-        # :nocov:
+        # simplecov:disable
         ::Kernel.raise "Invalid mode #{mode}"
-        # :nocov:
+        # simplecov:enable
       end
 
       @__object_id__ = ::Object.instance_method(:object_id).bind_call(self)
@@ -131,9 +131,9 @@ module Ractorize
           # Seems SimpleCov branch coverage doesn't like that we don't test the non-exhaustive
           # pattern path, but since that's purely defensive I have no interest in testing it.
 
-          # :nocov:
+          # simplecov:disable
           case data
-          # :nocov:
+          # simplecov:enable
           in :return, value
             stop = true
           in :yield, [yielded_args, yielded_opts, yielded_block], block_result_port
@@ -167,19 +167,19 @@ module Ractorize
         value = return_port.receive
         return_port.close
 
-        # :nocov:
+        # simplecov:disable
         ::Kernel.raise ::Ractorize::Thunk::EscapingRactorError if ::Ractorize::Thunk === value
-        # :nocov:
+        # simplecov:enable
 
         value
       end
     end
 
     def respond_to?(method_name, include_all = false)
-      # :nocov:
+      # simplecov:disable
       # This line is only here for when commenting out < BasicObject when debugging stuff
       return super if ::Object === self
-      # :nocov:
+      # simplecov:enable
 
       respond_to_missing?(method_name, include_all)
     end

data/src/ractorize/thunk/thunk_ractor.rb

@@ -7,9 +7,9 @@ module Ractorize
         def new
           super do
             # SimpleCov seems to want us to handle the case where nothing matches but that would be an error
-            # :nocov:
+            # simplecov:disable
             case receive
-            # :nocov:
+            # simplecov:enable
             in :__close__
               # do nothing
             in :success, value

data/src/ractorize/thunk.rb

@@ -10,13 +10,20 @@ module Ractorize
       ::Ractorize::GarbageCollection.track_thunk(self)
     end
 
-    def initialize_clone(...)
-      # :nocov:
-      raise "CAREFUL! THUNK CLONED!!"
-      # :nocov:
-      # is this actually necessary?? Seems so?
+    def initialize_clone(original_thunk)
+      if __resolved__? && original_thunk.__resolved__?
+        # Seems this could happen if we had a frozen thunk whose @__value__ is not shareable
+        # Since the thunk's ractor is already gone nothing to worry about
+        return
+      end
+
+      self.__object_id__ = ::Object.instance_method(:object_id).bind_call(self)
+
+      ::Ractorize::GarbageCollection.thunk_cloned(original_thunk, self)
     end
 
+    def __resolved__? = defined?(@__value__)
+
     def method_missing(...)
       __value__.__send__(...)
     end

data/src/ractorize.rb

@@ -25,16 +25,16 @@ module Ractorize
       @auto_freeze = @auto_freeze ? @auto_freeze.dup : []
 
       unless Ractor.shareable?(target)
-        # :nocov:
+        # simplecov:disable
         raise "#{target} isn't shareable so can't use it to auto-freeze"
-        # :nocov:
+        # simplecov:enable
       end
 
       @auto_freeze << if class_or_proc
                         unless Ractor.shareable?(class_or_proc)
-                          # :nocov:
+                          # simplecov:disable
                           raise "#{class_or_proc} isn't shareable so can't use it to auto-freeze"
-                          # :nocov:
+                          # simplecov:enable
                         end
 
                         [target, class_or_proc]
@@ -49,16 +49,16 @@ module Ractorize
       @move_arg = @move_arg ? @move_arg.dup : []
 
       unless Ractor.shareable?(target)
-        # :nocov:
+        # simplecov:disable
         raise "#{target} isn't shareable so can't use it to auto-freeze"
-        # :nocov:
+        # simplecov:enable
       end
 
       @move_arg << if class_or_proc
                      unless Ractor.shareable?(class_or_proc)
-                       # :nocov:
+                       # simplecov:disable
                        raise "#{class_or_proc} isn't shareable so can't use it to auto-freeze"
-                       # :nocov:
+                       # simplecov:enable
                      end
 
                      [target, class_or_proc]
@@ -189,9 +189,9 @@ module Ractorize
         when :done
           break
         else
-          # :nocov:
+          # simplecov:disable
           ::Kernel.raise "Unknown class_by_arg arg type #{arg_type}"
-          # :nocov:
+          # simplecov:enable
         end
       end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ractorize
 version: !ruby/object:Gem::Version
-  version: 0.0.9
+  version: 0.0.11
 platform: ruby
 authors:
 - Miles Georgi
@@ -46,13 +46,16 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ">="
     - !ruby/object:Gem::Version
       version: 4.0.0
+  - - "<"
+    - !ruby/object:Gem::Version
+      version: '4.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.10
+rubygems_version: 4.0.16
 specification_version: 4
 summary: Turn objects into ractors with ease!
 test_files: []