Use structuredClone

Author: JaredCE

src/definitionGenerator.js

@@ -624,7 +624,7 @@ class DefinitionGenerator {
           continue;
         }
 
-        const obj = JSON.parse(JSON.stringify(this.DEFAULT_CORS_HEADERS[key]));
+        const obj = structuredClone(this.DEFAULT_CORS_HEADERS[key]);
 
         if (key === "Access-Control-Allow-Origin") {
           if (

src/openAPIGenerator.js

@@ -241,7 +241,7 @@ class OpenAPIGenerator {
     };
 
     PostmanGenerator.convert(
-      { type: "json", data: JSON.parse(JSON.stringify(openAPI)) },
+      { type: "json", data: structuredClone(openAPI) },
       {},
       postmanGeneration
     );

test/unit/definitionGenerator.spec.js

@@ -23,7 +23,7 @@ describe("DefinitionGenerator", () => {
   );
 
   beforeEach(function () {
-    mockServerless = JSON.parse(JSON.stringify(serverlessMock));
+    mockServerless = structuredClone(serverlessMock);
     Object.assign(mockServerless.service.custom.documentation, modelsDocument);
   });
 
@@ -39,9 +39,7 @@ describe("DefinitionGenerator", () => {
     });
 
     it("should default to version 3.0.0 of openAPI when openAPI version is not passed in", function () {
-      const serverlessWithoutOpenAPIVersion = JSON.parse(
-        JSON.stringify(mockServerless)
-      );
+      const serverlessWithoutOpenAPIVersion = structuredClone(mockServerless);
       delete serverlessWithoutOpenAPIVersion.processedInput;
       let expected = new DefinitionGenerator(
         serverlessWithoutOpenAPIVersion,
@@ -108,9 +106,7 @@ describe("DefinitionGenerator", () => {
     });
 
     it("should respect the version of openAPI when passed in", function () {
-      const serverlessWithOpenAPIVersion = JSON.parse(
-        JSON.stringify(mockServerless)
-      );
+      const serverlessWithOpenAPIVersion = structuredClone(mockServerless);
       serverlessWithOpenAPIVersion.processedInput.options.openApiVersion =
         "3.0.2";
       let expected = new DefinitionGenerator(

test/unit/openAPIGenerator.spec.js

@@ -107,9 +107,7 @@ describe("OpenAPIGenerator", () => {
       const getAllFuncsStub = sinon
         .stub(sls.service, "getAllFunctions")
         .returns(["createUser"]);
-      const basicInvalidFunction = JSON.parse(
-        JSON.stringify(basicValidFunction)
-      );
+      const basicInvalidFunction = structuredClone(basicValidFunction);
 
       delete basicInvalidFunction.createUser.events[0].http.documentation
         .methodResponses[0].responseModels;
@@ -145,9 +143,7 @@ describe("OpenAPIGenerator", () => {
       const getAllFuncsStub = sinon
         .stub(sls.service, "getAllFunctions")
         .returns(["createUser"]);
-      const basicInvalidFunction = JSON.parse(
-        JSON.stringify(basicValidFunction)
-      );
+      const basicInvalidFunction = structuredClone(basicValidFunction);
 
       const getFuncStub = sinon
         .stub(sls.service, "getFunction")
@@ -182,9 +178,7 @@ describe("OpenAPIGenerator", () => {
         .stub(sls.service, "getAllFunctions")
         .returns(["createUser"]);
 
-      const basicInvalidFunction = JSON.parse(
-        JSON.stringify(basicValidFunction)
-      );
+      const basicInvalidFunction = structuredClone(basicValidFunction);
 
       delete basicInvalidFunction.createUser.events[0].http.documentation
         .pathParams;

test/unit/owasp.spec.js

@@ -56,7 +56,7 @@ describe(`owasp`, function () {
     });
 
     it(`adds any properties contained in a new release`, async function () {
-      const newOWASPJSONAdded = JSON.parse(JSON.stringify(newOWASPJSON));
+      const newOWASPJSONAdded = structuredClone(newOWASPJSON);
       newOWASPJSONAdded.headers.push({ name: "x-added", value: "true" });
 
       nock("https://owasp.org")

test/unit/schemaHandler.spec.js

@@ -43,12 +43,12 @@ describe(`SchemaHandler`, function () {
   };
 
   beforeEach(function () {
-    mockServerless = JSON.parse(JSON.stringify(serverlessMock));
-    modelsDocument = JSON.parse(JSON.stringify(modelsDocumentOG));
-    modelsAltDocument = JSON.parse(JSON.stringify(modelsAltDocumentOG));
-    modelsListDocument = JSON.parse(JSON.stringify(modelsListDocumentOG));
-    modelsListAltDocument = JSON.parse(JSON.stringify(modelsListAltDocumentOG));
-    openAPI = JSON.parse(JSON.stringify(openAPISchema));
+    mockServerless = structuredClone(serverlessMock);
+    modelsDocument = structuredClone(modelsDocumentOG);
+    modelsAltDocument = structuredClone(modelsAltDocumentOG);
+    modelsListDocument = structuredClone(modelsListDocumentOG);
+    modelsListAltDocument = structuredClone(modelsListAltDocumentOG);
+    openAPI = structuredClone(openAPISchema);
   });
 
   describe(`constuctor`, function () {
@@ -135,7 +135,7 @@ describe(`SchemaHandler`, function () {
       });
 
       it(`should standardise mixed models syntax in to the correct format`, function () {
-        const newModelsDocument = JSON.parse(JSON.stringify(modelsDocument));
+        const newModelsDocument = structuredClone(modelsDocument);
         Object.assign(
           mockServerless.service.custom.documentation,
           newModelsDocument
@@ -168,9 +168,7 @@ describe(`SchemaHandler`, function () {
       });
 
       it(`should standardise mixed modelsList syntax in to the correct format`, function () {
-        const newModelsDocument = JSON.parse(
-          JSON.stringify(modelsListDocument)
-        );
+        const newModelsDocument = structuredClone(modelsListDocument);
         Object.assign(
           mockServerless.service.custom.documentation,
           newModelsDocument
@@ -203,9 +201,7 @@ describe(`SchemaHandler`, function () {
       });
 
       it(`should standardise mixed models and modelsList syntax in to the correct format`, function () {
-        const newModelsDocument = JSON.parse(
-          JSON.stringify(modelsListDocument)
-        );
+        const newModelsDocument = structuredClone(modelsListDocument);
         Object.assign(
           mockServerless.service.custom.documentation,
           newModelsDocument