fix(tour): move auto-start flag into timer, fix truncate selector conflict

- Move hasAutoStarted flag inside setTimeout callback so it's only set
  when the timer fires, allowing retry if disabled changes during delay
- Add data-popover-scroll attribute to showArrow scroll wrapper and
  exclude it from the flex-1 truncate selector to prevent overflow
  conflict

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/components/product-tour/use-tour.ts

@@ -145,9 +145,9 @@ export function useTour({
   /** Auto-start on first visit */
   useEffect(() => {
     if (disabled || hasAutoStarted.current) return
-    hasAutoStarted.current = true
 
     const timer = setTimeout(() => {
+      hasAutoStarted.current = true
       if (!isTourCompleted(storageKey)) {
         setStepIndex(0)
         setIsEntrance(true)

apps/sim/components/emcn/components/popover/popover.tsx

@@ -610,7 +610,8 @@ const PopoverContent = React.forwardRef<
           showArrow ? 'overflow-visible' : 'overflow-auto',
           STYLES.colorScheme[colorScheme].content,
           STYLES.content,
-          hasUserWidthConstraint && '[&_.flex-1]:truncate [&_[data-popover-section]]:truncate',
+          hasUserWidthConstraint &&
+            '[&_.flex-1:not([data-popover-scroll])]:truncate [&_[data-popover-section]]:truncate',
           border && 'border border-[var(--border-1)]',
           className
         )}
@@ -628,7 +629,13 @@ const PopoverContent = React.forwardRef<
           ...style,
         }}
       >
-        {showArrow ? <div className='flex-1 overflow-auto'>{children}</div> : children}
+        {showArrow ? (
+          <div data-popover-scroll className='min-h-0 flex-1 overflow-auto'>
+            {children}
+          </div>
+        ) : (
+          children
+        )}
         {showArrow && (
           <PopoverPrimitive.Arrow width={14} height={7} asChild>
             <svg