use helper functions in vatKeeper

Agoric · Apr 27, 2023 · 0c2e6ec · 0c2e6ec
1 parent 7cd9127
commit 0c2e6ec
Showing 1 changed file with 27 additions and 11 deletions.
diff --git a/packages/SwingSet/src/kernel/state/vatKeeper.js b/packages/SwingSet/src/kernel/state/vatKeeper.js
@@ -516,6 +516,31 @@ export function makeVatKeeper(
     return { totalEntries, snapshottedEntries };
   }
 
+  /**
+   * @param {string} snapshotID
+   * @returns {TranscriptEntry}
+   */
+  function makeSaveSnapshotItem(snapshotID) {
+    return {
+      d: /** @type {TDSaveSnapshot} */ ['save-snapshot'],
+      sc: [],
+      r: { status: 'ok', snapshotID },
+    };
+  }
+
+  /**
+   * @param {string} snapshotID
+   * @returns {TranscriptEntry}
+   */
+  function makeLoadSnapshotItem(snapshotID) {
+    const loadConfig = { snapshotID };
+    return {
+      d: /** @type {TDLoadSnapshot} */ ['load-snapshot', loadConfig],
+      sc: [],
+      r: { status: 'ok' },
+    };
+  }
+
   /**
    * Store a snapshot, if given a snapStore.
    *
@@ -540,24 +565,15 @@ export function makeVatKeeper(
     } = info;
 
     // push a save-snapshot transcript entry
-    addToTranscript({
-      d: /** @type {TDSaveSnapshot} */ ['save-snapshot'],
-      sc: [],
-      r: { status: 'ok', snapshotID },
-    });
+    addToTranscript(makeSaveSnapshotItem(snapshotID));
 
     // then start a new transcript span
     transcriptStore.rolloverSpan(vatID);
 
     // then push a load-snapshot entry, so that the current span
     // always starts with an initialize-worker or load-snapshot
     // pseudo-delivery
-    const loadConfig = { snapshotID };
-    addToTranscript({
-      d: /** @type {TDLoadSnapshot} */ ['load-snapshot', loadConfig],
-      sc: [],
-      r: { status: 'ok' },
-    });
+    addToTranscript(makeLoadSnapshotItem(snapshotID));
 
     kernelSlog.write({
       type: 'heap-snapshot-save',