@uniweb/build 1.8.3 → 1.8.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uniweb/build",
-  "version": "1.8.3",
+  "version": "1.8.5",
   "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.5",
-    "@uniweb/projections": "~1.1.7",
+    "@uniweb/content-reader": "~1.5.6",
     "@uniweb/content-writer": "~1.3.0",
     "@uniweb/schemas": "~1.6.5",
     "@uniweb/semantic-parser": "~1.13.0",
+    "@uniweb/projections": "~1.1.7",
     "@uniweb/theming": "~1.2.1"
   },
   "optionalDependencies": {
-    "@uniweb/runtime": "~1.1.18"
+    "@uniweb/runtime": "~1.1.19"
   },
   "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.3",
-    "@uniweb/runtime": "~1.1.18"
+    "@uniweb/core": "~1.4.4",
+    "@uniweb/runtime": "~1.1.19"
   },
   "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/i18n/sync.js

@@ -26,6 +26,8 @@ export function syncManifests(previous, current) {
 
   const previousHashes = new Set(Object.keys(previousUnits))
   const currentHashes = new Set(Object.keys(currentUnits))
+  // Previous units a new one replaced, so that no two new units claim the same one.
+  const replaced = new Set()
 
   // Check each current unit
   for (const hash of currentHashes) {
@@ -66,10 +68,11 @@ export function syncManifests(previous, current) {
     } else {
       // New hash - could be new content or changed content
       // Check if any previous context now has this new hash
-      const matchingContext = findMatchingContext(currentUnit.contexts, previousUnits)
+      const matchingContext = findMatchingContext(currentUnit.contexts, previousUnits, currentHashes, replaced)
 
       if (matchingContext) {
         // Same context, different hash = content changed
+        replaced.add(matchingContext.hash)
         report.changed.push({
           hash,
           previousHash: matchingContext.hash,
@@ -94,8 +97,7 @@ export function syncManifests(previous, current) {
     if (!currentHashes.has(hash)) {
       const prevUnit = previousUnits[hash]
       // Only mark as removed if not detected as changed above
-      const wasChanged = report.changed.some(c => c.previousHash === hash)
-      if (!wasChanged) {
+      if (!replaced.has(hash)) {
         report.removed.push({
           hash,
           source: prevUnit.source,
@@ -128,16 +130,22 @@ function contextsEqual(contexts1, contexts2) {
 }
 
 /**
- * Find if any context in the current unit matches a context in previous units
- * Returns the previous unit info if found
+ * The previous unit a new one replaced: one in a context of the new unit that is gone from the
+ * current manifest and not yet replaced. Returns its info, or null.
+ *
+ * ⛔ Until 2026-10-24 it was the first previous unit in such a context, gone or not. Removing a
+ * hero's second button then reported the hero's unchanged title as changed into the remaining
+ * button, and the button paragraph it did replace as removed (measured that day on a site's
+ * `uniweb i18n extract`).
  */
-function findMatchingContext(currentContexts, previousUnits) {
+function findMatchingContext(currentContexts, previousUnits, currentHashes, replaced) {
   // One key shape for both lanes: a page/section pair, or a record's identity.
   const key = (c) => (c.record ? `record:${c.record}` : `${c.page}:${c.section}`)
   for (const context of currentContexts) {
     const contextKey = key(context)
 
     for (const [hash, unit] of Object.entries(previousUnits)) {
+      if (currentHashes.has(hash) || replaced.has(hash)) continue
       const unitContexts = unit.contexts || []
       const hasContext = unitContexts.some((c) => key(c) === contextKey)
       if (hasContext) {