refactor: extract named backups list facade

Author: ndycode

lib/storage.ts

@@ -61,6 +61,7 @@ import {
 	collectNamedBackups,
 	type NamedBackupSummary,
 } from "./storage/named-backups.js";
+import { getNamedBackupsEntry } from "./storage/named-backups-entry.js";
 import {
 	findProjectRoot,
 	getConfigDir,
@@ -794,7 +795,9 @@ export function buildNamedBackupPath(name: string): string {
 }
 
 export async function getNamedBackups(): Promise<NamedBackupSummary[]> {
-	return collectNamedBackups(getStoragePath(), {
+	return getNamedBackupsEntry({
+		getStoragePath,
+		collectNamedBackups,
 		loadAccountsFromPath: (path) =>
 			loadAccountsFromPath(path, {
 				normalizeAccountStorage,

lib/storage/named-backups-entry.ts

@@ -0,0 +1,23 @@
+import type { NamedBackupSummary } from "../storage.js";
+
+export async function getNamedBackupsEntry(params: {
+	getStoragePath: () => string;
+	collectNamedBackups: (
+		storagePath: string,
+		deps: {
+			loadAccountsFromPath: (
+				path: string,
+			) => Promise<{ normalized: { accounts: unknown[] } | null }>;
+			logDebug: (message: string, details: Record<string, unknown>) => void;
+		},
+	) => Promise<NamedBackupSummary[]>;
+	loadAccountsFromPath: (
+		path: string,
+	) => Promise<{ normalized: { accounts: unknown[] } | null }>;
+	logDebug: (message: string, details: Record<string, unknown>) => void;
+}): Promise<NamedBackupSummary[]> {
+	return params.collectNamedBackups(params.getStoragePath(), {
+		loadAccountsFromPath: params.loadAccountsFromPath,
+		logDebug: params.logDebug,
+	});
+}

test/named-backups-entry.test.ts

@@ -0,0 +1,23 @@
+import { describe, expect, it, vi } from "vitest";
+import { getNamedBackupsEntry } from "../lib/storage/named-backups-entry.js";
+
+describe("named backups entry", () => {
+	it("passes storage path and dependencies through to named backup collection", async () => {
+		const collectNamedBackups = vi.fn(async () => []);
+		const loadAccountsFromPath = vi.fn(async () => ({ normalized: null }));
+		const logDebug = vi.fn();
+
+		const result = await getNamedBackupsEntry({
+			getStoragePath: () => "/tmp/accounts.json",
+			collectNamedBackups,
+			loadAccountsFromPath,
+			logDebug,
+		});
+
+		expect(collectNamedBackups).toHaveBeenCalledWith("/tmp/accounts.json", {
+			loadAccountsFromPath,
+			logDebug,
+		});
+		expect(result).toEqual([]);
+	});
+});