@backloghq/opslog 0.1.2 → 0.1.4

package/README.md

@@ -103,6 +103,8 @@ await store.batch(() => {         // Multiple ops, single disk write
 });
 ```
 
+Empty batches (no `set`/`delete` calls) are no-ops — no I/O is performed.
+
 ### History
 
 ```typescript

package/dist/store.js

@@ -46,7 +46,18 @@ export class Store {
         }
         else {
             // Load existing state
-            const { records, version: storedVersion } = await loadSnapshot(dir, manifest.currentSnapshot);
+            let snapshotData;
+            try {
+                snapshotData = await loadSnapshot(dir, manifest.currentSnapshot);
+            }
+            catch (err) {
+                const isNotFound = err instanceof Error && "code" in err && err.code === "ENOENT";
+                if (isNotFound) {
+                    throw new Error(`Snapshot file not found: ${manifest.currentSnapshot}. The data directory may be corrupted.`, { cause: err });
+                }
+                throw err;
+            }
+            const { records, version: storedVersion } = snapshotData;
             this.records = records;
             this.version = storedVersion;
             this.activeOpsPath = manifest.activeOps;
@@ -154,6 +165,7 @@ export class Store {
         this.batchOps = [];
         try {
             fn();
+            // Empty batches are no-ops — no I/O if fn() didn't call set/delete
             if (this.batchOps.length > 0) {
                 await appendOps(join(this.dir, this.activeOpsPath), this.batchOps);
                 this.ops.push(...this.batchOps);
@@ -245,7 +257,7 @@ export class Store {
     }
     async loadArchive(segment) {
         this.ensureOpen();
-        const segmentPath = this.archiveSegments.find((s) => s.includes(segment));
+        const segmentPath = this.archiveSegments.find((s) => s === `archive/archive-${segment}.json`) || this.archiveSegments.find((s) => s.includes(segment));
         if (!segmentPath)
             throw new Error(`Archive segment '${segment}' not found`);
         return loadArchiveSegment(this.dir, segmentPath);

package/dist/validate.js

@@ -3,13 +3,24 @@ export function validateOp(raw) {
         throw new Error("Invalid operation: not an object");
     }
     const obj = raw;
-    if (typeof obj.ts !== "string")
-        throw new Error("Invalid operation: missing ts");
-    if (obj.op !== "set" && obj.op !== "delete") {
-        throw new Error(`Invalid operation: unknown op "${obj.op}"`);
+    if (typeof obj.ts !== "string" || obj.ts.length === 0)
+        throw new Error("Invalid operation: ts must be a non-empty string");
+    if (typeof obj.op !== "string" || (obj.op !== "set" && obj.op !== "delete")) {
+        throw new Error(`Invalid operation: op must be "set" or "delete", got "${obj.op}"`);
     }
-    if (typeof obj.id !== "string")
-        throw new Error("Invalid operation: missing id");
+    if (typeof obj.id !== "string" || obj.id.length === 0)
+        throw new Error("Invalid operation: id must be a non-empty string");
+    if (!("prev" in obj))
+        throw new Error("Invalid operation: missing prev");
+    if (obj.prev !== null && (typeof obj.prev !== "object" || Array.isArray(obj.prev))) {
+        throw new Error("Invalid operation: prev must be an object or null");
+    }
+    if (obj.op === "set" && (!("data" in obj) || obj.data === null))
+        throw new Error("Invalid operation: set op must have non-null data");
+    if (obj.op === "delete" && obj.prev === null)
+        throw new Error("Invalid operation: delete op must have non-null prev");
+    if (obj.op === "delete" && "data" in obj)
+        throw new Error("Invalid operation: delete op must not have data field");
     return raw;
 }
 export function validateManifest(raw) {
@@ -17,28 +28,33 @@ export function validateManifest(raw) {
         throw new Error("Invalid manifest: not an object");
     }
     const obj = raw;
-    if (typeof obj.version !== "number")
-        throw new Error("Invalid manifest: missing version");
-    if (typeof obj.currentSnapshot !== "string")
+    if (typeof obj.version !== "number" || !Number.isFinite(obj.version) || !Number.isInteger(obj.version) || obj.version < 1) {
+        throw new Error("Invalid manifest: version must be a positive finite integer");
+    }
+    if (typeof obj.currentSnapshot !== "string" || obj.currentSnapshot.length === 0)
         throw new Error("Invalid manifest: missing currentSnapshot");
-    if (typeof obj.activeOps !== "string")
+    if (typeof obj.activeOps !== "string" || obj.activeOps.length === 0)
         throw new Error("Invalid manifest: missing activeOps");
     if (!Array.isArray(obj.archiveSegments))
         throw new Error("Invalid manifest: archiveSegments must be an array");
+    for (const seg of obj.archiveSegments) {
+        if (typeof seg !== "string" || seg.length === 0)
+            throw new Error("Invalid manifest: archiveSegments entries must be non-empty strings");
+    }
     if (typeof obj.stats !== "object" || obj.stats === null || Array.isArray(obj.stats)) {
         throw new Error("Invalid manifest: missing stats");
     }
     const stats = obj.stats;
-    if (typeof stats.activeRecords !== "number")
-        throw new Error("Invalid manifest: stats.activeRecords must be a number");
-    if (typeof stats.archivedRecords !== "number")
-        throw new Error("Invalid manifest: stats.archivedRecords must be a number");
-    if (typeof stats.opsCount !== "number")
-        throw new Error("Invalid manifest: stats.opsCount must be a number");
-    if (typeof stats.created !== "string")
-        throw new Error("Invalid manifest: stats.created must be a string");
-    if (typeof stats.lastCheckpoint !== "string")
-        throw new Error("Invalid manifest: stats.lastCheckpoint must be a string");
+    if (typeof stats.activeRecords !== "number" || !Number.isFinite(stats.activeRecords) || !Number.isInteger(stats.activeRecords) || stats.activeRecords < 0)
+        throw new Error("Invalid manifest: stats.activeRecords must be a non-negative integer");
+    if (typeof stats.archivedRecords !== "number" || !Number.isFinite(stats.archivedRecords) || !Number.isInteger(stats.archivedRecords) || stats.archivedRecords < 0)
+        throw new Error("Invalid manifest: stats.archivedRecords must be a non-negative integer");
+    if (typeof stats.opsCount !== "number" || !Number.isFinite(stats.opsCount) || !Number.isInteger(stats.opsCount) || stats.opsCount < 0)
+        throw new Error("Invalid manifest: stats.opsCount must be a non-negative integer");
+    if (typeof stats.created !== "string" || stats.created.length === 0)
+        throw new Error("Invalid manifest: stats.created must be a non-empty string");
+    if (typeof stats.lastCheckpoint !== "string" || stats.lastCheckpoint.length === 0)
+        throw new Error("Invalid manifest: stats.lastCheckpoint must be a non-empty string");
     return raw;
 }
 export function validateSnapshot(raw) {
@@ -46,8 +62,11 @@ export function validateSnapshot(raw) {
         throw new Error("Invalid snapshot: not an object");
     }
     const obj = raw;
-    if (typeof obj.version !== "number")
-        throw new Error("Invalid snapshot: missing version");
+    if (typeof obj.version !== "number" || !Number.isFinite(obj.version) || !Number.isInteger(obj.version) || obj.version < 1) {
+        throw new Error("Invalid snapshot: version must be a positive finite integer");
+    }
+    if (typeof obj.timestamp !== "string" || obj.timestamp.length === 0)
+        throw new Error("Invalid snapshot: timestamp must be a non-empty string");
     if (typeof obj.records !== "object" || obj.records === null || Array.isArray(obj.records)) {
         throw new Error("Invalid snapshot: records must be an object");
     }
@@ -58,12 +77,13 @@ export function validateArchiveSegment(raw) {
         throw new Error("Invalid archive segment: not an object");
     }
     const obj = raw;
-    if (typeof obj.version !== "number")
-        throw new Error("Invalid archive segment: missing version");
-    if (typeof obj.period !== "string")
-        throw new Error("Invalid archive segment: missing period");
-    if (typeof obj.timestamp !== "string")
-        throw new Error("Invalid archive segment: missing timestamp");
+    if (typeof obj.version !== "number" || !Number.isFinite(obj.version) || !Number.isInteger(obj.version) || obj.version < 1) {
+        throw new Error("Invalid archive segment: version must be a positive finite integer");
+    }
+    if (typeof obj.period !== "string" || obj.period.length === 0)
+        throw new Error("Invalid archive segment: period must be a non-empty string");
+    if (typeof obj.timestamp !== "string" || obj.timestamp.length === 0)
+        throw new Error("Invalid archive segment: timestamp must be a non-empty string");
     if (typeof obj.records !== "object" || obj.records === null || Array.isArray(obj.records)) {
         throw new Error("Invalid archive segment: records must be an object");
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@backloghq/opslog",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "description": "Embedded event-sourced document store. Append-only operation log with immutable snapshots, zero native dependencies.",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",