fix: re-enable forceQuotes while also fixing systemVars behaviour

diplodoc-platform · Sep 18, 2024 · 2c4a8b3 · 2c4a8b3
1 parent d3c80c2
commit 2c4a8b3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/services/metadata/mergeMetadata.ts b/src/services/metadata/mergeMetadata.ts
@@ -24,9 +24,7 @@ export const mergeMetadata = ({
     // That way js-yaml wouldn't include it in the serialized YAML
     // However, that way, this would overwrite (delete) existing properties, e.g.: sourcePath
     // Because of this, we spread objects to create properties if necessary
-    const systemVarsMetadataToSpread = isObject(systemVars)
-        ? {__system: JSON.stringify(systemVars)}
-        : undefined;
+    const systemVarsMetadataToSpread = isObject(systemVars) ? {__system: systemVars} : undefined;
     const innerMetadataToSpread =
         mergedInnerMetadata.length > 0 ? {metadata: mergedInnerMetadata} : undefined;
 

diff --git a/src/services/metadata/parse.ts b/src/services/metadata/parse.ts
@@ -103,7 +103,7 @@ export const parseExistingMetadata = (
 };
 
 export const serializeMetadata = (objectMetadata: FileMetadata) => {
-    const dumped = unescapeLiquidSubstitutionSyntax(dump(objectMetadata).trimEnd());
+    const dumped = unescapeLiquidSubstitutionSyntax(dump(objectMetadata, {forceQuotes: true}));
 
     // This empty object check is a bit naive
     // The other option would be to check if all own fields are `undefined`,