@backloghq/opslog 0.1.1 → 0.1.3

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/archive.js

@@ -11,8 +11,10 @@ export async function writeArchiveSegment(dir, period, records) {
         const parsed = validateArchiveSegment(JSON.parse(content));
         existing = parsed.records;
     }
-    catch {
-        // First write to this period
+    catch (err) {
+        const isNotFound = err instanceof Error && "code" in err && err.code === "ENOENT";
+        if (!isNotFound)
+            throw err;
     }
     const merged = { ...existing, ...Object.fromEntries(records) };
     const segment = {

package/dist/manifest.js

@@ -3,13 +3,14 @@ import { join } from "node:path";
 import { validateManifest } from "./validate.js";
 const MANIFEST_FILE = "manifest.json";
 export async function readManifest(dir) {
+    let content;
     try {
-        const content = await readFile(join(dir, MANIFEST_FILE), "utf-8");
-        return validateManifest(JSON.parse(content));
+        content = await readFile(join(dir, MANIFEST_FILE), "utf-8");
     }
     catch {
-        return null;
+        return null; // File not found — fresh store
     }
+    return validateManifest(JSON.parse(content));
 }
 export async function writeManifest(dir, manifest) {
     const path = join(dir, MANIFEST_FILE);

package/dist/store.js

@@ -46,13 +46,24 @@ 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;
             this.created = manifest.stats.created;
             this.archiveSegments = manifest.archiveSegments;
-            this.archivedRecordCount = manifest.stats?.archivedRecords ?? 0;
+            this.archivedRecordCount = manifest.stats.archivedRecords;
             // Migrate if needed
             if (storedVersion < this.options.version) {
                 for (const [id, record] of this.records) {
@@ -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);

package/dist/validate.js

@@ -17,12 +17,29 @@ 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.version !== "number" || !Number.isInteger(obj.version) || obj.version < 1) {
+        throw new Error("Invalid manifest: version must be a positive integer");
+    }
     if (typeof obj.currentSnapshot !== "string")
         throw new Error("Invalid manifest: missing currentSnapshot");
     if (typeof obj.activeOps !== "string")
         throw new Error("Invalid manifest: missing activeOps");
+    if (!Array.isArray(obj.archiveSegments))
+        throw new Error("Invalid manifest: archiveSegments must be an array");
+    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");
     return raw;
 }
 export function validateSnapshot(raw) {
@@ -30,8 +47,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.isInteger(obj.version) || obj.version < 1) {
+        throw new Error("Invalid snapshot: version must be a positive integer");
+    }
+    if (typeof obj.timestamp !== "string")
+        throw new Error("Invalid snapshot: missing timestamp");
     if (typeof obj.records !== "object" || obj.records === null || Array.isArray(obj.records)) {
         throw new Error("Invalid snapshot: records must be an object");
     }
@@ -42,10 +62,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.version !== "number" || !Number.isInteger(obj.version) || obj.version < 1) {
+        throw new Error("Invalid archive segment: version must be a positive 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.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.1",
+  "version": "0.1.3",
   "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",