[DevTask] Explicit virtual environment management in ComfyInstallation class

[huchenlei]

Currently virtual environment is recreated everytime basePath is updated implicitly.

desktop/src/main-process/comfyInstallation.ts

Lines 38 to 47 in a6a59f6

	_basePath: string;
	/** The base path of the desktop app. Models, nodes, and configuration are saved here by default. */
	get basePath() {
	return this._basePath;
	}
	set basePath(value: string) {
	// Duplicated in constructor to avoid non-nullable type assertions.
	this._basePath = value;
	this.virtualEnvironment = this.createVirtualEnvironment(value);
	}

We should make this more explicit, i.e. move the side effect in the setter to the callsites.

┆Issue is synchronized with this Notion page by Unito

[webfiltered]

Thought I already had an issue for this, but apparently not - thanks!