patch(fix): octo hook and module registrations are sync.

Author: rash805115

packages/octo/src/decorators/enable-hook.decorator.ts

@@ -1,16 +1,19 @@
-import { AHook } from '../functions/hook/hook.abstract.js';
-import { Container } from './container.js';
+import { PostModelActionHook } from '../functions/hook/post-model-action.hook.js';
+import { PreCommitHook } from '../functions/hook/pre-commit.hook.js';
 
 type Hook = 'PostModelActionHook' | 'PreCommitHook';
 
 export function EnableHook(hook: Hook): (target: any, propertyKey: string, descriptor: PropertyDescriptor) => void {
   return function (target: any, propertyKey: string, descriptor: PropertyDescriptor) {
-    Container.get<AHook>(hook)
-      .then((aHook) => {
-        aHook.registrar(target.constructor, propertyKey, descriptor);
-      })
-      .catch((error) => {
-        console.error(error);
-      });
+    switch (hook) {
+      case 'PostModelActionHook':
+        PostModelActionHook.getInstance().registrar(target.constructor, propertyKey, descriptor);
+        break;
+      case 'PreCommitHook':
+        PreCommitHook.getInstance().registrar(target.constructor, propertyKey, descriptor);
+        break;
+      default:
+        throw new Error('Invalid hook!');
+    }
   };
 }

packages/octo/src/decorators/module.decorator.ts

@@ -1,7 +1,6 @@
 import { Constructable } from '../app.type.js';
 import { PostModelActionCallback, PostModelActionHook } from '../functions/hook/post-model-action.hook.js';
 import { PreCommitCallback, PreCommitHook } from '../functions/hook/pre-commit.hook.js';
-import { Container } from './container.js';
 
 export function Module({
   imports = [],
@@ -14,23 +13,11 @@ export function Module({
 }): (constructor: any) => void {
   return function (constructor: any) {
     if (postModelActionHandles.length > 0) {
-      Container.get(PostModelActionHook)
-        .then((aHook) => {
-          aHook.register(constructor.name, { imports, postModelActionHandles });
-        })
-        .catch((error) => {
-          console.error(error);
-        });
+      PostModelActionHook.getInstance().register(constructor.name, { imports, postModelActionHandles });
     }
 
     if (preCommitHandles.length > 0) {
-      Container.get(PreCommitHook)
-        .then((aHook) => {
-          aHook.register(constructor.name, { imports, preCommitHandles });
-        })
-        .catch((error) => {
-          console.error(error);
-        });
+      PreCommitHook.getInstance().register(constructor.name, { imports, preCommitHandles });
     }
   };
 }

packages/octo/src/functions/hook/post-model-action.hook.ts

@@ -6,6 +6,8 @@ import { AHook } from './hook.abstract.js';
 export type PostModelActionCallback = (output: ActionOutputs) => Promise<ActionOutputs>;
 
 export class PostModelActionHook extends AHook {
+  private static instance: PostModelActionHook;
+
   private readonly callbacks: { [key: string]: PostModelActionCallback[] } = {};
 
   private isInstanceOfModelAction(instance: any): instance is IModelAction {
@@ -23,6 +25,13 @@ export class PostModelActionHook extends AHook {
     }
   }
 
+  static getInstance(): PostModelActionHook {
+    if (!this.instance) {
+      this.instance = new PostModelActionHook();
+    }
+    return this.instance;
+  }
+
   override registrar(constructor: Constructable<unknown>, propertyKey: string, descriptor: PropertyDescriptor): void {
     if (!this.isInstanceOfModelAction(constructor.prototype)) {
       throw new Error('PostModelActionHook can only be used with ModelAction!');

packages/octo/src/functions/hook/pre-commit.hook.ts

@@ -5,6 +5,8 @@ import { AHook } from './hook.abstract.js';
 export type PreCommitCallback = (...args: any[]) => Promise<void>;
 
 export class PreCommitHook extends AHook {
+  private static instance: PreCommitHook;
+
   private readonly callbacks: PreCommitCallback[] = [];
 
   override generateCallbacks(): void {
@@ -15,6 +17,13 @@ export class PreCommitHook extends AHook {
     }
   }
 
+  static getInstance(): PreCommitHook {
+    if (!this.instance) {
+      this.instance = new PreCommitHook();
+    }
+    return this.instance;
+  }
+
   override registrar(constructor: Constructable<unknown>, propertyKey: string, descriptor: PropertyDescriptor): void {
     const originalMethod = descriptor.value;
     const self = this; // eslint-disable-line @typescript-eslint/no-this-alias
@@ -36,7 +45,7 @@ export class PreCommitHookFactory {
 
   static async create(): Promise<PreCommitHook> {
     if (!this.instance) {
-      this.instance = new PreCommitHook();
+      this.instance = PreCommitHook.getInstance();
     }
     return this.instance;
   }