Verify restore_mask workflow matches PR #1172 fix

[djdarcy]

Description

Ensure that the restore_mask functionality in Preview Bridge Extended correctly handles the standard inpainting workflow that PR #1172 addressed in the original Impact-Pack.

Standard Workflow to Validate

1. Run SDXL/generation workflow → output goes to Preview Bridge
2. Preview Bridge blocks (block=if_empty_editor)
3. User locks seed value in their seed node
4. User draws mask in MaskEditor, saves
5. User re-runs workflow
6. Mask is preserved correctly for inpainting step
7. Repeat steps 4-6 for iterative refinement

Key Scenarios to Test

• restore_mask=always: Mask preserved across all re-runs
• restore_mask=if_same_size: Mask preserved only when image dimensions match
• restore_mask=never: Mask cleared each run (fresh start)
• Multiple iterations: Mask edits accumulate correctly
• Image change detection: Cache invalidates when new image arrives

Related

• Impact-Pack PR: fix(PreviewBridge): ComfyUI v1.34+ compatibility - widget.value non-configurable ltdrdata/ComfyUI-Impact-Pack#1172
• Any fixes/improvements should be mentioned in PR #1172 comments

Milestone

v0.3.3

[djdarcy]

Bug Found During Testing: restore_mask=if_same_size Bypass

While testing the restore_mask scenarios, discovered a bug where restore_mask=if_same_size was being bypassed.

Root Cause

Clipspace file persists on disk across image changes. When loading clipspace in _load_clipspace_mask(), it was resized to new image dimensions regardless of restore_mask setting.

Fix (v0.3.2-alpha)

Added restore_mask check before resizing clipspace when dimensions differ:

• if_same_size: Clear mask when sizes don't match
• never: Clear mask when sizes don't match
• always: Resize and restore (unchanged)

Analysis

See 2026-01-12__19-27-47__restore-mask-if-same-size-bug.md for detailed DEV WORKFLOW PROCESS analysis.

Commit

1f54b11 - fix(0.3.2): restore_mask=if_same_size bypassed by clipspace persistence