@nextclaw/ui 0.20.1 → 0.20.3

package/src/features/cron/components/cron-config.tsx

@@ -17,12 +17,8 @@ import {
   CronTaskComposer,
   CronTemplateGallery,
 } from "@/features/cron/components/cron-task-discovery";
-import {
-  useCronJobs,
-  useDeleteCronJob,
-  useRunCronJob,
-  useToggleCronJob,
-} from "@/features/cron/hooks/use-cron-jobs";
+import { useCronJobs } from "@/features/cron/hooks/use-cron-jobs";
+import { useCronJobActions } from "@/features/cron/hooks/use-cron-job-actions";
 import { Button } from "@/shared/components/ui/button";
 import { IconActionButton } from "@/shared/components/ui/actions/icon-action-button";
 import { Input } from "@/shared/components/ui/input";
@@ -31,7 +27,6 @@ import type {
   CronListStatus,
   CronListSummaryView,
 } from "@/shared/lib/api";
-import { useConfirmDialog } from "@/shared/hooks/use-confirm-dialog";
 import { t } from "@/shared/lib/i18n";
 import { cn } from "@/shared/lib/utils";
 
@@ -167,10 +162,7 @@ export function CronConfig() {
     query: deferredQuery,
     status,
   });
-  const deleteCronJob = useDeleteCronJob();
-  const toggleCronJob = useToggleCronJob();
-  const runCronJob = useRunCronJob();
-  const { confirm, ConfirmDialog } = useConfirmDialog();
+  const cronActions = useCronJobActions();
   const jobs = cronQuery.data?.jobs ?? [];
   const total = cronQuery.data?.total ?? 0;
   const summary = cronQuery.data?.summary;
@@ -190,46 +182,28 @@ export function CronConfig() {
     composerRef.current?.focus();
   };
 
-  const handleDelete = async (job: CronJobView) => {
-    const confirmed = await confirm({
-      title: `${t("cronDeleteConfirm")}?`,
-      description: job.name ? `${job.name} (${job.id})` : job.id,
-      variant: "destructive",
-      confirmLabel: t("delete"),
+  const handleDelete = (job: CronJobView) => {
+    void cronActions.deleteJob(job, () => {
+      setExpandedJobId(null);
+      setSelectedJobId(null);
+      if (jobs.length === 1 && page > 0) {
+        setPage(page - 1);
+      }
     });
-    if (!confirmed) {
-      return;
-    }
-    setExpandedJobId(null);
-    setSelectedJobId(null);
-    if (jobs.length === 1 && page > 0) {
-      setPage(page - 1);
-    }
-    deleteCronJob.mutate({ id: job.id });
   };
 
   const handleToggle = (job: CronJobView, enabled: boolean) => {
-    const leavesCurrentFilter =
-      (status === "enabled" && !enabled) || (status === "disabled" && enabled);
-    if (leavesCurrentFilter && jobs.length === 1 && page > 0) {
-      setPage(page - 1);
-    }
-    toggleCronJob.mutate({ id: job.id, enabled });
+    void cronActions.toggleJob(job, enabled, () => {
+      const leavesCurrentFilter =
+        (status === "enabled" && !enabled) || (status === "disabled" && enabled);
+      if (leavesCurrentFilter && jobs.length === 1 && page > 0) {
+        setPage(page - 1);
+      }
+    });
   };
 
-  const handleRun = async (job: CronJobView) => {
-    const force = !job.enabled;
-    if (force) {
-      const confirmed = await confirm({
-        title: `${t("cronRunForceConfirm")}?`,
-        description: job.name ? `${job.name} (${job.id})` : job.id,
-        confirmLabel: t("cronRunNow"),
-      });
-      if (!confirmed) {
-        return;
-      }
-    }
-    runCronJob.mutate({ id: job.id, force });
+  const handleRun = (job: CronJobView) => {
+    void cronActions.runJob(job);
   };
 
   const handlePageChange = (nextPage: number) => {
@@ -446,7 +420,7 @@ export function CronConfig() {
         onRun={handleRun}
         onToggle={handleToggle}
       />
-      <ConfirmDialog />
+      <cronActions.ConfirmDialog />
     </PageLayout>
   );
 }

package/src/features/cron/components/cron-job-row.tsx

@@ -166,7 +166,7 @@ export function CronJobRow({
                     size="icon"
                     className="h-8 w-8"
                     onClick={() => onEdit(job)}
-                    aria-label={`${t("edit")} ${jobLabel}`}
+                    aria-label={`${t("cronEditDetails")} ${jobLabel}`}
                   >
                     <Pencil className="h-3.5 w-3.5" />
                   </Button>

package/src/features/cron/hooks/__tests__/use-cron-job-actions.test.tsx

@@ -0,0 +1,95 @@
+import { act, renderHook } from "@testing-library/react";
+import { beforeEach, describe, expect, it, vi } from "vitest";
+
+import type { CronJobView } from "@/shared/lib/api";
+import { setLanguage } from "@/shared/lib/i18n";
+import { useCronJobActions } from "@/features/cron/hooks/use-cron-job-actions";
+
+const mocks = vi.hoisted(() => ({
+  confirm: vi.fn(),
+  deleteJob: vi.fn(),
+  runJob: vi.fn(),
+  toggleJob: vi.fn(),
+}));
+
+vi.mock("@/features/cron/hooks/use-cron-jobs", () => ({
+  useDeleteCronJob: () => ({ mutateAsync: mocks.deleteJob }),
+  useRunCronJob: () => ({ mutateAsync: mocks.runJob }),
+  useToggleCronJob: () => ({ mutateAsync: mocks.toggleJob }),
+}));
+
+vi.mock("@/shared/hooks/use-confirm-dialog", () => ({
+  useConfirmDialog: () => ({
+    confirm: mocks.confirm,
+    ConfirmDialog: () => null,
+  }),
+}));
+
+function createJob(overrides: Partial<CronJobView> = {}): CronJobView {
+  return {
+    id: "paint-daily",
+    name: "每日绘画",
+    enabled: true,
+    schedule: { kind: "cron", expr: "0 9 * * *", tz: "Asia/Shanghai" },
+    payload: {
+      kind: "agent_turn",
+      message: "生成一张今日绘画。",
+      agentId: "main",
+      sessionId: "session:paint",
+    },
+    state: {
+      nextRunAt: "2026-08-27T01:00:00.000Z",
+      lastRunAt: null,
+      lastStatus: null,
+      lastError: null,
+    },
+    createdAt: "2026-08-26T01:00:00.000Z",
+    updatedAt: "2026-08-26T01:00:00.000Z",
+    deleteAfterRun: false,
+    ...overrides,
+  };
+}
+
+describe("useCronJobActions", () => {
+  beforeEach(() => {
+    setLanguage("zh");
+    vi.clearAllMocks();
+    mocks.confirm.mockResolvedValue(true);
+    mocks.deleteJob.mockResolvedValue({ deleted: true });
+    mocks.runJob.mockResolvedValue({ job: createJob(), executed: true });
+    mocks.toggleJob.mockResolvedValue({ job: createJob() });
+  });
+
+  it("requires confirmation and forces a paused task to run without enabling it", async () => {
+    const { result } = renderHook(() => useCronJobActions());
+    const job = createJob({ enabled: false });
+
+    await act(async () => {
+      await result.current.runJob(job);
+    });
+
+    expect(mocks.confirm).toHaveBeenCalledWith({
+      title: "任务已暂停，仍要立即执行?",
+      description: "每日绘画 (paint-daily)",
+      confirmLabel: "立即执行",
+    });
+    expect(mocks.runJob).toHaveBeenCalledWith({ id: "paint-daily", force: true });
+    expect(mocks.toggleJob).not.toHaveBeenCalled();
+  });
+
+  it("uses a destructive confirmation before deleting", async () => {
+    const { result } = renderHook(() => useCronJobActions());
+
+    await act(async () => {
+      await result.current.deleteJob(createJob());
+    });
+
+    expect(mocks.confirm).toHaveBeenCalledWith({
+      title: "确认删除定时任务?",
+      description: "每日绘画 (paint-daily)",
+      confirmLabel: "删除",
+      variant: "destructive",
+    });
+    expect(mocks.deleteJob).toHaveBeenCalledWith({ id: "paint-daily" });
+  });
+});

package/src/features/cron/hooks/use-cron-job-actions.ts

@@ -0,0 +1,109 @@
+import { useState } from "react";
+
+import type { CronJobView } from "@/shared/lib/api";
+import { useConfirmDialog } from "@/shared/hooks/use-confirm-dialog";
+import { t } from "@/shared/lib/i18n";
+import {
+  useDeleteCronJob,
+  useRunCronJob,
+  useToggleCronJob,
+} from "./use-cron-jobs";
+
+type AfterSuccess = () => void;
+
+function describeJob(job: CronJobView): string {
+  return job.name ? `${job.name} (${job.id})` : job.id;
+}
+
+export function useCronJobActions() {
+  const deleteCronJob = useDeleteCronJob();
+  const runCronJob = useRunCronJob();
+  const toggleCronJob = useToggleCronJob();
+  const { confirm, ConfirmDialog } = useConfirmDialog();
+  const [pendingJobIds, setPendingJobIds] = useState<ReadonlySet<string>>(
+    new Set(),
+  );
+
+  const execute = async (
+    job: CronJobView,
+    action: () => Promise<unknown>,
+    afterSuccess?: AfterSuccess,
+  ): Promise<boolean> => {
+    setPendingJobIds((ids) => new Set(ids).add(job.id));
+    try {
+      await action();
+      afterSuccess?.();
+      return true;
+    } catch {
+      return false;
+    } finally {
+      setPendingJobIds((ids) => {
+        const nextIds = new Set(ids);
+        nextIds.delete(job.id);
+        return nextIds;
+      });
+    }
+  };
+
+  const deleteJob = async (
+    job: CronJobView,
+    afterSuccess?: AfterSuccess,
+  ): Promise<boolean> => {
+    const confirmed = await confirm({
+      title: `${t("cronDeleteConfirm")}?`,
+      description: describeJob(job),
+      confirmLabel: t("delete"),
+      variant: "destructive",
+    });
+    if (!confirmed) {
+      return false;
+    }
+    return await execute(
+      job,
+      () => deleteCronJob.mutateAsync({ id: job.id }),
+      afterSuccess,
+    );
+  };
+
+  const runJob = async (
+    job: CronJobView,
+    afterSuccess?: AfterSuccess,
+  ): Promise<boolean> => {
+    const force = !job.enabled;
+    if (force) {
+      const confirmed = await confirm({
+        title: `${t("cronRunForceConfirm")}?`,
+        description: describeJob(job),
+        confirmLabel: t("cronRunNow"),
+      });
+      if (!confirmed) {
+        return false;
+      }
+    }
+    return await execute(
+      job,
+      () => runCronJob.mutateAsync({ id: job.id, force }),
+      afterSuccess,
+    );
+  };
+
+  const toggleJob = async (
+    job: CronJobView,
+    enabled: boolean,
+    afterSuccess?: AfterSuccess,
+  ): Promise<boolean> => {
+    return await execute(
+      job,
+      () => toggleCronJob.mutateAsync({ id: job.id, enabled }),
+      afterSuccess,
+    );
+  };
+
+  return {
+    ConfirmDialog,
+    deleteJob,
+    isPending: (job: CronJobView) => pendingJobIds.has(job.id),
+    runJob,
+    toggleJob,
+  };
+}

package/src/features/cron/index.ts

@@ -1,4 +1,6 @@
 export { CronConfig } from "./components/cron-config";
+export { CronJobDetailDialog } from "./components/cron-job-detail-dialog";
+export { useCronJobActions } from "./hooks/use-cron-job-actions";
 export {
   useCronJobs,
   useDeleteCronJob,

package/src/features/inbox/pages/inbox-page.tsx

@@ -12,6 +12,7 @@ import { Button } from "@/shared/components/ui/button";
 import { useConfirmDialog } from "@/shared/hooks/use-confirm-dialog";
 import { formatDateShort, formatDateTime, t } from "@/shared/lib/i18n";
 import { cn } from "@/shared/lib/utils";
+import { useScrollRestoration } from "@/shared/hooks/use-scroll-restoration";
 
 type InboxFilter = "unread" | "all" | "archived";
 
@@ -64,6 +65,10 @@ function InboxListPane({
   filter: InboxFilter;
   onFilterChange: (filter: InboxFilter) => void;
 }) {
+  const scrollRestoration = useScrollRestoration<HTMLDivElement>({
+    restorationKey: "inbox:list",
+  });
+  const { onScroll, scrollRef } = scrollRestoration;
   const filterItems: Array<{ id: InboxFilter; label: string; count: number }> = [
     {
       id: "unread",
@@ -100,7 +105,11 @@ function InboxListPane({
           </button>
         ))}
       </div>
-      <div className="custom-scrollbar min-h-0 flex-1 overflow-y-auto p-2">
+      <div
+        ref={scrollRef}
+        onScroll={onScroll}
+        className="custom-scrollbar min-h-0 flex-1 overflow-y-auto p-2"
+      >
         {filteredDeliveries.length > 0 ? (
           <ul className="space-y-0.5">
             {filteredDeliveries.map((delivery) => (
@@ -166,6 +175,10 @@ function InboxDetailPane({
   onDelete: () => void;
   onReadToggle: () => void;
 }) {
+  const scrollRestoration = useScrollRestoration<HTMLDivElement>({
+    restorationKey: delivery ? `inbox:delivery:${delivery.id}` : null,
+  });
+  const { onScroll, scrollRef } = scrollRestoration;
   if (!delivery) {
     return <main className="flex min-h-0 flex-col"><InboxEmptyState selection /></main>;
   }
@@ -204,7 +217,10 @@ function InboxDetailPane({
         isHtml
           ? "p-3 sm:p-4"
           : "custom-scrollbar overflow-y-auto px-5 py-5 sm:px-6 sm:py-6",
-      )}>
+      )}
+      ref={scrollRef}
+      onScroll={onScroll}
+      >
         <div className={cn("mx-auto max-w-5xl", isHtml && "h-full")}>
           <InboxDeliveryContent
             className={isHtml ? "h-full" : undefined}

package/src/features/marketplace/components/marketplace-page.tsx

@@ -30,6 +30,7 @@ import { PageLayout } from "@/app/components/layout/page-layout";
 import { useEffect, useMemo, useState } from "react";
 import { useNavigate, useParams } from "react-router-dom";
 import { useInfiniteScrollLoader } from "@/shared/hooks/use-infinite-scroll-loader";
+import { useScrollRestoration } from "@/shared/hooks/use-scroll-restoration";
 import { Tabs, TabsList, TabsTrigger } from "@/shared/components/ui/tabs";
 
 const PAGE_SIZE = 20;
@@ -95,6 +96,11 @@ export function MarketplacePage({
     onLoadMore: () => itemsQuery.fetchNextPage(),
     watchValue: `${typeFilter}:${scope}:${query}:${sceneParam ?? ""}:${sort}:${itemsQuery.data?.loadedItems ?? 0}:${itemsQuery.data?.loadedPages ?? 0}`,
   });
+  const scrollRestoration = useScrollRestoration({
+    restorationKey: `marketplace:${forcedType ?? "all"}`,
+    scrollRef: listContainerRef,
+  });
+  const { onScroll: onScrollPositionSave } = scrollRestoration;
 
   useEffect(() => {
     const container = listContainerRef.current;
@@ -177,6 +183,7 @@ export function MarketplacePage({
       <section className="flex min-h-0 flex-1 flex-col">
         <div
           ref={listContainerRef}
+          onScroll={onScrollPositionSave}
           className="min-h-0 flex-1 overflow-y-auto custom-scrollbar pr-1"
           aria-busy={listModel.showListSkeleton || itemsQuery.isFetchingNextPage}
         >

package/src/features/panel-apps/pages/panel-app-main-page.tsx

@@ -12,6 +12,7 @@ import {
 } from "@/features/panel-apps/utils/panel-app-iframe.utils";
 import { openApps } from "@/features/panel-apps/utils/panel-app-doc-browser.utils";
 import { useDocBrowser } from "@/shared/components/doc-browser";
+import { usePanelAppScrollRestoration } from "@/shared/hooks/use-panel-app-scroll-restoration";
 import { t } from "@/shared/lib/i18n";
 
 export function PanelAppMainPage() {
@@ -25,6 +26,12 @@ export function PanelAppMainPage() {
     [appId, panelApps.data?.entries],
   );
   const runtime = usePanelAppMainRuntime(entry);
+  const restoreScroll = usePanelAppScrollRestoration({
+    currentUrl: entry?.contentPath ?? null,
+    iframeRef,
+    isEnabled: Boolean(entry),
+    restorationKey: entry ? `panel-app:main:${entry.appId}` : null,
+  });
 
   useEffect(() => {
     const handleMessage = (event: MessageEvent) => {
@@ -88,6 +95,7 @@ export function PanelAppMainPage() {
         title={entry.title}
         sandbox={PANEL_APP_IFRAME_SANDBOX}
         className="block h-full w-full border-0 bg-background"
+        onLoad={restoreScroll}
         onPointerOver={(event) => focusPanelAppIframe(event.currentTarget)}
       />
     </div>

package/src/features/settings/pages/__tests__/privacy-settings-page.test.tsx

@@ -6,7 +6,16 @@ const mocks = vi.hoisted(() => ({ mutate: vi.fn() }));
 
 vi.mock('@/shared/hooks/use-config', () => ({
   useConfig: () => ({
-    data: { productAnalytics: { enabled: false, audience: 'external' } },
+    data: { productAnalytics: { schemaVersion: 2, enabled: true, audience: 'external' } },
+    isLoading: false
+  }),
+  useProductAnalyticsStatus: () => ({
+    data: {
+      lastAttemptAt: '2026-08-25T12:00:00.000Z',
+      lastSuccessAt: '2026-08-25T12:00:01.000Z',
+      lastError: null,
+      pendingReceiptCount: 0
+    },
     isLoading: false
   }),
   useUpdateProductAnalytics: () => ({
@@ -18,13 +27,13 @@ vi.mock('@/shared/hooks/use-config', () => ({
 describe('PrivacySettingsPage', () => {
   beforeEach(() => mocks.mutate.mockClear());
 
-  it('requires explicit opt-in and saves the selected test audience', () => {
+  it('shows anonymous reporting enabled by default and saves opt-out and test audience', () => {
     render(<PrivacySettingsPage />);
 
-    const shareSwitch = screen.getByRole('switch', { name: 'Share product activity' });
-    expect(shareSwitch.getAttribute('aria-checked')).toBe('false');
+    const shareSwitch = screen.getByRole('switch', { name: 'Send anonymous usage analytics' });
+    expect(shareSwitch.getAttribute('aria-checked')).toBe('true');
     fireEvent.click(shareSwitch);
-    expect(mocks.mutate).toHaveBeenCalledWith({ data: { enabled: true } });
+    expect(mocks.mutate).toHaveBeenCalledWith({ data: { enabled: false } });
 
     fireEvent.click(screen.getByRole('combobox', { name: 'Analytics audience' }));
     fireEvent.click(screen.getByRole('option', { name: 'QA testing' }));

package/src/features/settings/pages/privacy-settings-page.tsx

@@ -2,14 +2,19 @@ import { SettingsPage } from '@/shared/components/settings/settings-page';
 import { SettingRow, SettingsGroup, SettingsSection } from '@/shared/components/settings/setting-row';
 import { Switch } from '@/shared/components/ui/switch';
 import { Select, SelectContent, SelectItem, SelectTrigger, SelectValue } from '@/shared/components/ui/select';
-import { useConfig, useUpdateProductAnalytics } from '@/shared/hooks/use-config';
+import {
+  useConfig,
+  useProductAnalyticsStatus,
+  useUpdateProductAnalytics
+} from '@/shared/hooks/use-config';
 import type { ProductAnalyticsAudience } from '@/shared/lib/api';
 import { t } from '@/shared/lib/i18n';
 
 export function PrivacySettingsPage(): JSX.Element {
   const configQuery = useConfig();
+  const statusQuery = useProductAnalyticsStatus();
   const updateMutation = useUpdateProductAnalytics();
-  const enabled = configQuery.data?.productAnalytics.enabled ?? false;
+  const enabled = configQuery.data?.productAnalytics.enabled ?? true;
   const audience = configQuery.data?.productAnalytics.audience ?? 'external';
 
   const updateEnabled = (nextEnabled: boolean) => {
@@ -78,6 +83,37 @@ export function PrivacySettingsPage(): JSX.Element {
           />
         </SettingsGroup>
       </SettingsSection>
+
+      <SettingsSection title={t('productAnalyticsStatusSectionTitle')}>
+        <SettingsGroup>
+          <SettingRow
+            layout='stacked'
+            title={t('productAnalyticsStatusTitle')}
+            description={formatProductAnalyticsStatus(statusQuery.data, statusQuery.isLoading)}
+          />
+          {statusQuery.data?.lastError ? (
+            <SettingRow
+              layout='stacked'
+              title={t('productAnalyticsLastErrorTitle')}
+              description={statusQuery.data.lastError}
+            />
+          ) : null}
+        </SettingsGroup>
+      </SettingsSection>
     </SettingsPage>
   );
 }
+
+function formatProductAnalyticsStatus(
+  status: ReturnType<typeof useProductAnalyticsStatus>['data'],
+  isLoading: boolean,
+): string {
+  if (isLoading) return t('productAnalyticsStatusLoading');
+  if (!status?.lastAttemptAt) return t('productAnalyticsStatusNever');
+  const attemptedAt = new Date(status.lastAttemptAt).toLocaleString();
+  if (!status.lastSuccessAt) {
+    return `${t('productAnalyticsLastAttemptLabel')} ${attemptedAt} · ${status.pendingReceiptCount} ${t('productAnalyticsPendingLabel')}`;
+  }
+  const succeededAt = new Date(status.lastSuccessAt).toLocaleString();
+  return `${t('productAnalyticsLastSuccessLabel')} ${succeededAt} · ${status.pendingReceiptCount} ${t('productAnalyticsPendingLabel')}`;
+}

package/src/platforms/desktop/components/desktop-app-shell.tsx

@@ -14,6 +14,7 @@ import type { DocBrowserTabMenuGroupsResolver } from "@/shared/components/doc-br
 import { cn } from "@/shared/lib/utils";
 import { useViewportLayoutStore } from "@/app/stores/viewport-layout.store";
 import { SIDEBAR_RAIL_WIDTH_PX } from "@/app/components/layout/sidebar-rail.styles";
+import { useScrollRestoration } from "@/shared/hooks/use-scroll-restoration";
 
 const DocBrowser = lazy(async () => ({
   default: (await import("@/shared/components/doc-browser/doc-browser"))
@@ -55,6 +56,10 @@ export function DesktopAppShell({
     : isMainRoute
       ? "280px"
       : "240px";
+  const settingsScroll = useScrollRestoration<HTMLElement>({
+    restorationKey: isMainRoute ? null : `settings-page:${pathname}`,
+  });
+  const { onScroll: onSettingsScroll, scrollRef: settingsScrollRef } = settingsScroll;
 
   return (
     <div
@@ -82,7 +87,11 @@ export function DesktopAppShell({
             {isMainRoute ? (
               <div className="flex-1 h-full overflow-hidden">{children}</div>
             ) : (
-              <main className="flex-1 overflow-auto p-8 pb-16 custom-scrollbar">
+              <main
+                ref={settingsScrollRef}
+                onScroll={onSettingsScroll}
+                className="flex-1 overflow-auto p-8 pb-16 custom-scrollbar"
+              >
                 <div className="mx-auto h-full max-w-6xl animate-fade-in">
                   {children}
                 </div>

package/src/shared/components/doc-browser/doc-browser.tsx

@@ -215,7 +215,6 @@ export function DocBrowser({
     currentTab,
     iframeRef,
     isEnabled: supportsScrollRestoration,
-    tabs,
   });
   const { handleUrlSubmit, setUrlInput, urlInput } = useDocBrowserAddressBar({
     activeTabId,