test: reset transaction mock implementations between repair-commands tests

vi.clearAllMocks() wipes call history but keeps implementations set via
mockImplementation, so the storage transaction mocks overridden inline by
individual tests bled into later tests. Reset all three in beforeEach;
their factory defaults are bare vi.fn(), so mockReset restores the exact
initial state.

https://claude.ai/code/session_01XNtnkLbBiXZxfQQYLMpucB

Author: claude

test/repair-commands.test.ts

@@ -158,6 +158,12 @@ function createDeps(
 describe("repair-commands direct deps coverage", () => {
 	beforeEach(() => {
 		vi.clearAllMocks();
+		// clearAllMocks wipes call history but keeps mockImplementation, so the
+		// transaction mocks set inline by individual tests must be reset here or
+		// they bleed into later tests that rely on the default behavior.
+		storageMocks.withAccountStorageTransaction.mockReset();
+		storageMocks.withFlaggedStorageTransaction.mockReset();
+		storageMocks.withAccountAndFlaggedStorageTransaction.mockReset();
 		existsSyncMock.mockReturnValue(false);
 		quotaCacheMocks.loadQuotaCache.mockResolvedValue(null);
 		codexCliStateMocks.loadCodexCliState.mockResolvedValue(null);