@pack/hydrogen 3.2.5 → 3.2.7

package/README.md

@@ -1,6 +1,6 @@
 # `@pack/hydrogen`
 
-Hydrogen runtime integration for Pack content, preview, A/B testing, and server-side error tracking.
+Hydrogen runtime integration for Pack content, preview, and A/B testing.
 
 ## Install
 
@@ -19,25 +19,18 @@ npm install @pack/hydrogen
 ## Minimal setup
 
 ```ts
-import {
-  createPackClient,
-  handleRequest,
-} from '@pack/hydrogen';
-import { createServerErrorHandler } from '@pack/errors';
+import { createPackClient, handleRequest } from "@pack/hydrogen";
 
-// Optional if you want route-level error capture in addition to handleRequest wrapping.
-// Duplicate reports for the same Error instance are deduplicated automatically.
-export const handleError = createServerErrorHandler();
-
-export default async function fetch(request: Request, env: Env, ctx: ExecutionContext) {
+export default async function fetch(
+  request: Request,
+  env: Env,
+  ctx: ExecutionContext,
+) {
   const pack = createPackClient({
     cache: caches.default,
     waitUntil: ctx.waitUntil.bind(ctx),
     storeId: env.PACK_STORE_ID,
     token: env.PACK_API_TOKEN,
-    errorTracking: {
-      dsn: env.PUBLIC_PACK_ERROR_TRACKING_DSN,
-    },
     session,
     testSession,
   });

package/dist/index.d.ts

@@ -1,5 +1,4 @@
 import { PackClient } from '@pack/client';
-import { ErrorTrackingOptions } from '@pack/errors';
 import { CacheCustom } from '@shopify/hydrogen';
 import { SessionStorage, Session, ActionFunctionArgs, LoaderFunctionArgs } from 'react-router';
 import * as react from 'react';
@@ -86,8 +85,6 @@ interface CreatePackClientOptions extends EnvironmentOptions {
     i18n?: I18nOptions;
     /** Default theme data to use when no token is provided */
     defaultThemeData?: DefaultThemeData;
-    /** Configuration for server-side error tracking */
-    errorTracking?: ErrorTrackingOptions;
     /**
      * Initial request to extract query parameters from.
      * If not provided, it will be captured from the first handleRequest call.
@@ -145,14 +142,12 @@ interface Pack {
         packAbTest: Test$1 | null | undefined;
         packIsPreviewMode: boolean;
         packCustomizerMeta: PackCustomizerMeta | null;
-        packErrorTracking?: ErrorTrackingOptions;
     };
     handleRequest(request: Request): Promise<(response: Response) => void>;
     isPreviewModeEnabled: () => boolean;
     isValidEditToken: PackClient["isValidEditToken"];
     query: <T = any>(query: string, options: QueryOptions) => Promise<QueryResponse<T>>;
     session: PackSession;
-    errorTracking?: ErrorTrackingOptions;
     testSession: PackTestSession;
 }
 interface DefaultThemeData {

package/dist/index.js

@@ -4356,9 +4356,6 @@ var require_dist = __commonJS({
   }
 });
 
-// src/handle-request.ts
-import { sendServerErrorEvent } from "@pack/errors";
-
 // ../packlytics/dist/utils/get-packlytics-id.js
 function sha256(ascii) {
   function rightRotate(value, amount) {
@@ -4671,28 +4668,12 @@ async function packlytics(pack, request, next) {
 }
 
 // src/handle-request.ts
-function getServerStatusError(request, status) {
-  let pathname = request.url;
-  try {
-    pathname = new URL(request.url).pathname;
-  } catch {
-  }
-  return new Error(`HTTP ${status} response for ${request.method} ${pathname}`);
-}
 async function handleRequest(pack, request, handleRequest2) {
   const packHandleResponse = await pack.handleRequest(request);
   let response;
-  try {
-    response = await packlytics(pack, request, () => {
-      return handleRequest2(request);
-    });
-  } catch (error) {
-    sendServerErrorEvent(error, request, pack);
-    throw error;
-  }
-  if (response.status >= 500 && !request.signal.aborted) {
-    sendServerErrorEvent(getServerStatusError(request, response.status), request, pack);
-  }
+  response = await packlytics(pack, request, () => {
+    return handleRequest2(request);
+  });
   packHandleResponse(response);
   response.headers.append("powered-by", "Shopify, Hydrogen + Pack Digital");
   response.headers.append("Set-Cookie", await pack.session.commit());
@@ -5811,71 +5792,62 @@ async function getTestInfo({
   );
   if (testSessionData?.id && testSessionData?.testVariant) {
     debug3("[Pack Test] Found existing test data in session");
-    if (!testSessionExpireAt || new Date(testSessionExpireAt) >= /* @__PURE__ */ new Date()) {
-      debug3(
-        "[Pack Test] Test session not expired by time, fetching rules to validate if test is still active"
+    try {
+      const validationResult = await packClientFetchTestByRules(
+        packClient,
+        testTargetAudienceAttributes,
+        testSession,
+        request,
+        withCache,
+        token,
+        null,
+        true
+        // validateOnly - but will assign new test if current is inactive
       );
-      try {
-        const validationResult = await packClientFetchTestByRules(
-          packClient,
-          testTargetAudienceAttributes,
-          testSession,
-          request,
-          withCache,
-          token,
-          null,
-          true
-          // validateOnly - but will assign new test if current is inactive
-        );
-        if (validationResult) {
-          if (validationResult.id === testSessionData.id) {
-            debug3(
-              "[Pack Test] Test is still active, using refreshed data",
-              JSON.stringify({
-                testId: validationResult.id,
-                testHandle: validationResult.handle,
-                variantId: validationResult.testVariant.id,
-                variantHandle: validationResult.testVariant.handle
-              })
-            );
-            return validationResult;
-          } else {
-            debug3(
-              "[Pack Test] Original test no longer active, new test assigned",
-              JSON.stringify({
-                oldTestId: testSessionData.id,
-                newTestId: validationResult.id,
-                newTestHandle: validationResult.handle,
-                newVariantId: validationResult.testVariant.id,
-                newVariantHandle: validationResult.testVariant.handle
-              })
-            );
-            return {
-              ...validationResult,
-              isFirstExposure: true
-            };
-          }
+      if (validationResult) {
+        if (validationResult.id === testSessionData.id) {
+          debug3(
+            "[Pack Test] Test is still active, using refreshed data",
+            JSON.stringify({
+              testId: validationResult.id,
+              testHandle: validationResult.handle,
+              variantId: validationResult.testVariant.id,
+              variantHandle: validationResult.testVariant.handle
+            })
+          );
+          return validationResult;
         } else {
           debug3(
-            "[Pack Test] Test is no longer active and no new test was assigned"
+            "[Pack Test] Original test no longer active, new test assigned",
+            JSON.stringify({
+              oldTestId: testSessionData.id,
+              newTestId: validationResult.id,
+              newTestHandle: validationResult.handle,
+              newVariantId: validationResult.testVariant.id,
+              newVariantHandle: validationResult.testVariant.handle
+            })
           );
-          return void 0;
+          return {
+            ...validationResult,
+            isFirstExposure: true
+          };
         }
-      } catch (error) {
+      } else {
         debug3(
-          "[Pack Test] Error validating test, using cached data:",
-          JSON.stringify(error)
+          "[Pack Test] Test is no longer active and no new test was assigned"
         );
-        console.error(
-          "[Pack Test] Error validating test, using cached data:",
-          error
-        );
-        return { ...testSessionData, isFirstExposure: void 0 };
+        return void 0;
       }
-    } else {
-      debug3("[Pack Test] Test session has expired by time");
-      debug3("[Pack Test] Clearing expired test session data");
-      testSession.clearTestData();
+    } catch (error) {
+      debug3(
+        "[Pack Test] Error validating test, using cached data:",
+        JSON.stringify(error)
+      );
+      console.error(
+        "[Pack Test] Error validating test, using cached data:",
+        error
+      );
+      return { ...testSessionData, isFirstExposure: void 0 };
     }
   }
   if (exposedTestCookieString) {
@@ -6220,7 +6192,6 @@ function createPackClient(options) {
     token,
     apiUrl,
     defaultThemeData,
-    errorTracking,
     i18n,
     request
   } = options;
@@ -6290,8 +6261,7 @@ function createPackClient(options) {
           packSessionId: session.id,
           packAbTest: getCurrentAbTest(),
           packIsPreviewMode: previewEnabled,
-          packCustomizerMeta: session.get("customizerMeta"),
-          packErrorTracking: errorTracking
+          packCustomizerMeta: session.get("customizerMeta")
         };
       },
       isPreviewModeEnabled: () => previewEnabled,
@@ -6306,7 +6276,6 @@ function createPackClient(options) {
         return { data, error: null };
       },
       session,
-      errorTracking,
       testSession
     };
   }
@@ -6348,8 +6317,7 @@ function createPackClient(options) {
         packSessionId: session.id,
         packAbTest: getCurrentAbTest(),
         packIsPreviewMode: previewEnabled,
-        packCustomizerMeta: session.get("customizerMeta"),
-        packErrorTracking: errorTracking
+        packCustomizerMeta: session.get("customizerMeta")
       };
     },
     handleRequest: handleRequest2,
@@ -6477,7 +6445,6 @@ function createPackClient(options) {
       }
     },
     session,
-    errorTracking,
     testSession
   };
 }