@faststore/core 3.0.152 → 3.0.153

package/.turbo/turbo-build.log

@@ -38,8 +38,8 @@ https://nextjs.org/telemetry
    Collecting page data ...
    Generating static pages (0/7) ...
 
   Generating static pages (1/7) 
-
   Generating static pages (3/7) 
 Warning: Dynamic Content not found for the page: home. Refer to the Dynamic Content documentation at https://developers.vtex.com/docs/guides/faststore/dynamic-content-overview for mapping the page and the corresponding data-fetching function.
+
   Generating static pages (3/7) 
 
   Generating static pages (5/7) 
 
 ✓ Generating static pages (7/7) 
    Finalizing page optimization ...
@@ -74,4 +74,4 @@ Route (pages)                              Size     First Load JS
 ○  (Static)  automatically rendered as static HTML (uses no initial props)
 ●  (SSG)     automatically generated as static HTML + JSON (uses getStaticProps)
 
-Done in 71.36s.
+Done in 70.84s.

package/.turbo/turbo-lint.log

@@ -1,4 +1,4 @@
 yarn run v1.22.22
 $ next lint
 ✔ No ESLint warnings or errors
-Done in 6.03s.
+Done in 6.02s.

package/.turbo/turbo-test.log

@@ -1,12 +1,12 @@
 yarn run v1.22.22
 $ jest
-PASS test/utils/multipleTemplates.test.ts (34.286 s)
-PASS test/server/cms/index.test.ts (34.42 s)
-PASS test/server/index.test.ts (38.295 s)
+PASS test/utils/multipleTemplates.test.ts (33.959 s)
+PASS test/server/cms/index.test.ts (34.228 s)
+PASS test/server/index.test.ts (38.562 s)
 
 Test Suites: 3 passed, 3 total
 Tests:       19 passed, 19 total
 Snapshots:   0 total
-Time:        39.241 s
+Time:        39.653 s
 Ran all test suites.
-Done in 40.61s.
+Done in 41.17s.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@faststore/core",
-  "version": "3.0.152",
+  "version": "3.0.153",
   "license": "MIT",
   "repository": "vtex/faststore",
   "browserslist": "supports es6-module and not dead",
@@ -128,5 +128,5 @@
     "node": "18.19.0",
     "yarn": "1.19.1"
   },
-  "gitHead": "98f762825022beee952d25f893b012f35f335cb0"
+  "gitHead": "6f2e5b5970de162e5c425f70ce20552fe195a3de"
 }

package/src/pages/api/preview.ts

@@ -1,4 +1,5 @@
 import type { NextApiHandler, NextApiRequest } from 'next'
+import type { Locator } from '@vtex/client-cms'
 
 import { clientCMS } from 'src/server/cms'
 import { previewRedirects } from '../../../discovery.config'
@@ -20,10 +21,7 @@ const pickParam = (req: NextApiRequest, parameter: string) => {
   const maybeParam = req.query[parameter]
 
   if (typeof maybeParam !== 'string') {
-    throw new StatusError(
-      `Parameter ${parameter} missing from querystring`,
-      400
-    )
+    return undefined
   }
 
   return maybeParam
@@ -32,14 +30,37 @@ const pickParam = (req: NextApiRequest, parameter: string) => {
 // TODO: Improve security by disabling CMS preview in production
 const handler: NextApiHandler = async (req, res) => {
   try {
-    const locator = {
-      contentType: pickParam(req, 'contentType'),
-      documentId: pickParam(req, 'documentId'),
-      versionId: pickParam(req, 'versionId'),
+    const locator = [
+      'contentType',
+      'documentId',
+      'versionId',
+      'releaseId',
+    ].reduce((acc, param) => {
+      const value = pickParam(req, param)
+
+      if (value !== undefined) acc[param] = value
+
+      return acc
+    }, {} as Record<string, string>)
+
+    // Check if required path params are present
+    if (!locator.contentType || !locator.documentId) {
+      throw new StatusError(
+        `The following path params are required: contentType, documentId`,
+        400
+      )
+    }
+
+    // Check if at least one of the querystring params are present
+    if (!locator.versionId && !locator.releaseId) {
+      throw new StatusError(
+        `One of the following querystring params are required: versionId, releaseId`,
+        400
+      )
     }
 
     // Fetch CMS to check if the provided `locator` exists
-    const page = await clientCMS.getCMSPage(locator)
+    const page = await clientCMS.getCMSPage(locator as Locator)
 
     // If the content doesn't exist prevent preview mode from being enabled
     if (!page) {
@@ -56,8 +77,9 @@ const handler: NextApiHandler = async (req, res) => {
     })
 
     // Redirect to the path from the fetched locator
-    if (previewRedirects[locator.contentType]) {
-      res.redirect(previewRedirects[locator.contentType])
+    const redirects = previewRedirects as Record<string, string>
+    if (redirects[locator.contentType]) {
+      res.redirect(redirects[locator.contentType])
       return
     }