codify-plugin-lib 1.0.182-beta76 → 1.0.182-beta77

package/dist/plan/plan.js

@@ -206,14 +206,12 @@ export class Plan {
             const arrayStatefulParameters = Object.fromEntries(Object.entries(filteredCurrent)
                 .filter(([k, v]) => isArrayParameterWithFiltering(k, v))
                 .map(([k, v]) => [k, filterArrayParameterForDeletes(k, v)]));
-            console.log('Result for destroy filtering', { ...filteredCurrent, ...arrayStatefulParameters });
             return { ...filteredCurrent, ...arrayStatefulParameters };
         }
         // TODO: Add object handling here in addition to arrays in the future
         const arrayStatefulParameters = Object.fromEntries(Object.entries(filteredCurrent)
             .filter(([k, v]) => isArrayParameterWithFiltering(k, v))
             .map(([k, v]) => [k, filterArrayParameterForStatelessMode(k, v)]));
-        console.log('Result', { ...filteredCurrent, ...arrayStatefulParameters });
         return { ...filteredCurrent, ...arrayStatefulParameters };
         function filterCurrent() {
             if (!current) {
@@ -255,7 +253,6 @@ export class Plan {
         }
         // For stateless mode, we must filter the current array so that the diff algorithm will not detect any deletes
         function filterArrayParameterForStatelessMode(k, v) {
-            console.log('Attempting to filter stateless (key, value)', k, v);
             const desiredArray = desired[k];
             const matcher = settings.parameterSettings[k].type === 'stateful'
                 ? settings.parameterSettings[k]
@@ -279,13 +276,11 @@ export class Plan {
                 return result;
             });
             const filterParameter = getFilterParameter(k);
-            const result = typeof filterParameter === 'function'
+            return typeof filterParameter === 'function'
                 ? filterParameter(desiredCopy, currentCopy)
                 : defaultFilterMethod(desiredCopy, currentCopy);
-            return result.length === 0 ? null : result;
         }
         function filterArrayParameterForDeletes(k, v) {
-            console.log('Attempting to filter deletes (key, value)', k, v);
             const stateArray = state[k];
             const matcher = settings.parameterSettings[k].type === 'stateful'
                 ? settings.parameterSettings[k]
@@ -309,10 +304,9 @@ export class Plan {
                 return result;
             });
             const filterParameter = getFilterParameter(k);
-            const result = typeof filterParameter === 'function'
+            return typeof filterParameter === 'function'
                 ? filterParameter(stateCopy, currentCopy)
                 : defaultFilterMethod(stateCopy, currentCopy);
-            return result.length === 0 ? null : result;
         }
     }
     // TODO: This needs to be revisited. I don't think this is valid anymore.

package/package.json

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

package/src/plan/plan.ts

@@ -339,8 +339,6 @@ export class Plan<T extends StringIndexedObject> {
           .map(([k, v]) => [k, filterArrayParameterForDeletes(k, v)])
       )
 
-      console.log('Result for destroy filtering', { ...filteredCurrent, ...arrayStatefulParameters });
-
       return { ...filteredCurrent, ...arrayStatefulParameters }
     }
 
@@ -351,8 +349,6 @@ export class Plan<T extends StringIndexedObject> {
         .map(([k, v]) => [k, filterArrayParameterForStatelessMode(k, v)])
     )
 
-    console.log('Result', { ...filteredCurrent, ...arrayStatefulParameters });
-
     return { ...filteredCurrent, ...arrayStatefulParameters }
 
     function filterCurrent(): Partial<T> | null {
@@ -408,9 +404,7 @@ export class Plan<T extends StringIndexedObject> {
     }
 
     // For stateless mode, we must filter the current array so that the diff algorithm will not detect any deletes
-    function filterArrayParameterForStatelessMode(k: string, v: unknown[]): null | unknown[] {
-      console.log('Attempting to filter stateless (key, value)', k, v);
-
+    function filterArrayParameterForStatelessMode(k: string, v: unknown[]): unknown[] {
       const desiredArray = desired![k] as unknown[];
       const matcher = settings.parameterSettings![k]!.type === 'stateful'
         ? ((settings.parameterSettings![k] as ParsedStatefulParameterSetting)
@@ -441,16 +435,12 @@ export class Plan<T extends StringIndexedObject> {
       })
 
       const filterParameter = getFilterParameter(k);
-      const result = typeof filterParameter === 'function'
+      return typeof filterParameter === 'function'
         ? filterParameter(desiredCopy, currentCopy)
         : defaultFilterMethod(desiredCopy, currentCopy);
-
-      return result.length === 0 ? null : result;
     }
 
-    function filterArrayParameterForDeletes(k: string, v: unknown[]): null | unknown[] {
-      console.log('Attempting to filter deletes (key, value)', k, v);
-
+    function filterArrayParameterForDeletes(k: string, v: unknown[]): unknown[] {
       const stateArray = state![k] as unknown[];
       const matcher = settings.parameterSettings![k]!.type === 'stateful'
         ? ((settings.parameterSettings![k] as ParsedStatefulParameterSetting)
@@ -482,11 +472,9 @@ export class Plan<T extends StringIndexedObject> {
 
 
       const filterParameter = getFilterParameter(k);
-      const result = typeof filterParameter === 'function'
+      return typeof filterParameter === 'function'
         ? filterParameter(stateCopy, currentCopy)
         : defaultFilterMethod(stateCopy, currentCopy);
-
-      return result.length === 0 ? null : result;
     }
   }