@agent-native/core 0.97.0 → 0.98.0

package/corpus/templates/analytics/actions/rename-analysis.ts

@@ -5,7 +5,7 @@ import {
 } from "@agent-native/core/server";
 import { z } from "zod";
 
-import { upsertAnalysis } from "../server/lib/dashboards-store";
+import { upsertAnalysisWithRetry } from "../server/lib/dashboards-store";
 
 function resolveScope() {
   const orgId = getRequestOrgId() || null;
@@ -25,7 +25,13 @@ export default defineAction({
     if (!name) throw new Error("name is required");
 
     const ctx = resolveScope();
-    const analysis = await upsertAnalysis(args.id, { name }, ctx);
+    // Fenced through the retry helper: the patch below only ever touches
+    // `name`, so a concurrent edit (e.g. save-analysis re-running with fresh
+    // results) racing this rename is never silently overwritten — a lost
+    // race just re-reads the freshest record and reapplies the rename.
+    const analysis = await upsertAnalysisWithRetry(args.id, ctx, () => ({
+      name,
+    }));
     return { id: analysis.id, name: analysis.name };
   },
 });

package/corpus/templates/analytics/actions/rename-dashboard.ts

@@ -10,7 +10,7 @@ import {
 } from "@agent-native/core/server";
 import { z } from "zod";
 
-import { getDashboard, upsertDashboard } from "../server/lib/dashboards-store";
+import { upsertDashboardWithRetry } from "../server/lib/dashboards-store";
 
 function resolveScope() {
   const orgId = getRequestOrgId() || null;
@@ -47,12 +47,13 @@ export default defineAction({
     if (!name) throw new Error("name is required");
 
     const ctx = resolveScope();
-    const dashboard = await getDashboard(args.id, ctx);
-    if (!dashboard) throw new Error("Dashboard not found");
-
-    const config = { ...dashboard.config, name };
-    const updated = await upsertDashboard(args.id, dashboard.kind, config, ctx);
-    await syncToCollab(args.id, config);
+    // Recomputed on every retry attempt from the freshest dashboard config, so
+    // a concurrent panel edit (mutate-dashboard/update-dashboard) racing this
+    // rename is never silently overwritten by a stale config snapshot.
+    const updated = await upsertDashboardWithRetry(args.id, ctx, (existing) => {
+      return { kind: existing.kind, body: { ...existing.config, name } };
+    });
+    await syncToCollab(args.id, updated.config);
     return { id: updated.id, name: updated.title };
   },
 });

package/corpus/templates/analytics/actions/view-screen.ts

@@ -466,34 +466,15 @@ export default defineAction({
       const email = getRequestUserEmail();
       if (email) {
         const orgId = getRequestOrgId() || null;
-        const [keys, catalog] = await Promise.all([
-          listAnalyticsPublicKeys({
-            userEmail: email,
-            orgId,
-          }),
-          listDashboardCatalog({
-            email,
-            orgId,
-          }),
-        ]);
-        const llmTemplate = catalog.find(
-          (template) => template.id === "agent-observability-llm",
-        );
+        const keys = await listAnalyticsPublicKeys({
+          userEmail: email,
+          orgId,
+        });
         screen.firstPartyAnalytics = {
           activeKeys: keys.filter((key: any) => !key.revokedAt).length,
           serverEnv: "AGENT_NATIVE_ANALYTICS_PUBLIC_KEY",
           browserEnv: "VITE_AGENT_NATIVE_ANALYTICS_PUBLIC_KEY",
         };
-        if (llmTemplate) {
-          screen.llmObservabilityDashboard = {
-            templateId: llmTemplate.id,
-            name: llmTemplate.name,
-            installed: llmTemplate.installed,
-            installedDashboardIds: llmTemplate.installedDashboards.map(
-              (dashboard) => dashboard.id,
-            ),
-          };
-        }
       }
     } else if (nav?.view === "settings") {
       screen.page = "settings";

package/corpus/templates/analytics/app/components/ResourceLoadError.tsx

@@ -0,0 +1,42 @@
+import { Button } from "@/components/ui/button";
+
+type ResourceLoadErrorProps = {
+  message: string;
+  retryLabel: string;
+  onRetry: () => void;
+  inline?: boolean;
+};
+
+export function ResourceLoadError({
+  message,
+  retryLabel,
+  onRetry,
+  inline = false,
+}: ResourceLoadErrorProps) {
+  if (inline) {
+    return (
+      <div className="px-2 py-2 text-xs text-destructive" role="status">
+        <span>{message}</span>{" "}
+        <button
+          type="button"
+          className="font-medium underline underline-offset-2 hover:text-destructive/80"
+          onClick={onRetry}
+        >
+          {retryLabel}
+        </button>
+      </div>
+    );
+  }
+
+  return (
+    <div
+      className="flex h-64 flex-col items-center justify-center gap-2 text-sm text-destructive"
+      role="status"
+    >
+      <span>{message}</span>
+      <Button size="sm" variant="outline" onClick={onRetry}>
+        {retryLabel}
+      </Button>
+    </div>
+  );
+}

package/corpus/templates/analytics/app/components/layout/CommandPalette.tsx

@@ -18,6 +18,7 @@ import {
   IconMoon,
   IconHistory,
   IconLanguage,
+  IconRefresh,
 } from "@tabler/icons-react";
 import { useQuery } from "@tanstack/react-query";
 import { useTheme } from "next-themes";
@@ -126,70 +127,54 @@ function CommandLoadingGroup({
 }
 
 async function fetchSavedConfigs(): Promise<SavedConfig[]> {
-  try {
-    const rows = await callAction(
-      "list-explorer-configs",
-      {},
-      { method: "GET" },
-    );
-    return (Array.isArray(rows) ? rows : []) as SavedConfig[];
-  } catch {
-    return [];
-  }
+  const rows = await callAction("list-explorer-configs", {}, { method: "GET" });
+  return (Array.isArray(rows) ? rows : []) as SavedConfig[];
 }
 
 async function fetchExplorerDashboards(): Promise<ExplorerDashboard[]> {
-  try {
-    const result = await callAction(
-      "list-explorer-dashboards",
-      {
-        hidden: "all",
-      },
-      { method: "GET" },
-    );
-    const dashboards =
-      result && typeof result === "object" && "dashboards" in result
-        ? (result as { dashboards: unknown[] }).dashboards
-        : [];
-    return (Array.isArray(dashboards) ? dashboards : [])
-      .filter((d: any) => d && d.name)
-      .map((d: any) => ({
-        id: d.id,
-        name: d.name,
-        hiddenAt: typeof d.hiddenAt === "string" ? d.hiddenAt : null,
-      }));
-  } catch {
-    return [];
-  }
+  const result = await callAction(
+    "list-explorer-dashboards",
+    {
+      hidden: "all",
+    },
+    { method: "GET" },
+  );
+  const dashboards =
+    result && typeof result === "object" && "dashboards" in result
+      ? (result as { dashboards: unknown[] }).dashboards
+      : [];
+  return (Array.isArray(dashboards) ? dashboards : [])
+    .filter((d: any) => d && d.name)
+    .map((d: any) => ({
+      id: d.id,
+      name: d.name,
+      hiddenAt: typeof d.hiddenAt === "string" ? d.hiddenAt : null,
+    }));
 }
 
 async function fetchSqlDashboards(
   t: (key: string) => string,
 ): Promise<{ id: string; name: string; hiddenAt: string | null }[]> {
-  try {
-    const rows = await callAction(
-      "list-sql-dashboards",
-      { hidden: "all" },
-      { method: "GET" },
-    );
-    return (Array.isArray(rows) ? rows : [])
-      .filter((d: any) => d && typeof d.id === "string" && d.id.length > 0)
-      .map((d: any) => ({
-        id: d.id,
-        name:
-          typeof d.name === "string" && d.name.trim().length > 0
-            ? d.name
-            : t("commandPalette.untitledDashboard"),
-        hiddenAt: typeof d.hiddenAt === "string" ? d.hiddenAt : null,
-      }));
-  } catch {
-    return [];
-  }
+  const rows = await callAction(
+    "list-sql-dashboards",
+    { hidden: "all" },
+    { method: "GET" },
+  );
+  return (Array.isArray(rows) ? rows : [])
+    .filter((d: any) => d && typeof d.id === "string" && d.id.length > 0)
+    .map((d: any) => ({
+      id: d.id,
+      name:
+        typeof d.name === "string" && d.name.trim().length > 0
+          ? d.name
+          : t("commandPalette.untitledDashboard"),
+      hiddenAt: typeof d.hiddenAt === "string" ? d.hiddenAt : null,
+    }));
 }
 
 async function fetchExtensions(): Promise<ExtensionSearchItem[]> {
   const res = await fetch(agentNativePath("/_agent-native/extensions"));
-  if (!res.ok) return [];
+  if (!res.ok) throw new Error(`Failed to load extensions (${res.status})`);
   const data = await res.json();
   return (Array.isArray(data) ? data : [])
     .filter((extension: any) => {
@@ -230,7 +215,7 @@ export function CommandPalette() {
   const { resolvedTheme, setTheme } = useTheme();
   const isDark = resolvedTheme === "dark";
 
-  const { data: savedCharts = [], isFetching: savedChartsFetching } = useQuery({
+  const savedChartsQuery = useQuery({
     queryKey: ["explorer-configs-palette"],
     queryFn: fetchSavedConfigs,
     staleTime: 30_000,
@@ -239,10 +224,7 @@ export function CommandPalette() {
 
   const dashboardsSync = useChangeVersions(["dashboards", "action"]);
 
-  const {
-    data: explorerDashboards = [],
-    isFetching: explorerDashboardsFetching,
-  } = useQuery({
+  const explorerDashboardsQuery = useQuery({
     queryKey: ["explorer-dashboards-palette", dashboardsSync],
     queryFn: fetchExplorerDashboards,
     staleTime: 30_000,
@@ -250,18 +232,15 @@ export function CommandPalette() {
     placeholderData: (prev) => prev,
   });
 
-  const { data: sqlDashboards = [], isFetching: sqlDashboardsFetching } =
-    useQuery({
-      queryKey: ["sql-dashboards-palette", dashboardsSync],
-      queryFn: () => fetchSqlDashboards(t),
-      staleTime: 30_000,
-      enabled: open,
-      placeholderData: (prev) => prev,
-    });
-
-  const { data: extensions = [], isFetching: extensionsFetching } = useQuery<
-    ExtensionSearchItem[]
-  >({
+  const sqlDashboardsQuery = useQuery({
+    queryKey: ["sql-dashboards-palette", dashboardsSync],
+    queryFn: () => fetchSqlDashboards(t),
+    staleTime: 30_000,
+    enabled: open,
+    placeholderData: (prev) => prev,
+  });
+
+  const extensionsQuery = useQuery<ExtensionSearchItem[]>({
     queryKey: ["extensions"],
     queryFn: fetchExtensions,
     staleTime: 30_000,
@@ -269,6 +248,28 @@ export function CommandPalette() {
     placeholderData: (prev) => prev,
   });
 
+  const savedCharts = savedChartsQuery.data ?? [];
+  const explorerDashboards = explorerDashboardsQuery.data ?? [];
+  const sqlDashboards = sqlDashboardsQuery.data ?? [];
+  const extensions = extensionsQuery.data ?? [];
+  const savedChartsFetching = savedChartsQuery.isFetching;
+  const explorerDashboardsFetching = explorerDashboardsQuery.isFetching;
+  const sqlDashboardsFetching = sqlDashboardsQuery.isFetching;
+  const extensionsFetching = extensionsQuery.isFetching;
+  const asyncGroupsErrored =
+    savedChartsQuery.isError ||
+    explorerDashboardsQuery.isError ||
+    sqlDashboardsQuery.isError ||
+    extensionsQuery.isError;
+  const retryAsyncGroups = () => {
+    void Promise.all([
+      savedChartsQuery.refetch(),
+      explorerDashboardsQuery.refetch(),
+      sqlDashboardsQuery.refetch(),
+      extensionsQuery.refetch(),
+    ]);
+  };
+
   useEffect(() => {
     const handler = (e: KeyboardEvent) => {
       if ((e.metaKey || e.ctrlKey) && e.key === "k") {
@@ -310,6 +311,7 @@ export function CommandPalette() {
     <>
       <CommandDialog
         open={open}
+        motion="instant"
         onOpenChange={(next) => {
           setOpen(next);
           if (!next) setSearchQuery("");
@@ -321,10 +323,23 @@ export function CommandPalette() {
           onValueChange={setSearchQuery}
         />
         <CommandList>
-          {!asyncGroupsLoading && (
+          {!asyncGroupsLoading && !asyncGroupsErrored && (
             <CommandEmpty>{t("commandPalette.noResults")}</CommandEmpty>
           )}
 
+          {asyncGroupsErrored && (
+            <CommandGroup heading={t("commandPalette.loadFailed")} forceMount>
+              <CommandItem
+                forceMount
+                value="retry failed command palette data"
+                onSelect={retryAsyncGroups}
+              >
+                <IconRefresh className="me-2 h-4 w-4 text-muted-foreground" />
+                {t("sidebar.retry")}
+              </CommandItem>
+            </CommandGroup>
+          )}
+
           {explorerDashboardsFetching && explorerDashboards.length === 0 && (
             <CommandLoadingGroup
               heading={t("commandPalette.groupExplorerDashboards")}