The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		May 26, 2026 9:28am

PR Summary

Medium Risk
Changes where undo history is stored and how it is migrated/cleared on load and sign-out; mistakes could lose history or briefly race during rehydration, though tests cover migration and write ordering.

Overview
Moves workflow undo/redo persistence off localStorage onto IndexedDB via a new indexedDBStorage adapter (same idb-keyval pattern as the terminal console store), so large undo snapshots no longer compete for the origin’s ~5 MB localStorage quota.

On first load, a one-time migration copies any existing workflow-undo-redo payload into IndexedDB, sets a migration flag, and removes the legacy localStorage key. The adapter awaits migration on reads/writes, gates writes until the first rehydration getItem finishes (avoids overwriting persisted state with empty in-memory state), and swallows most IDB errors on get/set/remove so the app keeps running.

The undo/redo Zustand store drops the inline safeStorageAdapter and uses createJSONStorage(() => indexedDBStorage). Sign-out (clearUserData) now also calls clearPersistedUndoRedo() so undo history does not linger across sessions. New unit tests cover migration, adapter behavior, clear-on-logout, and the hydration race.

^{Reviewed by Cursor Bugbot for commit 2a8bf5f. Bugbot is set up for automated code reviews on this repo. Configure here.}

Greptile Summary

This PR migrates the workflow-undo-redo Zustand persist store from localStorage to IndexedDB via idb-keyval, matching the pattern already established by terminal-console-store. The inline safeStorageAdapter (which silently swallowed quota errors without freeing origin budget) is replaced with an async indexedDBStorage adapter that includes a one-time migration on first load.

• storage.ts: New adapter wrapping idb-keyval with a module-level migration routine that copies any existing localStorage payload to IndexedDB, removes the legacy key, and marks completion with a sentinel — preventing re-runs and leaving localStorage intact on IDB failure so the migration can retry.
• store.ts: Single-line change swapping safeStorageAdapter for indexedDBStorage in the persist config.
• storage.test.ts: 10 test cases covering migration idempotency, IDB failure resilience, and all three adapter operations (get/set/remove).

Confidence Score: 4/5

Safe to merge; the migration is idempotent and adapter errors are all caught, so there is no data-loss path.

The migration logic is sound — IDB writes are awaited before the localStorage key is removed, the sentinel prevents re-runs, and failures leave localStorage intact for a retry. The only open questions are a minor SSR-guard naming inconsistency with the sibling storage file and the absence of a migration-gate in setItem, which is safe in the current Zustand flow but could surprise future callers. Test coverage is solid across 10 cases with one small gap (no removeItem error-swallowing test).

apps/sim/stores/undo-redo/storage.ts — the setItem/removeItem migration gate and SSR guard style are worth a second look before merging.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant Module as storage.ts (module load)
    participant IDB as IndexedDB (idb-keyval)
    participant LS as localStorage
    participant Store as useUndoRedoStore (Zustand persist)

    Module->>IDB: get(MIGRATION_KEY)
    alt already migrated
        IDB-->>Module: true → return early
    else first load
        IDB-->>Module: undefined
        Module->>LS: getItem("workflow-undo-redo")
        alt data exists
            LS-->>Module: legacyPayload
            Module->>IDB: set("workflow-undo-redo", legacyPayload)
            Module->>LS: removeItem("workflow-undo-redo")
        end
        Module->>IDB: set(MIGRATION_KEY, true)
    end

    Note over Module: migrationPromiseInternal resolves → set to null

    Store->>Module: indexedDBStorage.getItem("workflow-undo-redo")
    Module->>Module: await migrationPromiseInternal (if still pending)
    Module->>IDB: get("workflow-undo-redo")
    IDB-->>Store: persisted JSON string (or null)

    Store->>Module: indexedDBStorage.setItem("workflow-undo-redo", newState)
    Module->>IDB: set("workflow-undo-redo", newState)

_{Reviews (1): Last reviewed commit: "feat(undo-redo): migrate persistence fro..." | Re-trigger Greptile}