fixed bug in summary and deleted mutation object summary

Author: Iurii Golovinskii

src/apitypes/rest/rest.changes.ts

@@ -144,7 +144,7 @@ async function reclassifyBreakingChanges(
   }
 
 
-  // mark removed required status of the property as semi-breaking
+  // mark removed required status of the property as risky
   if (diffs.length) {
     const requiredProperties = findRequiredRemovedProperties(mergedJso, diffs)
     //console.log('requiredProperties----->', requiredProperties)

src/utils/builder.ts

@@ -22,7 +22,6 @@ import {
   DEPRECATED_CHANGE_TYPE,
   ImpactedOperationSummary,
   NON_BREAKING_CHANGE_TYPE,
-  SEMI_BREAKING_CHANGE_TYPE,
   UNCLASSIFIED_CHANGE_TYPE,
 } from '../types'
 import { API_KIND } from '../consts'

src/utils/transformToDto.ts

@@ -197,13 +197,16 @@ export function replacePropertyInChangesSummary<
   if (Object.prototype.hasOwnProperty.call(obj, origin)) {
     // eslint-disable-next-line @typescript-eslint/ban-ts-comment
     // @ts-ignore
-    obj[override] = obj[origin]
+    const copyObj= {...obj}
     // eslint-disable-next-line @typescript-eslint/ban-ts-comment
     // @ts-ignore
-    delete obj[origin]
+    copyObj[override] = copyObj[origin]
     // eslint-disable-next-line @typescript-eslint/ban-ts-comment
     // @ts-ignore
-    return obj
+    delete copyObj[origin]
+    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+    // @ts-ignore
+    return copyObj
   } else {
     // eslint-disable-next-line @typescript-eslint/ban-ts-comment
     // @ts-ignore

test/helpers/registry/local.ts

@@ -23,7 +23,7 @@ import {
   BuilderResolvers,
   BuildResult,
   ChangeSummary,
-  EMPTY_CHANGE_SUMMARY, EMPTY_CHANGE_SUMMARY_DTO,
+  EMPTY_CHANGE_SUMMARY,
   FILE_FORMAT,
   GRAPHQL_API_TYPE,
   graphqlApiBuilder, MESSAGE_SEVERITY,