@decocms/blocks 7.26.1 → 7.26.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@decocms/blocks",
-  "version": "7.26.1",
+  "version": "7.26.3",
   "type": "module",
   "engines": {
     "node": ">=24"

package/src/cms/schema.test.ts

@@ -271,19 +271,18 @@ describe("commerce SEO section schemas", () => {
     expect(props.jsonLD.hide).toBe(true);
   });
 
-  it("registers the sections so existing pages resolve, without polluting the picker", () => {
+  it("registers the defs + manifest blocks and offers them as page.seo options", () => {
+    // Unconditional (no commerce gate): meta.gen.json is baked at generation
+    // time when the loader registry is empty, and the runtime serves that baked
+    // meta as-is, so a registry-based gate would never bake the options.
     const meta = composeMeta(emptySiteMeta());
     for (const key of [
       "commerce/sections/Seo/SeoPLPV2.tsx",
       "commerce/sections/Seo/SeoPDPV2.tsx",
     ]) {
-      // Registered as a manifest block + def → the SEO editor can resolve a
-      // schema for a page already on this type.
       expect(meta.manifest.blocks.sections).toHaveProperty(key);
       expect(meta.schema.definitions).toHaveProperty(b64(key));
-      // But deliberately NOT offered as a new pickable section everywhere
-      // (legacy commerce types have no component in start).
-      expect(meta.schema.root.sections.anyOf).not.toContainEqual(b64Ref(key));
+      expect(meta.schema.root.sections.anyOf).toContainEqual(b64Ref(key));
     }
   });
 });

package/src/cms/schema.ts

@@ -842,11 +842,13 @@ function buildFrameworkSections(sectionAnyOf: any[]) {
   // data-source block-ref set once when the page is created; it round-trips via
   // the form (kept `hide`) rather than being re-picked in the SEO panel.
   //
-  // NOT pushed to `extraAnyOf`: unlike the website Seo sections these are legacy
-  // deco-cx types with no component in start, so we only register the def +
-  // manifest block (enough for the SEO editor to resolve a schema for pages
-  // already on these types) without offering them as a new pickable section on
-  // every — including non-commerce — site.
+  // Added to `extraAnyOf` (offered as a selectable page.seo type) unconditionally,
+  // like the website Seo sections above. Gating this on "is a commerce site" via
+  // the loader registry does NOT work: meta.gen.json is baked at generation time
+  // (blocks-cli), when the registry is empty, and the runtime serves that baked,
+  // already-composed meta as-is (composeMeta is idempotent on it) — so a gate
+  // would drop the options at bake and never re-add them. Non-commerce sites
+  // seeing these two extra options is a minor, acceptable cost.
   const SEO_PLP_V2_TYPE = "commerce/sections/Seo/SeoPLPV2.tsx";
   const seoPlpV2Key = toBase64(SEO_PLP_V2_TYPE);
   definitions[seoPlpV2Key] = {
@@ -882,6 +884,7 @@ function buildFrameworkSections(sectionAnyOf: any[]) {
     $ref: `#/definitions/${seoPlpV2Key}`,
     namespace: "commerce",
   };
+  extraAnyOf.push({ $ref: `#/definitions/${seoPlpV2Key}` });
 
   // --- commerce/sections/Seo/SeoPDPV2.tsx ---
   // Product-details SEO. Studio "pdp" mode counterpart of the PLP section above.
@@ -914,6 +917,7 @@ function buildFrameworkSections(sectionAnyOf: any[]) {
     $ref: `#/definitions/${seoPdpV2Key}`,
     namespace: "commerce",
   };
+  extraAnyOf.push({ $ref: `#/definitions/${seoPdpV2Key}` });
 
   // --- website/flags/multivariate/section.ts ---
   const MV_SECTION_TYPE = "website/flags/multivariate/section.ts";