refactor: added schema for delete tool, added some comments, changed …

…response object
codex-team · Feb 23, 2024 · 5b4e277 · 5b4e277
1 parent 4ab2ab3
commit 5b4e277
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 2 deletions.
diff --git a/src/presentation/http/router/user.ts b/src/presentation/http/router/user.ts
@@ -152,6 +152,20 @@ const UserRouter: FastifyPluginCallback<UserRouterOptions> = (fastify, opts, don
           description: 'Unique editor tool id',
         },
       },
+      response: {
+        '2xx': {
+          description: 'Removed editor tool id',
+          content: {
+            'application/json': {
+              schema: {
+                removeId: {
+                  type: 'string',
+                },
+              },
+            },
+          },
+        },
+      },
     },
   }, async (request, reply) => {
     const toolId = request.body.toolId;
@@ -163,7 +177,7 @@ const UserRouter: FastifyPluginCallback<UserRouterOptions> = (fastify, opts, don
     });
 
     return reply.send({
-      data: toolId,
+      removeId: toolId,
     });
   });
 

diff --git a/src/repository/storage/postgres/orm/sequelize/user.ts b/src/repository/storage/postgres/orm/sequelize/user.ts
@@ -176,7 +176,12 @@ export default class UserSequelizeStorage {
     toolId,
   }: AddUserToolOptions): Promise<void> {
     await this.model.update({
-      editorTools: literal(`COALESCE(editor_tools, '[]'::jsonb) || '["${toolId}"]'::jsonb`),
+      editorTools: literal(
+        /**
+         * If editorTools is null, then set it to empty array
+         * Then add the tool to the list
+         */
+        `COALESCE(editor_tools, '[]'::jsonb) || '["${toolId}"]'::jsonb`),
     }, {
       where: {
         id: userId,