@narumitw/pi-sync 0.53.1 → 0.54.0

package/src/ui/manager-state.ts

@@ -25,11 +25,12 @@ import { countValidSyncSetups } from "./sync-setups-ui.js";
 import { safeTerminalText } from "./terminal-text.js";
 
 export const MAIN_MENU_ACTIONS = [
-  "Sync now (recommended)",
-  "Switch sync setup",
-  "Status & changes",
+  "Sync now",
+  "Pull from remote…",
+  "Push to remote…",
   "Settings",
-  "More…",
+  "History",
+  "Diagnostics",
 ] as const;
 
 export interface ManagerDescription {
@@ -125,7 +126,9 @@ export async function describeManagerState(
           ? "Never synced"
           : "Unavailable";
     const canSwitch = (await countValidSyncSetups(configuredTargets, signal)) > 1;
-    const mainActions = MAIN_MENU_ACTIONS.filter((action) => action !== "Switch sync setup" || canSwitch);
+    const mainActions: string[] = [...MAIN_MENU_ACTIONS];
+    if (canSwitch) mainActions.push("Switch sync setup");
+    if (syncState?.unresolved?.length) mainActions.unshift("Review unresolved conflicts");
     const ordinaryTitle = [
       "Manage sync",
       "",
@@ -137,7 +140,7 @@ export async function describeManagerState(
       `Last applied: ${lastAppliedSnapshot}`,
       ...(syncState?.unresolved?.length
         ? [
-            `Unresolved: ${syncState.unresolved.length} dependency groups · review History & recovery`,
+            `Unresolved: ${syncState.unresolved.length} dependency groups · choose Review unresolved conflicts`,
             `Last observed: ${safeTerminalText(syncState.lastObservedSnapshot ?? "unknown")}`,
           ]
         : []),
@@ -189,7 +192,7 @@ export async function describeManagerState(
         "",
         "What do you want to do?",
       ].join("\n"),
-      actions: ["Sync setups…", "Storage connections…", "History & recovery…", "Help"],
+      actions: ["Sync setups…", "Storage connections…", "History", "Help"],
     };
   }
 }
@@ -201,13 +204,19 @@ function operationActions(
   mainActions: string[],
 ) {
   if (operationCanRecover(operation)) {
-    return ["Restore sync access… (recommended)", "Status & changes", "History & recovery…", "Help"];
+    return ["Restore sync access… (recommended)", "History", "Help"];
   }
   if (operation.kind !== "free") {
-    return ["Refresh operation status", "Status & changes", "History & recovery…", "Help"];
+    return ["Refresh operation status", "History", "Help"];
   }
   return noSyncedContent
-    ? ["Settings", ...(canSwitch ? ["Switch sync setup"] : []), "Status & changes", "More…"]
+    ? [
+        ...(mainActions.includes("Review unresolved conflicts") ? ["Review unresolved conflicts"] : []),
+        "Settings",
+        ...(canSwitch ? ["Switch sync setup"] : []),
+        "History",
+        "Diagnostics",
+      ]
     : mainActions;
 }
 
@@ -217,16 +226,24 @@ function operationStatusLines(operation: OperationAvailability): string[] {
       return [];
     case "live": {
       const command = truncateToWidth(safeTerminalText(operation.lock.command), 16, "…");
-      return [`Running: ${command} (pid ${operation.lock.pid}). Wait, then refresh; Settings and More return.`];
+      return [
+        `Running: ${command} (pid ${operation.lock.pid}). Wait, then refresh; Transfer actions and Settings return.`,
+      ];
     }
     case "busy":
-      return ["Pi-sync may be starting or finishing. Wait, then refresh; Settings and More remain unavailable."];
+      return [
+        "Pi-sync may be starting or finishing. Wait, then refresh; Transfer actions and Settings remain unavailable.",
+      ];
     case "recoverable-stale":
-      return ["Sync paused: old lock remains. Close other Pi sessions then restore; Settings and More return."];
+      return [
+        "Sync paused: old lock remains. Close other Pi sessions then restore; Transfer actions and Settings return.",
+      ];
     case "recoverable-unreadable":
-      return ["Sync paused: owner unknown. Close other Pi sessions then restore; Settings and More return."];
+      return [
+        "Sync paused: owner unknown. Close other Pi sessions then restore; Transfer actions and Settings return.",
+      ];
     case "inspection-error":
-      return ["Lock check failed. Fix path access, then refresh; Settings and More remain unavailable."];
+      return ["Lock check failed. Fix path access, then refresh; Transfer actions and Settings remain unavailable."];
   }
 }
 

package/src/ui/manager-ui.ts

@@ -1,7 +1,7 @@
 import type { ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
 import { type ActionMenuItem, defineMenu, runMenu } from "@narumitw/pi-tui-kit";
 import { loadConfig } from "../settings/config.js";
-import { operationBlocksChanges, operationCanRecover } from "../state/operation-availability.js";
+import { operationBlocksChanges } from "../state/operation-availability.js";
 import { type RunRoute, runCancellableOperation } from "./cancellable-operation.js";
 import {
   attentionMainMenuItems,
@@ -28,12 +28,11 @@ export async function showSyncManager(
     await runRoute("help");
     return;
   }
-  type Screen = "main" | "more" | "recovery";
+  type Screen = "main";
   type Action =
     | "review-attention"
     | "sync"
     | "switch"
-    | "diff"
     | "settings"
     | "pull"
     | "push"
@@ -42,12 +41,10 @@ export async function showSyncManager(
     | "history"
     | "conflicts"
     | "doctor"
-    | "unlock"
     | "recover"
     | "refresh"
     | "help"
-    | "init"
-    | "back";
+    | "init";
   interface State {
     manager: Awaited<ReturnType<typeof describeManagerState>>;
   }
@@ -63,54 +60,15 @@ export async function showSyncManager(
         return {
           kind: "actions",
           title: "Manage sync",
-          lines: state.manager.title.split("\n").slice(1),
+          // Frequent transfers stay visible; status is advisory and Help remains a direct route.
+          lines: [
+            ...state.manager.title.split("\n").slice(1),
+            "Help: /sync help · Fresh status: /sync status · Differences: /sync diff",
+          ],
           items: operationFirst ? [...managerItems, ...attentionItems] : [...attentionItems, ...managerItems],
           hint: "close",
         };
       },
-      more: () => ({
-        kind: "actions",
-        title: "More options",
-        items: [
-          { id: "pull", label: "Pull from remote…", action: "pull" },
-          { id: "push", label: "Push to remote…", action: "push" },
-          { id: "setups", label: "Sync setups…", action: "setups" },
-          {
-            id: "connections",
-            label: "Storage connections…",
-            action: "connections",
-          },
-          {
-            id: "doctor",
-            label: "Check setup",
-            description: "Check current setup access. WebDAV also probes writes and repairs history.",
-            action: "doctor",
-          },
-          { id: "recovery", label: "History & recovery…", to: "recovery" },
-          { id: "help", label: "Help", action: "help" },
-          { id: "back", label: "Back", action: "back" },
-        ],
-        hint: "back",
-      }),
-      recovery: ({ state }) => ({
-        kind: "actions",
-        title: "History & recovery",
-        items: [
-          { id: "history", label: "Browse history", action: "history" },
-          { id: "conflicts", label: "Review unresolved conflicts", action: "conflicts" },
-          {
-            id: "doctor",
-            label: "Check setup",
-            description: "WebDAV also probes writes and repairs history.",
-            action: "doctor",
-          },
-          ...(state.manager.operation && operationCanRecover(state.manager.operation)
-            ? [{ id: "unlock", label: "Recover stale operation", action: "unlock" as const }]
-            : []),
-          { id: "back", label: "Back", action: "back" },
-        ],
-        hint: "back",
-      }),
     },
     actions: {
       "review-attention": async () => {
@@ -170,15 +128,9 @@ export async function showSyncManager(
         const result = await showSetupSwitcher(ctx, runRoute, undefined, sessionSignal);
         return result === "pull-attempted" || result === "closed" ? { kind: "close" } : { kind: "stay" };
       },
-      diff: async () => {
-        const result = await runCancellableOperation(ctx, "Checking current sync setup…", "diff", runRoute, {
-          signal: sessionSignal,
-        });
-        return result.kind === "closed" ? { kind: "close" } : { kind: "stay" };
-      },
       settings: async () => {
-        await showSyncSettings(ctx, runRoute, sessionSignal);
-        return { kind: "stay" };
+        const result = await showSyncSettings(ctx, runRoute, sessionSignal);
+        return { kind: result === "exit" ? "close" : "stay" };
       },
       pull: async () => {
         const result = await runCancellableOperation(ctx, "Checking remote changes…", "pull", runRoute, {
@@ -207,8 +159,13 @@ export async function showSyncManager(
         return { kind: "stay" };
       },
       history: async () => {
-        await runRoute("history");
-        return { kind: "stay" };
+        const result = await runCancellableOperation(ctx, "Loading sync history…", "history", runRoute, {
+          signal: sessionSignal,
+          commitAware: true,
+          cancelAcrossDialogs: true,
+          cancelledMessage: "History review cancelled; no files were changed.",
+        });
+        return { kind: result.kind === "closed" ? "close" : "stay" };
       },
       doctor: async () => {
         const result = await runCancellableOperation(ctx, "Checking setup access…", "doctor", runRoute, {
@@ -216,11 +173,6 @@ export async function showSyncManager(
         });
         return { kind: result.kind === "closed" ? "close" : "stay" };
       },
-      unlock: async ({ state, signal: actionSignal }) => {
-        const result = await recoverSyncAccess(ctx, state.manager, runRoute, sessionSignal, actionSignal);
-        if (result === "close") return { kind: "close" };
-        return result === "restored" ? { kind: "to", screen: "main" } : { kind: "stay" };
-      },
       recover: async ({ state, signal: actionSignal }) => {
         const result = await recoverSyncAccess(ctx, state.manager, runRoute, sessionSignal, actionSignal);
         return { kind: result === "close" ? "close" : "stay" };
@@ -242,7 +194,6 @@ export async function showSyncManager(
         const disposition = await dispatchManagerResult(ctx, result, "sync", runRoute, sessionSignal);
         return disposition.kind === "close" ? { kind: "close" } : { kind: "stay" };
       },
-      back: async () => ({ kind: "back" }),
     },
   });
   await runMenu(ctx, menu, {
@@ -273,26 +224,47 @@ export async function showSyncManager(
 function syncMainMenuItem(
   label: string,
 ): ActionMenuItem<
-  "main" | "more" | "recovery",
-  "sync" | "switch" | "diff" | "settings" | "setups" | "connections" | "recover" | "refresh" | "help" | "init"
+  "main",
+  | "sync"
+  | "pull"
+  | "push"
+  | "switch"
+  | "settings"
+  | "setups"
+  | "connections"
+  | "history"
+  | "conflicts"
+  | "doctor"
+  | "recover"
+  | "refresh"
+  | "help"
+  | "init"
 > {
-  if (label === "More…") return { id: "more", label, to: "more" };
-  if (label === "History & recovery…") return { id: "recovery", label, to: "recovery" };
-  const actions = new Map<
-    string,
-    "sync" | "switch" | "diff" | "settings" | "setups" | "connections" | "recover" | "refresh" | "help" | "init"
-  >([
-    ["Sync now (recommended)", "sync"],
-    ["Switch sync setup", "switch"],
-    ["Status & changes", "diff"],
-    ["Settings", "settings"],
-    ["Restore sync access… (recommended)", "recover"],
-    ["Refresh operation status", "refresh"],
-    ["Sync setups…", "setups"],
-    ["Storage connections…", "connections"],
-    ["Help", "help"],
-    ["Set up sync", "init"],
-    ["Use existing settings", "init"],
-  ]);
-  return { id: actions.get(label) ?? "help", label, action: actions.get(label) ?? "help" };
+  const actions = {
+    "Sync now": "sync",
+    "Pull from remote…": "pull",
+    "Push to remote…": "push",
+    "Switch sync setup": "switch",
+    Settings: "settings",
+    History: "history",
+    Diagnostics: "doctor",
+    "Review unresolved conflicts": "conflicts",
+    "Restore sync access… (recommended)": "recover",
+    "Refresh operation status": "refresh",
+    "Sync setups…": "setups",
+    "Storage connections…": "connections",
+    Help: "help",
+    "Set up sync": "init",
+    "Use existing settings": "init",
+  } as const;
+  const action = actions[label as keyof typeof actions];
+  if (!action) throw new Error(`Unknown sync manager action: ${label}`);
+  return {
+    id: action,
+    label,
+    action,
+    ...(action === "doctor"
+      ? { description: "Check setup access; WebDAV also probes writes and repairs history." }
+      : {}),
+  };
 }

package/src/ui/operation-dialogs.ts

@@ -0,0 +1,29 @@
+import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
+
+/** Track History's Pi-owned selector/confirmation lifetime without inspecting renderer internals. */
+export function trackOperationDialogs<Context extends ExtensionContext>(
+  ctx: Context,
+  onDialog?: (active: boolean) => void,
+): Context {
+  if (!onDialog) return ctx;
+  const runDialog = async <Value>(task: () => Promise<Value>): Promise<Value> => {
+    onDialog(true);
+    try {
+      return await task();
+    } finally {
+      onDialog(false);
+    }
+  };
+  const ui: ExtensionContext["ui"] = {
+    ...ctx.ui,
+    select: (...args) => runDialog(() => ctx.ui.select(...args)),
+    confirm: (...args) => runDialog(() => ctx.ui.confirm(...args)),
+  };
+  // Forward live context properties: copying the context would hide session
+  // replacement from mutation-owner revalidation after a dialog settles.
+  return new Proxy(ctx, {
+    get(target, key, receiver) {
+      return key === "ui" ? ui : Reflect.get(target, key, receiver);
+    },
+  });
+}

package/src/ui/settings-ui.ts

@@ -14,7 +14,9 @@ import {
 import { captureMutationOwner } from "../sync/sync-local.js";
 import type { RunRoute } from "./cancellable-operation.js";
 import { dispatchManagerResult } from "./manager-result-dispatcher.js";
+import { showEditSyncSetupStorage, showSyncSetupManager } from "./setup/setup-actions.js";
 import { AUTOMATIC_SYNC_DESCRIPTION } from "./setup/setup-prompts.js";
+import { showStorageConnections } from "./storage-connections-ui.js";
 import { configureSyncStatus } from "./sync-status.js";
 import { safeTerminalText } from "./terminal-text.js";
 
@@ -24,13 +26,46 @@ export async function showSyncSettings(
   ctx: ExtensionCommandContext,
   runRoute: SyncSettingsRoute,
   signal?: AbortSignal,
-) {
+): Promise<"back" | "exit"> {
+  let disposition: "back" | "exit" = "back";
+  const exitSettings = () => {
+    disposition = "exit";
+    return { kind: "close" as const };
+  };
   if (ctx.mode !== "tui") {
     ctx.ui.notify(`Edit pi-sync settings manually: ${safeTerminalText(localConfigPath())}`, "info");
-    return;
+    if (ctx.mode === "rpc" && ctx.hasUI) {
+      // Preserve RPC catalog management; preference/credential editing stays TUI-only.
+      const management = defineMenu<undefined, "management", "setups" | "connections", ExtensionCommandContext>({
+        start: "management",
+        screens: {
+          management: () => ({
+            kind: "actions",
+            title: "Sync settings management",
+            items: [
+              { id: "setups", label: "Manage sync setups", action: "setups" },
+              { id: "connections", label: "Manage storage connections", action: "connections" },
+            ],
+            hint: "back",
+          }),
+        },
+        actions: {
+          setups: async ({ signal: actionSignal }) => {
+            const result = await showSyncSetupManager(ctx, runRoute, actionSignal);
+            return result === "exit" ? exitSettings() : { kind: "stay" };
+          },
+          connections: async ({ signal: actionSignal }) => {
+            await showStorageConnections(ctx, actionSignal);
+            return { kind: "stay" };
+          },
+        },
+      });
+      await runMenu(ctx, management, { getState: () => undefined, signal, isCurrent: () => !signal?.aborted });
+    }
+    return disposition;
   }
   const initial = await loadConfig();
-  if (signal?.aborted) return;
+  if (signal?.aborted) return "back";
   const setupName = initial.setupName;
   type Action =
     | "automatic"
@@ -42,7 +77,10 @@ export async function showSyncSettings(
     | "show-status"
     | "on-switch"
     | "include"
-    | "remote-include";
+    | "remote-include"
+    | "storage-location"
+    | "setups"
+    | "connections";
   const menu = defineMenu<Awaited<ReturnType<typeof loadConfig>>, "settings", Action, ExtensionCommandContext>({
     start: "settings",
     screens: {
@@ -52,39 +90,9 @@ export async function showSyncSettings(
         lines: [
           `Sync setup: ${safeTerminalText(state.setupName)} · Storage connection: ${safeTerminalText(state.connectionName)}`,
         ],
+        // Flat, searchable task ordering: Content, Destination, Automation, Merge, Other.
+        // Kit has no heading rows; avoid focusable fake headings or a new component.
         items: [
-          {
-            id: "automatic",
-            label: "Automatic sync",
-            description: AUTOMATIC_SYNC_DESCRIPTION,
-            currentValue: state.automatic ? "On" : "Off",
-            values: ["On", "Off"],
-            action: "automatic",
-          },
-          {
-            id: "skipSecretScan",
-            label: "Skip secret scan (all setups)",
-            description: "All setups: allow pushes without checking managed local files for possible secrets.",
-            currentValue: state.skipSecretScan ? "On" : "Off",
-            values: ["On", "Off"],
-            action: "skip-secret-scan",
-          },
-          {
-            id: "showStatus",
-            label: "Show status (all setups)",
-            description: "All setups: show sync progress and attention in Pi status.",
-            currentValue: state.showStatus ? "On" : "Off",
-            values: ["On", "Off"],
-            action: "show-status",
-          },
-          {
-            id: "onSwitch",
-            label: "After switching setup (all setups)",
-            description: "All setups: ask before a pull review, open it automatically, or switch without pulling.",
-            currentValue: setupSwitchActionLabel(state.onSwitch),
-            values: SETUP_SWITCH_ACTION_OPTIONS.map(({ label }) => label),
-            action: "on-switch",
-          },
           {
             id: "include",
             label: "Included content",
@@ -99,6 +107,49 @@ export async function showSyncSettings(
             currentValue: "Review",
             action: "remote-include",
           },
+          {
+            id: "localFields",
+            label: "Machine-local settings fields",
+            description:
+              "Root field names omitted from future portable snapshots. Policy changes require directional migration; old history remains.",
+            currentValue: `${state.localFields?.length ?? 0} fields · Edit`,
+            action: "local-fields",
+          },
+          {
+            id: "storageLocation",
+            label: "Storage location",
+            description: "Destination: edit this setup's bucket, branch, or path; remote history is not moved.",
+            currentValue:
+              state.backend.type === "s3"
+                ? `${safeTerminalText(state.backend.destination.bucket)} · ${safeTerminalText(state.backend.destination.prefix)}`
+                : state.backend.type === "git"
+                  ? `${safeTerminalText(state.backend.destination.branch)} · ${safeTerminalText(state.backend.destination.directory)}`
+                  : safeTerminalText(state.backend.destination.path),
+            action: "storage-location",
+          },
+          {
+            id: "setups",
+            label: "Manage sync setups",
+            description: "Destination: add, switch, edit, or remove sync setups.",
+            currentValue: "Open",
+            action: "setups",
+          },
+          {
+            id: "connections",
+            label: "Manage storage connections",
+            description:
+              "Destination: manage shared server addresses and credentials; edits may affect multiple setups.",
+            currentValue: "Open",
+            action: "connections",
+          },
+          {
+            id: "automatic",
+            label: "Automatic sync",
+            description: AUTOMATIC_SYNC_DESCRIPTION,
+            currentValue: state.automatic ? "On" : "Off",
+            values: ["On", "Off"],
+            action: "automatic",
+          },
           {
             id: "automaticTransfer",
             label: "Automatic transfer at startup",
@@ -108,6 +159,14 @@ export async function showSyncSettings(
             values: ["On", "Off"],
             action: "automatic-transfer",
           },
+          {
+            id: "onSwitch",
+            label: "After switching setup (all setups)",
+            description: "All setups: ask before a pull review, open it automatically, or switch without pulling.",
+            currentValue: setupSwitchActionLabel(state.onSwitch),
+            values: SETUP_SWITCH_ACTION_OPTIONS.map(({ label }) => label),
+            action: "on-switch",
+          },
           {
             id: "mergeSettings",
             label: "Settings field merge (experimental)",
@@ -117,14 +176,6 @@ export async function showSyncSettings(
             values: ["On", "Off"],
             action: "merge-settings",
           },
-          {
-            id: "localFields",
-            label: "Machine-local settings fields",
-            description:
-              "Root field names omitted from future portable snapshots. Policy changes require directional migration; old history remains.",
-            currentValue: `${state.localFields?.length ?? 0} fields · Edit`,
-            action: "local-fields",
-          },
           {
             id: "contentPolicy",
             label: "Content / partial sync (experimental)",
@@ -140,10 +191,41 @@ export async function showSyncSettings(
             values: ["Off", "Content only", "Content & partial", "Partial only"],
             action: "content-policy",
           },
+          {
+            id: "skipSecretScan",
+            label: "Skip secret scan (all setups)",
+            description: "All setups: allow pushes without checking managed local files for possible secrets.",
+            currentValue: state.skipSecretScan ? "On" : "Off",
+            values: ["On", "Off"],
+            action: "skip-secret-scan",
+          },
+          {
+            id: "showStatus",
+            label: "Show status (all setups)",
+            description: "All setups: show sync progress and attention in Pi status.",
+            currentValue: state.showStatus ? "On" : "Off",
+            values: ["On", "Off"],
+            action: "show-status",
+          },
         ],
       }),
     },
     actions: {
+      "storage-location": async ({ signal: actionSignal }) => {
+        const editorSignal = signal ? AbortSignal.any([signal, actionSignal]) : actionSignal;
+        await showEditSyncSetupStorage(ctx, setupName, editorSignal);
+        return resumeSettings(editorSignal);
+      },
+      setups: async ({ signal: actionSignal }) => {
+        const editorSignal = signal ? AbortSignal.any([signal, actionSignal]) : actionSignal;
+        const result = await showSyncSetupManager(ctx, runRoute, editorSignal);
+        return result === "exit" ? exitSettings() : resumeSettings(editorSignal);
+      },
+      connections: async ({ signal: actionSignal }) => {
+        const editorSignal = signal ? AbortSignal.any([signal, actionSignal]) : actionSignal;
+        await showStorageConnections(ctx, editorSignal);
+        return resumeSettings(editorSignal);
+      },
       "content-policy": async ({ value, signal: actionSignal }) => {
         const mutationSignal = signal ? AbortSignal.any([signal, actionSignal]) : actionSignal;
         const validate = captureMutationOwner(ctx, mutationSignal);
@@ -356,17 +438,29 @@ export async function showSyncSettings(
         if (reviewSignal.aborted) return { kind: "rejected" };
         if (review.kind === "route-result") {
           const disposition = await dispatchManagerResult(ctx, review.result, review.route, runRoute, reviewSignal);
-          return disposition.kind === "close" ? { kind: "close" } : { kind: "stay" };
+          return disposition.kind === "close" ? exitSettings() : { kind: "stay" };
         }
-        return review.kind === "closed" || review.kind === "stale" ? { kind: "close" } : { kind: "stay" };
+        return review.kind === "closed" || review.kind === "stale" ? exitSettings() : { kind: "stay" };
       },
     },
   });
+  async function resumeSettings(editorSignal: AbortSignal) {
+    if (editorSignal.aborted) return { kind: "close" as const };
+    try {
+      const active = await loadConfig();
+      if (editorSignal.aborted || active.setupName !== setupName) return { kind: "close" as const };
+      // Stay reloads authoritative rows without pushing a duplicate screen or clearing search.
+      return { kind: "stay" as const };
+    } catch {
+      return { kind: "close" as const };
+    }
+  }
   await runMenu(ctx, menu, {
     getState: () => loadConfig(setupName),
     signal,
     isCurrent: () => !signal?.aborted,
   });
+  return disposition;
 }
 
 function notifySaveFailure(ctx: ExtensionCommandContext, error: unknown) {

package/src/ui/setup/setup-actions.ts

@@ -22,7 +22,7 @@ export async function showSyncSetupManager(ctx: ExtensionCommandContext, runRout
         await showAddTarget(ctx, setupSignal);
       },
       edit: async (name, setupSignal) => {
-        await showEditTarget(ctx, name, setupSignal);
+        await showEditSyncSetupStorage(ctx, name, setupSignal);
       },
       makeCurrent: async (name, setupSignal) => {
         const result = await showSetupSwitcher(ctx, runRoute, name, setupSignal);
@@ -84,7 +84,7 @@ async function showAddTarget(ctx: ExtensionCommandContext, signal?: AbortSignal)
   await showAddS3Target(ctx, raw, profile, name, signal);
 }
 
-async function showEditTarget(ctx: ExtensionCommandContext, name: string, signal?: AbortSignal) {
+export async function showEditSyncSetupStorage(ctx: ExtensionCommandContext, name: string, signal?: AbortSignal) {
   const partial = await loadPartialConfig(name);
   if (signal?.aborted) return;
   if (!partial.setupName) {