@antora/pdf-extension 1.0.0-alpha.6 → 1.0.0-alpha.8

package/README.md

@@ -21,7 +21,7 @@ antora:
 # ...
 ```
 
-You can configure the behavior of the extension using the optional *antora-assembler.yml* file.
+You can configure the behavior of the extension using the optional _antora-assembler.yml_ file.
 
 ## Copyright and License
 

package/lib/convert-document-to-pdf.js

@@ -1,28 +1,23 @@
 'use strict'
 
-const { runCommand } = (() => {
-  try {
-    return require('@antora/assembler')
-  } catch {
-    return require('../../assembler')
-  }
-})()
-const ospath = require('path')
+const fsp = require('node:fs/promises')
+const ospath = require('node:path')
 
 // Q: how can we simplify this to avoid redundant code?
 // Q: rename to convertDocumentToPDF?
-function convertDocumentToPdf (doc, buildConfig) {
+async function convertDocumentToPdf (doc, buildConfig, runCommand) {
   const {
     asciidoc: { attributes: baseAttributes } = { attributes: {} },
     contents: input,
-    src: { component, version, basename, extname: docfilesuffix },
+    src: { component, version, basename, extname = doc.extname },
   } = doc
-  const { command = 'asciidoctor-pdf', cwd, dir } = buildConfig
+  const { cwd = process.cwd(), dir = cwd } = buildConfig
+  const command = buildConfig.command ?? (await scopeDefaultCommand(cwd))
   const docfile = `${version}@${component}::pdf$${basename}`
-  const docname = basename.substr(0, basename.length - docfilesuffix.length)
+  const docname = basename.slice(0, basename.length - extname.length)
   const convertAttributes = Object.assign({}, baseAttributes, {
     docfile,
-    docfilesuffix,
+    docfilesuffix: extname,
     'docname@': docname,
     imagesdir: dir,
   })
@@ -33,6 +28,8 @@ function convertDocumentToPdf (doc, buildConfig) {
     []
   )
   const output = ospath.join(dir, doc.path)
+  // Q: should mkdirs be the default behavior?
+  if (buildConfig.mkdirs) await fsp.mkdir(ospath.dirname(output), { recursive: true, force: true })
   argv.push('-o', output)
   // Q: should runCommand accept outputFlag and automatically append to argv?
   return runCommand(command, argv, { cwd, input, output }).then((contents) =>
@@ -40,4 +37,11 @@ function convertDocumentToPdf (doc, buildConfig) {
   )
 }
 
+function scopeDefaultCommand (cwd, baseCommand = 'asciidoctor-pdf') {
+  return fsp.stat(ospath.join(cwd, 'Gemfile.lock')).then(
+    () => `bundle exec ${baseCommand}`,
+    () => baseCommand
+  )
+}
+
 module.exports = convertDocumentToPdf

package/lib/index.js

@@ -1,26 +1,13 @@
 'use strict'
 
 const convertDocumentToPdf = require('./convert-document-to-pdf')
-const { assembleContent } = (() => {
-  try {
-    return require('@antora/assembler')
-  } catch {
-    return require('../../assembler')
-  }
-})()
 
 module.exports.register = function () {
-  //this.on('beforeProcess', ({ asciidocConfig }) => {
-  //  asciidocConfig.keepSource = true
-  //})
-  this.on('contentClassified', ({ contentCatalog }) => {
-    contentCatalog.getPages((page) => {
-      if (!page.out) return
-      page.src.contents = page.contents
-      page.src = new Proxy(page.src, { deleteProperty: (o, p) => (p === 'contents' ? true : delete o[p]) })
-    })
+  this.once('beforeProcess', ({ siteAsciiDocConfig }) => {
+    siteAsciiDocConfig.keepSource = true
+  })
+  this.once('beforePublish', ({ playbook, contentCatalog, siteCatalog }) => {
+    const { assembleContent } = this.require('@antora/assembler')
+    return assembleContent.call(this, playbook, contentCatalog, convertDocumentToPdf, { siteCatalog })
   })
-  this.on('beforePublish', ({ playbook, contentCatalog, siteCatalog }) =>
-    assembleContent.call(this, playbook, contentCatalog, convertDocumentToPdf, { siteCatalog })
-  )
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@antora/pdf-extension",
-  "version": "1.0.0-alpha.6",
+  "version": "1.0.0-alpha.8",
   "description": "An Antora extension that assembles content pages into PDF files by version and publishes them with the site.",
   "license": "MPL-2.0",
   "author": "OpenDevise Inc. (https://opendevise.com)",
@@ -14,15 +14,19 @@
     "url": "https://gitlab.com/antora/antora-assembler/issues"
   },
   "scripts": {
-    "prepublishOnly": "node $npm_config_local_prefix/npm/prepublishOnly.js",
-    "postpublish": "node $npm_config_local_prefix/npm/postpublish.js"
+    "test": "_mocha test",
+    "prepublishOnly": "npx -y downdoc --prepublish",
+    "postpublish": "npx -y downdoc --postpublish"
   },
   "main": "lib/index.js",
   "exports": {
     ".": "./lib/index.js"
   },
+  "imports": {
+    "#convert-document-to-pdf": "./lib/convert-document-to-pdf.js"
+  },
   "dependencies": {
-    "@antora/assembler": "1.0.0-alpha.6"
+    "@antora/assembler": "1.0.0-alpha.8"
   },
   "engines": {
     "node": ">=16.0.0"