@uniweb/build 1.10.4 → 1.10.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uniweb/build",
-  "version": "1.10.4",
+  "version": "1.10.6",
   "description": "Build tooling for the Uniweb Component Web Platform",
   "type": "module",
   "exports": {
@@ -56,15 +56,15 @@
     "js-yaml": "^4.3.2",
     "sharp": "^0.35.5",
     "yaml": "^2.5.0",
-    "@uniweb/semantic-parser": "~1.13.0",
     "@uniweb/content-reader": "~1.5.7",
-    "@uniweb/content-writer": "~1.3.1",
     "@uniweb/schemas": "~1.6.5",
-    "@uniweb/projections": "~1.2.1",
-    "@uniweb/theming": "~1.2.1"
+    "@uniweb/semantic-parser": "~1.13.0",
+    "@uniweb/theming": "~1.2.1",
+    "@uniweb/projections": "~1.2.2",
+    "@uniweb/content-writer": "~1.3.1"
   },
   "optionalDependencies": {
-    "@uniweb/runtime": "~1.2.3"
+    "@uniweb/runtime": "~1.2.4"
   },
   "peerDependencies": {
     "vite": "^5.0.0 || ^6.0.0 || ^7.0.0",
@@ -73,8 +73,8 @@
     "@tailwindcss/vite": "^4.0.0",
     "@vitejs/plugin-react": "^4.0.0 || ^5.0.0",
     "vite-plugin-svgr": "^4.0.0",
-    "@uniweb/core": "~1.4.7",
-    "@uniweb/runtime": "~1.2.3"
+    "@uniweb/core": "~1.4.8",
+    "@uniweb/runtime": "~1.2.4"
   },
   "peerDependenciesMeta": {
     "vite": {

package/src/site/content-collector.js

@@ -40,6 +40,7 @@ import { resolveDefaultLocale, resolvePublishableLocales, validateLanguageConfig
 import { authoredRedirectTarget } from '@uniweb/core/resolve-route'
 import { parentRouteOf } from '@uniweb/core/route-match'
 import { parseFrontmatter } from '../utils/frontmatter.js'
+import { valuesCutByComment } from '../utils/yaml-comment-trap.js'
 import { parseGrid } from '@uniweb/schemas/grid'
 import { runtimeServicesConfig, refuseRetiredServiceKeys, refuseUnreadableServices } from '../uwx/services-request.js'
 import { refuseUnreadableAddress } from './site-address.js'
@@ -438,6 +439,20 @@ let yamlFailures = []
  */
 let sectionTypes = null
 
+/** The site being collected, so a warning names a file as its author sees it. Reset per collect, as `yamlFailures` is. */
+let collectRoot = null
+
+/**
+ * Say each value the file's YAML cut short at a `#` its author meant as part of it — a
+ * gradient, a color (`../utils/yaml-comment-trap.js`). A warning, not a failure: the YAML is
+ * valid, and the site builds without the value, as it did before anything said so.
+ */
+function warnCutByComment(text, parsed, file) {
+  for (const { message } of valuesCutByComment(text, parsed)) {
+    console.warn(`[content-collector] ${collectRoot ? relative(collectRoot, file) : file}: ${message}`)
+  }
+}
+
 /**
  * Parse YAML, recording a failure against the file it came from.
  *
@@ -449,7 +464,9 @@ let sectionTypes = null
  */
 function parseYaml(yamlString, source = '<unknown>') {
   try {
-    return yaml.load(yamlString, YAML_OPTIONS) || {}
+    const parsed = yaml.load(yamlString, YAML_OPTIONS) || {}
+    warnCutByComment(yamlString, parsed, source)
+    return parsed
   } catch (err) {
     yamlFailures.push({ source, message: err.message })
     console.warn(`[content-collector] YAML parse error in ${source}: ${err.message}`)
@@ -1097,6 +1114,7 @@ async function processMarkdownFile(filePath, id, siteRoot, defaultStableId = nul
     const parsed = parseFrontmatter(content, filePath)
     frontMatter = parsed.frontmatter
     markdown = parsed.body
+    warnCutByComment(parsed.yaml, frontMatter, filePath)
   } catch (err) {
     yamlFailures.push({ source: filePath, message: err.message })
     console.warn(`[content-collector] ${err.message}`)
@@ -2556,6 +2574,7 @@ export async function collectSiteContent(sitePath, options = {}) {
   // still be counted.
   yamlFailures = []
   sectionTypes = null
+  collectRoot = sitePath
 
   // Read site config and raw theme config
   const siteConfig = await readYamlFile(join(sitePath, configFile))

package/src/site/deploy-config-writer.js

@@ -308,11 +308,14 @@ export async function forgetDeployYml(siteDir) {
  *
  * A target's record moves to `deploys:` when that target has none there; when it has, the
  * one in `deploys:` is newer, and the old is dropped. `autoSave: off` becomes
- * `saveDeploys: false`; any other value is the default, and goes.
+ * `saveDeploys: false`; any other value is the default, and goes. And `saveDeploys:` holding
+ * one of `autoSave:`'s values — the key renamed by hand, its value kept — is read the same way:
+ * `off` is `false`, `lastDeploy` the default.
  *
  * @param {string} siteDir
- * @returns {Promise<{ moved: string[], dropped: string[], saveDeploys: boolean|null } | null>}
- *   null when the file has neither key, or there is no file
+ * @returns {Promise<{ moved: string[], dropped: string[], saveDeploys: boolean|null, renamed: string|null } | null>}
+ *   `renamed` is the old value found under `saveDeploys:`; null when the file has none of
+ *   these, or there is no file
  */
 export async function retireDeployYmlKeys(siteDir) {
   const path = join(siteDir, 'deploy.yml')
@@ -321,7 +324,9 @@ export async function retireDeployYmlKeys(siteDir) {
   if (doc.errors.length) throw new Error(`${path}: ${doc.errors[0].message}`)
   const old = doc.get('lastDeploy')
   const autoSave = doc.get('autoSave')
-  if (old === undefined && autoSave === undefined) return null
+  const renamed = doc.get('saveDeploys')
+  const kept = renamed === 'off' || renamed === 'lastDeploy' ? renamed : undefined
+  if (old === undefined && autoSave === undefined && kept === undefined) return null
   const moved = []
   const dropped = []
   if (isMap(old)) {
@@ -338,6 +343,14 @@ export async function retireDeployYmlKeys(siteDir) {
   }
   doc.delete('lastDeploy')
   let saveDeploys = null
+  if (kept !== undefined) {
+    if (kept === 'off') {
+      doc.set('saveDeploys', false)
+      saveDeploys = false
+    } else {
+      doc.delete('saveDeploys')
+    }
+  }
   if (autoSave !== undefined) {
     if (String(autoSave) === 'off' && !doc.has('saveDeploys')) {
       doc.set('saveDeploys', false)
@@ -346,5 +359,5 @@ export async function retireDeployYmlKeys(siteDir) {
     doc.delete('autoSave')
   }
   await writeFile(path, String(doc))
-  return { moved, dropped, saveDeploys }
+  return { moved, dropped, saveDeploys, renamed: kept ?? null }
 }

package/src/site/deploy-config.js

@@ -186,7 +186,15 @@ function validate(doc, path) {
   // now: two targets naming DIFFERENT backends are two sites, and two naming the SAME
   // backend are one site shipped twice. The reason is gone, so the rejection is too.
   if (doc.saveDeploys !== undefined && typeof doc.saveDeploys !== 'boolean') {
-    throw new Error(`${path}: \`saveDeploys\` must be true or false.`)
+    // `lastDeploy` and `off` were the values of `autoSave:`, which `saveDeploys:` replaced: the
+    // key renamed by hand, its value kept (measured 2026-10-26 on a copied project).
+    const retired = doc.saveDeploys === 'lastDeploy' || doc.saveDeploys === 'off'
+    throw new Error(
+      `${path}: \`saveDeploys\` must be true or false` +
+        (retired
+          ? ` — \`${doc.saveDeploys}\` is a value of the retired \`autoSave:\`. \`uniweb doctor --fix deploy-yml-retired-key\` writes what it meant.`
+          : '.')
+    )
   }
   if (doc.default !== undefined && typeof doc.default !== 'string') {
     throw new Error(`${path}: \`default\` must be a string.`)

package/src/utils/frontmatter.js

@@ -51,24 +51,25 @@ import { YAML_OPTIONS } from './yaml-schema.js'
  * @param {string} [filepath] - where it came from. Optional only because a
  *   caller may genuinely not have one; pass it whenever you do, since the file
  *   name is the difference between a fixable error and a hunt.
- * @returns {{ frontmatter: object, body: string }}
+ * @returns {{ frontmatter: object, body: string, yaml: string }} `yaml` is the block as
+ *   written, for a caller that reads what the parse cannot say (`./yaml-comment-trap.js`)
  * @throws {Error} when a `---` block is present and does not parse as YAML
  */
 export function parseFrontmatter(raw, filepath) {
   const text = raw ?? ''
   if (!text.trimStart().startsWith('---')) {
-    return { frontmatter: {}, body: text }
+    return { frontmatter: {}, body: text, yaml: '' }
   }
 
   const parts = text.split('---\n')
   if (parts.length < 3) {
-    return { frontmatter: {}, body: text }
+    return { frontmatter: {}, body: text, yaml: '' }
   }
 
   try {
     const frontmatter = yaml.load(parts[1], YAML_OPTIONS) || {}
     const body = parts.slice(2).join('---\n')
-    return { frontmatter, body }
+    return { frontmatter, body, yaml: parts[1] }
   } catch (err) {
     const where = filepath ? `${filepath}: ` : ''
     throw new Error(

package/src/utils/yaml-comment-trap.js

@@ -0,0 +1,77 @@
+/**
+ * Values YAML cut short at a `#` the author meant as part of them.
+ *
+ * YAML reads a `#` with a space before it as the start of a comment, so in a plain (unquoted)
+ * value it ends the value there: `gradient: linear-gradient(165deg, #0c1226 0%, #05060f 100%)`
+ * is `linear-gradient(165deg,` — an invalid value, which the browser drops — and
+ * `primary: #3b82f6` is no value at all. YAML is right; nothing in the file is malformed, so
+ * nothing fails, and the color or the gradient is just not there.
+ *
+ * Said only where the file shows it, so a comment meant as one is never questioned:
+ *   - **cut** — a value left with a `(` it never closes, followed in the file by ` #`;
+ *   - **empty** — a key left with no value, followed in the file by a color: `#` and 3 to 8
+ *     hex digits.
+ *
+ * ⭐ Quoting the value is the fix, and the one the message gives:
+ * `gradient: "linear-gradient(165deg, #0c1226 0%, #05060f 100%)"`. Measured 2026-10-26: a book
+ * page's gradient had not shown since the page was written, in either renderer, and nothing
+ * said so.
+ *
+ * @module @uniweb/build/utils/yaml-comment-trap
+ */
+
+const escapeRe = (s) => s.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
+
+/** More `(` than `)`: a function the value opens and never closes. */
+function unclosed(value) {
+  let depth = 0
+  for (const ch of value) {
+    if (ch === '(') depth++
+    else if (ch === ')' && depth > 0) depth--
+  }
+  return depth > 0
+}
+
+/**
+ * @param {string} text - the YAML as written
+ * @param {*} parsed - what it parsed to
+ * @returns {{ path: string, kind: 'cut'|'empty', message: string }[]}
+ */
+export function valuesCutByComment(text, parsed) {
+  if (!text || !text.includes('#') || !parsed || typeof parsed !== 'object') return []
+  const found = []
+  const walk = (value, path, key) => {
+    if (Array.isArray(value)) {
+      value.forEach((v, i) => walk(v, `${path}[${i}]`, null))
+      return
+    }
+    if (value && typeof value === 'object' && !(value instanceof Date)) {
+      for (const [k, v] of Object.entries(value)) walk(v, path ? `${path}.${k}` : k, k)
+      return
+    }
+    if (typeof value === 'string' && unclosed(value)) {
+      const m = text.match(new RegExp(`${escapeRe(value)}[ \\t]+(#.*?)[ \\t]*$`, 'm'))
+      if (!m) return
+      const meant = `${value} ${m[1]}`
+      found.push({
+        path,
+        kind: 'cut',
+        message:
+          `\`${path}\` is cut short at \`${value}\` — YAML reads a \`#\` after a space as a comment. ` +
+          `Quote the value: ${key ?? path}: "${meant.replace(/"/g, '\\"')}"`
+      })
+      return
+    }
+    if (value === null && key) {
+      const m = text.match(new RegExp(`^[ \\t]*(?:-[ \\t]+)?${escapeRe(key)}[ \\t]*:[ \\t]+(#[0-9A-Fa-f]{3,8})\\b`, 'm'))
+      if (!m) return
+      found.push({
+        path,
+        kind: 'empty',
+        message: `\`${path}\` is empty — YAML reads \`${m[1]}\` as a comment. Quote the value: ${key}: "${m[1]}"`
+      })
+    }
+  }
+  walk(parsed, '', null)
+  return found
+}