@modelnex/sdk 0.5.55 → 0.5.59

package/dist/index.js

@@ -3939,6 +3939,11 @@ function shouldExecuteTourCommandBatch(isPlaybackActive) {
 function shouldAcceptTourStart(state) {
   return !state.isPlaybackActive && !state.startRequested;
 }
+function shouldAcceptTourExperienceType(hookType, incomingType, isResumed) {
+  const matchesType = !incomingType || incomingType === hookType;
+  const isCompatibleResumption = isResumed && (hookType === "onboarding" && incomingType === "tour" || hookType === "tour" && incomingType === "onboarding");
+  return matchesType || isCompatibleResumption;
+}
 function shouldRunTourAutoDiscovery(state) {
   return state.enableAutoDiscovery && !state.disabled && !state.isPlaybackActive && !state.startRequested && !state.hasPendingTour;
 }
@@ -3978,6 +3983,25 @@ function releaseTourPlaybackOwnership(key, ownerId) {
 
 // src/hooks/useTourPlayback.ts
 init_dev_logging();
+function shouldSuppressCompletedDraftPreview({
+  endingTourId,
+  endingPreviewRunId
+}) {
+  return Boolean(endingTourId && endingPreviewRunId);
+}
+function getCompletedDraftPreviewToSuppress({
+  endingTourId,
+  endingPreviewRunId,
+  experienceType
+}) {
+  if (!shouldSuppressCompletedDraftPreview({ endingTourId, endingPreviewRunId })) {
+    return null;
+  }
+  return {
+    id: endingTourId,
+    experienceType
+  };
+}
 function resolveElement(step) {
   const el = step.element;
   if (!el) return null;
@@ -4799,7 +4823,7 @@ function useTourPlayback({
               }
               await waitForUserAction(gateTarget, segment.gate.event);
             }
-            if (segmentText && showCaptionsRef.current && reviewModeRef.current) {
+            if (segmentText && showCaptionsRef.current) {
               showCaption(segmentText);
             }
             const nextSegmentText = (segments[index + 1]?.text ?? "").trim();
@@ -4826,7 +4850,7 @@ function useTourPlayback({
           if (params.removeHighlightAtEnd !== false) {
             removeHighlight();
           }
-          if (showCaptionsRef.current && reviewModeRef.current) {
+          if (showCaptionsRef.current) {
             removeCaption();
           }
           return !!params.waitForInput;
@@ -4836,7 +4860,7 @@ function useTourPlayback({
           if (!text.trim()) {
             return { result: null };
           }
-          if (showCaptionsRef.current && reviewModeRef.current) {
+          if (showCaptionsRef.current) {
             showCaption(text);
           }
           const settlePromise = voice.speak(text, tourRef.current?.voice?.ttsVoice, {
@@ -5265,9 +5289,8 @@ function useTourPlayback({
       }
       const tour = tourData.tourContext ?? tourRef.current;
       const expType = experienceTypeRef.current;
-      const matchesType = !tour?.type || tour.type === expType;
-      const isCompatibleResumption = !!tourData.resumed && (expType === "onboarding" && tour?.type === "tour" || expType === "tour" && tour?.type === "onboarding");
-      if (!matchesType && !isCompatibleResumption) {
+      const isCompatible = shouldAcceptTourExperienceType(expType, tour?.type, !!tourData.resumed);
+      if (!isCompatible) {
         emitSdkDebugLog("[TourClient] Ignoring tour start for mismatched experience type", {
           incomingType: tour?.type,
           hookType: expType,
@@ -5415,10 +5438,10 @@ function useTourPlayback({
     return () => clearTimeout(timer);
   }, [disabled, emitTourInit, websiteId, userProfile?.userId, userProfile?.type]);
   (0, import_react12.useEffect)(() => {
-    if (!showCaptions || !isReviewMode) {
+    if (!showCaptions) {
       removeCaption();
     }
-  }, [showCaptions, isReviewMode]);
+  }, [showCaptions]);
   (0, import_react12.useEffect)(() => {
     if (!isActiveRef.current) return;
     emitSocketEvent(socketRef.current, "tour:client_state", {
@@ -5544,6 +5567,11 @@ function useTourPlayback({
     const endingTourId = tourRef.current?.id;
     const endingPreviewRunId = previewRunIdRef.current;
     const endingStepOrder = stepIndexRef.current;
+    const completedPreviewDraft = getCompletedDraftPreviewToSuppress({
+      endingTourId,
+      endingPreviewRunId,
+      experienceType
+    });
     isActiveRef.current = false;
     startRequestedRef.current = false;
     releasePlaybackOwnership();
@@ -5553,6 +5581,10 @@ function useTourPlayback({
     runCleanup(pendingManualWaitCleanupRef.current);
     pendingManualWaitCleanupRef.current = null;
     clearPendingManualInputSync();
+    if (completedPreviewDraft) {
+      persistSuppressedDraftPreview(completedPreviewDraft);
+      clearActiveDraftPreview(experienceType);
+    }
     if (!skipRequestedRef.current && userProfile?.userId && tourRef.current) {
       markTourComplete(serverUrl, toursApiBaseRef.current, tourRef.current.id, userProfile.userId, experienceType, websiteId);
     }
@@ -5577,7 +5609,7 @@ function useTourPlayback({
       });
     }
     if (reviewModeRef.current) {
-      if (endingTourId) {
+      if (endingTourId && !completedPreviewDraft) {
         persistSuppressedDraftPreview({
           id: endingTourId,
           experienceType

package/dist/index.mjs

@@ -3111,6 +3111,11 @@ function shouldExecuteTourCommandBatch(isPlaybackActive) {
 function shouldAcceptTourStart(state) {
   return !state.isPlaybackActive && !state.startRequested;
 }
+function shouldAcceptTourExperienceType(hookType, incomingType, isResumed) {
+  const matchesType = !incomingType || incomingType === hookType;
+  const isCompatibleResumption = isResumed && (hookType === "onboarding" && incomingType === "tour" || hookType === "tour" && incomingType === "onboarding");
+  return matchesType || isCompatibleResumption;
+}
 function shouldRunTourAutoDiscovery(state) {
   return state.enableAutoDiscovery && !state.disabled && !state.isPlaybackActive && !state.startRequested && !state.hasPendingTour;
 }
@@ -3149,6 +3154,25 @@ function releaseTourPlaybackOwnership(key, ownerId) {
 }
 
 // src/hooks/useTourPlayback.ts
+function shouldSuppressCompletedDraftPreview({
+  endingTourId,
+  endingPreviewRunId
+}) {
+  return Boolean(endingTourId && endingPreviewRunId);
+}
+function getCompletedDraftPreviewToSuppress({
+  endingTourId,
+  endingPreviewRunId,
+  experienceType
+}) {
+  if (!shouldSuppressCompletedDraftPreview({ endingTourId, endingPreviewRunId })) {
+    return null;
+  }
+  return {
+    id: endingTourId,
+    experienceType
+  };
+}
 function resolveElement(step) {
   const el = step.element;
   if (!el) return null;
@@ -3970,7 +3994,7 @@ function useTourPlayback({
               }
               await waitForUserAction(gateTarget, segment.gate.event);
             }
-            if (segmentText && showCaptionsRef.current && reviewModeRef.current) {
+            if (segmentText && showCaptionsRef.current) {
               showCaption(segmentText);
             }
             const nextSegmentText = (segments[index + 1]?.text ?? "").trim();
@@ -3997,7 +4021,7 @@ function useTourPlayback({
           if (params.removeHighlightAtEnd !== false) {
             removeHighlight();
           }
-          if (showCaptionsRef.current && reviewModeRef.current) {
+          if (showCaptionsRef.current) {
             removeCaption();
           }
           return !!params.waitForInput;
@@ -4007,7 +4031,7 @@ function useTourPlayback({
           if (!text.trim()) {
             return { result: null };
           }
-          if (showCaptionsRef.current && reviewModeRef.current) {
+          if (showCaptionsRef.current) {
             showCaption(text);
           }
           const settlePromise = voice.speak(text, tourRef.current?.voice?.ttsVoice, {
@@ -4436,9 +4460,8 @@ function useTourPlayback({
       }
       const tour = tourData.tourContext ?? tourRef.current;
       const expType = experienceTypeRef.current;
-      const matchesType = !tour?.type || tour.type === expType;
-      const isCompatibleResumption = !!tourData.resumed && (expType === "onboarding" && tour?.type === "tour" || expType === "tour" && tour?.type === "onboarding");
-      if (!matchesType && !isCompatibleResumption) {
+      const isCompatible = shouldAcceptTourExperienceType(expType, tour?.type, !!tourData.resumed);
+      if (!isCompatible) {
         emitSdkDebugLog("[TourClient] Ignoring tour start for mismatched experience type", {
           incomingType: tour?.type,
           hookType: expType,
@@ -4586,10 +4609,10 @@ function useTourPlayback({
     return () => clearTimeout(timer);
   }, [disabled, emitTourInit, websiteId, userProfile?.userId, userProfile?.type]);
   useEffect10(() => {
-    if (!showCaptions || !isReviewMode) {
+    if (!showCaptions) {
       removeCaption();
     }
-  }, [showCaptions, isReviewMode]);
+  }, [showCaptions]);
   useEffect10(() => {
     if (!isActiveRef.current) return;
     emitSocketEvent(socketRef.current, "tour:client_state", {
@@ -4715,6 +4738,11 @@ function useTourPlayback({
     const endingTourId = tourRef.current?.id;
     const endingPreviewRunId = previewRunIdRef.current;
     const endingStepOrder = stepIndexRef.current;
+    const completedPreviewDraft = getCompletedDraftPreviewToSuppress({
+      endingTourId,
+      endingPreviewRunId,
+      experienceType
+    });
     isActiveRef.current = false;
     startRequestedRef.current = false;
     releasePlaybackOwnership();
@@ -4724,6 +4752,10 @@ function useTourPlayback({
     runCleanup(pendingManualWaitCleanupRef.current);
     pendingManualWaitCleanupRef.current = null;
     clearPendingManualInputSync();
+    if (completedPreviewDraft) {
+      persistSuppressedDraftPreview(completedPreviewDraft);
+      clearActiveDraftPreview(experienceType);
+    }
     if (!skipRequestedRef.current && userProfile?.userId && tourRef.current) {
       markTourComplete(serverUrl, toursApiBaseRef.current, tourRef.current.id, userProfile.userId, experienceType, websiteId);
     }
@@ -4748,7 +4780,7 @@ function useTourPlayback({
       });
     }
     if (reviewModeRef.current) {
-      if (endingTourId) {
+      if (endingTourId && !completedPreviewDraft) {
         persistSuppressedDraftPreview({
           id: endingTourId,
           experienceType

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@modelnex/sdk",
-  "version": "0.5.55",
+  "version": "0.5.59",
   "description": "React SDK for natural language control of web apps via AI agents",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",