@uniweb/build 0.44.0 → 0.44.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uniweb/build",
-  "version": "0.44.0",
+  "version": "0.44.2",
   "description": "Build tooling for the Uniweb Component Web Platform",
   "type": "module",
   "exports": {
@@ -20,7 +20,8 @@
     "./hosts": "./src/hosts/index.js",
     "./i18n": "./src/i18n/index.js",
     "./uwx": "./src/uwx/index.js",
-    "./import-map-plugin": "./src/import-map-plugin.js"
+    "./import-map-plugin": "./src/import-map-plugin.js",
+    "./emit-surface": "./src/uwx/emit-surface.json"
   },
   "bin": {},
   "files": [
@@ -56,15 +57,15 @@
     "js-yaml": "^4.1.0",
     "sharp": "^0.35.3",
     "yaml": "^2.5.0",
-    "@uniweb/content-reader": "^1.2.4",
-    "@uniweb/schemas": "^0.2.13",
+    "@uniweb/projections": "^0.6.0",
     "@uniweb/semantic-parser": "^1.4.0",
-    "@uniweb/content-writer": "^0.3.4",
     "@uniweb/theming": "^0.1.15",
-    "@uniweb/projections": "^0.5.13"
+    "@uniweb/content-writer": "^0.3.4",
+    "@uniweb/schemas": "^0.2.13",
+    "@uniweb/content-reader": "^1.2.4"
   },
   "optionalDependencies": {
-    "@uniweb/runtime": "^0.19.3"
+    "@uniweb/runtime": "^0.19.4"
   },
   "peerDependencies": {
     "vite": "^5.0.0 || ^6.0.0 || ^7.0.0",
@@ -73,7 +74,7 @@
     "@tailwindcss/vite": "^4.0.0",
     "@vitejs/plugin-react": "^4.0.0 || ^5.0.0",
     "vite-plugin-svgr": "^4.0.0",
-    "@uniweb/core": "^0.24.2"
+    "@uniweb/core": "^0.24.3"
   },
   "peerDependenciesMeta": {
     "vite": {
@@ -99,6 +100,7 @@
     }
   },
   "scripts": {
-    "test": "vitest run"
+    "test": "vitest run",
+    "emit-surface": "node scripts/gen-emit-surface.mjs"
   }
 }

package/src/foundation/derive-supports.js

@@ -133,6 +133,30 @@ function isResolveServiceCall(node) {
   return false
 }
 
+/**
+ * A `.isServiceEnabled(name)` call on anything.
+ *
+ * ⛔ **Its name argument is at index 0, not 1.** `resolveService(website, name)`
+ * passes the website first; the method form has already bound it as the
+ * receiver. Reading index 1 here would find `undefined`, fall to the blind
+ * branch, and under-report `uniweb.supports` with a warning nobody is watching
+ * for — which is why `derive-supports.test.js` asserts a foundation calling
+ * `isSubmitEnabled()` derives `submit`.
+ *
+ * ⭐ This is how every `@uniweb/kit` service predicate is seen: they compile to
+ * `website.isServiceEnabled('<name>')`, and kit is bundled into the foundation,
+ * so the literal survives the shake. One matcher covers all of them — the name
+ * is an argument, so the family does not grow a rule per service.
+ */
+function isServiceEnabledCall(node) {
+  return (
+    node.type === 'CallExpression' &&
+    node.callee?.type === 'MemberExpression' &&
+    !node.callee.computed &&
+    node.callee.property?.name === 'isServiceEnabled'
+  )
+}
+
 /** A `.isSearchEnabled()` call on anything. */
 function isSearchEnabledCall(node) {
   return (
@@ -164,6 +188,11 @@ function collectSurvivors(bundle) {
 /**
  * Derive the services this foundation reaches for.
  *
+ * Three call shapes are read, and two of them carry the name as an argument so
+ * they cover the open registry without a rule per service:
+ * `resolveService(website, name)`, `website.isServiceEnabled(name)` (what every
+ * `@uniweb/kit` predicate compiles to), and the two hand-written gates below.
+ *
  * @param {object} bundle - Rollup's bundle, as handed to `writeBundle`
  * @param {object} ctx - the Rollup plugin context (`this` in the hook)
  * @returns {{services: string[], blind: boolean, blindAt: string[]}}
@@ -191,9 +220,12 @@ export function deriveSupports(bundle, ctx) {
         services.add('search')
         return
       }
-      if (!isResolveServiceCall(node)) return
+      const isMethod = isServiceEnabledCall(node)
+      if (!isMethod && !isResolveServiceCall(node)) return
 
-      const arg = node.arguments?.[1]
+      // The name is argument 0 on the method (the website is the receiver) and
+      // argument 1 on the free function. Getting this wrong under-reports.
+      const arg = node.arguments?.[isMethod ? 0 : 1]
       if (typeof arg?.value === 'string') {
         services.add(arg.value)
       } else if (arg?.type === 'Identifier' && consts.has(arg.name)) {

package/src/i18n/index.js

@@ -23,7 +23,8 @@ import {
   translateRecordData,
   RECORDS_DIR
 } from './records.js'
-import { generateSearchIndex, isSearchEnabled } from '@uniweb/projections'
+import { generateSearchIndex } from '@uniweb/projections'
+import { searchDeclaredOn } from '../site/search-declared.js'
 
 // Free-form translation support
 import {
@@ -400,7 +401,7 @@ export async function buildLocalizedContent(siteRoot, options = {}) {
     outputs[locale] = { content: contentOutputPath }
 
     // Generate search index for this locale if search is enabled
-    if (generateSearchIndexes && isSearchEnabled(translated)) {
+    if (generateSearchIndexes && searchDeclaredOn(translated)) {
       const searchConfig = translated.config?.search || {}
       const searchIndex = generateSearchIndex(translated, {
         locale,

package/src/site/plugin.js

@@ -209,8 +209,9 @@ async function processDevSectionFetches(sections, fetchOptions) {
     }
   }
 }
-import { generateSearchIndex, isSearchEnabled, getSearchIndexFilename } from '@uniweb/projections'
+import { generateSearchIndex, getSearchIndexFilename } from '@uniweb/projections'
 import { mergeTranslations } from '../i18n/merge.js'
+import { searchDeclaredOn } from './search-declared.js'
 import { mountDevApi } from '../dev/api-mount.js'
 
 /*
@@ -1145,7 +1146,7 @@ export function siteContentPlugin(options = {}) {
         // Serve search-index.json in dev mode (supports locale prefixes)
         const searchIndexMatch = req.url.match(new RegExp(`^(?:\\/(${LOCALE_RE}))?\\/search-index\\.json$`))
         if (searchIndexMatch && siteContent) {
-          const searchEnabled = searchPluginConfig.enabled !== false && isSearchEnabled(siteContent)
+          const searchEnabled = searchPluginConfig.enabled !== false && searchDeclaredOn(siteContent)
           if (searchEnabled) {
             const searchConfig = siteContent.config?.search || {}
             const defaultLocale = resolveDefaultLocale(siteContent.config)
@@ -1511,7 +1512,7 @@ export function siteContentPlugin(options = {}) {
       }
 
       // Generate search index if enabled
-      const searchEnabled = searchPluginConfig.enabled !== false && isSearchEnabled(finalContent)
+      const searchEnabled = searchPluginConfig.enabled !== false && searchDeclaredOn(finalContent)
       if (searchEnabled) {
         const searchConfig = finalContent.config?.search || {}
         const defaultLocale = resolveDefaultLocale(finalContent.config)

package/src/site/search-declared.js

@@ -0,0 +1,34 @@
+/**
+ * Is search on for this site, as the AUTHOR declared it?
+ *
+ * ⭐ **The build's own decision about its own output.** Deliberately not
+ * `@uniweb/projections`': that package is a helper library — it derives
+ * artifacts from content and must not gate on a site setting. Deciding whether
+ * to generate is the caller's job *[Diego, 2026-09-10]*.
+ *
+ * ⛔ **The boolean form is why this is not a one-liner.**
+ * `config?.search?.enabled !== false` reads `false.enabled` when an author
+ * writes `search: false`, because optional chaining short-circuits on
+ * `null`/`undefined` only — so `undefined !== false` is `true` and the natural
+ * spelling of "off" still shipped a `search-index.json` carrying every page's
+ * text. On a static export that file is world-readable. `Website.isSearchEnabled`
+ * fixed the same expression in `@uniweb/core`; the copy in projections never got
+ * it and now needs none. Pinned by `tests/search-declared.test.js`.
+ *
+ * ⚖️ **Author tier only, and that is correct here.** A host's `services` block
+ * decides where queries are *answered* at render; it has no bearing on whether
+ * this build emits an index. The render-time question is
+ * `Website.isServiceEnabled('search')`.
+ *
+ * ⛔ **A leaf on purpose.** It lived in `site/plugin.js` for one commit, which
+ * made `i18n/index.js` import a 26-module Vite plugin to reach four lines — and
+ * put one future import in the other direction away from a cycle.
+ *
+ * @param {object} siteContent - parsed site content, or anything with `.config`
+ * @returns {boolean}
+ */
+export function searchDeclaredOn(siteContent) {
+  const search = siteContent?.config?.search
+  if (typeof search === 'boolean') return search
+  return search?.enabled !== false
+}

package/src/uwx/emit-surface.json

@@ -0,0 +1,119 @@
+{
+  "schemaVersion": 1,
+  "model": "@uniweb/site-content",
+  "producer": "@uniweb/build/src/uwx/site.js",
+  "kinds": {
+    "closed": "A fixed key set. Every key is listed; a consumer may assert it.",
+    "passthrough": "Author keys copied verbatim, with no allowlist. ⛔ No closed key set exists — a consumer must NOT assert one.",
+    "records": "Content records with their own field shapes, not site.yml config keys."
+  },
+  "systemKeys": [
+    "$uuid",
+    "$id",
+    "$model"
+  ],
+  "sections": {
+    "info": {
+      "kind": "closed",
+      "keys": {
+        "description": [
+          "site.yml::description"
+        ],
+        "favicon": [
+          "site.yml::favicon"
+        ],
+        "foundation": [
+          "site.yml::foundation"
+        ],
+        "name": [
+          "site.yml::name"
+        ],
+        "tags": [
+          "site.yml::tags"
+        ],
+        "template": [
+          "site.yml::template"
+        ]
+      }
+    },
+    "settings": {
+      "kind": "closed",
+      "keys": {
+        "agents": [
+          "site.yml::agents"
+        ],
+        "assistant": [
+          "site.yml::assistant"
+        ],
+        "base": [
+          "site.yml::base"
+        ],
+        "build": [
+          "site.yml::build"
+        ],
+        "default_language": [
+          "site.yml::defaultLanguage"
+        ],
+        "fetch": [
+          "site.yml::fetch",
+          "site.yml::data"
+        ],
+        "fetcher": [
+          "site.yml::fetcher"
+        ],
+        "head_html": null,
+        "keywords": [
+          "site.yml::keywords"
+        ],
+        "languages": [
+          "site.yml::languages"
+        ],
+        "layout": [
+          "site.yml::layout"
+        ],
+        "paths": [
+          "site.yml::paths"
+        ],
+        "placeholders": [
+          "site.yml::placeholders"
+        ],
+        "publish_languages": [
+          "site.yml::publishLanguages"
+        ],
+        "search": [
+          "site.yml::search"
+        ],
+        "seo": [
+          "site.yml::seo"
+        ],
+        "submit": [
+          "site.yml::submit"
+        ],
+        "theme": null,
+        "tracking": [
+          "site.yml::tracking"
+        ]
+      }
+    },
+    "services": {
+      "kind": "passthrough",
+      "authoredAs": "site.yml::$services"
+    },
+    "secrets": {
+      "kind": "passthrough",
+      "authoredAs": "site.yml::$secrets"
+    },
+    "pages": {
+      "kind": "records"
+    },
+    "layout_sections": {
+      "kind": "records"
+    },
+    "extensions": {
+      "kind": "records"
+    },
+    "queries": {
+      "kind": "records"
+    }
+  }
+}

package/src/uwx/site.js

@@ -852,7 +852,9 @@ function queriesNested(declarations, uuids = null) {
 // config is bound where the service is provisioned, and arrives here by `pull`.
 //
 // ⚖️ WHAT THESE ARE NOT. A site's OWN service declarations — `search:`, `submit:`,
-// `assistant:`, `tracking:` — stay top-level `info.*` keys and are untouched. Those
+// `assistant:`, `tracking:` — ride the `settings` Section and are untouched here.
+// (They were top-level `info.*` keys until 2026-09-09; `settingsNested` below is
+// where they are now written.) Those
 // are authored, they resolve at the SITE tier (`config.<name>`, first choice in
 // `@uniweb/core`'s `resolveService`), and moving them here would flip them to the
 // host tier, where a block's mere presence declines every service it does not name.