@uniweb/build 0.73.1 → 0.73.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uniweb/build",
-  "version": "0.73.1",
+  "version": "0.73.2",
   "description": "Build tooling for the Uniweb Component Web Platform",
   "type": "module",
   "exports": {
@@ -57,11 +57,11 @@
     "js-yaml": "^4.1.0",
     "sharp": "^0.35.3",
     "yaml": "^2.5.0",
-    "@uniweb/content-reader": "^1.2.7",
-    "@uniweb/schemas": "^0.9.2",
+    "@uniweb/schemas": "^0.9.3",
+    "@uniweb/content-writer": "^0.3.4",
     "@uniweb/projections": "^0.7.11",
     "@uniweb/semantic-parser": "^1.4.1",
-    "@uniweb/content-writer": "^0.3.4",
+    "@uniweb/content-reader": "^1.2.7",
     "@uniweb/theming": "^0.2.0"
   },
   "optionalDependencies": {

package/src/prerender.js

@@ -32,14 +32,34 @@ import { FONT_LINKS_MARKER } from './site/head-markers.js'
 import { getAdapter } from './hosts/index.js'
 import { detectCiContext } from './hosts/detect-ci-context.js'
 import { stripBasePath } from './site/extension-urls.js'
+import { isSiteRelative, resolveExtensionDist } from './site/emit-extensions.js'
 
 /**
- * Resolve an extension URL to a filesystem path for prerender.
+ * Resolve an extension URL to the file prerender imports.
  * Browser URLs like "/effects/entry.js" need mapping to local files.
  *
+ * ⭐ AN EXTENSION IS IMPORTED FROM ITS OWN BUILD, as the primary foundation is
+ * (`<foundation>/dist/entry.js`, in prerenderSite). A foundation build leaves its
+ * externals — `@uniweb/core`, `react`, `react/jsx-runtime` — as bare imports, and
+ * Node resolves them from where the FILE sits: inside the extension's own
+ * package, which depends on them.
+ *
+ * ⛔ Not from the copy the site build emits into `dist/` (site/emit-extensions.js).
+ * That copy is the browser's, which resolves the same imports through the page's
+ * import map; here it sits under the site package, which need not depend on
+ * `@uniweb/core` at all. It was the first candidate until 2026-09-28 — an order
+ * written when nothing put a file there — and once the build began emitting one
+ * (2026-08-05), every workspace extension failed here with "Cannot find package
+ * '@uniweb/core'", and the static HTML carried `Component not found` where its
+ * sections belonged until the page ran in the browser.
+ *
  * Resolution order:
- * 1. dist directory (post-build copy target, e.g., site/dist/effects/entry.js)
- * 2. Project root with dist subdir (dev layout, e.g., project/effects/dist/entry.js)
+ * 1. The extension's own build — `<project>/<name>/dist/` or
+ *    `<project>/extensions/<name>/dist/` — found by the emitter's lookup
+ *    (`resolveExtensionDist`), so the two lanes find the same build.
+ * 2. A file in the site's dist, for an extension no workspace package builds (a
+ *    pre-built module placed in `public/`). Its bare imports resolve only where
+ *    the site's own dependencies carry them.
  * 3. Original URL (absolute or remote — let import() handle it)
  */
 export function resolveExtensionPath(url, distDir, projectRoot, base) {
@@ -50,32 +70,18 @@ export function resolveExtensionPath(url, distDir, projectRoot, base) {
   // so a payload produced before this change still resolves.
   url = stripBasePath(url, base)
 
-  // Only resolve URLs that look like root-relative paths
-  if (url.startsWith('/')) {
-    // Try dist directory first (production: files copied to site/dist/)
-    const distPath = join(distDir, url)
-    if (existsSync(distPath)) return distPath
-
-    // Workspace layouts: "/effects/entry.js" → "<pkg>/dist/entry.js", checked
-    // both at the project root and under an `extensions/` parent — the standard
-    // multi-foundation layout puts extensions in `extensions/<name>/`, so the
-    // bare-root candidate alone misses the built module and prerender can't load
-    // the extension.
-    const parts = url.slice(1).split('/')
-    if (parts.length >= 2) {
-      const pkgName = parts[0]
-      const rest = parts.slice(1).join('/')
-      const candidates = [
-        join(projectRoot, pkgName, 'dist', rest),
-        join(projectRoot, 'extensions', pkgName, 'dist', rest),
-      ]
-      for (const devPath of candidates) {
-        if (existsSync(devPath)) return devPath
-      }
-    }
+  // Return as-is for absolute paths or remote URLs
+  if (!isSiteRelative(url)) return url
+
+  const build = resolveExtensionDist(url, projectRoot)
+  if (build) {
+    const inBuild = join(build.distDir, build.rest)
+    if (existsSync(inBuild)) return inBuild
   }
 
-  // Return as-is for absolute paths or remote URLs
+  const inDist = join(distDir, url)
+  if (existsSync(inDist)) return inDist
+
   return url
 }
 

package/src/schema.js

@@ -17,7 +17,7 @@ import { isFontVar } from '@uniweb/theming'
 import { join, dirname, extname, basename } from 'node:path'
 import { pathToFileURL } from 'node:url'
 import { describeChildren, describeVisuals } from '@uniweb/schemas/component'
-import { SECTION_PARAMS } from '@uniweb/schemas/section'
+import { SECTION_PARAMS, SECTION_KEYS } from '@uniweb/schemas/section'
 import { collectSchemaRefs, buildDataSchemaMap, ownSchemaRefs } from './resolve-data-schema.js'
 import {
   composeSupports,
@@ -958,19 +958,32 @@ export function reportPlacementDeclarations(components) {
           `provides renders "Component not found".`
       )
     }
-    // A name framework reserves in a section's params whose value the component never
-    // receives — `component: false` in `@uniweb/schemas/section`, as `grid` is.
+    // A name framework reserves in a section's params: a setting of the SECTION, which framework
+    // applies — `component: false` in `@uniweb/schemas/section`. A component that declares one in
+    // its `params:` declares a setting it is never given. ⛔ Until 2026-09-28 this said the param
+    // "never receives the author's theme:" and to rename it — read as "the section loses the
+    // author's value", the opposite of what happens: the section gets it, and framework applies it.
     const params = entry.params && typeof entry.params === 'object' ? entry.params : {}
     for (const [param, reserved] of Object.entries(SECTION_PARAMS)) {
       if (reserved.component || !Object.hasOwn(params, param)) continue
       console.warn(
-        `Warning: ${name} (meta.js): a param named "${param}" never receives the author's ${param}: — framework ` +
-          `reserves it in a section's params (@uniweb/schemas/section). Rename the param.` +
+        `Warning: ${name} (meta.js): params declares "${param}", a setting of the section, not of its ` +
+          `component — framework applies an author's ${param}: to the section and never passes it to the ` +
+          `component, and an editor offers it for every section (@uniweb/schemas/section). Remove it from params:.` +
           (param === 'grid'
             ? ` To offer layouts for the child sections, declare children: { grid: [...] } and lay them out with kit's ChildGrid.`
             : '')
       )
     }
+    // …and a key of a section's frontmatter that is not a param at all (`SECTION_KEYS`): the build
+    // takes it out before the params are stored, so a param of that name is never given one.
+    for (const [key, { is }] of Object.entries(SECTION_KEYS)) {
+      if (!Object.hasOwn(params, key)) continue
+      console.warn(
+        `Warning: ${name} (meta.js): params declares "${key}", a key a section's frontmatter keeps for ` +
+          `itself — ${is} — so the component is never given it (@uniweb/schemas/section). Remove it from params:.`
+      )
+    }
   }
 }
 

package/src/site/content-collector.js

@@ -23,6 +23,7 @@
 
 import { readFile, readdir, stat } from 'node:fs/promises'
 import { resolveQueriesConfig, toConfigQueries } from './queries-config.js'
+import { isSectionKey } from '@uniweb/schemas/section'
 import { parseNumericPrefix, compareByNumericPrefix } from '../utils/numeric-prefix.js'
 import { isMarkdownFile, isIgnoredFolder } from '../utils/content-files.js'
 import { readLayoutFolder, DEFAULT_LAYOUT, layoutAreaRoute } from './layout-folder.js'
@@ -1016,9 +1017,14 @@ async function processMarkdownFile(filePath, id, siteRoot, defaultStableId = nul
     console.warn(`[content-collector] ${err.message}`)
   }
 
-  // `query`, `fetch` and `data` are never params: `query:` / `fetch:` declare the
-  // section's own data, and a leftover `data:` is refused (`declaredFetch`).
-  const { type, hidden, preset, input, props, fetch, query, data, id: frontmatterId, ...params } = frontMatter
+  // ⭐ WHAT IS NOT A PARAM is `SECTION_KEYS` (`@uniweb/schemas/section`) — the list the build's
+  // warning on a component's `params:` reads too — and `fetch`, one of the section's settings,
+  // kept on the section as a field of its own. `query:` / `fetch:` declare the section's own
+  // data, and a leftover `data:` is refused (`declaredFetch`).
+  const { type, hidden, preset, input, props, fetch, query, data, id: frontmatterId } = frontMatter
+  const params = Object.fromEntries(
+    Object.entries(frontMatter).filter(([key]) => !isSectionKey(key) && key !== 'fetch')
+  )
   // ⭐ `hidden: true` — a draft section, the page's meaning one level down: left out of a
   // published build, kept by `uniweb dev`, and synced as the section's own `hidden`
   // [Diego, 2026-09-28]. ⛔ Until then it was an ordinary param: the component received it

package/src/site/emit-extensions.js

@@ -39,6 +39,11 @@
  * Same browser/internal split the runtime's distribution channel draws, for the
  * same reason: what a visitor fetches and what a renderer needs are different
  * sets, and only one of them belongs on a public origin.
+ *
+ * ⛔ The emitted copy is the BROWSER's, and only the browser's. Prerender imports
+ * the extension from its own build (`resolveExtensionPath`), because the copy
+ * sits under the site package, where the extension's bare imports
+ * (`@uniweb/core`, `react`) do not resolve.
  */
 
 import { existsSync, readdirSync, readFileSync, statSync } from 'node:fs'
@@ -69,26 +74,31 @@ function isBrowserAsset(rel) {
 }
 
 /**
- * Locate the built `dist/` behind a site-relative extension URL.
+ * Locate the built `dist/` behind a site-relative extension URL:
+ * `/effects/entry.js` → `<project>/effects/dist/` or
+ * `<project>/extensions/effects/dist/`, with `rest` the path inside it.
  *
- * The same candidates `resolveExtensionPath` walks for prerender, kept in step
- * deliberately: if prerender can load an extension from the workspace but the
- * build cannot find it to emit, that is exactly the split that produced the
- * bug — one lane resolving it and the other not.
+ * One lookup for both lanes that need the extension's build: this emitter, which
+ * copies it into the site's output, and prerender, which imports it from where it
+ * was built (`resolveExtensionPath`). Neither can find an extension the other
+ * cannot, by construction rather than by two lists kept in step by hand.
  *
- * @returns {{ distDir: string, urlBase: string }|null}
+ * @param {string} url - a site-relative URL, with any deployment base removed
+ * @param {string} projectRoot - the directory holding the site and its extensions
+ * @returns {{ distDir: string, urlBase: string, rest: string }|null}
  */
-export function resolveExtensionDist(url, siteDir) {
+export function resolveExtensionDist(url, projectRoot) {
   const parts = url.replace(/^\//, '').split('/')
   if (parts.length < 2) return null
   const pkgName = parts[0]
-  const projectRoot = resolve(siteDir, '..')
 
   for (const candidate of [
     join(projectRoot, pkgName, 'dist'),
     join(projectRoot, 'extensions', pkgName, 'dist')
   ]) {
-    if (existsSync(candidate)) return { distDir: candidate, urlBase: pkgName }
+    if (existsSync(candidate)) {
+      return { distDir: candidate, urlBase: pkgName, rest: parts.slice(1).join('/') }
+    }
   }
   return null
 }
@@ -108,11 +118,12 @@ export function collectExtensionAssets(extensions, siteDir) {
   const emit = []
   const unresolved = []
   if (!Array.isArray(extensions)) return { emit, unresolved }
+  const projectRoot = resolve(siteDir, '..')
 
   for (const decl of extensions) {
     if (!isSiteRelative(decl)) continue // absolute URL or a ref — someone else serves it
     const url = typeof decl === 'string' ? decl : decl.url
-    const found = resolveExtensionDist(url, siteDir)
+    const found = resolveExtensionDist(url, projectRoot)
     if (!found) {
       unresolved.push(url)
       continue