@uniweb/core 0.24.3 → 0.24.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uniweb/core",
-  "version": "0.24.3",
+  "version": "0.24.4",
   "description": "Core classes for the Uniweb platform - Uniweb, Website, Page, Block",
   "type": "module",
   "exports": {

package/src/services.js

@@ -12,10 +12,14 @@ import { applyBasePath } from './base-path.js'
  * address comes from configuration, and there are exactly two places it can
  * come from:
  *
- *   1. **The site**, authored — `search:`, `submit:`, `assistant:`, `tracking:`
- *      in site.yml. The operator's own declaration, and it wins.
- *   2. **The host**, served — `config.services.<name>` in the payload. What the
- *      deployment offers, which the site never had to know about.
+ *   1. **The host**, served — `config.services.<name>` in the payload. What the
+ *      deployment offers, which the site never had to know about. When a host
+ *      offers a service, that is the answer: it is the authority on what a
+ *      hosted site is given.
+ *   2. **The site**, authored — `search:`, `submit:`, `assistant:`, `tracking:`
+ *      in site.yml. The operator's own declaration — used for anything the host
+ *      does not provide, and on a static site, where no host speaks, the whole
+ *      answer.
  *
  * Absent from both means the site has no such service, and the caller acts on
  * that rather than guessing an address. That is the same rule for every service,
@@ -117,7 +121,37 @@ const ABSOLUTE_URL_RE = /^(?:[a-z][a-z0-9+.-]*:|\/\/)/i
  * @param {*} declaration
  * @returns {string} the endpoint, or '' when there is none
  */
+/**
+ * Has this declaration switched the service OFF?
+ *
+ * ⭐ `false`, or an object with `enabled: false` — on either tier, for every
+ * name. It is the spelling `search` has always honoured
+ * (`Website.isSearchEnabled`), now read for every service.
+ *
+ * ⛔ **Until 2026-09-10 this module read it for NO service** — so the same
+ * spelling behaved two ways: `search: false` turned search off, while
+ * `submit: false` or `submit: { endpoint, enabled: false }` drew the form
+ * anyway. On a static site the authored key is the only place a service can be
+ * declared, so there the switch worked for search alone.
+ *
+ * ⚖️ **A site's refusal does not override a host's offer** — step 3 of
+ * `resolveService` says why.
+ *
+ * Module-private on purpose: it is a rule of the two readers below, not a
+ * question a caller should ask. `resolveService` is the question.
+ */
+function isServiceRefused(declaration) {
+  return (
+    declaration === false ||
+    (declaration !== null && typeof declaration === 'object' && declaration.enabled === false)
+  )
+}
+
 export function readEndpoint(declaration) {
+  // A refused service offers no address, whatever else its block carries — so
+  // a host row with `enabled: false` reads as a decline, and the records lane
+  // (`records-service.js`) withholds its address the same way.
+  if (isServiceRefused(declaration)) return ''
   if (typeof declaration === 'string') return declaration.trim()
   if (typeof declaration?.endpoint === 'string') return declaration.endpoint.trim()
   return ''
@@ -168,28 +202,42 @@ export function resolveServiceUrl(endpoint, basePath = '') {
  * @param {string} name - service name, e.g. 'submit' · 'search' · 'tracking'
  * @returns {{ url: string|null, source: 'site'|'host'|null }}
  *   `url` is the whole answer for acting. `source` says which declaration
- *   answered — a diagnostic, and the thing to check when a host's value appears
- *   not to be taking effect. `'host'` with a null `url` means the host answered
+ *   answered — a diagnostic, and the thing to check when a value you set
+ *   appears not to be taking effect. `'host'` with a null `url` means the host answered
  *   and offered no address; `null` means nothing declared the service at all.
  */
 export function resolveService(website, name) {
   const config = website?.config
   const basePath = website?.basePath
 
-  // 1 — the site's own declaration wins. An operator who named an endpoint
-  // means it, including on a host that offers one.
-  const authored = readEndpoint(config?.[name])
-  if (authored) {
-    return { url: resolveServiceUrl(authored, basePath), source: 'site' }
-  }
-
-  // 2 — what the host says it offers.
+  // 1 — what the host OFFERS. A host that returns an address is the authority
+  // on what a hosted site is given: nothing the site declares overrides it —
+  // not its own address (step 2), not its off switch (step 3).
   const hostDeclaration = config?.services?.[name]
   const hostEndpoint = readEndpoint(hostDeclaration)
   if (hostEndpoint) {
     return { url: resolveServiceUrl(hostEndpoint, basePath), source: 'host' }
   }
 
+  // 2 — the site's own ADDRESS, for a service the host does not provide: an
+  // operator bringing their own provider — a form service, self-hosted search
+  // on a host that does not sell it — keeps working. On a static site no host
+  // speaks, so this is the whole answer. (Until 2026-09-10 it came first and
+  // outranked a host's offer too.)
+  const authored = readEndpoint(config?.[name])
+  if (authored) {
+    return { url: resolveServiceUrl(authored, basePath), source: 'site' }
+  }
+
+  // 3 — ⛔ A SITE THAT SWITCHED THE SERVICE OFF — but only where no host offers
+  // it. A host that returns an address is the authority on what a hosted site
+  // is given: the site asked for the service where the host manages it, and a
+  // hosted service is turned off there — not with a second switch here that
+  // would hide a service the host is still providing. On a static site no host
+  // speaks, so this is the only switch there is, and it decides. (It briefly
+  // ran first, on 2026-09-10, and beat a host's offer — the wrong way round.)
+  if (isServiceRefused(config?.[name])) return { url: null, source: 'site' }
+
   // A host may declare the name while offering no address — a decline. It is
   // still the host answering, which is all a caller can use: any *wording* for
   // that state would be ours to invent, in one language, for a visitor who has

package/src/website.js

@@ -1140,13 +1140,15 @@ export default class Website {
    * @returns {boolean}
    */
   isSearchEnabled() {
-    const search = this.config?.search
-    if (typeof search === 'boolean') {
-      if (!search) return false
-    } else if (search?.enabled === false) {
-      return false
-    }
-
+    // ⭐ THE ORDER IS resolveService's, the same for every service: the host's
+    // offer, then the site's own endpoint, then the site's `false` /
+    // `{ enabled: false }` — which decides only where no host offers search. So
+    // an author's `search: false` turns the local index off on a static site
+    // and does not hide search a host provides; a hosted service is turned off
+    // where the host provides it. Until 2026-09-10 this method read the
+    // author's switch FIRST, so `search: false` beat a host's offer — the one
+    // service where it did, and the opposite of every other.
+    //
     // ⭐ THE HOST GETS A SAY, and this is the half that reaches an already-
     // published foundation. A foundation bundles its own frozen copy of
     // `@uniweb/kit`, so a fix made in kit never reaches one built before it.
@@ -1164,14 +1166,18 @@ export default class Website {
     // error, and any text a visitor reads is site content — authored and
     // localized — never a string a service layer invents.
     //
-    // A site's OWN `search.endpoint` still wins: `resolveService` answers from
-    // the site tier first, so self-hosted search on a host that does not sell
-    // it is untouched.
+    // A site's own `search.endpoint` still works where the host does not offer
+    // search, so self-hosted search on a host that does not sell it is
+    // untouched. Where the host offers search, its offer wins.
     const { url, source } = resolveService(this, 'search')
-    if (source === 'host' && !url) return false
-
-    // Enabled by default — absent means enabled, and a host that publishes no
-    // services block at all has declined nothing (the static-host path).
+    // An address from anyone — the site's own endpoint, or a host's offer.
+    if (url) return true
+    // 'site': switched off, with no host offering it. 'host': the host declines.
+    if (source) return false
+
+    // Nobody spoke: enabled by default — absent means enabled, and a host that
+    // publishes no services block at all has declined nothing (the static-host
+    // path, where the build emits the local index).
     return true
   }