@agent-native/core 0.141.6 → 0.144.0

package/corpus/templates/clips/app/routes/r.$recordingId.tsx

@@ -183,21 +183,6 @@ function nativeSaveFailureMessage(reason: string | null | undefined): string {
   return "The desktop recorder finished and saved a local copy, but Clips could not upload it. You can retry from the Clips menu without recording again.";
 }
 
-function InsightsUnavailableState() {
-  const t = useT();
-
-  return (
-    <div className="flex h-full flex-col items-center justify-center px-8 py-12 text-center">
-      <p className="text-sm font-medium text-foreground">
-        {t("sharePage.ownerInsights")}
-      </p>
-      <p className="mt-2 max-w-[240px] text-sm leading-5 text-muted-foreground">
-        {t("sharePage.ownerInsightsDescription")}
-      </p>
-    </div>
-  );
-}
-
 function parseTimeParam(raw: string | null): number {
   if (!raw) return 0;
   const value = raw.trim();
@@ -299,18 +284,6 @@ export default function RecordingPage() {
   const readyMediaPollRef = useRef<{ key: string; until: number } | null>(null);
   const [metadataRefreshUntil, setMetadataRefreshUntil] = useState(0);
 
-  useEffect(() => {
-    if (
-      panelParam === "agent" ||
-      panelParam === "comments" ||
-      panelParam === "transcript" ||
-      panelParam === "insights" ||
-      panelParam === "settings"
-    ) {
-      setPanel(panelParam);
-    }
-  }, [panelParam]);
-
   const playerDataQ = useActionQuery<any>(
     "get-recording-player-data",
     {
@@ -414,6 +387,25 @@ export default function RecordingPage() {
   const transcriptCleanup = playerDataQ.data?.transcript?.cleanup ?? null;
   const ctas = playerDataQ.data?.ctas ?? [];
   const canEdit = role === "owner" || role === "admin" || role === "editor";
+  useEffect(() => {
+    if (!canEdit && (panel === "insights" || panel === "settings")) {
+      setPanel("comments");
+    }
+  }, [canEdit, panel]);
+
+  useEffect(() => {
+    if (
+      (panelParam === "agent" ||
+        panelParam === "comments" ||
+        panelParam === "transcript" ||
+        panelParam === "insights" ||
+        panelParam === "settings") &&
+      ((panelParam !== "insights" && panelParam !== "settings") || canEdit)
+    ) {
+      setPanel(panelParam);
+    }
+  }, [canEdit, panelParam]);
+
   const builderCredits =
     (playerDataQ.data?.builderCredits as BuilderCreditsStatus | null) ?? null;
   const titleGenerationPaused = Boolean(
@@ -1149,7 +1141,7 @@ export default function RecordingPage() {
       trigger("comments", t("recordingPage.activity")),
       trigger("transcript", t("recordingPage.transcript")),
       trigger("agent", t("recordingPage.agent")),
-      trigger("insights", t("recordingPage.insights")),
+      canEdit ? trigger("insights", t("recordingPage.insights")) : null,
       canEdit ? trigger("settings", t("recordingPage.settings")) : null,
     ];
 
@@ -1162,7 +1154,7 @@ export default function RecordingPage() {
         <TabsList
           className={cn(
             "mx-3 mt-3 grid w-auto",
-            canEdit ? "grid-cols-5" : "grid-cols-4",
+            canEdit ? "grid-cols-5" : "grid-cols-3",
           )}
         >
           {triggers}
@@ -1259,19 +1251,17 @@ export default function RecordingPage() {
             presentation={compact ? "share" : "default"}
           />
         </TabsContent>
-        <TabsContent
-          value="insights"
-          className="flex-1 min-h-0 mt-3 overflow-y-auto data-[state=inactive]:hidden"
-        >
-          {canEdit ? (
+        {canEdit ? (
+          <TabsContent
+            value="insights"
+            className="flex-1 min-h-0 mt-3 overflow-y-auto data-[state=inactive]:hidden"
+          >
             <InsightsPanel
               recordingId={recording.id}
               durationMs={recording.durationMs}
             />
-          ) : (
-            <InsightsUnavailableState />
-          )}
-        </TabsContent>
+          </TabsContent>
+        ) : null}
         {canEdit ? (
           <TabsContent
             value="settings"

package/corpus/templates/clips/app/routes/share.$shareId.tsx

@@ -529,6 +529,12 @@ export default function ShareRoute() {
   const viewerCanOpenDashboard = Boolean(
     dataQ.data?.data?.viewer?.canOpenDashboard,
   );
+  useEffect(() => {
+    if (!viewerCanEdit && panel === "insights") {
+      setPanel("comments");
+    }
+  }, [panel, viewerCanEdit]);
+
   const viewCount = Number(dataQ.data?.data?.viewCount ?? 0);
   const agentViewCount = Number(dataQ.data?.data?.agentViewCount ?? 0);
   const showTitleSkeleton = recording
@@ -1102,7 +1108,9 @@ export default function ShareRoute() {
           onValueChange={setPanel}
           className="flex h-full flex-col"
         >
-          <TabsList className="mx-3 mt-3 grid w-auto grid-cols-4">
+          <TabsList
+            className={`mx-3 mt-3 grid w-auto ${viewerCanEdit ? "grid-cols-4" : "grid-cols-3"}`}
+          >
             <TabsTrigger value="comments" className="text-xs gap-1">
               {t("recordingPage.activity")}
               {comments.length > 0 ? (
@@ -1117,9 +1125,11 @@ export default function ShareRoute() {
             <TabsTrigger value="agent" className="text-xs">
               {t("sharePage.agent")}
             </TabsTrigger>
-            <TabsTrigger value="insights" className="text-xs">
-              {t("sharePage.insights")}
-            </TabsTrigger>
+            {viewerCanEdit ? (
+              <TabsTrigger value="insights" className="text-xs">
+                {t("sharePage.insights")}
+              </TabsTrigger>
+            ) : null}
           </TabsList>
           <TabsContent
             value="agent"
@@ -1187,19 +1197,17 @@ export default function ShareRoute() {
               presentation="share"
             />
           </TabsContent>
-          <TabsContent
-            value="insights"
-            className="mt-3 min-h-0 flex-1 data-[state=inactive]:hidden"
-          >
-            {viewerCanEdit ? (
+          {viewerCanEdit ? (
+            <TabsContent
+              value="insights"
+              className="mt-3 min-h-0 flex-1 data-[state=inactive]:hidden"
+            >
               <InsightsPanel
                 recordingId={recording.id}
                 durationMs={recording.durationMs}
               />
-            ) : (
-              <PublicInsightsState />
-            )}
-          </TabsContent>
+            </TabsContent>
+          ) : null}
         </Tabs>
       </aside>
 
@@ -1316,20 +1324,6 @@ function PublicAgentEmptyState({
   );
 }
 
-function PublicInsightsState() {
-  const t = useT();
-  return (
-    <div className="flex h-full flex-col items-center justify-center px-8 py-12 text-center">
-      <p className="text-sm font-medium text-foreground">
-        {t("sharePage.ownerInsights")}
-      </p>
-      <p className="mt-2 max-w-[240px] text-sm leading-5 text-muted-foreground">
-        {t("sharePage.ownerInsightsDescription")}
-      </p>
-    </div>
-  );
-}
-
 function EndState({
   title,
   message,

package/corpus/templates/clips/changelog/2026-08-06-clips-recovers-from-stalled-recording-setup-without-requirin.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-08-06
+---
+
+Clips recovers from stalled recording setup without requiring a restart

package/corpus/templates/clips/changelog/2026-08-06-clips-starts-faster-after-a-deploy-or-idle-period-startup-no.md

@@ -0,0 +1,6 @@
+---
+type: improved
+date: 2026-08-06
+---
+
+Clips starts faster after a deploy or idle period — startup no longer re-scans recordings or re-probes columns one at a time on every cold start.

package/corpus/templates/clips/changelog/2026-08-06-deleting-a-recording-returns-cleanly-to-the-library-and-fit-.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-08-06
+---
+
+Deleting a recording returns cleanly to the library, and Fit to width fits the whole timeline.

package/corpus/templates/clips/changelog/2026-08-06-hide-clip-insights-from-viewers-without-analytics-access.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-08-06
+---
+
+Hide clip insights from viewers without analytics access

package/corpus/templates/clips/desktop/src/app.tsx

@@ -980,6 +980,7 @@ export function App() {
   );
   const [rewindHomeOpen, setRewindHomeOpen] = useState(false);
   const [recorder, setRecorder] = useState<RecorderHandle | null>(null);
+  const recordingStartAbortRef = useRef<AbortController | null>(null);
   const [recError, setRecError] = useState<string | null>(null);
   const [cameraError, setCameraError] = useState<string | null>(null);
   const [shortcutRegistrationError, setShortcutRegistrationError] = useState<
@@ -2267,14 +2268,14 @@ export function App() {
       try {
         await invoke("show_toolbar");
         if (cancelled) return;
-        // Seed disabled — previous recordings may have latched it on in
-        // the toolbar's React state (the window is destroyed on
-        // `hide_overlays`, so this is mostly defensive, but free).
-        emit("clips:toolbar-enabled", false).catch(() => {});
       } catch (err) {
         console.error("[clips-popover] show_toolbar failed:", err);
       }
     })();
+    // Seed disabled before the asynchronous window creation. A late seed can
+    // otherwise arrive after the recorder's enabled event and strand the
+    // toolbar at 0:00.
+    emit("clips:toolbar-enabled", false).catch(() => {});
     return () => {
       cancelled = true;
       // In screen-only mode the bubble effect never runs, so its
@@ -2889,6 +2890,9 @@ export function App() {
 
     let handle: RecorderHandle | null = null;
     let startError: unknown = null;
+    const startController = new AbortController();
+    recordingStartAbortRef.current = startController;
+    let parkPopoverTimer: number | null = null;
     try {
       // Per Steve: "when we hit Start Recording the popover should disappear
       // BEFORE the screen picker shows up — otherwise you might accidentally
@@ -2940,11 +2944,11 @@ export function App() {
         preAcquiredCameraStream,
         preAcquiredDisplayStream: options?.resumeCapture?.displayStream ?? null,
         preAcquiredAudioStream: options?.resumeCapture?.audioStream ?? null,
+        signal: startController.signal,
       });
-      // macOS: park the popover to its 2×2 pinhole IMMEDIATELY so it
-      // doesn't appear in the screen picker window list. The native
-      // Rust recorder used for full-screen doesn't need getDisplayMedia
-      // at all, so parking is always safe on macOS.
+      // macOS: give WebKit a short window to dispatch getDisplayMedia before
+      // parking the popover. Parking synchronously can leave the picker
+      // request pending in a long-lived tray webview.
       //
       // Windows: do NOT park before getDisplayMedia resolves. On Windows,
       // the WebView2 screen picker UI renders within the popover webview —
@@ -2953,24 +2957,25 @@ export function App() {
       // popover itself (line ~2165) AFTER the streams are acquired, which
       // is the correct time on Windows.
       if (isMacPlatform()) {
-        invoke("park_popover_offscreen").catch(() => {});
-        emit("clips:popover-visible", false).catch(() => {});
+        parkPopoverTimer = window.setTimeout(() => {
+          if (!startController.signal.aborted && recordingFlowGateRef.current) {
+            invoke("park_popover_offscreen").catch(() => {});
+            emit("clips:popover-visible", false).catch(() => {});
+          }
+        }, 250);
       }
-
-      // No watchdog — the macOS screen picker can stay open indefinitely
-      // (a user deciding which window to capture may take 20, 60, 180
-      // seconds). A false-positive timeout here fires recovery mid-setup,
-      // which flips `recordingFlowActive` back to false → the bubble
-      // session effect's cleanup runs and stops the popover-owned camera
-      // stream → the recorder ends up with a dead track when the screen
-      // picker finally resolves. If the user actually wants to abort,
-      // canceling the picker throws NotAllowedError and we recover through
-      // the normal error path.
       handle = await recordingPromise;
       console.log("[clips-popover] recorder handle received");
     } catch (err) {
       startError = err;
     } finally {
+      if (parkPopoverTimer !== null) {
+        window.clearTimeout(parkPopoverTimer);
+        parkPopoverTimer = null;
+      }
+      if (recordingStartAbortRef.current === startController) {
+        recordingStartAbortRef.current = null;
+      }
       // If the recorder handle was NEVER set, ALWAYS run recovery here —
       // even if downstream code throws before reaching the failure
       // branch. This makes the tray-dead symptom impossible: regardless
@@ -3064,6 +3069,29 @@ export function App() {
   // include a function that is recreated every render.
   handleStartRecordingRef.current = handleStartRecording;
 
+  // The toolbar exists before a recorder handle does. Keep its Cancel action
+  // useful during capture setup, when the normal recorder event listener has
+  // not been installed yet.
+  useEffect(() => {
+    let cancelled = false;
+    let unlisten: (() => void) | null = null;
+    listen("clips:recorder-cancel", () => {
+      recordingStartAbortRef.current?.abort();
+    })
+      .then((nextUnlisten) => {
+        if (cancelled) {
+          nextUnlisten();
+          return;
+        }
+        unlisten = nextUnlisten;
+      })
+      .catch(() => {});
+    return () => {
+      cancelled = true;
+      unlisten?.();
+    };
+  }, []);
+
   // Gates every start-recording gesture (button, global shortcut, permission
   // retry) on the mic toggle. When the mic is off we hold the actual
   // getDisplayMedia/getUserMedia call until the user confirms in