foobara 0.0.72 → 0.0.74

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4f37230e03dfeded35416a8552cb0877017b3258cf994be9b223da77a833fc5a
-  data.tar.gz: 8eeb368cb063d05208513067d39e353e43074abe54d40767867b593cbd9157b7
+  metadata.gz: 354b23b90ef7ec29ac1126a3ee0ddda231253be5e33731041f80d77414c64e66
+  data.tar.gz: 2270ba278df777156963416623bb69418b596490cdbda7d40a2464322c17afd0
 SHA512:
-  metadata.gz: e6732b1d9abb5f2b9a8e738adf2db35cfaba3f8e0decd30de843ae09efb007a5ead774075390e8648030c3f52c17b2c124b1647614b56a6a22c2634eef8c5bdf
-  data.tar.gz: 176d48303bb79763ef67b0485cd90ff657fa406f1727969ae5c674d3f3b874ca7e466d4190c48910d1b968666665cfe23cfbf36302d38d88ac3ec48b0a575721
+  metadata.gz: bed611a57a66223eeba89f046a2aefdcaaca8ba7afadcd2a77a63817104c7bb39c449148109f6d0d2a3061fa00fb4665c7c0ca06384ecb0cd81ca7738101d268
+  data.tar.gz: 7463d91432204996f68590e23d79ffba7d0b067d946a37462a5da7a50f4e726383075f0d4d15d6a8639ae8387b24d5d867ba7b8cf74e66705c02287978d2ca75

data/CHANGELOG.md

@@ -1,3 +1,12 @@
+# [0.0.74] - 2025-03-15
+
+- fix bug where a command connector won't expose the non-global domain of a type if the command it depends on
+  is in the global domain
+
+# [0.0.73] - 2025-03-11
+
+- Fix bug preventing false from being used as a default in the Attributes::Dsl
+
 # [0.0.72] - 2025-03-07
 
 - Fix bug in find_many_by when there are required attributes not present in the filter

data/README.md

@@ -1052,7 +1052,7 @@ Let's install foobara-remote-imports:
 $ gem install foobara-remote-imports
 ```
 
-And lLet's create a new script and import our various Capybara commands over HTTP:
+And let's create a new script and import our various Capybara commands over HTTP:
 
 ```ruby
 #!/usr/bin/env ruby

data/projects/builtin_types/src/duck/supported_casters/allow_nil.rb

@@ -12,6 +12,7 @@ module Foobara
               false
             end
           end
+
           def applicable?(value)
             value.nil? && allow_nil?
           end

data/projects/command_connectors/src/command_connector.rb

@@ -354,8 +354,6 @@ module Foobara
 
       # Drive all of this off of the list of exposed commands...
       to_include = Set.new
-      to_include << command_registry.exposed_global_organization
-      to_include << command_registry.exposed_global_domain
 
       command_registry.foobara_each do |exposed_whatever|
         to_include << exposed_whatever

data/projects/command_connectors/src/command_registry/exposed_command.rb

@@ -94,7 +94,26 @@ module Foobara
         @full_command_symbol ||= Util.underscore_sym(full_command_name)
       end
 
+      def root_registry
+        parent = scoped_namespace
+        parent = parent.scoped_namespace while parent.scoped_namespace
+        parent
+      end
+
       def foobara_manifest(to_include: Set.new)
+        # A bit of a hack here. We don't have an exposed type class to encapsulate including exposed domains/orgs
+        # which leads to a bug when a global command is exposed that depends on a type in a non-global domain
+        # but there being no other reason to include that non-global domain.
+        transformed_command_class.types_depended_on.select(&:registered?).each do |type|
+          full_domain_name = type.foobara_domain.scoped_full_name
+
+          unless root_registry.foobara_lookup_domain(full_domain_name)
+            exposed_domain = root_registry.build_and_register_exposed_domain(full_domain_name)
+            to_include << exposed_domain
+            to_include << exposed_domain.foobara_organization
+          end
+        end
+
         transformed_command_class.foobara_manifest(to_include:).merge(super).merge(
           Util.remove_blank(
             scoped_category: :command,

data/projects/command_connectors/src/command_registry/exposed_domain.rb

@@ -15,6 +15,8 @@ module Foobara
 
       # TODO: unable to address types here so it is handled as a hack higher up...
       def foobara_manifest(to_include: Set.new)
+        to_include << foobara_organization
+
         domain_manifest = domain_module.foobara_manifest(to_include: Set.new)
         mode = Foobara::Namespace::LookupMode::DIRECT
         commands = foobara_all_command(mode:).map(&:foobara_manifest_reference).sort
@@ -22,6 +24,17 @@ module Foobara
         domain_manifest.merge(commands:)
       end
 
+      def foobara_organization
+        full_org_name = domain_module.foobara_organization.scoped_full_name
+        root_registry.foobara_lookup_organization(full_org_name)
+      end
+
+      def root_registry
+        parent = scoped_namespace
+        parent = parent.scoped_namespace while parent.scoped_namespace
+        parent
+      end
+
       def foobara_manifest_reference
         domain_module.foobara_manifest_reference
       end

data/projects/command_connectors/src/command_registry.rb

@@ -100,16 +100,6 @@ module Foobara
       exposed_organization
     end
 
-    def exposed_global_domain
-      exposed_global_organization.foobara_lookup_domain("") ||
-        build_and_register_exposed_domain("")
-    end
-
-    def exposed_global_organization
-      foobara_lookup_organization("") ||
-        build_and_register_exposed_organization("")
-    end
-
     def [](name)
       if name.is_a?(Class)
         name.full_command_name

data/projects/model/src/model.rb

@@ -182,7 +182,8 @@ module Foobara
 
       if options[:ignore_unexpected_attributes]
         Thread.foobara_with_var(:foobara_ignore_unexpected_attributes, true) do
-          return initialize(attributes, options.except(:ignore_unexpected_attributes))
+          initialize(attributes, options.except(:ignore_unexpected_attributes))
+          return
         end
       end
 

data/projects/type_declarations/src/dsl/attributes.rb

@@ -107,9 +107,9 @@ module Foobara
               _add_to_required(attribute_name)
             end
 
-            default = declaration.delete(:default)
+            if declaration.key?(:default)
+              default = declaration.delete(:default)
 
-            if default
               _add_to_defaults(attribute_name, default)
             end
 

data/projects/value/src/processor.rb

@@ -208,7 +208,7 @@ module Foobara
         always_applicable?
       end
 
-      # This means its applicable regardless of value to transform. Override if different behavior is needed.
+      # This means it's applicable regardless of value to transform. Override if different behavior is needed.
       def always_applicable?
         !!declaration_data
       end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: foobara
 version: !ruby/object:Gem::Version
-  version: 0.0.72
+  version: 0.0.74
 platform: ruby
 authors:
 - Miles Georgi
 bindir: bin
 cert_chain: []
-date: 2025-03-08 00:00:00.000000000 Z
+date: 2025-03-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bigdecimal