codify-plugin-lib 1.0.68 → 1.0.69

package/dist/entities/errors.d.ts

@@ -1,11 +1,4 @@
-import { Plan } from './plan.js';
-import { StringIndexedObject } from 'codify-schemas';
 export declare class SudoError extends Error {
     command: string;
     constructor(command: string);
 }
-export declare class ApplyValidationError<T extends StringIndexedObject> extends Error {
-    desiredPlan: Plan<T>;
-    validatedPlan: Plan<T>;
-    constructor(desiredPlan: Plan<T>, validatedPlan: Plan<T>);
-}

package/dist/entities/errors.js

@@ -5,12 +5,3 @@ export class SudoError extends Error {
         this.command = command;
     }
 }
-export class ApplyValidationError extends Error {
-    desiredPlan;
-    validatedPlan;
-    constructor(desiredPlan, validatedPlan) {
-        super();
-        this.desiredPlan = desiredPlan;
-        this.validatedPlan = validatedPlan;
-    }
-}

package/dist/entities/plugin.js

@@ -1,7 +1,5 @@
-import { ResourceOperation } from 'codify-schemas';
 import { Plan } from './plan.js';
 import { splitUserConfig } from '../utils/utils.js';
-import { ApplyValidationError } from './errors.js';
 export class Plugin {
     name;
     resources;
@@ -64,10 +62,6 @@ export class Plugin {
             throw new Error('Malformed plan with resource that cannot be found');
         }
         await resource.apply(plan);
-        const validationPlan = await resource.plan(plan.desiredConfig, plan.currentConfig, true);
-        if (validationPlan.changeSet.operation !== ResourceOperation.NOOP) {
-            throw new ApplyValidationError(plan, validationPlan);
-        }
     }
     resolvePlan(data) {
         const { planId, plan: planRequest } = data;

package/dist/messages/handlers.js

@@ -1,7 +1,7 @@
 import addFormats from 'ajv-formats';
 import { ApplyRequestDataSchema, ApplyResponseDataSchema, InitializeRequestDataSchema, InitializeResponseDataSchema, IpcMessageSchema, MessageStatus, PlanRequestDataSchema, PlanResponseDataSchema, ResourceSchema, ValidateRequestDataSchema, ValidateResponseDataSchema } from 'codify-schemas';
 import Ajv2020 from 'ajv/dist/2020.js';
-import { ApplyValidationError, SudoError } from '../entities/errors.js';
+import { SudoError } from '../entities/errors.js';
 const SupportedRequests = {
     'initialize': {
         requestValidator: InitializeRequestDataSchema,
@@ -89,20 +89,6 @@ export class MessageHandler {
                 data: `Plugin: '${this.plugin.name}'. Forbidden usage of sudo for command '${e.command}'. Please contact the plugin developer to fix this.`,
             });
         }
-        if (e instanceof ApplyValidationError) {
-            return process.send?.({
-                cmd,
-                status: MessageStatus.ERROR,
-                data: `Plugin: '${this.plugin.name}'. Apply validation was not successful (additional changes are needed to match the desired plan).
-
-Validation plan:
-${JSON.stringify(e.validatedPlan, null, 2)},
-        
-User desired plan:
-${JSON.stringify(e.desiredPlan, null, 2)}
-`
-            });
-        }
         const isDebug = process.env.DEBUG?.includes('*') ?? false;
         process.send?.({
             cmd,

package/package.json

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

package/src/entities/errors.ts

@@ -9,17 +9,3 @@ export class SudoError extends Error {
     this.command = command;
   }
 }
-
-export class ApplyValidationError<T extends StringIndexedObject> extends Error {
-  desiredPlan: Plan<T>;
-  validatedPlan: Plan<T>;
-
-  constructor(
-    desiredPlan: Plan<T>,
-    validatedPlan: Plan<T>
-  ) {
-    super();
-    this.desiredPlan = desiredPlan;
-    this.validatedPlan = validatedPlan;
-  }
-}

package/src/entities/plugin.ts

@@ -11,7 +11,6 @@ import {
 } from 'codify-schemas';
 import { Plan } from './plan.js';
 import { splitUserConfig } from '../utils/utils.js';
-import { ApplyValidationError } from './errors.js';
 
 export class Plugin {
   planStorage: Map<string, Plan<any>>;
@@ -92,17 +91,6 @@ export class Plugin {
     }
 
     await resource.apply(plan);
-
-    // Perform a validation check after to ensure that the plan was properly applied.
-    // Sometimes no errors are returned (exit code 0) but the apply was not successful
-    const validationPlan = await resource.plan(
-      plan.desiredConfig,
-      plan.currentConfig,
-      true,
-    );
-    if (validationPlan.changeSet.operation !== ResourceOperation.NOOP) {
-      throw new ApplyValidationError(plan, validationPlan);
-    }
   }
 
   private resolvePlan(data: ApplyRequestData): Plan<ResourceConfig> {

package/src/messages/handlers.ts

@@ -15,7 +15,7 @@ import {
   ValidateResponseDataSchema
 } from 'codify-schemas';
 import Ajv2020, { SchemaObject, ValidateFunction } from 'ajv/dist/2020.js';
-import { ApplyValidationError, SudoError } from '../entities/errors.js';
+import { SudoError } from '../entities/errors.js';
 
 const SupportedRequests: Record<string, { requestValidator: SchemaObject; responseValidator: SchemaObject; handler: (plugin: Plugin, data: any) => Promise<unknown> }> = {
   'initialize': {
@@ -124,21 +124,6 @@ export class MessageHandler {
       })
     }
 
-    if (e instanceof ApplyValidationError) {
-      return process.send?.({
-        cmd,
-        status: MessageStatus.ERROR,
-        data: `Plugin: '${this.plugin.name}'. Apply validation was not successful (additional changes are needed to match the desired plan).
-
-Validation plan:
-${JSON.stringify(e.validatedPlan, null, 2)},
-        
-User desired plan:
-${JSON.stringify(e.desiredPlan, null, 2)}
-`
-      })
-    }
-
     const isDebug = process.env.DEBUG?.includes('*') ?? false;
 
     process.send?.({