Merge branch 'dev/ptrottier/presets' into 'main'

REMIX-4985: Added Presets + Various Other Improvements

See merge request lightspeedrtx/lss-ai-tools/comfyui-rtx_remix!45

Author: po-trottier

CHANGELOG.md

@@ -15,6 +15,33 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Removed
 
+## [3.0.0] - 2026-02-10
+
+### Added
+
+- **Preset Panel**: New sidebar panel to save, manage, and apply preset configurations for tagged inputs
+  - Save current input values as named presets
+  - Apply presets to restore saved values
+  - Set default presets that auto-apply on workflow load
+  - Edit and delete presets via context menu
+  - Group presets for organization
+- **Grouped List Component**: Reusable collapsible grouped list UI component
+- **Popover Component**: Reusable popover UI component for contextual actions
+- **Form Field Components**: Standardized input components (text, number, checkbox, select, textarea)
+- **Graph Store**: Centralized store for managing graph state and tagged inputs
+- **Preset Store**: Centralized store for managing presets with localStorage persistence
+- **Frontend Documentation**: Added web/README.md with architecture and contribution guidelines
+- **Download Node**: Auto-reset hash and archive fields when URL changes to prevent stale data
+- **Download Node**: Progress bar in UI during file downloads
+- **Download Node**: Cancellation support - stops download and cleans up partial files when prompt is cancelled
+
+### Changed
+
+- **Export Dialog**: Improved layout, better details panel, pending changes detection
+- **UI Architecture**: Refactored to controller/core/store pattern for better separation of concerns
+- **Keyboard Shortcuts**: Improved keybind handling and consistency
+- **Styling**: Enhanced CSS with new utility classes, variables, and animations
+
 ## [2.2.1] - 2026-02-23
 
 ### Fixed

README.md

@@ -45,8 +45,9 @@ Mixing approaches is also _possible_ but **not recommended**.
 **Key benefits:**
 
 - Simple to set up and use: Tag your inputs and outputs, export the workflow, and the RTX Remix Toolkit will discover the workflow automatically.
-- Works directly from within the RTX RemixToolkit interface: Queue up jobs and set the workflow input values directly from the RTX Remix Toolkit interface.
+- Works directly from within the RTX Remix Toolkit interface: Queue up jobs and set the workflow input values directly from the RTX Remix Toolkit interface.
 - Easy workflow development and iteration: use standard ComfyUI nodes like `Load Image` to test your workflow without the RTX Remix Toolkit running
+- Built-in [Presets](#managing-presets) system for saving and switching between different parameter configurations during development
 
 👉 **[Jump to Integration Workflow Guide](#integration-workflow-guide)**
 
@@ -85,6 +86,8 @@ The integration workflow system consists of three key steps:
 2. **Tag inputs and outputs** - Mark which inputs/outputs the RTX Remix Toolkit should control
 3. **Export the workflow** - Save the workflow for the RTX Remix Toolkit to use
 
+Optionally, use **[Presets](#managing-presets)** to save and compare different parameter configurations for your tagged inputs during workflow development.
+
 Once exported, your workflow can be selected and run directly from within the RTX Remix Toolkit, which will:
 
 - Provide input texture paths and parameters to tagged input slots
@@ -170,6 +173,8 @@ Output nodes are the final nodes in your workflow that produce the textures/mesh
 - **Green circles** on input slots = Tagged input
 - **Green outline** around entire node = Tagged output (must save files to disk)
 
+**Tip:** Tagged inputs also appear in the [Presets](#managing-presets) sidebar panel, where you can save different value configurations and organize inputs into groups.
+
 ### Step 3: Export the Workflow
 
 Once you've tagged your inputs and outputs, export the workflow for use in the RTX Remix Toolkit.
@@ -199,7 +204,8 @@ The exported workflow is saved to the `user/rtx-remix/workflows/` and `user/rtx-
 - **Set texture types for outputs** - Configure which texture map type each output represents (albedo, normal, roughness, etc.) - essential for proper shader input matching when using multiple outputs
 - **Add descriptions** - Provide user guidance for each parameter
 - **Set value ranges** - Define min/max constraints for numeric inputs
-- **Reorder inputs/outputs** - Drag and drop rows to change the order
+- **Reorder inputs/outputs** - Drag and drop rows to change order within a group
+- **Reorder groups** - Drag group headers to change the order of entire groups
 - **Expand metadata** - Click the chevron to see and edit additional fields
 
 ### Using Your Workflow in the RTX Remix Toolkit
@@ -220,6 +226,93 @@ After exporting, your workflow becomes available in the RTX Remix Toolkit's work
 - **Outputs:** Your workflow saves textures/meshes to disk (e.g., using `RTX Remix Save Texture` or `Save Image` nodes)
 - **Integration:** The RTX Remix Toolkit retrieves the output file paths from ComfyUI's output history and imports the processed assets into your project
 
+### Managing Presets
+
+Presets let you save and switch between different sets of parameter values for your tagged inputs. They're useful for:
+
+- Testing your workflow with different parameter combinations (e.g., different upscale strengths, model choices, or prompt variations)
+- Shipping workflows with pre-configured settings for different use cases
+- Quickly comparing results across configurations without manually adjusting every input
+
+Presets only work with [tagged inputs](#tagging-input-slots) — tag at least one input before creating presets.
+
+#### The Preset Panel
+
+Open the preset panel by clicking the **★** (star) icon in the ComfyUI sidebar. The panel has two sections:
+
+- **Preset list** (top) — All presets for the current workflow, with search, create, edit, and delete controls
+- **Tagged inputs** (bottom) — All tagged inputs for the active preset, with inline value editing and per-input actions
+
+#### How Presets Work
+
+Presets use a **default + override** model:
+
+1. **Default Values** — A special preset that stores the baseline value for every tagged input. Created automatically when you make your first custom preset.
+2. **Custom presets** — Store only the values that **differ** from the default. Any value not explicitly overridden falls back to the Default Values.
+
+**Value resolution order:** Custom preset override → Default Values → Current widget value
+
+This keeps presets lightweight — you only configure what's different, and everything else inherits the baseline.
+
+#### Creating and Managing Presets
+
+| Action | How |
+|--------|-----|
+| **Create** | Click **Create Preset** in the panel header. Enter a name and optional description. The preset captures any values that currently differ from the Default Values. |
+| **Switch** | Click a preset in the list to activate it. Its values are applied to all tagged widgets. If you have unsaved changes, you'll be prompted to save or discard. |
+| **Rename** | Click the edit icon on a preset. The Default Values preset cannot be renamed. |
+| **Delete** | Click the delete icon. Default Values can only be cleared when no other presets exist. |
+
+#### Editing Values and Overrides
+
+Values can be edited in two ways — both stay in sync:
+
+- **In the preset panel** — Edit inputs directly in the sidebar
+- **On the canvas** — Edit node widgets as usual; changes flow to the panel
+
+Inputs whose values differ from the Default Values show a **colored override indicator** on the left side of the row, making it easy to see which values are customized in the current preset.
+
+**Per-input actions** (click the **⋮** menu on any input row):
+
+- **Focus Node** — Center the canvas on the input's node
+- **Edit Metadata** — Change the input's export name, group, tooltip, or min/max/step values
+- **Apply to Default** — Promote this input's current value to the Default Values preset
+- **Reset to Default** — Revert this input back to the Default Values
+- **Untag Input** — Remove the RTX Remix tag from this input (removes it from all presets)
+
+**Bulk actions** (buttons above the inputs list):
+
+- **Apply All** — Apply all overrides in the current preset to the Default Values
+- **Reset All** — Revert all overrides in the current preset back to Default Values
+
+Changes are held as **pending** until you click **Save**. You can also enable auto-save to skip the manual step.
+
+#### Auto-Save
+
+Click the **▾** dropdown arrow next to the Save button to access save options:
+
+- **Discard** — Discard all pending changes and revert to the last saved state
+- **Auto-save** toggle — When enabled, changes save immediately as you edit. The manual Save button is disabled while auto-save is active.
+
+#### Organizing Inputs with Groups
+
+When a workflow has many tagged inputs, groups help keep them organized:
+
+1. Click **⋮** on any input → **Edit Metadata** → set a **Group** name
+2. Inputs sharing the same group name are displayed together in collapsible sections
+3. Drag group headers to reorder groups
+4. Drag rows within a group to reorder inputs
+5. Use the group header **⋮** menu to untag all inputs in a group at once
+
+Groups are shared between the preset panel and the [export dialog](#step-3-export-the-workflow) — organizing in one place updates the other.
+
+#### Preset Storage
+
+- Presets are **embedded in the workflow file** (stored in the graph's extra data). Loading a workflow restores all its presets automatically.
+- There are no external preset files — everything travels with the workflow.
+- Before exporting, the system checks for unsaved preset changes and prompts you to save or discard.
+- Presets manage input values for your development workflow. The exported workflow uses whatever values are currently set in the widgets at export time.
+
 ### Integration Workflow Nodes
 
 #### RTX Remix Download Model

VERSION.md

@@ -1 +1 @@
-2.2.1
+3.0.0

api/configs/dynamic_ui.py

@@ -27,7 +27,8 @@
     NodeUI,
     UrlHandler,
     UrlParser,
-    WidgetVisibility,
+    WidgetResetRule,
+    WidgetVisibilityRule,
 )
 from ...nodes.constant import ModelSource
 
@@ -61,12 +62,12 @@ class NodeUIConfig(str, Enum):
         ),
         visibility_rules=[
             # Show model_source only after a URL is entered
-            WidgetVisibility(
+            WidgetVisibilityRule(
                 source_field="url",
                 show_when_filled=["model_source"],
             ),
             # Show source-specific fields based on detected source
-            WidgetVisibility(
+            WidgetVisibilityRule(
                 source_field="model_source",
                 mapping={
                     ModelSource.HUGGINGFACE: [
@@ -79,13 +80,20 @@ class NodeUIConfig(str, Enum):
                 },
             ),
             # Show archive fields when extract_archive is enabled
-            WidgetVisibility(
+            WidgetVisibilityRule(
                 source_field="extract_archive",
                 mapping={
                     True: ["archive_model_filename", "extracted_model_hash"],
                 },
             ),
         ],
+        reset_rules=[
+            # Reset hash and archive fields when URL changes
+            WidgetResetRule(
+                source_field="url",
+                reset_fields=["file_hash", "archive_model_filename", "extracted_model_hash"],
+            ),
+        ],
         info_button=DynamicHelp(
             source_field="model_source",
             configs={

api/models/__init__.py

@@ -21,7 +21,8 @@
     NodeUI,
     UrlHandler,
     UrlParser,
-    WidgetVisibility,
+    WidgetResetRule,
+    WidgetVisibilityRule,
 )
 
 __all__ = [
@@ -30,5 +31,6 @@
     "NodeUI",
     "UrlHandler",
     "UrlParser",
-    "WidgetVisibility",
+    "WidgetResetRule",
+    "WidgetVisibilityRule",
 ]

api/models/ui.py

@@ -18,7 +18,8 @@
 __all__ = [
     "UrlParser",
     "UrlHandler",
-    "WidgetVisibility",
+    "WidgetVisibilityRule",
+    "WidgetResetRule",
     "HelpContent",
     "DynamicHelp",
     "NodeUI",
@@ -51,7 +52,7 @@ class UrlHandler(BaseModel):
     """URL parsers keyed by source value"""
 
 
-class WidgetVisibility(BaseModel):
+class WidgetVisibilityRule(BaseModel):
     """Shows/hides widgets based on another widget's value."""
 
     source_field: str
@@ -64,6 +65,16 @@ class WidgetVisibility(BaseModel):
     """Widget names to show when source_field has any non-empty value"""
 
 
+class WidgetResetRule(BaseModel):
+    """Resets widget values when another widget's value changes."""
+
+    source_field: str
+    """Widget whose change triggers the reset (e.g., "url")"""
+
+    reset_fields: list[str] = []
+    """Widget names to clear when source_field changes"""
+
+
 class HelpContent(BaseModel):
     """Content for a help dialog."""
 
@@ -93,8 +104,11 @@ class NodeUI(BaseModel):
     url_handler: UrlHandler | None = None
     """Auto-detect URLs and populate fields"""
 
-    visibility_rules: list[WidgetVisibility] = []
+    visibility_rules: list[WidgetVisibilityRule] = []
     """Conditional widget visibility rules"""
 
+    reset_rules: list[WidgetResetRule] = []
+    """Reset widget values when source fields change"""
+
     info_button: DynamicHelp | None = None
     """Context-sensitive help button"""

nodes/download.py

@@ -33,7 +33,9 @@
 
 import folder_paths
 from comfy_execution.utils import get_executing_context
+from comfy_execution.progress import get_progress_state
 from comfy_api.latest import io
+from comfy.model_management import processing_interrupted, InterruptProcessingException
 from server import PromptServer
 
 from .constant import (
@@ -310,7 +312,7 @@ def execute(
 
         # === PHASE 3: Download the file ===
         logger.info(f"Downloading {download_info.filename} from {model_source}")
-        cls._download_file(download_info.url, target_path, timeout, headers=download_info.headers)
+        cls._download_file(download_info.url, target_path, timeout, headers=download_info.headers, node_id=unique_id)
         logger.info(f"Downloaded to {target_path}")
 
         # === PHASE 4: Verify hash (if provided) ===
@@ -647,15 +649,18 @@ def _compute_file_hash(filepath: Path, hash_func=hashlib.sha256, prefix: bytes |
         return file_hash.hexdigest()
 
     @staticmethod
-    def _download_file(url: str, filepath: Path, timeout: int, headers: dict | None = None) -> None:
+    def _download_file(
+        url: str, filepath: Path, timeout: int, headers: dict | None = None, node_id: str | None = None
+    ) -> None:
         """
-        Download a file from URL to filepath with progress logging.
+        Download a file from URL to filepath with progress logging and UI updates.
 
         Args:
             url: The URL to download from
             filepath: Path where the file will be saved
             timeout: Download timeout in seconds
             headers: Optional HTTP headers to include in the request
+            node_id: Optional node ID for progress updates in the UI
 
         Raises:
             ValueError: If URL returns HTML content instead of a file
@@ -681,28 +686,57 @@ def _download_file(url: str, filepath: Path, timeout: int, headers: dict | None
         downloaded_size = 0
         last_logged_percent = -1
 
+        # Send initial progress
+        if node_id and total_size > 0:
+            get_progress_state().update_progress(node_id=node_id, value=0, max_value=total_size)
+
         # Download and validate content
         first_chunk = None
-        with filepath.open("wb") as f:
-            for chunk in response.iter_content(chunk_size=CHUNK_SIZE_BYTES):
-                if first_chunk is None:
-                    first_chunk = chunk
-                f.write(chunk)
-                downloaded_size += len(chunk)
-
-                # Log progress every 10%
-                if total_size > 0:
-                    percent = int((downloaded_size / total_size) * 100)
-                    if percent >= last_logged_percent + 10:
-                        last_logged_percent = (percent // 10) * 10
-                        downloaded_mb = downloaded_size / (1024 * 1024)
-                        total_mb = total_size / (1024 * 1024)
-                        logger.info(f"Downloading: {downloaded_mb:.1f} MB / {total_mb:.1f} MB ({last_logged_percent}%)")
-
-        # Log completion
+        try:
+            with filepath.open("wb") as f:
+                for chunk in response.iter_content(chunk_size=CHUNK_SIZE_BYTES):
+                    # Check if user cancelled the prompt
+                    if processing_interrupted():
+                        raise InterruptProcessingException()
+
+                    if first_chunk is None:
+                        first_chunk = chunk
+                    f.write(chunk)
+                    downloaded_size += len(chunk)
+
+                    # Update progress every 10%
+                    if total_size > 0:
+                        percent = int((downloaded_size / total_size) * 100)
+                        if percent >= last_logged_percent + 10:
+                            last_logged_percent = (percent // 10) * 10
+                            downloaded_mb = downloaded_size / (1024 * 1024)
+                            total_mb = total_size / (1024 * 1024)
+                            logger.info(f"Downloading: {downloaded_mb:.1f} MB / {total_mb:.1f} MB ({last_logged_percent}%)")
+
+                            # Update UI progress bar
+                            if node_id:
+                                get_progress_state().update_progress(
+                                    node_id=node_id,
+                                    value=downloaded_size,
+                                    max_value=total_size,
+                                )
+        except InterruptProcessingException:
+            # Clean up partial file on cancellation
+            logger.info("Download cancelled by user")
+            try:
+                if filepath.exists():
+                    filepath.unlink()
+                    logger.info(f"Deleted partial download: {filepath.name}")
+            except Exception as e:
+                logger.warning(f"Failed to delete partial download {filepath.name}: {e}")
+            raise
+
+        # Log completion and update UI
         if total_size > 0:
             total_mb = total_size / (1024 * 1024)
             logger.info(f"Download complete: {total_mb:.1f} MB (100%)")
+            if node_id:
+                get_progress_state().update_progress(node_id=node_id, value=total_size, max_value=total_size)
         else:
             downloaded_mb = downloaded_size / (1024 * 1024)
             logger.info(f"Download complete: {downloaded_mb:.1f} MB")