Remove unreliable settings backup

src/config/comfyConfigManager.ts

@@ -60,23 +60,12 @@ export class ComfyConfigManager {
     }
     this.createComfyDirectories(localComfyDirectory);
     const userSettingsPath = path.join(localComfyDirectory, 'user', 'default');
-    this.createComfyConfigFile(userSettingsPath, true);
+    this.createComfyConfigFile(userSettingsPath);
   }
 
-  public static createComfyConfigFile(userSettingsPath: string, overwrite: boolean = false): void {
+  public static createComfyConfigFile(userSettingsPath: string): void {
     const configFilePath = path.join(userSettingsPath, 'comfy.settings.json');
 
-    if (fs.existsSync(configFilePath) && overwrite) {
-      const backupFilePath = path.join(userSettingsPath, 'old_comfy.settings.json');
-      try {
-        fs.renameSync(configFilePath, backupFilePath);
-        log.info(`Renaming existing user settings file to: ${backupFilePath}`);
-      } catch (error) {
-        log.error(`Failed to backup existing user settings file: ${error}`);
-        return;
-      }
-    }
-
     try {
       fs.writeFileSync(configFilePath, JSON.stringify(this.DEFAULT_CONFIG, null, 2));
       log.info(`Created new ComfyUI config file at: ${configFilePath}`);

tests/unit/comfyConfigManager.test.ts

@@ -89,33 +89,11 @@ describe('ComfyConfigManager', () => {
     it('should create new config file when none exists', () => {
       (fs.existsSync as jest.Mock).mockReturnValue(false);
 
-      ComfyConfigManager.createComfyConfigFile('/fake/path', false);
+      ComfyConfigManager.createComfyConfigFile('/fake/path');
 
       expect(fs.writeFileSync).toHaveBeenCalledTimes(1);
       expect(fs.renameSync).not.toHaveBeenCalled();
     });
-
-    it('should backup existing config file when overwrite is true', () => {
-      (fs.existsSync as jest.Mock).mockImplementation((path: string) => {
-        return path === '/user/default/comfy.settings.json';
-      });
-
-      ComfyConfigManager.createComfyConfigFile('/user/default', true);
-
-      expect(fs.renameSync).toHaveBeenCalledTimes(1);
-      expect(fs.writeFileSync).toHaveBeenCalledTimes(1);
-    });
-
-    it('should handle backup failure gracefully', () => {
-      (fs.existsSync as jest.Mock).mockReturnValue(true);
-      (fs.renameSync as jest.Mock).mockImplementation(() => {
-        throw new Error('Backup failed');
-      });
-
-      ComfyConfigManager.createComfyConfigFile('/fake/path', true);
-
-      expect(fs.writeFileSync).not.toHaveBeenCalled();
-    });
   });
 
   describe('createNestedDirectories', () => {