Merge pull request #1885 from contentstack/fix/DX-2894

Fix: Added check for removing field

Author: cs-raj

packages/contentstack-audit/src/config/index.ts

@@ -13,6 +13,7 @@ const config = {
     'field-rules',
   ],
   'fix-fields': ['reference', 'global_field', 'json:rte', 'json:extension', 'blocks', 'group', 'content_types'],
+  'schema-fields-data-type': ['blocks','group','global_field'],
   moduleConfig: {
     'content-types': {
       name: 'content type',

packages/contentstack-audit/src/modules/content-types.ts

@@ -185,7 +185,6 @@ export default class ContentType {
       field.schema = this.runFixOnSchema(tree, field.schema as ContentTypeSchemaType[]);
     }
     for (let child of field.schema ?? []) {
-      
       if (!fixTypes.includes(child.data_type) && child.data_type !== 'json') continue;
 
       switch (child.data_type) {
@@ -487,8 +486,13 @@ export default class ContentType {
       })
       .filter((val: any) => {
         if (this.config.skipFieldTypes.includes(val?.data_type)) return true;
-        if (val?.schema && isEmpty(val?.schema)) return false;
-        if (val?.reference_to && isEmpty(val?.reference_to) && val.data_type==='reference') return false;
+        if (
+          val?.schema &&
+          isEmpty(val?.schema) &&
+          (!val?.data_type || this.config['schema-fields-data-type'].includes(val.data_type))
+        )
+          return false;
+        if (val?.reference_to && isEmpty(val?.reference_to) && val.data_type === 'reference') return false;
 
         return !!val;
       }) as ContentTypeSchemaType[];