@eventop/sdk 1.2.12 → 1.2.13

package/dist/index.cjs

@@ -315,18 +315,28 @@ function EventopTarget({
   navigate,
   navigateWaitFor,
   advanceOn,
-  waitFor,
-  ...rest
+  waitFor
 }) {
   const registry = useRegistry();
-  const ref = react.useRef(null);
+  const wrapperRef = react.useRef(null);
   const dataAttr = `data-evtp-${id}`;
   const selector = `[${dataAttr}]`;
   react.useEffect(() => {
+    var _wrapperRef$current;
     if (!id || !name) {
       console.warn('[Eventop] <EventopTarget> requires id and name props.');
       return;
     }
+
+    // Set the attribute directly on the first real child DOM element.
+    // This bypasses React's prop system entirely — works regardless of whether
+    // the wrapped component forwards unknown props.
+    const firstChild = (_wrapperRef$current = wrapperRef.current) === null || _wrapperRef$current === void 0 ? void 0 : _wrapperRef$current.firstElementChild;
+    if (firstChild) {
+      firstChild.setAttribute(dataAttr, '');
+    } else {
+      console.warn(`[Eventop] <EventopTarget id="${id}"> could not find a child DOM element to attach to. ` + `Make sure the wrapped component renders at least one DOM element.`);
+    }
     registry.registerFeature({
       id,
       name,
@@ -341,32 +351,24 @@ function EventopTarget({
         ...advanceOn
       } : null
     });
-    return () => registry.unregisterFeature(id);
-  }, [id, name, description, route]);
-  const child = react.Children.only(children);
-  let wrapped;
-  try {
-    wrapped = /*#__PURE__*/react.cloneElement(child, {
-      [dataAttr]: '',
-      ref: node => {
-        ref.current = node;
-        const originalRef = child.ref;
-        if (typeof originalRef === 'function') originalRef(node);else if (originalRef && 'current' in originalRef) originalRef.current = node;
-      }
-    });
-  } catch {
-    wrapped = /*#__PURE__*/jsxRuntime.jsx("span", {
-      [dataAttr]: '',
-      ref: ref,
+    return () => {
+      var _wrapperRef$current2;
+      // Clean up the injected attribute on unmount
+      const el = (_wrapperRef$current2 = wrapperRef.current) === null || _wrapperRef$current2 === void 0 ? void 0 : _wrapperRef$current2.firstElementChild;
+      if (el) el.removeAttribute(dataAttr);
+      registry.unregisterFeature(id);
+    };
+  }, [id, name, description, route]); // eslint-disable-line react-hooks/exhaustive-deps
+
+  return /*#__PURE__*/jsxRuntime.jsx(EventopFeatureScopeContext.Provider, {
+    value: id,
+    children: /*#__PURE__*/jsxRuntime.jsx("span", {
+      ref: wrapperRef,
       style: {
         display: 'contents'
       },
-      children: child
-    });
-  }
-  return /*#__PURE__*/jsxRuntime.jsx(EventopFeatureScopeContext.Provider, {
-    value: id,
-    children: wrapped
+      children: react.Children.only(children)
+    })
   });
 }
 
@@ -381,7 +383,7 @@ function EventopStep({
   const registry = useRegistry();
   const featureScope = useFeatureScope();
   const featureId = feature || featureScope;
-  const ref = react.useRef(null);
+  const wrapperRef = react.useRef(null);
   if (!featureId) {
     console.warn('[Eventop] <EventopStep> needs either a feature prop or an <EventopTarget> ancestor.');
   }
@@ -391,7 +393,14 @@ function EventopStep({
   const dataAttr = `data-evtp-step-${featureId}-${parentStep != null ? `${parentStep}-` : ''}${index}`;
   const selector = `[${dataAttr}]`;
   react.useEffect(() => {
+    var _wrapperRef$current;
     if (!featureId || index == null) return;
+
+    // Inject attribute directly onto the first real child DOM element
+    const firstChild = (_wrapperRef$current = wrapperRef.current) === null || _wrapperRef$current === void 0 ? void 0 : _wrapperRef$current.firstElementChild;
+    if (firstChild) {
+      firstChild.setAttribute(dataAttr, '');
+    }
     registry.registerStep(featureId, index, parentStep ?? null, {
       selector,
       waitFor: waitFor || null,
@@ -400,30 +409,21 @@ function EventopStep({
         ...advanceOn
       } : null
     });
-    return () => registry.unregisterStep(featureId, index, parentStep ?? null);
-  }, [featureId, index, parentStep]);
-  const child = react.Children.only(children);
-  let wrapped;
-  try {
-    wrapped = /*#__PURE__*/react.cloneElement(child, {
-      [dataAttr]: '',
-      ref: node => {
-        ref.current = node;
-        const originalRef = child.ref;
-        if (typeof originalRef === 'function') originalRef(node);else if (originalRef && 'current' in originalRef) originalRef.current = node;
-      }
-    });
-  } catch {
-    wrapped = /*#__PURE__*/jsxRuntime.jsx("span", {
-      [dataAttr]: '',
-      ref: ref,
-      style: {
-        display: 'contents'
-      },
-      children: child
-    });
-  }
-  return wrapped;
+    return () => {
+      var _wrapperRef$current2;
+      const el = (_wrapperRef$current2 = wrapperRef.current) === null || _wrapperRef$current2 === void 0 ? void 0 : _wrapperRef$current2.firstElementChild;
+      if (el) el.removeAttribute(dataAttr);
+      registry.unregisterStep(featureId, index, parentStep ?? null);
+    };
+  }, [featureId, index, parentStep]); // eslint-disable-line react-hooks/exhaustive-deps
+
+  return /*#__PURE__*/jsxRuntime.jsx("span", {
+    ref: wrapperRef,
+    style: {
+      display: 'contents'
+    },
+    children: react.Children.only(children)
+  });
 }
 
 // ═══════════════════════════════════════════════════════════════════════════

package/dist/index.js

@@ -1,4 +1,4 @@
-import { createContext, useContext, useRef, useCallback, useEffect, Children, cloneElement, useState } from 'react';
+import { createContext, useContext, useRef, useCallback, useEffect, Children, useState } from 'react';
 import { jsx } from 'react/jsx-runtime';
 
 /**
@@ -313,18 +313,28 @@ function EventopTarget({
   navigate,
   navigateWaitFor,
   advanceOn,
-  waitFor,
-  ...rest
+  waitFor
 }) {
   const registry = useRegistry();
-  const ref = useRef(null);
+  const wrapperRef = useRef(null);
   const dataAttr = `data-evtp-${id}`;
   const selector = `[${dataAttr}]`;
   useEffect(() => {
+    var _wrapperRef$current;
     if (!id || !name) {
       console.warn('[Eventop] <EventopTarget> requires id and name props.');
       return;
     }
+
+    // Set the attribute directly on the first real child DOM element.
+    // This bypasses React's prop system entirely — works regardless of whether
+    // the wrapped component forwards unknown props.
+    const firstChild = (_wrapperRef$current = wrapperRef.current) === null || _wrapperRef$current === void 0 ? void 0 : _wrapperRef$current.firstElementChild;
+    if (firstChild) {
+      firstChild.setAttribute(dataAttr, '');
+    } else {
+      console.warn(`[Eventop] <EventopTarget id="${id}"> could not find a child DOM element to attach to. ` + `Make sure the wrapped component renders at least one DOM element.`);
+    }
     registry.registerFeature({
       id,
       name,
@@ -339,32 +349,24 @@ function EventopTarget({
         ...advanceOn
       } : null
     });
-    return () => registry.unregisterFeature(id);
-  }, [id, name, description, route]);
-  const child = Children.only(children);
-  let wrapped;
-  try {
-    wrapped = /*#__PURE__*/cloneElement(child, {
-      [dataAttr]: '',
-      ref: node => {
-        ref.current = node;
-        const originalRef = child.ref;
-        if (typeof originalRef === 'function') originalRef(node);else if (originalRef && 'current' in originalRef) originalRef.current = node;
-      }
-    });
-  } catch {
-    wrapped = /*#__PURE__*/jsx("span", {
-      [dataAttr]: '',
-      ref: ref,
+    return () => {
+      var _wrapperRef$current2;
+      // Clean up the injected attribute on unmount
+      const el = (_wrapperRef$current2 = wrapperRef.current) === null || _wrapperRef$current2 === void 0 ? void 0 : _wrapperRef$current2.firstElementChild;
+      if (el) el.removeAttribute(dataAttr);
+      registry.unregisterFeature(id);
+    };
+  }, [id, name, description, route]); // eslint-disable-line react-hooks/exhaustive-deps
+
+  return /*#__PURE__*/jsx(EventopFeatureScopeContext.Provider, {
+    value: id,
+    children: /*#__PURE__*/jsx("span", {
+      ref: wrapperRef,
       style: {
         display: 'contents'
       },
-      children: child
-    });
-  }
-  return /*#__PURE__*/jsx(EventopFeatureScopeContext.Provider, {
-    value: id,
-    children: wrapped
+      children: Children.only(children)
+    })
   });
 }
 
@@ -379,7 +381,7 @@ function EventopStep({
   const registry = useRegistry();
   const featureScope = useFeatureScope();
   const featureId = feature || featureScope;
-  const ref = useRef(null);
+  const wrapperRef = useRef(null);
   if (!featureId) {
     console.warn('[Eventop] <EventopStep> needs either a feature prop or an <EventopTarget> ancestor.');
   }
@@ -389,7 +391,14 @@ function EventopStep({
   const dataAttr = `data-evtp-step-${featureId}-${parentStep != null ? `${parentStep}-` : ''}${index}`;
   const selector = `[${dataAttr}]`;
   useEffect(() => {
+    var _wrapperRef$current;
     if (!featureId || index == null) return;
+
+    // Inject attribute directly onto the first real child DOM element
+    const firstChild = (_wrapperRef$current = wrapperRef.current) === null || _wrapperRef$current === void 0 ? void 0 : _wrapperRef$current.firstElementChild;
+    if (firstChild) {
+      firstChild.setAttribute(dataAttr, '');
+    }
     registry.registerStep(featureId, index, parentStep ?? null, {
       selector,
       waitFor: waitFor || null,
@@ -398,30 +407,21 @@ function EventopStep({
         ...advanceOn
       } : null
     });
-    return () => registry.unregisterStep(featureId, index, parentStep ?? null);
-  }, [featureId, index, parentStep]);
-  const child = Children.only(children);
-  let wrapped;
-  try {
-    wrapped = /*#__PURE__*/cloneElement(child, {
-      [dataAttr]: '',
-      ref: node => {
-        ref.current = node;
-        const originalRef = child.ref;
-        if (typeof originalRef === 'function') originalRef(node);else if (originalRef && 'current' in originalRef) originalRef.current = node;
-      }
-    });
-  } catch {
-    wrapped = /*#__PURE__*/jsx("span", {
-      [dataAttr]: '',
-      ref: ref,
-      style: {
-        display: 'contents'
-      },
-      children: child
-    });
-  }
-  return wrapped;
+    return () => {
+      var _wrapperRef$current2;
+      const el = (_wrapperRef$current2 = wrapperRef.current) === null || _wrapperRef$current2 === void 0 ? void 0 : _wrapperRef$current2.firstElementChild;
+      if (el) el.removeAttribute(dataAttr);
+      registry.unregisterStep(featureId, index, parentStep ?? null);
+    };
+  }, [featureId, index, parentStep]); // eslint-disable-line react-hooks/exhaustive-deps
+
+  return /*#__PURE__*/jsx("span", {
+    ref: wrapperRef,
+    style: {
+      display: 'contents'
+    },
+    children: Children.only(children)
+  });
 }
 
 // ═══════════════════════════════════════════════════════════════════════════

package/dist/react/index.cjs

@@ -315,18 +315,28 @@ function EventopTarget({
   navigate,
   navigateWaitFor,
   advanceOn,
-  waitFor,
-  ...rest
+  waitFor
 }) {
   const registry = useRegistry();
-  const ref = react.useRef(null);
+  const wrapperRef = react.useRef(null);
   const dataAttr = `data-evtp-${id}`;
   const selector = `[${dataAttr}]`;
   react.useEffect(() => {
+    var _wrapperRef$current;
     if (!id || !name) {
       console.warn('[Eventop] <EventopTarget> requires id and name props.');
       return;
     }
+
+    // Set the attribute directly on the first real child DOM element.
+    // This bypasses React's prop system entirely — works regardless of whether
+    // the wrapped component forwards unknown props.
+    const firstChild = (_wrapperRef$current = wrapperRef.current) === null || _wrapperRef$current === void 0 ? void 0 : _wrapperRef$current.firstElementChild;
+    if (firstChild) {
+      firstChild.setAttribute(dataAttr, '');
+    } else {
+      console.warn(`[Eventop] <EventopTarget id="${id}"> could not find a child DOM element to attach to. ` + `Make sure the wrapped component renders at least one DOM element.`);
+    }
     registry.registerFeature({
       id,
       name,
@@ -341,32 +351,24 @@ function EventopTarget({
         ...advanceOn
       } : null
     });
-    return () => registry.unregisterFeature(id);
-  }, [id, name, description, route]);
-  const child = react.Children.only(children);
-  let wrapped;
-  try {
-    wrapped = /*#__PURE__*/react.cloneElement(child, {
-      [dataAttr]: '',
-      ref: node => {
-        ref.current = node;
-        const originalRef = child.ref;
-        if (typeof originalRef === 'function') originalRef(node);else if (originalRef && 'current' in originalRef) originalRef.current = node;
-      }
-    });
-  } catch {
-    wrapped = /*#__PURE__*/jsxRuntime.jsx("span", {
-      [dataAttr]: '',
-      ref: ref,
+    return () => {
+      var _wrapperRef$current2;
+      // Clean up the injected attribute on unmount
+      const el = (_wrapperRef$current2 = wrapperRef.current) === null || _wrapperRef$current2 === void 0 ? void 0 : _wrapperRef$current2.firstElementChild;
+      if (el) el.removeAttribute(dataAttr);
+      registry.unregisterFeature(id);
+    };
+  }, [id, name, description, route]); // eslint-disable-line react-hooks/exhaustive-deps
+
+  return /*#__PURE__*/jsxRuntime.jsx(EventopFeatureScopeContext.Provider, {
+    value: id,
+    children: /*#__PURE__*/jsxRuntime.jsx("span", {
+      ref: wrapperRef,
       style: {
         display: 'contents'
       },
-      children: child
-    });
-  }
-  return /*#__PURE__*/jsxRuntime.jsx(EventopFeatureScopeContext.Provider, {
-    value: id,
-    children: wrapped
+      children: react.Children.only(children)
+    })
   });
 }
 
@@ -381,7 +383,7 @@ function EventopStep({
   const registry = useRegistry();
   const featureScope = useFeatureScope();
   const featureId = feature || featureScope;
-  const ref = react.useRef(null);
+  const wrapperRef = react.useRef(null);
   if (!featureId) {
     console.warn('[Eventop] <EventopStep> needs either a feature prop or an <EventopTarget> ancestor.');
   }
@@ -391,7 +393,14 @@ function EventopStep({
   const dataAttr = `data-evtp-step-${featureId}-${parentStep != null ? `${parentStep}-` : ''}${index}`;
   const selector = `[${dataAttr}]`;
   react.useEffect(() => {
+    var _wrapperRef$current;
     if (!featureId || index == null) return;
+
+    // Inject attribute directly onto the first real child DOM element
+    const firstChild = (_wrapperRef$current = wrapperRef.current) === null || _wrapperRef$current === void 0 ? void 0 : _wrapperRef$current.firstElementChild;
+    if (firstChild) {
+      firstChild.setAttribute(dataAttr, '');
+    }
     registry.registerStep(featureId, index, parentStep ?? null, {
       selector,
       waitFor: waitFor || null,
@@ -400,30 +409,21 @@ function EventopStep({
         ...advanceOn
       } : null
     });
-    return () => registry.unregisterStep(featureId, index, parentStep ?? null);
-  }, [featureId, index, parentStep]);
-  const child = react.Children.only(children);
-  let wrapped;
-  try {
-    wrapped = /*#__PURE__*/react.cloneElement(child, {
-      [dataAttr]: '',
-      ref: node => {
-        ref.current = node;
-        const originalRef = child.ref;
-        if (typeof originalRef === 'function') originalRef(node);else if (originalRef && 'current' in originalRef) originalRef.current = node;
-      }
-    });
-  } catch {
-    wrapped = /*#__PURE__*/jsxRuntime.jsx("span", {
-      [dataAttr]: '',
-      ref: ref,
-      style: {
-        display: 'contents'
-      },
-      children: child
-    });
-  }
-  return wrapped;
+    return () => {
+      var _wrapperRef$current2;
+      const el = (_wrapperRef$current2 = wrapperRef.current) === null || _wrapperRef$current2 === void 0 ? void 0 : _wrapperRef$current2.firstElementChild;
+      if (el) el.removeAttribute(dataAttr);
+      registry.unregisterStep(featureId, index, parentStep ?? null);
+    };
+  }, [featureId, index, parentStep]); // eslint-disable-line react-hooks/exhaustive-deps
+
+  return /*#__PURE__*/jsxRuntime.jsx("span", {
+    ref: wrapperRef,
+    style: {
+      display: 'contents'
+    },
+    children: react.Children.only(children)
+  });
 }
 
 // ═══════════════════════════════════════════════════════════════════════════

package/dist/react/index.js

@@ -1,4 +1,4 @@
-import { createContext, useContext, useRef, useCallback, useEffect, Children, cloneElement, useState } from 'react';
+import { createContext, useContext, useRef, useCallback, useEffect, Children, useState } from 'react';
 import { jsx } from 'react/jsx-runtime';
 
 /**
@@ -313,18 +313,28 @@ function EventopTarget({
   navigate,
   navigateWaitFor,
   advanceOn,
-  waitFor,
-  ...rest
+  waitFor
 }) {
   const registry = useRegistry();
-  const ref = useRef(null);
+  const wrapperRef = useRef(null);
   const dataAttr = `data-evtp-${id}`;
   const selector = `[${dataAttr}]`;
   useEffect(() => {
+    var _wrapperRef$current;
     if (!id || !name) {
       console.warn('[Eventop] <EventopTarget> requires id and name props.');
       return;
     }
+
+    // Set the attribute directly on the first real child DOM element.
+    // This bypasses React's prop system entirely — works regardless of whether
+    // the wrapped component forwards unknown props.
+    const firstChild = (_wrapperRef$current = wrapperRef.current) === null || _wrapperRef$current === void 0 ? void 0 : _wrapperRef$current.firstElementChild;
+    if (firstChild) {
+      firstChild.setAttribute(dataAttr, '');
+    } else {
+      console.warn(`[Eventop] <EventopTarget id="${id}"> could not find a child DOM element to attach to. ` + `Make sure the wrapped component renders at least one DOM element.`);
+    }
     registry.registerFeature({
       id,
       name,
@@ -339,32 +349,24 @@ function EventopTarget({
         ...advanceOn
       } : null
     });
-    return () => registry.unregisterFeature(id);
-  }, [id, name, description, route]);
-  const child = Children.only(children);
-  let wrapped;
-  try {
-    wrapped = /*#__PURE__*/cloneElement(child, {
-      [dataAttr]: '',
-      ref: node => {
-        ref.current = node;
-        const originalRef = child.ref;
-        if (typeof originalRef === 'function') originalRef(node);else if (originalRef && 'current' in originalRef) originalRef.current = node;
-      }
-    });
-  } catch {
-    wrapped = /*#__PURE__*/jsx("span", {
-      [dataAttr]: '',
-      ref: ref,
+    return () => {
+      var _wrapperRef$current2;
+      // Clean up the injected attribute on unmount
+      const el = (_wrapperRef$current2 = wrapperRef.current) === null || _wrapperRef$current2 === void 0 ? void 0 : _wrapperRef$current2.firstElementChild;
+      if (el) el.removeAttribute(dataAttr);
+      registry.unregisterFeature(id);
+    };
+  }, [id, name, description, route]); // eslint-disable-line react-hooks/exhaustive-deps
+
+  return /*#__PURE__*/jsx(EventopFeatureScopeContext.Provider, {
+    value: id,
+    children: /*#__PURE__*/jsx("span", {
+      ref: wrapperRef,
       style: {
         display: 'contents'
       },
-      children: child
-    });
-  }
-  return /*#__PURE__*/jsx(EventopFeatureScopeContext.Provider, {
-    value: id,
-    children: wrapped
+      children: Children.only(children)
+    })
   });
 }
 
@@ -379,7 +381,7 @@ function EventopStep({
   const registry = useRegistry();
   const featureScope = useFeatureScope();
   const featureId = feature || featureScope;
-  const ref = useRef(null);
+  const wrapperRef = useRef(null);
   if (!featureId) {
     console.warn('[Eventop] <EventopStep> needs either a feature prop or an <EventopTarget> ancestor.');
   }
@@ -389,7 +391,14 @@ function EventopStep({
   const dataAttr = `data-evtp-step-${featureId}-${parentStep != null ? `${parentStep}-` : ''}${index}`;
   const selector = `[${dataAttr}]`;
   useEffect(() => {
+    var _wrapperRef$current;
     if (!featureId || index == null) return;
+
+    // Inject attribute directly onto the first real child DOM element
+    const firstChild = (_wrapperRef$current = wrapperRef.current) === null || _wrapperRef$current === void 0 ? void 0 : _wrapperRef$current.firstElementChild;
+    if (firstChild) {
+      firstChild.setAttribute(dataAttr, '');
+    }
     registry.registerStep(featureId, index, parentStep ?? null, {
       selector,
       waitFor: waitFor || null,
@@ -398,30 +407,21 @@ function EventopStep({
         ...advanceOn
       } : null
     });
-    return () => registry.unregisterStep(featureId, index, parentStep ?? null);
-  }, [featureId, index, parentStep]);
-  const child = Children.only(children);
-  let wrapped;
-  try {
-    wrapped = /*#__PURE__*/cloneElement(child, {
-      [dataAttr]: '',
-      ref: node => {
-        ref.current = node;
-        const originalRef = child.ref;
-        if (typeof originalRef === 'function') originalRef(node);else if (originalRef && 'current' in originalRef) originalRef.current = node;
-      }
-    });
-  } catch {
-    wrapped = /*#__PURE__*/jsx("span", {
-      [dataAttr]: '',
-      ref: ref,
-      style: {
-        display: 'contents'
-      },
-      children: child
-    });
-  }
-  return wrapped;
+    return () => {
+      var _wrapperRef$current2;
+      const el = (_wrapperRef$current2 = wrapperRef.current) === null || _wrapperRef$current2 === void 0 ? void 0 : _wrapperRef$current2.firstElementChild;
+      if (el) el.removeAttribute(dataAttr);
+      registry.unregisterStep(featureId, index, parentStep ?? null);
+    };
+  }, [featureId, index, parentStep]); // eslint-disable-line react-hooks/exhaustive-deps
+
+  return /*#__PURE__*/jsx("span", {
+    ref: wrapperRef,
+    style: {
+      display: 'contents'
+    },
+    children: Children.only(children)
+  });
 }
 
 // ═══════════════════════════════════════════════════════════════════════════

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eventop/sdk",
-  "version": "1.2.12",
+  "version": "1.2.13",
   "description": "AI-powered guided tours for any web app. Drop-in, themeable, provider-agnostic.",
   "keywords": [
     "onboarding",