The global Alt+Left/Right keydown handler called preventDefault() on the
browser's Back/Forward shortcut for the whole document — the most likely
cause of "can't switch workflow" after the previous deploy, and a poor
default regardless. Remove the handler (stepToSnapshot remains for a
future conflict-free binding or a timeline button).
Also wrap the post-switch baseline seeding in try/catch so any failure
there can never propagate into ComfyUI's openWorkflow action and block a
workflow switch.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Browsing the timeline still created a snapshot per swap — the exact spam
this work set out to remove. Two causes:
1. After loadGraphData, the dedup baseline (lastCapturedHashMap) was
seeded from the stored record.graphData, but captureSnapshot hashes a
fresh app.graph.serialize() of the loaded graph. ComfyUI's
re-serialization isn't byte-identical to the saved bytes, so the hash
never matched and the pre-swap "Current" capture fired every time.
Seed the baseline from the live re-serialization instead, so the next
capture dedupes to a no-op.
2. The pre-swap/pre-restore "Current" capture used skipCosmetic=false, so
post-load cosmetic settling (node size/position nudges) or a stray move
still produced a snapshot. Make these automatic captures skip cosmetic
like the auto path — only explicit manual saves (Ctrl+S / Snapshot
button) keep cosmetic-only changes now.
Net effect: browsing between snapshots produces zero snapshots; only a
genuine unsaved meaningful edit is preserved before a load.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Reworks snapshot capture and navigation toward a Fusion-360-style
non-destructive timeline, addressing three pain points.
- Workflow-switch spam: re-seed the dedup/change-detection baseline for
the newly-opened tab (seedWorkflowBaseline) and suppress auto-capture
for a short guard window (suppressAutoCapture/SWITCH_GUARD_MS), so the
graphChanged fired by loadGraphData no longer spawns a redundant
snapshot of a workflow you only just opened.
- Quieter autosave: detectChangeType now recognises resize, collapse and
pin (previously saved as "unknown") and classifies pure
move/resize/collapse as "cosmetic"; a cosmetic flag never escalates a
real edit. Auto-capture skips cosmetic-only changes; mode (mute/bypass)
is treated as a meaningful change. The cosmetic gate applies to the
auto path only — manual saves and the pre-swap/pre-restore "Current"
capture still preserve everything (no silent layout loss).
- Semantic "what changed": getLiveWidgetNames/widgetNameFor map
widgets_values indices to widget names (by exact node id), so diffs and
tooltips read "seed", "text", "cfg" instead of "Value[6]"; the diff
modal shows meaningful params first and collapses position/size into a
single muted "Layout" line.
- Non-destructive navigation: Alt+Left/Right step through history via
stepToSnapshot (quiet, re-entrancy-guarded swap with a position toast);
jumping between saved states is a storage no-op and never deletes later
snapshots.
Includes research report + implementation plan under docs/plans.
Verified: node --check passes; 19 unit tests on the diff/classification
logic pass.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
5 tasks: hide branching, skip move autosaves, compute diff at capture,
show diff in sidebar tooltip, show diff in timeline tooltip.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add cycle detection and O(n) path building to getAllBranches, and a
concurrency guard on timeline refresh to drop overlapping calls.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Store snapshots and profiles under <user_dir>/snapshot_manager/ instead
of <extension>/data/ so user data survives extension updates and reinstalls.
Automatically migrates existing data on first load.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace the single global branchingEnabled boolean with a two-tier
system: a ComfyUI settings default and per-workflow overrides via the
sidebar Branch button, persisted independently in localStorage.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The footer "Clear All Snapshots" button deleted records from the server
but left stale in-memory state (lastCapturedIdMap, activeSnapshotId,
etc.) intact. New captures then referenced deleted parents, creating
isolated branches. Reset all branching state on both clear-all paths.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
When SaveSnapshot receives an image tensor, extract a 200x150 JPEG
thumbnail (base64) and include it in the snapshot record. Sidebar shows
a small preview, tooltip displays the generated image instead of SVG,
and the preview modal shows the image above the graph.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Concurrent captureSnapshot calls (e.g. manual Ctrl+S overlapping a
debounced auto-capture) could both pass the hash check before either
sets the hash, creating duplicates. A simple boolean flag now ensures
only one capture runs at a time.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Store branchingEnabled and timelineExpanded in localStorage. Restore
on init with matching UI state (button class, expand arrow, bar class).
Also persist the forced collapse when branching is disabled.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Pruning could delete unlocked intermediate nodes between locked
snapshots and the root, creating orphan branches. Now ancestors of
all locked snapshots are added to the protected set in both regular
and node capture paths.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
When activeBranchSelections is empty (after reload or tab switch),
selectBranchContaining is called with the effective active ID so the
highlighted branch matches the active snapshot ring.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Shows × on hover for each workflow row. Confirms before deleting all
snapshots. Clears stale in-memory state for the deleted workflow.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Seed lastCapturedIdMap, lastCapturedHashMap, and lastGraphDataMap
from most recent snapshot on startup so the active ring appears
immediately after page reload
- Seed lastCapturedIdMap on workflow tab switch so the ring appears
when switching to a tab with existing snapshots
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Active ring now falls back to lastCapturedIdMap when activeSnapshotId
is null (cleared on auto-capture), so the ring persists
- Latest snapshot gets a yellow glow ring for quick identification
- Cache getWorkflowKey() in restore/swap/refresh/populatePicker
- Inline getEffectiveWorkflowKey() to avoid redundant getWorkflowKey()
- Replace double filter() with single loop for record counting
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Fetches the latest snapshot and loads it as a named temporary workflow
tab, keeping the current workflow untouched.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ComfyUI only supports one sidebar at a time, so opening the
workflows panel just replaces the snapshot manager. Revert to
simple view-only behavior for cross-workflow snapshot browsing.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Instead of trying to programmatically switch the active workflow,
open the ComfyUI workflows sidebar panel so the user can switch
from there. The snapshot picker still shows the selected workflow's
snapshots in view-only mode.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The store's openWorkflow only updates Pinia state without switching
the canvas. Use app.loadGraphData with the workflow object instead,
matching ComfyUI's internal workflow service pattern.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Expanded timeline: non-active branch rows now only show their unique
markers after the fork point, indented to align with the active branch.
Removes the dimmed-ancestor approach in favor of cleaner layout.
Workflow picker: clicking a workflow that is open in ComfyUI now
switches to that tab instead of just viewing its snapshots.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Adds a toggle button (▾/▴) to the timeline bar that expands it
vertically, displaying every branch as its own row. The active
branch is highlighted, shared ancestors are dimmed on other rows,
and clicking any marker on a non-active row switches to that branch.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Top arrow aligns content to flex-start (top edge), bottom arrow to
flex-end (bottom edge), so both arrow tips sit at their respective
edges instead of centering in the available space.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Arrows inherit the marker's --snap-marker-color so they match the
node color (blue for regular, purple for node snapshots, green for
current, etc).
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Fork group fills full timeline height with space-between layout.
Arrow buttons flex to fill remaining space so both arrow tips reach
their respective edges of the timeline bar.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Use margin-bottom:-3px on the top arrow to compensate for the Unicode
glyph's built-in whitespace and pull it tight against the marker.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Restore 8px size for both arrows. Top arrow uses align-items:flex-end
to push glyph down against marker, bottom arrow uses flex-start to
push glyph up against marker, making them symmetrical.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Reduce arrow button font-size and height from 8px to 6px and set
line-height to 0 so both arrows sit symmetrically tight against the
marker.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
New "Branch" button next to "Hide Auto" in the search row. When
toggled off: captures have no parentId, sidebar/timeline show a flat
timestamp-sorted list, branch navigators are hidden, and pruning
skips tree-aware protection.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Remove the fork count badge from timeline markers and the wrapper div.
Reduce arrow button height and remove extra spacing so the up arrow
sits flush against the marker.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace horizontal arrow layout with vertical stack: up/down arrows
above and below the marker, with a small blue badge showing the fork
count on the marker node.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add documentation for snapshot branching, session profiles,
hide auto-saves toggle, and timeline branch navigation.
Update features list, usage sections, architecture, and FAQ.
Align pyproject.toml and README badge to 3.0.0.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Toggle button next to the search bar hides Auto and Initial
snapshots to reduce clutter. Filter combines with text search
and persists across refreshes.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Branching: snapshots now track parentId to form a tree structure.
Swapping to an old snapshot and editing forks into a new branch.
Sidebar and timeline show < 1/3 > navigators at fork points to
switch between branches. Pruning protects ancestors and fork points.
Deleting a fork point re-parents its children.
Profiles: save/load named sets of workflows as session profiles.
Backend stores profiles as JSON in data/profiles/. Sidebar has a
collapsible Profiles section with save, load, and delete.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
List, prune, and delete operations now use a lightweight in-memory cache
of snapshot metadata (everything except graphData). Only get_full_record()
and update_meta() hit disk after warm-up, keeping sidebar loads and
auto-capture prune cycles fast.
Key changes:
- snapshot_storage.py: cache layer (_cache, _cache_warmed, _extract_meta,
_ensure_cached), new get_full_record() and update_meta() functions,
all existing functions updated to use cache
- snapshot_routes.py: new /get and /update-meta endpoints
- snapshot_manager.js: db_getFullRecord() and db_updateMeta() helpers,
lazy graphData fetch in restore/swap/diff/preview/tooltip, label/notes/
lock use update_meta instead of full put to preserve graphData on disk
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The sidebar refresh didn't trigger a timeline update, leaving stale
markers visible until the next auto-capture or manual action.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Ethanfel