perf: optimize SessionListItem style recalculation

src/__tests__/cli/services/agent-spawner.test.ts

@@ -48,7 +48,7 @@ vi.mock('child_process', async (importOriginal) => {
 // Mock fs module
 vi.mock('fs', async (importOriginal) => {
 	const actual = await importOriginal<typeof import('fs')>();
-	return {
+	const mockFsModule = {
 		...actual,
 		readFileSync: vi.fn(),
 		writeFileSync: vi.fn(),
@@ -60,6 +60,10 @@ vi.mock('fs', async (importOriginal) => {
 			X_OK: 1,
 		},
 	};
+	return {
+		...mockFsModule,
+		default: mockFsModule,
+	};
 });
 
 // Mock os module

src/__tests__/integration/group-chat.integration.test.ts

@@ -70,7 +70,7 @@ function createMockProcessManager(): IProcessManager & {
 				toolType: config.toolType,
 				prompt: config.prompt,
 			});
-			return { pid: Math.floor(Math.random() * 10000), success: true };
+			return Promise.resolve({ pid: Math.floor(Math.random() * 10000), success: true });
 		},
 
 		write(sessionId: string, data: string) {

src/__tests__/main/debug-package/collectors.test.ts

@@ -38,6 +38,11 @@ vi.mock('fs', () => ({
 	statSync: vi.fn(() => ({ size: 0, isDirectory: () => false })),
 	readdirSync: vi.fn(() => []),
 	readFileSync: vi.fn(() => ''),
+	promises: {
+		stat: vi.fn(() => Promise.resolve({ size: 0, isDirectory: () => false })),
+		readdir: vi.fn(() => Promise.resolve([])),
+		readFile: vi.fn(() => Promise.resolve('')),
+	},
 }));
 
 // Mock cliDetection
@@ -817,14 +822,13 @@ describe('Debug Package Collectors', () => {
 			const { app } = await import('electron');
 
 			vi.mocked(app.getPath).mockReturnValue('/mock/userData');
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.statSync).mockImplementation((path: any) => {
+			vi.mocked(fs.promises.stat).mockImplementation(async (path: any) => {
 				if (path.includes('maestro-sessions.json')) {
 					return { size: 1024, isDirectory: () => false } as any;
 				}
 				return { size: 0, isDirectory: () => true } as any;
 			});
-			vi.mocked(fs.readdirSync).mockReturnValue([]);
+			vi.mocked(fs.promises.readdir).mockResolvedValue([]);
 
 			const { collectStorage } = await import('../../../main/debug-package/collectors/storage');
 
@@ -867,13 +871,12 @@ describe('Debug Package Collectors', () => {
 			const { app } = await import('electron');
 
 			vi.mocked(app.getPath).mockReturnValue('/mock/userData');
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readdirSync).mockReturnValue([
+			vi.mocked(fs.promises.readdir).mockResolvedValue([
 				'chat-1.json',
 				'chat-1.log.json',
 				'chat-2.json',
 			] as any);
-			vi.mocked(fs.readFileSync).mockImplementation((path: any) => {
+			vi.mocked(fs.promises.readFile).mockImplementation(async (path: any) => {
 				if (path.includes('chat-1.json') && !path.includes('.log')) {
 					return JSON.stringify({
 						id: 'chat-1',
@@ -930,7 +933,7 @@ describe('Debug Package Collectors', () => {
 		it('should handle missing group chats directory', async () => {
 			const fs = await import('fs');
 
-			vi.mocked(fs.existsSync).mockReturnValue(false);
+			vi.mocked(fs.promises.readdir).mockRejectedValue(new Error('Directory does not exist'));
 
 			const { collectGroupChats } =
 				await import('../../../main/debug-package/collectors/group-chats');