@antora/content-classifier 3.1.7 → 3.1.9

package/lib/classify-content.js

@@ -2,6 +2,8 @@
 
 const ContentCatalog = require('./content-catalog')
 const collateAsciiDocAttributes = require('@antora/asciidoc-loader/config/collate-asciidoc-attributes')
+const logger = require('./logger')
+const summarizeFileLocation = require('./util/summarize-file-location')
 
 /**
  * Organizes the raw aggregate of virtual files into a {ContentCatalog}.
@@ -34,8 +36,16 @@ function classifyContent (playbook, aggregate, siteAsciiDocConfig = {}) {
     .forEach((componentVersionData, componentVersion) => {
       const { name, version } = componentVersion
       const { files, nav, startPage } = componentVersionData
+      const navResolved = nav && (nav.resolved = new Set())
       componentVersionData.files = undefined // clean up memory
       files.forEach((file) => allocateSrc(file, name, version, nav) && contentCatalog.addFile(file))
+      if (navResolved && nav.length > navResolved.size && new Set(nav).size > navResolved.size) {
+        const loc = summarizeFileLocation({ path: 'antora.yml', src: { origin: nav.origin } })
+        for (const filepath of nav) {
+          if (navResolved.has(filepath)) continue
+          logger.warn('Could not resolve nav entry for %s@%s defined in %s: %s', version, name, loc, filepath)
+        }
+      }
       contentCatalog.registerComponentVersionStartPage(name, componentVersion, startPage)
     })
   contentCatalog.registerSiteStartPage(playbook.site.startPage)
@@ -45,9 +55,9 @@ function classifyContent (playbook, aggregate, siteAsciiDocConfig = {}) {
 function allocateSrc (file, component, version, nav) {
   const extname = file.src.extname
   const filepath = file.path
-  const navInfo = nav && getNavInfo(filepath, nav)
   const pathSegments = filepath.split('/')
-  if (navInfo) {
+  let navInfo
+  if (nav && (navInfo = getNavInfo(filepath, nav))) {
     if (extname !== '.adoc') return // ignore file
     file.nav = navInfo
     file.src.family = 'nav'
@@ -57,14 +67,14 @@ function allocateSrc (file, component, version, nav) {
       file.src.relative = pathSegments.slice(2).join('/')
       file.src.moduleRootPath = calculateRootPath(pathSegments.length - 3)
     } else {
-      // relative to root
+      // relative to content source root
       file.src.relative = filepath
     }
   } else if (pathSegments[0] === 'modules') {
     let familyFolder = pathSegments[2]
     switch (familyFolder) {
       case 'pages':
-        // pages/_partials location for partials is @deprecated; special designation scheduled to be removed in Antora 4
+        // pages/_partials location for partials is @deprecated; special designation scheduled for removal in Antora 4
         if (pathSegments[3] === '_partials') {
           file.src.family = 'partial'
           // relative to modules/<module>/pages/_partials
@@ -127,7 +137,7 @@ function allocateSrc (file, component, version, nav) {
  */
 function getNavInfo (filepath, nav) {
   const index = nav.findIndex((candidate) => candidate === filepath)
-  if (~index) return { index }
+  if (~index) return nav.resolved.add(filepath) && { index }
 }
 
 function resolveAsciiDocConfig (siteAsciiDocConfig, { asciidoc, origins = [] }) {

package/lib/content-catalog.js

@@ -7,6 +7,7 @@ const { lookup: resolveMimeType } = require('./mime-types-with-asciidoc')
 const parseResourceId = require('./util/parse-resource-id')
 const { posix: path } = require('path')
 const resolveResource = require('./util/resolve-resource')
+const summarizeFileLocation = require('./util/summarize-file-location')
 const versionCompare = require('./util/version-compare-desc')
 
 const { ROOT_INDEX_ALIAS_ID, ROOT_INDEX_PAGE_ID } = require('./constants')
@@ -72,6 +73,7 @@ class ContentCatalog {
         }
       }
     }
+    // NOTE if no AsciiDoc attributes are defined in the component descriptor, asciidoc is the siteAsciiDocConfig object
     if (asciidoc) componentVersion.asciidoc = asciidoc
     const component = this[$components].get(name)
     if (component) {
@@ -143,7 +145,7 @@ class ContentCatalog {
         throw new Error(`Duplicate alias: ${generateResourceSpec(src)}`)
       } else {
         const details = [filesForFamily.get(key), file]
-          .map((it, idx) => `${idx + 1}: ${getFileLocation(it)}`)
+          .map((it, idx) => `${idx + 1}: ${summarizeFileLocation(it)}`)
           .join(LOG_WRAP)
         if (family === 'nav') {
           throw new Error(`Duplicate nav in ${src.version}@${src.component}: ${file.path}${LOG_WRAP}${details}`)
@@ -365,10 +367,10 @@ class ContentCatalog {
         throw new Error(
           existingPage === target
             ? `Page cannot define alias that references itself: ${generateResourceSpec(src)}` +
-              ` (specified as: ${spec})${LOG_WRAP}source: ${getFileLocation(existingPage)}`
+              ` (specified as: ${spec})${LOG_WRAP}source: ${summarizeFileLocation(existingPage)}`
             : `Page alias cannot reference an existing page: ${generateResourceSpec(src)} (specified as: ${spec})` +
-              `${LOG_WRAP}source: ${getFileLocation(target)}` +
-              `${LOG_WRAP}existing page: ${getFileLocation(existingPage)}`
+              `${LOG_WRAP}source: ${summarizeFileLocation(target)}` +
+              `${LOG_WRAP}existing page: ${summarizeFileLocation(existingPage)}`
         )
       }
     } else if (src.version == null) {
@@ -380,7 +382,7 @@ class ContentCatalog {
     if (existingAlias) {
       throw new Error(
         `Duplicate alias: ${generateResourceSpec(src)} (specified as: ${spec})` +
-          `${LOG_WRAP}source: ${getFileLocation(target)}`
+          `${LOG_WRAP}source: ${summarizeFileLocation(target)}`
       )
     }
     // NOTE the redirect producer will populate contents when the redirect facility is 'static'
@@ -598,13 +600,4 @@ function createSymbolicVersionAlias (component, version, symbolicVersionSegment,
     : Object.assign(symbolicVersionAlias, { out: undefined, rel: originalVersionAlias })
 }
 
-function getFileLocation ({ path: path_, src: { abspath, origin } }) {
-  if (!origin) return abspath || path_
-  const { url, gitdir, worktree, refname, tag, reftype = tag ? 'tag' : 'branch', remote, startPath } = origin
-  let details = `${reftype}: ${refname}`
-  if ('worktree' in origin) details += worktree ? ' <worktree>' : remote ? ` <remotes/${remote}>` : ''
-  if (startPath) details += ` | start path: ${startPath}`
-  return `${abspath || path.join(startPath, path_)} in ${'worktree' in origin ? worktree || gitdir : url} (${details})`
-}
-
 module.exports = ContentCatalog

package/lib/util/summarize-file-location.js

@@ -0,0 +1,14 @@
+'use strict'
+
+const { posix: path } = require('path')
+
+function summarizeFileLocation ({ path: path_, src: { abspath, origin } }) {
+  if (!origin) return abspath || path_
+  const { url, gitdir, worktree, refname, tag, reftype = tag ? 'tag' : 'branch', remote, startPath } = origin
+  let details = `${reftype}: ${refname}`
+  if ('worktree' in origin) details += worktree ? ' <worktree>' : remote ? ` <remotes/${remote}>` : ''
+  if (startPath) details += ` | start path: ${startPath}`
+  return `${abspath || path.join(startPath, path_)} in ${'worktree' in origin ? worktree || gitdir : url} (${details})`
+}
+
+module.exports = summarizeFileLocation

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@antora/content-classifier",
-  "version": "3.1.7",
+  "version": "3.1.9",
   "description": "Organizes aggregated content into a virtual file catalog for use in an Antora documentation pipeline.",
   "license": "MPL-2.0",
   "author": "OpenDevise Inc. (https://opendevise.com)",
@@ -27,10 +27,10 @@
     "#constants": "./lib/constants.js"
   },
   "dependencies": {
-    "@antora/asciidoc-loader": "3.1.7",
-    "@antora/logger": "3.1.7",
+    "@antora/asciidoc-loader": "3.1.9",
+    "@antora/logger": "3.1.9",
     "mime-types": "~2.1",
-    "vinyl": "~2.2"
+    "vinyl": "~3.0"
   },
   "engines": {
     "node": ">=16.0.0"