@antora/assembler 1.0.0-rc.4 → 1.0.0-rc.5

package/lib/assemble-content.js

@@ -23,7 +23,8 @@ async function assembleContent (playbook, contentCatalog, converter, { configSou
     backend: targetBackend = targetExtname.substring(1),
     embedReferenceStyle = 'relative',
     mediaType: targetMediaType,
-    loggerName = PACKAGE_NAME,
+    loggerName = `${PACKAGE_NAME} [${targetBackend}-exporter]`,
+    xmlCompliant,
   } = converter ?? {}
   const assemblerConfig = await loadConfig.call(this, configSource, playbook, '-' + targetBackend)
   if (assemblerConfig.enabled === false || !contentCatalog.publishableFamilies?.has('export')) return []
@@ -38,8 +39,9 @@ async function assembleContent (playbook, contentCatalog, converter, { configSou
   const { loadAsciiDoc = require('@antora/asciidoc-loader') } = generatorFunctions
   const { assembly: assemblyConfig, build: buildConfig } = assemblerConfig
   assemblyConfig.embedReferenceStyle = embedReferenceStyle
+  if (xmlCompliant) assemblyConfig.xmlIds ??= true
   const profile = (assemblyConfig.profile ??= targetBackend)
-  const intrinsicAttributes = { 'loader-assembler': '' }
+  const intrinsicAttributes = { 'loader-assembler': '', 'assembler-root-level': assemblyConfig.rootLevel }
   const cwd = (buildConfig.cwd ??= process.cwd())
   if (profile) {
     buildConfig.dir ??= ospath.join(playbook.dir ?? process.cwd(), `build/assembler/${profile}`)
@@ -150,16 +152,21 @@ function generateSelectAssemblyProfile (context, contentCatalog, baseModel, intr
     const attributes = Object.assign({}, baseModel.attributes)
     const model = Object.assign({}, baseModel, overrides, { attributes, logger })
     if (!overrides) return Object.assign(model, { navigation: navigation || componentVersion.navigation })
+    if ('rootLevel' in overrides) attributes['assembler-root-level'] = overrides.rootLevel
     delete model.navFiles
     delete model.messages
-    if (overrides.attributes) Object.entries(overrides.attributes).forEach(([name, val]) => (attributes[name] = val))
+    if (overrides.attributes) {
+      for (const [name, val] of Object.entries(overrides.attributes)) {
+        if (!(name in intrinsicAttributes)) attributes[name] = val
+      }
+    }
     if (navigation) return Object.assign(model, { navigation })
     overrides.messages?.forEach(boundSendToLog)
     const navFiles = overrides.navFiles
     if (!navFiles) return Object.assign(model, { navigation: componentVersion.navigation })
     if (!navFiles.length) return Object.assign(model, { navigation: [] })
     model.navigation = buildAlternateNavigation(contentCatalog, componentVersion, navFiles, {
-      attributes: intrinsicAttributes,
+      attributes: model.attributes,
     })
     return model
   }
@@ -182,7 +189,7 @@ function prepareConvertAttributes (doc, targetExtname, relativeToOutput, assembl
     docdir,
     docfile,
     docfilesuffix,
-    'docname@': docname,
+    docname: `${docname}@`,
     imagesdir: '',
     outdir,
     outfile,
@@ -206,6 +213,17 @@ function prepareConvertAttributes (doc, targetExtname, relativeToOutput, assembl
       return args
     },
   })
+  Object.defineProperty(attributes, 'outfilesuffix', {
+    get () {
+      return ospath.extname(this.outfile)
+    },
+    set (value) {
+      const outfile = this.outfile
+      const extname = ospath.extname(outfile)
+      if (extname && outfile.endsWith(extname)) this.outfile = outfile.substring(0, outfile.length - extname.length)
+      this.outfile += value
+    },
+  })
   return Object.defineProperty(attributes, 'toArgs', { enumerable: false })
 }
 
@@ -265,12 +283,14 @@ async function resolveFileOrContents (convertResult, convertAttributes, buildCon
       fileOrContents = convertResult.stdout
     }
     let logger, match
-    if (buildConfig.stderrSink === 'log' && convertResult.stderr.length && (logger = this.getLogger(loggerName))) {
+    if (buildConfig.stderrSink === 'log' && convertResult.stderr?.length && (logger = this.getLogger(loggerName))) {
       const docfile = convertAttributes.docfile
       const command = buildConfig.command
+      const file = { path: docfile }
       const stderr = convertResult.stderr.toString().trimEnd()
+      const additionalLines = ['Additional stderr lines from command:']
       stderr.split(NEWLINE_RX).forEach((line) => {
-        const ctx = { command, file: { path: docfile } }
+        const ctx = { command, file }
         if (line.charAt() === '{' && line.charAt(line.length - 1) === '}') {
           const entry = JSON.parse(line)
           if (entry.name) ctx.program = entry.name
@@ -286,9 +306,10 @@ async function resolveFileOrContents (convertResult, convertAttributes, buildCon
           if (lineno) ctx.line = parseInt(lineno, 10)
           logger[level === 'WARNING' ? 'warn' : level.toLowerCase()](ctx, msg)
         } else {
-          logger.info(ctx, line)
+          additionalLines.push(line)
         }
       })
+      if (additionalLines.length > 1) logger.info({ command, file }, additionalLines.join('\n'))
     }
   } else if (convertResult !== undefined) {
     return convertResult

package/lib/configure.js

@@ -17,7 +17,8 @@ function internalConfigure (converter, config = {}, providers = {}) {
 
   this.once('navigationBuilt', async ({ playbook, contentCatalog }) => {
     const { assembleContent = require('./assemble-content'), ...assembleContentConfig } = providers
-    if (assembleContentConfig.configSource?.constructor === Object) {
+    if (config.configSource?.constructor === Object) {
+      assembleContentConfig.configSource = config.configSource
       await assembleContent.call(this, playbook, contentCatalog, converter, assembleContentConfig)
     } else {
       const singleConfig = !('configFiles' in config)

package/lib/load-config.js

@@ -9,6 +9,7 @@ const yaml = require('js-yaml')
 const { LEGACY_ASSEMBLY_KEYS } = require('./constants')
 const CAMEL_CASE_STOP_PATHS = ['asciidoc.attributes', 'assembly.attributes']
 const PACKAGE_NAME = require('../package.json').name
+const YAML_SCHEMA = yaml.CORE_SCHEMA.withTags(yaml.mergeTag)
 
 function loadConfig (configSource, playbook, preferredQualifier = '') {
   let resolvedConfigSource
@@ -36,11 +37,11 @@ function loadConfig (configSource, playbook, preferredQualifier = '') {
               }
               return {}
             }
-            return fsp
-              .readFile(resolvedConfigSource)
-              .then((data) =>
-                Object.assign(camelCaseKeys(yaml.load(data), CAMEL_CASE_STOP_PATHS), { file: resolvedConfigSource })
-              )
+            return fsp.readFile(resolvedConfigSource).then((data) =>
+              Object.assign(camelCaseKeys(yaml.load(data, { schema: YAML_SCHEMA }), CAMEL_CASE_STOP_PATHS), {
+                file: resolvedConfigSource,
+              })
+            )
           })
   ).then((config) => {
     if (config.enabled === false) return config

package/lib/produce-assembly-file.js

@@ -198,6 +198,7 @@ function mergeAsciiDoc (
       Object.assign({}, page, { contents: trimAsciiDoc(contents), mediaType: page.src.mediaType })
     )
     const doc = loadAsciiDoc(pageAsAsciiDoc, contentCatalog, asciidocConfig)
+    doc.catalog.syntheticIds = {}
     doc.logger = logger
       ? Object.assign(doc.getLogger().$dup(), {
           delegate: {
@@ -475,7 +476,7 @@ function mergeAsciiDoc (
       if (~ignoreLines.indexOf(idx)) continue
       let line = lines[idx]
       if (line.startsWith('//')) {
-        if (line[2] !== '/') continue
+        if (line.charAt(2) !== '/') continue
         if (line.length > 3 && line === '/'.repeat(line.length)) {
           if (skipping) {
             if (line === skipping) skipping = undefined
@@ -807,10 +808,18 @@ function safePush (onto, entries) {
 
 function generateIdFromTitle (titleAsciiDoc, idSeparators, idLeader) {
   const Section = this.$class().$const_get('::Asciidoctor::Section')
-  const baseId = Section.$generate_id(titleAsciiDoc, this)
-  if (idLeader) return `_${idSeparators.coordinate}${idLeader}${baseId}`
-  this.getCatalog().refs['$[]='](baseId, true)
-  return `_${idSeparators.coordinate}${baseId}`
+  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 candidate
+    while ((candidate = `${id}${sep}${cnt}`) in syntheticIds) cnt++
+    id = candidate
+  }
+  syntheticIds[id] = true
+  return id
 }
 
 module.exports = produceAssemblyFile

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@antora/assembler",
-  "version": "1.0.0-rc.4",
+  "version": "1.0.0-rc.5",
   "description": "A JavaScript library that merges AsciiDoc content from multiple pages in an Antora site into assembly files and delegates to an exporter to convert those files to another format, such as PDF.",
   "license": "MPL-2.0",
   "author": "OpenDevise Inc. (https://opendevise.com)",
@@ -41,7 +41,7 @@
     "@asciidoctor/reducer": "~1.1",
     "@antora/expand-path-helper": "~3.0",
     "@antora/run-command-helper": "~1.1",
-    "js-yaml": "~4.3"
+    "js-yaml": "~5.2"
   },
   "devDependencies": {
     "@antora/asciidoc-loader": "3.2.0-rc.2",