@agoric/swing-store 0.9.2-dev-de95355.0 → 0.9.2-dev-b6c58f2.0

package/docs/snapstore.md

@@ -42,7 +42,7 @@ The SnapStore doesn't provide an explicit API to call when a vat is first create
 
 When terminating a vat, the kernel should first call `snapStore.stopUsingLastSnapshot(vatID)`, the same call it would make at the end of an incarnation, to indicate that we're no longer using the last snapshot. This results in zero in-use snapshots.
 
-Then, the kernel must either call `snapStore.deleteVatSnapshots(vatID, undefined)` to delete everything at once, or make a series of calls (spread out over time/blocks) to `snapStore.deleteVatSnapshots(vatID, budget)`. Each will return `{ done, cleanups }`, which can be used to manage the rate-limiting and know when the process is finished.
+Then, the kernel must either call `snapStore.deleteVatSnapshots(vatID)` or `deleteVatSnapshots(vatID, Infinity)` to delete everything at once, or make a series of calls (spread out over time/blocks) to `snapStore.deleteVatSnapshots(vatID, budget)`. Each will return `{ done, cleanups }`, which can be used to manage the rate-limiting and know when the process is finished.
 
 The `stopUsingLastSnapshot()` is a performance improvement, but is not mandatory. If omitted, exports will continue to include the vat's snapshot artifacts until the first call to `deleteVatSnapshots()`, after which they will go away. Snapshots are deleted in descending `snapPos` order, so the first call will delete the only `inUse = 1` snapshot, after which exports will omit all artifacts for the vatID. `stopUsingLastSnapshot()` is idempotent, and extra calls will leave the DB unchanged.
 

package/docs/transcriptstore.md

@@ -63,7 +63,7 @@ Unlike the [SnapStore](./snapstore.md), the TranscriptStore *does* have an expli
 
 When a vat is terminated, the kernel should first call `transcriptStore.stopUsingTranscript(vatID)`. This will mark the single current span as `isCurrent = 0`. The kernel must not attempt to read, add, or rollover spans or items while in this state. While in this state, exports (export for `mode = debug`) will not emit artifacts for this VatID: export-data records will still exist for all spans, as these must be deleted slowly, however there will be no associated artifacts or artifact names.
 
-Then, the kernel should either call `transcriptStore.deleteVatTranscripts(vatID, undefined)` exactly once, or it should call `transcriptStore.deleteVatTranscripts(vatID, budget)` until it returns `{ done: true }`.
+Then, the kernel should either call `transcriptStore.deleteVatTranscripts(vatID)` exactly once, or it should call `transcriptStore.deleteVatTranscripts(vatID, budget)` until it returns `{ done: true }`.
 
 As with snapshots, the `stopUsingTranscript()` is a non-mandatory performance improvement. If omitted, exports will continue to include (many) span artifacts for this vat until the first call to `deleteVatTranscripts()` removes the one `isCurrent = 1` span (since spans are deleted most-recent-first). After that point, exports will stop including any artifacts for the vatID. `stopUsingTranscript()` is idempotent, and extra calls will leave the DB unchanged.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agoric/swing-store",
-  "version": "0.9.2-dev-de95355.0+de95355",
+  "version": "0.9.2-dev-b6c58f2.0+b6c58f2",
   "description": "Persistent storage for SwingSet",
   "type": "module",
   "main": "./src/index.js",
@@ -21,7 +21,7 @@
     "lint:eslint": "eslint ."
   },
   "dependencies": {
-    "@agoric/internal": "0.3.3-dev-de95355.0+de95355",
+    "@agoric/internal": "0.3.3-dev-b6c58f2.0+b6c58f2",
     "@endo/base64": "^1.0.6",
     "@endo/bundle-source": "^3.3.0",
     "@endo/check-bundle": "^1.0.8",
@@ -51,5 +51,5 @@
   "typeCoverage": {
     "atLeast": 76.31
   },
-  "gitHead": "de9535585f4913e792caf662d869d6f041881985"
+  "gitHead": "b6c58f297e8f902f046cc2c71fe7f6162fe0c76d"
 }

package/src/snapStore.js

@@ -393,9 +393,9 @@ export function makeSnapStore(
    * @param {number} [budget]
    * @returns {{ done: boolean, cleanups: number }}
    */
-  function deleteVatSnapshots(vatID, budget = undefined) {
+  function deleteVatSnapshots(vatID, budget = Infinity) {
     ensureTxn();
-    const deleteAll = budget === undefined;
+    const deleteAll = budget === Infinity;
     assert(deleteAll || budget >= 1, 'budget must be undefined or positive');
     // We can't use .iterate because noteExport can write to the DB,
     // and overlapping queries are not supported.

package/src/transcriptStore.js

@@ -394,9 +394,9 @@ export function makeTranscriptStore(
    * @param {number} [budget]
    * @returns {{ done: boolean, cleanups: number }}
    */
-  function deleteVatTranscripts(vatID, budget = undefined) {
+  function deleteVatTranscripts(vatID, budget = Infinity) {
     ensureTxn();
-    const deleteAll = budget === undefined;
+    const deleteAll = budget === Infinity;
     assert(deleteAll || budget >= 1, 'budget must be undefined or positive');
     // We can't use .iterate because noteExport can write to the DB,
     // and overlapping queries are not supported.