cocina-models 0.78.0 → 0.81.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '09a3e28b24593aeb0e75c0318067e72fc8e10bb98e189d00b3f166357b198cab'
-  data.tar.gz: 688dbfe3ef449fe8caccd5c3518a6cd059cff2920ad9af3d9fb11553516b520e
+  metadata.gz: 0d8392fadece0bb4ec12ed93e958c99abf04e29ad0a78c14ee8651ec8ecd1e37
+  data.tar.gz: db4a5de344bb53d98b2b71f725e8b1d1b9e0d5e0b28416989ad55f19df826376
 SHA512:
-  metadata.gz: 3bc1bce3e8624fa829ce0f41b69362d723e77549d102d561b41245c462c178cbcac5733fcd0816fd7750fa7bf736611e46de503e1e6b93dfce009ae6bd33eea4
-  data.tar.gz: 5cb526603a08485e0c04abccfa962b92b890fbf9d966d5dc00cb5ddd9aec64b24ebb4aa555397d769d5814edb914960d7d83a1b7f5c23be6752252f34b64a033
+  metadata.gz: 4880e4eb86935116e0bec017926aeab5e5aa12d8792ee5066d0aeef5f514e6f13f63c8bc68c03035fdff2748f95aa7215f236384e109a8e532bd76e9677ae53c
+  data.tar.gz: 22154804ee4b103bdbd3aaaee449327bb4a6c9bc1dce9431a5311ed06c5e76039ccb420634d275f56d644a60ffa529bd9cb822f9c549be057f4972d791a34bf7

data/description_types.yml

@@ -391,6 +391,8 @@ identifier:
   - value: record id
   - value: Senate Number
   - value: Series
+  - value: SICI
+    code: sici
   - value: SIRSI
   - value: Source ID
   - value: sourceID

data/docs/description_types.md

@@ -385,6 +385,7 @@ _Path: identifier.type_
   * record id
   * Senate Number
   * Series
+  * SICI
   * SIRSI
   * Source ID
   * sourceID

data/lib/cocina/generator/schema.rb

@@ -13,8 +13,8 @@ module Cocina
           # frozen_string_literal: true
 
           module Cocina
-            module Models
-              class #{name} < Struct
+            module Models#{'              '}
+              #{preamble}class #{name} < Struct
 
                 #{validate}
                 #{types}

data/lib/cocina/generator/schema_base.rb

@@ -41,6 +41,12 @@ module Cocina
         "# #{schema_doc.description}\n"
       end
 
+      def deprecation
+        return '' unless schema_doc.deprecated?
+
+        "# DEPRECATED\n"
+      end
+
       def example
         return '' unless schema_doc.example
 
@@ -82,6 +88,10 @@ module Cocina
           'Strict::String'
         end
       end
+
+      def preamble
+        "#{deprecation}#{description}#{example}#{relaxed_comment}"
+      end
     end
   end
 end

data/lib/cocina/generator/schema_value.rb

@@ -19,10 +19,6 @@ module Cocina
         "Types::#{dry_datatype(schema_doc)}#{optional}#{default}#{enum}"
       end
 
-      def preamble
-        "#{description}#{example}#{relaxed_comment}"
-      end
-
       def enum
         return '' if !schema_doc.enum || relaxed
 

data/lib/cocina/models/access_role.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Access role conferred by an AdminPolicy to objects within it. (used by Argo)
     class AccessRole < Struct
       # Name of role
       attribute :name, Types::Strict::String.enum('dor-apo-depositor', 'dor-apo-manager', 'dor-apo-viewer', 'sdr-administrator', 'sdr-viewer', 'hydrus-collection-creator', 'hydrus-collection-manager', 'hydrus-collection-depositor', 'hydrus-collection-item-depositor', 'hydrus-collection-reviewer', 'hydrus-collection-viewer')

data/lib/cocina/models/access_role_member.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Represents a user or group that is a member of an AccessRole
     class AccessRoleMember < Struct
       include Checkable
 

data/lib/cocina/models/admin_policy_access_template.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Provides the template of access settings that is copied to the items goverend by an AdminPolicy. This is almost the same as DROAccess, but it provides no defaults and has no embargo.
     class AdminPolicyAccessTemplate < Struct
       attribute? :view, Types::Strict::String.enum('world', 'stanford', 'location-based', 'citation-only', 'dark')
       # Available for controlled digital lending.

data/lib/cocina/models/admin_policy_administrative.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Administrative properties for an AdminPolicy
     class AdminPolicyAdministrative < Struct
       attribute(:accessTemplate, AdminPolicyAccessTemplate.default { AdminPolicyAccessTemplate.new })
       attribute :registrationWorkflow, Types::Strict::Array.of(Types::Strict::String).default([].freeze)

data/lib/cocina/models/admin_policy_with_metadata.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Admin Policy with addition object metadata.
     class AdminPolicyWithMetadata < Struct
       include Validatable
 

data/lib/cocina/models/applies_to.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Property model for indicating the parts, aspects, or versions of the resource to which a descriptive element is applicable.
     class AppliesTo < Struct
       attribute :appliesTo, Types::Strict::Array.of(DescriptiveBasicValue).default([].freeze)
     end

data/lib/cocina/models/builders/name_title_group_builder.rb

@@ -92,7 +92,6 @@ module Cocina
           # ignoring groupedValue
           slices.flatten
         end
-        # private_class_method :value_slices
 
         # for a given Hash (from a Cocina DescriptiveValue or Title or Name or ...)
         # result will be either

data/lib/cocina/models/collection.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # A group of Digital Repository Objects that indicate some type of conceptual grouping within the domain that is worth reusing across the system.
     class Collection < Struct
       include Validatable
 

data/lib/cocina/models/collection_access.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Access metadata for collections
     class CollectionAccess < Struct
       # Access level
       attribute? :view, Types::Strict::String.default('dark').enum('world', 'dark')

data/lib/cocina/models/collection_with_metadata.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Collection with addition object metadata.
     class CollectionWithMetadata < Struct
       include Validatable
 

data/lib/cocina/models/contributor.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Property model for describing agents contributing in some way to the creation and history of the resource.
     class Contributor < Struct
       attribute :name, Types::Strict::Array.of(DescriptiveValue).default([].freeze)
       # Entity type of the contributor (person, organization, etc.). See https://github.com/sul-dlss/cocina-models/blob/main/docs/description_types.md for valid types.

data/lib/cocina/models/descriptive_access_metadata.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Information about how to access digital and physical versions of the object.
     class DescriptiveAccessMetadata < Struct
       attribute :url, Types::Strict::Array.of(DescriptiveValue).default([].freeze)
       attribute :physicalLocation, Types::Strict::Array.of(DescriptiveValue).default([].freeze)

data/lib/cocina/models/descriptive_admin_metadata.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Information about this resource description.
     class DescriptiveAdminMetadata < Struct
       attribute :contributor, Types::Strict::Array.of(Contributor).default([].freeze)
       attribute :event, Types::Strict::Array.of(Event).default([].freeze)

data/lib/cocina/models/descriptive_basic_value.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Basic value model for descriptive elements. Can only have one of value, parallelValue, groupedValue, or structuredValue.
     class DescriptiveBasicValue < Struct
       attribute :structuredValue, Types::Strict::Array.of(DescriptiveValue).default([].freeze)
       attribute :parallelValue, Types::Strict::Array.of(DescriptiveValue).default([].freeze)

data/lib/cocina/models/descriptive_geographic_metadata.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Value model for mods geographic extension metadata
     class DescriptiveGeographicMetadata < Struct
       attribute :form, Types::Strict::Array.of(DescriptiveValue).default([].freeze)
       attribute :subject, Types::Strict::Array.of(DescriptiveValue).default([].freeze)

data/lib/cocina/models/descriptive_grouped_value.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Value model for a set of descriptive elements grouped together in an unstructured way.
     class DescriptiveGroupedValue < Struct
       attribute :groupedValue, Types::Strict::Array.of(DescriptiveValue).default([].freeze)
     end

data/lib/cocina/models/descriptive_parallel_contributor.rb

@@ -2,6 +2,8 @@
 
 module Cocina
   module Models
+    # DEPRECATED
+    # Value model for multiple representations of information about the same contributor (e.g. in different languages).
     class DescriptiveParallelContributor < Struct
       attribute :name, Types::Strict::Array.of(DescriptiveValue).default([].freeze)
       # Entity type of the contributor (person, organization, etc.). See https://github.com/sul-dlss/cocina-models/blob/main/docs/description_types.md for valid types.

data/lib/cocina/models/descriptive_parallel_event.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Value model for multiple representations of information about the same event (e.g. in different languages).
     class DescriptiveParallelEvent < Struct
       attribute :structuredValue, Types::Strict::Array.of(DescriptiveValue).default([].freeze)
       # Description of the event (creation, publication, etc.).

data/lib/cocina/models/descriptive_parallel_value.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Value model for multiple representations of the same information (e.g. in different languages).
     class DescriptiveParallelValue < Struct
       attribute :parallelValue, Types::Strict::Array.of(DescriptiveValue).default([].freeze)
     end

data/lib/cocina/models/descriptive_structured_value.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Value model for descriptive elements structured as typed, ordered values.
     class DescriptiveStructuredValue < Struct
       attribute :structuredValue, Types::Strict::Array.of(DescriptiveValue).default([].freeze)
     end

data/lib/cocina/models/descriptive_value.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Default value model for descriptive elements.
     class DescriptiveValue < Struct
       attribute :structuredValue, Types::Strict::Array.of(DescriptiveValue).default([].freeze)
       attribute :parallelValue, Types::Strict::Array.of(DescriptiveValue).default([].freeze)

data/lib/cocina/models/descriptive_value_language.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Language of the descriptive element value
     class DescriptiveValueLanguage < Struct
       # Code representing the standard or encoding.
       attribute? :code, Types::Strict::String

data/lib/cocina/models/dro.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Domain-defined abstraction of a 'work'. Digital Repository Objects' abstraction is describable for our domain’s purposes, i.e. for management needs within our system.
     class DRO < Struct
       include Validatable
 

data/lib/cocina/models/dro_structural.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Structural metadata
     class DROStructural < Struct
       attribute :contains, Types::Strict::Array.of(FileSet).default([].freeze)
       attribute :hasMemberOrders, Types::Strict::Array.of(Sequence).default([].freeze)

data/lib/cocina/models/dro_with_metadata.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # DRO with addition object metadata.
     class DROWithMetadata < Struct
       include Validatable
 

data/lib/cocina/models/event.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Property model for describing events in the history of the resource.
     class Event < Struct
       attribute :structuredValue, Types::Strict::Array.of(DescriptiveValue).default([].freeze)
       # Description of the event (creation, publication, etc.).

data/lib/cocina/models/file.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Binaries that are the basis of what our domain manages. Binaries here do not include metadata files generated for the domain's own management purposes.
     class File < Struct
       include Checkable
 

data/lib/cocina/models/file_access.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Access metadata for files
     class FileAccess < Struct
       # Access level.
       # Validation of this property is relaxed. See the openapi for full validation.

data/lib/cocina/models/file_set.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Relevant groupings of Files. Also called a File Grouping.
     class FileSet < Struct
       include Checkable
 

data/lib/cocina/models/file_set_structural.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Structural metadata
     class FileSetStructural < Struct
       attribute :contains, Types::Strict::Array.of(File).default([].freeze)
     end

data/lib/cocina/models/geographic.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Geographic metadata
     class Geographic < Struct
       # Geographic ISO 19139 XML metadata
       attribute :iso19139, Types::Strict::String

data/lib/cocina/models/language.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Languages, scripts, symbolic systems, and notations used in all or part of a resource or its descriptive metadata.
     class Language < Struct
       attribute :appliesTo, Types::Strict::Array.of(DescriptiveBasicValue).default([].freeze)
       # Code value of the descriptive element.

data/lib/cocina/models/mapping/from_mods/name_builder.rb

@@ -36,13 +36,13 @@ module Cocina
 
           def build_parallel
             names = {
-              parallelValue: name_elements.map { |name_node| build_parallel_name(name_node) },
+              parallelValue: build_parallel_values,
               type: type_for(name_elements.first['type']),
               status: name_elements.filter_map { |name_element| name_element['usage'] }.first
             }.compact
             { name: [names] }.tap do |attrs|
-              roles = name_elements.flat_map { |name_node| build_roles(name_node) }.compact.uniq
-              attrs[:role] = roles.presence
+              attrs[:role] = name_elements.flat_map { |name_node| build_roles(name_node) }.compact.uniq.presence
+              attrs[:note] = name_elements.flat_map { |name_node| build_affiliation_notes(name_node) }.compact.uniq.presence
             end.compact
           end
 
@@ -51,8 +51,8 @@ module Cocina
               status: name_node['usage']
             }.compact.merge(common_lang_script(name_node))
 
-            name_attrs = name_attrs.merge(common_name(name_node, name_attrs[:name]))
-            name_parts = build_name_parts(name_node)
+            name_attrs = name_attrs.merge(common_name(name_node, name_attrs[:name], is_parallel: true))
+            name_parts = build_name_parts(name_node, parallel: true)
             notifier.warn('Missing name/namePart element') if name_parts.all?(&:empty?)
             name_parts.each { |name_part| name_attrs = name_part.merge(name_attrs) }
             name_attrs.compact
@@ -85,9 +85,9 @@ module Cocina
             }.compact.merge(common_name(name_node, name_parts))
           end
 
-          def common_name(name_node, name)
+          def common_name(name_node, name, is_parallel: false)
             {
-              note: build_notes(name_node),
+              note: build_notes(name_node, is_parallel: is_parallel),
               identifier: build_identifier(name_node)
             }.tap do |attrs|
               roles = build_roles(name_node)
@@ -106,7 +106,17 @@ module Cocina
             end.compact
           end
 
-          def build_name_parts(name_node)
+          def build_parallel_values
+            parallel_values = []
+            name_elements.each do |name_node|
+              parallel_values << build_parallel_name(name_node)
+              display_val = display_value(name_node)
+              parallel_values << display_val if display_val
+            end
+            parallel_values.compact
+          end
+
+          def build_name_parts(name_node, parallel: false)
             name_part_nodes = name_node.xpath('mods:namePart', mods: Description::DESC_METADATA_NS)
             alternative_name_nodes = name_node.xpath('mods:alternativeName', mods: Description::DESC_METADATA_NS)
 
@@ -116,23 +126,64 @@ module Cocina
               parts << { valueAt: name_node['xlink:href'] } if name_node['xlink:href']
               parts << common_authority(name_node) if name_node['valueURI']
             when 1
-              parts << build_name_part(name_node, name_part_nodes.first,
-                                       default_type: alternative_name_nodes.present?)
-                       .merge(common_authority(name_node)).merge(common_lang_script(name_node)).presence
+              name = build_simple_value_name(name_node, name_part_nodes.first, alternative_name_nodes, parallel)
+              parts << name.merge(common_authority(name_node)).merge(common_lang_script(name_node)).presence
             else
-              vals = name_part_nodes.filter_map do |name_part|
-                build_name_part(name_node, name_part, default_type: name_node['type'] != 'corporate').presence
-              end
-              parts << { structuredValue: vals }.merge(common_authority(name_node)).merge(common_lang_script(name_node))
+              name = build_structured_value_name(name_node, name_part_nodes)
+              parts << name.merge(common_authority(name_node)).merge(common_lang_script(name_node))
             end
 
             parts = build_alternative_name(alternative_name_nodes, parts) if alternative_name_nodes.present?
-
-            display_form = name_node.xpath('mods:displayForm', mods: Description::DESC_METADATA_NS).first
-            parts << { value: display_form.text, type: 'display' } if display_form
             parts.compact
           end
 
+          def build_simple_value_name(name_node, name_part_node, alternative_name_nodes, parallel)
+            name_value_hash = build_name_part(name_node, name_part_node, default_type: alternative_name_nodes.present?)
+            display_form_node = name_node.xpath('mods:displayForm', mods: Description::DESC_METADATA_NS).first
+            if display_form_node.present? && !parallel
+              cocina_contrib_name =
+                {
+                  parallelValue: [
+                    name_value_hash
+                  ]
+                }
+              add_display_parallel_value(name_node, cocina_contrib_name)
+            else
+              name_value_hash
+            end
+          end
+
+          def build_structured_value_name(name_node, name_part_nodes)
+            vals = name_part_nodes.filter_map { |name_part_node| build_name_part(name_node, name_part_node, default_type: name_node['type'] != 'corporate').presence }
+            display_form_node = name_node.xpath('mods:displayForm', mods: Description::DESC_METADATA_NS).first
+            if display_form_node.present?
+              cocina_contrib_name =
+                {
+                  parallelValue: [
+                    { structuredValue: vals }
+                  ]
+                }
+              add_display_parallel_value(name_node, cocina_contrib_name)
+            else
+              { structuredValue: vals }
+            end
+          end
+
+          def add_display_parallel_value(name_node, cocina_contrib_name)
+            display_form = name_node.xpath('mods:displayForm', mods: Description::DESC_METADATA_NS)&.text
+            return cocina_contrib_name if display_form.blank?
+
+            display_parallel_value = display_value(name_node)
+            cocina_contrib_name[:parallelValue] << display_parallel_value if display_parallel_value && cocina_contrib_name[:parallelValue].present?
+
+            cocina_contrib_name
+          end
+
+          def display_value(name_node)
+            display_form = name_node.xpath('mods:displayForm', mods: Description::DESC_METADATA_NS)&.text
+            { value: display_form, type: 'display' } if display_form.present?
+          end
+
           def build_name_part(name_node, name_part_node, default_type: true)
             if name_part_node.content.blank? && !name_part_node['xlink:href']
               notifier.warn('name/namePart missing value')
@@ -224,12 +275,8 @@ module Cocina
             end.presence
           end
 
-          def build_notes(name_node)
+          def build_notes(name_node, is_parallel:)
             [].tap do |parts|
-              name_node.xpath('mods:affiliation', mods: Description::DESC_METADATA_NS).each do |affiliation_node|
-                parts << { value: affiliation_node.text, type: 'affiliation' }
-              end
-
               description = name_node.xpath('mods:description', mods: Description::DESC_METADATA_NS).first
               if description
                 parts << if description.text == UNCITED_DESCRIPTION
@@ -238,9 +285,18 @@ module Cocina
                            { value: description.text, type: 'description' }
                          end
               end
+              parts.concat(build_affiliation_notes(name_node)) unless is_parallel
             end.presence
           end
 
+          def build_affiliation_notes(name_node)
+            [].tap do |parts|
+              name_node.xpath('mods:affiliation', mods: Description::DESC_METADATA_NS).each do |affiliation_node|
+                parts << { value: affiliation_node.text, type: 'affiliation' }
+              end
+            end
+          end
+
           def build_roles(name_node)
             role_nodes = name_node.xpath('mods:role', mods: Description::DESC_METADATA_NS)
             role_nodes.filter_map { |role_node| role_for(role_node) }.presence

data/lib/cocina/models/mapping/normalizers/mods/name_normalizer.rb

@@ -20,6 +20,7 @@ module Cocina
 
             def normalize
               normalize_parallel_name_role
+              normalize_parallel_affiliation_note
               normalize_text_role_term
               normalize_role_term
               normalize_role # must be after normalize_role_term
@@ -35,10 +36,13 @@ module Cocina
 
             attr_reader :ng_xml
 
+            def grouped_name_nodes
+              name_nodes = ng_xml.root.xpath('//mods:name[@altRepGroup]', mods: ModsNormalizer::MODS_NS)
+              name_nodes.group_by { |name_node| name_node['altRepGroup'] }.values.reject { |name_node_group| name_node_group.size == 1 }
+            end
+
             def normalize_parallel_name_role
               # For parallel names, all should have the same roles.
-              name_nodes = ng_xml.root.xpath('//mods:name[@altRepGroup]', mods: ModsNormalizer::MODS_NS)
-              grouped_name_nodes = name_nodes.group_by { |name_node| name_node['altRepGroup'] }.values.reject { |name_node_group| name_node_group.size == 1 }
               grouped_name_nodes.each do |name_node_group|
                 name_node_with_role = name_node_group.find { |name_node| role_node_for(name_node) }
                 next unless name_node_with_role
@@ -58,6 +62,27 @@ module Cocina
               name_node.xpath('mods:role', mods: ModsNormalizer::MODS_NS).first
             end
 
+            def normalize_parallel_affiliation_note
+              # For parallel names, all should have the same affiliation.
+              grouped_name_nodes.each do |name_node_group|
+                name_node_with_affiliation = name_node_group.find { |name_node| affiliation_node_for(name_node) }
+                next unless name_node_with_affiliation
+
+                name_node_group.each do |name_node|
+                  next if name_node == name_node_with_affiliation
+
+                  existing_affiliation_node = affiliation_node_for(name_node)
+                  existing_affiliation_node&.remove
+
+                  name_node << affiliation_node_for(name_node_with_affiliation).dup
+                end
+              end
+            end
+
+            def affiliation_node_for(name_node)
+              name_node.xpath('mods:affiliation', mods: ModsNormalizer::MODS_NS).first
+            end
+
             def normalize_text_role_term
               # Add the type="text" attribute to roleTerms that don't have a type (seen in MODS 3.3 druid:yy910cj7795)
               ng_xml.root.xpath('//mods:roleTerm[not(@type)]', mods: ModsNormalizer::MODS_NS).each do |role_term_node|

data/lib/cocina/models/mapping/to_mods/name_writer.rb

@@ -36,25 +36,11 @@ module Cocina
             if contributor.type == 'unspecified others'
               write_etal
             elsif contributor.name.present?
+              # Expect contributor to have a single value for name property
               contrib_name = contributor.name.first
-              parallel_values = contrib_name.parallelValue
-              if parallel_values.present?
-                altrepgroup_id = id_generator.next_altrepgroup
-                parallel_values.each do |parallel_contrib_name|
-                  Cocina::Models::Builders::NameTitleGroupBuilder.value_slices(parallel_contrib_name)&.each do |parallel_contrib_name_slice|
-                    if name_title_vals_index[parallel_contrib_name_slice]
-                      name_title_group = name_title_vals_index[parallel_contrib_name_slice]&.values&.first
-                      write_parallel_contributor(contributor, contrib_name, parallel_contrib_name,
-                                                 name_title_group, altrepgroup_id)
-                    else
-                      # TODO:  want a way to notify that we hit a problem - either notifier or HB error (issue #3751)
-                      #  OR validate for semantic correctness upon creation/update so we can't get here.
-                      #  notifier.warn("For contributor name '#{parallel_contrib_name_val}', no title matching '#{title_from_contrib}'")
-                      write_parallel_contributor(contributor, contrib_name, parallel_contrib_name, nil,
-                                                 altrepgroup_id)
-                    end
-                  end
-                end
+              parallel_name_values = contrib_name.parallelValue
+              if parallel_name_values.present?
+                write_contributor_with_parallel_names(parallel_name_values, contrib_name)
               else
                 write_contributor(contributor)
               end
@@ -105,7 +91,50 @@ module Cocina
             end
           end
 
-          def write_parallel_contributor(contributor, name, parallel_name, name_title_group, altrepgroup_id)
+          def write_contributor_with_parallel_names(parallel_name_values, contrib_name)
+            display_type_parallel_name = display_type_parallel_name(parallel_name_values)
+            if parallel_name_values.size == 1
+              contrib_name_value_slice = Cocina::Models::Builders::NameTitleGroupBuilder.value_slices(parallel_name_values.first)
+              name_title_group = name_title_vals_index[contrib_name_value_slice]&.values&.first
+              write_name_from_parallel(contributor, contributor.name.first, parallel_name_values, name_title_group, nil)
+            elsif parallel_name_values.size == 2 && display_type_parallel_name
+              contrib_name_value_slice = Cocina::Models::Builders::NameTitleGroupBuilder.value_slices(parallel_name_values.first)
+              name_title_group = name_title_vals_index[contrib_name_value_slice]&.values&.first
+              write_name_with_display_form(contributor, contributor.name.first, parallel_name_values, 0, name_title_group, nil)
+            else
+              write_multiple_parallel_contributors(parallel_name_values, contrib_name)
+            end
+          end
+
+          def write_multiple_parallel_contributors(parallel_name_values, contrib_name)
+            altrepgroup_id = id_generator.next_altrepgroup
+            parallel_name_values.each_with_index do |parallel_contrib_name, index|
+              display_name_present = parallel_name_values[index + 1].present? && parallel_name_values[index + 1].type == 'display'
+              Cocina::Models::Builders::NameTitleGroupBuilder.value_slices(parallel_contrib_name)&.each do |parallel_contrib_name_slice|
+                if name_title_vals_index[parallel_contrib_name_slice]
+                  name_title_group = name_title_vals_index[parallel_contrib_name_slice]&.values&.first
+                  if display_name_present
+                    # associate type 'display' with the previous value
+                    write_name_with_display_form(contributor, contrib_name, parallel_name_values, index, name_title_group, altrepgroup_id)
+                  else
+                    write_name_from_parallel(contributor, contrib_name, parallel_contrib_name, name_title_group, altrepgroup_id)
+                  end
+                elsif display_name_present
+                  # TODO:  want a way to notify that we hit a problem - either notifier or HB error (issue #3751)
+                  #  OR validate for semantic correctness upon creation/update so we can't get here.
+                  #  notifier.warn("For contributor name '#{parallel_contrib_name_slice}', no contrib matching")
+                  write_name_with_display_form(contributor, contrib_name, parallel_name_values, index, nil, altrepgroup_id)
+                elsif parallel_name_values[index].type == 'display'
+                  # we assume we included this as part of a previous name
+                  next
+                else
+                  write_name_from_parallel(contributor, contrib_name, parallel_contrib_name, nil, altrepgroup_id)
+                end
+              end
+            end
+          end
+
+          def write_name_from_parallel(contributor, name, parallel_name, name_title_group, altrepgroup_id)
             attributes = parallel_name_attributes(name, parallel_name, name_title_group, altrepgroup_id)
             type_attr = NAME_TYPE.fetch(contributor.type, name_title_group ? 'personal' : nil)
             attributes[:type] = type_attr if type_attr
@@ -121,6 +150,37 @@ module Cocina
             end
           end
 
+          # rubocop:disable Metrics/ParameterLists
+          def write_name_with_display_form(contributor, name, parallel_name_values, index, name_title_group, altrepgroup_id)
+            display_type_parallel_name = display_type_parallel_name(parallel_name_values)
+            parallel_name = parallel_name_values[index]
+            return if parallel_name.blank?
+
+            attributes = if altrepgroup_id.present?
+                           parallel_name_attributes(name, parallel_name, name_title_group, altrepgroup_id)
+                         else
+                           name_attributes(contributor, name, nil)
+                         end
+            type_attr = NAME_TYPE.fetch(contributor.type, name_title_group ? 'personal' : nil)
+            attributes[:type] = type_attr if type_attr
+            xml.name attributes do
+              if parallel_name.structuredValue.present?
+                write_structured(parallel_name)
+              else
+                write_basic(parallel_name)
+              end
+              write_display_form(display_type_parallel_name) if display_type_parallel_name.present?
+              write_identifier(contributor) if contributor.identifier.present?
+              write_note(contributor)
+              write_roles(contributor)
+            end
+          end
+          # rubocop:enable Metrics/ParameterLists
+
+          def display_type_parallel_name(parallel_name_values)
+            parallel_name_values.detect { |parallel_value| parallel_value.type == 'display' }
+          end
+
           def parallel_name_attributes(name, parallel_name, name_title_group, altrepgroup_id)
             {
               nameTitleGroup: name_title_group,
@@ -199,8 +259,6 @@ module Cocina
                 xml.affiliation note.value
               when 'description'
                 xml.description note.value
-              when 'citation status'
-                xml.description UNCITED_DESCRIPTION if note.value == 'false'
               end
             end
           end

data/lib/cocina/models/mapping/to_mods/subject.rb

@@ -394,7 +394,7 @@ module Cocina
             xml.name name_attrs do
               write_name_part(subject_value)
               write_display_form(display_values)
-              write_roles(subject.note)
+              write_roles(subject_value.note)
               write_other_notes(subject.note, 'description')
               write_other_notes(subject.note, 'affiliation')
             end
@@ -410,7 +410,7 @@ module Cocina
               write_display_form(display_values)
               write_roles(subject.note)
               write_other_notes(subject.note, 'description')
-              write_other_notes(subject.note, 'affiliation')
+              write_other_notes(subject_value.note, 'affiliation')
             end
             write_genres(subject_value)
           end

data/lib/cocina/models/message_digest.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # The output of the message digest algorithm.
     class MessageDigest < Struct
       include Checkable
 

data/lib/cocina/models/object_metadata.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Metadata for a cocina object.
     class ObjectMetadata < Struct
       # When the object was created.
       attribute? :created, Types::Params::DateTime

data/lib/cocina/models/presentation.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Presentation data for the File.
     class Presentation < Struct
       # Height in pixels
       attribute? :height, Types::Strict::Integer

data/lib/cocina/models/related_resource.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Other resource associated with the described resource.
     class RelatedResource < Struct
       # The relationship of the related resource to the described resource.
       attribute? :type, Types::Strict::String

data/lib/cocina/models/release_tag.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # A tag that indicates the item or collection should be released.
     class ReleaseTag < Struct
       # Who did this release
       # example: petucket

data/lib/cocina/models/request_admin_policy.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Same as an AdminPolicy, but doesn't have an externalIdentifier as one will be created
     class RequestAdminPolicy < Struct
       include Validatable
 

data/lib/cocina/models/request_collection.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Same as a Collection, but doesn't have an externalIdentifier as one will be created
     class RequestCollection < Struct
       include Validatable
 

data/lib/cocina/models/request_description.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Description that is included in a request to create a DRO. This is the same as a Description, except excludes PURL.
     class RequestDescription < Struct
       include Validatable
 

data/lib/cocina/models/request_dro.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # A request to create a DRO.  This has the same general structure as a DRO but doesn't have externalIdentifier and doesn't require the access subschema. If no access subschema is provided, these values will be inherited from the AdminPolicy.
     class RequestDRO < Struct
       include Validatable
 

data/lib/cocina/models/request_dro_structural.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Structural metadata
     class RequestDROStructural < Struct
       attribute :contains, Types::Strict::Array.of(RequestFileSet).default([].freeze)
       attribute :hasMemberOrders, Types::Strict::Array.of(Sequence).default([].freeze)

data/lib/cocina/models/request_file_set_structural.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Structural metadata
     class RequestFileSetStructural < Struct
       attribute :contains, Types::Strict::Array.of(RequestFile).default([].freeze)
     end

data/lib/cocina/models/request_identification.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Same as a Identification, but requires a sourceId and doesn't permit a DOI.
     class RequestIdentification < Struct
       # A barcode
       attribute? :barcode, Types::Nominal::Any

data/lib/cocina/models/sequence.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # A sequence or ordering of resources within a Collection or Object.
     class Sequence < Struct
       attribute :members, Types::Strict::Array.of(Types::Strict::String).default([].freeze)
       # The direction that a sequence of canvases should be displayed to the user

data/lib/cocina/models/source.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Property model for indicating the vocabulary, authority, or other origin for a term, code, or identifier.
     class Source < Struct
       # Code representing the value source.
       attribute? :code, Types::Strict::String

data/lib/cocina/models/standard.rb

@@ -2,6 +2,7 @@
 
 module Cocina
   module Models
+    # Property model for indicating the encoding, standard, or syntax to which a value conforms (e.g. RDA).
     class Standard < Struct
       # Code representing the standard or encoding.
       attribute? :code, Types::Strict::String

data/lib/cocina/models/validators/associated_name_validator.rb

@@ -11,7 +11,7 @@ module Cocina
 
         def initialize(clazz, attributes)
           @clazz = clazz
-          @attributes = attributes.deep_symbolize_keys
+          @attributes = attributes
           @error_paths = []
         end
 
@@ -29,9 +29,7 @@ module Cocina
         attr_reader :clazz, :attributes, :error_paths
 
         def meets_preconditions?
-          resources.any? do |resource|
-            titles_with_associated_name_note_for(resource).present?
-          end
+          [Cocina::Models::Description, Cocina::Models::RequestDescription].include?(clazz)
         end
 
         def valid?(resource)

data/lib/cocina/models/validators/description_types_validator.rb

@@ -11,7 +11,7 @@ module Cocina
 
         def initialize(clazz, attributes)
           @clazz = clazz
-          @attributes = attributes.deep_symbolize_keys
+          @attributes = attributes
           @error_paths = []
         end
 
@@ -30,8 +30,7 @@ module Cocina
         attr_reader :clazz, :attributes, :error_paths
 
         def meets_preconditions?
-          attributes.key?(:description) || [Cocina::Models::Description,
-                                            Cocina::Models::RequestDescription].include?(clazz)
+          [Cocina::Models::Description, Cocina::Models::RequestDescription].include?(clazz)
         end
 
         def validate_hash(hash, path)

data/lib/cocina/models/validators/description_values_validator.rb

@@ -11,7 +11,7 @@ module Cocina
 
         def initialize(clazz, attributes)
           @clazz = clazz
-          @attributes = attributes.deep_symbolize_keys
+          @attributes = attributes
           @error_paths = []
         end
 
@@ -30,8 +30,7 @@ module Cocina
         attr_reader :clazz, :attributes, :error_paths
 
         def meets_preconditions?
-          attributes.key?(:description) || [Cocina::Models::Description,
-                                            Cocina::Models::RequestDescription].include?(clazz)
+          [Cocina::Models::Description, Cocina::Models::RequestDescription].include?(clazz)
         end
 
         def validate_hash(hash, path)

data/lib/cocina/models/validators/validator.rb

@@ -26,8 +26,7 @@ module Cocina
           # In the meantime, copying code.
           attributes_hash = deep_transform_values(attributes.to_h) do |value|
             value.class.name.starts_with?('Cocina::Models') ? value.to_h : value
-          end.with_indifferent_access
-
+          end.deep_symbolize_keys.with_indifferent_access
           VALIDATORS.each { |validator| validator.validate(clazz, attributes_hash) }
         end
 

data/lib/cocina/models/version.rb

@@ -2,6 +2,6 @@
 
 module Cocina
   module Models
-    VERSION = '0.78.0'
+    VERSION = '0.81.0'
   end
 end

data/lib/cocina/models.rb

@@ -119,7 +119,7 @@ module Cocina
     # @param [DROWithMetadata,CollectionWithMetadata,AdminPolicyWithMetadata] cocina_object
     # @return [DRO,Collection,AdminPolicy]
     def self.without_metadata(cocina_object)
-      build(cocina_object.to_h.except(:created, :modified, :lock))
+      build(cocina_object.to_h.except(:created, :modified, :lock), validate: false)
     end
 
     # Adds metadata to a DRO, Collection, AdminPolicy
@@ -144,7 +144,7 @@ module Cocina
               else
                 AdminPolicyWithMetadata
               end
-      clazz.new(props)
+      clazz.new(props, false, false)
     end
 
     def self.type_for(dyn)

data/lib/cocina/rspec/factories.rb

@@ -45,24 +45,31 @@ module Cocina
         type: Cocina::Models::ObjectType.object,
         id: 'druid:bc234fg5678',
         version: 1,
-        label: 'test object',
-        title: 'test object',
+        label: 'factory DRO label',
+        title: 'factory DRO title',
         source_id: 'sul:1234',
         admin_policy_id: 'druid:hv992ry2431'
       }.freeze
 
       REQUEST_DRO_DEFAULTS = DRO_DEFAULTS.except(:id)
 
-      COLLECTION_DEFAULTS = DRO_DEFAULTS.except(:source_id).merge(type: Cocina::Models::ObjectType.collection)
+      COLLECTION_DEFAULTS = {
+        type: Cocina::Models::ObjectType.collection,
+        id: 'druid:bb222ff5555',
+        version: 1,
+        label: 'factory collection label',
+        title: 'factory collection title',
+        admin_policy_id: 'druid:hv992ry2431'
+      }.freeze
 
       REQUEST_COLLECTION_DEFAULTS = COLLECTION_DEFAULTS.except(:id)
 
       ADMIN_POLICY_DEFAULTS = {
         type: Cocina::Models::ObjectType.admin_policy,
-        id: 'druid:bc234fg5678',
+        id: 'druid:cb432gf8765',
         version: 1,
-        label: 'test admin policy',
-        title: 'test admin policy',
+        label: 'factory APO label',
+        title: 'factory APO title',
         admin_policy_id: 'druid:hv992ry2431',
         agreement_id: 'druid:hp308wm0436'
       }.freeze

data/openapi.yml

@@ -779,6 +779,7 @@ components:
             $ref: "#/components/schemas/DescriptiveValue"
     DescriptiveParallelContributor:
       description: Value model for multiple representations of information about the same contributor (e.g. in different languages).
+      deprecated: true
       type: object
       additionalProperties: false
       properties:

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cocina-models
 version: !ruby/object:Gem::Version
-  version: 0.78.0
+  version: 0.81.0
 platform: ruby
 authors:
 - Justin Coyne
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-05-04 00:00:00.000000000 Z
+date: 2022-06-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport