@antora/assembler 1.0.0-rc.6 → 1.0.0-rc.8

package/lib/produce-assembly-file.js

@@ -11,20 +11,22 @@ const {
   rewriteStyleAttribute,
 } = require('./util/rewriter')
 const sanitize = require('./util/sanitize')
+const toHash = require('./util/to-hash')
 const unconvertInlineAsciiDoc = require('./util/unconvert-inline-asciidoc')
 
 const ATTR_ENTRY_RX = /^:(!?[\p{Alpha}0-9_][^:]*):(?: |$)/u
 const BUILT_IN_NAMED_ENTITIES = { amp: '&', apos: "'", gt: '>', lt: '<', nbsp: ' ', quot: '"' }
 const CHAR_REF_RX = /&(?:([a-z][a-z]+\d{0,2})|#(?:(\d{2,6})|x([a-z\d]{2,5})));/g
 const DISCARD_ATTRIBUTE_NAMES = [
-  'doctype',
-  'leveloffset',
-  'preface-title',
   'assembly-doctype',
   'assembly-header-attributes',
   'assembly-navtitle',
   'assembly-slug',
   'assembly-style',
+  'doctype',
+  'leveloffset',
+  'page-aliases',
+  'preface-title',
   'underscore',
 ]
 const { NAMED_ID_ATTR_RX } = require('./util/rx')
@@ -42,33 +44,37 @@ function produceAssemblyFile (
   const pagesByUrl = files.reduce((map, it) => (it.src.family === 'page' ? map.set(it.pub.url, it) : map), new Map())
   const pagesInOutline = selectPagesInOutline(outline, pagesByUrl)
   if (outline.urlType === 'internal' && !pagesByUrl.has(outline.pathname) && !outline.items?.length) return
+  const { name: component, version } = componentVersion
   const { rootLevel, xmlIds } = assemblyModel
+  const scratchDoc = loadAsciiDoc(
+    {
+      contents: Buffer.alloc(0),
+      src: { component, version, module: 'ROOT', family: 'page', relative: '_.adoc', path: '_.adoc' },
+    },
+    undefined,
+    asciidocConfig
+  )
   const idSeparators = {
     prefix:
       'assembler-idprefix' in asciidocConfig.attributes ? (asciidocConfig.attributes['assembler-idprefix'] ?? '') : '_',
     scope: xmlIds ? '---' : ':::',
     coordinate: xmlIds ? '----' : ':',
-    generateIdFromTitle: generateIdFromTitle.bind(
-      loadAsciiDoc(
-        { contents: Buffer.alloc(0), src: { family: 'page', relative: 'generate-id-from-title.adoc' } },
-        undefined,
-        asciidocConfig
-      )
-    ),
+    generateIdFromTitle: generateIdFromTitle.bind(scratchDoc),
   }
-  const { name: component, version } = componentVersion
   asciidocConfig = prepareAsciiDocConfig(
     contentCatalog,
     { component, version },
     pagesInOutline,
     asciidocConfig,
     assemblyModel,
-    idSeparators
+    idSeparators,
+    scratchDoc
   )
-  const buffer = mergeAsciiDoc(
+  const builder = buildAsciiDocHeader(componentVersion, outline.content, assemblyModel)
+  mergeAsciiDoc(
     loadAsciiDoc,
     contentCatalog,
-    buildAsciiDocHeader(componentVersion, outline.content, assemblyModel),
+    builder,
     componentVersion,
     outline,
     files,
@@ -78,24 +84,23 @@ function produceAssemblyFile (
     mutableAttributes,
     assemblyModel
   )
-  if (buffer[buffer.doctypeIdx] === ':doctype: article') buffer.splice(buffer.doctypeIdx, 1)
   const stem =
-    (buffer.slug ? sanitizeSlug(buffer.slug) : generateSlug(rootLevel === 0 ? 'index' : buffer.navtitle)) ||
+    (builder.slug ? sanitizeSlug(builder.slug) : generateSlug(rootLevel === 0 ? 'index' : builder.navtitle)) ||
     'export-' + (assemblyModel.stemSeq = (assemblyModel.exportSeq ?? 0) + 1)
   const downloadStem = [component, version, stem === 'index' ? '' : stem].filter((it) => it).join('-')
   const file = contentCatalog.createFile({
     asciidoc: asciidocConfig,
     assembler: { assembled: pagesInOutline.assembled, downloadStem, rootLevel },
-    contents: Buffer.from(buffer.join('\n') + '\n'),
+    contents: builder.serialize(),
     src: { component, version, componentVersion, module: 'ROOT', family: 'export', relative: stem + '.adoc' },
     pub: false,
   })
-  file.path = file.src.path = file.out.path // use out path as path so assets can be published to same hierarchy
+  file.path = file.out.path // use out path as path so assets can be published to same hierarchy
   delete file.out
   return file
 }
 
-function prepareAsciiDocConfig (contentCatalog, ctx, pagesInOutline, asciidocConfig, assemblyModel, idSeparators) {
+function prepareAsciiDocConfig (contentCatalog, ctx, pagesInOutline, asciidocConfig, assemblyModel, idSeparators, doc) {
   let attributesModified
   const configShared = asciidocConfig.$shared
   const sharedAttributes = asciidocConfig.attributes
@@ -116,7 +121,17 @@ function prepareAsciiDocConfig (contentCatalog, ctx, pagesInOutline, asciidocCon
     for (const [name, val] of Object.entries(sharedAttributes)) {
       if (!(val?.constructor === String && ~val.indexOf(':'))) continue
       if (~val.indexOf('xref:')) {
-        const newVal = rewriteXrefs(val, contentCatalog, assemblyModel, ctx, false, pagesInOutline, idSeparators)
+        const newVal = rewriteXrefs(
+          val,
+          contentCatalog,
+          assemblyModel,
+          ctx,
+          false,
+          pagesInOutline,
+          idSeparators,
+          null,
+          doc
+        )
         if (newVal !== val) (attributesModified ??= {})[name] = newVal
       }
     }
@@ -126,25 +141,44 @@ function prepareAsciiDocConfig (contentCatalog, ctx, pagesInOutline, asciidocCon
 }
 
 function buildAsciiDocHeader (componentVersion, navtitle, assemblyModel) {
+  const { name: componentName, version, displayVersion, title } = componentVersion
   const navtitlePlain = sanitize(navtitle)
   const navtitleAsciiDoc = unconvertInlineAsciiDoc(navtitle)
-  const doctitle = (componentVersion.title === navtitlePlain ? '' : componentVersion.title + ': ') + navtitleAsciiDoc
-  const version = componentVersion.version === 'master' ? '' : componentVersion.version
-  const displayVersion = componentVersion.displayVersion
-  const buffer = [
-    `= ${doctitle}`,
-    ...(version ? [`:revnumber: ${displayVersion}`] : []),
-    `:doctype: ${assemblyModel.doctype ?? 'book'}`,
-    ':underscore: _',
-    // Q: should we pass these via the CLI so they cannot be modified?
-    `:page-component-name: ${componentVersion.name}`,
-    `:page-component-version:${version ? ' ' + version : ''}`,
-    ':page-version: {page-component-version}',
-    `:page-component-display-version: ${displayVersion}`,
-    `:page-component-title: ${componentVersion.title}`,
-  ]
-  buffer.doctypeIdx = version ? 2 : 1
-  return Object.assign(buffer, { navtitle })
+  const doctitle = (title === navtitlePlain ? '' : title + ': ') + navtitleAsciiDoc
+  const versioned = !!(version && version !== 'master')
+  return {
+    header: [
+      `= ${doctitle}`,
+      ...(versioned ? [`:revnumber: ${displayVersion}`] : []),
+      `:doctype: ${(assemblyModel.doctype ??= 'book')}`,
+      ':underscore: _',
+      `:page-component-name: ${componentName}`,
+      `:page-component-version:${versioned ? ' ' + version : ''}`,
+      ':page-version: {page-component-version}',
+      `:page-component-display-version: ${displayVersion}`,
+      `:page-component-title: ${title}`,
+    ],
+    currentComponentVersion: componentVersion,
+    navtitle,
+    setAuthor (author) {
+      this.header.splice(1, 0, author)
+    },
+    setDoctitle (doctitle_) {
+      this.header[0] = `= ${doctitle_}`
+    },
+    setDoctype (doctype) {
+      this.header[this.header.findIndex((line) => line.startsWith(':doctype: '))] = `:doctype: ${doctype}`
+    },
+    setDocid (docid) {
+      this.header.unshift(`[#${docid}]`)
+    },
+    serialize () {
+      const normalizedHeader = this.header.filter((it) => it !== ':doctype: article')
+      return this.body
+        ? Buffer.from(normalizedHeader.join('\n') + '\n\n' + this.body.join('\n') + '\n')
+        : Buffer.from(normalizedHeader.join('\n') + '\n')
+    },
+  }
 }
 
 function selectPagesInOutline (outlineEntry, pagesByUrl, accum) {
@@ -165,7 +199,7 @@ function selectPagesInOutline (outlineEntry, pagesByUrl, accum) {
 function mergeAsciiDoc (
   loadAsciiDoc,
   contentCatalog,
-  buffer,
+  builder,
   componentVersion,
   outlineEntry,
   files,
@@ -174,15 +208,14 @@ function mergeAsciiDoc (
   asciidocConfig,
   mutableAttributes,
   assemblyModel,
-  lastComponentVersion = componentVersion,
   level = 0,
   supportsParts = false
 ) {
   const { items = [], roles = [], unresolved, urlType, url, pathname, hash } = outlineEntry
   // TODO: we could try to be smart about it and make sure the page with fragment is included at least once
-  if (roles.includes('site-only')) {
-    buffer.inBody ??= false
-    return buffer
+  if (roles.includes('site-only') || (urlType === 'external' && !URL.canParse(url))) {
+    builder.body ??= []
+    return builder
   }
   const assembled = pagesInOutline.assembled
   // FIXME: ideally, resource ID would be stored in navigation so we can look up the page more efficiently
@@ -191,18 +224,18 @@ function mergeAsciiDoc (
   let navtitlePlain = sanitize(navtitle)
   let navtitleAsciiDoc = unconvertInlineAsciiDoc(navtitle)
   const { filetype, linkReferenceStyle, pubRoot, sectionMergeStrategy, siteRoot, logger, rootLevel } = assemblyModel
-  const doctype = assemblyModel.doctype ?? 'book'
-  const atDocumentRoot = !buffer.inBody
+  const doctype = assemblyModel.doctype
+  const atDocumentRoot = !builder.body?.length
   let atBookRoot = atDocumentRoot && !level && doctype === 'book' && (supportsParts = true)
   const hasItems = items.length > 0
   if (page && !assembled.pages.has(page)) {
     const contents = page.src.contents
     if (contents == null) {
-      buffer.inBody ??= false
-      return buffer
+      builder.body ??= []
+      return builder
     }
     const pageAsAsciiDoc = new page.constructor(
-      Object.assign({}, page, { contents: trimAsciiDoc(contents), mediaType: page.src.mediaType })
+      Object.assign({}, page, { contents: Buffer.from(contents.toString().trimEnd()), mediaType: page.src.mediaType })
     )
     const doc = loadAsciiDoc(pageAsAsciiDoc, contentCatalog, asciidocConfig)
     doc.catalog.syntheticIds = {}
@@ -215,25 +248,25 @@ function mergeAsciiDoc (
           },
         })
       : doc.getLogger()
-    doc.source_header_attributes ??= doc.parent.$to_h()
+    doc.source_header_attributes ??= toHash()
     const isRootPage = atDocumentRoot && !level
     const { component, version, module: module_, relative, origin } = page.src
     const doctypeOverride = doc.getAttribute('assembly-doctype')
     if (doctypeOverride) {
-      if (doctypeOverride !== doctype) buffer[buffer.doctypeIdx] = `:doctype: ${doctypeOverride}`
+      if (doctypeOverride !== doctype) builder.setDoctype(doctypeOverride)
       if (doctypeOverride !== 'book') atBookRoot = supportsParts = false
     }
-    if (isRootPage && doc.hasAttribute('assembly-slug')) buffer.slug = doc.getAttribute('assembly-slug')
+    if (isRootPage && doc.hasAttribute('assembly-slug')) builder.slug = doc.getAttribute('assembly-slug')
     let hasAssemblyNavtitleAttr
     if ((hasAssemblyNavtitleAttr = !!doc.getAttribute('assembly-navtitle'))) {
       navtitleAsciiDoc = doc.getAttribute('assembly-navtitle')
       navtitlePlain = sanitize((navtitle = doc.$apply_reftext_subs(navtitleAsciiDoc)))
-      if (buffer.inBody == null) {
-        buffer.navtitle = navtitle
+      if (builder.body == null) {
+        builder.navtitle = navtitle
         if (rootLevel) {
-          const doctitle =
+          builder.setDoctitle(
             (componentVersion.title === navtitlePlain ? '' : componentVersion.title + ': ') + navtitleAsciiDoc
-          buffer[0] = `= ${doctitle}`
+          )
         }
       }
     }
@@ -241,7 +274,7 @@ function mergeAsciiDoc (
     const docname = doc.getAttribute('docname')
     const pageId = generateScopedId(page.src, componentVersion, idSeparators, filetype)
     const pageIdLeader = pageId + idSeparators.scope
-    let pageFragment = ''
+    let pageFragment = `#${pageId}`
     let pageRoles = []
     let pageStyle = doc.getAttribute('assembly-style', '')
     if (!pageStyle && atBookRoot && rootLevel === 0 && !doc.source_header_attributes['$key?']('assembly-style')) {
@@ -259,13 +292,10 @@ function mergeAsciiDoc (
     let nextSectionLevel = 1
     const lines = doc.getSourceLines()
     const ignoreLines = []
-    if (!buffer.inBody) {
-      buffer.inBody = true
-      buffer.endHeaderIdx = buffer.length - 1
-    }
-    buffer.push('')
+    const buffer = (builder.body ??= [])
+    if (buffer.length) buffer.push('')
     buffer.push(`:docname: ${docname}`)
-    if (component !== lastComponentVersion.name) {
+    if (component !== builder.currentComponentVersion.name) {
       const thisComponentVersion =
         component === componentVersion.name && version === componentVersion.version
           ? componentVersion
@@ -276,7 +306,7 @@ function mergeAsciiDoc (
         buffer.push(':page-version: {page-component-version}')
         buffer.push(`:page-component-display-version: ${thisComponentVersion.displayVersion}`)
         buffer.push(`:page-component-title: ${thisComponentVersion.title}`)
-        lastComponentVersion = thisComponentVersion
+        builder.currentComponentVersion = thisComponentVersion
       }
     }
     buffer.push(`:page-module: ${module_}`)
@@ -286,13 +316,10 @@ function mergeAsciiDoc (
     buffer.push(`:page-origin-refname: ${origin.branch || origin.tag}`)
     buffer.push(`:page-origin-reftype: ${origin.branch ? 'branch' : 'tag'}`)
     buffer.push(`:page-origin-refhash: ${origin.worktree ? '(worktree)' : origin.refhash}`)
-    let headerHasBlockAttrs
     if (doc.hasHeader()) {
       for (const entry of processDocumentHeader(doc, lines, ignoreLines, isRootPage)) {
         if (entry.type === 'author_line') {
-          buffer.splice(1, 0, entry.lines[0])
-          buffer.doctypeIdx += 1
-          buffer.endHeaderIdx += 1
+          builder.setAuthor(entry.lines[0])
         } else if (entry.type === 'attribute_entry') {
           const name = entry.name
           if (!entry.negated && !doc.isAttributeLocked(name)) {
@@ -300,7 +327,7 @@ function mergeAsciiDoc (
             if (
               name.endsWith('-image') &&
               (val = doc.getAttribute(name)) &&
-              (newVal = rewriteImageAttr(val, contentCatalog, assemblyModel, page.src, assembled.assets))
+              (newVal = rewriteImageAttr(val, contentCatalog, assemblyModel, page.src, assembled.assets)) != null
             ) {
               if (newVal !== val) entry.lines = [`:${name}: ${newVal}`]
             } else if (~(val = entry.lines.join('\n').substring(name.length + 3)).indexOf(':')) {
@@ -333,14 +360,9 @@ function mergeAsciiDoc (
               if (newVal !== val) entry.lines = `:${entry.name}: ${newVal}`.split('\n')
             }
           }
-          if (entry.promote) {
-            buffer.splice(buffer.endHeaderIdx + 1, 0, ...entry.lines)
-            buffer.endHeaderIdx += entry.lines.length
-          } else {
-            buffer.push(...entry.lines)
-          }
+          ;(entry.promote ? builder.header : buffer).push(...entry.lines)
         } else {
-          if (entry.type === 'attrlist') headerHasBlockAttrs = true
+          if (entry.type === 'attrlist') builder.headerHasBlockAttrs = true
           buffer.push(...entry.lines)
         }
       }
@@ -364,15 +386,15 @@ function mergeAsciiDoc (
         if (htitleOverride != null) {
           const keepSections = hasSections && sectionMergeStrategy !== 'discrete'
           if (htitleOverride === '') {
-            if (headerHasBlockAttrs || keepSections) {
+            if (builder.headerHasBlockAttrs || keepSections) {
               htitleAsciiDoc = '{empty}'
               notitle = true
             } else {
-              if (hasPrefaceTitleAttr) buffer.splice(++buffer.endHeaderIdx, 0, ':preface-title:')
+              if (hasPrefaceTitleAttr) builder.header.push(':preface-title:')
               htitleAsciiDoc = undefined
             }
-          } else if (hasPrefaceTitleAttr && !headerHasBlockAttrs && !keepSections) {
-            buffer.splice(++buffer.endHeaderIdx, 0, `:preface-title: ${unconvertInlineAsciiDoc(htitleOverride)}`)
+          } else if (hasPrefaceTitleAttr && !builder.headerHasBlockAttrs && !keepSections) {
+            builder.header.push(`:preface-title: ${unconvertInlineAsciiDoc(htitleOverride)}`)
             htitleAsciiDoc = undefined
           } else {
             htitleAsciiDoc = unconvertInlineAsciiDoc(htitleOverride)
@@ -383,10 +405,7 @@ function mergeAsciiDoc (
         if (atDocumentRoot) {
           // Q: should we use htitleAsciiDoc to generate ID if not {empty} instead of pageStyle
           pageFragment = `#_${idSeparators.coordinate}${pageIdLeader}${pageStyle}`
-          buffer.unshift(`[#${pageId}]`)
-          buffer.doctypeIdx += 1
-        } else {
-          pageFragment = `#${pageId}`
+          builder.setDocid(pageId)
         }
         heading = { title: htitleAsciiDoc, level: part ? 1 : 2 }
         if (notitle) heading.notitle = true
@@ -396,14 +415,12 @@ function mergeAsciiDoc (
       if (atDocumentRoot && rootLevel === 0 && navtitlePlain === componentVersion.title && !hasAssemblyNavtitleAttr) {
         level--
       } else {
-        pageFragment = `#${pageId}`
         if (part && level === 1) level--
         if ((heading = { title: navtitleAsciiDoc, level: level + 1 }).level > 6) {
           Object.assign(heading, { level: 6, style: `discrete.h${heading.level}` })
         }
       }
     } else if (atDocumentRoot && rootLevel === 0 && hasAssemblyNavtitleAttr) {
-      pageFragment = `#${pageId}`
       heading = { title: navtitleAsciiDoc, level: (nextSectionLevel = part ? 1 : 2) }
     } else if (atBookRoot) {
       nextSectionLevel = undefined
@@ -414,16 +431,17 @@ function mergeAsciiDoc (
       buffer.push(`[${heading.style ?? pageStyle}${pageFragment}${rolesAttr}${notitleAttrs}]`)
       buffer.push(`${'='.repeat(heading.level)} ${heading.title}`)
     } else if (atDocumentRoot) {
-      buffer.unshift(`[#${pageId}]`)
-      buffer.doctypeIdx += 1
+      builder.setDocid(pageId)
     }
+    if (isRootPage || (atDocumentRoot && !heading)) assembled.pages.rootPageFragment = pageFragment
+    assembled.pages.set(page, pageFragment)
     if (sectionMergeStrategy === 'enclose' && hasItems && hasSections && !(atDocumentRoot && heading)) {
       // TODO: make overview section title configurable
       //let overviewTitle = doc.getDocumentTitle()
       //if (overviewTitle === navtitle) overviewTitle = doc.getAttribute('overview-title', 'Overview')
       const overviewTitle = doc.getAttribute('overview-title', 'Overview')
       const syntheticId = idSeparators.generateIdFromTitle(overviewTitle, idSeparators, pageIdLeader)
-      if (heading) buffer.push('')
+      if (heading && buffer.length) buffer.push('')
       let hlevel = level + (nextSectionLevel = 2)
       if (hlevel > 6) {
         const blockStyle = `discrete.h${hlevel}`
@@ -434,7 +452,6 @@ function mergeAsciiDoc (
       }
       buffer.push(`${'='.repeat(hlevel)} ${overviewTitle}`)
     }
-    assembled.pages.set(page, pageFragment)
     if (hasSections) fixSectionLevels(doc.getSections(), nextSectionLevel)
     const allBlocks = doc.findBy({ traverse_documents: true }, (it) =>
       it.getContext() === 'document'
@@ -444,7 +461,7 @@ function mergeAsciiDoc (
     if (doc.getDoctype() === 'manpage') {
       const firstSectionIdx = hasSections ? doc.getSections()[0].getLineNumber() - 1 : lines.length
       for (let idx = 0; idx < firstSectionIdx; idx++) {
-        if (~ignoreLines.indexOf(idx)) continue
+        if (ignoreLines.includes(idx)) continue
         const line = lines[idx]
         if (line.startsWith('== ') && line.length > 3) {
           allBlocks.unshift({
@@ -488,7 +505,7 @@ function mergeAsciiDoc (
     })
     let skipping
     for (let idx = 0, lastIdx = lines.length - 1; idx <= lastIdx; idx++) {
-      if (~ignoreLines.indexOf(idx)) continue
+      if (ignoreLines.includes(idx)) continue
       let line = lines[idx]
       if (line.startsWith('//')) {
         if (line.charAt(2) !== '/') continue
@@ -604,10 +621,13 @@ function mergeAsciiDoc (
       buffer,
       lines.filter((it) => it !== undefined)
     )
-    const attributeEntries = Object.entries(doc.source_header_attributes.$$smap)
+    const attributeEntries = doc.source_header_attributes.$entries()
     if (attributeEntries.length) {
       const resolvedAttributeEntries = attributeEntries.reduce(
         (accum, [name, val]) => {
+          if (val) {
+            if (val['$nil?']()) val = null
+          }
           // Q: couldn't we just check if attribute is locked?
           if (name in mutableAttributes) {
             const initialVal = mutableAttributes[name]
@@ -626,12 +646,11 @@ function mergeAsciiDoc (
       if (resolvedAttributeEntries.length > 1) safePush(buffer, resolvedAttributeEntries)
     }
   } else if (level && !hash) {
+    const buffer = (builder.body ??= [])
     if (atDocumentRoot && rootLevel === 0 && navtitlePlain === componentVersion.title) {
-      buffer.inBody ??= false
       level--
     } else {
-      buffer.inBody = true
-      buffer.push('')
+      if (buffer.length) buffer.push('')
       const syntheticId = idSeparators.generateIdFromTitle(navtitleAsciiDoc, idSeparators)
       let sectionTitle = navtitleAsciiDoc
       if (urlType === 'external') {
@@ -647,38 +666,39 @@ function mergeAsciiDoc (
           }
         }
       }
+      if (supportsParts && level === 1 && roles.includes('part')) {
+        roles.splice(roles.indexOf('part', 1))
+        level--
+      }
       const hlevel = level > 5 ? 6 : level + 1
       const hroles = urlType === 'internal' ? roles.slice(1) : roles
       buffer.push(`[${hlevel > 6 ? 'discrete' : ''}#${syntheticId}${hroles.reduce((str, it) => str + '.' + it, '')}]`)
       buffer.push(`${'='.repeat(hlevel)} ${sectionTitle}`)
     }
   }
-  if (hasItems) {
-    const nextLevel = level + 1
-    // NOTE: drop first child if same as parent; should we keep if content is different?
-    ;(urlType === 'internal' && items[0].urlType === 'internal' && pathname === items[0].url && !items[0].items?.length
-      ? items.slice(1)
-      : items
-    ).forEach((item) => {
-      mergeAsciiDoc(
-        loadAsciiDoc,
-        contentCatalog,
-        buffer,
-        componentVersion,
-        item,
-        files,
-        pagesInOutline,
-        idSeparators,
-        asciidocConfig,
-        mutableAttributes,
-        assemblyModel,
-        lastComponentVersion,
-        nextLevel,
-        atBookRoot
-      )
-    })
-  }
-  return buffer
+  if (!hasItems) return
+  const nextLevel = level + 1
+  // NOTE: drop first child if same as parent; should we keep if content is different?
+  ;(urlType === 'internal' && items[0].urlType === 'internal' && pathname === items[0].url && !items[0].items?.length
+    ? items.slice(1)
+    : items
+  ).forEach((item) => {
+    mergeAsciiDoc(
+      loadAsciiDoc,
+      contentCatalog,
+      builder,
+      componentVersion,
+      item,
+      files,
+      pagesInOutline,
+      idSeparators,
+      asciidocConfig,
+      mutableAttributes,
+      assemblyModel,
+      nextLevel,
+      atBookRoot
+    )
+  })
 }
 
 function processDocumentHeader (doc, lines, ignoreLines, isRootPage) {
@@ -717,7 +737,7 @@ function processDocumentHeader (doc, lines, ignoreLines, isRootPage) {
     }
     const line = lines[idx]
     if (open === ':' || open === '-:') {
-      if (open === ':') current.lines.push(line)
+      if (open === ':') current.appendLine(line)
       if (!line?.endsWith(' \\')) current = open = undefined
     } else if (line) {
       let attributeEntryMatch
@@ -725,13 +745,14 @@ function processDocumentHeader (doc, lines, ignoreLines, isRootPage) {
       if (chr0 === '/' && line.charAt(1) === '/') {
         if (line.startsWith('////') && line === '/'.repeat(line.length)) {
           if (open) {
-            current.lines.push(line)
+            current.appendLine(line)
             if (open === line) current = open = undefined
           } else {
             entries.push((current = { type: 'block_comment', lines: [(open = line)] }))
+            current.appendLine = Array.prototype.push.bind(current.lines)
           }
         } else if (open) {
-          current.lines.push(line)
+          current.appendLine(line)
         } else if (belowDoctitle && line.charAt(2) === '/') {
           break
         } else {
@@ -739,7 +760,7 @@ function processDocumentHeader (doc, lines, ignoreLines, isRootPage) {
           current = undefined
         }
       } else if (open) {
-        current.lines.push(line)
+        current.appendLine(line)
       } else if (chr0 === ':' && ~line.indexOf(':', 2) && (attributeEntryMatch = line.match(ATTR_ENTRY_RX))) {
         let name = attributeEntryMatch[1]
         const negated = name.charAt() === '!' || name.charAt(name.length - 1) === '!'
@@ -748,7 +769,9 @@ function processDocumentHeader (doc, lines, ignoreLines, isRootPage) {
           if (line.endsWith(' \\')) open = '-:' // disallow value continuation
         } else {
           if (line.endsWith(' \\')) open = ':'
-          entries.push((current = { type: 'attribute_entry', name, negated, lines: [line], lineno: idx + 1 }))
+          entries.push((current = { type: 'attribute_entry', name, negated, lines: [], lineno: idx + 1 }))
+          current.lines.push(negated ? `:!${name}:` : line)
+          current.appendLine = Array.prototype.push.bind(negated ? [] : current.lines)
           if (assemblyHeaderAttributes.has(name)) current.promote = true
         }
       } else if (belowDoctitle) {
@@ -799,36 +822,22 @@ function fixSectionLevels (sections, expectedLevel) {
   })
 }
 
-// NOTE: blank lines at top and bottom of document create mismatch when using line numbers to navigate source lines
-// IMPORTANT: this must not leave behind lines the parser will drop!
-// IDEA: another option is to capture initial lineno of reader and use as offset (but preseves those blank lines)
-function trimAsciiDoc (buffer) {
-  return Buffer.from(
-    buffer
-      .toString()
-      .replace(/^(?:[ \t]*\r\n?|[ \t]*\n)+/, '')
-      .trimRight()
-  )
-}
-
 function safePush (onto, entries) {
   try {
     onto.push(...entries)
-  } catch (err) {
-    /* istanbul ignore if */
-    if (!(err instanceof RangeError)) throw err
+  } catch {
     for (const e of entries) onto.push(e)
   }
 }
 
 function generateIdFromTitle (titleAsciiDoc, idSeparators, idLeader) {
-  const Section = this.$class().$const_get('::Asciidoctor::Section')
+  const Section = this.$class().$$base_module.Section
   const baseId = Section.$generate_id(titleAsciiDoc, this) // refs in catalog will always be empty
   let id = `_${idSeparators.coordinate}${idLeader ?? ''}${baseId}`
   const syntheticIds = (this.catalog.syntheticIds ??= {})
   if (id in syntheticIds) {
     const sep = (this.getAttribute('idseparator') ?? '_').charAt()
-    let cnt = this.$class().$const_get('::Asciidoctor::Compliance').unique_id_start_index
+    let cnt = this.$class().$$base_module.Compliance.unique_id_start_index
     let candidate
     while ((candidate = `${id}${sep}${cnt}`) in syntheticIds) cnt++
     id = candidate