Merge pull request #67 from TypingMind/refactor-youtube-transcript

refactor(youtube transcript):  Refactor OpenAPI document for Youtube Transcript

Author: dat-devuap

src/api-docs/openAPIDocumentGenerator.ts

@@ -4,12 +4,12 @@ import { healthCheckRegistry } from '@/routes/healthCheck/healthCheckRouter';
 import { powerpointGeneratorRegistry } from '@/routes/powerpointGenerator/powerpointGeneratorRouter';
 import { articleReaderRegistry } from '@/routes/webPageReader/webPageReaderRouter';
 import { wordGeneratorRegistry } from '@/routes/wordGenerator/wordGeneratorRouter';
-import { transcriptRegistry } from '@/routes/youtubeTranscript/transcriptRouter';
+import { youtubeTranscriptRegistry } from '@/routes/youtubeTranscript/youtubeTranscriptRouter';
 
 export function generateOpenAPIDocument() {
   const registry = new OpenAPIRegistry([
     healthCheckRegistry,
-    transcriptRegistry,
+    youtubeTranscriptRegistry,
     articleReaderRegistry,
     powerpointGeneratorRegistry,
     wordGeneratorRegistry,

src/routes/webPageReader/webPageReaderRouter.ts

@@ -67,12 +67,12 @@ export const webPageReaderRouter: Router = (() => {
     path: '/web-page-reader/get-content',
     tags: ['Web Page Reader'],
     request: {
-      params: WebPageReaderRequestParamSchema,
+      query: WebPageReaderRequestParamSchema,
     },
     responses: createApiResponse(WebPageReaderResponseSchema, 'Success'),
   });
 
-  router.get('/', async (_req: Request, res: Response) => {
+  router.get('/get-content', async (_req: Request, res: Response) => {
     const { url } = _req.query;
 
     if (typeof url !== 'string') {

src/routes/youtubeTranscript/transcriptModel.ts

@@ -0,0 +1,14 @@
+import { extendZodWithOpenApi } from '@asteasolutions/zod-to-openapi';
+import { z } from 'zod';
+
+extendZodWithOpenApi(z);
+
+export type YoutubeTranscriptResponse = z.infer<typeof YoutubeTranscriptResponseSchema>;
+export const YoutubeTranscriptResponseSchema = z.object({
+  textOnly: z.string(),
+});
+
+export type YoutubeTranscriptRequestParam = z.infer<typeof YoutubeTranscriptRequestParamSchema>;
+export const YoutubeTranscriptRequestParamSchema = z.object({
+  videoId: z.string().describe('The id of the Youtube video to retrieve the transcript'),
+});

src/routes/youtubeTranscript/youtubeTranscriptModel.ts

@@ -7,22 +7,25 @@ import { createApiResponse } from '@/api-docs/openAPIResponseBuilders';
 import { ResponseStatus, ServiceResponse } from '@/common/models/serviceResponse';
 import { handleServiceResponse } from '@/common/utils/httpHandlers';
 
-import { TranscriptSchema } from './transcriptModel';
+import { YoutubeTranscriptRequestParamSchema, YoutubeTranscriptResponseSchema } from './youtubeTranscriptModel';
 
-export const transcriptRegistry = new OpenAPIRegistry();
-transcriptRegistry.register('Transcript', TranscriptSchema);
+export const youtubeTranscriptRegistry = new OpenAPIRegistry();
+youtubeTranscriptRegistry.register('YoutubeTranscript', YoutubeTranscriptResponseSchema);
 
-export const transcriptRouter: Router = (() => {
+export const youtubeTranscriptRouter: Router = (() => {
   const router = express.Router();
 
-  transcriptRegistry.registerPath({
+  youtubeTranscriptRegistry.registerPath({
     method: 'get',
-    path: '/transcript',
+    path: '/youtube-transcript/get-transcript',
     tags: ['Youtube Transcript'],
-    responses: createApiResponse(TranscriptSchema, 'Success'),
+    request: {
+      query: YoutubeTranscriptRequestParamSchema,
+    },
+    responses: createApiResponse(YoutubeTranscriptResponseSchema, 'Success'),
   });
 
-  router.get('/', async (_req: Request, res: Response) => {
+  router.get('/get-transcript', async (_req: Request, res: Response) => {
     const { videoId } = _req.query;
 
     if (!videoId) {

…es/youtubeTranscript/transcriptRouter.ts …ubeTranscript/youtubeTranscriptRouter.tssrc/routes/youtubeTranscript/transcriptRouter.ts renamed to src/routes/youtubeTranscript/youtubeTranscriptRouter.ts src/routes/youtubeTranscript/transcriptRouter.ts renamed to src/routes/youtubeTranscript/youtubeTranscriptRouter.ts

@@ -13,7 +13,7 @@ import { healthCheckRouter } from '@/routes/healthCheck/healthCheckRouter';
 import { powerpointGeneratorRouter } from './routes/powerpointGenerator/powerpointGeneratorRouter';
 import { webPageReaderRouter } from './routes/webPageReader/webPageReaderRouter';
 import { wordGeneratorRouter } from './routes/wordGenerator/wordGeneratorRouter';
-import { transcriptRouter } from './routes/youtubeTranscript/transcriptRouter';
+import { youtubeTranscriptRouter } from './routes/youtubeTranscript/youtubeTranscriptRouter';
 const logger = pino({ name: 'server start' });
 const app: Express = express();
 
@@ -36,7 +36,7 @@ app.use(requestLogger());
 // Routes
 app.use('/health-check', healthCheckRouter);
 app.use('/images', express.static('public/images'));
-app.use('/transcript', transcriptRouter);
+app.use('/youtube-transcript', youtubeTranscriptRouter);
 app.use('/web-page-reader', webPageReaderRouter);
 app.use('/powerpoint-generator', powerpointGeneratorRouter);
 app.use('/word-generator', wordGeneratorRouter);