[CODE-60] Apply verification (#24)

* [CODE-60] Add apply verification

* [CODE-60] Refactored files

* [CODE-60] Remove json diff

Author: kevinwang5658

src/utils/errors.ts src/common/errors.tssrc/utils/errors.ts renamed to src/common/errors.ts

@@ -1,17 +1,17 @@
 import { Project } from '../entities/project.js';
 import { ctx, SubProcessName } from '../events/context.js';
-import { DependencyMap, PluginCollection } from '../plugins/plugin-collection.js';
+import { DependencyMap, PluginManager } from '../plugins/plugin-manager.js';
 
 export const CommonOrchestrator = {
   async initializePlugins(project?: Project, secureMode = false): Promise<{
     dependencyMap: DependencyMap
-    pluginCollection: PluginCollection,
+    pluginManager: PluginManager,
   }> {
     ctx.subprocessStarted(SubProcessName.INITIALIZE_PLUGINS)
-    const pluginCollection = new PluginCollection();
-    const dependencyMap = await pluginCollection.initialize(project, secureMode);
+    const pluginManager = new PluginManager();
+    const dependencyMap = await pluginManager.initialize(project, secureMode);
     ctx.subprocessFinished(SubProcessName.INITIALIZE_PLUGINS)
 
-    return { dependencyMap, pluginCollection };
+    return { dependencyMap, pluginManager };
   },
 };

src/common/orchestrator.ts

@@ -2,7 +2,7 @@ import { ProjectSchema } from 'codify-schemas';
 
 import { ConfigClass } from '../parser/language-definition.js';
 import { ajv } from '../utils/ajv.js';
-import { RemoveMethods } from '../utils/types.js';
+import { RemoveMethods } from '../common/types.js';
 import { ConfigBlock } from './config.js';
 
 /** Project JSON supported format

src/utils/types.ts src/common/types.tssrc/utils/types.ts renamed to src/common/types.ts

@@ -1,7 +1,7 @@
 import { ValidateResponseData } from 'codify-schemas';
 
 import { ctx } from '../events/context.js';
-import { DependencyMap } from '../plugins/plugin-collection.js';
+import { DependencyMap } from '../plugins/plugin-manager.js';
 import { DependencyGraphResolver } from '../utils/dependency-graph-resolver.js';
 import { ProjectConfig } from './project-config.js';
 import { ResourceConfig } from './resource-config.js';

src/entities/project-config.ts

@@ -2,7 +2,7 @@ import { ResourceSchema } from 'codify-schemas';
 
 import { ConfigClass } from '../parser/language-definition.js';
 import { ajv } from '../utils/ajv.js';
-import { RemoveMethods } from '../utils/types.js';
+import { RemoveMethods } from '../common/types.js';
 import { ConfigBlock } from './config.js';
 
 /** Resource JSON supported format

src/entities/project.ts

@@ -5,12 +5,12 @@ import { PlanOrchestratorResponse } from './plan.js';
 
 export const ApplyOrchestrator = {
   async run(planResult: PlanOrchestratorResponse): Promise<void> {
-    const { plan, pluginCollection } = planResult;
+    const { plan, pluginManager, project } = planResult;
     const filteredPlan = plan
       .filter((p) => p.operation !== ResourceOperation.NOOP)
 
     ctx.processStarted(ProcessName.APPLY);
-    await pluginCollection.apply(filteredPlan);
+    await pluginManager.apply(project, filteredPlan);
     ctx.processFinished(ProcessName.APPLY);
   },
 };

src/entities/resource-config.ts

@@ -1,15 +1,15 @@
 import { PlanResponseData } from 'codify-schemas';
 
+import { CommonOrchestrator } from '../common/orchestrator.js';
 import { Project } from '../entities/project.js';
 import { ctx, ProcessName, SubProcessName } from '../events/context.js';
 import { Parser } from '../parser/index.js';
-import { PluginCollection } from '../plugins/plugin-collection.js';
+import { PluginManager } from '../plugins/plugin-manager.js';
 import { createStartupShellScriptsIfNotExists } from '../utils/file.js';
-import { CommonOrchestrator } from '../common/orchestrator.js';
 
 export interface PlanOrchestratorResponse {
   plan: PlanResponseData[],
-  pluginCollection: PluginCollection;
+  pluginManager: PluginManager;
   project: Project;
 }
 
@@ -24,27 +24,27 @@ export const PlanOrchestrator = {
     project.addXCodeToolsConfig();
     ctx.subprocessFinished(SubProcessName.PARSE);
 
-    const { dependencyMap, pluginCollection } = await CommonOrchestrator.initializePlugins(project, secureMode);
+    const { dependencyMap, pluginManager } = await CommonOrchestrator.initializePlugins(project, secureMode);
     await createStartupShellScriptsIfNotExists();
 
     ctx.subprocessStarted(SubProcessName.VALIDATE)
     project.validateWithResourceMap(dependencyMap);
     project.resolveResourceDependencies(dependencyMap);
 
-    const validationResults = await pluginCollection.validate(project);
+    const validationResults = await pluginManager.validate(project);
     project.handlePluginResourceValidationResults(validationResults);
     project.calculateEvaluationOrder();
     ctx.subprocessFinished(SubProcessName.VALIDATE)
 
     ctx.subprocessStarted(SubProcessName.GENERATE_PLAN)
-    const plan = await pluginCollection.getPlan(project);
+    const plan = await pluginManager.getPlan(project);
     ctx.subprocessFinished(SubProcessName.GENERATE_PLAN)
 
     ctx.processFinished(ProcessName.PLAN)
 
     return {
       plan,
-      pluginCollection,
+      pluginManager,
       project,
     };
   },

src/orchestrators/apply.ts

@@ -18,11 +18,11 @@ export const UninstallOrchestrator = {
       resourceType: type,
     } as PlanResponseData))
 
-    const { pluginCollection } = await CommonOrchestrator.initializePlugins(undefined, secureMode);
+    const { pluginManager } = await CommonOrchestrator.initializePlugins(undefined, secureMode);
     await createStartupShellScriptsIfNotExists();
 
     ctx.processStarted(ProcessName.UNINSTALL);
-    await pluginCollection.apply(plan);
+    // await pluginManager.apply(plan);
     ctx.processFinished(ProcessName.UNINSTALL);
   },
 };

src/orchestrators/plan.ts

@@ -2,7 +2,7 @@ import { ConfigBlock } from '../entities/config.js';
 import { Project } from '../entities/project.js';
 import { ProjectConfig } from '../entities/project-config.js';
 import { ResourceConfig } from '../entities/resource-config.js';
-import { InternalError } from '../utils/errors.js';
+import { InternalError } from '../common/errors.js';
 import { ConfigClass } from './language-definition.js';
 import { FileParser } from './parser/index.js';
 import { JsonFileParser } from './parser/json/file-parser.js';