@cascayd/experiment 0.3.18 → 0.3.20

package/dist/client.d.ts

@@ -1,7 +1,7 @@
 import { InitOptions, ExperimentConfigResponse, EventType, RecordOptions, VariantStatusResponse } from './types';
 export declare function initCascayd(opts: InitOptions): void;
 export declare function getVariantStatus(experimentId: string, variantId: string): Promise<VariantStatusResponse>;
-export declare function assignVariant(experimentId: string, detectedVariants?: string[], route?: string, shop?: string): Promise<{
+export declare function assignVariant(experimentId: string, detectedVariants?: string[], route?: string): Promise<{
     variantId: string;
     config: ExperimentConfigResponse;
 }>;

package/dist/client.js

@@ -1,3 +1,4 @@
+//! SSR: Imports cookie functions that use document.cookie
 import { getOrCreateSession, readVariantChoice } from './cookies.js';
 let API_KEY = '';
 let BASE_URL = 'https://ab-mvp-backend.onrender.com';
@@ -14,6 +15,7 @@ export function initCascayd(opts) {
     });
 }
 async function fetchConfig(experimentId) {
+    //! SSR: fetch works in Node 18+ but may need request context (headers, cookies) for auth
     const res = await fetch(`${BASE_URL}/experiments/${encodeURIComponent(experimentId)}/config`);
     if (!res.ok)
         throw new Error(`Config load failed: ${res.status}`);
@@ -21,6 +23,7 @@ async function fetchConfig(experimentId) {
 }
 export async function getVariantStatus(experimentId, variantId) {
     console.log('[cascayd-sdk] getVariantStatus', experimentId, variantId);
+    //! SSR: fetch works in Node 18+ but may need request context (headers, cookies) for auth
     const res = await fetch(`${BASE_URL}/experiments/${encodeURIComponent(experimentId)}/variants/${encodeURIComponent(variantId)}/status`);
     const data = (await res.json());
     console.log('[cascayd-sdk] getVariantStatus response', data);
@@ -44,6 +47,7 @@ function chooseByWeight(variants, experimentId) {
     console.log('[cascayd-sdk] chooseByWeight called with variants:', variants);
     const total = variants.reduce((s, v) => s + v.weight, 0);
     console.log('[cascayd-sdk] chooseByWeight total weight:', total);
+    //! SSR: Date.now() is non-deterministic - variant will change on every render/request
     // Create a seed based on current time (milliseconds) and experiment ID
     // This makes it change over time while being deterministic for the same millisecond
     const now = Date.now();
@@ -71,7 +75,7 @@ function chooseByWeight(variants, experimentId) {
     console.log('[cascayd-sdk] chooseByWeight fallback to:', fallback);
     return fallback;
 }
-export async function assignVariant(experimentId, detectedVariants, route, shop) {
+export async function assignVariant(experimentId, detectedVariants, route) {
     const baseStatus = await getVariantStatus(experimentId, 'control');
     let weights = baseStatus.weights || {};
     console.log('[cascayd-sdk] assignVariant weights received:', weights);
@@ -86,11 +90,10 @@ export async function assignVariant(experimentId, detectedVariants, route, shop)
                 const ensureBody = {
                     experiment_id: experimentId,
                     variant_ids: detectedVariants,
+                    //! SSR: window.location.pathname is browser-only, route should be passed explicitly on server
                     route: route || (typeof window !== 'undefined' ? window.location.pathname : "/"),
                 };
-                if (shop) {
-                    ensureBody.shop = shop;
-                }
+                //! SSR: fetch works in Node 18+ but may need request context (headers, cookies) for auth
                 const ensureRes = await fetch(`${BASE_URL}/experiments/ensure`, {
                     method: 'POST',
                     headers: {
@@ -159,6 +162,7 @@ function baseStatusToVariants(status) {
     return entries.map(([id, weight]) => ({ id, weight }));
 }
 export async function record(type, opts = {}) {
+    //! SSR: getOrCreateSession uses document.cookie
     const sessionId = getOrCreateSession();
     const body = {
         type,
@@ -166,6 +170,7 @@ export async function record(type, opts = {}) {
     };
     if (opts.experimentId) {
         body.experiment_id = opts.experimentId;
+        //! SSR: readVariantChoice uses document.cookie
         const fromCookie = readVariantChoice(opts.experimentId);
         const variantId = opts.variantId || fromCookie;
         if (variantId) {
@@ -179,10 +184,12 @@ export async function record(type, opts = {}) {
         body.value = opts.value;
     if (opts.route) {
         body.route = opts.route;
+        //! SSR: window.location.pathname is browser-only, route should be passed explicitly on server
     }
     else if (typeof window !== 'undefined' && window.location) {
         body.route = window.location.pathname;
     }
+    //! SSR: fetch works in Node 18+ but may need request context (headers, cookies) for auth
     const res = await fetch(`${BASE_URL}/events`, {
         method: 'POST',
         headers: {
@@ -194,7 +201,8 @@ export async function record(type, opts = {}) {
     if (!res.ok)
         throw new Error(`Record failed: ${res.status}`);
 }
-// Expose functions to window for browser/Shopify theme compatibility
+//! SSR: Exposing to window object is browser-only, should be conditional or removed for SSR
+// Expose functions to window for browser compatibility
 if (typeof window !== 'undefined') {
     ;
     window.initCascayd = initCascayd;

package/dist/cookies.js

@@ -1,19 +1,23 @@
 function getCookie(name) {
+    //! SSR: document.cookie is browser-only, needs server-side cookie abstraction
     const match = document.cookie.match(new RegExp('(?:^|; )' + name.replace(/[.$?*|{}()\[\]\\\/\+^]/g, '\\$&') + '=([^;]*)'));
     return match ? decodeURIComponent(match[1]) : null;
 }
 function setCookie(name, value) {
+    //! SSR: document.cookie is browser-only, needs server-side cookie abstraction
     document.cookie = `${name}=${encodeURIComponent(value)}; path=/; SameSite=Lax`;
 }
 function randomId() {
-    // lightweight uuid v4-ish
+    // lightweight uuid v4-ish.
     return 'xxxxxxxx-xxxx-4xxx-yxxx-xxxxxxxxxxxx'.replace(/[xy]/g, (c) => {
+        //! SSR: crypto.getRandomValues works in Node 18+ but should use Web Crypto API abstraction
         const r = (crypto.getRandomValues(new Uint8Array(1))[0] & 0xf) >>> 0;
         const v = c === 'x' ? r : (r & 0x3) | 0x8;
         return v.toString(16);
     });
 }
 export function getOrCreateSession() {
+    //! SSR: Uses getCookie/setCookie which depend on document.cookie
     const key = 'cascayd:session';
     let id = getCookie(key);
     if (id)
@@ -26,8 +30,10 @@ export function getVariantCookieKey(experimentId) {
     return `cascayd:${experimentId}`;
 }
 export function readVariantChoice(experimentId) {
+    //! SSR: Uses getCookie which depends on document.cookie
     return getCookie(getVariantCookieKey(experimentId));
 }
 export function persistVariantChoice(experimentId, variantId) {
+    //! SSR: Uses setCookie which depends on document.cookie
     setCookie(getVariantCookieKey(experimentId), variantId);
 }

package/dist/react/Experiment.js

@@ -1,8 +1,11 @@
 import { useEffect, useMemo, useRef, useState } from 'react';
+//! SSR: Imports client functions that use fetch without request context
 import { getVariantStatus, record } from '../client';
+//! SSR: Imports cookie functions that use document.cookie
 import { getOrCreateSession, persistVariantChoice, readVariantChoice } from '../cookies';
 const sentImpressions = new Set();
 export function Experiment({ id, children }) {
+    //! SSR: Initial null state causes hydration mismatch - server renders null, client renders variant
     const [active, setActive] = useState(null);
     const hasRunRef = useRef(false);
     // collect variant children (with optional weights)
@@ -15,12 +18,15 @@ export function Experiment({ id, children }) {
         }
         return arr;
     }, [children]);
+    //! SSR: useEffect doesn't run during SSR, variant assignment happens only on client
     useEffect(() => {
         if (hasRunRef.current)
             return;
         hasRunRef.current = true;
+        //! SSR: readVariantChoice uses document.cookie
         const existing = readVariantChoice(id);
         let cancelled = false;
+        //! SSR: getOrCreateSession uses document.cookie
         getOrCreateSession();
         void (async () => {
             try {
@@ -31,6 +37,7 @@ export function Experiment({ id, children }) {
                     });
                     console.log('[cascayd-sdk] current children', childrenArray.map((c) => c.id));
                     try {
+                        //! SSR: getVariantStatus uses fetch without request context
                         const status = await getVariantStatus(id, existing);
                         if (cancelled)
                             return;
@@ -51,6 +58,7 @@ export function Experiment({ id, children }) {
                     }
                 }
                 console.log('[cascayd-sdk] fetching base status', { experimentId: id, variantId: 'control' });
+                //! SSR: getVariantStatus uses fetch without request context
                 const baseStatus = await getVariantStatus(id, 'control');
                 if (cancelled)
                     return;
@@ -67,6 +75,7 @@ export function Experiment({ id, children }) {
                         variantId: candidate,
                     });
                     try {
+                        //! SSR: getVariantStatus uses fetch without request context
                         const status = await getVariantStatus(id, candidate);
                         if (cancelled)
                             return;
@@ -102,6 +111,7 @@ export function Experiment({ id, children }) {
             cancelled = true;
         };
     }, [id, childrenArray]);
+    //! SSR: Returning null on server causes hydration mismatch - server renders null, client renders variant
     if (!active)
         return null;
     const match = childrenArray.find((c) => c.id === active);
@@ -135,6 +145,7 @@ function resolveVariant(candidate, children) {
     return children[0]?.id ?? 'control';
 }
 function applyAssignment(experimentId, variantId, setActive) {
+    //! SSR: persistVariantChoice uses document.cookie
     persistVariantChoice(experimentId, variantId);
     setActive(variantId);
 }
@@ -144,6 +155,7 @@ function sendImpression(experimentId, variantId) {
     if (sentImpressions.has(`${experimentId}:${variantId}`))
         return;
     sentImpressions.add(`${experimentId}:${variantId}`);
+    //! SSR: record uses fetch and cookies without request context
     void record('impression', { experimentId, variantId });
 }
 function fallbackAssign(children, experimentId) {
@@ -165,6 +177,7 @@ function seededRandom(seed) {
 function chooseByWeight(items, experimentId) {
     console.log('[cascayd-sdk] choosing by weight', items);
     const total = items.reduce((sum, item) => sum + item.weight, 0) || 1;
+    //! SSR: Date.now() is non-deterministic - variant will change on every render/request
     // Create a seed based on current time (milliseconds) and experiment ID
     const now = Date.now();
     const seedString = `${experimentId}-${now}`;

package/dist/shopify-handler-browser.js

@@ -339,10 +339,10 @@
   }
 
   // Setup conversion tracking for an experiment
-  function setupConversionTracking(experimentId) {
+  function setupConversionTracking(experimentId, sdk) {
     const selector = '[data-cascayd-experiment="' + experimentId + '"] [data-cascayd-conversion]';
     const elements = document.querySelectorAll(selector);
-    console.log('[cascayd-shopify] 🎯 Setting up conversion tracking for experiment:', experimentId, 'found', elements.length, 'conversion elements');
+    console.log('[cascayd-shopify] 🎯 Setting up conversion tracking for experiment:', experimentId, 'found', elements.length, 'conversion elements', 'hasSDK:', !!sdk);
 
     elements.forEach(function(el) {
       // Skip if already tracked
@@ -358,16 +358,31 @@
         console.log('[cascayd-shopify] 🎯 Conversion click detected!');
         const variantId = findVariantIdFromElement(el, experimentId);
         if (variantId) {
-          console.log('[cascayd-shopify] 📊 Recording conversion:', { experimentId, variantId });
-          // Use direct API call since we're in standalone mode
-          recordDirect('conversion', {
-            experimentId: experimentId,
-            variantId: variantId
-          }).then(function() {
-            console.log('[cascayd-shopify] ✅ Conversion recorded successfully');
-          }).catch(function(error) {
-            console.error('[cascayd-shopify] ❌ Failed to record conversion:', error);
-          });
+          console.log('[cascayd-shopify] 📊 Recording conversion:', { experimentId, variantId, hasSDK: !!sdk, hasRecordFn: !!(sdk && sdk.record) });
+          
+          // Use SDK's record function if available (same as impressions), otherwise fallback to direct API call
+          if (sdk && sdk.record) {
+            console.log('[cascayd-shopify] ✅ Using SDK record function for conversion');
+            sdk.record('conversion', {
+              experimentId: experimentId,
+              variantId: variantId
+            }).then(function() {
+              console.log('[cascayd-shopify] ✅ Conversion recorded successfully via SDK');
+            }).catch(function(error) {
+              console.error('[cascayd-shopify] ❌ Failed to record conversion via SDK:', error);
+            });
+          } else {
+            console.log('[cascayd-shopify] ⚠️ SDK record not available, using direct API call');
+            // Fallback to direct API call
+            recordDirect('conversion', {
+              experimentId: experimentId,
+              variantId: variantId
+            }).then(function() {
+              console.log('[cascayd-shopify] ✅ Conversion recorded successfully via direct API');
+            }).catch(function(error) {
+              console.error('[cascayd-shopify] ❌ Failed to record conversion via direct API:', error);
+            });
+          }
         } else {
           console.warn('[cascayd-shopify] ⚠️ No variant found for experiment', experimentId, 'when recording conversion');
         }
@@ -450,7 +465,7 @@
     });
 
     // Setup conversion tracking after variant is shown
-    setupConversionTracking(experimentId);
+    setupConversionTracking(experimentId, sdk);
     console.log('[cascayd-shopify] ✅ Finished processing experiment:', experimentId);
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cascayd/experiment",
-  "version": "0.3.18",
+  "version": "0.3.20",
   "description": "A lightweight A/B testing SDK for React applications with server-side analytics integration",
   "keywords": [
     "ab-testing",