feat: archivesWrapper -> relatedItem@type=series

local/archivesWrapper/subseries -> relatedItem@type=series,@displayLabel=Subseries
which contains a second series for the archivesWrapper/series text
no identifiers yet. Ref #108

Author: phette23

collection-export/strict-mods.js

@@ -911,6 +911,103 @@ export function removeEmptyClassifications(doc) {
     return doc
 }
 
+/**
+ * Convert local/archivesWrapper (series/subseries) to nested relatedItem elements
+ * Maps archives metadata to MODS relatedItem structure:
+ * - subseries -> relatedItem type="series" displayLabel="subseries" containing
+ * - series -> nested relatedItem type="series" displayLabel="series"
+ * If only series exists, creates single relatedItem with displayLabel="series"
+ * 
+ * @param {Document} doc - XML DOM document
+ * @returns {Document} Modified document
+ */
+export function convertArchivesWrapper(doc) {
+    if (!doc) {
+        return doc
+    }
+
+    const archivesWrappers = safeSelect('//local/archivesWrapper', doc)
+    const select = xpath.useNamespaces({})
+
+    for (let wrapper of archivesWrappers) {
+        const local = wrapper.parentNode // archivesWrapper's parent is local
+        
+        // Find the mods element - local is a sibling of mods, not a child
+        // Navigate to parent (xml) then find mods child
+        const modsElements = safeSelect('//mods', doc)
+        if (modsElements.length === 0) {
+            continue
+        }
+        const mods = modsElements[0]
+        
+        // Get series and subseries BEFORE removing wrapper
+        const seriesEl = select('series', wrapper)[0]
+        const subseriesEl = select('subseries', wrapper)[0]
+        
+        const seriesText = seriesEl?.textContent?.trim() || ''
+        const subseriesText = subseriesEl?.textContent?.trim() || ''
+        
+        // Remove the archivesWrapper from local
+        local.removeChild(wrapper)
+        
+        // Skip if both are empty
+        if (!seriesText && !subseriesText) {
+            continue
+        }
+        
+        if (subseriesText && seriesText) {
+            // Both exist: create nested structure
+            // Outer relatedItem is subseries
+            const outerRelatedItem = doc.createElement('relatedItem')
+            outerRelatedItem.setAttribute('type', 'series')
+            outerRelatedItem.setAttribute('displayLabel', 'subseries')
+            
+            const outerTitleInfo = doc.createElement('titleInfo')
+            const outerTitle = doc.createElement('title')
+            outerTitle.textContent = subseriesText
+            outerTitleInfo.appendChild(outerTitle)
+            outerRelatedItem.appendChild(outerTitleInfo)
+            
+            // Inner relatedItem is series
+            const innerRelatedItem = doc.createElement('relatedItem')
+            innerRelatedItem.setAttribute('type', 'series')
+            innerRelatedItem.setAttribute('displayLabel', 'series')
+            
+            const innerTitleInfo = doc.createElement('titleInfo')
+            const innerTitle = doc.createElement('title')
+            innerTitle.textContent = seriesText
+            innerTitleInfo.appendChild(innerTitle)
+            innerRelatedItem.appendChild(innerTitleInfo)
+            
+            outerRelatedItem.appendChild(innerRelatedItem)
+            mods.appendChild(outerRelatedItem)
+            
+        } else if (seriesText) {
+            // Only series: create single relatedItem
+            const relatedItem = doc.createElement('relatedItem')
+            relatedItem.setAttribute('type', 'series')
+            relatedItem.setAttribute('displayLabel', 'series')
+            
+            const titleInfo = doc.createElement('titleInfo')
+            const title = doc.createElement('title')
+            title.textContent = seriesText
+            titleInfo.appendChild(title)
+            relatedItem.appendChild(titleInfo)
+            
+            mods.appendChild(relatedItem)
+        }
+        // Note: subseries without series shouldn't happen based on data analysis
+    }
+
+    // Remove empty local elements (if all archivesWrappers were removed)
+    const localElements = safeSelect('//local[not(*)]', doc)
+    for (let local of localElements) {
+        local.parentNode.removeChild(local)
+    }
+
+    return doc
+}
+
 /**
  * Fix nonstandard mods/name/subNameWrapper elements, several operations:
  * - affiliation -> name/affiliation ("CCAC")
@@ -1065,6 +1162,9 @@ export function toStrictMODS(xmlString) {
     // Fix nonstandard subNameWrapper elements under mods/name
     convertSubNameWrapper(doc)
 
+    // Convert local/archivesWrapper (series/subseries) to nested relatedItem structure
+    convertArchivesWrapper(doc)
+
     // Remove usage="secondary" attribute from name elements
     removeBadNameUsageAttrs(doc)
 

collection-export/strict-mods.test.js

@@ -3,7 +3,7 @@ import { describe, it } from 'mocha'
 import xpath from 'xpath'
 import { DOMParser as xmldom } from '@xmldom/xmldom'
 
-import { removeBadNameUsageAttrs, unwrapSimpleElement, fixTitleAttributes, unwrapDateCreated, unwrapDateOther, fixDateCreatedKeyDate, fixDateCreatedQualifer, renameElement, removeElement, removeEmptyElements, removeAttribute, convertAuthorityElement, moveClassificationToSubject, wrapElement, wrapTextWithChild, moveAndRenameElement, convertNamePartDate, convertSubNameWrapper, wrapCopyInformation, wrapLocationTextContent, removeEmptyClassifications, convertSpeakerReleaseDetail, toStrictMODS } from './strict-mods.js'
+import { removeBadNameUsageAttrs, unwrapSimpleElement, fixTitleAttributes, unwrapDateCreated, unwrapDateOther, fixDateCreatedKeyDate, fixDateCreatedQualifer, renameElement, removeElement, removeEmptyElements, removeAttribute, convertAuthorityElement, moveClassificationToSubject, wrapElement, wrapTextWithChild, moveAndRenameElement, convertNamePartDate, convertSubNameWrapper, wrapCopyInformation, wrapLocationTextContent, removeEmptyClassifications, convertSpeakerReleaseDetail, convertArchivesWrapper, toStrictMODS } from './strict-mods.js'
 import { hasDirectTextContent } from './strict-mods-helpers.js'
 
 // Test fixtures
@@ -779,6 +779,100 @@ const fixtures = {
         expected: `<xml><mods>
             <titleInfo><title>Test</title></titleInfo>
         </mods></xml>`
+    },
+
+    archivesWrapperBoth: {
+        input: `<xml><mods>
+            <titleInfo><title>Provost's Office Emails</title></titleInfo>
+            <local>
+                <archivesWrapper>
+                    <series>I. Administrative Materials</series>
+                    <subseries>7. General Admin Files</subseries>
+                    <seriesStaging>I. Administrative Materials\\7. General Admin Files</seriesStaging>
+                </archivesWrapper>
+            </local>
+        </mods></xml>`,
+        expected: `<xml><mods>
+            <titleInfo><title>Provost's Office Emails</title></titleInfo>
+            <relatedItem type="series" displayLabel="subseries">
+                <titleInfo>
+                    <title>7. General Admin Files</title>
+                </titleInfo>
+                <relatedItem type="series" displayLabel="series">
+                    <titleInfo>
+                        <title>I. Administrative Materials</title>
+                    </titleInfo>
+                </relatedItem>
+            </relatedItem>
+        </mods></xml>`
+    },
+
+    archivesWrapperSeriesOnly: {
+        input: `<xml><mods>
+            <titleInfo><title>College Newsletter</title></titleInfo>
+            <local>
+                <archivesWrapper>
+                    <series>II. Publications</series>
+                    <seriesStaging>II. Publications</seriesStaging>
+                </archivesWrapper>
+            </local>
+        </mods></xml>`,
+        expected: `<xml><mods>
+            <titleInfo><title>College Newsletter</title></titleInfo>
+            <relatedItem type="series" displayLabel="series">
+                <titleInfo>
+                    <title>II. Publications</title>
+                </titleInfo>
+            </relatedItem>
+        </mods></xml>`
+    },
+
+    archivesWrapperEmpty: {
+        input: `<xml><mods>
+            <titleInfo><title>Test Item</title></titleInfo>
+            <local>
+                <archivesWrapper>
+                    <series></series>
+                    <seriesStaging></seriesStaging>
+                </archivesWrapper>
+            </local>
+        </mods></xml>`,
+        expected: `<xml><mods>
+            <titleInfo><title>Test Item</title></titleInfo>
+        </mods></xml>`
+    },
+
+    archivesWrapperMultiple: {
+        input: `<xml><mods>
+            <titleInfo><title>Multiple Archives</title></titleInfo>
+            <local>
+                <archivesWrapper>
+                    <series>I. First Series</series>
+                    <subseries>A. First Subseries</subseries>
+                </archivesWrapper>
+                <archivesWrapper>
+                    <series>II. Second Series</series>
+                </archivesWrapper>
+            </local>
+        </mods></xml>`,
+        expected: `<xml><mods>
+            <titleInfo><title>Multiple Archives</title></titleInfo>
+            <relatedItem type="series" displayLabel="subseries">
+                <titleInfo>
+                    <title>A. First Subseries</title>
+                </titleInfo>
+                <relatedItem type="series" displayLabel="series">
+                    <titleInfo>
+                        <title>I. First Series</title>
+                    </titleInfo>
+                </relatedItem>
+            </relatedItem>
+            <relatedItem type="series" displayLabel="series">
+                <titleInfo>
+                    <title>II. Second Series</title>
+                </titleInfo>
+            </relatedItem>
+        </mods></xml>`
     }
 }
 
@@ -3934,6 +4028,84 @@ describe('Strict MODS Conversion', () => {
         })
     })
 
+    describe('convertArchivesWrapper', () => {
+        it('should convert both series and subseries to nested relatedItem', () => {
+            const parser = new xmldom()
+            const doc = parser.parseFromString(fixtures.archivesWrapperBoth.input, 'text/xml')
+
+            convertArchivesWrapper(doc)
+
+            const result = normalizeXML(doc.toString())
+            const expected = normalizeXML(fixtures.archivesWrapperBoth.expected)
+
+            assert.strictEqual(result, expected)
+        })
+
+        it('should convert series only to single relatedItem', () => {
+            const parser = new xmldom()
+            const doc = parser.parseFromString(fixtures.archivesWrapperSeriesOnly.input, 'text/xml')
+
+            convertArchivesWrapper(doc)
+
+            const result = normalizeXML(doc.toString())
+            const expected = normalizeXML(fixtures.archivesWrapperSeriesOnly.expected)
+
+            assert.strictEqual(result, expected)
+        })
+
+        it('should remove empty archivesWrapper', () => {
+            const parser = new xmldom()
+            const doc = parser.parseFromString(fixtures.archivesWrapperEmpty.input, 'text/xml')
+
+            convertArchivesWrapper(doc)
+
+            const result = normalizeXML(doc.toString())
+            const expected = normalizeXML(fixtures.archivesWrapperEmpty.expected)
+
+            assert.strictEqual(result, expected)
+        })
+
+        it('should handle multiple archivesWrapper elements', () => {
+            const parser = new xmldom()
+            const doc = parser.parseFromString(fixtures.archivesWrapperMultiple.input, 'text/xml')
+
+            convertArchivesWrapper(doc)
+
+            const result = normalizeXML(doc.toString())
+            const expected = normalizeXML(fixtures.archivesWrapperMultiple.expected)
+
+            assert.strictEqual(result, expected)
+        })
+
+        it('should handle null document gracefully', () => {
+            const result = convertArchivesWrapper(null)
+
+            assert.strictEqual(result, null)
+        })
+
+        it('should verify nested structure is created correctly', () => {
+            const parser = new xmldom()
+            const doc = parser.parseFromString(fixtures.archivesWrapperBoth.input, 'text/xml')
+
+            convertArchivesWrapper(doc)
+
+            const select = xpath.useNamespaces({})
+            const outerRelated = select('//relatedItem[@displayLabel="subseries"]', doc)
+            const innerRelated = select('//relatedItem[@displayLabel="series"]', doc)
+            
+            assert.strictEqual(outerRelated.length, 1, 'Should have one subseries relatedItem')
+            assert.strictEqual(innerRelated.length, 1, 'Should have one series relatedItem')
+            assert.strictEqual(outerRelated[0].getAttribute('type'), 'series')
+            assert.strictEqual(innerRelated[0].getAttribute('type'), 'series')
+            
+            // Verify titles
+            const subseriesTitle = select('titleInfo/title', outerRelated[0])[0]
+            const seriesTitle = select('titleInfo/title', innerRelated[0])[0]
+            assert.strictEqual(subseriesTitle.textContent, '7. General Admin Files')
+            assert.strictEqual(seriesTitle.textContent, 'I. Administrative Materials')
+        })
+    })
+
     describe('Helper functions', () => {
         describe('hasDirectTextContent', () => {
             it('should return true for element with direct text content', () => {