feat: remove POST /api/v1/generate-reports endpoint

This now can be achieved by running `curl -X POST http://localhost:3000/api/v1/workflow/generate-reports/run   -H "Content-Type: application/json"`

Author: UlisesGascon

__tests__/httpServer/apiV1.test.js

@@ -20,7 +20,6 @@ jest.mock('../../src/cli/workflows', () => ({
 }))
 
 const request = require('supertest')
-const { generateStaticReports } = require('../../src/reports')
 const knexInit = require('knex')
 const { getConfig } = require('../../src/config')
 const { resetDatabase, initializeStore } = require('../../__utils__')
@@ -223,42 +222,4 @@ describe('HTTP Server API V1', () => {
 
     test.todo('should return 500 when workflow execution times out')
   })
-
-  describe('POST /api/v1/generate-reports', () => {
-    test('should return status completed when report generation succeeds', async () => {
-      generateStaticReports.mockResolvedValueOnce()
-
-      const response = await app.post('/api/v1/generate-reports')
-
-      expect(generateStaticReports).toHaveBeenCalledWith(expect.anything(), { clearPreviousReports: true })
-      expect(response.status).toBe(202)
-      expect(response.body).toHaveProperty('status', 'completed')
-      expect(response.body).toHaveProperty('startedAt')
-      expect(response.body).toHaveProperty('finishedAt')
-
-      const startedAt = new Date(response.body.startedAt)
-      const finishedAt = new Date(response.body.finishedAt)
-      expect(startedAt.toISOString()).toBe(response.body.startedAt)
-      expect(finishedAt.toISOString()).toBe(response.body.finishedAt)
-      expect(finishedAt.getTime()).toBeGreaterThanOrEqual(startedAt.getTime())
-    })
-
-    test('should return status failed when report generation fails', async () => {
-      generateStaticReports.mockRejectedValueOnce(new Error('Report generation failed'))
-
-      const response = await app.post('/api/v1/generate-reports')
-
-      expect(generateStaticReports).toHaveBeenCalledWith(expect.anything(), { clearPreviousReports: true })
-      expect(response.status).toBe(500)
-      expect(response.body).toHaveProperty('status', 'failed')
-      expect(response.body).toHaveProperty('startedAt')
-      expect(response.body).toHaveProperty('finishedAt')
-
-      const startedAt = new Date(response.body.startedAt)
-      const finishedAt = new Date(response.body.finishedAt)
-      expect(startedAt.toISOString()).toBe(response.body.startedAt)
-      expect(finishedAt.toISOString()).toBe(response.body.finishedAt)
-      expect(finishedAt.getTime()).toBeGreaterThanOrEqual(startedAt.getTime())
-    })
-  })
 })

src/httpServer/routers/apiV1.js

@@ -1,4 +1,3 @@
-const { generateStaticReports } = require('../../reports')
 const pkg = require('../../../package.json')
 const { logger } = require('../../utils')
 const { initializeStore } = require('../../store')
@@ -105,24 +104,6 @@ function createApiRouter (knex, express) {
     }
   })
 
-  router.post('/generate-reports', async (req, res) => {
-    const startTs = new Date().toISOString()
-    try {
-      await generateStaticReports(knex, { clearPreviousReports: true })
-      res.status(202).json({
-        status: 'completed',
-        startedAt: startTs,
-        finishedAt: new Date().toISOString()
-      })
-    } catch (error) {
-      logger.error(error)
-      res.status(500).json({
-        status: 'failed',
-        startedAt: startTs,
-        finishedAt: new Date().toISOString()
-      })
-    }
-  })
   return router
 }
 

src/httpServer/swagger/api-v1.yml

@@ -197,62 +197,6 @@ paths:
             application/json:
               schema:
                 $ref: '#/components/schemas/ErrorResponse'
-  /api/v1/generate-reports:
-    post:
-      summary: Generate static reports
-      description: Triggers the generation of static reports
-      operationId: generateReports
-      tags:
-        - Reports
-      responses:
-        '202':
-          description: Report generation accepted
-          content:
-            application/json:
-              schema:
-                type: object
-                additionalProperties: false
-                properties:
-                  status:
-                    type: string
-                    enum: [completed, failed]
-                    example: completed
-                  startedAt:
-                    type: string
-                    format: date-time
-                    example: '2025-05-03T07:20:16.000Z'
-                  finishedAt:
-                    type: string
-                    format: date-time
-                    example: '2025-05-03T07:20:20.000Z'
-                required:
-                  - status
-                  - startedAt
-                  - finishedAt
-        '500':
-          description: Report generation failed
-          content:
-            application/json:
-              schema:
-                type: object
-                additionalProperties: false
-                properties:
-                  status:
-                    type: string
-                    enum: [failed]
-                    example: failed
-                  startedAt:
-                    type: string
-                    format: date-time
-                    example: '2025-05-03T07:20:16.000Z'
-                  finishedAt:
-                    type: string
-                    format: date-time
-                    example: '2025-05-03T07:20:20.000Z'
-                required:
-                  - status
-                  - startedAt
-                  - finishedAt
 
 components:
   schemas: