@zohofinance/commerce 2.1.0 → 2.2.0

package/ts-dist/commands/serve-theme.js

@@ -13,13 +13,14 @@ import { findAvailablePort, watchFolders } from '../utils/serve-theme-utils.js';
 import { APIHandler } from '../utils/api-handler.js';
 import { loadThemeGlobals, mergePageData } from '../utils/load-theme-globals.js';
 import StackTrace from '../utils/stack-trace.js';
-import { validateTheme, formatValidationIssuesForConsole } from '../utils/theme-validation.js';
+import { validateTheme, formatValidationIssuesForConsole, validationWarnings } from '../utils/theme-validation.js';
 import { buildValidationErrorReport, buildRuntimeErrorReport, renderDevErrorPage } from '../utils/error-report.js';
 import { STOREFRONT_ROUTES, resolveApiPath, resolvePageTemplate } from '../routes/storefront-routes.js';
 import { resolveSchemaSettings } from '../utils/resolve-schema-settings.js';
-import { resolveSchemaTranslations as resolveSchemaT, resolveSchemaName } from '../utils/resolve-schema-translations.js';
+import { resolveSchemaTranslations as resolveSchemaT, resolveSchemaName, invalidateSchemaLocaleCache } from '../utils/resolve-schema-translations.js';
 import { prepareSectionData } from '../utils/prepare-section-data.js';
 import { buildSectionComponentAttr } from '../utils/editor-attributes.js';
+import { withThemeBlockSchema } from '../utils/theme-block-schema.js';
 function normalizeStoreUrl(input) {
     if (!input)
         return undefined;
@@ -39,8 +40,6 @@ function normalizeStoreUrl(input) {
 async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true, storeUrl) {
     const liveReloadPort = await findAvailablePort();
     const stackTrace = new StackTrace();
-    const themeGlobals = loadThemeGlobals();
-    let runtimeGlobals = themeGlobals;
     const normalizedStoreUrl = normalizeStoreUrl(storeUrl);
     const shouldUseMock = useMock || !normalizedStoreUrl;
     const cwd = process.cwd();
@@ -50,7 +49,11 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
     let validationResult = await validateTheme(cwd, { mode: 'dev' });
     if (!validationResult.valid) {
         console.error(chalk.yellow('\nTheme validation failed. Pages will show validation errors until the issues are fixed.'));
-        console.error(chalk.red(formatValidationIssuesForConsole(validationResult.issues)));
+        console.error(formatValidationIssuesForConsole(validationResult.issues));
+    }
+    else if (validationWarnings(validationResult.issues).length > 0) {
+        console.log(chalk.yellow('\nTheme validation passed with warnings. Preview is still available.'));
+        console.log(formatValidationIssuesForConsole(validationResult.issues));
     }
     else {
         console.log(chalk.green('Theme validation passed.'));
@@ -59,18 +62,78 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
         useMock: shouldUseMock,
         baseURL: normalizedStoreUrl ?? ''
     });
-    if (!shouldUseMock && normalizedStoreUrl) {
+    // Page-specific keys of the live /home response; they must not leak into other pages.
+    const PAGE_SPECIFIC_KEYS = [
+        'product',
+        'products',
+        'collection',
+        'collections',
+        'category',
+        'categories',
+        'blog',
+        'blogs',
+        'page',
+        'page_context',
+        'cart',
+        'checkout',
+        'order',
+        'comments',
+        'page_title',
+        'seo'
+    ];
+    let liveGlobals = null;
+    async function ensureLiveGlobals() {
+        if (shouldUseMock || liveGlobals)
+            return;
         try {
-            const homeData = await apiHandler.fetch('/home');
-            runtimeGlobals = mergePageData(themeGlobals, homeData);
+            const homeData = {
+                ...(await apiHandler.fetch('/home'))
+            };
+            for (const key of PAGE_SPECIFIC_KEYS) {
+                delete homeData[key];
+            }
+            liveGlobals = homeData;
         }
         catch (e) {
-            console.warn('Could not load live storefront globals. Using local theme globals instead.', e);
+            console.warn('Could not load live storefront globals. Will retry on the next request.', e);
         }
     }
-    const engine = new TemplateEngine({ globals: runtimeGlobals }, stackTrace);
+    // Rebuilt on every access so edits to settings/theme.json show up without a restart.
+    function getRuntimeGlobals() {
+        const themeGlobals = loadThemeGlobals(!shouldUseMock);
+        return liveGlobals ? mergePageData(themeGlobals, liveGlobals) : themeGlobals;
+    }
+    await ensureLiveGlobals();
+    const productHandleById = new Map();
+    function indexProductHandles(value, depth = 0) {
+        if (depth > 8 || !value || typeof value !== 'object')
+            return;
+        if (Array.isArray(value)) {
+            for (const item of value)
+                indexProductHandles(item, depth + 1);
+            return;
+        }
+        const record = value;
+        const { id, handle } = record;
+        if ((typeof id === 'string' || typeof id === 'number') &&
+            typeof handle === 'string' &&
+            handle) {
+            productHandleById.set(String(id), handle);
+        }
+        for (const nested of Object.values(record)) {
+            indexProductHandles(nested, depth + 1);
+        }
+    }
+    function resolveProductPath(productId) {
+        return productHandleById.get(productId) ?? productId;
+    }
+    const engine = new TemplateEngine({ globals: getRuntimeGlobals() }, stackTrace);
     if (!shouldUseMock && normalizedStoreUrl) {
-        engine.fetcher = (url) => apiHandler.fetch(url);
+        engine.fetcher = async (url) => {
+            const data = await apiHandler.fetch(url);
+            indexProductHandles(data);
+            return data;
+        };
     }
     function parseSectionsParam(req) {
         const raw = req.body?.sections ?? req.query?.sections;
@@ -144,11 +207,19 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
         }
         return sections;
     }
+    // Declared before revalidateTheme so a file change can reset it.
+    const sectionSchemaCache = new Map();
     async function revalidateTheme() {
+        sectionSchemaCache.clear();
+        invalidateSchemaLocaleCache();
         validationResult = await validateTheme(cwd, { mode: 'dev' });
         if (!validationResult.valid) {
             console.error(chalk.yellow('\nTheme validation failed after a file change.'));
-            console.error(chalk.red(formatValidationIssuesForConsole(validationResult.issues)));
+            console.error(formatValidationIssuesForConsole(validationResult.issues));
+        }
+        else if (validationWarnings(validationResult.issues).length > 0) {
+            console.log(chalk.yellow('\nTheme validation passed with warnings after a file change.'));
+            console.log(formatValidationIssuesForConsole(validationResult.issues));
         }
         else {
             console.log(chalk.green('Theme validation passed.'));
@@ -158,9 +229,24 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
         const report = buildValidationErrorReport(validationResult, cwd);
         return renderDevErrorPage(report);
     }
-    async function fetchPageData(apiPath) {
-        const pageData = await apiHandler.fetch(apiPath);
-        return mergePageData(runtimeGlobals, pageData);
+    async function fetchPageData(apiPath, query = '') {
+        await ensureLiveGlobals();
+        // Sort, pagination and filter params must reach the live store.
+        const pageData = await apiHandler.fetch(query ? `${apiPath}?${query}` : apiPath);
+        indexProductHandles(pageData);
+        return mergePageData(getRuntimeGlobals(), pageData);
+    }
+    function pageQueryString(req) {
+        const params = new URLSearchParams();
+        for (const [key, value] of Object.entries(req.query)) {
+            if (key === 'format' || key === 'sections')
+                continue;
+            for (const v of Array.isArray(value) ? value : [value]) {
+                if (typeof v === 'string')
+                    params.append(key, v);
+            }
+        }
+        return params.toString();
     }
     async function renderPage(pathName, data, req) {
         if (req?.query.format === 'json') {
@@ -187,7 +273,7 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
     async function handlePageRoute(routePageTemplate, apiPath, req, res, next) {
         try {
             const pageTemplate = resolvePageTemplate(routePageTemplate);
-            const data = await fetchPageData(apiPath);
+            const data = await fetchPageData(apiPath, pageQueryString(req));
             // Always render in editor mode so sections get data-zcommerce-component attributes
             engine.engine.__editorMode = true;
             engine.engine.__editorJsonType = 'template';
@@ -208,7 +294,7 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
             else {
                 html += bridgeTag;
             }
-            res.status(result.status ?? 200).send(html);
+            res.status(result.status ?? 200).set('Cache-Control', 'no-store').send(html);
         }
         catch (error) {
             engine.engine.__editorMode = false;
@@ -236,11 +322,14 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
     app.engine('liquid', engine.express());
     app.set('views', ['./pages']);
     app.set('view engine', 'liquid');
-    if (!livereloadOff) {
+    if (livereloadOff) {
+        watchFolders(undefined, revalidateTheme);
+    }
+    else {
         app.use(connectLiveReload({ port: liveReloadPort }));
         const liveReloadServer = livereload.createServer({ port: liveReloadPort });
+        watchFolders(liveReloadServer, revalidateTheme);
         liveReloadServer.server.once('connection', () => {
-            watchFolders(liveReloadServer, revalidateTheme);
             setTimeout(() => {
                 liveReloadServer.refresh('/');
             }, 100);
@@ -250,12 +339,62 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
     app.get('/', (_req, res) => {
         res.redirect('/home');
     });
+    const PREDICTIVE_SEARCH_SECTION = 'search-result';
+    function withSearchContext(data, req) {
+        const existing = data.search && typeof data.search === 'object' && !Array.isArray(data.search)
+            ? { ...data.search }
+            : {};
+        if (existing.products === undefined) {
+            existing.products = data.products ?? [];
+        }
+        if (existing.q === undefined) {
+            existing.q = typeof req.query.q === 'string' ? req.query.q : '';
+        }
+        return { ...data, search: existing };
+    }
+    async function handlePredictiveSearch(req, res, next) {
+        try {
+            const requested = parseSectionsParam(req);
+            const sectionNames = requested.length > 0 ? requested : [PREDICTIVE_SEARCH_SECTION];
+            const data = await fetchPageData('/search', pageQueryString(req));
+            const sections = await renderSectionsForResponse(sectionNames, withSearchContext(data, req));
+            res.json({ sections });
+        }
+        catch (error) {
+            next(error);
+        }
+    }
+    app.get('/home/search', handlePredictiveSearch);
+    // /search is the search page as well, so only a sections param marks it as
+    // an API call; a plain navigation falls through to the page route below.
+    app.get('/search', (req, res, next) => {
+        if (parseSectionsParam(req).length === 0) {
+            next();
+            return;
+        }
+        return handlePredictiveSearch(req, res, next);
+    });
     for (const route of STOREFRONT_ROUTES) {
         app.get(route.expressPath, (req, res, next) => {
             const apiPath = resolveApiPath(route, req.params);
             return handlePageRoute(route.pageTemplate, apiPath, req, res, next);
         });
     }
+    // Live stores link to singular /category/<handle> and /collection/<handle> URLs.
+    const ROUTE_ALIASES = [
+        ['/category/:category_name', '/categories/:category_name'],
+        ['/collection/:collection_name', '/collections/:collection_name'],
+        ['/product/:product_id', '/products/:product_id']
+    ];
+    for (const [aliasPath, targetPath] of ROUTE_ALIASES) {
+        const route = STOREFRONT_ROUTES.find((r) => r.expressPath === targetPath);
+        if (!route)
+            continue;
+        app.get(aliasPath, (req, res, next) => {
+            const apiPath = resolveApiPath(route, req.params);
+            return handlePageRoute(route.pageTemplate, apiPath, req, res, next);
+        });
+    }
     app.post('/addtocart', upload.none(), async (req, res, next) => {
         try {
             const response = (await apiHandler.addToCart(req.body));
@@ -263,7 +402,7 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
             if (sectionNames.length > 0) {
                 const cartData = await resolveCartForSections(response);
                 const sections = await renderSectionsForResponse(sectionNames, {
-                    ...runtimeGlobals,
+                    ...getRuntimeGlobals(),
                     cart: cartData
                 });
                 res.send({ ...response, sections });
@@ -283,7 +422,7 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
             if (sectionNames.length > 0) {
                 const cartData = await resolveCartForSections(response);
                 const sections = await renderSectionsForResponse(sectionNames, {
-                    ...runtimeGlobals,
+                    ...getRuntimeGlobals(),
                     cart: cartData
                 });
                 res.send({ ...response, sections });
@@ -303,7 +442,7 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
             if (sectionNames.length > 0) {
                 const cartData = await resolveCartForSections(response);
                 const sections = await renderSectionsForResponse(sectionNames, {
-                    ...runtimeGlobals,
+                    ...getRuntimeGlobals(),
                     cart: cartData
                 });
                 res.send({ ...response, sections });
@@ -396,18 +535,14 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
             const sectionNames = parseSectionsParam(req);
             if (sectionNames.length === 0) {
                 res.status(400).json({
-                    code: 1,
                     message: 'Missing sections query param',
                     sections: {}
                 });
                 return;
             }
-            const data = await fetchPageData(`/products/${req.params.product_id}`);
+            const data = await fetchPageData(`/products/${encodeURIComponent(resolveProductPath(req.params.product_id))}`);
             const sections = await renderSectionsForResponse(sectionNames, data);
-            res.json({
-                code: 0,
-                sections
-            });
+            res.json({ sections });
         }
         catch (error) {
             next(error);
@@ -415,7 +550,34 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
     });
     app.get('/products/:product_id/reviews', async (req, res, next) => {
         try {
-            res.json(await apiHandler.getProductReviews(req.params.product_id));
+            const params = new URLSearchParams();
+            const response = (await apiHandler.getProductReviews(req.params.product_id, params.toString()));
+            res.json(response);
+            return;
+        }
+        catch (error) {
+            next(error);
+        }
+    });
+    app.post('/products/:product_id/reviews', async (req, res, next) => {
+        try {
+            res.json(await apiHandler.submitProductReview('POST', req.params.product_id, req.body));
+        }
+        catch (error) {
+            next(error);
+        }
+    });
+    app.put('/products/:product_id/reviews', async (req, res, next) => {
+        try {
+            res.json(await apiHandler.submitProductReview('PUT', req.params.product_id, req.body));
+        }
+        catch (error) {
+            next(error);
+        }
+    });
+    app.get('/editreview/:product_id', async (req, res, next) => {
+        try {
+            res.json(await apiHandler.getReviewForEdit(req.params.product_id));
         }
         catch (error) {
             next(error);
@@ -437,7 +599,7 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
             if (sectionNames.length > 0) {
                 const cartData = await resolveCartForSections(response);
                 const sections = await renderSectionsForResponse(sectionNames, {
-                    ...runtimeGlobals,
+                    ...getRuntimeGlobals(),
                     cart: cartData
                 });
                 res.send({ ...response, sections });
@@ -459,7 +621,7 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
                 const cartData = await resolveCartForSections(response);
                 const saveForLaterData = resolveSaveForLaterForSections(response);
                 const sectionData = {
-                    ...runtimeGlobals,
+                    ...getRuntimeGlobals(),
                     cart: cartData
                 };
                 if (saveForLaterData) {
@@ -485,7 +647,7 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
                 const cartData = await resolveCartForSections(response);
                 const saveForLaterData = resolveSaveForLaterForSections(response);
                 const sectionData = {
-                    ...runtimeGlobals,
+                    ...getRuntimeGlobals(),
                     cart: cartData
                 };
                 if (saveForLaterData) {
@@ -511,7 +673,7 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
                 const cartData = await resolveCartForSections(response);
                 const saveForLaterData = resolveSaveForLaterForSections(response);
                 const sectionData = {
-                    ...runtimeGlobals,
+                    ...getRuntimeGlobals(),
                     cart: cartData
                 };
                 if (saveForLaterData) {
@@ -537,7 +699,7 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
                 const cartData = await resolveCartForSections(response);
                 const saveForLaterData = resolveSaveForLaterForSections(response);
                 const sectionData = {
-                    ...runtimeGlobals,
+                    ...getRuntimeGlobals(),
                     cart: cartData
                 };
                 if (saveForLaterData) {
@@ -804,8 +966,7 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
             console.warn('[dev] Failed to persist full JSON:', e);
         }
     }
-    // Cache parsed section schemas (type → { name, blocks }) to avoid re-reading files.
-    const sectionSchemaCache = new Map();
+    // sectionSchemaCache (declared above) caches parsed section schemas (type → { name, blocks }).
     function getSectionSchema(sectionType) {
         if (sectionSchemaCache.has(sectionType))
             return sectionSchemaCache.get(sectionType);
@@ -901,11 +1062,15 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
                     catch { /* skip invalid schema */ }
                 }
                 const resolved = resolveSchemaT({ settings: schema.settings || [], name: schema.name || sectionType, blocks: schema.blocks || [] }, cwd);
-                const blocks = (resolved.blocks || []).map((b) => ({
-                    ...b,
-                    id: `setting_section_${sectionType}.${b.type}`,
-                    settings: b.settings || [],
-                }));
+                const blocks = (schema.blocks || []).map((rawBlock) => {
+                    const merged = withThemeBlockSchema(cwd, rawBlock, rawBlock.type) ?? rawBlock;
+                    const b = resolveSchemaT(merged, cwd);
+                    return {
+                        ...b,
+                        id: `setting_section_${sectionType}.${b.type}`,
+                        settings: b.settings || [],
+                    };
+                });
                 return {
                     id: `setting_section_${sectionType}`,
                     name: resolved.name || sectionType,
@@ -1005,11 +1170,9 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
                 || STOREFRONT_ROUTES[0];
             const apiPath = resolveApiPath(route, {});
             const data = await fetchPageData(apiPath);
-            // Temporarily override runtimeGlobals.settings so the engine renders with
-            // the new values. Restored in the finally block regardless of outcome.
-            const savedSettings = runtimeGlobals.settings;
+            // Override the settings on this request's data so the engine renders with the new values.
             if (settingsData)
-                runtimeGlobals.settings = settingsData;
+                data.settings = settingsData;
             engineInternal.__editorMode = true;
             engineInternal.__editorJsonType = 'template';
             try {
@@ -1024,7 +1187,6 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
                 res.json({ html });
             }
             finally {
-                runtimeGlobals.settings = savedSettings;
                 engineInternal.__editorMode = false;
             }
         }
@@ -1062,7 +1224,7 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
             // Deep copy BEFORE any mutation (resolveSchemaSettings, prepareSectionData both mutate sectionJson).
             // This copy goes back to the editor and is later saved to disk — must have original unmodified URLs.
             const editorSectionJson = JSON.parse(JSON.stringify(sectionJson));
-            const data = { ...runtimeGlobals };
+            const data = { ...getRuntimeGlobals() };
             if (schemaData?.settings && sectionJson.settings) {
                 await resolveSchemaSettings(schemaData.settings, sectionJson.settings, data, engine.fetcher);
             }
@@ -1154,11 +1316,13 @@ async function serveTheme(portNumber, openInWeb, livereloadOff, useMock = true,
             const fileContent = fs.readFileSync(filePath, 'utf-8');
             const schemaMatch = fileContent.match(/{%\s*schema\s*%}([\s\S]*?){%\s*endschema\s*%}/);
             if (!schemaMatch) {
-                res.json({ schema: { settings: [], name: blockType } });
+                const themeBlock = withThemeBlockSchema(cwd, undefined, blockType);
+                res.json({ schema: resolveSchemaT({ settings: themeBlock?.settings || [], name: themeBlock?.name || blockType }, cwd) });
                 return;
             }
             const schema = JSON.parse(schemaMatch[1].trim());
-            const blockSchema = (schema.blocks || []).find((b) => b.type === blockType);
+            const sectionBlockEntry = (schema.blocks || []).find((b) => b.type === blockType);
+            const blockSchema = withThemeBlockSchema(cwd, sectionBlockEntry, blockType);
             res.json({ schema: resolveSchemaT({ settings: blockSchema?.settings || [], name: blockSchema?.name || blockType }, cwd) });
         }
         catch (error) {

package/ts-dist/mocks/global-objects.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"global-objects.d.ts","sourceRoot":"","sources":["../../src/mocks/global-objects.ts"],"names":[],"mappings":"AAAA,KAAK,aAAa,GAAG;IACnB,CAAC,GAAG,EAAE,MAAM,GAAG,GAAG,CAAC;CACpB,CAAC;AAEF,QAAA,MAAM,aAAa,EAAE,aAw0fpB,CAAC;AAEF,eAAe,aAAa,CAAC"}
+{"version":3,"file":"global-objects.d.ts","sourceRoot":"","sources":["../../src/mocks/global-objects.ts"],"names":[],"mappings":"AAAA,KAAK,aAAa,GAAG;IACnB,CAAC,GAAG,EAAE,MAAM,GAAG,GAAG,CAAC;CACpB,CAAC;AAEF,QAAA,MAAM,aAAa,EAAE,aAy0fpB,CAAC;AAEF,eAAe,aAAa,CAAC"}

package/ts-dist/mocks/global-objects.js

@@ -182,6 +182,7 @@ const globalObjects = {
             show_zoho_powered_by: false,
             is_product_filters_enabled: false,
             is_sorting_enabled: false,
+            is_product_reviews_enabled: true,
             search_box_enabled: false,
             is_footer_enabled: false,
             product_specifications_enabled: false,

package/ts-dist/mocks/init-theme/i18n/en.schema.json

@@ -43,7 +43,8 @@
           "default": "Shop Now"
         },
         "cta_url": {
-          "label": "Button URL"
+          "label": "Button URL",
+          "default": "Shop"
         }
       }
     },

package/ts-dist/mocks/init-theme/sections/footer.liquid

@@ -6,6 +6,8 @@
 
 {% schema %}
 {
-  "name": "t:sections.footer.name"
+  "name": "t:sections.footer.name",
+  "available_in": { "templates": ["footer-group"] },
+  "presets": [{ "name": "t:sections.footer.name" }]
 }
 {% endschema %}

package/ts-dist/mocks/init-theme/sections/header-topbar.liquid

@@ -23,6 +23,7 @@
 {% schema %}
 {
   "name": "t:sections.header_topbar.name",
+  "available_in": { "templates": ["header-group"] },
   "settings": [
     {
       "type": "select",
@@ -47,6 +48,7 @@
         }
       ]
     }
-  ]
+  ],
+  "presets": [{ "name": "t:sections.header_topbar.name" }]
 }
 {% endschema %}

package/ts-dist/mocks/init-theme/sections/product-collection.liquid

@@ -25,6 +25,7 @@
 {% schema %}
 {
   "name": "t:sections.product_collection.name",
+  "available_in": { "templates": ["index", "collection", "category"] },
   "settings": [
     {
       "type": "collection",
@@ -38,6 +39,7 @@
       "type": "zc_theme__section-heading",
       "name": "Section Heading"
     }
-  ]
+  ],
+  "presets": [{ "name": "t:sections.product_collection.name" }]
 }
 {% endschema %}

package/ts-dist/mocks/init-theme/sections/promo-banner.liquid

@@ -50,7 +50,11 @@
     {
       "type": "url",
       "id": "cta_url",
-      "label": "t:sections.promo_banner.settings.cta_url.label"
+      "label": "t:sections.promo_banner.settings.cta_url.label",
+      "default": {
+        "display_name": "t:sections.promo_banner.settings.cta_url.default",
+        "handle": "collections/all-products"
+      }
     }
   ],
   "blocks": [

package/ts-dist/mocks/reviews.d.ts

@@ -0,0 +1,6 @@
+type ProductReviews = {
+    [key: string]: any;
+};
+declare const reviews: ProductReviews;
+export default reviews;
+//# sourceMappingURL=reviews.d.ts.map

package/ts-dist/mocks/reviews.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"reviews.d.ts","sourceRoot":"","sources":["../../src/mocks/reviews.ts"],"names":[],"mappings":"AAAA,KAAK,cAAc,GAAG;IACpB,CAAC,GAAG,EAAE,MAAM,GAAG,GAAG,CAAC;CACpB,CAAC;AAIF,QAAA,MAAM,OAAO,EAAE,cAkDd,CAAC;AAEF,eAAe,OAAO,CAAC"}

package/ts-dist/mocks/reviews.js

@@ -0,0 +1,52 @@
+// Mirrors the live /products/:product_id/reviews response so a theme renders the
+// same markup with or without --storeUrl.
+const reviews = {
+    is_reviewed: false,
+    allow_registered_only: false,
+    is_logged_in: false,
+    is_review_enabled: true,
+    reviews: [
+        {
+            id: '8858475000000094101',
+            user_name: 'Arun Kumar',
+            rating: 5,
+            title: 'Excellent build quality',
+            content: 'The product arrived well packed and works exactly as described. Battery life is great and the finish feels premium.',
+            last_modified_time: '14 June 2026'
+        },
+        {
+            id: '8858475000000094102',
+            user_name: 'Divya Raman',
+            rating: 4,
+            title: 'Good value for the price',
+            content: 'Happy with the purchase overall. Setup was simple and the sound quality is clear, though the case scratches easily.',
+            last_modified_time: '02 June 2026'
+        }
+    ],
+    user_review: null,
+    is_captcha_enabled: false,
+    average_rating: 4.5,
+    rating_stats: {
+        star_5_percentage: 50,
+        star_5_count: 1,
+        overall_average: 4.5,
+        star_4_percentage: 50,
+        star_4_count: 1,
+        star_3_percentage: 0,
+        star_3_count: 0,
+        star_2_percentage: 0,
+        star_2_count: 0,
+        star_1_percentage: 0,
+        star_1_count: 0,
+        total_reviews: 2
+    },
+    page_context: {
+        per_page: 10,
+        items_count: '2',
+        total_pages: 1,
+        current_page: 1,
+        page_param: 'page'
+    },
+    is_purchased: false
+};
+export default reviews;

package/ts-dist/routes/storefront-routes.js

@@ -126,7 +126,7 @@ export const STOREFRONT_ROUTES = [
     }
 ];
 export function resolveApiPath(route, params) {
-    return route.apiPath.replace(/:([a-z_]+)/gi, (_, key) => params[key] ?? '');
+    return route.apiPath.replace(/:([a-z_]+)/gi, (_, key) => encodeURIComponent(params[key] ?? ''));
 }
 export function pageTemplateExists(pageTemplate) {
     return fs.existsSync(path.join(process.cwd(), 'pages', `${pageTemplate}.json`));

package/ts-dist/utils/api-handler.d.ts

@@ -13,6 +13,7 @@ export declare class APIHandler {
         useMock: boolean;
         baseURL?: string;
     }): Promise<APIHandler>;
+    get cookieHeader(): string;
     fetch(requestUrl: string): Promise<Record<string, unknown>>;
     getMockCartData(): Record<string, unknown>;
     getMockSaveForLaterData(): Record<string, unknown>[];
@@ -27,7 +28,11 @@ export declare class APIHandler {
     wishlistAction(method: 'POST' | 'DELETE', query: string, body?: Record<string, unknown>): Promise<any>;
     getQuote(): Promise<any>;
     getProductRecommendations(productId: string): Promise<any>;
-    getProductReviews(productId: string): Promise<any>;
+    getProductReviews(productId: string, query?: string): Promise<{
+        [key: string]: any;
+    }>;
+    submitProductReview(method: 'POST' | 'PUT', productId: string, data: Record<string, unknown>): Promise<unknown>;
+    getReviewForEdit(productId: string): Promise<unknown>;
     bulkAddToCart(data: Record<string, unknown>): Promise<any>;
     saveForLater(method: 'PUT' | 'DELETE', query: string): Promise<unknown>;
     moveToCart(query: string): Promise<unknown>;