@manuscripts/transform 4.3.22 → 4.3.23

package/README.md

@@ -4,7 +4,7 @@ ProseMirror transformer for Manuscripts applications.
 
 It provides a way to import/export [Manuscript JSON Schema](https://gitlab.com/mpapp-public/manuscripts-json-schema) formatted data from and to other formats such as (JATS XML, STS XML, HTML, [ProseMirror Model](https://prosemirror.net/docs/guide/#doc))
 
-# Components 
+# Components
 
 **Decoder**: converts Manuscript JSON Schema to ProseMirror Model.
 
@@ -24,7 +24,7 @@ It provides a way to import/export [Manuscript JSON Schema](https://gitlab.com/m
 
 ### Manuscript JSON Schema to ProseMirror Model
 
-```typescript
+```typescript 
 import { Decoder, ContainedModel } from '@manuscripts/manuscript-transform'
 
 

package/dist/cjs/jats/__tests__/jats-exporter.test.js

@@ -31,17 +31,16 @@ describe('JATS exporter', () => {
     test('export latest version', async () => {
         const transformer = new jats_exporter_1.JATSExporter();
         const input = await (0, files_1.readAndParseFixture)('jats-example-full.xml');
-        const { node, journal } = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
+        const node = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
         const result = await transformer.serializeToJATS(node, {
             csl: citations_1.DEFAULT_CSL_OPTIONS,
-            journal,
         });
         expect(result).toMatchSnapshot('jats-export');
     });
     test('export with & and < in bibliography metadata', async () => {
         const transformer = new jats_exporter_1.JATSExporter();
         const input = await (0, files_1.readAndParseFixture)('jats-example-full.xml');
-        const { node, journal } = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
+        const node = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
         let biblio = null;
         node.descendants((n) => {
             if ((0, schema_1.isBibliographyItemNode)(n)) {
@@ -51,17 +50,15 @@ describe('JATS exporter', () => {
         biblio.attrs.title += ' & Sons 55 < 135';
         const result = await transformer.serializeToJATS(node, {
             csl: citations_1.DEFAULT_CSL_OPTIONS,
-            journal,
         });
         expect(result).toMatchSnapshot('jats-export-with-xml-unsafe-in-biblios');
     });
     test('export v1.1', async () => {
         const transformer = new jats_exporter_1.JATSExporter();
         const input = await (0, files_1.readAndParseFixture)('jats-example-full.xml');
-        const { node, journal } = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
+        const node = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
         const result = await transformer.serializeToJATS(node, {
             csl: citations_1.DEFAULT_CSL_OPTIONS,
-            journal,
             version: '1.1',
         });
         expect(result).toMatchSnapshot('jats-export-1.1');
@@ -69,11 +66,10 @@ describe('JATS exporter', () => {
     test('export unknown version', async () => {
         const transformer = new jats_exporter_1.JATSExporter();
         const input = await (0, files_1.readAndParseFixture)('jats-example-full.xml');
-        const { node, journal } = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
+        const node = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
         await expect(async () => {
             await transformer.serializeToJATS(node, {
                 csl: citations_1.DEFAULT_CSL_OPTIONS,
-                journal,
                 version: '1.0',
             });
         }).rejects.toThrow(Error);
@@ -81,10 +77,9 @@ describe('JATS exporter', () => {
     test('export table-wrap-foot', async () => {
         const transformer = new jats_exporter_1.JATSExporter();
         const input = await (0, files_1.readAndParseFixture)('jats-tables-example.xml');
-        const { node, journal } = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
+        const node = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
         const xml = await transformer.serializeToJATS(node, {
             csl: citations_1.DEFAULT_CSL_OPTIONS,
-            journal,
         });
         const resultDoc = parseXMLWithDTD(xml);
         const tableWrapFoot = resultDoc.get('//table-wrap/table-wrap-foot');
@@ -92,36 +87,12 @@ describe('JATS exporter', () => {
         expect(paragraph).not.toBeUndefined();
         expect(tableWrapFoot).not.toBeUndefined();
     });
-    test('add journal ID', async () => {
-        const transformer = new jats_exporter_1.JATSExporter();
-        const input = await (0, files_1.readAndParseFixture)('jats-example-full.xml');
-        const { node, journal } = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
-        const xml = await transformer.serializeToJATS(node, {
-            csl: citations_1.DEFAULT_CSL_OPTIONS,
-            journal,
-        });
-        expect(xml).toMatchSnapshot('jats-export-submitted');
-    });
-    test('journal metadata', async () => {
-        const transformer = new jats_exporter_1.JATSExporter();
-        const input = await (0, files_1.readAndParseFixture)('jats-import.xml');
-        const { node, journal } = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
-        const xml = await transformer.serializeToJATS(node, {
-            csl: citations_1.DEFAULT_CSL_OPTIONS,
-            journal,
-        });
-        const output = parseXMLWithDTD(xml);
-        expect(output.errors).toHaveLength(0);
-        expect(output.get('//journal-title').text()).toBe('Brain and Behavior');
-        expect(output.get('//issn').text()).toBe('2162-3279');
-    });
     test('DTD validation', async () => {
         const transformer = new jats_exporter_1.JATSExporter();
         const input = await (0, files_1.readAndParseFixture)('jats-import.xml');
-        const { node, journal } = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
+        const node = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
         const xml = await transformer.serializeToJATS(node, {
             csl: citations_1.DEFAULT_CSL_OPTIONS,
-            journal,
         });
         const { errors } = parseXMLWithDTD(xml);
         expect(errors).toHaveLength(0);
@@ -129,10 +100,9 @@ describe('JATS exporter', () => {
     test('DTD validation: article with title markup and citations', async () => {
         const transformer = new jats_exporter_1.JATSExporter();
         const input = await (0, files_1.readAndParseFixture)('jats-document.xml');
-        const { node, journal } = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
+        const node = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
         const xml = await transformer.serializeToJATS(node, {
             csl: citations_1.DEFAULT_CSL_OPTIONS,
-            journal,
         });
         const { errors } = parseXMLWithDTD(xml);
         expect(errors).toHaveLength(0);
@@ -140,10 +110,9 @@ describe('JATS exporter', () => {
     test('Export link', async () => {
         const transformer = new jats_exporter_1.JATSExporter();
         const input = await (0, files_1.readAndParseFixture)('jats-import.xml');
-        const { node, journal } = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
+        const node = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
         const xml = await transformer.serializeToJATS(node, {
             csl: citations_1.DEFAULT_CSL_OPTIONS,
-            journal,
         });
         const output = parseXMLWithDTD(xml);
         const link = output.get('//ext-link[@ext-link-type="uri"]');
@@ -159,10 +128,9 @@ describe('JATS exporter', () => {
     test('DTD validation for MathML representation', async () => {
         const transformer = new jats_exporter_1.JATSExporter();
         const input = await (0, files_1.readAndParseFixture)('jats-example-doc.xml');
-        const { node, journal } = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
+        const node = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
         const xml = await transformer.serializeToJATS(node, {
             csl: citations_1.DEFAULT_CSL_OPTIONS,
-            journal,
         });
         const { errors } = parseXMLWithDTD(xml);
         expect(errors).toHaveLength(0);
@@ -170,10 +138,9 @@ describe('JATS exporter', () => {
     test('export with supplement', async () => {
         const transformer = new jats_exporter_1.JATSExporter();
         const input = await (0, files_1.readAndParseFixture)('jats-import.xml');
-        const { node, journal } = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
+        const node = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
         const xml = await transformer.serializeToJATS(node, {
             csl: citations_1.DEFAULT_CSL_OPTIONS,
-            journal,
         });
         const resultDoc = parseXMLWithDTD(xml);
         const supplementaryMaterial = resultDoc.get('//supplementary-material');
@@ -192,10 +159,9 @@ describe('JATS exporter', () => {
     test('export footnotes', async () => {
         const transformer = new jats_exporter_1.JATSExporter();
         const input = await (0, files_1.readAndParseFixture)('jats-fn-group.xml');
-        const { node, journal } = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
+        const node = (0, parse_jats_article_1.parseJATSArticle)(input, section_categories_1.sectionCategories);
         const xml = await transformer.serializeToJATS(node, {
             csl: citations_1.DEFAULT_CSL_OPTIONS,
-            journal,
         });
         const footnoteCategories = [
             'con',

package/dist/cjs/jats/__tests__/jats-importer.test.js

@@ -24,7 +24,7 @@ describe('JATS importer', () => {
     describe('title node', () => {
         it('should have title node with content if title element exists', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const titleNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.title);
             expect(titleNode).toBeDefined();
             (0, utils_1.updateNodeID)(titleNode);
@@ -34,7 +34,7 @@ describe('JATS importer', () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
             const titleEl = jats.querySelector('article-meta > title-group > article-title');
             titleEl?.remove();
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const titleNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.title);
             (0, utils_1.updateNodeID)(titleNode);
             expect(titleNode).toMatchSnapshot();
@@ -43,39 +43,17 @@ describe('JATS importer', () => {
     describe('contributors node', () => {
         it('should have contributors node with content if contributors element exists', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
-            const contributorsNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.contributors);
-            expect(node).toBeDefined();
-            expect(contributorsNode).toBeDefined();
-            expect((0, utils_1.findNodesByType)(contributorsNode, schema_1.schema.nodes.contributor)).toHaveLength(2);
-        });
-        it('should correctly parse contributor nodes', async () => {
-            const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
-            const contributorsEl = jats.querySelector('article-meta > contrib-group');
-            if (!contributorsEl) {
-                throw new Error('Contributors element not found');
-            }
-            const contributors = contributorsEl.querySelectorAll('contrib[contrib-type="author"]');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            (0, utils_1.changeIDs)(node);
             const contributorNodes = (0, utils_1.findNodesByType)(node, schema_1.schema.nodes.contributor);
-            expect(contributorNodes).toHaveLength(contributors.length);
-            contributorNodes.forEach((node) => {
-                (0, utils_1.updateNodeID)(node);
-                node.attrs.affiliations = node.attrs.affiliations.map(() => 'MPAffiliation:test');
-                node.attrs.footnote = node.attrs.footnote.map((footnote) => {
-                    return { ...footnote, noteID: 'MPFootnote:test' };
-                });
-                node.attrs.corresp = node.attrs.corresp.map((corresp) => {
-                    return { ...corresp, correspID: 'MPCorrespondance:test' };
-                });
-            });
+            expect(contributorNodes).toHaveLength(2);
             expect(contributorNodes).toMatchSnapshot();
         });
         it('should not have contributors node if contributors element does not exist', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
-            const contributorsEl = jats.querySelector('article-meta > contrib-group');
-            contributorsEl?.remove();
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const $contribGroup = jats.querySelector('article-meta > contrib-group');
+            $contribGroup.remove();
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const contributorsNode = (0, utils_1.findNodesByType)(node, schema_1.schema.nodes.contributors);
             const contributorNodes = (0, utils_1.findNodesByType)(node, schema_1.schema.nodes.contributor);
             expect(contributorNodes).toHaveLength(0);
@@ -85,7 +63,7 @@ describe('JATS importer', () => {
     describe('affiliations', () => {
         it('should correctly parse affiliation nodes', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const affiliationNodes = (0, utils_1.findNodesByType)(node, schema_1.schema.nodes.affiliation);
             affiliationNodes.forEach(utils_1.updateNodeID);
             expect(affiliationNodes).toMatchSnapshot();
@@ -96,7 +74,7 @@ describe('JATS importer', () => {
             affiliationsElements.forEach((aff) => {
                 aff.remove();
             });
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const affiliationsNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.affiliations);
             const affiliationNodes = (0, utils_1.findNodesByType)(node, schema_1.schema.nodes.affiliation);
             expect(affiliationNodes).toHaveLength(0);
@@ -106,7 +84,7 @@ describe('JATS importer', () => {
     describe('author-notes', () => {
         it('should have author notes node with content if author notes element exists', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const authorNotesNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.author_notes);
             expect(authorNotesNode).toBeDefined();
             expect((0, utils_1.findNodesByType)(authorNotesNode, schema_1.schema.nodes.footnote)).toHaveLength(2);
@@ -118,7 +96,7 @@ describe('JATS importer', () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
             const authorNotesEl = jats.querySelector('article-meta > author-notes');
             authorNotesEl?.remove();
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const authorNotesNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.authorNotes);
             expect(authorNotesNode).toBeUndefined();
         });
@@ -126,14 +104,14 @@ describe('JATS importer', () => {
     describe('awards', () => {
         it('should have awards node if awards element exists', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const awardsNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.awards);
             expect(awardsNode).toBeDefined();
             expect((0, utils_1.findNodesByType)(awardsNode, schema_1.schema.nodes.award).length).toBeGreaterThan(0);
         });
         it('should correctly parse award nodes', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const awardsNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.awards);
             (0, utils_1.changeIDs)(awardsNode);
             expect(awardsNode).toMatchSnapshot();
@@ -142,7 +120,7 @@ describe('JATS importer', () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
             const awardsEl = jats.querySelector('article-meta > funding-group');
             awardsEl?.remove();
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const awardsNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.awards);
             expect(awardsNode).toBeUndefined();
         });
@@ -150,7 +128,7 @@ describe('JATS importer', () => {
     describe('keywords', () => {
         it('should have keywords node with content if keywords element exists', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const keywords = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.keywords);
             expect(keywords).toBeDefined();
             const keywordsNodes = (0, utils_1.findNodesByType)(keywords, schema_1.schema.nodes.keyword);
@@ -164,7 +142,7 @@ describe('JATS importer', () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
             const keywordGroup = jats.querySelector('kwd-group');
             keywordGroup?.remove();
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const keywords = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.keywords);
             expect(keywords).toBeUndefined();
         });
@@ -172,7 +150,7 @@ describe('JATS importer', () => {
     describe('supplements', () => {
         it('should have supplements node with content if supplementary-material elements exist', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const supplementsNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.supplement);
             (0, utils_1.changeIDs)(supplementsNode);
             expect(supplementsNode).toMatchSnapshot();
@@ -183,7 +161,7 @@ describe('JATS importer', () => {
             supplementryMaterial.forEach((supplement) => {
                 supplement.remove();
             });
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const supplementsNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.supplements);
             expect(supplementsNode).toBeUndefined();
         });
@@ -191,7 +169,7 @@ describe('JATS importer', () => {
     describe('comments', () => {
         it('should parse keyword comment', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-comments.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const group = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.keyword_group);
             const markers = (0, utils_1.findNodesByType)(group, schema_1.schema.nodes.highlight_marker);
             expect(markers.length).toBe(0);
@@ -202,7 +180,7 @@ describe('JATS importer', () => {
         });
         it('should parse abstract comment', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-comments.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const abstracts = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.abstracts);
             const paragraph = (0, utils_1.findNodeByType)(abstracts, schema_1.schema.nodes.paragraph);
             const marker = (0, utils_1.findNodeByType)(paragraph, schema_1.schema.nodes.highlight_marker);
@@ -215,7 +193,7 @@ describe('JATS importer', () => {
         });
         it('should parse body comment', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-comments.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const body = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.body);
             const paragraph = (0, utils_1.findNodeByType)(body, schema_1.schema.nodes.paragraph);
             const marker = (0, utils_1.findNodeByType)(paragraph, schema_1.schema.nodes.highlight_marker);
@@ -228,7 +206,7 @@ describe('JATS importer', () => {
         });
         it('should parse back comment', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-comments.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const back = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.backmatter);
             const paragraph = (0, utils_1.findNodeByType)(back, schema_1.schema.nodes.paragraph);
             const marker = (0, utils_1.findNodeByType)(paragraph, schema_1.schema.nodes.highlight_marker);
@@ -241,7 +219,7 @@ describe('JATS importer', () => {
         });
         it('should parse ref-list comment', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-comments.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const element = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.bibliography_element);
             const markers = (0, utils_1.findNodesByType)(element, schema_1.schema.nodes.highlight_marker);
             expect(markers.length).toBe(0);
@@ -255,14 +233,14 @@ describe('JATS importer', () => {
     describe('attachments', () => {
         it('should have attachment node if attachment exists', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const attachmentsNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.attachments);
             expect(attachmentsNode).toBeDefined();
             expect((0, utils_1.findNodesByType)(attachmentsNode, schema_1.schema.nodes.attachment).length).toBeGreaterThan(0);
         });
         it('should correctly parse self-uri element', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const attachmentNodes = (0, utils_1.findNodesByType)(node, schema_1.schema.nodes.attachment);
             (0, utils_1.changeIDs)(attachmentNodes[0]);
             expect(attachmentNodes[0].attrs.type).toBe('document');
@@ -270,7 +248,7 @@ describe('JATS importer', () => {
         });
         it('should correctly parse multiple self-uri element', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const attachmentNodes = (0, utils_1.findNodesByType)(node, schema_1.schema.nodes.attachment);
             expect(attachmentNodes.length).toBe(2);
             (0, utils_1.changeIDs)(attachmentNodes[0]);
@@ -281,7 +259,7 @@ describe('JATS importer', () => {
         });
         it('should not have attachments node if no attachment element does not exist', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-example.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const attachmentsNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.attachments);
             expect(attachmentsNode).toBeUndefined();
         });
@@ -293,7 +271,7 @@ describe('JATS importer', () => {
             if (!abstractsEl) {
                 throw new Error('Abstract element not found');
             }
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const abstractsNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.abstracts);
             expect(abstractsNode).toBeDefined();
         });
@@ -301,7 +279,7 @@ describe('JATS importer', () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
             const abstractsEl = jats.querySelector('front > article-meta > abstract');
             abstractsEl?.remove();
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const abstractsNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.abstracts);
             expect(abstractsNode).toBeDefined();
         });
@@ -311,7 +289,7 @@ describe('JATS importer', () => {
             if (!abstractEl) {
                 throw new Error('Abstract element not found');
             }
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const abstractsNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.abstracts);
             const sections = (0, utils_1.findNodesByType)(abstractsNode, schema_1.schema.nodes.section);
             expect(sections).toHaveLength(6);
@@ -322,7 +300,7 @@ describe('JATS importer', () => {
             if (!abstractEl) {
                 throw new Error('Abstract element not found');
             }
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const abstractsNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.abstracts);
             const sections = (0, utils_1.findNodesByType)(abstractsNode, schema_1.schema.nodes.section);
             const firstSection = sections[0];
@@ -337,13 +315,13 @@ describe('JATS importer', () => {
             if (!bodyEl) {
                 throw new Error('Body element not found');
             }
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const bodyNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.body);
             expect(bodyNode).toBeDefined();
         });
         it('should have the correct number of sections', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const bodyNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.body);
             const sections = (0, utils_1.findNodesByType)(bodyNode, schema_1.schema.nodes.section, false);
             expect(sections).toHaveLength(5);
@@ -352,19 +330,19 @@ describe('JATS importer', () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
             const bodyEl = jats.querySelector('body');
             bodyEl?.remove();
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const bodyNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.body);
             expect(bodyNode).toBeDefined();
         });
         it('should create body element and append to article when body is missing', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-abstract-no-body.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const bodyNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.body);
             expect(bodyNode).toBeDefined();
         });
         it('should have abstracts node even when no body element exists in JATS', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-abstract-no-body.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const abstractNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.abstracts);
             expect(abstractNode).toBeDefined();
         });
@@ -376,7 +354,7 @@ describe('JATS importer', () => {
             if (!backEl) {
                 throw new Error('Back element not found');
             }
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const backNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.backmatter);
             expect(backNode).toBeDefined();
         });
@@ -386,7 +364,7 @@ describe('JATS importer', () => {
             if (!appGroup) {
                 throw new Error('App group element not found');
             }
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const backNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.backmatter);
             const appNode = (0, utils_1.findNodesByType)(backNode, schema_1.schema.nodes.section, false).filter((node) => node.attrs.category === 'appendices');
             expect(appNode).toHaveLength(1);
@@ -397,14 +375,14 @@ describe('JATS importer', () => {
             if (!backEl) {
                 throw new Error('Back element not found');
             }
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const backNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.backmatter);
             const availabilitySection = (0, utils_1.findNodesByType)(backNode, schema_1.schema.nodes.section, false).filter((node) => node.attrs.category === 'availability');
             expect(availabilitySection).toHaveLength(1);
         });
         it('should create sections for special footnotes', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-example-full.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const backNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.backmatter);
             const con = (0, utils_1.findNodesByType)(backNode, schema_1.schema.nodes.section, false).filter((node) => node.attrs.category === 'con');
             const financialDisclosure = (0, utils_1.findNodesByType)(backNode, schema_1.schema.nodes.section, false).filter((node) => node.attrs.category === 'financial-disclosure');
@@ -415,7 +393,7 @@ describe('JATS importer', () => {
         });
         it('should have an endnotes section if either an endnotes section exists or there are footnotes', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-example-full.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const backNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.backmatter);
             const endNotesSection = (0, utils_1.findNodesByType)(backNode, schema_1.schema.nodes.footnotes_section, false);
             expect(endNotesSection).toHaveLength(1);
@@ -426,7 +404,7 @@ describe('JATS importer', () => {
             if (!refList) {
                 throw new Error('Ref list element not found');
             }
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const backNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.backmatter);
             const refSection = (0, utils_1.findNodesByType)(backNode, schema_1.schema.nodes.bibliography_section, false);
             expect(refSection).toHaveLength(1);
@@ -437,7 +415,7 @@ describe('JATS importer', () => {
             if (!ack) {
                 throw new Error('Ack element not found');
             }
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const backNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.backmatter);
             const ackSection = (0, utils_1.findNodesByType)(backNode, schema_1.schema.nodes.section, false).filter((node) => node.attrs.category === 'acknowledgements');
             expect(ackSection).toHaveLength(1);
@@ -448,7 +426,7 @@ describe('JATS importer', () => {
             if (!refList) {
                 throw new Error('Ref list element not found');
             }
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const backNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.backmatter);
             const refSection = (0, utils_1.findNodeByType)(backNode, schema_1.schema.nodes.bibliography_section);
             const bibliographyItems = (0, utils_1.findNodesByType)(refSection, schema_1.schema.nodes.bibliography_item);
@@ -462,7 +440,7 @@ describe('JATS importer', () => {
         });
         it('should have the correct number of sections', async () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-example-full.xml');
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const backNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.backmatter);
             expect(backNode.childCount).toBe(8);
         });
@@ -470,43 +448,43 @@ describe('JATS importer', () => {
             const jats = await (0, files_1.readAndParseFixture)('jats-import.xml');
             const backEl = jats.querySelector('back');
             backEl?.remove();
-            const { node } = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
+            const node = (0, parse_jats_article_1.parseJATSArticle)(jats, section_categories_1.sectionCategories);
             const backNode = (0, utils_1.findNodeByType)(node, schema_1.schema.nodes.backmatter);
             expect(backNode).toBeDefined();
         });
     });
     test('parses JATS to Manuscripts document', async () => {
-        const { node } = await (0, utils_1.createNodeFromJATS)('jats-import.xml');
+        const node = await (0, utils_1.createNodeFromJATS)('jats-import.xml');
         (0, utils_1.changeIDs)(node);
         expect(node).toMatchSnapshot();
     });
     test('parses JATS AuthorQueries example to Manuscripts document', async () => {
-        const { node } = await (0, utils_1.createNodeFromJATS)('jats-document.xml');
+        const node = await (0, utils_1.createNodeFromJATS)('jats-document.xml');
         (0, utils_1.changeIDs)(node);
         expect(node).toMatchSnapshot();
     });
     test('parses full JATS example to Manuscripts document', async () => {
-        const { node } = await (0, utils_1.createNodeFromJATS)('jats-example-doc.xml');
+        const node = await (0, utils_1.createNodeFromJATS)('jats-example-doc.xml');
         (0, utils_1.changeIDs)(node);
         expect(node).toMatchSnapshot();
     });
     test("parses JATS article without references and doesn't create empty references section", async () => {
-        const { node } = await (0, utils_1.createNodeFromJATS)('jats-import-no-refs.xml');
+        const node = await (0, utils_1.createNodeFromJATS)('jats-import-no-refs.xml');
         (0, utils_1.changeIDs)(node);
         expect(node).toMatchSnapshot();
     });
     test('parses JATS article to Manuscripts document', async () => {
-        const { node } = await (0, utils_1.createNodeFromJATS)('jats-example.xml');
+        const node = await (0, utils_1.createNodeFromJATS)('jats-example.xml');
         (0, utils_1.changeIDs)(node);
         expect(node).toMatchSnapshot();
     });
     test('parses JATS article with tables and table footnotes', async () => {
-        const { node } = await (0, utils_1.createNodeFromJATS)('jats-tables-example.xml');
+        const node = await (0, utils_1.createNodeFromJATS)('jats-tables-example.xml');
         (0, utils_1.changeIDs)(node);
         expect(node).toMatchSnapshot();
     });
     test('parses JATS example full', async () => {
-        const { node } = await (0, utils_1.createNodeFromJATS)('jats-example-full.xml');
+        const node = await (0, utils_1.createNodeFromJATS)('jats-example-full.xml');
         (0, utils_1.changeIDs)(node);
         expect(node).toMatchSnapshot();
     });

package/dist/cjs/jats/__tests__/jats-roundtrip.test.js

@@ -29,10 +29,9 @@ const parseXMLWithDTD = (data) => (0, libxmljs2_1.parseXml)(data, {
 const roundtrip = async (filename) => {
     const input = await (0, files_1.readFixture)(filename);
     const doc = new DOMParser().parseFromString(input, 'application/xml');
-    const { node, journal } = (0, parse_jats_article_1.parseJATSArticle)(doc, section_categories_1.sectionCategories);
+    const node = (0, parse_jats_article_1.parseJATSArticle)(doc, section_categories_1.sectionCategories);
     const exporter = new jats_exporter_1.JATSExporter();
     return await exporter.serializeToJATS(node, {
-        journal: journal,
         csl: citations_1.DEFAULT_CSL_OPTIONS,
     });
 };