chore: manifest.json should have stable json

packages/@cdktf/cli-core/src/test/lib/cdktf-project.test.ts

@@ -161,7 +161,7 @@ describe("CdktfProject", () => {
       });
 
       return expect(cdktfProject.diff()).rejects.toMatchInlineSnapshot(
-        `[Error: Usage Error: Found more than one stack, please specify a target stack. Run cdktf diff <stack> with one of these stacks: first, second, third, fourth, fifth ]`
+        `[Error: Usage Error: Found more than one stack, please specify a target stack. Run cdktf diff <stack> with one of these stacks: fifth, first, fourth, second, third ]`
       );
     });
   });

packages/cdktf/lib/manifest.ts

@@ -4,6 +4,7 @@ import * as path from "path";
 import * as fs from "fs";
 import { TerraformStack } from "./terraform-stack";
 import { AnnotationMetadataEntryType } from "./annotations";
+import stringify = require("json-stable-stringify");
 
 export interface StackAnnotation {
   readonly constructPath: string;
@@ -87,7 +88,7 @@ export class Manifest implements IManifest {
   public writeToFile() {
     fs.writeFileSync(
       path.join(this.outdir, Manifest.fileName),
-      JSON.stringify(this.buildManifest(), undefined, 2)
+      stringify(this.buildManifest(), { space: 2 })
     );
   }
 }

packages/cdktf/test/manifest.test.ts

@@ -53,18 +53,20 @@ test("write manifest", () => {
   expect(fs.readFileSync(path.join(outdir, Manifest.fileName)).toString())
     .toMatchInlineSnapshot(`
     "{
-      "version": "0.0.0",
       "stacks": {
         "this-is-a-stack": {
-          "name": "this-is-a-stack",
+          "annotations": [
+          ],
           "constructPath": "this-is-a-stack",
-          "workingDirectory": "stacks/this-is-a-stack",
-          "synthesizedStackPath": "stacks/this-is-a-stack/cdk.tf.json",
+          "dependencies": [
+          ],
+          "name": "this-is-a-stack",
           "stackMetadataPath": "stacks/this-is-a-stack/metadata.json",
-          "annotations": [],
-          "dependencies": []
+          "synthesizedStackPath": "stacks/this-is-a-stack/cdk.tf.json",
+          "workingDirectory": "stacks/this-is-a-stack"
         }
-      }
+      },
+      "version": "0.0.0"
     }"
   `);
 });
@@ -88,14 +90,8 @@ describe("manifest annotations", () => {
     expect(fs.readFileSync(path.join(outdir, Manifest.fileName)).toString())
       .toMatchInlineSnapshot(`
       "{
-        "version": "stubbed",
         "stacks": {
           "this-is-a-stack": {
-            "name": "this-is-a-stack",
-            "constructPath": "this-is-a-stack",
-            "workingDirectory": "stacks/this-is-a-stack",
-            "synthesizedStackPath": "stacks/this-is-a-stack/cdk.tf.json",
-            "stackMetadataPath": "stacks/this-is-a-stack/metadata.json",
             "annotations": [
               {
                 "constructPath": "this-is-a-stack",
@@ -113,9 +109,16 @@ describe("manifest annotations", () => {
                 "message": "an error"
               }
             ],
-            "dependencies": []
+            "constructPath": "this-is-a-stack",
+            "dependencies": [
+            ],
+            "name": "this-is-a-stack",
+            "stackMetadataPath": "stacks/this-is-a-stack/metadata.json",
+            "synthesizedStackPath": "stacks/this-is-a-stack/cdk.tf.json",
+            "workingDirectory": "stacks/this-is-a-stack"
           }
-        }
+        },
+        "version": "stubbed"
       }"
     `);
   });