@backloghq/opslog 0.1.3 → 0.1.4

package/dist/store.js

@@ -257,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,29 +28,33 @@ export function validateManifest(raw) {
         throw new Error("Invalid manifest: not an object");
     }
     const obj = raw;
-    if (typeof obj.version !== "number" || !Number.isInteger(obj.version) || obj.version < 1) {
-        throw new Error("Invalid manifest: version must be a positive integer");
+    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")
+    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) {
@@ -47,11 +62,11 @@ export function validateSnapshot(raw) {
         throw new Error("Invalid snapshot: not an object");
     }
     const obj = raw;
-    if (typeof obj.version !== "number" || !Number.isInteger(obj.version) || obj.version < 1) {
-        throw new Error("Invalid snapshot: version must be a positive integer");
+    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")
-        throw new Error("Invalid snapshot: missing timestamp");
+    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");
     }
@@ -62,13 +77,13 @@ export function validateArchiveSegment(raw) {
         throw new Error("Invalid archive segment: not an object");
     }
     const obj = raw;
-    if (typeof obj.version !== "number" || !Number.isInteger(obj.version) || obj.version < 1) {
-        throw new Error("Invalid archive segment: version must be a positive integer");
+    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")
-        throw new Error("Invalid archive segment: missing period");
-    if (typeof obj.timestamp !== "string")
-        throw new Error("Invalid archive segment: missing timestamp");
+    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.3",
+  "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",