Ethanfel/Comfyui-Workflow-Snapshot-Manager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Skip redundant auto-save when browsing between snapshots

Browse Source 
When activeSnapshotId is set, the current graph state is already a
saved snapshot. Swapping to another one no longer creates a duplicate
"Current" entry — the auto-save only fires on the first swap away
from an unsaved graph state.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Ethanfel
2026-02-25 00:46:38 +01:00
parent e554dc9973
commit 73cde5ade1
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
js/snapshot_manager.js
View File

@@ -544,11 +544,13 @@ async function restoreSnapshot(record) {
} }
async function swapSnapshot(record) { async function swapSnapshot(record) {
// Auto-save current state before swapping (so user can get back) // Auto-save current state before swapping (so user can get back),
// but skip if the graph is already a saved snapshot (browsing between old ones)
const prevCurrentId = currentSnapshotId; const prevCurrentId = currentSnapshotId;
const capturedId = await captureSnapshot("Current"); if (!activeSnapshotId) {
// captureSnapshot clears currentSnapshotId; restore or update it const capturedId = await captureSnapshot("Current");
currentSnapshotId = capturedId || prevCurrentId; currentSnapshotId = capturedId || prevCurrentId;
}
await withRestoreLock(async () => { await withRestoreLock(async () => {
if (!validateSnapshotData(record.graphData)) { if (!validateSnapshotData(record.graphData)) {