codify-plugin-lib 1.0.23 → 1.0.25

package/dist/entities/resource.js

@@ -29,7 +29,9 @@ class Resource {
         const resourceOperation = parameterChangeSet
             .filter((change) => change.operation !== codify_schemas_1.ParameterOperation.NOOP)
             .reduce((operation, curr) => {
-            const newOperation = this.calculateOperation(curr);
+            const newOperation = !this.statefulParameters.has(curr.name)
+                ? this.calculateOperation(curr)
+                : codify_schemas_1.ResourceOperation.MODIFY;
             return change_set_1.ChangeSet.combineResourceOperations(operation, newOperation);
         }, codify_schemas_1.ResourceOperation.NOOP);
         return plan_1.Plan.create(new change_set_1.ChangeSet(resourceOperation, parameterChangeSet), desiredConfig);

package/dist/utils/utils.d.ts

@@ -12,6 +12,8 @@ type CodifySpawnOptions = {
     cwd?: string;
     stdioString?: boolean;
 } & SpawnOptions;
-export declare function codifySpawn(cmd: string, args?: string[], opts?: Omit<CodifySpawnOptions, 'stdio' | 'stdioString' | 'shell'>, extras?: Record<any, any>): Promise<SpawnResult>;
+export declare function codifySpawn(cmd: string, args?: string[], opts?: Omit<CodifySpawnOptions, 'stdio' | 'stdioString' | 'shell'> & {
+    throws?: boolean;
+}, extras?: Record<any, any>): Promise<SpawnResult>;
 export declare function isDebug(): boolean;
 export {};

package/dist/utils/utils.js

@@ -25,10 +25,16 @@ async function codifySpawn(cmd, args, opts, extras) {
             data: status === SpawnStatus.SUCCESS ? result.stdout : result.stderr
         };
     }
-    catch (e) {
+    catch (error) {
+        if (opts?.throws ?? true) {
+            throw error;
+        }
+        if (isDebug()) {
+            console.error(`CodifySpawn Error for command ${cmd} ${args}`, error);
+        }
         return {
             status: SpawnStatus.ERROR,
-            data: e,
+            data: error,
         };
     }
 }

package/package.json

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

package/src/entities/resource.ts

@@ -45,7 +45,9 @@ export abstract class Resource<T extends ResourceConfig> {
     const resourceOperation = parameterChangeSet
       .filter((change) => change.operation !== ParameterOperation.NOOP)
       .reduce((operation: ResourceOperation, curr: ParameterChange) => {
-        const newOperation = this.calculateOperation(curr);
+        const newOperation = !this.statefulParameters.has(curr.name)
+          ? this.calculateOperation(curr)
+          : ResourceOperation.MODIFY; // All stateful parameters are modify only
         return ChangeSet.combineResourceOperations(operation, newOperation);
       }, ResourceOperation.NOOP);
 

package/src/utils/utils.ts

@@ -16,10 +16,23 @@ type CodifySpawnOptions = {
   stdioString?: boolean;
 } & SpawnOptions
 
+/**
+ *
+ * @param cmd Command to run. Ex: `rm -rf`
+ * @param args Optional additional arguments to append
+ * @param opts Standard options for node spawn. Additional argument:
+ * throws determines if a shell will throw a JS error. Defaults to true
+ * @param extras From PromiseSpawn
+ *
+ * @see promiseSpawn
+ * @see spawn
+ *
+ * @returns SpawnResult { status: SUCCESS | ERROR; data: string }
+ */
 export async function codifySpawn(
   cmd: string,
   args?: string[],
-  opts?: Omit<CodifySpawnOptions, 'stdio' | 'stdioString' | 'shell'>,
+  opts?: Omit<CodifySpawnOptions, 'stdio' | 'stdioString' | 'shell'> & { throws?: boolean },
   extras?: Record<any, any>,
 ): Promise<SpawnResult> {
   try {
@@ -43,10 +56,18 @@ export async function codifySpawn(
       status,
       data: status === SpawnStatus.SUCCESS ? result.stdout : result.stderr
     }
-  } catch (e) {
+  } catch (error) {
+    if (opts?.throws ?? true) {
+      throw error;
+    }
+
+    if (isDebug()) {
+      console.error(`CodifySpawn Error for command ${cmd} ${args}`, error);
+    }
+
     return {
       status: SpawnStatus.ERROR,
-      data: e as string,
+      data: error as string,
     }
   }
 }