feat(tests): noteList.test

.github/workflows/build-and-push-docker-image.yml → .github/workflows/deploy-API.yml

@@ -1,35 +1,21 @@
-name: Build and push docker image
+name: Deploy API
 
 on:
   push:
     branches:
-      - '*'
+      - 'main'
     tags:
       - 'v*'
 
 env:
   REGISTRY: ghcr.io
 
 jobs:
-  eslint:
-    name: Run eslint check
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v3
-
-      - name: Install dependencies
-        run: |
-          yarn install
-
-      - name: Run ESLint
-        run: yarn lint
-
   build:
     runs-on: ubuntu-22.04
 
     steps:
-      - name: Checkout repository
+      - name: Checkout Repository
         uses: actions/checkout@v3
 
       - name: Get full image name

.github/workflows/run-build-check.yml

@@ -0,0 +1,33 @@
+name: Run build check
+
+on:
+  push:
+    branches:
+      - 'main'
+    tags:
+      - 'v*'
+
+  pull_request:
+    types: [opened, synchronize, reopened]
+
+jobs:
+  build:
+    name: Run yarn build
+    runs-on: ubuntu-22.04
+
+    steps:
+      - name: Checkout Repository
+        uses: actions/checkout@v3
+
+      - name: Setup node
+        uses: actions/setup-node@v3
+        with:
+          node-version: 20
+
+      - name: Install dependencies
+        run: yarn install
+
+      - name: Build
+        run: yarn build
+
+

.github/workflows/run-eslint-check.yml

@@ -0,0 +1,26 @@
+name: Run ESlint check
+
+on:
+  push:
+    branches:
+      - 'main'
+    tags:
+      - 'v*'
+
+  pull_request:
+    types: [opened, synchronize, reopened]
+
+jobs:
+  eslint:
+    name: Run eslint check
+    runs-on: ubuntu-22.04
+    steps:
+      - name: Checkout Repository
+        uses: actions/checkout@v3
+
+      - name: Install dependencies
+        run: |
+          yarn install
+
+      - name: Run ESLint
+        run: yarn lint

.github/workflows/run-tests.yml

@@ -1,9 +1,14 @@
 name: Run tests
 
 on:
-  pull_request:
+  push:
     branches:
-      - '*'
+      - 'main'
+    tags:
+      - 'v*'
+
+  pull_request:
+    types: [opened, synchronize, reopened]
 
 jobs:
   tests:

migrations/tenant/0008-users@add-editor-tools-remove-extensions.sql

@@ -0,0 +1,22 @@
+
+-- Adds "editor_tools" column at "users" if not exists
+DO $$
+BEGIN
+  IF NOT EXISTS(SELECT *
+    FROM information_schema.columns
+    WHERE table_name='users' and column_name='editor_tools')
+  THEN
+    ALTER TABLE "public"."users" ADD COLUMN "editor_tools" jsonb;
+  END IF;
+END $$;
+
+-- Removes "extensions" column at "users" if exists
+DO $$
+BEGIN
+  IF EXISTS(SELECT *
+    FROM information_schema.columns
+    WHERE table_name='users' and column_name='extensions')
+  THEN
+    ALTER TABLE "public"."users" DROP COLUMN "extensions";
+  END IF;
+END $$;

migrations/tenant/0009-editor-tools@rename-camelcase-to-underscore.sql

@@ -0,0 +1,33 @@
+-- "editor_tools" table:
+
+-- Rename column "isDefault" to "is_default" if "isDefault" exists and "is_default" not exists
+DO $$
+BEGIN
+  IF EXISTS(SELECT *
+    FROM information_schema.columns
+    WHERE table_name='editor_tools' and column_name='isDefault')
+  THEN
+    IF NOT EXISTS(SELECT *
+      FROM information_schema.columns
+      WHERE table_name='editor_tools' and column_name='is_default')
+    THEN
+      ALTER TABLE "public"."editor_tools" RENAME COLUMN "isDefault" TO "is_default";
+    END IF;
+  END IF;
+END $$;
+
+-- Rename column "exportName" to "export_name" if "exportName" exists and "export_name" not exists
+DO $$
+BEGIN
+  IF EXISTS(SELECT *
+    FROM information_schema.columns
+    WHERE table_name='editor_tools' and column_name='exportName')
+  THEN
+    IF NOT EXISTS(SELECT *
+      FROM information_schema.columns
+      WHERE table_name='editor_tools' and column_name='export_name')
+    THEN
+      ALTER TABLE "public"."editor_tools" RENAME COLUMN "exportName" TO "export_name";
+    END IF;
+  END IF;
+END $$;

src/domain/entities/user.ts

@@ -1,4 +1,4 @@
-import type UserExtensions from '@domain/entities/userExtensions.js';
+import type EditorTool from './editorTools';
 
 /**
  * User entity
@@ -30,8 +30,7 @@ export default interface User {
   photo?: string;
 
   /**
-   * Custom plugins from the marketplace that improve
-   * editor or notes environment
+   * Custom plugins ids from the marketplace that improve editor or notes environment
    */
-  extensions?: UserExtensions;
+  editorTools?: EditorTool['id'][];
 }

src/domain/index.ts

@@ -63,10 +63,14 @@ export function init(repositories: Repositories, appConfig: AppConfig): DomainSe
     repositories.userSessionRepository
   );
 
-  const userService = new UserService(repositories.userRepository);
-
-  const aiService = new AIService(repositories.aiRepository);
   const editorToolsService = new EditorToolsService(repositories.editorToolsRepository);
+  const userService = new UserService(repositories.userRepository, {
+    editorTools: editorToolsService,
+    /**
+     * @todo find a way how to resolve circular dependency
+     */
+  });
+  const aiService = new AIService(repositories.aiRepository);
 
   return {
     noteService,

src/domain/service/editorTools.ts

@@ -1,11 +1,12 @@
 import type EditorToolsRepository from '@repository/editorTools.repository.js';
 import type EditorTool from '@domain/entities/editorTools.js';
 import { createEditorToolId } from '@infrastructure/utils/id.js';
+import type EditorToolsServiceSharedMethods from './shared/editorTools.js';
 
 /**
  * Editor tools service
  */
-export default class EditorToolsService {
+export default class EditorToolsService implements EditorToolsServiceSharedMethods {
   /**
    * User repository instance
    */
@@ -32,10 +33,17 @@ export default class EditorToolsService {
    *
    * @param editorToolIds - tool ids
    */
-  public async getToolsByIds(editorToolIds: EditorTool['id'][] ): Promise<EditorTool[] | null> {
+  public async getToolsByIds(editorToolIds: EditorTool['id'][]): Promise<EditorTool[]> {
     return await this.repository.getToolsByIds(editorToolIds);
   }
 
+  /**
+   * Return tools that are available at Editor by default
+   */
+  public async getDefaultTools(): Promise<EditorTool[]> {
+    return await this.repository.getDefaultTools();
+  }
+
   /**
    * Adding custom editor tool
    *

src/domain/service/shared/README.md

@@ -0,0 +1,30 @@
+# Shared Domain Services
+
+Sometimes you may want to call some domain method from other domain. We can inject them in constructor, but it creates a direct dependency.
+
+One way do decouple domains is to create a Shared Interfaces — domain will "expose" some public methods though it. You can call it "Contract".
+So dependant domain will depend on it instead of direct dependency.
+
+
+## Example
+
+```ts
+interface DomainASharedMethods {
+  someMethodA: () => void;
+}
+
+export type SharedDomainMethods = {
+  domainA: DomainASharedMethods;
+};
+
+
+class DomainA implements DomainASharedMethods {
+  public someMethodA (){}
+}
+
+class DomainB {
+  constructor(private readonly shared: SharedDomainMethods) {
+    this.shared.domainA.someMethodA(); // here we call method of Domain A, but without direct dependency
+  }
+}
+```

src/domain/service/shared/editorTools.ts

@@ -0,0 +1,19 @@
+import type EditorTool from '@domain/entities/editorTools';
+
+/**
+ * Which methods of Domain can be used by other domains
+ * Uses to decouple domains from each other
+ */
+export default interface  EditorToolsServiceSharedMethods {
+  /**
+   * Return tools that are available at Editor by default
+   */
+  getDefaultTools(): Promise<EditorTool[]>;
+
+  /**
+   * Get bunch of editor tools by their ids
+   *
+   * @param ids - tool ids to resolve
+   */
+  getToolsByIds(ids: EditorTool['id'][]): Promise<EditorTool[]>;
+}

src/domain/service/shared/index.ts

@@ -0,0 +1,5 @@
+import type EditorToolsServiceSharedMethods from './editorTools';
+
+export type SharedDomainMethods = {
+  editorTools: EditorToolsServiceSharedMethods;
+};

src/domain/service/user.ts

@@ -2,6 +2,7 @@ import type UserRepository from '@repository/user.repository.js';
 import { Provider } from '@repository/user.repository.js';
 import type User from '@domain/entities/user.js';
 import type EditorTool from '@domain/entities/editorTools';
+import type { SharedDomainMethods } from './shared/index.js';
 
 export {
   Provider
@@ -20,8 +21,9 @@ export default class UserService {
    * User service constructor
    *
    * @param repository - user repository instance
+   * @param shared - shared domains
    */
-  constructor(repository: UserRepository) {
+  constructor(repository: UserRepository, private readonly shared: SharedDomainMethods) {
     this.repository = repository;
   }
 
@@ -47,15 +49,28 @@ export default class UserService {
   }
 
   /**
-   * Get user extensions that contains only editoTools for now
-   * TODO: Simplify extenisons
+   * Get user editor tools ids
    *
    * @param userId - user unique identifier
    */
-  public async getUserExtensions(userId: User['id']): Promise<User['extensions']> {
+  public async getUserEditorTools(userId: User['id']): Promise<EditorTool[]> {
     const user = await this.getUserById(userId);
 
-    return user?.extensions ?? {};
+    if (user === null) {
+      throw new Error('User not found');
+    }
+
+    const userToolsIds = user.editorTools ?? [];
+    const defaultTools = await this.shared.editorTools.getDefaultTools();
+    const uniqueDefaultEditorTools = defaultTools.filter(({ id }) => !userToolsIds.includes(id));
+    const userTools = await this.shared.editorTools.getToolsByIds(userToolsIds) ?? [];
+
+    /**
+     * Combine user tools and default tools
+     *
+     * @todo load tools in notes service
+     */
+    return [...userTools, ...uniqueDefaultEditorTools];
   }
 
   /**
@@ -65,16 +80,14 @@ export default class UserService {
    */
   public async addUserEditorTool({
     userId,
-    editorToolId,
+    toolId,
   }: {
     userId: User['id'],
-    editorToolId: EditorTool['id'],
+    toolId: EditorTool['id'],
   }): Promise<void> {
     return await this.repository.addUserEditorTool({
       userId,
-      tool: {
-        id: editorToolId,
-      },
+      toolId,
     });
   }
 }