npm - @uniweb/build - Versions diffs - 0.20.0 → 0.21.0 - Mend

@uniweb/build 0.20.0 → 0.21.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (3) hide show

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@uniweb/build",
-  "version": "0.20.0",
+  "version": "0.21.0",
   "description": "Build tooling for the Uniweb Component Web Platform",
   "type": "module",
   "exports": {
@@ -59,14 +59,14 @@
     "js-yaml": "^4.1.0",
     "sharp": "^0.35.3",
     "yaml": "^2.5.0",
-    "@uniweb/content-writer": "^0.3.3",
-    "@uniweb/projections": "^0.2.8",
     "@uniweb/schemas": "^0.2.10",
-    "@uniweb/theming": "^0.1.15"
+    "@uniweb/projections": "^0.3.0",
+    "@uniweb/theming": "^0.1.15",
+    "@uniweb/content-writer": "^0.3.3"
   },
   "optionalDependencies": {
-    "@uniweb/content-reader": "^1.2.2",
     "@uniweb/schemas": "^0.2.10",
+    "@uniweb/content-reader": "^1.2.2",
     "@uniweb/semantic-parser": "^1.2.2",
     "@uniweb/runtime": "^0.11.7"
   },

package/src/prerender.js CHANGED Viewed

@@ -148,6 +148,41 @@ async function executeAllFetches(siteContent, siteDir, onProgress, localeInfo) {
  * @param {function} onProgress - Progress callback
  * @returns {Array} Expanded pages array with dynamic pages replaced by concrete instances
  */
+/**
+ * Where a content-less container's redirect stub should point, in THIS locale.
+ *
+ * A folder with no body of its own gets a `<meta http-equiv="refresh">` stub so
+ * the redirect works without JS. `page.getNavigableRoute()` answers in CANONICAL
+ * routes — right, because the hierarchy is canonical and "does this folder have
+ * a page of its own?" is not a language question. But the stub is per-locale, so
+ * the destination has to be localized exactly the way `outputRoute` is: translate
+ * the slug, then re-apply the locale prefix.
+ *
+ * Emitting the canonical route raw sent a reader who asked for `/fr/<container>`
+ * to the DEFAULT-locale page — a language switch they never asked for, from a
+ * URL that was correct. The runtime's copy of this redirect (PageRenderer.jsx)
+ * had the identical bug and was fixed first; this is its twin, and the two must
+ * agree or a container behaves differently on a cold load than on an in-app
+ * navigation.
+ *
+ * The DECISION to redirect stays canonical (`target !== page.route`) — only the
+ * destination is localized. Comparing a localized destination against a
+ * canonical route would make a folder with an index child redirect to itself.
+ *
+ * @param {string} target - Canonical route from getNavigableRoute()
+ * @param {Object} ctx
+ * @param {Object} ctx.website - Website instance (for translateRoute / basePath)
+ * @param {string} ctx.locale - Locale being rendered
+ * @param {boolean} ctx.isDefault - Is this the site's default locale?
+ * @param {string} ctx.routePrefix - Locale prefix for this pass ('' or '/fr')
+ * @returns {string} Absolute path for the stub's refresh + canonical link
+ */
+export function localizeRedirectTarget(target, { website, locale, isDefault, routePrefix = '' }) {
+  const localized = isDefault ? target : routePrefix + website.translateRoute(target, locale)
+  const withSlash = localized.startsWith('/') ? localized : '/' + localized
+  return (website.basePath || '') + withSlash
+}
 export function expandDynamicPages(pages, pageFetchedData, onProgress) {
   const expandedPages = []
@@ -704,8 +739,12 @@ export async function prerenderSite(siteDir, options = {}) {
       if (!page.hasContent()) {
         const target = page.getNavigableRoute()
         if (target && target !== page.route) {
-          const base = website.basePath || ''
-          const targetPath = base + (target.startsWith('/') ? target : '/' + target)
+          const targetPath = localizeRedirectTarget(target, {
+            website,
+            locale,
+            isDefault,
+            routePrefix
+          })
           onProgress(`  Auto-redirect ${outputRoute} → ${targetPath}`)
           const redirectHtml = `<!DOCTYPE html><html><head><meta charset="utf-8"><meta http-equiv="refresh" content="0;url=${targetPath}"><link rel="canonical" href="${targetPath}"><title>Redirecting...</title></head><body><p>Redirecting to <a href="${targetPath}">${targetPath}</a></p></body></html>`
           const outputPath = getOutputPath(distDir, outputRoute)

package/src/site/plugin.js CHANGED Viewed

@@ -43,6 +43,7 @@ import {
   pageMarkdownFilename,
   branchIndexFilename,
   applyRouteTranslation,
+  partitionKnowledgePages,
   INDEX_FILENAME
 } from '@uniweb/projections'
 import { collectSiteContent, mountEntriesOf } from './content-collector.js'
@@ -505,6 +506,51 @@ export function siteContentPlugin(options = {}) {
     robots: seo.robots || {}
   }
+  // Warn once per build, not once per collection — the dev server re-collects
+  // on every content change and would otherwise repeat this on every keystroke.
+  let warnedAboutKnowledge = false
+  /**
+   * Collect the site, then drop `knowledge:` pages.
+   *
+   * ⛔ **This is the bundle lane, and the bundle lane has no agent endpoint.**
+   * `uniweb export` and `uniweb deploy --host <adapter>` both run this pipeline
+   * (`cli/src/commands/export.js`, `deploy.js`), and their destinations are
+   * static hosts — no worker, no backend, nothing that can gate a request. A
+   * `knowledge:` page is content the author marked for an agent and not for a
+   * visitor; here there is no agent to serve it to and no gate to withhold it
+   * with, so shipping it is disclosure with **no consumer on the other side**.
+   *
+   * Measured 2026-08-13, before this existed: a knowledge page became
+   * `dist/kb/index.html` — a public route — and its body rode in the
+   * `__SITE_CONTENT__` of *every* prerendered page including `404.html`,
+   * because that script carries the whole site.
+   *
+   * ⚠️ **Deliberately NOT inside `collectSiteContent`.** The link lane
+   * (`build-site-data.js`) calls that same collector and **must keep** these
+   * pages: it hands `site-content.json` to a backend that derives an agent
+   * corpus from it and gates access to that corpus. Same flag, two correct
+   * answers, decided by what is behind the destination — so the subtraction
+   * belongs to the pipeline, not to the collector.
+   */
+  async function collectForBundle(path, options) {
+    const collected = await collectSiteContent(path, options)
+    const { knowledgePages, renderedPages } = partitionKnowledgePages(collected.pages || [])
+    if (!knowledgePages.length) return collected
+    if (!warnedAboutKnowledge) {
+      warnedAboutKnowledge = true
+      const routes = knowledgePages.map(page => page.route).join(', ')
+      console.warn(
+        `[site-content] Dropped ${knowledgePages.length} knowledge page(s) from this build: ${routes}\n` +
+          `  This build targets a host that serves files only, so there is no agent endpoint to read them ` +
+          `and no gate to keep them from visitors. Use \`uniweb deploy\` to make them agent-readable.`
+      )
+    }
+    return { ...collected, pages: renderedPages }
+  }
   let siteContent = null
   let resolvedSitePath = null
   let resolvedPublicDir = null
@@ -738,7 +784,7 @@ export function siteContentPlugin(options = {}) {
       if (!isProduction) {
         try {
           // Do an early content collection to get the collections config
-          const earlyContent = await collectSiteContent(resolvedSitePath, { foundationPath })
+          const earlyContent = await collectForBundle(resolvedSitePath, { foundationPath })
           collectionsConfig = earlyContent.config?.collections
           // Resolve content directory paths from site.yml paths: group
@@ -788,7 +834,7 @@ export function siteContentPlugin(options = {}) {
         // pages stay in the graph so in-progress drafts remain previewable.
         // strict on a production build: a mount that contributes no pages is a
         // warning while you author and a broken deploy once you ship.
-        siteContent = await collectSiteContent(resolvedSitePath, { foundationPath, dropUnpublished: isProduction, base: basePath, strict: isProduction })
+        siteContent = await collectForBundle(resolvedSitePath, { foundationPath, dropUnpublished: isProduction, base: basePath, strict: isProduction })
         headHtml = await loadHeadHtml()
         console.log(`[site-content] Collected ${siteContent.pages?.length || 0} pages`)
@@ -841,7 +887,7 @@ export function siteContentPlugin(options = {}) {
           rebuildTimeout = setTimeout(async () => {
             console.log('[site-content] Content changed, rebuilding...')
             try {
-              siteContent = await collectSiteContent(resolvedSitePath, { foundationPath, base: basePath })
+              siteContent = await collectForBundle(resolvedSitePath, { foundationPath, base: basePath })
               headHtml = await loadHeadHtml()
               // Execute fetches for the updated content
               await executeDevFetches(siteContent, resolvedSitePath)