@uniweb/build 1.8.2 → 1.8.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uniweb/build",
-  "version": "1.8.2",
+  "version": "1.8.4",
   "description": "Build tooling for the Uniweb Component Web Platform",
   "type": "module",
   "exports": {
@@ -56,15 +56,15 @@
     "js-yaml": "^4.1.0",
     "sharp": "^0.35.3",
     "yaml": "^2.5.0",
-    "@uniweb/content-reader": "~1.5.4",
-    "@uniweb/projections": "~1.1.7",
+    "@uniweb/content-reader": "~1.5.6",
+    "@uniweb/semantic-parser": "~1.13.0",
     "@uniweb/schemas": "~1.6.5",
-    "@uniweb/semantic-parser": "~1.12.2",
-    "@uniweb/theming": "~1.2.1",
-    "@uniweb/content-writer": "~1.3.0"
+    "@uniweb/content-writer": "~1.3.0",
+    "@uniweb/projections": "~1.1.7",
+    "@uniweb/theming": "~1.2.1"
   },
   "optionalDependencies": {
-    "@uniweb/runtime": "~1.1.16"
+    "@uniweb/runtime": "~1.1.18"
   },
   "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.1",
-    "@uniweb/runtime": "~1.1.16"
+    "@uniweb/runtime": "~1.1.18",
+    "@uniweb/core": "~1.4.3"
   },
   "peerDependenciesMeta": {
     "vite": {

package/src/i18n/merge.js

@@ -17,19 +17,16 @@ import { loadFreeformTranslation } from './freeform.js'
 import { elementText, blockElements, translationContext, SITE_META_CONTEXT } from './extract.js'
 import { visitDataBlockStrings } from './data-strings.js'
 
-// Inline-markdown → ProseMirror inline fragment, for resolving a whole-element
+// Inline markdown → ProseMirror inline nodes, for resolving a whole-element
 // translation VALUE (which carries marks/links/icons as inline markdown). Same
 // lazy-import-with-fallback pattern as freeform.js / collection-processor.js, so
 // the synchronous merge path has the converter ready at call time.
-let markdownToProseMirror
+let inlineMarkdownToProseMirror
 try {
   const contentReader = await import('@uniweb/content-reader')
-  markdownToProseMirror = contentReader.markdownToProseMirror
+  inlineMarkdownToProseMirror = contentReader.inlineMarkdownToProseMirror
 } catch {
-  markdownToProseMirror = (markdown) => ({
-    type: 'doc',
-    content: [{ type: 'paragraph', content: [{ type: 'text', text: markdown.trim() }] }]
-  })
+  inlineMarkdownToProseMirror = (markdown) => [{ type: 'text', text: markdown.trim() }]
 }
 
 /**
@@ -183,9 +180,6 @@ async function mergeTranslationsAsync(siteContent, translations, options) {
   return translated
 }
 
-// The blocks that hold words, which `toPlainText` reads as one run each.
-const TEXT_BLOCKS = new Set(['paragraph', 'heading', 'codeBlock'])
-
 /**
  * ⭐ A TRANSLATION VALUE IS INLINE MARKDOWN, AND A PLAIN-TEXT SLOT RENDERS ITS TEXT.
  *
@@ -221,17 +215,19 @@ export function toPlainText(value) {
     return value
   }
   try {
-    const doc = markdownToProseMirror(value)
-    // A block's words run together; blocks are apart. ⛔ Until 2026-10-16 every level joined with
-    // nothing, so a value of two paragraphs read as one word at the seam ("…17hFermé…").
+    // Each paragraph's words, read as inline markdown and never as blocks, so `1. *Introduction*`
+    // keeps its number; paragraphs are apart. ⛔ Until 2026-10-16 every level joined with nothing,
+    // so a value of two paragraphs read as one word at the seam ("…17hFermé…"). ⛔ Until 2026-10-24
+    // the value was read as a document, and one that began like a block lost its marker: an ordered
+    // list's number, a bullet, a `#`.
     const inline = (nodes) =>
       (nodes || []).map((n) => (n.type === 'text' ? n.text || '' : n.type === 'hardBreak' ? ' ' : inline(n.content))).join('')
-    const blocks = (nodes) =>
-      (nodes || [])
-        .map((n) => (TEXT_BLOCKS.has(n.type) ? inline(n.content) : n.type === 'text' ? n.text || '' : blocks(n.content)))
-        .filter((t) => t.trim())
-        .join(' ')
-    const text = blocks(doc?.content).trim()
+    const text = value
+      .split(/\n[ \t]*\n/)
+      .map((paragraph) => inline(inlineMarkdownToProseMirror(paragraph.trim())))
+      .filter((t) => t.trim())
+      .join(' ')
+      .trim()
     return text || value
   } catch {
     return value
@@ -639,17 +635,15 @@ export function resolveDocForLocale(sourceDoc, table, context = { page: '', sect
   return elements || data ? doc : null
 }
 
-// Parse an inline-markdown translation value into a ProseMirror inline fragment.
-// A value is expected to be one element's inline content (one paragraph once
-// parsed); take that paragraph's inline children. If the converter yields
-// several blocks, flatten their inline content rather than drop any.
+// Parse an inline-markdown translation value into a ProseMirror inline fragment: one element's
+// words, read as inline markdown and never as blocks (`inlineMarkdownToProseMirror`).
+// ⛔ Until 2026-10-24 it was read as a document and each block's content taken, so a value that
+// began like a block — `1. Informations que nous collectons`, the French of a numbered heading —
+// was an ordered list, and the heading was handed its list item. It rendered empty, and a pull
+// refused the site, since no file can say a heading holds a list item.
 function inlineMarkdownToFragment(value) {
   if (typeof value !== 'string' || value.trim() === '') return null
-  const doc = markdownToProseMirror(value)
-  const inline = []
-  for (const block of doc?.content || []) {
-    if (Array.isArray(block.content)) inline.push(...block.content)
-  }
+  const inline = inlineMarkdownToProseMirror(value.trim())
   return inline.length ? inline : [{ type: 'text', text: value.trim() }]
 }
 

package/src/site/content-collector.js

@@ -241,6 +241,22 @@ function detectVersions(folderNames) {
   return versions
 }
 
+/**
+ * An id an author wrote — a page's `id:` in `page.yml`, a section's in its frontmatter — as the
+ * string it names. YAML reads `id: 404` as a number, and an id is never one: a `page:404` link
+ * names the string, and a store holds a stable id as text.
+ *
+ * ⛔ *Until 2026-10-24 the parsed value was kept, so a page with `id: 404` matched no `page:`
+ * link, and a push sent its stable id as a number, which a backend refuses (measured that day:
+ * HTTP 400, "item.data.stable_id must be a string").*
+ *
+ * @param {*} value
+ * @returns {string|null} null for an id not written, or written empty
+ */
+export function authoredId(value) {
+  return value === undefined || value === null || value === '' ? null : String(value)
+}
+
 /**
  * `query:` — the shorthand for `fetch: { query }`, at every level that declares
  * data: a section's frontmatter, `page.yml`, `folder.yml` and `site.yml`, on the
@@ -1168,7 +1184,7 @@ async function processMarkdownFile(filePath, id, siteRoot, defaultStableId = nul
 
   // Stable ID for scroll targeting: frontmatter id > filename-derived > null
   // This ID is stable across reordering (unlike the positional id)
-  const stableId = frontmatterId || defaultStableId || null
+  const stableId = authoredId(frontmatterId) || defaultStableId || null
 
   const section = {
     id,
@@ -1434,7 +1450,7 @@ async function processPage(pagePath, pageName, siteRoot, { isIndex = false, pare
       page: {
         route,
         sourcePath: parentRoute === '/' ? `/${pageName}` : `${parentRoute}/${pageName}`,
-        id: pageConfig.id || null,
+        id: authoredId(pageConfig.id),
         isIndex,
         title: resolveDisplayTitle(pageConfig.title, pageName, null),
         description: pageConfig.description || '',
@@ -1460,7 +1476,7 @@ async function processPage(pagePath, pageName, siteRoot, { isIndex = false, pare
       page: {
         route,
         sourcePath: parentRoute === '/' ? `/${pageName}` : `${parentRoute}/${pageName}`,
-        id: pageConfig.id || null,
+        id: authoredId(pageConfig.id),
         isIndex,
         title: resolveDisplayTitle(pageConfig.title, pageName, null),
         description: pageConfig.description || '',
@@ -1731,7 +1747,7 @@ async function processPage(pagePath, pageName, siteRoot, { isIndex = false, pare
     page: {
       route,
       sourcePath, // Original folder-based path (for ancestor checking in navigation)
-      id: pageConfig.id || null, // Stable page ID for page: links (survives reorganization)
+      id: authoredId(pageConfig.id), // Stable page ID for page: links (survives reorganization)
       isIndex, // Marks this page as the index for its parent route
       title: resolveDisplayTitle(pageConfig.title, pageName, hierarchicalSections),
       description: pageConfig.description || '',

package/src/uwx/site.js

@@ -71,6 +71,7 @@ import {
   rootOrderConfig,
   composeLocalizedRoute,
   isValidSlugSegment,
+  authoredId,
 } from '../site/content-collector.js'
 import { refuseBinding, refuseUnder, refuseOutsideLanguage, warnDuplicateBindings } from '../site/data-fetcher.js'
 import { readLayoutFolder, layoutAreaRoute } from '../site/layout-folder.js'
@@ -312,7 +313,8 @@ function buildPageData(config, ctx) {
   // `localizeRouteSlugs`, once the whole tree is known. `mode` is the plain
   // delivery mode.
   const data = { slug: { [sourceLocale]: slug }, mode } // both required by the entity type
-  setIf(data, 'stable_id', config.id)
+  // An authored id is a string (`authoredId`): `id: 404` reads as a number, which a store refuses.
+  setIf(data, 'stable_id', authoredId(config.id) ?? undefined)
   setIf(data, 'title', localizeScalar(config.title, sourceLocale, translations))
   setIf(data, 'description', localizeScalar(config.description, sourceLocale, translations))
   setIf(data, 'label', localizeScalar(config.label, sourceLocale, translations))