refactor: extract named backup facade

Author: ndycode

lib/storage.ts

@@ -56,6 +56,7 @@ import {
 	migrateV1ToV3,
 	type RateLimitStateV3,
 } from "./storage/migrations.js";
+import { exportNamedBackupEntry } from "./storage/named-backup-entry.js";
 import {
 	collectNamedBackups,
 	type NamedBackupSummary,
@@ -825,14 +826,13 @@ export async function exportNamedBackup(
 	name: string,
 	options?: { force?: boolean },
 ): Promise<string> {
-	return exportNamedBackupFile(
+	return exportNamedBackupEntry({
 		name,
-		{
-			getStoragePath,
-			exportAccounts,
-		},
 		options,
-	);
+		exportNamedBackupFile,
+		getStoragePath,
+		exportAccounts,
+	});
 }
 
 export function getFlaggedAccountsPath(): string {

lib/storage/named-backup-entry.ts

@@ -0,0 +1,31 @@
+export async function exportNamedBackupEntry(params: {
+	name: string;
+	options?: { force?: boolean };
+	exportNamedBackupFile: (
+		name: string,
+		deps: {
+			getStoragePath: () => string;
+			exportAccounts: (
+				filePath: string,
+				force?: boolean,
+				beforeCommit?: (resolvedPath: string) => Promise<void> | void,
+			) => Promise<void>;
+		},
+		options?: { force?: boolean },
+	) => Promise<string>;
+	getStoragePath: () => string;
+	exportAccounts: (
+		filePath: string,
+		force?: boolean,
+		beforeCommit?: (resolvedPath: string) => Promise<void> | void,
+	) => Promise<void>;
+}): Promise<string> {
+	return params.exportNamedBackupFile(
+		params.name,
+		{
+			getStoragePath: params.getStoragePath,
+			exportAccounts: params.exportAccounts,
+		},
+		params.options,
+	);
+}

test/named-backup-entry.test.ts

@@ -0,0 +1,27 @@
+import { describe, expect, it, vi } from "vitest";
+import { exportNamedBackupEntry } from "../lib/storage/named-backup-entry.js";
+
+describe("named backup entry", () => {
+	it("passes name, deps, and options through to the named backup exporter", async () => {
+		const exportNamedBackupFile = vi.fn(async () => "/tmp/backup.json");
+		const exportAccounts = vi.fn(async () => undefined);
+
+		const result = await exportNamedBackupEntry({
+			name: "manual-backup",
+			options: { force: true },
+			exportNamedBackupFile,
+			getStoragePath: () => "/tmp/accounts.json",
+			exportAccounts,
+		});
+
+		expect(exportNamedBackupFile).toHaveBeenCalledWith(
+			"manual-backup",
+			{
+				getStoragePath: expect.any(Function),
+				exportAccounts,
+			},
+			{ force: true },
+		);
+		expect(result).toBe("/tmp/backup.json");
+	});
+});