@wuyaos/pi-sync 1.2.0 → 1.2.1

package/extensions/_shared/enhanced-select.ts

@@ -85,6 +85,12 @@ export interface EnhancedSelectOptions {
    * key takes precedence over filter input, mirroring shortcut-key behavior.
    */
   actionKeys?: Array<{ key: string; label: string }>;
+  /**
+   * Initial highlighted index (0-based). Defaults to 0. Clamped to the
+   * valid item range. Use to preserve cursor position across re-renders in
+   * looping menus (e.g. a config toggle that re-enters the select).
+   */
+  initialIndex?: number;
 }
 
 /** Sentinel prefix marking an action-key result from enhancedSelect. */
@@ -143,6 +149,13 @@ class EnhancedSelectComponent {
         this.shortcutMap.set(key, i);
       }
     }
+
+    // Preserve caller-provided cursor position; clamp to valid range so a
+    // stale index from a previous (longer) item list cannot overflow.
+    const initial = this.options.initialIndex ?? 0;
+    this.selectedIdx = this.items.length > 0
+      ? Math.max(0, Math.min(initial, this.items.length - 1))
+      : 0;
   }
 
   // ── Fuzzy filter state ──────────────────────────────────────────────

package/extensions/sync/i18n.ts

@@ -6,9 +6,9 @@ const EN = {
   enabled: "enabled",
   disabled: "disabled",
   keepAll: "keep all",
-  cancel: "❌ Cancel",
-  back: "x Back",
-  save: "s Save",
+  cancel: "x ✕ Cancel",
+  back: "b ← Back",
+  save: "s ✓ Save",
 
   menuTitle: "Pi WebDAV Backup",
   uploadAllBackups: "⬆️  Back Up All",
@@ -101,9 +101,9 @@ const ZH: Record<TranslationKey, string> = {
   enabled: "已启用",
   disabled: "已禁用",
   keepAll: "全部保留",
-  cancel: "❌ 取消",
-  back: "x 返回",
-  save: "s 保存",
+  cancel: "x ✕ 取消",
+  back: "b ← 返回",
+  save: "s ✓ 保存",
 
   menuTitle: "Pi WebDAV 备份",
   uploadAllBackups: "⬆️  全部备份",

package/extensions/sync/menus.ts

@@ -73,8 +73,9 @@ async function selectAction<T extends string>(
   ctx: ExtensionCommandContext,
   title: string,
   items: SelectItem<T>[],
+  initialIndex = 0,
 ): Promise<T | undefined> {
-  const selected = await enhancedSelect(ctx, title, items.map((item) => item.label));
+  const selected = await enhancedSelect(ctx, title, items.map((item) => item.label), { initialIndex });
   return items.find((item) => item.label === selected)?.id;
 }
 
@@ -135,6 +136,7 @@ function sessionSelectionLabel(config: SyncConfig): string {
 }
 
 async function showSessionProjectSelect(ctx: ExtensionCommandContext, config: SyncConfig): Promise<void> {
+  let cursor = 0;
   while (true) {
     const language = config.language;
     const projects = listSessionProjects();
@@ -149,8 +151,9 @@ async function showSessionProjectSelect(ctx: ExtensionCommandContext, config: Sy
       })),
       { id: "back", label: t(language, "back") },
     ];
-    const action = await selectAction(ctx, t(language, "selectSessionProjects", { mode: modeLabel }), items);
+    const action = await selectAction(ctx, t(language, "selectSessionProjects", { mode: modeLabel }), items, cursor);
     if (!action || action === "back") return;
+    cursor = Math.max(0, items.findIndex((item) => item.id === action));
     if (action === "mode") {
       config.sessionProjectMode = config.sessionProjectMode === "whitelist" ? "blacklist" : "whitelist";
     } else if (action === "all") {
@@ -182,6 +185,7 @@ export async function showSetupWizard(ctx: ExtensionCommandContext): Promise<boo
 
 export async function showConfigureSettings(ctx: ExtensionCommandContext): Promise<void> {
   const config = cloneConfig(loadConfig());
+  let cursor = 0;
   while (true) {
     const language = config.language;
     const items: SelectItem<string>[] = [
@@ -199,8 +203,9 @@ export async function showConfigureSettings(ctx: ExtensionCommandContext): Promi
       { id: "save", label: t(language, "save") },
       { id: "back", label: t(language, "back") },
     ];
-    const action = await selectAction(ctx, t(language, "configureTitle"), items);
+    const action = await selectAction(ctx, t(language, "configureTitle"), items, cursor);
     if (!action || action === "back") return;
+    cursor = Math.max(0, items.findIndex((item) => item.id === action));
     if (action === "save") {
       saveConfig(config, ctx);
       ctx.ui.notify(t(language, "configSaved"), "info");
@@ -361,6 +366,19 @@ async function showRestorePackage(
         )) return false;
       }
       const restored = kind === "pi" ? await extractPiAgentZip(local) : await extractAgentSkillsZip(local);
+      if (kind === "pi") {
+        try {
+          // ctx.reload() 只重载扩展等资源，不会刷新内存中的 ModelRegistry。
+          // 禁用网络请求，仅重读刚恢复的 ~/.pi/agent/models.json，使模型选择器立即可见最新目录。
+          const refresh = await ctx.modelRegistry.refresh({ allowNetwork: false });
+          if (refresh.errors.size > 0) {
+            console.warn(`[pi-sync] Model catalog refresh completed with ${refresh.errors.size} error(s)`);
+          }
+        } catch (error) {
+          // 文件已恢复；模型刷新失败不应让整个恢复操作误报失败，重启 Pi 后仍会重新读取 models.json。
+          console.warn(`[pi-sync] Failed to refresh restored model catalog: ${error instanceof Error ? error.message : String(error)}`);
+        }
+      }
       ctx.ui.notify(t(language, "restoreCompleted", { kind: label, contents: restored.join("\n") }), "info");
       if (kind === "pi" && reloadAfter && await ctx.ui.confirm(t(language, "reloadRuntimeTitle"), t(language, "reloadRuntimeBody"))) {
         await ctx.reload();

package/extensions/sync/restore.ts

@@ -15,6 +15,9 @@ async function extractToTemp(archivePath: string, prefix: string): Promise<strin
 }
 
 function assertSafeRestoreParent(root: string, destination: string): void {
+  // destination 就是 root 本身（sessions 顶层合并的目标）是安全的：
+  // 此时 dirname(destination) 在 root 之外，会被下面的检查误判为路径逃逸。
+  if (path.resolve(destination) === path.resolve(root)) return;
   const relative = path.relative(root, path.dirname(destination));
   if (relative.startsWith("..") || path.isAbsolute(relative)) throw new Error(`Restore destination escapes root: ${destination}`);
   let current = root;
@@ -134,13 +137,14 @@ export async function extractAgentSkillsZip(archivePath: string, targetDir = AGE
   }
 }
 
-export async function extractSessionsArchiveZip(archivePath: string): Promise<string[]> {
+export async function extractSessionsArchiveZip(archivePath: string, targetDir = SESSIONS_DIR): Promise<string[]> {
   const tempDir = await extractToTemp(archivePath, "pi_sessions_extract");
   try {
     const source = path.join(tempDir, "sessions");
     if (!fs.existsSync(source)) throw new Error("Archive does not contain sessions/.");
-    fs.mkdirSync(SESSIONS_DIR, { recursive: true });
-    const fileCount = copyExtractedTree(source, SESSIONS_DIR, SESSIONS_DIR);
+    const target = path.resolve(targetDir);
+    fs.mkdirSync(target, { recursive: true });
+    const fileCount = copyExtractedTree(source, target, target);
     return [`Session archive merged: ${fileCount} file(s)`];
   } finally { fs.rmSync(tempDir, { recursive: true, force: true }); }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wuyaos/pi-sync",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "description": "WebDAV archive backup and restore for Pi agent data, shared skills, and per-project sessions",
   "keywords": [
     "pi-package",
@@ -11,7 +11,7 @@
   ],
   "repository": {
     "type": "git",
-    "url": "https://github.com/wuyaos/pi-packages.git",
+    "url": "git+https://github.com/wuyaos/pi-packages.git",
     "directory": "pi-sync"
   },
   "publishConfig": {