restrict Migrator migrate function to manifest types

polywrap · Sep 30, 2022 · 49b29ca · 49b29ca
1 parent 373e84b
commit 49b29ca
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 6 deletions.
diff --git a/packages/js/manifests/polywrap/src/__tests__/migrations.spec.ts b/packages/js/manifests/polywrap/src/__tests__/migrations.spec.ts
@@ -8,27 +8,27 @@ describe("Manifest migration pathfinding", () => {
     {
       from: "0.1.0",
       to: "0.2.0",
-      migrate: () => {},
+      migrate: (x) => x,
     },
     {
       from: "0.2.0",
       to: "0.3.0",
-      migrate: () => {},
+      migrate: (x) => x,
     },
     {
       from: "0.3.0",
       to: "0.4.0",
-      migrate: () => {},
+      migrate: (x) => x,
     },
     {
       from: "0.4.0",
       to: "0.5.0",
-      migrate: () => {},
+      migrate: (x) => x,
     },
     {
       from: "0.2.0",
       to: "0.4.0",
-      migrate: () => {},
+      migrate: (x) => x,
     },
   ];
 

diff --git a/packages/js/manifests/polywrap/src/migrations/migration.ts b/packages/js/manifests/polywrap/src/migrations/migration.ts
@@ -1,5 +1,26 @@
+import {
+  AnyAppManifest,
+  AnyBuildManifest,
+  AnyDeployManifest,
+  AnyInfraManifest,
+  AnyMetaManifest,
+  AnyPluginManifest,
+  AnyPolywrapManifest,
+  AnyPolywrapWorkflow,
+} from "../formats";
+
+type AnyManifest =
+  | AnyPolywrapManifest
+  | AnyPluginManifest
+  | AnyAppManifest
+  | AnyInfraManifest
+  | AnyDeployManifest
+  | AnyBuildManifest
+  | AnyMetaManifest
+  | AnyPolywrapWorkflow;
+
 export type Migrator = {
   from: string;
   to: string;
-  migrate: (manifest: unknown) => unknown;
+  migrate: (manifest: AnyManifest) => AnyManifest;
 };