@pack/hydrogen 3.2.0 → 3.2.1

package/dist/index.js

@@ -5046,11 +5046,26 @@ var cookie2 = __toESM(require_cookie(), 1);
 // src/tests/local-test-resolver.ts
 var import_json_rules_engine = __toESM(require_dist(), 1);
 var import_debug2 = __toESM(require_browser(), 1);
-var debug2 = (0, import_debug2.default)("pack:ab-testing:local-resolver");
-function getImpressionSectionIds(testVariant) {
+
+// src/tests/impression.ts
+function getVariantImpressionSectionIds(testVariant) {
   const sectionIds = testVariant?.sectionTestVariants?.map((variant) => variant?.section?.id).filter((id) => Boolean(id)) || [];
   return [...new Set(sectionIds)];
 }
+function getImpressionSectionIdsForVariant(test, testVariant) {
+  const selectedVariantSectionIds = getVariantImpressionSectionIds(testVariant);
+  if (selectedVariantSectionIds.length > 0) {
+    return selectedVariantSectionIds;
+  }
+  if (testVariant?.handle !== "control") {
+    return [];
+  }
+  const fallbackSectionIds = test?.testVariants?.filter((variant) => variant.handle !== "control").flatMap((variant) => getVariantImpressionSectionIds(variant)) || [];
+  return [...new Set(fallbackSectionIds)];
+}
+
+// src/tests/local-test-resolver.ts
+var debug2 = (0, import_debug2.default)("pack:ab-testing:local-resolver");
 var LocalTestResolver = class {
   testRules = [];
   constructor() {
@@ -5154,7 +5169,10 @@ var LocalTestResolver = class {
         })
       );
       if (accumulatedPercentage >= randomPercentage) {
-        const impressionSectionIds = getImpressionSectionIds(variant);
+        const impressionSectionIds = getImpressionSectionIdsForVariant(
+          selectedTest,
+          variant
+        );
         const result = {
           id: selectedTest.id,
           handle: selectedTest.handle,
@@ -5366,10 +5384,6 @@ var QUERY_TESTS_BY_RULES = `#graphql
     }
   }
 `;
-function getImpressionSectionIds2(testVariant) {
-  const sectionIds = testVariant?.sectionTestVariants?.map((variant) => variant?.section?.id).filter((id) => Boolean(id)) || [];
-  return [...new Set(sectionIds)];
-}
 var localTestResolver = new LocalTestResolver();
 function generateTestRulesCacheKey(storeId, contentEnvironment) {
   return `pack-tests:${storeId}:${contentEnvironment || "default"}`;
@@ -5583,7 +5597,10 @@ async function packClientFetchTestByRules(packClient, testTargetAudienceAttribut
                 (v) => v.id === currentTestData.testVariant.id
               );
               if (currentVariant) {
-                const impressionSectionIds = getImpressionSectionIds2(currentVariant);
+                const impressionSectionIds = getImpressionSectionIdsForVariant(
+                  freshTestData,
+                  currentVariant
+                );
                 const refreshedTest = {
                   id: freshTestData.id,
                   handle: freshTestData.handle,
@@ -5595,7 +5612,10 @@ async function packClientFetchTestByRules(packClient, testTargetAudienceAttribut
                   impression: impressionSectionIds.length > 0 ? { sectionIds: impressionSectionIds } : void 0,
                   isFirstExposure: void 0
                 };
-                const { isFirstExposure: isFirstExposure2, ...testDataWithoutFlag2 } = refreshedTest;
+                const {
+                  isFirstExposure: _isFirstExposure2,
+                  ...testDataWithoutFlag2
+                } = refreshedTest;
                 testSession.setTestData(testDataWithoutFlag2);
                 testSession.setExpireAt(getExpireAtDate().toISOString());
                 return refreshedTest;
@@ -5640,7 +5660,10 @@ async function packClientFetchTestByRules(packClient, testTargetAudienceAttribut
           (v) => v.id === exposedTest.testVariant?.id
         );
         if (exposedVariant) {
-          const impressionSectionIds = getImpressionSectionIds2(exposedVariant);
+          const impressionSectionIds = getImpressionSectionIdsForVariant(
+            freshTestData,
+            exposedVariant
+          );
           const result2 = {
             id: freshTestData.id,
             handle: freshTestData.handle,
@@ -5652,7 +5675,7 @@ async function packClientFetchTestByRules(packClient, testTargetAudienceAttribut
             impression: impressionSectionIds.length > 0 ? { sectionIds: impressionSectionIds } : void 0,
             isFirstExposure: void 0
           };
-          const { isFirstExposure: isFirstExposure2, ...testDataWithoutFlag2 } = result2;
+          const { isFirstExposure: _isFirstExposure2, ...testDataWithoutFlag2 } = result2;
           testSession.setTestData(testDataWithoutFlag2);
           testSession.setExpireAt(getExpireAtDate().toISOString());
           return result2;
@@ -5707,7 +5730,7 @@ async function packClientFetchTestByRules(packClient, testTargetAudienceAttribut
         variantHandle: result.testVariant.handle
       })
     );
-    const { isFirstExposure, ...testDataWithoutFlag } = result;
+    const { isFirstExposure: _isFirstExposure, ...testDataWithoutFlag } = result;
     testSession.setTestData(testDataWithoutFlag);
     testSession.setExpireAt(getExpireAtDate().toISOString());
     debug3("[Pack Test] Returning test info with first exposure flag");