[GH-ISSUE #815] [DevTask] Refactor virtual environment uv pty (terminal) to separate class #227

New issue

Closed

opened 2026-10-31 22:10:19 +00:00 by bsakaguchi · 1 comment

bsakaguchi commented

2026-10-31 22:10:19 +00:00

Owner

Originally created by @webfiltered on GitHub (Feb 4, 2025).
Original GitHub issue: https://github.com/Comfy-Org/desktop/issues/815

Originally assigned to: @webfiltered on GitHub.

The VirtualEnvironment class requires refactoring into smaller, more maintainable units.

Splitting the pseudo terminal code out to its own class would have the largest impact.

┆Issue is synchronized with this Notion page by Unito

Originally created by @webfiltered on GitHub (Feb 4, 2025). Original GitHub issue: https://github.com/Comfy-Org/desktop/issues/815 Originally assigned to: @webfiltered on GitHub. The `VirtualEnvironment` class requires refactoring into smaller, more maintainable units. Splitting the pseudo terminal code out to its own class would have the largest impact. ┆Issue is synchronized with this [Notion page](https://www.notion.so/Issue-815-DevTask-Refactor-virtual-environment-uv-pty-terminal-to-separate-class-1906d73d365081d8bdbaf4dbed755eda) by [Unito](https://www.unito.io)

bsakaguchi

2026-10-31 22:10:19 +00:00

closed this issue
added the
enhancement
label

bsakaguchi commented

2026-10-31 22:10:20 +00:00

Author

Owner

@dosubot[bot] commented on GitHub (May 6, 2025):

Hi, @webfiltered. I'm Dosu, and I'm helping the desktop team manage their backlog. I'm marking this issue as stale.

Issue Summary:

Proposal to refactor the VirtualEnvironment class.
Aim is to isolate pseudo terminal code into a separate class.
Expected to improve code clarity and modularity.
No comments or activities have been made on this issue yet.

Next Steps:

Is this issue still relevant to the latest version of the desktop repository? If so, please comment to keep the discussion open.
If there are no updates, the issue will be automatically closed in 7 days.

Thank you for your understanding and contribution!

@dosubot[bot] commented on GitHub (May 6, 2025): Hi, @webfiltered. I'm [Dosu](https://dosu.dev), and I'm helping the desktop team manage their backlog. I'm marking this issue as stale. **Issue Summary:** - Proposal to refactor the `VirtualEnvironment` class. - Aim is to isolate pseudo terminal code into a separate class. - Expected to improve code clarity and modularity. - No comments or activities have been made on this issue yet. **Next Steps:** - Is this issue still relevant to the latest version of the desktop repository? If so, please comment to keep the discussion open. - If there are no updates, the issue will be automatically closed in 7 days. Thank you for your understanding and contribution!

bsakaguchi referenced this issue

2026-10-31 22:16:07 +00:00

[PR #227] [MERGED] Rename ComfyUI-Manager to manager-core. #703

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

Comfy-Org/desktop#227

No description provided.

Rows
Columns