fix(webhook-custom-actions): register webhook actions configured from…

… the frontend (#686)

packages/agent/src/agent.ts

@@ -73,13 +73,13 @@ export default class Agent<S extends TSchema = TSchema> extends FrameworkMounter
   override async start(): Promise<void> {
     const { isProduction, logger, skipSchemaUpdate, typingsPath, typingsMaxDepth } = this.options;
 
-    const dataSource = await this.nocodeCustomizer.getDataSource(logger);
-
     await this.nocodeCustomizer.use(
       this.actionCustomizationService.addWebhookActions,
       this.options.experimental?.webhookCustomActions,
     );
 
+    const dataSource = await this.nocodeCustomizer.getDataSource(logger);
+
     const [router] = await Promise.all([
       this.getRouter(dataSource),
       !skipSchemaUpdate ? this.sendSchema(dataSource) : Promise.resolve(),

packages/agent/test/agent.test.ts

@@ -104,6 +104,28 @@ describe('Agent', () => {
       expect(mockCustomizer.use).toHaveBeenCalledTimes(1);
     });
 
+    test('use should be called before getDatasource in order to be correctly applied', async () => {
+      const agent = new Agent(options);
+      let useCalled = false;
+      let getDataSourceCalledAfterUseCalled = false;
+
+      mockNocodeCustomizer.use.mockImplementationOnce(async () => {
+        useCalled = true;
+      });
+
+      mockNocodeCustomizer.getDataSource.mockImplementationOnce(async () => {
+        getDataSourceCalledAfterUseCalled = useCalled;
+
+        return factories.dataSource.build();
+      });
+
+      await agent.start();
+
+      expect(mockNocodeCustomizer.use).toHaveBeenCalledTimes(1);
+      expect(mockNocodeCustomizer.getDataSource).toHaveBeenCalledTimes(1);
+      expect(getDataSourceCalledAfterUseCalled).toBe(true);
+    });
+
     // eslint-disable-next-line max-len
     test("should add the customizer's factory as a datasource for the nocode customizer", async () => {
       mockCustomizer.getFactory.mockReturnValueOnce('factory');