@crewhaus/migration-engine 0.1.1 → 0.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@crewhaus/migration-engine",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "type": "module",
   "description": "IR version migrations: register up/down chains; migrate(spec, fromVersion, toVersion) walks the chain",
   "main": "src/index.ts",
@@ -12,13 +12,13 @@
     "test": "bun test src"
   },
   "dependencies": {
-    "@crewhaus/errors": "0.1.1"
+    "@crewhaus/errors": "0.1.2"
   },
   "license": "Apache-2.0",
   "author": {
     "name": "Max Meier",
-    "email": "max@studiomax.io",
-    "url": "https://studiomax.io"
+    "email": "max@crewhaus.ai",
+    "url": "https://crewhaus.ai"
   },
   "repository": {
     "type": "git",
@@ -30,12 +30,7 @@
     "url": "https://github.com/crewhaus/factory/issues"
   },
   "publishConfig": {
-    "access": "restricted"
+    "access": "public"
   },
-  "files": [
-    "src",
-    "README.md",
-    "LICENSE",
-    "NOTICE"
-  ]
+  "files": ["src", "README.md", "LICENSE", "NOTICE"]
 }

package/src/index.test.ts

@@ -115,4 +115,46 @@ describe("migration-engine — T4 multi-step chain replay", () => {
     });
     expect(e.list()).toEqual(["0→1", "1→2"]);
   });
+
+  test("clear empties the registry", () => {
+    const e = createDefaultEngine();
+    expect(e.list()).toEqual(["0→1"]);
+    e.clear();
+    expect(e.list()).toEqual([]);
+    // After clearing, the previously-registered step is gone.
+    expect(() => e.migrate({ version: 0 }, 1)).toThrow(MigrationError);
+  });
+});
+
+describe("migration-engine — additional coverage", () => {
+  test("NOOP_0_TO_1.up stamps version 1 and preserves other keys", () => {
+    const out = NOOP_0_TO_1.up({ name: "x", version: 0 });
+    expect(out).toEqual({ name: "x", version: 1 });
+  });
+
+  test("NOOP_0_TO_1.down stamps version 0 and preserves other keys", () => {
+    const out = NOOP_0_TO_1.down({ name: "x", version: 1 });
+    expect(out).toEqual({ name: "x", version: 0 });
+  });
+
+  test("migrate treats a spec with no version field as version 0", () => {
+    const e = createDefaultEngine();
+    // No `version` key at all -> fromVersion defaults to 0, so 0 -> 1 runs.
+    const out = e.migrate({ name: "no-version" }, 1);
+    expect(out.version).toBe(1);
+  });
+
+  test("downgrade throws when the needed step is not registered", () => {
+    const e = new MigrationEngine();
+    // Walking down from 1 to 0 needs the 0→1 step's down(); none registered.
+    expect(() => e.migrate({ version: 1 }, 0)).toThrow(MigrationError);
+  });
+
+  test("MigrationError carries the config code and an optional cause", () => {
+    const cause = new Error("root");
+    const err = new MigrationError("boom", cause);
+    expect(err).toBeInstanceOf(MigrationError);
+    expect(err.name).toBe("MigrationError");
+    expect(err.cause).toBe(cause);
+  });
 });

package/src/index.ts

@@ -33,7 +33,11 @@ export class MigrationError extends CrewhausError {
 }
 
 export class MigrationEngine {
-  private readonly migrations = new Map<string, Migration>();
+  private readonly migrations: Map<string, Migration>;
+
+  constructor() {
+    this.migrations = new Map<string, Migration>();
+  }
 
   register(m: Migration): void {
     if (m.from === m.to) {