iknow_view_models 3.4.3 → 3.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3560a98616dd2f0548ac7e663827e68d86fa9991969ac6ee49633a0090e7d628
-  data.tar.gz: deec44e5132f91dfc1deef0dd889ee20c1b01c6d5d7791d356f1e9dc99ec1c8a
+  metadata.gz: f51edf25fab0aec533446c939c44975f76806871e1341518aa4470f32c23d839
+  data.tar.gz: 140166cc68f15484a0291f4e8faa8b4e26c3233a2b3bcfacfa9a4972a2a5e0ef
 SHA512:
-  metadata.gz: 5af2b38b2e8fb8805480bd665ea24e7ad4b489951c8b72660e502a053a945798f18269b60991eb416ab6959fc46e4831a911a15005810b57e288e2eeaaf37fd5
-  data.tar.gz: 5a73108237e3ffae5751f50a6e693e8fd737a4d1936f087095138ec2db47cbd7ec852f81e6ef881f794a38d67ba85429825b2393a850c149f363121f321adf30
+  metadata.gz: 27e85b3cd097d4f19e28d66d3500db4142dedd4a65a44731754afd35430d86945bb2e7f928df476e4aadb665d34518fa10211113f17b1ad0718727f957c8156f
+  data.tar.gz: 7e740ea172d15e55b870336fd833a6138c3f5aaec97da84828813b6da57d9512bbb3cd98f404f24c58c7adf7b53d875639bd22561b200d429e74523770a3a483

data/iknow_view_models.gemspec

@@ -8,7 +8,7 @@ Gem::Specification.new do |spec|
   spec.name          = 'iknow_view_models'
   spec.version       = IknowViewModels::VERSION
   spec.authors       = ['iKnow Team']
-  spec.email         = ['edge@iknow.jp']
+  spec.email         = ['systems@iknow.jp']
   spec.summary       = 'ViewModels provide a means of encapsulating a collection of related data and specifying its JSON serialization.'
   spec.description   = ''
   spec.homepage      = 'https://github.com/iknow/cerego_view_models'

data/lib/iknow_view_models/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module IknowViewModels
-  VERSION = '3.4.3'
+  VERSION = '3.5.2'
 end

data/lib/view_model/active_record/association_manipulation.rb

@@ -3,6 +3,8 @@
 # Mix-in for VM::ActiveRecord providing direct manipulation of
 # directly-associated entities. Avoids loading entire collections.
 module ViewModel::ActiveRecord::AssociationManipulation
+  extend ActiveSupport::Concern
+
   def load_associated(association_name, scope: nil, eager_include: true, serialize_context: self.class.new_serialize_context)
     association_data = self.class._association_data(association_name)
     direct_reflection = association_data.direct_reflection
@@ -49,49 +51,63 @@ module ViewModel::ActiveRecord::AssociationManipulation
     end
   end
 
-  # Replace the current member(s) of an association with the provided hash(es).
+  # Replace the current member(s) of an association with the provided
+  # hash(es).  Only mentioned member(s) will be returned.
+  #
+  # This interface deals with associations directly where reasonable,
+  # with the notable exception of referenced+shared associations. That
+  # is to say, that owned associations should be presented in the form
+  # of direct update hashes, regardless of their
+  # referencing. Reference and shared associations are excluded to
+  # ensure that the update hash for a shared entity is unique, and
+  # that edits may only be specified once.
   def replace_associated(association_name, update_hash, references: {}, deserialize_context: self.class.new_deserialize_context)
-    association_data = self.class._association_data(association_name)
-
-    if association_data.referenced?
-      is_fupdate =
-        association_data.collection? &&
-        update_hash.is_a?(Hash) &&
-        update_hash[ViewModel::ActiveRecord::TYPE_ATTRIBUTE] == ViewModel::ActiveRecord::FUNCTIONAL_UPDATE_TYPE
-
-      if is_fupdate
-        update_hash[ViewModel::ActiveRecord::ACTIONS_ATTRIBUTE].each_with_index do |action, i|
-          action_type_name = action[ViewModel::ActiveRecord::TYPE_ATTRIBUTE]
-          if action_type_name == ViewModel::ActiveRecord::FunctionalUpdate::Remove::NAME
-            # Remove actions are always type/id refs; others need to be translated to proper refs
-            next
-          end
+    _updated_parent, changed_children =
+      self.class.replace_associated_bulk(
+        association_name,
+        { self.id => update_hash },
+        references: references,
+        deserialize_context: deserialize_context
+      ).first
+
+    changed_children
+  end
 
-          association_references = convert_updates_to_references(
-            action[ViewModel::ActiveRecord::VALUES_ATTRIBUTE],
-            key: "#{action_type_name}_#{i}")
-          references.merge!(association_references)
-          action[ViewModel::ActiveRecord::VALUES_ATTRIBUTE] =
-            association_references.each_key.map { |ref| { ViewModel::REFERENCE_ATTRIBUTE => ref } }
-        end
-      else
-        update_hash = ViewModel::Utils.wrap_one_or_many(update_hash) do |sh|
-          association_references = convert_updates_to_references(sh, key: 'replace')
-          references.merge!(association_references)
-          association_references.each_key.map { |ref| { ViewModel::REFERENCE_ATTRIBUTE => ref } }
-        end
+  class_methods do
+    # Replace the current member(s) of an association with the provided
+    # hash(es) for many viewmodels.  Only mentioned members will be returned.
+    #
+    # This is an interim implementation that requires loading the contents of
+    # all collections into memory and filtering for the mentioned entities,
+    # even for functional updates.  This is in contrast to append_associated,
+    # which only operates on the new entities.
+    def replace_associated_bulk(association_name, updates_by_parent_id, references:, deserialize_context: self.class.new_deserialize_context)
+      association_data = _association_data(association_name)
+
+      touched_ids = updates_by_parent_id.each_with_object({}) do |(parent_id, update_hash), acc|
+        acc[parent_id] =
+          mentioned_children(
+            update_hash,
+            references:       references,
+            association_data: association_data,
+          ).to_set
       end
-    end
 
-    root_update_hash = {
-      ViewModel::ID_ATTRIBUTE   => self.id,
-      ViewModel::TYPE_ATTRIBUTE => self.class.view_name,
-      association_name.to_s     => update_hash,
-    }
+      root_update_hashes = updates_by_parent_id.map do |parent_id, update_hash|
+        {
+          ViewModel::ID_ATTRIBUTE   => parent_id,
+          ViewModel::TYPE_ATTRIBUTE => view_name,
+          association_name.to_s     => update_hash,
+        }
+      end
 
-    root_update_viewmodel = self.class.deserialize_from_view(root_update_hash, references: references, deserialize_context: deserialize_context)
+      root_update_viewmodels = deserialize_from_view(
+        root_update_hashes, references: references, deserialize_context: deserialize_context)
 
-    root_update_viewmodel._read_association(association_name)
+      root_update_viewmodels.each_with_object({}) do |updated, acc|
+        acc[updated] = updated._read_association_touched(association_name, touched_ids: touched_ids.fetch(updated.id))
+      end
+    end
   end
 
   # Create or update members of a associated collection. For an ordered
@@ -313,7 +329,10 @@ module ViewModel::ActiveRecord::AssociationManipulation
     indirect_update_data, referenced_update_data = ViewModel::ActiveRecord::UpdateData.parse_hashes(subtree_hashes, references)
 
     # Convert associated update data to references
-    indirect_references = convert_updates_to_references(indirect_update_data, key: 'indirect_append')
+    indirect_references =
+      self.class.convert_updates_to_references(
+        indirect_update_data, key: 'indirect_append')
+
     referenced_update_data.merge!(indirect_references)
 
     # Find any existing models for the direct association: need to re-use any
@@ -352,12 +371,6 @@ module ViewModel::ActiveRecord::AssociationManipulation
     return direct_update_data, referenced_update_data
   end
 
-  def convert_updates_to_references(indirect_update_data, key:)
-    indirect_update_data.each.with_index.with_object({}) do |(update, i), indirect_references|
-      indirect_references["__#{key}_ref_#{i}"] = update
-    end
-  end
-
   # TODO: this functionality could reasonably be extracted into `acts_as_manual_list`.
   def select_append_positions(association_data, position_attr, append_count, before:, after:)
     direct_reflection = association_data.direct_reflection
@@ -395,10 +408,134 @@ module ViewModel::ActiveRecord::AssociationManipulation
   def check_association_type!(association_data, type)
     if type && !association_data.accepts?(type)
       raise ViewModel::SerializationError.new(
-              "Type error: association '#{direct_reflection.name}' can't refer to viewmodel #{type.view_name}")
+              "Type error: association '#{association_data.association_name}' can't refer to viewmodel #{type.view_name}")
     elsif association_data.polymorphic? && !type
       raise ViewModel::SerializationError.new(
-              "Need to specify target viewmodel type for polymorphic association '#{direct_reflection.name}'")
+              "Need to specify target viewmodel type for polymorphic association '#{association_data.association_name}'")
+    end
+  end
+
+  class_methods do
+    def convert_updates_to_references(indirect_update_data, key:)
+      indirect_update_data.each.with_index.with_object({}) do |(update, i), indirect_references|
+        indirect_references["__#{key}_ref_#{i}"] = update
+      end
+    end
+
+    def add_reference_indirection(update_hash, association_data:, references:, key:)
+      raise ArgumentError.new('Not a referenced association') unless association_data.referenced?
+
+      is_fupdate =
+        association_data.collection? &&
+          update_hash.is_a?(Hash) &&
+          update_hash[ViewModel::ActiveRecord::TYPE_ATTRIBUTE] == ViewModel::ActiveRecord::FUNCTIONAL_UPDATE_TYPE
+
+      if is_fupdate
+        update_hash[ViewModel::ActiveRecord::ACTIONS_ATTRIBUTE].each_with_index do |action, i|
+          action_type_name = action[ViewModel::ActiveRecord::TYPE_ATTRIBUTE]
+          if action_type_name == ViewModel::ActiveRecord::FunctionalUpdate::Remove::NAME
+            # Remove actions are always type/id refs; others need to be translated to proper refs
+            next
+          end
+
+          association_references = convert_updates_to_references(
+            action[ViewModel::ActiveRecord::VALUES_ATTRIBUTE],
+            key: "#{key}_#{action_type_name}_#{i}")
+          references.merge!(association_references)
+          action[ViewModel::ActiveRecord::VALUES_ATTRIBUTE] =
+            association_references.each_key.map { |ref| { ViewModel::REFERENCE_ATTRIBUTE => ref } }
+        end
+
+        update_hash
+      else
+        ViewModel::Utils.wrap_one_or_many(update_hash) do |sh|
+          association_references = convert_updates_to_references(sh, key: "#{key}_replace")
+          references.merge!(association_references)
+          association_references.each_key.map { |ref| { ViewModel::REFERENCE_ATTRIBUTE => ref } }
+        end
+      end
+    end
+
+    # Traverses literals and fupdates to return referenced children.
+    #
+    # Runs before the main parser, so must be defensive
+    def each_child_hash(assoc_update, association_data:)
+      return enum_for(__method__, assoc_update, association_data: association_data) unless block_given?
+
+      is_fupdate =
+        association_data.collection? &&
+          assoc_update.is_a?(Hash) &&
+          assoc_update[ViewModel::ActiveRecord::TYPE_ATTRIBUTE] == ViewModel::ActiveRecord::FUNCTIONAL_UPDATE_TYPE
+
+      if is_fupdate
+        assoc_update.fetch(ViewModel::ActiveRecord::ACTIONS_ATTRIBUTE).each do |action|
+          action_type_name = action[ViewModel::ActiveRecord::TYPE_ATTRIBUTE]
+          if action_type_name.nil?
+            raise ViewModel::DeserializationError::InvalidSyntax.new(
+              "Functional update missing '#{ViewModel::ActiveRecord::TYPE_ATTRIBUTE}'"
+            )
+          end
+
+          if action_type_name == ViewModel::ActiveRecord::FunctionalUpdate::Remove::NAME
+            # Remove actions are not considered children of the action.
+            next
+          end
+
+          values = action.fetch(ViewModel::ActiveRecord::VALUES_ATTRIBUTE) {
+            raise ViewModel::DeserializationError::InvalidSyntax.new(
+              "Functional update missing '#{ViewModel::ActiveRecord::VALUES_ATTRIBUTE}'"
+            )
+          }
+          values.each { |x| yield x }
+        end
+      else
+        ViewModel::Utils.wrap_one_or_many(assoc_update) do |assoc_updates|
+          assoc_updates.each { |u| yield u }
+        end
+      end
+    end
+
+    # Collects the ids of children that are mentioned in the update data.
+    #
+    # Runs before the main parser, so must be defensive.
+    def mentioned_children(assoc_update, references:, association_data:)
+      return enum_for(__method__, assoc_update, references: references, association_data: association_data) unless block_given?
+
+      each_child_hash(assoc_update, association_data: association_data).each do |child_hash|
+        unless child_hash.is_a?(Hash)
+          raise ViewModel::DeserializationError::InvalidSyntax.new(
+            "Expected update hash, received: #{child_hash}"
+          )
+        end
+
+        if association_data.referenced?
+          ref_handle = child_hash.fetch(ViewModel::REFERENCE_ATTRIBUTE) {
+            raise ViewModel::DeserializationError::InvalidSyntax.new(
+              "Reference hash missing '#{ViewModel::REFERENCE_ATTRIBUTE}'"
+            )
+          }
+
+          ref_update_hash = references.fetch(ref_handle) {
+            raise ViewModel::DeserializationError::InvalidSyntax.new(
+              "Reference '#{ref_handle}' does not exist in references"
+            )
+          }
+
+          unless ref_update_hash.is_a?(Hash)
+            raise ViewModel::DeserializationError::InvalidSyntax.new(
+              "Expected update hash, received: #{child_hash}"
+            )
+          end
+
+          if (id = ref_update_hash[ViewModel::ID_ATTRIBUTE])
+            yield id
+          end
+        else
+          if (id = child_hash[ViewModel::ID_ATTRIBUTE])
+            yield id
+          end
+        end
+      end
     end
   end
 end

data/lib/view_model/active_record/cloner.rb

@@ -52,22 +52,23 @@ class ViewModel::ActiveRecord::Cloner
           new_associated = associated
         else
           # Otherwise descend into the child, and attach the result
-          vm_class =
-            case
-            when association_data.through?
-              # descend into the synthetic join table viewmodel
-              association_data.direct_viewmodel
-            when association_data.collection?
-              association_data.viewmodel_class
-            else
-              association_data.viewmodel_class_for_model!(associated.class)
-            end
+          build_vm = ->(model) do
+            vm_class =
+              if association_data.through?
+                # descend into the synthetic join table viewmodel
+                association_data.direct_viewmodel
+              else
+                association_data.viewmodel_class_for_model!(model.class)
+              end
+
+            vm_class.new(model)
+          end
 
           new_associated =
             if ViewModel::Utils.array_like?(associated)
-              associated.map { |m| clone(vm_class.new(m)) }.compact
+              associated.map { |m| clone(build_vm.(m)) }.compact
             else
-              clone(vm_class.new(associated))
+              clone(build_vm.(associated))
             end
         end
       end

data/lib/view_model/active_record/collection_nested_controller.rb

@@ -3,8 +3,7 @@
 require 'view_model/active_record/nested_controller_base'
 
 # Controller mixin for accessing a root ViewModel which can be accessed in a
-# collection by a parent model. Enabled by calling `nested_in :parent, as:
-# :children` on the viewmodel controller
+# collection by a parent model.
 
 # Contributes the following routes:
 # PUT    /parents/:parent_id/children            #append  -- deserialize (possibly existing) children and append to collection
@@ -32,6 +31,10 @@ module ViewModel::ActiveRecord::CollectionNestedController
     write_association(serialize_context: serialize_context, deserialize_context: deserialize_context, lock_owner: lock_owner, &block)
   end
 
+  def replace_bulk(serialize_context: new_serialize_context, deserialize_context: new_deserialize_context, &block)
+    write_association_bulk(serialize_context: serialize_context, deserialize_context: deserialize_context, &block)
+  end
+
   def disassociate_all(serialize_context: new_serialize_context, deserialize_context: new_deserialize_context, lock_owner: nil)
     destroy_association(true, serialize_context: serialize_context, deserialize_context: deserialize_context, lock_owner: lock_owner)
   end

data/lib/view_model/active_record/controller_base.rb

@@ -102,15 +102,35 @@ module ActionDispatch
         def arvm_resources(resource_name, options = {}, &block)
           except             = options.delete(:except) { [] }
           add_shallow_routes = options.delete(:add_shallow_routes) { true }
+          defaults           = options.delete(:defaults) { {} }
 
           nested = shallow_nesting_depth > 0
 
+          association_name = options.delete(:association_name) { resource_name.to_s }
+          owner_viewmodel  = options.delete(:owner_viewmodel) do
+            if nested
+              parent_resource.name.to_s.singularize.camelize
+            end
+          end
+
+          defaults = {
+            association_name: association_name,
+            owner_viewmodel: owner_viewmodel,
+          }.merge(defaults)
+
           only_routes = []
           only_routes += [:create] unless nested
           only_routes += [:show, :destroy] if add_shallow_routes
           only_routes -= except
 
-          resources resource_name, shallow: true, only: only_routes, **options do
+          # Bulk replace
+          if nested && !except.include?(:replace_bulk)
+            collection do
+              post resource_name, controller: resource_name, action: :replace_bulk, as: nil, defaults: defaults
+            end
+          end
+
+          resources resource_name, shallow: true, only: only_routes, defaults: defaults, **options do
             instance_eval(&block) if block_given?
 
             if nested
@@ -149,16 +169,35 @@ module ActionDispatch
         def arvm_resource(resource_name, options = {}, &block)
           except             = options.delete(:except) { [] }
           add_shallow_routes = options.delete(:add_shallow_routes) { true }
+          defaults           = options.delete(:defaults) { {} }
 
           only_routes = []
-          is_shallow = false
-          resource resource_name, shallow: true, only: only_routes, **options do
-            is_shallow = shallow_nesting_depth > 1
+          nested = shallow_nesting_depth > 0
+
+          association_name = options.delete(:association_name) { resource_name.to_s }
+          owner_viewmodel  = options.delete(:owner_viewmodel) do
+            if nested
+              parent_resource.name.to_s.singularize.camelize
+            end
+          end
+
+          defaults = {
+            association_name: association_name,
+            owner_viewmodel: owner_viewmodel,
+          }.merge(defaults)
+
+          if nested && !except.include?(:create_associated_bulk)
+            collection do
+              post resource_name, controller: resource_name, action: :create_associated_bulk, as: nil, defaults: defaults
+            end
+          end
+
+          resource resource_name, shallow: true, only: only_routes, defaults: defaults, **options do
             instance_eval(&block) if block_given?
 
             name_route = { as: '' } # Only one route may take the name
 
-            if is_shallow
+            if nested
               post('',   action: :create_associated,  **name_route.extract!(:as)) unless except.include?(:create)
               get('',    action: :show_associated,    **name_route.extract!(:as)) unless except.include?(:show)
               delete('', action: :destroy_associated, **name_route.extract!(:as)) unless except.include?(:destroy)
@@ -170,7 +209,7 @@ module ActionDispatch
           end
 
           # singularly nested resources provide collection accessors at the top level
-          if is_shallow && add_shallow_routes
+          if nested && add_shallow_routes
             resources resource_name.to_s.pluralize, shallow: true, only: [:show, :destroy] - except do
               shallow_scope do
                 collection do

data/lib/view_model/active_record/nested_controller_base.rb

@@ -7,9 +7,45 @@ require 'view_model/active_record/controller_base'
 module ViewModel::ActiveRecord::NestedControllerBase
   extend ActiveSupport::Concern
 
+  class ParentProxyModel < ViewModel
+    # Prevent this from appearing in hooks
+    self.synthetic = true
+
+    attr_reader :parent, :association_data, :changed_children
+
+    def initialize(parent, association_data, changed_children)
+      @parent = parent
+      @association_data = association_data
+      @changed_children = changed_children
+    end
+
+    def serialize(json, serialize_context:)
+      ViewModel::Callbacks.wrap_serialize(parent, context: serialize_context) do
+        child_context = parent.context_for_child(association_data.association_name, context: serialize_context)
+
+        json.set!(ViewModel::ID_ATTRIBUTE, parent.id)
+        json.set!(ViewModel::BULK_UPDATE_ATTRIBUTE) do
+          if association_data.referenced? && !association_data.owned?
+            if association_data.collection?
+              json.array!(changed_children) do |child|
+                ViewModel.serialize_as_reference(child, json, serialize_context: child_context)
+              end
+            else
+              ViewModel.serialize_as_reference(changed_children, json, serialize_context: child_context)
+            end
+          else
+            ViewModel.serialize(changed_children, json, serialize_context: child_context)
+          end
+        end
+      end
+    end
+  end
+
   protected
 
   def show_association(scope: nil, serialize_context: new_serialize_context, lock_owner: nil)
+    require_external_referenced_association!
+
     associated_views = nil
     pre_rendered = owner_viewmodel.transaction do
       owner_view = owner_viewmodel.find(owner_viewmodel_id, eager_include: false, lock: lock_owner)
@@ -27,11 +63,27 @@ module ViewModel::ActiveRecord::NestedControllerBase
     associated_views
   end
 
+  # This method always takes direct update hashes, and returns
+  # viewmodels directly.
+  #
+  # There's no multi membership, so when viewing the children of a
+  # single parent each child can only appear once. This means it's
+  # safe to use update hashes directly.
   def write_association(serialize_context: new_serialize_context, deserialize_context: new_deserialize_context, lock_owner: nil)
+    require_external_referenced_association!
+
     association_view = nil
     pre_rendered = owner_viewmodel.transaction do
       update_hash, refs = parse_viewmodel_updates
 
+      update_hash =
+        ViewModel::ActiveRecord.add_reference_indirection(
+          update_hash,
+          association_data: association_data,
+          references:       refs,
+          key:              'write-association',
+        )
+
       owner_view = owner_viewmodel.find(owner_viewmodel_id, eager_include: false, lock: lock_owner)
 
       association_view = owner_view.replace_associated(association_name, update_hash,
@@ -49,7 +101,67 @@ module ViewModel::ActiveRecord::NestedControllerBase
     association_view
   end
 
+  # This method takes direct update hashes for owned associations, and
+  # reference hashes for shared associations. The return value matches
+  # the input structure.
+  #
+  # If an association is referenced and owned, each child may only
+  # appear once so each is guaranteed to have a unique update
+  # hash. This means it's only safe to use update hashes directly in
+  # this case.
+  def write_association_bulk(serialize_context: new_serialize_context, deserialize_context: new_deserialize_context, lock_owner: nil)
+    require_external_referenced_association!
+
+    updated_by_parent_viewmodel = nil
+
+    pre_rendered = owner_viewmodel.transaction do
+      updates_by_parent_id, references = parse_bulk_update
+
+      if association_data.owned?
+        updates_by_parent_id.transform_values!.with_index do |update_hash, index|
+          ViewModel::ActiveRecord.add_reference_indirection(
+            update_hash,
+            association_data: association_data,
+            references:       references,
+            key:              "write-association-bulk-#{index}",
+          )
+        end
+      end
+
+      updated_by_parent_viewmodel =
+        owner_viewmodel.replace_associated_bulk(
+          association_name,
+          updates_by_parent_id,
+          references:          references,
+          deserialize_context: deserialize_context,
+        )
+
+      views = updated_by_parent_viewmodel.flat_map { |_parent_viewmodel, updated_views| Array.wrap(updated_views) }
+
+      ViewModel.preload_for_serialization(views)
+
+      updated_by_parent_viewmodel = yield(updated_by_parent_viewmodel) if block_given?
+
+      return_updates = updated_by_parent_viewmodel.map do |owner_view, updated_views|
+        ParentProxyModel.new(owner_view, association_data, updated_views)
+      end
+
+      return_structure = {
+        ViewModel::TYPE_ATTRIBUTE         => ViewModel::BULK_UPDATE_TYPE,
+        ViewModel::BULK_UPDATES_ATTRIBUTE => return_updates,
+      }
+
+      prerender_viewmodel(return_structure, serialize_context: serialize_context)
+    end
+
+    render_json_string(pre_rendered)
+    updated_by_parent_viewmodel
+  end
+
+
   def destroy_association(collection, serialize_context: new_serialize_context, deserialize_context: new_deserialize_context, lock_owner: nil)
+    require_external_referenced_association!
+
     if lock_owner
       owner_viewmodel.find(owner_viewmodel_id, eager_include: false, lock: lock_owner)
     end
@@ -61,7 +173,7 @@ module ViewModel::ActiveRecord::NestedControllerBase
   end
 
   def association_data
-    owner_viewmodel._association_data(association_name)
+    @association_data ||= owner_viewmodel._association_data(association_name)
   end
 
   def owner_update_hash(update)
@@ -78,22 +190,22 @@ module ViewModel::ActiveRecord::NestedControllerBase
     parse_param(id_param_name, **default)
   end
 
-  included do
-    delegate :owner_viewmodel, :association_name, to: 'self.class'
+  def owner_viewmodel_class_for_name(name)
+    ViewModel::Registry.for_view_name(name)
   end
 
-  class_methods do
-    attr_accessor :owner_viewmodel, :association_name
-
-    def nested_in(owner, as:)
-      unless owner.is_a?(Class) && owner < ViewModel::Record
-        owner = ViewModel::Registry.for_view_name(owner.to_s.camelize)
-      end
+  def owner_viewmodel
+    name = params.fetch(:owner_viewmodel) { raise ArgumentError.new("No owner viewmodel present") }
+    owner_viewmodel_class_for_name(name.to_s.camelize)
+  end
 
-      self.owner_viewmodel = owner
-      raise ArgumentError.new("Could not find owner ViewModel class '#{owner_name}'") if owner_viewmodel.nil?
+  def association_name
+    params.fetch(:association_name) { raise ArgumentError.new('No association name from routes') }
+  end
 
-      self.association_name = as
+  def require_external_referenced_association!
+    unless association_data.referenced? && association_data.external?
+      raise ArgumentError.new("Expected referenced external association: '#{association_name}'")
     end
   end
 end

data/lib/view_model/active_record/singular_nested_controller.rb

@@ -1,8 +1,7 @@
 # frozen_string_literal: true
 
 # Controller mixin for accessing a root ViewModel which can be accessed
-# individually by a parent model. Enabled by calling `nested_in :parent, as:
-# :child` on the viewmodel controller
+# individually by a parent model.
 
 # Contributes the following routes:
 # POST   /parents/:parent_id/child   #create_associated  -- deserialize (possibly existing) child, replacing existing child
@@ -28,6 +27,10 @@ module ViewModel::ActiveRecord::SingularNestedController
     write_association(serialize_context: serialize_context, deserialize_context: deserialize_context, lock_owner: lock_owner, &block)
   end
 
+  def create_associated_bulk(serialize_context: new_serialize_context, deserialize_context: new_deserialize_context, &block)
+    write_association_bulk(serialize_context: serialize_context, deserialize_context: deserialize_context, &block)
+  end
+
   def destroy_associated(serialize_context: new_serialize_context, deserialize_context: new_deserialize_context, lock_owner: nil)
     destroy_association(false, serialize_context: serialize_context, deserialize_context: deserialize_context, lock_owner: lock_owner)
   end