@faststore/core 4.7.0-dev.2 → 4.7.0-dev.4

package/src/sdk/analytics/hooks/useStartRecommendationSession.ts

@@ -2,7 +2,7 @@ import { useEffect, useRef } from 'react'
 
 import { gql } from '@faststore/core/api'
 
-import { hasEnabledRecommendationShelf } from 'src/sdk/analytics/utils/hasEnabledRecommendationShelf'
+import storeConfig from 'discovery.config'
 import { useLazyQuery } from 'src/sdk/graphql/useLazyQuery'
 import { getCookie } from 'src/utils/getCookie'
 import { retry } from 'src/utils/retry'
@@ -20,9 +20,9 @@ const startRecommendationSessionMutation = gql(`
 type StartRecommendationSessionData = { startRecommendationSession: boolean }
 type StartRecommendationSessionVariables = Record<string, never>
 
-// In-memory lock so Layout re-renders (e.g. multiple RecommendationShelves
-// fetching) cannot start more than one session before the cookie lands.
-// Survives effect cleanup / Strict Mode remounts for this JS realm.
+// In-memory lock so Layout re-renders / shelf fallback cannot start more than
+// one session before the cookie lands. Survives effect cleanup / Strict Mode
+// remounts for this JS realm.
 let hasRequestedStartRecommendationSession = false
 
 /** @internal Test-only: clears the in-memory session-start lock. */
@@ -34,14 +34,14 @@ export function resetStartRecommendationSessionLock() {
  * Starts the anonymous recommendation/personalization session once per browser
  * session.
  *
- * Mounted from `Layout` with the current page props. Opt-in is controlled by
- * the CMS boolean prop `enableRecommendations` on `RecommendationShelf`
- * (default `false`): if no shelf on the page has it enabled, this hook is a
- * no-op and the store never hits the Recommendations API.
+ * Mounted from `Layout` (gated by `experimental.enableRecommendations`) so it
+ * runs on every page when the store opts in via discovery config. Also used as
+ * a fallback from `RecommendationShelf` when the flag is off and `userId` is
+ * missing (including while lookup is still pending) — pass `enabled` to
+ * override the feature flag for that call site.
  *
- * Multiple enabled shelves on the same page still produce a single call: the
- * hook runs once from Layout and is gated by an in-memory lock plus the
- * session cookie.
+ * Multiple callers still produce a single mutation: the in-memory lock plus the
+ * session cookie ensure at most one attempt per browser session.
  *
  * The product view is no longer reported through a mutation here: the PDP now
  * exposes the product data as `<meta property="product:*">` tags, which the
@@ -49,9 +49,14 @@ export function resetStartRecommendationSessionLock() {
  *
  * All work runs client-side in effects (after hydration), so it doesn't affect
  * SSR/TTFB or Lighthouse render metrics.
+ *
+ * @param enabled Optional override. When omitted, uses
+ *   `storeConfig.experimental.enableRecommendations`. When provided (e.g. by
+ *   RecommendationShelf fallback), that boolean wins.
  */
-export function useStartRecommendationSession(pageProps?: unknown) {
-  const enableRecommendations = hasEnabledRecommendationShelf(pageProps)
+export function useStartRecommendationSession(enabled?: boolean) {
+  const flagEnabled = storeConfig.experimental?.enableRecommendations === true
+  const enableRecommendations = enabled ?? flagEnabled
 
   const [runStartRecommendationSession] = useLazyQuery<
     StartRecommendationSessionData,

package/src/sdk/recommendations/useRecommendationShelf.ts

@@ -1,6 +1,7 @@
 import { usePDP } from '@faststore/core'
 import { useMemo } from 'react'
 
+import { useStartRecommendationSession } from 'src/sdk/analytics/hooks/useStartRecommendationSession'
 import { useCart } from 'src/sdk/cart'
 
 import type { ItemContext } from './types'
@@ -50,12 +51,6 @@ export function getRecommendationArguments(
 
 export type UseRecommendationShelfArgs = {
   readonly campaignVrn: string
-  /**
-   * Opt-in for VTEX Recommendations. When `false` (default) the campaign is
-   * never requested. The anonymous user id is still resolved, but it goes
-   * unused and never leaves the browser.
-   */
-  readonly enableRecommendations?: boolean
   /**
    * Where to read the products used as context for the recommendation request.
    * @default 'PDP'
@@ -79,14 +74,25 @@ const EMPTY_ITEMS: RecommendationResponse['products'] = []
  * campaign. It renders nothing — presentation lives in the shelf components
  * that consume it, so a new surface can reuse the campaign rules instead of
  * reimplementing them.
+ *
+ * When `userId` is missing (`undefined` or `null`), starts the personalization
+ * session as a fallback in parallel with cookie retry (Layout already starts
+ * it when `experimental.enableRecommendations` is on). Cookie + in-memory lock
+ * keep the mutation to one attempt per browser session.
  */
 export function useRecommendationShelf({
   campaignVrn,
-  enableRecommendations = false,
   itemsContext = 'PDP',
 }: UseRecommendationShelfArgs): UseRecommendationShelfResult {
   const userId = useRecommendationUserId(campaignVrn)
 
+  // Fallback when Layout did not start the session (flag off): start in
+  // parallel with cookie lookup. `undefined` and `null` both mean no id yet;
+  // waiting for `null` exhausts the retry budget before the mutation can set
+  // `vtex-rec-user-id`, so the first page would never fetch. Cookie + lock
+  // already no-op when a session exists.
+  useStartRecommendationSession(!userId)
+
   const { data: productDetailPage } = usePDP()
   const { items: cartItems } = useCart()
 
@@ -108,14 +114,10 @@ export function useRecommendationShelf({
     return pdpProduct ? [pdpProduct] : []
   }, [itemsContext, cartItems, productDetailPage])
 
-  // Gate the whole feature behind the CMS opt-in: when Recommendations is
-  // disabled the shelf never requests recommendations (nor renders).
-  const recommendationArgs = enableRecommendations
-    ? getRecommendationArguments(campaignVrn, {
-        userId,
-        contextProducts,
-      })
-    : null
+  const recommendationArgs = getRecommendationArguments(campaignVrn, {
+    userId,
+    contextProducts,
+  })
 
   const { data, isLoading, error } = useRecommendations(recommendationArgs)
 

package/test/components/RecommendationShelf/RecommendationShelf.browser.test.tsx

@@ -41,6 +41,10 @@ vi.mock('src/sdk/recommendations/useRecommendationUserId', () => ({
   useRecommendationUserId,
 }))
 
+vi.mock('src/sdk/analytics/hooks/useStartRecommendationSession', () => ({
+  useStartRecommendationSession: vi.fn(),
+}))
+
 import { RecommendationShelf } from 'src/components/sections/RecommendationShelf/RecommendationShelf'
 
 const CAMPAIGN_VRN = 'vrn:recommendations:acc:rec-top-items-v2:campaign-1'
@@ -71,7 +75,7 @@ afterEach(() => {
 })
 
 describe('RecommendationShelf', () => {
-  it('renders a product card per recommended product when enabled', async () => {
+  it('renders a product card per recommended product', async () => {
     useRecommendations.mockReturnValue({
       data: recommendationData,
       isLoading: false,
@@ -79,7 +83,7 @@ describe('RecommendationShelf', () => {
     })
 
     const { getAllByTestId, getByText } = render(
-      <RecommendationShelf campaignVrn={CAMPAIGN_VRN} enableRecommendations />
+      <RecommendationShelf campaignVrn={CAMPAIGN_VRN} />
     )
 
     await waitFor(() => {
@@ -98,9 +102,7 @@ describe('RecommendationShelf', () => {
       error: null,
     })
 
-    render(
-      <RecommendationShelf campaignVrn={CROSS_SELL_VRN} enableRecommendations />
-    )
+    render(<RecommendationShelf campaignVrn={CROSS_SELL_VRN} />)
 
     await waitFor(() => {
       const lastArgs = useRecommendations.mock.calls.at(-1)?.[0]
@@ -120,11 +122,7 @@ describe('RecommendationShelf', () => {
     })
 
     const { getByText } = render(
-      <RecommendationShelf
-        campaignVrn={CAMPAIGN_VRN}
-        enableRecommendations
-        title="Custom title"
-      />
+      <RecommendationShelf campaignVrn={CAMPAIGN_VRN} title="Custom title" />
     )
 
     await waitFor(() => {
@@ -140,25 +138,6 @@ describe('RecommendationShelf', () => {
       error: null,
     })
 
-    const { container, queryByTestId } = render(
-      <RecommendationShelf campaignVrn={CAMPAIGN_VRN} enableRecommendations />
-    )
-
-    await waitFor(() => {
-      const lastArgs = useRecommendations.mock.calls.at(-1)?.[0]
-      expect(lastArgs).toBeNull()
-    })
-    expect(queryByTestId('product-card')).toBeNull()
-    expect(container).toBeEmptyDOMElement()
-  })
-
-  it('skips the fetch when recommendations are disabled (default)', async () => {
-    useRecommendations.mockReturnValue({
-      data: undefined,
-      isLoading: false,
-      error: null,
-    })
-
     const { container, queryByTestId } = render(
       <RecommendationShelf campaignVrn={CAMPAIGN_VRN} />
     )
@@ -179,7 +158,7 @@ describe('RecommendationShelf', () => {
     })
 
     const { container } = render(
-      <RecommendationShelf campaignVrn={CAMPAIGN_VRN} enableRecommendations />
+      <RecommendationShelf campaignVrn={CAMPAIGN_VRN} />
     )
 
     expect(container).toBeEmptyDOMElement()
@@ -193,7 +172,7 @@ describe('RecommendationShelf', () => {
     })
 
     const { container, queryByTestId } = render(
-      <RecommendationShelf campaignVrn={CAMPAIGN_VRN} enableRecommendations />
+      <RecommendationShelf campaignVrn={CAMPAIGN_VRN} />
     )
 
     expect(queryByTestId('product-card')).toBeNull()
@@ -208,7 +187,7 @@ describe('RecommendationShelf', () => {
     })
 
     const { getByTestId } = render(
-      <RecommendationShelf campaignVrn={CAMPAIGN_VRN} enableRecommendations />
+      <RecommendationShelf campaignVrn={CAMPAIGN_VRN} />
     )
 
     expect(getByTestId('skeleton').getAttribute('data-loading')).toBe('true')

package/test/sdk/analytics/useStartRecommendationSession.test.ts

@@ -18,50 +18,21 @@ vi.mock('src/sdk/graphql/useLazyQuery', () => ({
 const getCookie = vi.hoisted(() => vi.fn())
 vi.mock('src/utils/getCookie', () => ({ getCookie }))
 
+const storeConfigMock = vi.hoisted(() => ({
+  experimental: { enableRecommendations: true },
+}))
+vi.mock('discovery.config', () => ({ default: storeConfigMock }))
+
 import {
   resetStartRecommendationSessionLock,
   useStartRecommendationSession,
 } from 'src/sdk/analytics/hooks/useStartRecommendationSession'
 
-const enabledPageProps = {
-  sections: [
-    {
-      name: 'RecommendationShelf',
-      data: { enableRecommendations: true },
-    },
-  ],
-}
-
-const multipleEnabledShelvesPageProps = {
-  sections: [
-    {
-      name: 'RecommendationShelf',
-      data: { enableRecommendations: true, campaignVrn: 'vrn:a' },
-    },
-    {
-      name: 'RecommendationShelf',
-      data: { enableRecommendations: true, campaignVrn: 'vrn:b' },
-    },
-    {
-      name: 'ProductShelf',
-      data: {},
-    },
-  ],
-}
-
-const disabledPageProps = {
-  sections: [
-    {
-      name: 'RecommendationShelf',
-      data: { enableRecommendations: false },
-    },
-  ],
-}
-
 beforeEach(() => {
   resetStartRecommendationSessionLock()
   useLazyQueryMock.mockReturnValue([runStartRecommendationSession, {}])
   runStartRecommendationSession.mockResolvedValue(true)
+  storeConfigMock.experimental.enableRecommendations = true
 })
 
 afterEach(() => {
@@ -69,18 +40,19 @@ afterEach(() => {
 })
 
 describe('useStartRecommendationSession', () => {
-  it('does not start a session when no RecommendationShelf is enabled in CMS', async () => {
+  it('does not start a session when experimental.enableRecommendations is false', async () => {
+    storeConfigMock.experimental.enableRecommendations = false
     getCookie.mockReturnValue(undefined)
 
-    renderHook(() => useStartRecommendationSession(disabledPageProps))
+    renderHook(() => useStartRecommendationSession())
 
     await waitFor(() => {
       expect(runStartRecommendationSession).not.toHaveBeenCalled()
     })
   })
 
-  it('does not start a session when page props are missing', async () => {
-    getCookie.mockReturnValue(undefined)
+  it('does not start a session when the session cookie already exists', async () => {
+    getCookie.mockReturnValue('already-started')
 
     renderHook(() => useStartRecommendationSession())
 
@@ -89,53 +61,50 @@ describe('useStartRecommendationSession', () => {
     })
   })
 
-  it('does not start a session when the session cookie already exists', async () => {
-    getCookie.mockReturnValue('already-started')
+  it('starts a session when the feature flag is on and no session cookie is present', async () => {
+    getCookie.mockReturnValue(undefined)
 
-    renderHook(() => useStartRecommendationSession(enabledPageProps))
+    renderHook(() => useStartRecommendationSession())
 
     await waitFor(() => {
-      expect(runStartRecommendationSession).not.toHaveBeenCalled()
+      expect(runStartRecommendationSession).toHaveBeenCalledTimes(1)
     })
   })
 
-  it('starts a session when CMS enableRecommendations is true and no session cookie is present', async () => {
+  it('starts a session when enabled override is true even if the feature flag is off', async () => {
+    storeConfigMock.experimental.enableRecommendations = false
     getCookie.mockReturnValue(undefined)
 
-    renderHook(() => useStartRecommendationSession(enabledPageProps))
+    renderHook(() => useStartRecommendationSession(true))
 
     await waitFor(() => {
       expect(runStartRecommendationSession).toHaveBeenCalledTimes(1)
     })
   })
 
-  it('starts a session only once when the page has multiple enabled RecommendationShelves', async () => {
+  it('does not start a session when enabled override is false even if the feature flag is on', async () => {
     getCookie.mockReturnValue(undefined)
 
-    const { rerender } = renderHook(
-      ({ pageProps }) => useStartRecommendationSession(pageProps),
-      { initialProps: { pageProps: multipleEnabledShelvesPageProps } }
-    )
-
-    // Simulate Layout re-renders caused by multiple shelves updating.
-    rerender({ pageProps: multipleEnabledShelvesPageProps })
-    rerender({ pageProps: multipleEnabledShelvesPageProps })
+    renderHook(() => useStartRecommendationSession(false))
 
     await waitFor(() => {
-      expect(runStartRecommendationSession).toHaveBeenCalledTimes(1)
+      expect(runStartRecommendationSession).not.toHaveBeenCalled()
     })
   })
 
-  it('does not start a second session when the hook mounts again without a cookie', async () => {
+  it('starts a session only once across remounts / re-renders (in-memory lock)', async () => {
     getCookie.mockReturnValue(undefined)
 
-    renderHook(() => useStartRecommendationSession(enabledPageProps))
+    const { rerender } = renderHook(() => useStartRecommendationSession())
+
+    rerender()
+    rerender()
 
     await waitFor(() => {
       expect(runStartRecommendationSession).toHaveBeenCalledTimes(1)
     })
 
-    renderHook(() => useStartRecommendationSession(enabledPageProps))
+    renderHook(() => useStartRecommendationSession())
 
     await waitFor(() => {
       expect(runStartRecommendationSession).toHaveBeenCalledTimes(1)

package/test/sdk/recommendations/useRecommendationShelf.browser.test.tsx

@@ -17,6 +17,11 @@ vi.mock('src/sdk/recommendations/useRecommendationUserId', () => ({
   useRecommendationUserId,
 }))
 
+const useStartRecommendationSession = vi.hoisted(() => vi.fn())
+vi.mock('src/sdk/analytics/hooks/useStartRecommendationSession', () => ({
+  useStartRecommendationSession,
+}))
+
 import { useRecommendationShelf } from 'src/sdk/recommendations/useRecommendationShelf'
 
 const TOP_ITEMS_VRN = 'vrn:recommendations:acc:rec-top-items-v2:campaign-1'
@@ -53,12 +58,9 @@ afterEach(() => {
 })
 
 describe('useRecommendationShelf', () => {
-  it('exposes the campaign payload when enabled', () => {
+  it('exposes the campaign payload', () => {
     const { result } = renderHook(() =>
-      useRecommendationShelf({
-        campaignVrn: TOP_ITEMS_VRN,
-        enableRecommendations: true,
-      })
+      useRecommendationShelf({ campaignVrn: TOP_ITEMS_VRN })
     )
 
     expect(result.current.items).toHaveLength(2)
@@ -66,32 +68,40 @@ describe('useRecommendationShelf', () => {
     expect(result.current.correlationId).toBe('corr-1')
   })
 
-  it('never fetches while recommendations are disabled', () => {
+  it('never fetches before a userId resolves', () => {
+    useRecommendationUserId.mockReturnValue(null)
+
     renderHook(() => useRecommendationShelf({ campaignVrn: TOP_ITEMS_VRN }))
 
     expect(lastArgs()).toBeNull()
   })
 
-  it('never fetches before a userId resolves', () => {
+  it('starts the recommendation session as a fallback when userId is missing', () => {
     useRecommendationUserId.mockReturnValue(null)
 
-    renderHook(() =>
-      useRecommendationShelf({
-        campaignVrn: TOP_ITEMS_VRN,
-        enableRecommendations: true,
-      })
-    )
+    renderHook(() => useRecommendationShelf({ campaignVrn: TOP_ITEMS_VRN }))
 
-    expect(lastArgs()).toBeNull()
+    expect(useStartRecommendationSession).toHaveBeenCalledWith(true)
+  })
+
+  it('starts the recommendation session while userId is still resolving', () => {
+    useRecommendationUserId.mockReturnValue(undefined)
+
+    renderHook(() => useRecommendationShelf({ campaignVrn: TOP_ITEMS_VRN }))
+
+    expect(useStartRecommendationSession).toHaveBeenCalledWith(true)
+  })
+
+  it('does not start the recommendation session when userId already exists', () => {
+    useRecommendationUserId.mockReturnValue('user-1')
+
+    renderHook(() => useRecommendationShelf({ campaignVrn: TOP_ITEMS_VRN }))
+
+    expect(useStartRecommendationSession).toHaveBeenCalledWith(false)
   })
 
   it('never fetches on a malformed campaign vrn', () => {
-    renderHook(() =>
-      useRecommendationShelf({
-        campaignVrn: 'not-a-vrn',
-        enableRecommendations: true,
-      })
-    )
+    renderHook(() => useRecommendationShelf({ campaignVrn: 'not-a-vrn' }))
 
     expect(lastArgs()).toBeNull()
   })
@@ -104,7 +114,6 @@ describe('useRecommendationShelf', () => {
     renderHook(() =>
       useRecommendationShelf({
         campaignVrn: CROSS_SELL_VRN,
-        enableRecommendations: true,
         itemsContext: 'CART',
       })
     )
@@ -120,7 +129,6 @@ describe('useRecommendationShelf', () => {
     renderHook(() =>
       useRecommendationShelf({
         campaignVrn: CROSS_SELL_VRN,
-        enableRecommendations: true,
         itemsContext: 'CART',
       })
     )
@@ -132,7 +140,6 @@ describe('useRecommendationShelf', () => {
     renderHook(() =>
       useRecommendationShelf({
         campaignVrn: TOP_ITEMS_VRN,
-        enableRecommendations: true,
         itemsContext: 'CART',
       })
     )
@@ -150,7 +157,6 @@ describe('useRecommendationShelf', () => {
     const { rerender } = renderHook(() =>
       useRecommendationShelf({
         campaignVrn: CROSS_SELL_VRN,
-        enableRecommendations: true,
         itemsContext: 'CART',
       })
     )
@@ -166,12 +172,7 @@ describe('useRecommendationShelf', () => {
       data: { product: { isVariantOf: { productGroupID: 'pg-pdp' } } },
     })
 
-    renderHook(() =>
-      useRecommendationShelf({
-        campaignVrn: CROSS_SELL_VRN,
-        enableRecommendations: true,
-      })
-    )
+    renderHook(() => useRecommendationShelf({ campaignVrn: CROSS_SELL_VRN }))
 
     expect(lastArgs()?.products).toEqual(['pg-pdp'])
   })
@@ -185,10 +186,7 @@ describe('useRecommendationShelf', () => {
     })
 
     const { result } = renderHook(() =>
-      useRecommendationShelf({
-        campaignVrn: TOP_ITEMS_VRN,
-        enableRecommendations: true,
-      })
+      useRecommendationShelf({ campaignVrn: TOP_ITEMS_VRN })
     )
 
     expect(result.current.items).toEqual([])

package/test/sdk/recommendations/useRecommendationShelf.sessionFallback.browser.test.tsx

@@ -0,0 +1,110 @@
+import { renderHook, waitFor } from '@testing-library/react'
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+
+const usePDP = vi.hoisted(() => vi.fn())
+vi.mock('@faststore/core', () => ({ usePDP }))
+
+vi.mock('@faststore/core/api', () => ({
+  gql: (s: string) => ({ __meta__: {}, query: s }),
+}))
+
+const useCart = vi.hoisted(() => vi.fn())
+vi.mock('src/sdk/cart', () => ({ useCart }))
+
+const useRecommendations = vi.hoisted(() => vi.fn())
+vi.mock('src/sdk/recommendations/useRecommendations', () => ({
+  useRecommendations,
+}))
+
+const runStartRecommendationSession = vi.hoisted(() => vi.fn())
+vi.mock('src/sdk/graphql/useLazyQuery', () => ({
+  useLazyQuery: () => [runStartRecommendationSession, {}],
+}))
+
+const cookies = vi.hoisted(() => ({
+  current: {} as Record<string, string | undefined>,
+}))
+vi.mock('src/utils/getCookie', () => ({
+  getCookie: (name: string) => cookies.current[name],
+}))
+
+const storeConfigMock = vi.hoisted(() => ({
+  experimental: { enableRecommendations: false },
+}))
+vi.mock('discovery.config', () => ({ default: storeConfigMock }))
+
+// Keep retry semantics (loop until `until` / attempts) but skip wall-clock
+// backoff so the cookie lookup can observe the session mutation in this test.
+vi.mock('src/utils/retry', async (importOriginal) => {
+  const actual = await importOriginal<typeof import('src/utils/retry')>()
+  return {
+    retry: <T,>(
+      fn: () => T | Promise<T>,
+      options?: Parameters<typeof actual.retry>[1]
+    ) => actual.retry(fn, { ...options, delayMs: 0, maxDelayMs: 0 }),
+  }
+})
+
+import { resetStartRecommendationSessionLock } from 'src/sdk/analytics/hooks/useStartRecommendationSession'
+import { useRecommendationShelf } from 'src/sdk/recommendations/useRecommendationShelf'
+
+const TOP_ITEMS_VRN = 'vrn:recommendations:acc:rec-top-items-v2:campaign-1'
+
+const lastArgs = () => useRecommendations.mock.calls.at(-1)?.[0]
+
+beforeEach(() => {
+  cookies.current = {}
+  resetStartRecommendationSessionLock()
+  storeConfigMock.experimental.enableRecommendations = false
+  usePDP.mockReturnValue({ data: undefined })
+  useCart.mockReturnValue({ items: [] })
+  useRecommendations.mockReturnValue({
+    data: undefined,
+    isLoading: false,
+    error: null,
+  })
+  runStartRecommendationSession.mockImplementation(async () => {
+    cookies.current['vtex-rec-user-id'] = 'user-from-session'
+    cookies.current['vtex-rec-user-start-session'] = '1'
+    return true
+  })
+})
+
+afterEach(() => {
+  vi.clearAllMocks()
+})
+
+describe('useRecommendationShelf session fallback race', () => {
+  it('starts the session in parallel with cookie retry so the first page can fetch', async () => {
+    renderHook(() => useRecommendationShelf({ campaignVrn: TOP_ITEMS_VRN }))
+
+    await waitFor(() => {
+      expect(runStartRecommendationSession).toHaveBeenCalledTimes(1)
+    })
+
+    await waitFor(() => {
+      expect(lastArgs()).toEqual({
+        userId: 'user-from-session',
+        campaignVrn: TOP_ITEMS_VRN,
+        products: [],
+      })
+    })
+  })
+
+  it('does not start a session when the user id cookie is already present', async () => {
+    cookies.current['vtex-rec-user-id'] = 'existing-user'
+    cookies.current['vtex-rec-user-start-session'] = '1'
+
+    renderHook(() => useRecommendationShelf({ campaignVrn: TOP_ITEMS_VRN }))
+
+    await waitFor(() => {
+      expect(lastArgs()).toEqual({
+        userId: 'existing-user',
+        campaignVrn: TOP_ITEMS_VRN,
+        products: [],
+      })
+    })
+
+    expect(runStartRecommendationSession).not.toHaveBeenCalled()
+  })
+})

package/src/sdk/analytics/utils/hasEnabledRecommendationShelf.ts

@@ -1,49 +0,0 @@
-type CmsSection = {
-  name?: string
-  $componentKey?: string
-  data?: {
-    enableRecommendations?: boolean
-  }
-}
-
-function pushSections(target: CmsSection[], value: unknown) {
-  if (Array.isArray(value)) {
-    for (const item of value) {
-      if (item && typeof item === 'object') {
-        target.push(item as CmsSection)
-      }
-    }
-    return
-  }
-
-  if (value && typeof value === 'object') {
-    const sections = (value as { sections?: unknown }).sections
-    if (Array.isArray(sections)) {
-      pushSections(target, sections)
-    }
-  }
-}
-
-/**
- * Returns true when the current page CMS data includes at least one
- * `RecommendationShelf` with `enableRecommendations: true`.
- */
-export function hasEnabledRecommendationShelf(pageProps: unknown): boolean {
-  if (!pageProps || typeof pageProps !== 'object') {
-    return false
-  }
-
-  const props = pageProps as Record<string, unknown>
-  const sections: CmsSection[] = []
-
-  pushSections(sections, props.sections)
-  pushSections(sections, props.globalSections)
-  pushSections(sections, props.page)
-
-  return sections.some(
-    (section) =>
-      (section.name === 'RecommendationShelf' ||
-        section.$componentKey === 'RecommendationShelf') &&
-      section.data?.enableRecommendations === true
-  )
-}