foobara 0.0.140 → 0.0.142

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e9ef91ca57cf88a6ce345e68697517ec5459bfbfda3be8958d2194cc1d46bc75
-  data.tar.gz: f77080a972b301496787bc4c1c0108e5721ab8364c1c3e818f2541498a4b131a
+  metadata.gz: a6defed2fda94d3c5f244f9f24eb9a32f4d06999bc773071dc58667565518499
+  data.tar.gz: 98a0f663a38ce6b105f78fbbefdc1e34328654ec54755daa379df824753f3214
 SHA512:
-  metadata.gz: 1b3c6ff758814d7706ccdaa3e3da6921929b8b0d2ac9d9404e2033d408af3654ffe96c66448b57c8394a2d144c95979092e1b75e6bd130d9363596fd1a9daae0
-  data.tar.gz: 7e4eb5c7411d3d02e1b65deca964a82bf0a495936c40613a1820afdada14a031be882d2f28aba5d2a3c43cad54381ed81888b77aa0c52c32c8582a44b9108a27
+  metadata.gz: 9c42abcbd62eba3583acfd4345df530ec04efdc18d19773ce1ed5dafa9e0d43d09364b2b45355a22874b2aeaf030d476726cbd214ec1a3626812b2a20c35133f
+  data.tar.gz: 7cc3b4e13250e4cc2b3af8468b2f5774861161fcc13972ecc87f17c1682bd8a69170a080b6f9cc72249ff5756d2a7f1ca509dfaeba2ac52bbf9896c8b9a4f3e6

data/CHANGELOG.md

@@ -1,3 +1,13 @@
+# [0.0.142] - 2025-08-10
+
+- Add LruCache to IsNamespace to speed up lookups
+
+# [0.0.141] - 2025-07-31
+
+- Fix bug returning a non-strict type declaration in the command connector manifest
+- Support using :attributes as a type to permit anything with symbolizable keys
+- Fix buggy global domain in command connector manifest
+
 # [0.0.140] - 2025-07-25
 
 - Support result :array, &block style for declaring result type

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

@@ -72,11 +72,11 @@ module Foobara
               possible_errors:,
               depends_on:,
               # TODO: allow inputs type to be nil or really any type?
-              inputs_type: inputs_type&.reference_or_declaration_data || {
+              inputs_type: inputs_type&.reference_or_declaration_data || GlobalDomain.foobara_type_from_declaration(
                 type: "::attributes",
                 element_type_declarations: {},
                 required: []
-              }
+              ).declaration_data
             ).merge(description:)
 
             if result_type

data/projects/command_connectors/src/command_connector.rb

@@ -628,7 +628,7 @@ module Foobara
                            fixed_scoped_path = error.scoped_full_path[index..]
                            fixed_scoped_name = fixed_scoped_path.join("::")
                            fixed_scoped_prefix = fixed_scoped_path[..-2]
-                           fixed_parent = [:domain, domain.full_domain_name]
+                           fixed_parent = [:domain, domain.reference]
 
                            error.relevant_manifest.merge(
                              parent: fixed_parent,

data/projects/common/src/error_collection.rb

@@ -27,7 +27,7 @@ module Foobara
 
     def error_array
       # :nocov:
-      warn "DEPRECATED: Do not call ErrorCollection#errors instead just use the collection directly."
+      warn "DEPRECATED: Do not call ErrorCollection#error_array instead just use the collection directly."
       self
       # :nocov:
     end

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

@@ -3,7 +3,7 @@ require_relative "type_declaration"
 module Foobara
   module Manifest
     class Attributes < TypeDeclaration
-      optional_keys :required, :defaults
+      optional_keys :required, :defaults, :element_type_declarations
 
       alias attributes_manifest relevant_manifest
 
@@ -19,6 +19,10 @@ module Foobara
         DataPath.value_at([:defaults, attribute_name], attributes_manifest)
       end
 
+      def has_attribute_declarations?
+        !element_type_declarations.nil?
+      end
+
       def attribute_declarations
         element_type_declarations.keys.to_h do |attribute_name|
           [

data/projects/namespace/src/is_namespace.rb

@@ -3,6 +3,16 @@ require_relative "scoped"
 module Foobara
   class Namespace
     module IsNamespace
+      class << self
+        def lru_cache
+          @lru_cache ||= Foobara::LruCache.new(100)
+        end
+
+        def clear_lru_cache!
+          @lru_cache = nil
+        end
+      end
+
       include Scoped
 
       def scoped_clear_caches
@@ -90,6 +100,8 @@ module Foobara
 
       def foobara_register(scoped)
         foobara_registry.register(scoped)
+
+        IsNamespace.clear_lru_cache!
         # awkward??
         scoped.scoped_namespace = self
 
@@ -108,6 +120,8 @@ module Foobara
         foobara_registry.unregister(scoped)
         foobara_children.delete(scoped)
         scoped.scoped_namespace = nil
+
+        IsNamespace.clear_lru_cache!
       end
 
       def foobara_unregister_all
@@ -116,17 +130,30 @@ module Foobara
         end
       end
 
+      def lru_cache
+        IsNamespace.lru_cache
+      end
+
       def foobara_lookup(
         path,
         filter: nil,
         mode: LookupMode::GENERAL,
-        visited: Set.new,
+        visited: nil,
         initial: true
       )
+        # TODO: make it so this is only necessary when initial
         path = Namespace.to_registry_path(path)
         visited_key = [path, mode, initial, self]
-        return nil if visited.include?(visited_key)
 
+        if initial
+          lru_key = [path, mode, self, filter]
+          found, value = lru_cache.get(lru_key)
+          return value if found
+        end
+
+        return nil if visited&.include?(visited_key)
+
+        visited ||= Set.new
         visited << visited_key
 
         LookupMode.validate!(mode)
@@ -139,7 +166,13 @@ module Foobara
             visited:,
             initial: false
           )
-          return scoped if scoped
+          if scoped
+            if initial
+              lru_cache.set_if_missing(lru_key, scoped)
+            end
+
+            return scoped
+          end
 
           candidates = foobara_children.map do |namespace|
             namespace.foobara_lookup(path, filter:, mode:, visited:, initial: false)
@@ -152,8 +185,14 @@ module Foobara
             # :nocov:
           end
 
-          return candidates.first ||
-                 foobara_parent_namespace&.foobara_lookup(path, filter:, mode:, visited:, initial: false)
+          scoped = candidates.first ||
+                   foobara_parent_namespace&.foobara_lookup(path, filter:, mode:, visited:, initial: false)
+
+          if initial
+            lru_cache.set_if_missing(lru_key, scoped)
+          end
+
+          return scoped
         end
 
         if path[0] == ""
@@ -171,11 +210,19 @@ module Foobara
         partial = foobara_registry.lookup(path, filter)
 
         if mode == LookupMode::DIRECT
+          if initial
+            lru_cache.set_if_missing(lru_key, partial)
+          end
+
           return partial
         end
 
         if partial
           if partial.scoped_path == path
+            if initial
+              lru_cache.set_if_missing(lru_key, partial)
+            end
+
             return partial
           end
         end
@@ -187,18 +234,39 @@ module Foobara
         end
 
         scoped = _lookup_in(path, to_consider, filter:, visited:)
-        return scoped if scoped
+
+        if scoped
+          if initial
+            lru_cache.set_if_missing(lru_key, scoped)
+          end
+
+          return scoped
+        end
 
         if [LookupMode::GENERAL, LookupMode::STRICT].include?(mode) && foobara_parent_namespace
           scoped = foobara_parent_namespace.foobara_lookup(
             path, filter:, mode: LookupMode::STRICT, visited:, initial: false
           )
-          return scoped if scoped
+
+          if scoped
+            if initial
+              lru_cache.set_if_missing(lru_key, scoped)
+            end
+
+            return scoped
+          end
         end
 
         if mode == LookupMode::GENERAL
           scoped = _lookup_in(path, foobara_depends_on_namespaces, filter:, visited:)
-          return scoped if scoped
+
+          if scoped
+            if initial
+              lru_cache.set_if_missing(lru_key, scoped)
+            end
+
+            return scoped
+          end
         end
 
         to_consider = case mode
@@ -219,7 +287,14 @@ module Foobara
         end
 
         scoped = candidates.first || partial
-        return scoped if scoped
+
+        if scoped
+          if initial
+            lru_cache.set_if_missing(lru_key, scoped)
+          end
+
+          return scoped
+        end
 
         # As a last resort we'll see if we're trying to fetch a builtin type from a different namespace
         # TODO: these lookup modes are really confusing and were designed prior to having multiple root
@@ -228,6 +303,7 @@ module Foobara
           scoped = Namespace.global.foobara_lookup(path, filter:, mode: LookupMode::ABSOLUTE, visited:, initial: false)
 
           if scoped.is_a?(Types::Type) && scoped.builtin?
+            lru_cache.set_if_missing(lru_key, scoped)
             scoped
           end
         end

data/projects/type_declarations/src/handlers/registered_type_declaration/short_type_name_desugarizer.rb

@@ -7,11 +7,10 @@ module Foobara
         class ShortTypeNameDesugarizer < TypeDeclarations::Desugarizer
           def applicable?(sugary_type_declaration)
             return false if TypeDeclarations.strict_stringified? || TypeDeclarations.strict?
+            return false unless sugary_type_declaration.is_a?(::Hash)
 
             sugary_type_declaration = sugary_type_declaration.dup
 
-            return false unless sugary_type_declaration.is_a?(::Hash)
-
             sugary_type_declaration = normalize_type(sugary_type_declaration)
 
             if sugary_type_declaration.key?(:type) &&

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

@@ -4,6 +4,13 @@ module Foobara
   module TypeDeclarations
     module SensitiveTypeRemovers
       class Attributes < SensitiveTypeRemover
+        def applicable?(strict_type_declaration)
+          return false unless super
+
+          element_type_declarations = strict_type_declaration[:element_type_declarations]
+          element_type_declarations && !element_type_declarations.empty?
+        end
+
         def transform(strict_type_declaration)
           to_change = {}
           to_remove = []

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: foobara
 version: !ruby/object:Gem::Version
-  version: 0.0.140
+  version: 0.0.142
 platform: ruby
 authors:
 - Miles Georgi
 bindir: bin
 cert_chain: []
-date: 2025-07-25 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bigdecimal
@@ -27,44 +27,44 @@ dependencies:
   name: foobara-lru-cache
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 0.0.2
+        version: 2.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 0.0.2
+        version: 2.0.0
 - !ruby/object:Gem::Dependency
   name: foobara-util
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 2.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 2.0.0
 - !ruby/object:Gem::Dependency
   name: inheritable-thread-vars
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 0.0.1
+        version: 2.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 0.0.1
+        version: 2.0.0
 description: A command-centric and discoverable software framework with a focus on
   domain concepts and abstracting away integration code
 email:
@@ -535,7 +535,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.2
+rubygems_version: 3.6.9
 specification_version: 4
 summary: A command-centric and discoverable software framework with a focus on domain
   concepts and abstracting away integration code