refactor: extract storage file path helpers

Author: ndycode

lib/storage.ts

@@ -23,6 +23,17 @@ export {
 } from "./storage/identity.js";
 
 import { normalizeEmailKey } from "./storage/identity.js";
+import {
+	ACCOUNTS_BACKUP_SUFFIX,
+	ACCOUNTS_WAL_SUFFIX,
+	getFlaggedAccountsPath as buildFlaggedAccountsPath,
+	getLegacyFlaggedAccountsPath as buildLegacyFlaggedAccountsPath,
+	getAccountsBackupPath,
+	getAccountsBackupRecoveryCandidates,
+	getAccountsWalPath,
+	getIntentionalResetMarkerPath,
+	RESET_MARKER_SUFFIX,
+} from "./storage/file-paths.js";
 import {
 	type AccountMetadataV1,
 	type AccountMetadataV3,
@@ -54,12 +65,8 @@ const log = createLogger("storage");
 const ACCOUNTS_FILE_NAME = "openai-codex-accounts.json";
 const FLAGGED_ACCOUNTS_FILE_NAME = "openai-codex-flagged-accounts.json";
 const LEGACY_FLAGGED_ACCOUNTS_FILE_NAME = "openai-codex-blocked-accounts.json";
-const ACCOUNTS_BACKUP_SUFFIX = ".bak";
-const ACCOUNTS_WAL_SUFFIX = ".wal";
-const ACCOUNTS_BACKUP_HISTORY_DEPTH = 3;
 const BACKUP_COPY_MAX_ATTEMPTS = 5;
 const BACKUP_COPY_BASE_DELAY_MS = 10;
-const RESET_MARKER_SUFFIX = ".reset-intent";
 let storageBackupEnabled = true;
 let lastAccountsSaveTimestamp = 0;
 
@@ -367,25 +374,6 @@ export function setStorageBackupEnabled(enabled: boolean): void {
 	storageBackupEnabled = enabled;
 }
 
-function getAccountsBackupPath(path: string): string {
-	return `${path}${ACCOUNTS_BACKUP_SUFFIX}`;
-}
-
-function getAccountsBackupPathAtIndex(path: string, index: number): string {
-	if (index <= 0) {
-		return getAccountsBackupPath(path);
-	}
-	return `${path}${ACCOUNTS_BACKUP_SUFFIX}.${index}`;
-}
-
-function getAccountsBackupRecoveryCandidates(path: string): string[] {
-	const candidates: string[] = [];
-	for (let i = 0; i < ACCOUNTS_BACKUP_HISTORY_DEPTH; i += 1) {
-		candidates.push(getAccountsBackupPathAtIndex(path, i));
-	}
-	return candidates;
-}
-
 async function getAccountsBackupRecoveryCandidatesWithDiscovery(
 	path: string,
 ): Promise<string[]> {
@@ -425,10 +413,6 @@ async function getAccountsBackupRecoveryCandidatesWithDiscovery(
 	return [...knownCandidates, ...discoveredOrdered];
 }
 
-function getAccountsWalPath(path: string): string {
-	return `${path}${ACCOUNTS_WAL_SUFFIX}`;
-}
-
 async function copyFileWithRetry(
 	sourcePath: string,
 	destinationPath: string,
@@ -605,10 +589,6 @@ function computeSha256(value: string): string {
 	return createHash("sha256").update(value).digest("hex");
 }
 
-function getIntentionalResetMarkerPath(path: string): string {
-	return `${path}${RESET_MARKER_SUFFIX}`;
-}
-
 function createEmptyStorageWithMetadata(
 	restoreEligible: boolean,
 	restoreReason: RestoreReason,
@@ -996,11 +976,14 @@ export async function exportNamedBackup(
 }
 
 export function getFlaggedAccountsPath(): string {
-	return join(dirname(getStoragePath()), FLAGGED_ACCOUNTS_FILE_NAME);
+	return buildFlaggedAccountsPath(getStoragePath(), FLAGGED_ACCOUNTS_FILE_NAME);
 }
 
 function getLegacyFlaggedAccountsPath(): string {
-	return join(dirname(getStoragePath()), LEGACY_FLAGGED_ACCOUNTS_FILE_NAME);
+	return buildLegacyFlaggedAccountsPath(
+		getStoragePath(),
+		LEGACY_FLAGGED_ACCOUNTS_FILE_NAME,
+	);
 }
 
 async function migrateLegacyProjectStorageIfNeeded(

lib/storage/file-paths.ts

@@ -0,0 +1,48 @@
+import { dirname, join } from "node:path";
+
+export const ACCOUNTS_BACKUP_SUFFIX = ".bak";
+export const ACCOUNTS_WAL_SUFFIX = ".wal";
+const ACCOUNTS_BACKUP_HISTORY_DEPTH = 3;
+export const RESET_MARKER_SUFFIX = ".reset-intent";
+
+export function getAccountsBackupPath(path: string): string {
+	return `${path}${ACCOUNTS_BACKUP_SUFFIX}`;
+}
+
+export function getAccountsBackupPathAtIndex(
+	path: string,
+	index: number,
+): string {
+	if (index <= 0) return getAccountsBackupPath(path);
+	return `${path}${ACCOUNTS_BACKUP_SUFFIX}.${index}`;
+}
+
+export function getAccountsBackupRecoveryCandidates(path: string): string[] {
+	const candidates: string[] = [];
+	for (let i = 0; i < ACCOUNTS_BACKUP_HISTORY_DEPTH; i += 1) {
+		candidates.push(getAccountsBackupPathAtIndex(path, i));
+	}
+	return candidates;
+}
+
+export function getAccountsWalPath(path: string): string {
+	return `${path}${ACCOUNTS_WAL_SUFFIX}`;
+}
+
+export function getIntentionalResetMarkerPath(path: string): string {
+	return `${path}${RESET_MARKER_SUFFIX}`;
+}
+
+export function getFlaggedAccountsPath(
+	storagePath: string,
+	fileName: string,
+): string {
+	return join(dirname(storagePath), fileName);
+}
+
+export function getLegacyFlaggedAccountsPath(
+	storagePath: string,
+	legacyFileName: string,
+): string {
+	return join(dirname(storagePath), legacyFileName);
+}