From 74848f4882e4b185367f34cb1a99e0407286dc9e Mon Sep 17 00:00:00 2001 From: krisbitney Date: Fri, 14 Apr 2023 09:36:35 +0000 Subject: [PATCH 1/2] removed top-level docker property during build manifest migration --- .../src/formats/polywrap.build/migrators/0.1.0_to_0.2.0.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/packages/js/manifests/polywrap/src/formats/polywrap.build/migrators/0.1.0_to_0.2.0.ts b/packages/js/manifests/polywrap/src/formats/polywrap.build/migrators/0.1.0_to_0.2.0.ts index f076051a3d..070eb39853 100644 --- a/packages/js/manifests/polywrap/src/formats/polywrap.build/migrators/0.1.0_to_0.2.0.ts +++ b/packages/js/manifests/polywrap/src/formats/polywrap.build/migrators/0.1.0_to_0.2.0.ts @@ -5,7 +5,8 @@ import { BuildManifest as NewManifest } from "../0.2.0"; export function migrate(old: OldManifest): NewManifest { return { - ...old, + config: old.config, + linked_packages: old.linked_packages, __type: "BuildManifest", format: "0.2.0", strategies: { From 49390a273f0db7cfea7f5a56a9773080ae477b88 Mon Sep 17 00:00:00 2001 From: krisbitney Date: Sat, 15 Apr 2023 11:19:18 +0000 Subject: [PATCH 2/2] removed possibility of empty object assigned to extension property in project manifest migrator 0.2.0_to_0.3.0.ts --- .../src/formats/polywrap/migrators/0.2.0_to_0.3.0.ts | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/packages/js/manifests/polywrap/src/formats/polywrap/migrators/0.2.0_to_0.3.0.ts b/packages/js/manifests/polywrap/src/formats/polywrap/migrators/0.2.0_to_0.3.0.ts index 5c9398f8b5..f3f862628e 100644 --- a/packages/js/manifests/polywrap/src/formats/polywrap/migrators/0.2.0_to_0.3.0.ts +++ b/packages/js/manifests/polywrap/src/formats/polywrap/migrators/0.2.0_to_0.3.0.ts @@ -1,7 +1,8 @@ -import { ILogger } from "@polywrap/logging-js"; import { PolywrapManifest as OldManifest } from "../0.2.0"; import { PolywrapManifest as NewManifest } from "../0.3.0"; +import { ILogger } from "@polywrap/logging-js"; + export function migrate(migrate: OldManifest, logger?: ILogger): NewManifest { if (migrate.extensions?.meta) { logger?.warn( @@ -21,8 +22,11 @@ export function migrate(migrate: OldManifest, logger?: ILogger): NewManifest { ); delete migrate.extensions.infra; } + const hasExtensions = + migrate.extensions && Object.keys(migrate.extensions).length > 0; return { ...migrate, format: "0.3.0", + extensions: hasExtensions ? migrate.extensions : undefined, }; }