Add test to validate.

Author: robinjhuang

src/handlers/pathHandlers.ts

@@ -34,6 +34,12 @@ export function registerPathHandlers() {
     // Remove OneDrive from documents path if present
     if (process.platform === 'win32') {
       documentsPath = documentsPath.replace(/OneDrive\\/, '');
+      // We should use path.win32.join for Windows paths
+      return {
+        appData: app.getPath('appData'),
+        appPath: app.getAppPath(),
+        defaultInstallPath: path.win32.join(documentsPath, 'ComfyUI'),
+      };
     }
 
     return {

tests/unit/handlers/pathHandler.test.ts

@@ -110,6 +110,32 @@ const getRegisteredHandler = <T extends (...args: never[]) => unknown>(
   return handler as unknown as HandlerType<T>;
 };
 
+type ElectronPathName =
+  | 'home'
+  | 'appData'
+  | 'userData'
+  | 'sessionData'
+  | 'temp'
+  | 'exe'
+  | 'module'
+  | 'desktop'
+  | 'documents'
+  | 'downloads'
+  | 'music'
+  | 'pictures'
+  | 'videos'
+  | 'recent'
+  | 'logs'
+  | 'crashDumps';
+
+const mockPaths = (overrides: Partial<Record<ElectronPathName, string>> = {}) => {
+  vi.mocked(app.getPath).mockImplementation((name: ElectronPathName): string => {
+    if (name in overrides) return overrides[name]!;
+    if (name in MOCK_PATHS) return MOCK_PATHS[name as keyof typeof MOCK_PATHS];
+    return path.normalize(`/mock/${name}`);
+  });
+};
+
 describe('PathHandlers', () => {
   beforeEach(() => {
     vi.mocked(app.getPath).mockImplementation(
@@ -328,6 +354,19 @@ describe('PathHandlers', () => {
         defaultInstallPath: path.join('/mock/documents', 'ComfyUI'),
       });
     });
+
+    it('Windows: should remove OneDrive from documents path', async () => {
+      // Mock Windows platform
+      const restorePlatform = mockProcessPlatform('win32');
+      mockPaths({ documents: String.raw`C:\Users\Test\OneDrive\Documents` });
+
+      const result = await getSystemPathsHandler({});
+      const expected = String.raw`C:\Users\Test\Documents\ComfyUI`;
+      expect(result.defaultInstallPath).toBe(expected);
+
+      // Restore original platform
+      restorePlatform();
+    });
   });
 
   describe('validate-comfyui-source', () => {