foobara 0.0.110 → 0.0.111

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 71a254889aa452b7a219cb8d9b4c7d1c2eb2c21ea452ae783e62d601720c387b
-  data.tar.gz: 451cf0735d2c75026c92b92381e0e8a3766ed69a8a08eed739e6866e88f5b134
+  metadata.gz: 0ff2cd0b6537dbf44f0ef865a8fc6c8ac339d7b306cb67ddfcb744b74d2cb73d
+  data.tar.gz: cc3a822e6a43b1f2f27070c85fe2a4c07bc6a0274b963816fa3ef55bb531b597
 SHA512:
-  metadata.gz: 6d758bae9e990d477c7775966b69dd1a51ae1878b938fec278a76ebbb8dd88a417e6c68f3a11174929cb86f858294bf1207206704bba70c458035e3798b12597
-  data.tar.gz: 9f7c770b990f942884aaa79f45f7ce344aaabaa8a939536d869802c25ccc2ada3b7ddb476119a543d5546019874ae44929af29d5437d0574e447a4f8acc398ed
+  metadata.gz: 4f1a8853b814a569aeaa4ac67d53363fe5349658f9e9f8214f2c05d649154fcc6b60be0f249e2912485dbc0689ade4ae1e67e2e5b913f478ff8a4151756a1542
+  data.tar.gz: c52ccae166bf419ba503653a29aa4bde3b8094c2ba190f611e877eec13ada47bd787c448db5506def4f5cebb73748a215d4de0d2d6dc30325f5c110347ad12bd

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+# [0.0.111] - 2025-04-25
+
+- Fix bug in DomainMapper.depends_on
+- Allow mutator instances to be used by connectors not just classes
+- Some WeakObjectSet tweaks
+
 # [0.0.110] - 2025-04-23
 
 - Automatically load associations needed for delegated attributes for

data/projects/command/src/command_pattern_implementation/concerns/subcommands.rb

@@ -59,7 +59,8 @@ module Foobara
             if subcommand_classes.empty?
               return @depends_on if defined?(@depends_on)
 
-              @depends_on = if self == Foobara::Command
+              # TODO: get Command and DomainMapper out of here!
+              @depends_on = if self == Foobara::Command || self == Foobara::DomainMapper
                               Set.new
                             else
                               superclass.depends_on.dup

data/projects/command/src/transformed_command.rb

@@ -122,8 +122,14 @@ module Foobara
 
         mutated_result_type = result_type
 
-        response_mutators&.reverse&.each do |mutator|
-          mutated_result_type = mutator.instance.result_type_from(mutated_result_type)
+        mutators = if response_mutators.size == 1
+                     [response_mutator]
+                   else
+                     response_mutator&.processors&.reverse
+                   end
+
+        mutators&.each do |mutator|
+          mutated_result_type = mutator.result_type_from(mutated_result_type)
         end
 
         @result_type_for_manifest = mutated_result_type
@@ -134,8 +140,14 @@ module Foobara
 
         mutated_inputs_type = inputs_type
 
-        request_mutators&.each do |mutator|
-          mutated_inputs_type = mutator.instance.inputs_type_from(mutated_inputs_type)
+        mutators = if request_mutators.size == 1
+                     [request_mutator]
+                   else
+                     request_mutator&.processors&.reverse
+                   end
+
+        mutators&.each do |mutator|
+          mutated_inputs_type = mutator.inputs_type_from(mutated_inputs_type)
         end
 
         @inputs_type_for_manifest = mutated_inputs_type

data/projects/persistence/src/entity_base/transaction_table/concerns/record_tracking.rb

@@ -64,9 +64,6 @@ module Foobara
             end
 
             def updated(record)
-              # Hacky way to handle situation where the primary key might have changed.
-              # TODO: make a better way of handling this.
-              tracked_records.delete(record)
               tracked_records << record
 
               # TODO: is this check redundant? Maybe have the entity explode directly instead?

data/projects/persistence/src/entity_base/transaction_table.rb

@@ -519,8 +519,6 @@ module Foobara
 
         def flush_created!
           marked_created.each do |record|
-            tracked_records.delete(record)
-
             flush_created_associations!(record)
 
             # TODO: do this in bulk
@@ -540,8 +538,6 @@ module Foobara
         end
 
         def flush_created_record!(record)
-          tracked_records.delete(record)
-
           flush_created_associations!(record)
 
           unmark_created(record)

data/projects/weak_object_set/src/weak_object_set.rb

@@ -97,22 +97,40 @@ module Foobara
     end
 
     def <<(object)
-      garbage_cleaner.track(object)
-
       object_id = object.object_id
 
-      objects[object_id] = WeakRef.new(object)
+      if include?(object)
+        if @key_method
+          key = object.send(@key_method)
+          old_key = object_id_to_key[object_id]
 
-      if @key_method
-        key = object.send(@key_method)
+          if key != old_key
+            key_to_object_id.delete(old_key)
 
-        if key
-          key_to_object_id[key] = object_id
-          object_id_to_key[object_id] = key
+            if key
+              key_to_object_id[key] = object_id
+              object_id_to_key[object_id] = key
+            else
+              object_id_to_key.delete(object_id)
+            end
+          end
         end
-      end
+      else
+        garbage_cleaner.track(object)
 
-      object
+        objects[object_id] = WeakRef.new(object)
+
+        if @key_method
+          key = object.send(@key_method)
+
+          if key
+            key_to_object_id[key] = object_id
+            object_id_to_key[object_id] = key
+          end
+        end
+
+        object
+      end
     end
 
     def include?(object_or_object_id)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foobara
 version: !ruby/object:Gem::Version
-  version: 0.0.110
+  version: 0.0.111
 platform: ruby
 authors:
 - Miles Georgi