fix(cli): an interactive restore that ticks nothing restores nothing

cli/src/contexts/framework/application/global/restore-all-use-case.ts

@@ -36,6 +36,16 @@ export class RestoreAllUseCase {
     if (manifest === null) throw new NoManifestError();
 
     const effectiveFiles = interactive ? await this.promptForFiles(projectRoot) : undefined;
+    if (effectiveFiles !== undefined && effectiveFiles.length === 0) {
+      return {
+        totalRestored: 0,
+        totalKept: 0,
+        pluginNamesRestored: [],
+        errors,
+        unrestorable: [],
+        nativeOnlyToolIds: [],
+      };
+    }
     const version = this.resolveVersion(manifest);
     const restoreResult = await this.runConfigRestore(
       projectRoot,
@@ -73,12 +83,12 @@ export class RestoreAllUseCase {
         .filter((d) => d.status === "modified" || d.status === "deleted")
         .map((d) => d.relativePath)
     );
-    if (driftedFiles.length === 0) return [];
+    if (driftedFiles.length === 0) return undefined;
     const selected = await this.prompter.checkbox(
       "Select files to restore:",
       driftedFiles.map((f) => ({ name: f, value: f }))
     );
-    return selected.length === 0 ? [] : selected;
+    return selected;
   }
 
   private async runConfigRestore(

cli/tests/contexts/framework/application/restore-all-use-case.unit.test.ts

@@ -140,6 +140,34 @@ describe("RestoreAllUseCase — the --force flag", () => {
   });
 });
 
+describe("RestoreAllUseCase — an interactive run that ticks nothing", () => {
+  it("restores nothing: an empty selection is a decision, not the absence of one", async () => {
+    const deps = await buildUnitDeps(PROJECT_ROOT);
+    await initAndInstall(deps, PROJECT_ROOT, "claude");
+    const manifest = await deps.manifestRepo.load();
+    const tracked = manifest?.getToolFiles("claude") ?? [];
+    const trackedPath = join(PROJECT_ROOT, tracked[0].relativePath);
+    await deps.fs.writeFile(trackedPath, "EDITED OUTSIDE THE CLI");
+    const prompter = new ScriptedPrompter([ScriptedPrompter.answer.checkbox([])]);
+
+    const result = await makeRestoreAllUseCase(
+      deps,
+      new PluginDistributionReaderAdapter(deps.fs),
+      prompter
+    ).execute(PROJECT_ROOT, false, true);
+
+    expect(deps.fs.getFile(trackedPath)).toBe("EDITED OUTSIDE THE CLI");
+    expect(result).toStrictEqual({
+      totalRestored: 0,
+      totalKept: 0,
+      pluginNamesRestored: [],
+      errors: [],
+      unrestorable: [],
+      nativeOnlyToolIds: [],
+    });
+  });
+});
+
 describe("RestoreAllUseCase — plugin materialization", () => {
   it("restores a corrupted plugin file with exactly one materialization call (translate-mode: claude)", async () => {
     const deps = await buildUnitDeps(PROJECT_ROOT);
@@ -552,7 +580,7 @@ describe("RestoreAllUseCase — the interactive file picker", () => {
     expect(asked.map((o) => o.files)).toStrictEqual([[KEYBINDINGS]]);
   });
 
-  it("forwards an empty selection as no file at all", async () => {
+  it("never delegates when the user ticked nothing: an empty selection is a decision", async () => {
     const deps = await vscodeProject();
     await deps.fs.writeFile(join(PROJECT_ROOT, KEYBINDINGS), "[]");
     const { asked, delegate } = recordingDelegate();
@@ -563,17 +591,17 @@ describe("RestoreAllUseCase — the interactive file picker", () => {
       true
     );
 
-    expect(asked.map((o) => o.files)).toStrictEqual([[]]);
+    expect(asked).toStrictEqual([]);
   });
 
-  it("asks nothing and selects nothing when no tracked entry drifted", async () => {
+  it("asks nothing and delegates with no selection when no tracked entry drifted", async () => {
     const deps = await vscodeProject();
     const prompter = new CheckboxRecordingPrompter([KEYBINDINGS]);
     const { asked, delegate } = recordingDelegate();
 
     await restoreAllDelegatingTo(deps, prompter, delegate).execute(PROJECT_ROOT, false, true);
 
     expect(prompter.asks).toStrictEqual([]);
-    expect(asked.map((o) => o.files)).toStrictEqual([[]]);
+    expect(asked.map((o) => o.files)).toStrictEqual([undefined]);
   });
 });