foobara 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 382dbd4fb1743f89af17bc70f7acdbef0ad2851767122f702e2503dcfacddf12
-  data.tar.gz: 02dfabf373e9ec1545ecbcaeeed6c31b0f6338bcc5e9095b3e0efe9bc9f61c03
+  metadata.gz: 5ff971dedd276b649d7ceb9b184734a47074357ab20b517b01a377bf7dfedfaa
+  data.tar.gz: a1b07c225fdddebd6dd859650db155a7e20aca66fe0647211d5daede966b305f
 SHA512:
-  metadata.gz: 535af52cbd9ad41609694db0b23e69f003d5959ccff113bc98dcedc13ff15df685b6abfcafe510f26243888d979b4724b061d00a76164805167d8f90cc3f4d39
-  data.tar.gz: 692827d4929a901e5127c706c5f63d5da40330dfc5a5e868ea0661ef5885b25dbb5a4db43f8b76e980076ded3d876637ea1328bd164a16fed9e86664d99b9d4c
+  metadata.gz: 1d797ddc6c22d97617ffe9c1c951d66257f305dceedccaa5df80f0da3283a902a3c7acfd086dd4603957e3efa2a9f2b06a0851bb6cc420884839f79f19c70816
+  data.tar.gz: f57df9169e0da18ffc23d57e9fd8b2c5d218714938e4b37f1d4de1b19344774756915702d152b19ace97302b481df9bf6b9e4467db590b34f606cae2861cf44f

data/CHANGELOG.md

@@ -1,4 +1,12 @@
-# [0.1.3] - 2025-08-25
+# [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/command_connectors/src/transformed_command.rb

@@ -369,27 +369,45 @@ 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
+
+        to_merge.each_pair do |key, value|
+          # TODO: we could probably remove empty strings and nils, too, and from the whole hash
+          # if (value.is_a?(::Hash) || value.is_a?(::Array)) && value.empty?
+          #   manifest.delete(key)
+          # else
+          #   manifest[key] = value
+          # end
+          # TODO: for now, just including everything for stability
+          # But we should remove all empty values in a future version
+          manifest[key] = value
+        end
+
+        manifest
       end
 
       def processors_to_manifest_symbols(processors)

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]

metadata

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