@adobe/helix-html-pipeline 6.28.2 → 6.28.4

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [6.28.4](https://github.com/adobe/helix-html-pipeline/compare/v6.28.3...v6.28.4) (2026-04-13)
+
+
+### Bug Fixes
+
+* treat section metadata style values like block options ([#1071](https://github.com/adobe/helix-html-pipeline/issues/1071)) ([#1072](https://github.com/adobe/helix-html-pipeline/issues/1072)) ([f6594bc](https://github.com/adobe/helix-html-pipeline/commit/f6594bcc7eca53e3416f085c6599428122d281ca))
+
+## [6.28.3](https://github.com/adobe/helix-html-pipeline/compare/v6.28.2...v6.28.3) (2026-04-10)
+
+
+### Bug Fixes
+
+* support mixed and comma-separated section metadata values ([#1068](https://github.com/adobe/helix-html-pipeline/issues/1068)) ([f18796b](https://github.com/adobe/helix-html-pipeline/commit/f18796b6a6fc03eda3ac9b0a9df86c0b8e590ea1))
+
 ## [6.28.2](https://github.com/adobe/helix-html-pipeline/compare/v6.28.1...v6.28.2) (2026-04-09)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-html-pipeline",
-  "version": "6.28.2",
+  "version": "6.28.4",
   "description": "Helix HTML Pipeline",
   "main": "src/index.js",
   "types": "src/index.d.ts",
@@ -84,7 +84,7 @@
     "esmock": "2.7.3",
     "husky": "9.1.7",
     "js-yaml": "4.1.1",
-    "jsdom": "28.1.0",
+    "jsdom": "29.0.1",
     "junit-report-builder": "5.1.1",
     "lint-staged": "16.4.0",
     "mocha": "11.7.5",

package/src/html-pipe.js

@@ -168,8 +168,8 @@ export async function htmlPipe(state, req) {
       await rewriteUrls(state);
       await fixSections(state);
       await createPageBlocks(state);
-      await extractSectionMetadata(state);
       await createPictures(state);
+      await extractSectionMetadata(state);
       await extractMetaData(state, req);
       await rewriteIcons(state);
       await addHeadingIds(state);

package/src/steps/extract-section-metadata.js

@@ -30,28 +30,49 @@ function isSectionMetadataEnabled(config) {
 }
 
 /**
- * Extracts a value from a HAST node by looking for image src or link href attributes.
- * Falls back to text content if no images or links are found.
+ * Extracts a value from a HAST node by collecting image srcs, link hrefs,
+ * and text tokens (split by comma/whitespace).
  * @param {object} $value the HAST value node
  * @returns {string} the extracted value
  */
 function getValueFromNode($value) {
-  const urls = [];
+  const items = [];
   visit($value, (node) => {
     if (node.tagName === 'img' && node.properties?.src) {
-      urls.push(node.properties.src);
+      items.push(node.properties.src);
       return SKIP;
     }
     if (node.tagName === 'a' && node.properties?.href) {
-      urls.push(node.properties.href);
+      items.push(node.properties.href);
       return SKIP;
     }
+    if (node.type === 'text') {
+      items.push(...node.value.trim().split(/[,\s]+/).filter(Boolean));
+    }
     return CONTINUE;
   });
-  if (urls.length) {
-    return urls.join(',');
-  }
-  return toString($value).trim();
+  return items.join(',');
+}
+
+/**
+ * Extracts style class names from a HAST node.
+ * Treats {@code <br>} and {@code <p>} boundaries as comma separators,
+ * then converts each segment to CSS class names via {@link toBlockCSSClassNames}.
+ * @param {object} $node the HAST node
+ * @returns {string[]} the extracted class names
+ */
+function getStyleClassNames($node) {
+  const parts = [];
+  visit($node, (node) => {
+    if (node.tagName === 'br') {
+      return SKIP;
+    }
+    if (node.type === 'text') {
+      parts.push(...node.value.split(','));
+    }
+    return CONTINUE;
+  });
+  return parts.flatMap(toBlockCSSClassNames);
 }
 
 /**
@@ -78,15 +99,13 @@ export default function extractSectionMetadata(state) {
         const [$name, $value] = $row.children;
         const name = toMetaName(toString($name));
         if (name) {
-          const value = getValueFromNode($value);
           if (name === 'style') {
             if (!parent.properties.className) {
               parent.properties.className = [];
             }
-            parent.properties.className.push(
-              ...value.split(/[,\s]+/).filter(Boolean).flatMap(toBlockCSSClassNames),
-            );
+            parent.properties.className.push(...getStyleClassNames($value));
           } else {
+            const value = getValueFromNode($value);
             parent.properties[`data-${name}`] = value;
           }
         }