foobara 0.1.3 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 382dbd4fb1743f89af17bc70f7acdbef0ad2851767122f702e2503dcfacddf12
-  data.tar.gz: 02dfabf373e9ec1545ecbcaeeed6c31b0f6338bcc5e9095b3e0efe9bc9f61c03
+  metadata.gz: b3eea1befe835410f5c6ecc372c5db94e3641f90711a773b0a7685a18afe677f
+  data.tar.gz: 7fe5e33084023a8d475e0fe491520a179d9123b5109c0ebc2ad52ed5643752bd
 SHA512:
-  metadata.gz: 535af52cbd9ad41609694db0b23e69f003d5959ccff113bc98dcedc13ff15df685b6abfcafe510f26243888d979b4724b061d00a76164805167d8f90cc3f4d39
-  data.tar.gz: 692827d4929a901e5127c706c5f63d5da40330dfc5a5e868ea0661ef5885b25dbb5a4db43f8b76e980076ded3d876637ea1328bd164a16fed9e86664d99b9d4c
+  metadata.gz: 67bbdfab3b295bbe38f39d978f2855a239092f9feba4a417bbb853759dae9527dcbd0ba3a34647925f2b90fc15693c03ddb56a5d5e9247678c22c236deedba2c
+  data.tar.gz: bc09a9a484f9c1e6487a7ce112e33aabb5132b77071f195ed06cafb79895c070ee895be8e3cdd01abf75c4fc4eb54c51dd23d0bd4b22f7d1e0ef3b584c5874d7

data/CHANGELOG.md

@@ -1,4 +1,22 @@
-# [0.1.3] - 2025-08-25
+# [0.1.6] - 2025-08-25
+
+- Allow constructing a thunk-like record when removing sensitive values from a thunk
+- Undo automatically converting connected entities to detached entities
+- Honor imported builtin types in manifest
+- Add Foobara version to manifest metadata
+- Add RootManifest#detached_entities
+- Remove some items from the manifest that should be safe to remove
+- Let's use GlobalDomain in the manifest for objects scoped to Foobara::Value
+
+# [0.1.4] - 2025-08-24
+
+- Fix TransformedCommand#foobara_manifest bug that failed to overwrite transformed possible_errors
+  if the new possible errors is empty
+- Make sure we set nil inputs_type in TransformedCommand#foobara_manifest to empty attributes
+  for compatibility with other projects
+- Sets the parent for various Foobara::Value children to be GlobalDomain in manifests
+
+# [0.1.3] - 2025-08-24
 
 - Add Command#abstract and exclude abstract commands from CommandConnector#connect when connecting domains/orgs
 

data/projects/builtin_types/src/builtin_types.rb

@@ -5,7 +5,9 @@ module Foobara
 
   module BuiltinTypes
     class << self
-      foobara_delegate :global_type_declaration_handler_registry, to: TypeDeclarations
+      def global_type_declaration_handler_registry
+        TypeDeclarations.global_type_declaration_handler_registry
+      end
 
       # TODO: break this up
       # TODO: much of this behavior is helpful to non-builtin types as well.

data/projects/callback/src/registry/chained_conditioned.rb

@@ -6,7 +6,13 @@ module Foobara
       class ChainedConditioned < Conditioned
         attr_accessor :other_conditions_registry
 
-        foobara_delegate :possible_conditions, :possible_condition_keys, to: :other_conditions_registry
+        def possible_conditions
+          other_conditions_registry.possible_conditions
+        end
+
+        def possible_condition_keys
+          other_conditions_registry.possible_condition_keys
+        end
 
         def initialize(other_conditions_registry)
           self.other_conditions_registry = other_conditions_registry

data/projects/callback/src/registry/chained_multiple_action.rb

@@ -8,7 +8,13 @@ module Foobara
 
         class InvalidConditions < StandardError; end
 
-        foobara_delegate :possible_actions, :allowed_types, to: :other_multiple_actions_registry
+        def possible_actions
+          other_multiple_actions_registry.possible_actions
+        end
+
+        def allowed_types
+          other_multiple_actions_registry.allowed_types
+        end
 
         def initialize(other_multiple_actions_registry)
           self.other_multiple_actions_registry = other_multiple_actions_registry

data/projects/callback/src/registry/joined_conditioned.rb

@@ -6,7 +6,13 @@ module Foobara
       class JoinedConditioned < Conditioned
         attr_accessor :first, :second
 
-        foobara_delegate :possible_conditions, :possible_condition_keys, to: :first
+        def possible_conditions
+          first.possible_conditions
+        end
+
+        def possible_condition_keys
+          first.possible_condition_keys
+        end
 
         def initialize(first, second)
           self.first = first

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

@@ -53,7 +53,9 @@ module Foobara
           super
         end
 
-        foobara_delegate :has_errors?, to: :error_collection
+        def has_errors?
+          error_collection.has_errors?
+        end
 
         private
 

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

@@ -61,7 +61,9 @@ module Foobara
           end
         end
 
-        foobara_delegate :inputs_type, to: :class
+        def inputs_type
+          self.class.inputs_type
+        end
       end
     end
   end

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

@@ -18,7 +18,9 @@ module Foobara
           end
         end
 
-        foobara_delegate :result_type, :raw_result_type_declaration, to: :class
+        def result_type
+          self.class.result_type
+        end
       end
     end
   end

data/projects/command_connectors/src/command_connector/commands/describe.rb

@@ -1,3 +1,5 @@
+require_relative "../../../../../version"
+
 module Foobara
   class CommandConnector
     module Commands
@@ -20,7 +22,7 @@ module Foobara
         end
 
         def stamp_request_metadata
-          manifest[:metadata] = { when: Time.now }
+          manifest[:metadata] = { when: Time.now, foobara_version: Foobara::Version::VERSION }
         end
       end
     end

data/projects/command_connectors/src/command_connector.rb

@@ -352,7 +352,9 @@ module Foobara
           args = args[1..]
           connected = []
 
-          registerable.foobara_each_command(mode: Namespace::LookupMode::DIRECT) do |command_class|
+          registerable = registerable.foobara_all_command(mode: Namespace::LookupMode::DIRECT)
+
+          registerable.each do |command_class|
             unless command_class.abstract?
               connected << connect(command_class, *args, **opts)
             end
@@ -567,29 +569,6 @@ module Foobara
                       additional_to_include << exposed_organization
                     end
                   end
-
-                  if o.extends_type?(BuiltinTypes[:entity])
-                    declaration_data = o.declaration_data
-
-                    if declaration_data.is_a?(::Hash) && declaration_data[:type] == :entity
-                      if o.foobara_root_namespace != command_registry.foobara_root_namespace
-                        # Let's swap it out with a detached type
-                        detached_entity = command_registry.foobara_lookup_type(o.scoped_full_name, mode:)
-
-                        unless detached_entity
-                          declaration_data = o.declaration_data.merge(
-                            type: :detached_entity,
-                            model_base_class: "Foobara::DetachedEntity"
-                          )
-
-                          detached_entity = exposed_domain.foobara_type_from_declaration(declaration_data)
-                        end
-
-                        additional_to_include << detached_entity
-                        next
-                      end
-                    end
-                  end
                 end
               end
 
@@ -637,7 +616,6 @@ module Foobara
       end
 
       h = normalize_manifest(h)
-
       patch_up_broken_parents_for_errors_with_missing_command_parents(h)
     end
 

data/projects/command_connectors/src/transformed_command.rb

@@ -61,10 +61,11 @@ module Foobara
         end
       end
 
-      foobara_delegate :description,
-                       :domain,
-                       :organization,
-                       to: :command_class
+      foobara_delegate :domain, :organization, to: :command_class
+
+      def description
+        command_class.description
+      end
 
       def result_transformers
         return @result_transformers if @considered_sensitive_value_remover
@@ -350,7 +351,7 @@ module Foobara
         response_mutators = processors_to_manifest_symbols(self.response_mutators)
         request_mutators = processors_to_manifest_symbols(self.request_mutators)
 
-        authenticator_details = if authenticator
+        authenticator_details = if requires_authentication && authenticator
                                   {
                                     symbol: authenticator.symbol,
                                     explanation: authenticator.explanation
@@ -369,27 +370,59 @@ module Foobara
           command_class.foobara_manifest
         end
 
+        # TODO: This should support nil as an inputs_type but it breaks other projects for now
+        inputs_type = inputs_type_for_manifest&.reference_or_declaration_data ||
+                      { type: :attributes, element_type_declarations: {} }
+
         # TODO: handle errors_types_depended_on!
-        manifest.merge(
-          Util.remove_blank(
-            inputs_types_depended_on:,
-            result_types_depended_on:,
-            types_depended_on: types,
-            inputs_type: inputs_type_for_manifest&.reference_or_declaration_data,
-            result_type: result_type&.reference_or_declaration_data,
-            possible_errors: possible_errors_manifest,
-            capture_unknown_error:,
-            inputs_transformers:,
-            result_transformers:,
-            errors_transformers:,
-            pre_commit_transformers:,
-            serializers:,
-            response_mutators:,
-            request_mutators:,
-            requires_authentication:,
-            authenticator: authenticator_details
-          )
-        )
+        to_merge = {
+          inputs_types_depended_on:,
+          result_types_depended_on:,
+          types_depended_on: types,
+          inputs_type:,
+          result_type: result_type&.reference_or_declaration_data,
+          possible_errors: possible_errors_manifest,
+          capture_unknown_error:,
+          inputs_transformers:,
+          result_transformers:,
+          errors_transformers:,
+          pre_commit_transformers:,
+          serializers:,
+          response_mutators:,
+          request_mutators:,
+          requires_authentication:,
+          authenticator: authenticator_details
+        }
+
+        manifest = manifest.dup
+
+        # TODO: we should remove all blank valued keys but keeping many for now for stability
+        safe_to_remove = [
+          :pre_commit_transformers,
+          :response_mutators,
+          :request_mutators,
+          :capture_unknown_error,
+          :inputs_transformers,
+          :result_transformers,
+          :errors_transformers,
+          :requires_authentication,
+          :authenticator
+        ]
+
+        to_merge.each_pair do |key, value|
+          # TODO: we could probably remove empty strings and nils, too, and from the whole hash
+          if value.nil? || ((value.is_a?(::Hash) || value.is_a?(::Array)) && value.empty?)
+            if safe_to_remove.include?(key)
+              manifest.delete(key)
+            else
+              manifest[key] = value
+            end
+          else
+            manifest[key] = value
+          end
+        end
+
+        manifest
       end
 
       def processors_to_manifest_symbols(processors)

data/projects/command_connectors/src/transformers/entity_to_primary_key_inputs_transformer.rb

@@ -54,6 +54,7 @@ module Foobara
 
               description = "#{to_type.target_class.model_name} #{to_type.target_class.primary_key_attribute}"
 
+              # TODO: doesn't everything that extends :entity also extend :detached_entity?
               unless to_type.extends_directly?(Foobara::BuiltinTypes[:detached_entity]) ||
                      to_type.extends_directly?(Foobara::BuiltinTypes[:entity])
 

data/projects/common/src/outcome.rb

@@ -55,11 +55,15 @@ module Foobara
       end
     end
 
-    foobara_delegate :has_errors?,
-                     :has_error?,
-                     :add_error,
-                     :add_errors,
-                     to: :error_collection
+    foobara_delegate :has_error, :add_errors, to: :error_collection
+
+    def has_errors?
+      error_collection.has_errors?
+    end
+
+    def add_errors(...)
+      error_collection.add_errors(...)
+    end
 
     def errors
       error_collection

data/projects/detached_entity/src/concerns/associations.rb

@@ -157,13 +157,11 @@ module Foobara
             result = {},
             initial: true
           )
-            if initial && type.extends?(BuiltinTypes[:detached_entity])
-              return construct_associations(type.element_types, path, result, initial: false)
-            end
-
             remove_sensitive = TypeDeclarations.foobara_manifest_context_remove_sensitive?
 
-            if type.extends?(BuiltinTypes[:entity])
+            if initial && type.extends?(BuiltinTypes[:detached_entity])
+              construct_associations(type.target_class.foobara_attributes_type, path, result, initial: false)
+            elsif type.extends?(BuiltinTypes[:entity])
               result[path.to_s] = type
             elsif type.extends?(BuiltinTypes[:tuple])
               element_types = type.element_types

data/projects/detached_entity/src/extensions/type_declarations/handlers/extend_detached_entity_type_declaration/to_type_transformer.rb

@@ -3,6 +3,7 @@ module Foobara
     module Handlers
       class ExtendDetachedEntityTypeDeclaration < ExtendModelTypeDeclaration
         class ToTypeTransformer < ExtendModelTypeDeclaration::ToTypeTransformer
+          # TODO: use constants for these arrays for performance reasons
           def non_processor_keys
             [:primary_key, *super]
           end

data/projects/domain/src/is_manifestable.rb

@@ -63,16 +63,26 @@ module Foobara
 
         if parent_category
           if include_processors || (parent_category != :processor && parent_category != :processor_class)
-            if candidate != Foobara::Value
-              parent = candidate
-              break
-            end
+            parent = if candidate == Foobara::Value
+                       GlobalDomain
+                     else
+                       candidate
+                     end
+            break
           end
         end
 
         candidate = candidate.scoped_namespace
       end
 
+      if parent == GlobalDomain
+        h[:scoped_path] = scoped_full_path
+        h[:scoped_name] = scoped_full_name
+        h[:scoped_prefix] = scoped_full_path[..-2]
+        h[:domain] = parent.foobara_manifest_reference
+        h[:organization] = parent.foobara_organization.foobara_manifest_reference
+      end
+
       if parent
         to_include << parent
         h[:parent] = [parent_category, parent.foobara_manifest_reference]

data/projects/entity/src/concerns/callbacks.rb

@@ -35,7 +35,9 @@ module Foobara
           end
         end
 
-        foobara_delegate :register_callback, to: :callback_registry
+        def register_callback(...)
+          callback_registry.register_callback(...)
+        end
 
         module ClassMethods
           def class_callback_registry
@@ -68,7 +70,13 @@ module Foobara
             end
           end
 
-          foobara_delegate :register_callback, :possible_actions, to: :class_callback_registry
+          def register_callback(...)
+            class_callback_registry.register_callback(...)
+          end
+
+          def possible_actions
+            class_callback_registry.possible_actions
+          end
         end
 
         on_include do

data/projects/entity/src/extensions/builtin_types/entity/validators/model_instance_is_valid.rb

@@ -4,7 +4,7 @@ module Foobara
       module Validators
         class ModelInstanceIsValid < DetachedEntity::Validators::ModelInstanceIsValid
           def applicable?(record)
-            record && (record.created? || record.built?)
+            record && (record.created? || record.built?) && !record.skip_validations
           end
 
           def expected_type_symbol

data/projects/entity/src/sensitive_type_removers/entity.rb

@@ -7,10 +7,14 @@ module Foobara
 
           if strict_type_declaration != new_type_declaration
             if new_type_declaration[:type] == :entity
-              new_type_declaration[:type] = :detached_entity
+              if Namespace.current.foobara_root_namespace == Namespace.global.foobara_root_namespace
+                # Nervous about creating two entities with the same name in the same namespace
+                # So going to create a detached entity instead
+                new_type_declaration[:type] = :detached_entity
 
-              if new_type_declaration[:model_base_class] == "Foobara::Entity"
-                new_type_declaration[:model_base_class] = "Foobara::DetachedEntity"
+                if new_type_declaration[:model_base_class] == "Foobara::Entity"
+                  new_type_declaration[:model_base_class] = "Foobara::DetachedEntity"
+                end
               end
             end
           end

data/projects/entity/src/sensitive_value_removers/entity.rb

@@ -8,11 +8,10 @@ module Foobara
           elsif record.persisted?
             # We will assume that we do not need to clean up the primary key itself as
             # we will assume we don't allow sensitive primary keys for now.
-            # We use .new because the target_class should be a detached entity
-            to_type.target_class.new(
-              { record.class.primary_key_attribute => record.primary_key },
-              { mutable: false, skip_validations: true }
-            )
+            thunkish = to_type.target_class.build(record.class.primary_key_attribute => record.primary_key)
+            thunkish.skip_validations = true
+            thunkish.mutable = false
+            thunkish
           else
             # :nocov:
             raise "Not sure what to do with a record that isn't loaded, created, or persisted"
@@ -21,14 +20,7 @@ module Foobara
         end
 
         def build_method
-          if to_type.extends_type?(BuiltinTypes[:entity])
-            # TODO: test this code path
-            # :nocov:
-            :build
-            # :nocov:
-          else
-            :new
-          end
+          :build
         end
       end
     end

data/projects/manifest/src/foobara/manifest/organization.rb

@@ -21,6 +21,10 @@ module Foobara
         domains.map(&:types).flatten
       end
 
+      def detached_entities
+        domains.map(&:detached_entities).flatten
+      end
+
       def entities
         domains.map(&:entities).flatten
       end

data/projects/manifest/src/foobara/manifest/root_manifest.rb

@@ -31,6 +31,10 @@ module Foobara
         end
       end
 
+      def detached_entities
+        organizations.map(&:detached_entities).flatten
+      end
+
       def entities
         organizations.map(&:entities).flatten
       end

data/projects/manifest/src/foobara/manifest/type.rb

@@ -72,7 +72,7 @@ module Foobara
 
       # TODO: replace this with primitive?
       def builtin?
-        BuiltinTypes.builtin_reference?(reference)
+        BuiltinTypes.builtin_reference?(reference) || self[:builtin]
       end
 
       def primitive?

data/projects/persistence/src/entity_base/transaction/concerns/state_transitions.rb

@@ -7,7 +7,11 @@ module Foobara
 
         module Concerns
           module StateTransitions
-            foobara_delegate :close!, :currently_open?, to: :state_machine
+            foobara_delegate :close, to: :state_machine
+
+            def currently_open?
+              state_machine.currently_open?
+            end
 
             def open!
               state_machine.open! do

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

@@ -14,7 +14,9 @@ module Foobara
           self.tables = {}
         end
 
-        foobara_delegate :entity_attributes_crud_driver, to: :entity_base
+        def entity_attributes_crud_driver
+          entity_base.entity_attributes_crud_driver
+        end
 
         def create(entity_class, attributes = {})
           Persistence.to_base(entity_class).transaction(existing_transaction: self) do

data/version.rb

@@ -0,0 +1,6 @@
+module Foobara
+  module Version
+    VERSION = "0.1.6".freeze
+    MINIMUM_RUBY_VERSION = ">= 3.4.0".freeze
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foobara
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.6
 platform: ruby
 authors:
 - Miles Georgi
@@ -491,6 +491,7 @@ files:
 - projects/value/src/validator.rb
 - projects/weak_object_set/lib/foobara/weak_object_set.rb
 - projects/weak_object_set/src/weak_object_set.rb
+- version.rb
 homepage: https://foobara.com
 licenses:
 - MPL-2.0