foobara 0.7.0 → 0.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 55727f8ba357338d057069b782e94dc171a87ce95ae4d1865c5df1e46252559a
-  data.tar.gz: 5aa4e8a6535fdaf3bc1b3dd968b5b00348ef00c723d4728e313f9076b4a02b5c
+  metadata.gz: 64b471d0d0bf93acf6326e484b67d1be35f7c0c883bc315738f680e4c8119613
+  data.tar.gz: 9477f64763952ea8d819576545d6f9d06d7bb1ad562ea6dca83532297331859a
 SHA512:
-  metadata.gz: 239417815a5ead7dd367120cb67b50fb20d0750dacdd3f8519d76f2bc1e69f565f6217d7dd8d194bd199e6dbf9e7f1d2fd8783e081461da8888ee675f1fc9e7a
-  data.tar.gz: c06a234449b4ee52855f153277fe2440111f14873cc03dc9cc8c4c3a2fe558f811800ca6db8b0714d1e1fc2a54223290fb278c13c854b49dc2b708a5687f5ca2
+  metadata.gz: 941279f67447a52f9632f620637f011f21bae87766b4a8040846744751e17ba779a21db495cd0ef010152f908af015f8aad87246ab0215a9cd684673bb2c8793
+  data.tar.gz: de42b7abaff5512bdc00eba73d862778866034dd1a5d9b354649648a27b2ba2208062550e99ae4c91df8f51456f96bad4f7b4b89d2e263e13cc87c6d1daed231

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+# [0.7.1] - 2026-10-09
+
+- Fix bug that removes Foobara::RuntimeError from GlobalDomain during type declaration init
+
 # [0.7.0] - 2026-10-09
 
 - Do not allow anything but domains (and orgs as a hack) to be children of orgs

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

@@ -57,6 +57,8 @@ module Foobara
             if result_type
               result result_type.reference_or_declaration_data
             end
+
+            # TODO: We don't have to clear out possible error types??
           end
         end
       end

data/projects/typesystem/projects/builtin_types/lib/foobara/builtin_types.rb

@@ -62,6 +62,11 @@ module Foobara
         associative_array = build_and_register!(:associative_array, duckture, ::Hash)
         # TODO: uh oh... we do some translations in the casting here...
         build_and_register!(:attributes, associative_array, nil)
+
+        if needs_to_handle_reregistered_types
+          GlobalDomain.foobara_each_command(&:handle_reregistered_types!)
+          self.needs_to_handle_reregistered_types = false
+        end
       end
 
       def reset_all

data/projects/typesystem/projects/builtin_types/src/builtin_types.rb

@@ -5,6 +5,8 @@ module Foobara
 
   module BuiltinTypes
     class << self
+      attr_accessor :needs_to_handle_reregistered_types
+
       # TODO: break this up
       # TODO: much of this behavior is helpful to non-builtin types as well.
       def build_and_register!(
@@ -86,7 +88,17 @@ module Foobara
 
         # TODO: really need to encapsulate this somehow...
         type.type_symbol = type_symbol
-        (type.foobara_parent_namespace || GlobalDomain).foobara_register(type)
+
+        domain = type.foobara_parent_namespace || GlobalDomain
+        mode = Namespace::LookupMode::DIRECT
+
+        exists = domain.foobara_registered?(type_symbol, mode:)
+        if exists
+          domain.foobara_unregister(domain.foobara_lookup_type(type_symbol, mode:))
+          self.needs_to_handle_reregistered_types = true
+        end
+
+        domain.foobara_register(type)
 
         supported_casters_module = Util.constant_value(builtin_type_module, :SupportedCasters)
         supported_caster_classes = if supported_casters_module

data/projects/typesystem/projects/namespace/src/scoped.rb

@@ -7,7 +7,6 @@ module Foobara
 
     def scoped_path
       if scoped_unregistered?
-        # TODO: can probably delete this defensive check
         # simplecov:disable
         raise "Cannot use unregistered Scoped object"
         # simplecov:enable

data/projects/typesystem/projects/type_declarations/lib/foobara/type_declarations.rb

@@ -17,35 +17,56 @@ module Foobara
         [
           :@foobara_children,
           :@foobara_registry,
-          :@foobara_type_builder
+          :@foobara_type_builder,
+          :@lru_cache
         ].each do |var_name|
-          # Don't we only have to do this for Foobara and not all of these??
-          [
-            Namespace.global,
-            Foobara::GlobalOrganization,
-            Foobara::GlobalDomain,
-            Domain,
-            Types::Type,
-            Value::Processor,
-            Error
-          ].each do |klass|
+          namespaces_to_reset.each do |klass|
             if klass.instance_variable_defined?(var_name)
               klass.remove_instance_variable(var_name)
             end
           end
         end
 
-        @original_scoped.each do |scoped|
-          Namespace.global.foobara_register(scoped)
-        end
+        Namespace.clear_lru_cache!
+
+        @original_scoped.each_key do |namespace|
+          scoped = @original_scoped[namespace]
+          to_remove = []
+
+          scoped.each do |child|
+            if child.scoped_unregistered?
+              to_remove << child
+              next
+            end
 
-        children = Namespace.global.foobara_children
+            unless namespace.foobara_registered?(child, mode: Namespace::LookupMode::DIRECT)
+              namespace.foobara_register(child)
+            end
+          end
 
-        @original_children.each do |child|
-          children.foobara_children << child unless children.include?(child)
+          children = namespace.foobara_children
+
+          to_remove.each do |child|
+            children.delete(child)
+            scoped.delete(child)
+          end
+
+          to_remove.clear
+
+          @original_children[namespace].each do |child|
+            if child.scoped_unregistered?
+              to_remove << child
+              next
+            end
+            children << child unless children.include?(child)
+          end
+
+          to_remove.each do |child|
+            @original_children[namespace].delete(child)
+          end
         end
 
-        GlobalOrganization.foobara_register(GlobalDomain)
+        Namespace.clear_lru_cache!
 
         register_type_declaration(Handlers::RegisteredTypeDeclaration.new)
         register_type_declaration(Handlers::ExtendRegisteredTypeDeclaration.new)
@@ -117,11 +138,33 @@ module Foobara
         capture_current_namespaces
       end
 
+      private
+
       def capture_current_namespaces
         # TODO: this feels like the wrong place to do this but doing it here for now to make sure it's done when
         # most important
-        @original_scoped = Namespace.global.foobara_registry.all_scoped.dup
-        @original_children = Namespace.global.foobara_children.dup
+        @original_scoped = {}
+        @original_children = {}
+
+        namespaces_to_reset.each do |namespace|
+          next unless namespace.is_a?(Namespace::IsNamespace)
+
+          @original_scoped[namespace] = namespace.foobara_registry.all_scoped.dup
+          @original_children[namespace] = namespace.foobara_children.dup
+        end
+      end
+
+      def namespaces_to_reset
+        [
+          Foobara::GlobalDomain,
+          Foobara::GlobalOrganization,
+          Value::Processor,
+          Types::Type,
+          Domain,
+          Error,
+          Foobara,
+          Namespace.global
+        ]
       end
     end
   end

data/version.rb

@@ -1,6 +1,6 @@
 module Foobara
   module Version
-    VERSION = "0.7.0".freeze
+    VERSION = "0.7.1".freeze
     MINIMUM_RUBY_VERSION = [">= 3.4.0", "< 4.1"].freeze
   end
 end

metadata

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