feat: Add ability to skip destroys for a resource

Author: kevinwang5658

package.json

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

src/plan/change-set.ts

@@ -72,6 +72,18 @@ export class ChangeSet<T extends StringIndexedObject> {
     return new ChangeSet(ResourceOperation.CREATE, parameterChanges);
   }
 
+  static noop<T extends StringIndexedObject>(parameters: Partial<T>): ChangeSet<T> {
+    const parameterChanges = Object.entries(parameters)
+      .map(([k, v]) => ({
+        name: k,
+        operation: ParameterOperation.NOOP,
+        previousValue: v ?? null,
+        newValue: v ?? null,
+      }))
+
+    return new ChangeSet(ResourceOperation.NOOP, parameterChanges);
+  }
+
   static destroy<T extends StringIndexedObject>(current: Partial<T>): ChangeSet<T> {
     const parameterChanges = Object.entries(current)
       .map(([k, v]) => ({

src/plan/plan.ts

@@ -126,6 +126,16 @@ export class Plan<T extends StringIndexedObject> {
 
     // DESTROY
     if (filteredCurrentParameters && !desired) {
+      // We can manually override destroys. If a resource cannot be destroyed (for instance the npm resource relies on NodeJS being created and destroyed)
+      if (!settings.canDestroy) {
+        return new Plan(
+          uuidV4(),
+          ChangeSet.noop(filteredCurrentParameters),
+          core,
+          isStateful,
+        )
+      }
+
       return new Plan(
         uuidV4(),
         ChangeSet.destroy(filteredCurrentParameters),

src/resource/parsed-resource-settings.ts

@@ -64,6 +64,10 @@ export class ParsedResourceSettings<T extends StringIndexedObject> implements Re
     return this.id;
   }
 
+  get canDestroy(): boolean {
+    return !this.settings.importAndDestroy?.preventDestroy;
+  }
+
   get statefulParameters(): Map<keyof T, StatefulParameterController<T, T[keyof T]>> {
     return this.getFromCacheOrCreate('statefulParameters', () => {
 

src/resource/resource-settings.ts

@@ -113,12 +113,18 @@ export interface ResourceSettings<T extends StringIndexedObject> {
   importAndDestroy?: {
     /**
      * Can this resources be imported? If set to false then the codifyCLI will skip over/not consider this
-     * resource valid for imports. Defaults to true.
+     * resource valid for imports. Defaults to false.
      *
      * Resources that can't be imported in the core library for example are: action resources
      */
     preventImport?: boolean;
 
+    /**
+     * Can this resources be destroyed? If set to false then the codifyCLI will skip over/not consider this
+     * resource valid for destroys. Defaults to false.
+     */
+    preventDestroy?: boolean;
+
     /**
      * Customize the required parameters needed to import this resource. By default, the `requiredParameters` are taken
      * from the identifyingParameters for allowMultiple. The `requiredParameters` parameter must be declared if a complex required is declared in