@kelet-ai/feedback-ui 0.7.0 → 1.0.0

package/dist/feedback-ui.umd.js

@@ -356,7 +356,7 @@
   }
   var jsxRuntimeExports = requireJsxRuntime();
   const KeletContext = require$$0$1.createContext(null);
-  const KeletBaseUrl = "https://api.kelet.ai/api";
+  const DefaultKeletBaseUrl = "https://api.kelet.ai/api";
   const useKelet = () => {
     const context = require$$0$1.useContext(KeletContext);
     if (!context) {
@@ -376,7 +376,7 @@
       return context.feedback;
     }
   };
-  const KeletProvider = ({ apiKey, project, children }) => {
+  const KeletProvider = ({ apiKey, project, baseUrl, children }) => {
     const parentContext = require$$0$1.useContext(KeletContext);
     const resolvedApiKey = apiKey || parentContext?.api_key;
     if (!resolvedApiKey) {
@@ -385,7 +385,8 @@
       );
     }
     const feedback = async (data) => {
-      const url = `${KeletBaseUrl}/projects/${project}/feedback`;
+      const resolvedBaseUrl = baseUrl || DefaultKeletBaseUrl;
+      const url = `${resolvedBaseUrl}/projects/${project}/feedback`;
       const req = {
         tx_id: data.tx_id,
         source: data.source || "EXPLICIT",
@@ -1867,7 +1868,7 @@
     }
     return result;
   }
-  function formatDiff(oldValue, newValue, diffType = "git", context = 3) {
+  function formatDiff(oldValue, newValue, diffType = "git", context = 1) {
     switch (diffType) {
       case "git":
         return formatGitDiff(oldValue, newValue, context);
@@ -1879,7 +1880,7 @@
         return formatGitDiff(oldValue, newValue, context);
     }
   }
-  function formatGitDiff(oldValue, newValue, context = 3) {
+  function formatGitDiff(oldValue, newValue, context = 1) {
     const oldStr = stringify(oldValue);
     const newStr = stringify(newValue);
     const patch = createTwoFilesPatch(
@@ -1895,7 +1896,20 @@
       "",
       { context }
     );
-    return patch.split("\n").slice(2).join("\n");
+    const lines = patch.split("\n");
+    const filtered = lines.filter((line) => {
+      if (!line) return false;
+      if (line === "\") return false;
+      if (line.startsWith("@@")) return false;
+      if (line.startsWith("---") || line.startsWith("+++")) return false;
+      if (line.startsWith(" ")) return false;
+      if (line.startsWith("+") || line.startsWith("-")) return true;
+      return false;
+    });
+    while (filtered.length && filtered[0].trim() === "") filtered.shift();
+    while (filtered.length && filtered[filtered.length - 1].trim() === "")
+      filtered.pop();
+    return filtered.join("\n");
   }
   function formatObjectDiff(oldValue, newValue) {
     const differences = deepDiffExports.diff(oldValue, newValue) || [];
@@ -2008,19 +2022,22 @@
   function useStateChangeTracking(currentState, tx_id, options) {
     const defaultFeedbackHandler = useDefaultFeedbackHandler();
     const feedbackHandler = options?.onFeedback || defaultFeedbackHandler;
-    const debounceMs = options?.debounceMs ?? 1500;
+    const debounceMs = options?.debounceMs ?? 3e3;
     const diffType = options?.diffType ?? "git";
     const compareWith = options?.compareWith;
     const defaultTriggerName = options?.default_trigger_name ?? "auto_state_change";
     const ignoreInitialNullish = options?.ignoreInitialNullish ?? true;
     const prevStateRef = require$$0$1.useRef(currentState);
-    const changeStartStateRef = require$$0$1.useRef(currentState);
     const isFirstRenderRef = require$$0$1.useRef(true);
     const initialStateRef = require$$0$1.useRef(currentState);
+    const initialWasNullishRef = require$$0$1.useRef(currentState == null);
     const hasHadNonNullishStateRef = require$$0$1.useRef(
       currentState != null
       // != null catches both null and undefined
     );
+    const hasEligibleBaselineRef = require$$0$1.useRef(
+      !(ignoreInitialNullish && currentState == null)
+    );
     const timeoutRef = require$$0$1.useRef(null);
     const currentTriggerNameRef = require$$0$1.useRef(void 0);
     const sendFeedback = require$$0$1.useCallback(
@@ -2055,7 +2072,7 @@
         const newTriggerName = trigger_name || defaultTriggerName;
         if (timeoutRef.current && currentTriggerNameRef.current && currentTriggerNameRef.current !== newTriggerName) {
           clearTimeout(timeoutRef.current);
-          const startState = changeStartStateRef.current;
+          const startState = initialStateRef.current;
           const currentStateBeforeChange = currentState;
           sendFeedback(
             startState,
@@ -2072,39 +2089,41 @@
       if (isFirstRenderRef.current) {
         isFirstRenderRef.current = false;
         prevStateRef.current = currentState;
-        changeStartStateRef.current = currentState;
+        if (!ignoreInitialNullish || currentState != null) {
+          initialStateRef.current = currentState;
+          hasEligibleBaselineRef.current = true;
+        }
         return;
       }
       const prevState = prevStateRef.current;
       const isEqual = compareWith ? compareWith(prevState, currentState) : JSON.stringify(prevState) === JSON.stringify(currentState);
       if (!isEqual) {
-        const shouldIgnoreChange = ignoreInitialNullish && initialStateRef.current == null && // Initial state was nullish
+        const shouldIgnoreChange = ignoreInitialNullish && initialWasNullishRef.current && // True initial state was nullish
         !hasHadNonNullishStateRef.current && // We haven't had non-nullish state before
         currentState != null;
         if (currentState != null) {
           hasHadNonNullishStateRef.current = true;
         }
         if (shouldIgnoreChange) {
+          initialStateRef.current = currentState;
+          hasEligibleBaselineRef.current = true;
           prevStateRef.current = currentState;
-          changeStartStateRef.current = currentState;
           return;
         }
-        if (!timeoutRef.current) {
-          changeStartStateRef.current = prevState;
-        }
         if (timeoutRef.current) {
           clearTimeout(timeoutRef.current);
         }
         prevStateRef.current = currentState;
         timeoutRef.current = setTimeout(() => {
-          const startState = changeStartStateRef.current;
+          const startState = initialStateRef.current;
           const finalState = currentState;
-          sendFeedback(
-            startState,
-            finalState,
-            currentTriggerNameRef.current || defaultTriggerName
-          );
-          changeStartStateRef.current = finalState;
+          if (hasEligibleBaselineRef.current) {
+            sendFeedback(
+              startState,
+              finalState,
+              currentTriggerNameRef.current || defaultTriggerName
+            );
+          }
           timeoutRef.current = null;
         }, debounceMs);
       }