@friggframework/core 2.0.0--canary.396.d50d765.0 → 2.0.0--canary.396.340648d.0

package/index.js

@@ -46,7 +46,6 @@ const { TimeoutCatcher } = require('./lambda/index');
 const { debug, initDebugLog, flushDebugLog } = require('./logs/index');
 const {
     Credential,
-    EntityManager,
     Entity,
     ModuleManager,
     ApiKeyRequester,
@@ -120,7 +119,6 @@ module.exports = {
 
     // module plugin
     Credential,
-    EntityManager,
     Entity,
     ModuleManager,
     ApiKeyRequester,

package/integrations/integration-router.js

@@ -12,6 +12,7 @@ const { GetCredentialForUser } = require('../credential/use-cases/get-credential
 const { CreateIntegration } = require('./use-cases/create-integration');
 const { ModuleService } = require('../module-plugin/module-service');
 const { ModuleRepository } = require('../module-plugin/module-repository');
+const { GetEntitiesForUser } = require('../module-plugin/use-cases/get-entities-for-user');
 const {
     loadAppDefinition,
 } = require('../handlers/app-definition-loader');
@@ -70,14 +71,21 @@ function createIntegrationRouter(params) {
         moduleService,
     });
 
+    const getEntitiesForUserUseCase = new GetEntitiesForUser({
+        moduleRepository,
+        moduleDefinitions: getModules(),
+    });
+
     const router = get(params, 'router', express());
     const factory = get(params, 'factory');
     const getUserFromBearerToken = get(params, 'getUserFromBearerToken');
 
+    // todo: moduleFactory in factory is not used here anymore, remove it
     setIntegrationRoutes(router, factory, getUserFromBearerToken, {
         createIntegration,
         deleteIntegrationForUser,
         getIntegrationsForUser,
+        getEntitiesForUserUseCase,
     });
     setEntityRoutes(router, factory, getUserFromBearerToken, {
         getCredentialForUser,
@@ -116,11 +124,12 @@ function checkRequiredParams(params, requiredKeys) {
  * @param {import('../user/use-cases/get-user-from-bearer-token').GetUserFromBearerToken} getUserFromBearerToken - Use case for retrieving a user from a bearer token
  */
 function setIntegrationRoutes(router, factory, getUserFromBearerToken, useCases) {
-    const { moduleFactory, integrationFactory } = factory;
+    const { integrationFactory } = factory;
     const {
         createIntegration,
         deleteIntegrationForUser,
         getIntegrationsForUser,
+        getEntitiesForUserUseCase,
     } = useCases;
 
     router.route('/api/integrations').get(
@@ -130,8 +139,7 @@ function setIntegrationRoutes(router, factory, getUserFromBearerToken, useCases)
             );
             const userId = user.getId();
             const results = await integrationFactory.getIntegrationOptions();
-            results.entities.authorized =
-                await moduleFactory.getEntitiesForUser(userId);
+            results.entities.authorized = await getEntitiesForUserUseCase.execute(userId);
             results.integrations =
                 await getIntegrationsForUser.execute(userId);
 
@@ -454,7 +462,7 @@ function setEntityRoutes(router, factory, getUserFromBearerToken, useCases) {
                 module.validateAuthorizationRequirements();
             if (!areRequirementsValid) {
                 throw new Error(
-                    `Error: EntityManager of type ${params.entityType} requires a valid url`
+                    `Error: Entity of type ${params.entityType} requires a valid url`
                 );
             }
 

package/integrations/use-cases/index.js

@@ -1,15 +1,11 @@
-const { GetIntegrationsForUserId } = require('./get-integrations-for-user');
-const { DeleteIntegrationForUserById } = require('./delete-integration-for-user-by-id');
-const { GetIntegrationById } = require('./get-integration-by-id');
-const { ListCredentials } = require('./list-credentials');
+const { GetIntegrationsForUser } = require('./get-integrations-for-user');
+const { DeleteIntegrationForUser } = require('./delete-integration-for-user');
 const { CreateIntegration } = require('./create-integration');
 const { GetIntegrationInstance } = require('./get-integration-instance');
 
 module.exports = {
-    GetIntegrationsForUserId,
-    DeleteIntegrationForUserById,
-    GetIntegrationById,
-    ListCredentials,
+    GetIntegrationsForUser,
+    DeleteIntegrationForUser,
     CreateIntegration,
     GetIntegrationInstance,
 }; 

package/module-plugin/index.js

@@ -1,5 +1,4 @@
 const { Credential } = require('./credential');
-const { EntityManager } = require('./entity-manager');
 const { Entity } = require('./entity');
 const { ModuleManager } = require('./manager');
 const { ApiKeyRequester } = require('./requester/api-key');
@@ -11,7 +10,6 @@ const { ModuleFactory } = require('./module-factory');
 
 module.exports = {
     Credential,
-    EntityManager,
     Entity,
     ModuleManager,
     ApiKeyRequester,

package/module-plugin/module-factory.js

@@ -13,19 +13,6 @@ class ModuleFactory {
         });
     }
 
-    async getEntitiesForUser(userId) {
-        let results = [];
-        for (const moduleDefinition of this.moduleDefinitions) {
-            const moduleInstance = new Module({
-                userId,
-                definition: moduleDefinition,
-            });
-            const list = await moduleInstance.getEntitiesForUserId(userId);
-            results.push(...list);
-        }
-        return results;
-    }
-
     checkIsValidType(entityType) {
         return this.moduleTypes.includes(entityType);
     }

package/module-plugin/module-repository.js

@@ -9,6 +9,14 @@ class ModuleRepository {
 
         return entity;
     }
+
+    async findEntitiesByUserId(userId) {
+        return Entity.find(
+            { user: userId },
+            '-dateCreated -dateUpdated -user -credentials -credential -__t -__v',
+            { lean: true }
+        );
+    }
 }
 
 module.exports = { ModuleRepository }; 

package/module-plugin/module.js

@@ -103,20 +103,21 @@ class Module extends Delegate {
         return this.CredentialModel;
     }
 
-    async getEntitiesForUserId(userId) {
-        // Only return non-internal fields. Leverages "select" and "options" to non-excepted fields and a pure object.
-        const list = await this.EntityModel.find(
-            { user: userId },
-            '-dateCreated -dateUpdated -user -credentials -credential -__t -__v',
-            { lean: true }
-        );
-        console.log('getEntitiesForUserId list', list, userId);
-        return list.map((entity) => ({
-            id: entity._id,
-            type: this.getName(),
-            ...entity,
-        }));
-    }
+    // todo: remove this method from all places
+    // async getEntitiesForUserId(userId) {
+    //     // Only return non-internal fields. Leverages "select" and "options" to non-excepted fields and a pure object.
+    //     const list = await this.EntityModel.find(
+    //         { user: userId },
+    //         '-dateCreated -dateUpdated -user -credentials -credential -__t -__v',
+    //         { lean: true }
+    //     );
+    //     console.log('getEntitiesForUserId list', list, userId);
+    //     return list.map((entity) => ({
+    //         id: entity._id,
+    //         type: this.getName(),
+    //         ...entity,
+    //     }));
+    // }
 
     async validateAuthorizationRequirements() {
         const requirements = await this.getAuthorizationRequirements();

package/module-plugin/use-cases/get-entities-for-user.js

@@ -0,0 +1,35 @@
+const { Module } = require('../module');
+
+class GetEntitiesForUser {
+    constructor({ moduleRepository, moduleDefinitions }) {
+        this.moduleRepository = moduleRepository;
+
+        this.definitionMap = new Map();
+        for (const definition of moduleDefinitions) {
+            const modelName =
+                Module.getEntityModelFromDefinition(definition).modelName;
+            this.definitionMap.set(modelName, definition);
+        }
+    }
+
+    async execute(userId) {
+        const entities = await this.moduleRepository.findEntitiesByUserId(
+            userId
+        );
+
+        return entities.map((entity) => {
+            const definition = this.definitionMap.get(entity.__t);
+            // todo: check if dont need to use this.
+            const type = definition ? definition.getName() : 'unknown';
+
+            const moduleInstance = new Module({
+                userId,
+                definition: definition,
+                entity: entity,
+            });
+            return moduleInstance;
+        });
+    }
+}
+
+module.exports = { GetEntitiesForUser }; 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@friggframework/core",
   "prettier": "@friggframework/prettier-config",
-  "version": "2.0.0--canary.396.d50d765.0",
+  "version": "2.0.0--canary.396.340648d.0",
   "dependencies": {
     "@hapi/boom": "^10.0.1",
     "aws-sdk": "^2.1200.0",
@@ -22,9 +22,9 @@
     "uuid": "^9.0.1"
   },
   "devDependencies": {
-    "@friggframework/eslint-config": "2.0.0--canary.396.d50d765.0",
-    "@friggframework/prettier-config": "2.0.0--canary.396.d50d765.0",
-    "@friggframework/test": "2.0.0--canary.396.d50d765.0",
+    "@friggframework/eslint-config": "2.0.0--canary.396.340648d.0",
+    "@friggframework/prettier-config": "2.0.0--canary.396.340648d.0",
+    "@friggframework/test": "2.0.0--canary.396.340648d.0",
     "@types/lodash": "4.17.15",
     "@typescript-eslint/eslint-plugin": "^8.0.0",
     "chai": "^4.3.6",
@@ -53,5 +53,5 @@
   },
   "homepage": "https://github.com/friggframework/frigg#readme",
   "description": "",
-  "gitHead": "d50d765cf88cd18a604ded97837926ad2ab0bc87"
+  "gitHead": "340648d415ccfb50a16dcdcfa02ee7367b3ca64f"
 }

package/types/integrations/index.d.ts

@@ -1,7 +1,6 @@
 declare module "@friggframework/integrations" {
   import { Delegate, IFriggDelegate } from "@friggframework/core";
   import { Model } from "mongoose";
-  import { EntityManager } from "@friggframework/module-plugin";
 
   export class Integration extends Model {
     entities: any[];
@@ -19,8 +18,7 @@ declare module "@friggframework/integrations" {
 
   export class IntegrationManager
     extends Delegate
-    implements IFriggIntegrationManager
-  {
+    implements IFriggIntegrationManager {
     integration: Integration;
     primaryInstance: any;
     targetInstance: any;
@@ -56,7 +54,6 @@ declare module "@friggframework/integrations" {
       entities: { id: string; user: any },
       userId: string,
       config: any,
-      EntityManager: EntityManager
     ): Promise<any>;
 
     static getFormattedIntegration(
@@ -116,8 +113,7 @@ declare module "@friggframework/integrations" {
   }
 
   export class IntegrationConfigManager
-    implements IFriggIntegrationConfigManager
-  {
+    implements IFriggIntegrationConfigManager {
     options: IntegrationOptions[];
     primary: any;
 

package/types/module-plugin/index.d.ts

@@ -9,21 +9,7 @@ declare module "@friggframework/module-plugin" {
     externalId: string;
   }
 
-  export class EntityManager implements IFriggEntityManager {
-    static primaryEntityClass: any;
-    static entityManagerClasses: any[];
-    static entityTypes: string[];
-    static getEntitiesForUser(userId: string): Promise<any[]>;
-    static checkIsValidType(entityType: string): boolean;
-    static getEntityManagerClass(entityType?: string): any;
-
-    static getEntityManagerInstanceFromEntityId(
-      entityId: string,
-      userId: string
-    ): Promise<any>;
-  }
-
-  interface IFriggEntityManager {}
+  interface IFriggEntityManager { }
 
   export class Entity extends Model {
     credentialId: string;
@@ -138,8 +124,7 @@ declare module "@friggframework/module-plugin" {
 
   export class ApiKeyRequester
     extends Requester
-    implements IFriggApiKeyRequester
-  {
+    implements IFriggApiKeyRequester {
     API_KEY_NAME: string;
     API_KEY_VALUE: any;
 
@@ -160,8 +145,7 @@ declare module "@friggframework/module-plugin" {
 
   export class BasicAuthRequester
     extends Requester
-    implements IFriggBasicAuthRequester
-  {
+    implements IFriggBasicAuthRequester {
     password: string;
     username: string;
 
@@ -189,8 +173,7 @@ declare module "@friggframework/module-plugin" {
 
   export class OAuth2Requester
     extends Requester
-    implements IFriggOAuth2Requester
-  {
+    implements IFriggOAuth2Requester {
     DLGT_TOKEN_DEAUTHORIZED: string;
     DLGT_TOKEN_UPDATE: string;
     accessTokenExpire: any;

package/module-plugin/entity-manager.js

@@ -1,70 +0,0 @@
-const { loadInstalledModules, Delegate } = require('../core');
-
-const { Entity } = require('./entity');
-const { ModuleManager } = require('./manager');
-
-class EntityManager {
-    static primaryEntityClass = null; //primaryEntity;
-
-    static entityManagerClasses = loadInstalledModules().map(
-        (m) => m.EntityManager
-    );
-
-    static entityTypes = EntityManager.entityManagerClasses.map(
-        (ManagerClass) => ManagerClass.getName()
-    );
-
-    static async getEntitiesForUser(userId) {
-        const results = [];
-        for (const Manager of this.entityManagerClasses) {
-            results.push(...(await Manager.getEntitiesForUserId(userId)));
-        }
-        return results;
-    }
-
-    static checkIsValidType(entityType) {
-        const indexOfEntity = EntityManager.entityTypes.indexOf(entityType);
-        return indexOfEntity >= 0;
-    }
-
-    static getEntityManagerClass(entityType = '') {
-        const normalizedType = entityType.toLowerCase();
-
-        const indexOfEntityType =
-            EntityManager.entityTypes.indexOf(normalizedType);
-        if (!EntityManager.checkIsValidType(normalizedType)) {
-            throw new Error(
-                `Error: Invalid entity type of ${normalizedType}, options are ${EntityManager.entityTypes.join(
-                    ', '
-                )}`
-            );
-        }
-
-        const managerClass =
-            EntityManager.entityManagerClasses[indexOfEntityType];
-
-        if (!(managerClass.prototype instanceof ModuleManager)) {
-            throw new Error('The Entity is not an instance of ModuleManager');
-        }
-
-        return managerClass;
-    }
-
-    static async getEntityManagerInstanceFromEntityId(entityId, userId) {
-        const entityMO = new Entity();
-        const entity = await entityMO.get(entityId);
-        let entityManagerClass;
-        for (const Manager of this.entityManagerClasses) {
-            if (entity instanceof Manager.Entity.Model) {
-                entityManagerClass = Manager;
-            }
-        }
-        const instance = await entityManagerClass.getInstance({
-            userId,
-            entityId,
-        });
-        return instance;
-    }
-}
-
-module.exports = { EntityManager };