feat: Added feature to remove un-necessary default values from imports

Author: kevinwang5658

package.json

@@ -1,6 +1,6 @@
 {
   "name": "codify-plugin-lib",
-  "version": "1.0.141",
+  "version": "1.0.142",
   "description": "Library plugin library",
   "main": "dist/index.js",
   "typings": "dist/index.d.ts",

src/resource/resource-controller.test.ts

@@ -711,4 +711,39 @@ describe('Resource tests', () => {
       }
     })
   })
+
+  it('Applies removes default values if they remain default for imports', async () => {
+    const resource = new class extends TestResource {
+      getSettings(): ResourceSettings<TestConfig> {
+        return {
+          id: 'resourceType',
+          parameterSettings: {
+            propA: { type: 'string', default: 'defaultValue' },
+            propB: { type: 'boolean', default: true }
+          },
+        }
+      }
+
+      async refresh(parameters: Partial<TestConfig>): Promise<Partial<TestConfig> | null> {
+        return {
+          propA: 'defaultValue',
+          propB: false,
+          propC: 'newPropC'
+        }
+      }
+    }
+
+    const controller = new ResourceController(resource);
+    const plan = await controller.import({ type: 'resourceType' }, {});
+
+    expect(plan![0]).toMatchObject({
+      'core': {
+        'type': 'resourceType'
+      },
+      'parameters': {
+        propB: false,
+        propC: 'newPropC'
+      }
+    })
+  })
 });

src/resource/resource-controller.ts

@@ -261,6 +261,8 @@ export class ResourceController<T extends StringIndexedObject> {
     const resultParameters = { ...currentParametersArray[0], ...statefulCurrentParameters };
 
     await this.applyTransformParameters(resultParameters, true);
+    this.removeDefaultValues(resultParameters, parameters)
+
     return [{ core, parameters: resultParameters }];
   }
 
@@ -377,6 +379,19 @@ ${JSON.stringify(refresh, null, 2)}
     }
   }
 
+  private removeDefaultValues(newConfig: Partial<T> | null, originalConfig: Partial<T>): void {
+    if (!newConfig) {
+      return;
+    }
+
+    for (const [key, defaultValue] of Object.entries(this.parsedSettings.defaultValues)) {
+      if (defaultValue !== undefined && (newConfig[key] === defaultValue || originalConfig[key] === undefined || originalConfig[key] === null)) {
+        delete newConfig[key];
+      }
+    }
+
+  }
+
   private async refreshNonStatefulParameters(resourceParameters: Partial<T>): Promise<Array<Partial<T>> | null> {
     const result = await this.resource.refresh(resourceParameters);
 

src/resource/resource-settings.ts

@@ -368,6 +368,10 @@ const ParameterTransformationDefaults: Partial<Record<ParameterSettingType, Inpu
   'string': {
     to: String,
     from: String,
+  },
+  'boolean': {
+    to: Boolean,
+    from: Boolean,
   }
 }