@adobe/helix-html-pipeline 6.28.1 → 6.28.3

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [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)
+
+
+### Bug Fixes
+
+* multiple regressions ([#1066](https://github.com/adobe/helix-html-pipeline/issues/1066)) ([e66aa9f](https://github.com/adobe/helix-html-pipeline/commit/e66aa9f13eaec74fdd4520591c4533e7fd5bee77))
+
 ## [6.28.1](https://github.com/adobe/helix-html-pipeline/compare/v6.28.0...v6.28.1) (2026-04-08)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-html-pipeline",
-  "version": "6.28.1",
+  "version": "6.28.3",
   "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

@@ -10,7 +10,7 @@
  * governing permissions and limitations under the License.
  */
 import { toString } from 'hast-util-to-string';
-import { SKIP, visit } from 'unist-util-visit';
+import { CONTINUE, SKIP, visit } from 'unist-util-visit';
 import { toMetaName } from '../utils/modifiers.js';
 import { toBlockCSSClassNames } from './utils.js';
 
@@ -29,6 +29,31 @@ function isSectionMetadataEnabled(config) {
   return new Date(config.created) >= new Date('2026-05-01');
 }
 
+/**
+ * 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 items = [];
+  visit($value, (node) => {
+    if (node.tagName === 'img' && node.properties?.src) {
+      items.push(node.properties.src);
+      return SKIP;
+    }
+    if (node.tagName === 'a' && 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;
+  });
+  return items.join(',');
+}
+
 /**
  * Processes section metadata blocks by applying their key/value pairs
  * as data attributes on the parent section div, with special handling
@@ -53,12 +78,14 @@ export default function extractSectionMetadata(state) {
         const [$name, $value] = $row.children;
         const name = toMetaName(toString($name));
         if (name) {
-          const value = toString($value).trim();
+          const value = getValueFromNode($value);
           if (name === 'style') {
             if (!parent.properties.className) {
               parent.properties.className = [];
             }
-            parent.properties.className.push(...toBlockCSSClassNames(value));
+            parent.properties.className.push(
+              ...value.split(/[,\s]+/).filter(Boolean).flatMap(toBlockCSSClassNames),
+            );
           } else {
             parent.properties[`data-${name}`] = value;
           }