Fix rewind tab: remove intro video, fix delayed screenshots (#5802)

## Summary
- Remove the rewind intro video overlay — users see their screenshots
directly on the Rewind tab
- Fix 5-minute delay before screenshots appear after onboarding:
notification auth callback was blocking `continueStartMonitoring`, and
first capture waited for the 3s timer interval instead of firing
immediately

## Test plan
- [ ] Fresh install: complete onboarding, navigate to Rewind tab — no
video overlay shown
- [ ] Screenshots appear within seconds of monitoring start, not minutes
- [ ] Notification permission still requested (in parallel) without
blocking capture

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Author: kodjima33

CLAUDE.md

@@ -8,6 +8,48 @@
 
 - Never ask for permission to access folders, run commands, search the web, or use tools. Just do it.
 - Never ask for confirmation. Just act. Make decisions autonomously and proceed without checking in.
+- You have full access to the user's computer — browser, desktop, all apps. Never ask the user to do something you can do yourself (sign in, click buttons, dismiss dialogs, etc.).
+
+## Computer Control (clicking, typing, screenshots)
+
+You have multiple MCP tools for controlling the Mac. Use the **right tool for each job** — don't bounce between tools.
+
+### For clicking at coordinates — use `cliclick` (FASTEST)
+```bash
+cliclick c:X,Y        # click
+cliclick dc:X,Y       # double-click
+cliclick rc:X,Y       # right-click
+cliclick m:X,Y        # move mouse
+cliclick t:"text"     # type text
+cliclick p            # print current mouse position
+cliclick kd:cmd ku:cmd  # key down/up
+```
+`cliclick` uses CGEvent, handles Retina correctly, works across all displays. No MCP overhead.
+
+### For screenshots — use `codriver`
+- `mcp__codriver__desktop_screenshot` — capture screen (use `scale: 0.5` for speed)
+- `mcp__codriver__desktop_ocr` — find text positions on screen
+- `mcp__codriver__desktop_windows` — list/focus windows
+
+### Workflow: screenshot → find target → click
+1. Take screenshot with `codriver` to see the screen
+2. Identify the coordinates of what to click (use OCR if needed)
+3. Click with `cliclick c:X,Y` via Bash — instant, reliable
+
+### For native macOS app testing — use `agent-swift`
+Already documented below. Use for connected SwiftUI/AppKit apps.
+
+### For browser interaction — priority order:
+1. **`playwright`** MCP — headless browser, most reliable for web automation
+2. **`claude-in-chrome`** — for existing browser tabs (only when extension is connected)
+3. **`codriver` screenshot + `cliclick`** — fallback if browser tools fail
+
+### Rules:
+- NEVER try 3+ different click tools for the same action — pick one and commit
+- For multi-monitor: always check coordinates against the screenshot scale factor
+- `codriver` screenshots at `scale: 0.5` means multiply coordinates by 2 before clicking
+- Prefer `cliclick` over `automac`/`mac-use-mcp` click — they have coordinate bugs on multi-monitor
+- When a tool errors (e.g., "helper binary not found", "extension not connected"), immediately switch to the fallback — don't retry the broken tool
 
 ## Setup
 

desktop/CHANGELOG.json

@@ -1,5 +1,8 @@
 {
-  "unreleased": [],
+  "unreleased": [
+    "Removed intro video from Rewind tab \u2014 screenshots are shown immediately",
+    "Fixed delayed screenshot capture after onboarding \u2014 first screenshot is now taken instantly"
+  ],
   "releases": [
     {
       "version": "0.11.133",

desktop/Desktop/Sources/ProactiveAssistants/ProactiveAssistantsPlugin.swift

@@ -208,9 +208,9 @@ public class ProactiveAssistantsPlugin: NSObject {
             return
         }
 
-        // Request notification permission but don't block on it
-        // Screen analysis can work without notifications - users just won't get alerts
-        UNUserNotificationCenter.current().requestAuthorization(options: [.alert, .sound, .badge]) { [weak self] granted, error in
+        // Request notification permission in parallel — don't block monitoring on it.
+        // Screen analysis can work without notifications - users just won't get alerts.
+        UNUserNotificationCenter.current().requestAuthorization(options: [.alert, .sound, .badge]) { granted, error in
             DispatchQueue.main.async {
                 if let error = error {
                     let nsError = error as NSError
@@ -232,11 +232,11 @@ public class ProactiveAssistantsPlugin: NSObject {
                 if !granted {
                     log("Notification permission not granted - screen analysis will work but notifications will be disabled")
                 }
-
-                // Continue with monitoring regardless of notification permission
-                self?.continueStartMonitoring(completion: completion)
             }
         }
+
+        // Start monitoring immediately — don't wait for notification permission callback
+        continueStartMonitoring(completion: completion)
     }
 
     /// Repair LaunchServices registration when notification authorization fails with "not allowed".
@@ -392,6 +392,12 @@ public class ProactiveAssistantsPlugin: NSObject {
         }
 
         isMonitoring = true
+
+        // Capture the first frame immediately so screenshots appear right away
+        // (don't wait for the first timer interval to elapse)
+        Task { @MainActor in
+            await self.captureFrame()
+        }
         isStartingMonitoring = false
 
         // Report resources after initialization

desktop/Desktop/Sources/Rewind/UI/RewindPage.swift

@@ -1,6 +1,5 @@
 import SwiftUI
 import AppKit
-import AVKit
 
 /// Main Rewind page - Timeline-first view with integrated search
 /// The timeline is the primary interface, with search results highlighted inline
@@ -41,8 +40,6 @@ struct RewindPage: View {
     @State private var showNameSpeakerSheet = false
     @State private var selectedSpeakerSegment: SpeakerSegment? = nil
 
-    // Rewind intro video (first-time experience)
-    @AppStorage("hasSeenRewindIntro") private var hasSeenRewindIntro = false
 
     enum SearchViewMode {
         case results  // Full-screen search results
@@ -132,10 +129,6 @@ struct RewindPage: View {
                 .frame(maxWidth: .infinity, maxHeight: .infinity, alignment: .top)
             }
 
-            // Rewind intro video overlay (first-time experience)
-            if !hasSeenRewindIntro {
-                rewindIntroOverlay
-            }
         }
         .focusable()
         .focused($isPageFocused)
@@ -906,49 +899,6 @@ struct RewindPage: View {
     }
 
 
-    // MARK: - Rewind Intro Video
-
-    private var rewindIntroOverlay: some View {
-        ZStack {
-            Color.black.opacity(0.85)
-                .ignoresSafeArea()
-
-            VStack(spacing: 20) {
-                RewindIntroVideoView()
-                    .frame(maxWidth: 700, maxHeight: 394) // 16:9 aspect
-                    .clipShape(RoundedRectangle(cornerRadius: 12))
-                    .shadow(color: OmiColors.purplePrimary.opacity(0.3), radius: 20)
-
-                Button(action: {
-                    withAnimation(.easeOut(duration: 0.25)) {
-                        hasSeenRewindIntro = true
-                    }
-                }) {
-                    Text("Get Started")
-                        .scaledFont(size: 15, weight: .semibold)
-                        .foregroundColor(.white)
-                        .padding(.horizontal, 32)
-                        .padding(.vertical, 10)
-                        .background(OmiColors.purplePrimary)
-                        .clipShape(RoundedRectangle(cornerRadius: 10))
-                }
-                .buttonStyle(.plain)
-
-                Button(action: {
-                    withAnimation(.easeOut(duration: 0.25)) {
-                        hasSeenRewindIntro = true
-                    }
-                }) {
-                    Text("Skip")
-                        .scaledFont(size: 13)
-                        .foregroundColor(.white.opacity(0.5))
-                }
-                .buttonStyle(.plain)
-            }
-        }
-        .transition(.opacity)
-    }
-
     // MARK: - Empty States
 
     private var emptyState: some View {
@@ -1882,44 +1832,6 @@ extension View {
     }
 }
 
-// MARK: - Rewind Intro Video Player
-
-struct RewindIntroVideoView: NSViewRepresentable {
-    func makeCoordinator() -> Coordinator {
-        Coordinator()
-    }
-
-    func makeNSView(context: Context) -> AVPlayerView {
-        let playerView = AVPlayerView()
-        if let url = Bundle.resourceBundle.url(forResource: "rewind-demo", withExtension: "mp4") {
-            let player = AVPlayer(url: url)
-            playerView.player = player
-            playerView.controlsStyle = .inline
-            playerView.showsFullScreenToggleButton = false
-            player.play()
-
-            NotificationCenter.default.addObserver(
-                context.coordinator,
-                selector: #selector(Coordinator.playerDidFinishPlaying(_:)),
-                name: .AVPlayerItemDidPlayToEndTime,
-                object: player.currentItem
-            )
-            context.coordinator.player = player
-        }
-        return playerView
-    }
-
-    func updateNSView(_ nsView: AVPlayerView, context: Context) {}
-
-    class Coordinator: NSObject {
-        var player: AVPlayer?
-
-        @objc func playerDidFinishPlaying(_ notification: Notification) {
-            player?.seek(to: .zero)
-            player?.play()
-        }
-    }
-}
 
 #Preview {
     RewindPage()