test: use resetAllMocks instead of clearAllMocks in runner-env-handlers tests

Convert from clearAllMocks() to resetAllMocks() to properly reset mock
implementations between tests. Tests already set up their own mock
implementations, so this provides cleaner isolation.

Resolves coderabbitai:comment_2939695330

apps/desktop/src/main/ipc-handlers/github/__tests__/runner-env-handlers.test.ts

@@ -269,9 +269,8 @@ function createProject(): Project {
 
 describe('GitHub TypeScript runner usage', () => {
   beforeEach(() => {
-    // Note: Using clearAllMocks() here instead of resetAllMocks() to preserve mock implementations
-    // TODO: Convert to resetAllMocks() and reset implementations in beforeEach
-    vi.clearAllMocks();
+    // Reset all mocks to clean state (resets both call history and implementations)
+    vi.resetAllMocks();
     mockIpcMain.reset();
     projectRef.current = createProject();
   });