Fix critical startup order bug - ensure ComfyUI setup before model downloads
Browse filesPreviously, download_models() created empty directories that caused ensure_setup()
to incorrectly skip cloning ComfyUI, resulting in 'main.py not found' errors.
This fix reorders the startup sequence to clone ComfyUI first, then download
models into the proper directories.
π€ Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
app.py
CHANGED
|
@@ -742,16 +742,16 @@ if __name__ == "__main__":
|
|
| 742 |
log_startup("=" * 50)
|
| 743 |
|
| 744 |
try:
|
| 745 |
-
# Step 1:
|
| 746 |
-
log_startup("π¨ Downloading Authenticity Stack models...")
|
| 747 |
-
download_models()
|
| 748 |
-
manager.models_downloaded = True
|
| 749 |
-
|
| 750 |
-
# Step 2: Ensure environment (cloning repos) is set up
|
| 751 |
if not manager.ensure_setup():
|
| 752 |
log_startup("β Initial setup failed", "ERROR")
|
| 753 |
raise RuntimeError("Setup failed")
|
| 754 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 755 |
# Step 3: Start server
|
| 756 |
log_startup("π§ Starting ComfyUI server...")
|
| 757 |
if not manager.start_server():
|
|
|
|
| 742 |
log_startup("=" * 50)
|
| 743 |
|
| 744 |
try:
|
| 745 |
+
# Step 1: Ensure environment (cloning repos) is set up
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 746 |
if not manager.ensure_setup():
|
| 747 |
log_startup("β Initial setup failed", "ERROR")
|
| 748 |
raise RuntimeError("Setup failed")
|
| 749 |
|
| 750 |
+
# Step 2: Download all models after ComfyUI is set up
|
| 751 |
+
log_startup("π¨ Downloading Authenticity Stack models...")
|
| 752 |
+
download_models()
|
| 753 |
+
manager.models_downloaded = True
|
| 754 |
+
|
| 755 |
# Step 3: Start server
|
| 756 |
log_startup("π§ Starting ComfyUI server...")
|
| 757 |
if not manager.start_server():
|