@continuum-dev/session 0.1.2 → 0.1.3

package/README.md

@@ -92,6 +92,7 @@ function hydrateOrCreate(options?: SessionOptions): Session;
 Persistence note:
 
 - When `options.persistence` is provided, snapshot writes are debounced by 200ms.
+- Pending writes are flushed on `beforeunload` so tab closes do not drop recent updates.
 - `SessionPersistenceOptions.maxBytes` enforces a payload size cap before writes.
 - `SessionPersistenceOptions.onError` receives `size_limit` and `storage_error` events.
 - Browser `storage` events are consumed for cross-tab session synchronization.
@@ -118,6 +119,8 @@ const stopIssues = session.onIssues((issues) => {
 });
 ```
 
+Snapshot listeners receive immutable top-level copies of `view` and `data`.
+
 ### 2) View and State Updates
 
 #### `session.pushView(view)`
@@ -148,6 +151,8 @@ session.recordIntent({
 });
 ```
 
+`recordIntent` clones incoming payload objects before storing them and deduplicates issues by `nodeId + code`.
+
 #### Viewport APIs
 
 ```typescript
@@ -310,6 +315,7 @@ Persistence behavior:
 
 - `serialize()` returns a JSON-safe payload with `formatVersion: 1`.
 - Automatic persistence writes are debounced (200ms) to reduce storage churn.
+- Pending writes are flushed on `beforeunload` to reduce data loss risk during tab close.
 - If `maxBytes` is exceeded, the write is skipped and `onError` is invoked.
 - Remote storage updates can rehydrate in-memory state for cross-tab continuity.
 

package/lib/session/checkpoint-manager.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"checkpoint-manager.d.ts","sourceRoot":"","sources":["../../../../../packages/session/src/lib/session/checkpoint-manager.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,qBAAqB,CAAC;AACtD,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAOvD;;;;;GAKG;AACH,wBAAgB,uBAAuB,CAAC,CAAC,EAAE,KAAK,EAAE,CAAC,GAAG,CAAC,CAEtD;AAED;;;;;;GAMG;AACH,wBAAgB,cAAc,CAAC,QAAQ,EAAE,YAAY,GAAG,IAAI,CAqB3D;AAED;;;;;GAKG;AACH,wBAAgB,sBAAsB,CAAC,QAAQ,EAAE,YAAY,GAAG,UAAU,CAezE;AAED;;;;;GAKG;AACH,wBAAgB,qBAAqB,CAAC,QAAQ,EAAE,YAAY,EAAE,EAAE,EAAE,UAAU,GAAG,IAAI,CAalF;AAED;;;;;GAKG;AACH,wBAAgB,MAAM,CAAC,QAAQ,EAAE,YAAY,EAAE,YAAY,EAAE,MAAM,GAAG,IAAI,CAkBzE"}
+{"version":3,"file":"checkpoint-manager.d.ts","sourceRoot":"","sources":["../../../../../packages/session/src/lib/session/checkpoint-manager.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,qBAAqB,CAAC;AACtD,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAOvD;;;;;GAKG;AACH,wBAAgB,uBAAuB,CAAC,CAAC,EAAE,KAAK,EAAE,CAAC,GAAG,CAAC,CAEtD;AAED;;;;;;GAMG;AACH,wBAAgB,cAAc,CAAC,QAAQ,EAAE,YAAY,GAAG,IAAI,CAqB3D;AAED;;;;;GAKG;AACH,wBAAgB,sBAAsB,CAAC,QAAQ,EAAE,YAAY,GAAG,UAAU,CAyBzE;AAED;;;;;GAKG;AACH,wBAAgB,qBAAqB,CAAC,QAAQ,EAAE,YAAY,EAAE,EAAE,EAAE,UAAU,GAAG,IAAI,CAalF;AAED;;;;;GAKG;AACH,wBAAgB,MAAM,CAAC,QAAQ,EAAE,YAAY,EAAE,YAAY,EAAE,MAAM,GAAG,IAAI,CAkBzE"}

package/lib/session/checkpoint-manager.js

@@ -59,6 +59,16 @@ export function createManualCheckpoint(internal) {
         trigger: 'manual',
     };
     internal.checkpoints.push(checkpoint);
+    if (internal.checkpoints.length > internal.maxCheckpoints) {
+        const overflow = internal.checkpoints.length - internal.maxCheckpoints;
+        for (let index = 0; index < overflow; index += 1) {
+            const removableIndex = internal.checkpoints.findIndex((cp) => cp.trigger === 'manual');
+            if (removableIndex === -1) {
+                break;
+            }
+            internal.checkpoints.splice(removableIndex, 1);
+        }
+    }
     return checkpoint;
 }
 /**

package/lib/session/event-log.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"event-log.d.ts","sourceRoot":"","sources":["../../../../../packages/session/src/lib/session/event-log.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAY,WAAW,EAA2B,MAAM,qBAAqB,CAAC;AAE1F,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAsDvD;;;;;;;;GAQG;AACH,wBAAgB,YAAY,CAC1B,QAAQ,EAAE,YAAY,EACtB,OAAO,EAAE,IAAI,CAAC,WAAW,EAAE,eAAe,GAAG,WAAW,GAAG,WAAW,GAAG,aAAa,CAAC,GACtF,IAAI,CAiFN"}
+{"version":3,"file":"event-log.d.ts","sourceRoot":"","sources":["../../../../../packages/session/src/lib/session/event-log.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAY,WAAW,EAA2B,MAAM,qBAAqB,CAAC;AAE1F,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAoEvD;;;;;;;;GAQG;AACH,wBAAgB,YAAY,CAC1B,QAAQ,EAAE,YAAY,EACtB,OAAO,EAAE,IAAI,CAAC,WAAW,EAAE,eAAe,GAAG,WAAW,GAAG,WAAW,GAAG,aAAa,CAAC,GACtF,IAAI,CA+EN"}

package/lib/session/event-log.js

@@ -3,6 +3,16 @@ import { generateId } from './session-state.js';
 import { buildSnapshotFromCurrentState, notifySnapshotAndIssueListeners } from './listeners.js';
 import { cloneCheckpointSnapshot } from './checkpoint-manager.js';
 import { validateNodeValue } from '@continuum-dev/runtime';
+function dedupeIssues(existing, incoming) {
+    if (incoming.length === 0) {
+        return existing;
+    }
+    const nextKeys = new Set(incoming.map((issue) => `${issue.nodeId ?? ''}:${issue.code}`));
+    return [
+        ...existing.filter((issue) => !nextKeys.has(`${issue.nodeId ?? ''}:${issue.code}`)),
+        ...incoming,
+    ];
+}
 function collectNodesByCanonicalId(nodes) {
     const byId = new Map();
     const walk = (items, parentPath) => {
@@ -75,24 +85,22 @@ export function recordIntent(internal, partial) {
     }
     const resolvedEntry = resolveNodeLookupEntry(internal.currentView.nodes, partial.nodeId);
     if (!resolvedEntry) {
-        internal.issues = [
-            ...internal.issues,
-            {
+        internal.issues = dedupeIssues(internal.issues, [{
                 severity: ISSUE_SEVERITY.WARNING,
                 nodeId: partial.nodeId,
                 message: `Node ${partial.nodeId} not found in current view`,
                 code: ISSUE_CODES.UNKNOWN_NODE,
-            },
-        ];
+            }]);
         notifySnapshotAndIssueListeners(internal);
         return;
     }
     const { canonicalId, node } = resolvedEntry;
+    const payload = { ...partial.payload };
     internal.currentData = {
         ...internal.currentData,
         values: {
             ...internal.currentData.values,
-            [canonicalId]: partial.payload,
+            [canonicalId]: payload,
         },
         lineage: {
             ...internal.currentData.lineage,
@@ -108,9 +116,9 @@ export function recordIntent(internal, partial) {
         },
     };
     if (internal.validateOnUpdate) {
-        const validationIssues = validateNodeValue(node, partial.payload);
+        const validationIssues = validateNodeValue(node, payload);
         if (validationIssues.length > 0) {
-            internal.issues = [...internal.issues, ...validationIssues];
+            internal.issues = dedupeIssues(internal.issues, validationIssues);
         }
     }
     const lastAutoCheckpoint = [...internal.checkpoints]

package/lib/session/listeners.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"listeners.d.ts","sourceRoot":"","sources":["../../../../../packages/session/src/lib/session/listeners.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qBAAqB,CAAC;AAC9D,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,oBAAoB,CAAC;AAC9D,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAEvD;;;;;GAKG;AACH,wBAAgB,6BAA6B,CAAC,QAAQ,EAAE,YAAY,GAAG,kBAAkB,GAAG,IAAI,CAG/F;AAED;;;;;;GAMG;AACH,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,YAAY,GAAG,IAAI,CASpE;AAED;;;;;;GAMG;AACH,wBAAgB,oBAAoB,CAAC,QAAQ,EAAE,YAAY,GAAG,IAAI,CAQjE;AAED;;;;GAIG;AACH,wBAAgB,+BAA+B,CAAC,QAAQ,EAAE,YAAY,GAAG,IAAI,CAG5E;AAED;;;;;;GAMG;AACH,wBAAgB,iBAAiB,CAC/B,QAAQ,EAAE,YAAY,EACtB,QAAQ,EAAE,CAAC,QAAQ,EAAE,kBAAkB,GAAG,IAAI,KAAK,IAAI,GACtD,MAAM,IAAI,CAGZ;AAED;;;;;;GAMG;AACH,wBAAgB,eAAe,CAC7B,QAAQ,EAAE,YAAY,EACtB,QAAQ,EAAE,CAAC,MAAM,EAAE,mBAAmB,EAAE,KAAK,IAAI,GAChD,MAAM,IAAI,CAGZ"}
+{"version":3,"file":"listeners.d.ts","sourceRoot":"","sources":["../../../../../packages/session/src/lib/session/listeners.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qBAAqB,CAAC;AAC9D,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,oBAAoB,CAAC;AAC9D,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAEvD;;;;;GAKG;AACH,wBAAgB,6BAA6B,CAAC,QAAQ,EAAE,YAAY,GAAG,kBAAkB,GAAG,IAAI,CAS/F;AAED;;;;;;GAMG;AACH,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,YAAY,GAAG,IAAI,CASpE;AAED;;;;;;GAMG;AACH,wBAAgB,oBAAoB,CAAC,QAAQ,EAAE,YAAY,GAAG,IAAI,CAQjE;AAED;;;;GAIG;AACH,wBAAgB,+BAA+B,CAAC,QAAQ,EAAE,YAAY,GAAG,IAAI,CAG5E;AAED;;;;;;GAMG;AACH,wBAAgB,iBAAiB,CAC/B,QAAQ,EAAE,YAAY,EACtB,QAAQ,EAAE,CAAC,QAAQ,EAAE,kBAAkB,GAAG,IAAI,KAAK,IAAI,GACtD,MAAM,IAAI,CAGZ;AAED;;;;;;GAMG;AACH,wBAAgB,eAAe,CAC7B,QAAQ,EAAE,YAAY,EACtB,QAAQ,EAAE,CAAC,MAAM,EAAE,mBAAmB,EAAE,KAAK,IAAI,GAChD,MAAM,IAAI,CAGZ"}

package/lib/session/listeners.js

@@ -7,7 +7,13 @@
 export function buildSnapshotFromCurrentState(internal) {
     if (!internal.currentView || !internal.currentData)
         return null;
-    return { view: internal.currentView, data: internal.currentData };
+    const snapshot = {
+        view: { ...internal.currentView },
+        data: { ...internal.currentData },
+    };
+    Object.freeze(snapshot.view);
+    Object.freeze(snapshot.data);
+    return Object.freeze(snapshot);
 }
 /**
  * Notifies all snapshot listeners with the latest snapshot value.

package/lib/session/persistence.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"persistence.d.ts","sourceRoot":"","sources":["../../../../../packages/session/src/lib/session/persistence.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAIvD,OAAO,KAAK,EAAE,yBAAyB,EAAE,MAAM,aAAa,CAAC;AAkB7D;;;;;;;;;GASG;AACH,wBAAgB,iBAAiB,CAC/B,QAAQ,EAAE,YAAY,EACtB,OAAO,EAAE,yBAAyB,GACjC,MAAM,IAAI,CAuEZ"}
+{"version":3,"file":"persistence.d.ts","sourceRoot":"","sources":["../../../../../packages/session/src/lib/session/persistence.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAIvD,OAAO,KAAK,EAAE,yBAAyB,EAAE,MAAM,aAAa,CAAC;AAkB7D;;;;;;;;;GASG;AACH,wBAAgB,iBAAiB,CAC/B,QAAQ,EAAE,YAAY,EACtB,OAAO,EAAE,yBAAyB,GACjC,MAAM,IAAI,CAmFZ"}

package/lib/session/persistence.js

@@ -31,36 +31,43 @@ export function attachPersistence(internal, options) {
     let timeout;
     let isApplyingRemote = false;
     const encoder = new TextEncoder();
+    const flushNow = () => {
+        if (timeout) {
+            clearTimeout(timeout);
+            timeout = undefined;
+        }
+        try {
+            const payload = JSON.stringify(serializeSession(internal));
+            const attemptedBytes = encoder.encode(payload).byteLength;
+            if (typeof options.maxBytes === 'number'
+                && Number.isFinite(options.maxBytes)
+                && options.maxBytes >= 0
+                && attemptedBytes > options.maxBytes) {
+                options.onError?.({
+                    reason: 'size_limit',
+                    key,
+                    attemptedBytes,
+                    maxBytes: options.maxBytes,
+                });
+                return;
+            }
+            storage.setItem(key, payload);
+        }
+        catch (cause) {
+            options.onError?.({
+                reason: 'storage_error',
+                key,
+                cause,
+            });
+        }
+    };
     const unsubscribe = subscribeSnapshot(internal, () => {
         if (isApplyingRemote)
             return;
         if (timeout)
             clearTimeout(timeout);
         timeout = setTimeout(() => {
-            try {
-                const payload = JSON.stringify(serializeSession(internal));
-                const attemptedBytes = encoder.encode(payload).byteLength;
-                if (typeof options.maxBytes === 'number'
-                    && Number.isFinite(options.maxBytes)
-                    && options.maxBytes >= 0
-                    && attemptedBytes > options.maxBytes) {
-                    options.onError?.({
-                        reason: 'size_limit',
-                        key,
-                        attemptedBytes,
-                        maxBytes: options.maxBytes,
-                    });
-                    return;
-                }
-                storage.setItem(key, payload);
-            }
-            catch (cause) {
-                options.onError?.({
-                    reason: 'storage_error',
-                    key,
-                    cause,
-                });
-            }
+            flushNow();
         }, 200);
     });
     const onStorage = (event) => {
@@ -75,26 +82,29 @@ export function attachPersistence(internal, options) {
             });
             isApplyingRemote = true;
             replaceInternalState(internal, next);
+            isApplyingRemote = false;
             notifySnapshotAndIssueListeners(internal);
         }
         catch {
-            return;
-        }
-        finally {
             isApplyingRemote = false;
+            return;
         }
     };
     const maybeAdd = globalThis.addEventListener;
     const maybeRemove = globalThis.removeEventListener;
     if (maybeAdd) {
         maybeAdd('storage', onStorage);
+        maybeAdd('beforeunload', flushNow);
     }
     return () => {
-        if (timeout)
+        if (timeout) {
             clearTimeout(timeout);
+            timeout = undefined;
+        }
         unsubscribe();
         if (maybeRemove) {
             maybeRemove('storage', onStorage);
+            maybeRemove('beforeunload', flushNow);
         }
     };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@continuum-dev/session",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "publishConfig": {
     "access": "public"
   },
@@ -40,7 +40,7 @@
     "LICENSE*"
   ],
   "dependencies": {
-    "@continuum-dev/contract": "^0.1.2",
-    "@continuum-dev/runtime": "^0.1.2"
+    "@continuum-dev/contract": "^0.1.3",
+    "@continuum-dev/runtime": "^0.1.3"
   }
 }