attr_json 2.1.0 → 2.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3e3a6009367cdd38597ef94a714897b21dd287f34251a501e54f0081981525e4
-  data.tar.gz: a4b4afb4af34a355eb8df4df1facbd17e03a34df8abb29a8335d4c0d589c4378
+  metadata.gz: eb4f8078039ec81cc25e7a95318ff5e42cb5bf85a955b40326f60f9022af16a2
+  data.tar.gz: 635f08ce5fa34ce45dfdd5256ae0e81fdcc2d455f4525bad5f71f5b0c4391c6b
 SHA512:
-  metadata.gz: f2613f3e9e0bfc676ef5ff965e2656780315cd645dc9ddace261ac3f340621ff32e0c1e7029309c379ed79b969aecabed361dd442848504c279753af0452af64
-  data.tar.gz: 5aa56646f724d7426353327f8c396abc0a84f5e462979aa7a92bb0b3563f4cefdfdbb0a0c0c058ba74c9119f2267ff511ca83e1209e13c8d077615fd5971ac38
+  metadata.gz: 22c35f674c7b5f3a4e91f348d20439df3e8fe175ca5d3c8938720a5e71739df2474cdddd5fdab5f0527f8751ed0fb6dee11ba7f75889c143daca92bd84afc86b
+  data.tar.gz: 636628f0083dd61c869de5b9eb65a0c11988560e26d5dcd8e822759788bdac4bdb3c6f0359e2257867ac91a61721fda32df54a36f51a001450ef883289be433a

data/.github/workflows/ci.yml

@@ -18,7 +18,10 @@ jobs:
   tests:
     services:
       db:
-        image: postgres:9.4
+        # used to be 9.4, should work on 9.4, only moved to 10.0
+        # because of rails 7.1 bug (rails intends to support 9 too!)
+        # https://github.com/jrochkind/attr_json/issues/211
+        image: postgres:10.0
         env:
           POSTGRES_USER: postgres
           POSTGRES_PASSWORD: postgres
@@ -50,6 +53,14 @@ jobs:
           - gemfile: rails_7_0
             ruby: 3.2
 
+          - gemfile: rails_7_1
+            ruby: '3.0'
+
+          - gemfile: rails_7_1
+            ruby: 3.1
+
+          - gemfile: rails_7_1
+            ruby: 3.2
 
     name: ${{ matrix.gemfile }}, ruby ${{ matrix.ruby }}
 

data/Appraisals

@@ -4,6 +4,11 @@ appraise "rails-6-0" do
   gem "rails", ">= 6.0.0", "< 6.1"
   gem "pg", "~> 1.0"
   gem "rspec-rails", "~> 5.0"
+
+
+  # Ruby 2.7 still needs webdrivers, since it can't use a new enough
+  # version of selenium-webdriver to download drivers itself
+  gem "webdrivers", ">= 5.3.1"
 end
 
 appraise "rails-6-1" do
@@ -25,6 +30,13 @@ appraise "rails-7-0" do
   gem "pg", "~> 1.0"
 end
 
+appraise "rails-7-1" do
+  gem 'combustion', "~> 1.0"
+
+  gem "rails", "~> 7.1.0"
+  gem "pg", "~> 1.0"
+end
+
 appraise "rails-edge" do
   # need combustion edge to work with rails edge, will no longer
   # be true on next combustion release, probably no later than Rails 7.1

data/CHANGELOG.md

@@ -4,7 +4,7 @@ Notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [Unreleased](https://github.com/jrochkind/attr_json/compare/v2.0.1...HEAD)
+## [Unreleased](https://github.com/jrochkind/attr_json/compare/v2.3.0...HEAD)
 
 ### Fixed
 
@@ -22,6 +22,30 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 *
 
+### Changed
+
+*
+
+*
+
+*
+
+## [2.3.0]((https://github.com/jrochkind/attr_json/compare/v2.2.0...v2.3.0))
+
+
+### Changed
+
+* Avoid private ActiveRecord API when lazily registering container attributes. (Compat with Rails post 7.1) https://github.com/jrochkind/attr_json/pull/214
+
+* Freeze more strings to reduce String allocations https://github.com/jrochkind/attr_json/pull/216
+
+## [2.2.0](https://github.com/jrochkind/attr_json/compare/v2.1.0...v2.2.0)
+
+### Added
+
+* Test and allow Rails 7.1 https://github.com/jrochkind/attr_json/pull/208 Thanks @JoeSouthan
+
+
 ## [2.1.0](https://github.com/jrochkind/attr_json/compare/v2.0.1...v2.1.0)
 
 ### Added

data/Gemfile

@@ -37,7 +37,6 @@ gem "sprockets-rails"
 
 
 gem 'capybara', "~> 3.0"
-gem 'webdrivers', '~> 5.0'
 gem "selenium-webdriver"
 
 gem "byebug"

data/README.md

@@ -48,7 +48,7 @@ class MyModel < ActiveRecord::Base
 
    # You can have an _array_ of those things too. It will ordinarily default to empty array.
    attr_json :int_array, :integer, array: true
-   
+
    # The empty array default can be disabled with the following setting
    attr_json :str_array, :string, array: true, default: AttrJson::AttributeDefinition::NO_DEFAULT_PROVIDED
 
@@ -189,10 +189,8 @@ attribute name, it'll actually query on store_key. And properly handles any
 `container_attribute` -- it'll look in the proper jsonb column.
 
 Anything you can do with `jsonb_contains` should be handled
-by a [postgres `USING GIN` index](https://www.postgresql.org/docs/9.5/static/datatype-json.html#JSON-INDEXING)
-(I think! can anyone help confirm/deny?). To be sure, I recommend you
-investigate: Check out `to_sql` on any query to see what jsonb SQL it generates,
-and explore if you have the indexes you need.
+by a [postgres `USING GIN` index](https://www.postgresql.org/docs/9.5/static/datatype-json.html#JSON-INDEXING). Figuring out how to use indexes for jsonb
+queries can be confusing, [here is a good blog post](https://blog.kiprosh.com/postgres-gin-index-in-rails/).
 
 <a name="nested"></a>
 ## Nested models -- Structured/compound data

data/attr_json.gemspec

@@ -47,7 +47,7 @@ attributes use as much of the existing ActiveRecord architecture as we can.}
   # dependency included just as it is here, should never be released
   # from an env tht has any of these variables set.
   unless ENV['APPRAISAL_INITIALIZED'] || ENV["TRAVIS"] || ENV['CI']
-    spec.add_runtime_dependency "activerecord", ">= 6.0.0", "< 7.1"
+    spec.add_runtime_dependency "activerecord", ">= 6.0.0", "< 7.2"
   end
 
   spec.add_development_dependency "bundler"

data/gemfiles/rails_6_0.gemfile

@@ -12,8 +12,8 @@ gem "jquery-rails"
 gem "coffee-rails"
 gem "sprockets-rails"
 gem "capybara", "~> 3.0"
-gem "webdrivers", "~> 5.0"
 gem "selenium-webdriver"
 gem "byebug"
+gem "webdrivers", ">= 5.3.1"
 
 gemspec path: "../"

data/gemfiles/rails_6_1.gemfile

@@ -12,7 +12,6 @@ gem "jquery-rails"
 gem "coffee-rails"
 gem "sprockets-rails"
 gem "capybara", "~> 3.0"
-gem "webdrivers", "~> 5.0"
 gem "selenium-webdriver"
 gem "byebug"
 

data/gemfiles/rails_7_0.gemfile

@@ -12,7 +12,6 @@ gem "jquery-rails"
 gem "coffee-rails"
 gem "sprockets-rails"
 gem "capybara", "~> 3.0"
-gem "webdrivers", "~> 5.0"
 gem "selenium-webdriver"
 gem "byebug"
 

data/gemfiles/rails_7_1.gemfile

@@ -0,0 +1,18 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "combustion", "~> 1.0"
+gem "rails", "~> 7.1.0"
+gem "pg", "~> 1.0"
+gem "rspec-rails", "~> 6.0"
+gem "simple_form", ">= 4.0"
+gem "cocoon", ">= 1.2"
+gem "jquery-rails"
+gem "coffee-rails"
+gem "sprockets-rails"
+gem "capybara", "~> 3.0"
+gem "selenium-webdriver"
+gem "byebug"
+
+gemspec path: "../"

data/gemfiles/rails_edge.gemfile

@@ -12,7 +12,6 @@ gem "jquery-rails"
 gem "coffee-rails"
 gem "sprockets-rails"
 gem "capybara", "~> 3.0"
-gem "webdrivers", "~> 5.0"
 gem "selenium-webdriver"
 gem "byebug"
 gem "rack", "~> 2.0"

data/lib/attr_json/attribute_definition/registry.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'attr_json/attribute_definition'
 
 module AttrJson
@@ -25,7 +27,10 @@ module AttrJson
       def initialize(hash = {})
         @name_to_definition = hash.dup
         @store_key_to_definition = {}
-        definitions.each { |d| store_key_index!(d) }
+
+        @name_to_definition.values.each { |d| store_key_index!(d) }
+
+        @container_attributes_registered = Hash.new { Set.new }
       end
 
       def fetch(key, *args, &block)
@@ -46,12 +51,13 @@ module AttrJson
 
       # Can return nil if none found.
       def store_key_lookup(container_attribute, store_key)
-        @store_key_to_definition[container_attribute.to_s] &&
-          @store_key_to_definition[container_attribute.to_s][store_key.to_s]
+        @store_key_to_definition[AttrJson.efficient_to_s(container_attribute)] &&
+          @store_key_to_definition[AttrJson.efficient_to_s(container_attribute)][AttrJson.efficient_to_s(store_key)]
       end
 
       def definitions
-        @name_to_definition.values
+        # Since we are immutable, we can cache this to avoid allocation in a hot spot
+        @stored_definitions ||= @name_to_definition.values
       end
 
       # Returns all registered attributes as an array of symbols
@@ -60,7 +66,7 @@ module AttrJson
       end
 
       def container_attributes
-        @store_key_to_definition.keys.collect(&:to_s)
+        @store_key_to_definition.keys.collect { |s| AttrJson.efficient_to_s(s) }
       end
 
       # This is how you register additional definitions, as a non-mutating
@@ -73,6 +79,32 @@ module AttrJson
         end
       end
 
+
+      # We need to lazily set the container type only the FIRST time
+      #
+      # While also avoiding this triggering ActiveRecord to actually go to DB,
+      # we don't want DB connection forced on boot, that's a problem for many apps,
+      # including that may not have a DB connection in initial development.
+      # (#type_for_attribute forces DB connection)
+      #
+      # AND we need to call container attriubte on SUB-CLASS AGAIN, iff sub-class
+      # has any of it's own new registrations, to make sure we get the right type in
+      # sub-class!
+      #
+      # So we just keep track of whether we've registered ourselves, so we can
+      # first time we need to.
+      #
+      # While current implementation is simple, this has ended up a bit fragile,
+      # a different API that doesn't require us to do this implicitly lazily
+      # might be preferred! But this is what we got for now.
+      def register_container_attribute(attribute_name:, model:)
+        @container_attributes_registered[attribute_name.to_sym] << model
+      end
+
+      def container_attribute_registered?(attribute_name:, model:)
+         @container_attributes_registered[attribute_name.to_sym].include?(model)
+      end
+
       protected
 
       def add!(definition)
@@ -81,17 +113,19 @@ module AttrJson
         end
         @name_to_definition[definition.name.to_sym] = definition
         store_key_index!(definition)
+
+        @stored_definitions = nil
       end
 
       def store_key_index!(definition)
-        container_hash = (@store_key_to_definition[definition.container_attribute.to_s] ||= {})
+        container_hash = (@store_key_to_definition[AttrJson.efficient_to_s(definition.container_attribute)] ||= {})
 
-        if container_hash.has_key?(definition.store_key.to_s)
-          existing = container_hash[definition.store_key.to_s]
+        if container_hash.has_key?(AttrJson.efficient_to_s(definition.store_key))
+          existing = container_hash[AttrJson.efficient_to_s(definition.store_key)]
           raise ArgumentError, "Can't add, store key `#{definition.store_key}` conflicts with existing attribute: #{existing.original_args}"
         end
 
-        container_hash[definition.store_key.to_s] = definition
+        container_hash[AttrJson.efficient_to_s(definition.store_key)] = definition
       end
     end
   end

data/lib/attr_json/attribute_definition.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
  require 'attr_json/type/array'
 
  module AttrJson
@@ -62,7 +64,7 @@
     end
 
     def store_key
-      (@store_key || name).to_s
+      AttrJson.efficient_to_s(@store_key || name)
     end
 
     def has_default?

data/lib/attr_json/model.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_support/concern'
 require 'active_model/type'
 
@@ -136,8 +138,8 @@ module AttrJson
       def new_from_serializable(attributes = {})
         attributes = attributes.collect do |key, value|
           # store keys in arguments get translated to attribute names on initialize.
-          if attribute_def = self.attr_json_registry.store_key_lookup("", key.to_s)
-            key = attribute_def.name.to_s
+          if attribute_def = self.attr_json_registry.store_key_lookup("".freeze, AttrJson.efficient_to_s(key))
+            key = AttrJson.efficient_to_s(attribute_def.name)
           end
 
           attr_type = self.attr_json_registry.has_attribute?(key) && self.attr_json_registry.type_for_attribute(key)
@@ -184,7 +186,7 @@ module AttrJson
 
       # like the ActiveModel::Attributes method, hash with name keys, and ActiveModel::Type values
       def attribute_types
-        attribute_names.collect { |name| [name.to_s, attr_json_registry.type_for_attribute(name)]}.to_h
+        attribute_names.collect { |name| [AttrJson.efficient_to_s(name), attr_json_registry.type_for_attribute(name)]}.to_h
       end
 
 
@@ -225,11 +227,11 @@ module AttrJson
 
         _attr_jsons_module.module_eval do
           define_method("#{name}=") do |value|
-            _attr_json_write(name.to_s, value)
+            _attr_json_write(AttrJson.efficient_to_s(name), value)
           end
 
           define_method("#{name}") do
-            attributes[name.to_s]
+            attributes[AttrJson.efficient_to_s(name)]
           end
         end
       end
@@ -427,7 +429,7 @@ module AttrJson
 
     def fill_in_defaults!
       self.class.attr_json_registry.definitions.each do |definition|
-        if definition.has_default? && !attributes.has_key?(definition.name.to_s)
+        if definition.has_default? && !attributes.has_key?(AttrJson.efficient_to_s(definition.name))
           self.send("#{definition.name.to_s}=", definition.provide_default!)
         end
       end
@@ -435,10 +437,10 @@ module AttrJson
 
     def _attr_json_write(key, value)
       if attribute_def = self.class.attr_json_registry[key.to_sym]
-        attributes[key.to_s] = attribute_def.cast(value)
+        attributes[AttrJson.efficient_to_s(key)] = attribute_def.cast(value)
       else
         # TODO, strict mode, ignore, raise, allow.
-        attributes[key.to_s] = value
+        attributes[AttrJson.efficient_to_s(key)] = value
       end
     end
 

data/lib/attr_json/nested_attributes/builder.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module AttrJson
   module NestedAttributes
     # Implementation of `build_` methods, called by the `build_` methods

data/lib/attr_json/nested_attributes/multiparameter_attribute_writer.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module AttrJson
   module NestedAttributes
     # Rails has a weird "multiparameter attribute" thing, that is used for simple_form's

data/lib/attr_json/nested_attributes/writer.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'attr_json/nested_attributes/multiparameter_attribute_writer'
 
 module AttrJson

data/lib/attr_json/nested_attributes.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'attr_json/nested_attributes/builder'
 require 'attr_json/nested_attributes/writer'
 

data/lib/attr_json/record/query_builder.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module AttrJson
   module Record
     # Implementation class called by #jsonb_contains scope method. Ordinarily

data/lib/attr_json/record/query_scopes.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'attr_json/record/query_builder'
 
 module AttrJson

data/lib/attr_json/record.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'attr_json/attribute_definition'
 require 'attr_json/attribute_definition/registry'
 require 'attr_json/type/container_attribute'
@@ -160,22 +162,14 @@ module AttrJson
         options.assert_valid_keys(AttributeDefinition::VALID_OPTIONS + [:validate, :accepts_nested_attributes])
         container_attribute = options[:container_attribute]
 
-        # TODO arg check container_attribute make sure it exists. Hard cause
-        # schema isn't loaded yet when class def is loaded. Maybe not.
 
-        # Want to lazily add an attribute cover to the json container attribute,
-        # only if it hasn't already been done. WARNING we are using internal
-        # Rails API here, but only way to do this lazily, which I thought was
-        # worth it. On the other hand, I think .attribute is idempotent, maybe we don't need it...
-        #
-        # We set default to empty hash, because that 'tricks' AR into knowing any
-        # application of defaults is a change that needs to be saved.
-        unless attributes_to_define_after_schema_loads[container_attribute.to_s] &&
-               attributes_to_define_after_schema_loads[container_attribute.to_s].first.is_a?(AttrJson::Type::ContainerAttribute) &&
-               attributes_to_define_after_schema_loads[container_attribute.to_s].first.model == self
-           # If this is already defined, but was for superclass, we need to define it again for
-           # this class.
+        # Make sure to "lazily" register attribute for *container* class if this is the first time
+        # this container attribute hsa been encountered for this specific class. The registry
+        # helps us keep track. Kinda messy, in future we may want a more explicit API
+        # that does not require us to implicitly track first-time per-container.
+        unless self.attr_json_registry.container_attribute_registered?(model: self, attribute_name: container_attribute.to_sym)
            attribute container_attribute.to_sym, AttrJson::Type::ContainerAttribute.new(self, container_attribute), default: -> { {} }
+           self.attr_json_registry.register_container_attribute(model: self, attribute_name: container_attribute.to_sym)
         end
 
         self.attr_json_registry = attr_json_registry.with(

data/lib/attr_json/serialization_coder_from_type.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module AttrJson
 
   # A little wrapper to provide an object that provides #dump and #load method for use

data/lib/attr_json/type/array.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module AttrJson
   module Type
     # You can wrap any ActiveModel::Type in one of these, and it's magically

data/lib/attr_json/type/container_attribute.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module AttrJson
   module Type
     # A type that gets applied to the AR container/store jsonb attribute,

data/lib/attr_json/type/model.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module AttrJson
   module Type
     # An ActiveModel::Type representing a particular AttrJson::Model

data/lib/attr_json/type/polymorphic_model.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module AttrJson
   module Type
     # AttrJson::Type::PolymorphicModel can be used to create attr_json attributes

data/lib/attr_json/version.rb

@@ -1,3 +1,3 @@
 module AttrJson
-  VERSION = "2.1.0"
+  VERSION = "2.3.0"
 end

data/lib/attr_json.rb

@@ -10,5 +10,25 @@ require 'attr_json/record/query_scopes'
 require 'attr_json/type/polymorphic_model'
 
 module AttrJson
-
+  # We need to convert Symbols to strings a lot at present -- ActiveRecord does too, so
+  # not too suprrising.
+  #
+  # In Rails 3.0 and above, we can use Symbol#name to get a frozen string back
+  # and avoid extra allocations. https://bugs.ruby-lang.org/issues/16150
+  #
+  # Ruby 2.7 doens't yet have it though. As long as we are supporting ruby 2.7,
+  # we'll just check at runtime to keep this lean
+  if RUBY_VERSION.split('.').first.to_i >= 3
+    def self.efficient_to_s(obj)
+      if obj.kind_of?(Symbol)
+        obj.name
+      else
+        obj.to_s
+      end
+    end
+  else
+    def self.efficient_to_s(obj)
+      obj.to_s
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: attr_json
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 2.3.0
 platform: ruby
 authors:
 - Jonathan Rochkind
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-07-13 00:00:00.000000000 Z
+date: 2023-12-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -19,7 +19,7 @@ dependencies:
         version: 6.0.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -29,7 +29,7 @@ dependencies:
         version: 6.0.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -164,6 +164,7 @@ files:
 - gemfiles/rails_6_0.gemfile
 - gemfiles/rails_6_1.gemfile
 - gemfiles/rails_7_0.gemfile
+- gemfiles/rails_7_1.gemfile
 - gemfiles/rails_edge.gemfile
 - lib/attr_json.rb
 - lib/attr_json/attribute_definition.rb
@@ -206,7 +207,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.26
+rubygems_version: 3.4.21
 signing_key:
 specification_version: 4
 summary: ActiveRecord attributes stored serialized in a json column, super smooth.