871-refactor: Refactor youtube api (#872)

Quiddlee · web-flow · commit 8734dfac7699 · 2025-05-21T23:03:12.000+03:00
* refactor: 871 - move youtube api to separate domain

* chore: 871 - remove redundant env variable type

* refactor: 871 - move business logic to store

* fix: 871 - function name typo

* chore: 871 - change to not log in prod
diff --git a/.env.example b/.env.example
@@ -1,7 +1,10 @@
-API_BASE_URL=https://cdn.rs.school
-YOUTUBE_API_KEY=YOUTUBE_API_KEY
+NEXT_PUBLIC_API_BASE_URL=https://cdn.rs.school
 LOG_QUERY=true
 
+# YouTube
+NEXT_PUBLIC_YOUTUBE_API_BASE_URL=https://www.googleapis.com/youtube/v3
+NEXT_PUBLIC_YOUTUBE_API_KEY=YOUTUBE_API_KEY
+
 # Contentful
 CONTENTFUL_SPACE_ID=CONTENTFUL_SPACE_ID
 CONTENTFUL_MANAGEMENT_TOKEN=CONTENTFUL_MANAGEMENT_TOKEN
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -8,8 +8,9 @@ on:
 
 env:
   NODE_VERSION: 20.x
-  API_BASE_URL: ${{ secrets.API_BASE_URL }}
-  YOUTUBE_API_KEY: ${{ secrets.YOUTUBE_API_KEY_DEVELOPMENT }}
+  NEXT_PUBLIC_API_BASE_URL: ${{ secrets.API_BASE_URL }}
+  NEXT_PUBLIC_YOUTUBE_API_BASE_URL: ${{ secrets.YOUTUBE_API_BASE_URL }}
+  NEXT_PUBLIC_YOUTUBE_API_KEY: ${{ secrets.YOUTUBE_API_KEY_DEVELOPMENT }}
 
 jobs:
   ci:
diff --git a/.github/workflows/preview-create.yml b/.github/workflows/preview-create.yml
@@ -15,8 +15,9 @@ env:
   AWS_ACCESS_KEY_ID: ${{ secrets.DEPLOY_AWS_ACCESS_KEY_ID }}
   AWS_SECRET_ACCESS_KEY: ${{ secrets.DEPLOY_AWS_SECRET_ACCESS_KEY }}
   AWS_REGION: 'eu-central-1'
-  API_BASE_URL: ${{ secrets.API_BASE_URL }}
-  YOUTUBE_API_KEY: ${{ secrets.YOUTUBE_API_KEY_DEVELOPMENT }}
+  NEXT_PUBLIC_API_BASE_URL: ${{ secrets.API_BASE_URL }}
+  NEXT_PUBLIC_YOUTUBE_API_BASE_URL: ${{ secrets.YOUTUBE_API_BASE_URL }}
+  NEXT_PUBLIC_YOUTUBE_API_KEY: ${{ secrets.YOUTUBE_API_KEY_DEVELOPMENT }}
 
 jobs:
   build-rs-school:
diff --git a/.github/workflows/production.yml b/.github/workflows/production.yml
@@ -20,8 +20,9 @@ env:
   AWS_ACCESS_KEY_ID: ${{ secrets.DEPLOY_AWS_ACCESS_KEY_ID }}
   AWS_SECRET_ACCESS_KEY: ${{ secrets.DEPLOY_AWS_SECRET_ACCESS_KEY }}
   AWS_REGION: 'eu-central-1'
-  API_BASE_URL: ${{ secrets.API_BASE_URL }}
-  YOUTUBE_API_KEY: ${{ secrets.YOUTUBE_API_KEY_PRODUCTION }}
+  NEXT_PUBLIC_API_BASE_URL: ${{ secrets.API_BASE_URL }}
+  NEXT_PUBLIC_YOUTUBE_API_BASE_URL: ${{ secrets.YOUTUBE_API_BASE_URL }}
+  NEXT_PUBLIC_YOUTUBE_API_KEY: ${{ secrets.YOUTUBE_API_KEY_PRODUCTION }}
 
 jobs:
   rs-school:
diff --git a/.github/workflows/visual-testing-on-comment.yml b/.github/workflows/visual-testing-on-comment.yml
@@ -7,8 +7,9 @@ on:
 
 env:
   NODE_VERSION: 20.x
-  API_BASE_URL: ${{ secrets.API_BASE_URL }}
-  YOUTUBE_API_KEY: ${{ secrets.YOUTUBE_API_KEY_DEVELOPMENT }}
+  NEXT_PUBLIC_API_BASE_URL: ${{ secrets.API_BASE_URL }}
+  NEXT_PUBLIC_YOUTUBE_API_BASE_URL: ${{ secrets.YOUTUBE_API_BASE_URL }}
+  NEXT_PUBLIC_YOUTUBE_API_KEY: ${{ secrets.YOUTUBE_API_KEY_DEVELOPMENT }}
 
 jobs:
   run-visial-testing:
diff --git a/.github/workflows/visual-testing-on-push.yml b/.github/workflows/visual-testing-on-push.yml
@@ -6,8 +6,9 @@ on:
 
 env:
   NODE_VERSION: 20.x
-  API_BASE_URL: ${{ secrets.API_BASE_URL }}
-  YOUTUBE_API_KEY: ${{ secrets.YOUTUBE_API_KEY_DEVELOPMENT }}
+  NEXT_PUBLIC_API_BASE_URL: ${{ secrets.API_BASE_URL }}
+  NEXT_PUBLIC_YOUTUBE_API_BASE_URL: ${{ secrets.YOUTUBE_API_BASE_URL }}
+  NEXT_PUBLIC_YOUTUBE_API_KEY: ${{ secrets.YOUTUBE_API_KEY_DEVELOPMENT }}
 
 jobs:
   visial-testing:
diff --git a/env.d.ts b/env.d.ts
@@ -1,7 +1,9 @@
 declare namespace NodeJS {
   interface ProcessEnv {
-    YOUTUBE_API_KEY: string;
-    API_BASE_URL: string;
+    API_URL: string;
+    NEXT_PUBLIC_API_BASE_URL: string;
+    NEXT_PUBLIC_YOUTUBE_API_BASE_URL: string;
+    NEXT_PUBLIC_YOUTUBE_API_KEY: string;
     LOG_QUERY: string;
   }
 }
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -46,7 +46,8 @@
     "remark-rehype": "^11.1.2",
     "remark-remove-comments": "^1.1.1",
     "remark-toc": "^9.0.0",
-    "swiper": "^11.2.6"
+    "swiper": "^11.2.6",
+    "swr": "^2.3.3"
   },
   "devDependencies": {
     "@eslint/js": "^9.25.1",
diff --git a/src/core/api/app-api.ts b/src/core/api/app-api.ts
@@ -1,4 +1,5 @@
 import { CourseApi } from '@/entities/course/api/course-api';
+import { MentorApi } from '@/entities/mentor/api/mentor-api';
 import { TrainerApi } from '@/entities/trainer/api/trainer-api';
 import { ApiBaseClass } from '@/shared/api/api-base-class';
 import { ApiServices } from '@/shared/types';
@@ -8,11 +9,19 @@ export class Api {
 
   public readonly trainer: TrainerApi;
   public readonly course: CourseApi;
+  public readonly mentor: MentorApi;
 
-  constructor(private readonly baseURI: string) {
-    this.services = { rest: new ApiBaseClass(this.baseURI) };
+  constructor(
+    private readonly baseURI: string,
+    private readonly youtubeBaseURI: string,
+  ) {
+    this.services = {
+      rest: new ApiBaseClass(this.baseURI),
+      youtube: new ApiBaseClass(this.youtubeBaseURI),
+    };
 
     this.trainer = new TrainerApi(this.services);
     this.course = new CourseApi(this.services);
+    this.mentor = new MentorApi(this.services);
   }
 }
diff --git a/src/entities/mentor/api/mentor-api.ts b/src/entities/mentor/api/mentor-api.ts
@@ -0,0 +1,19 @@
+import { MENTORSHIP_YOUTUBE_PLAYLIST_ID } from '../constants';
+import { MentorPlaylistResponse } from '../types';
+import { YOUTUBE_API_MAX_RESULTS_PER_PAGE } from '@/shared/constants';
+import { ApiServices } from '@/shared/types/';
+
+export class MentorApi {
+  constructor(private readonly services: ApiServices) {}
+
+  public queryMentorPlaylist() {
+    return this.services.youtube.get<MentorPlaylistResponse>(`/playlistItems`, {
+      params: {
+        part: ['snippet', 'status'],
+        maxResults: YOUTUBE_API_MAX_RESULTS_PER_PAGE,
+        key: process.env.NEXT_PUBLIC_YOUTUBE_API_KEY,
+        playlistId: MENTORSHIP_YOUTUBE_PLAYLIST_ID,
+      },
+    });
+  }
+}
diff --git a/src/entities/mentor/constants.ts b/src/entities/mentor/constants.ts
@@ -0,0 +1 @@
+export const MENTORSHIP_YOUTUBE_PLAYLIST_ID = 'PLzLiprpVuH8f7Jg8pgZUCeTN-Q6uVZNhg';
diff --git a/src/entities/mentor/helpers/transform-mentor-videos.ts b/src/entities/mentor/helpers/transform-mentor-videos.ts
@@ -0,0 +1,9 @@
+import type { Video } from '@/shared/types';
+
+export function transformMentorVideos(videos: GoogleApiYouTubePlaylistItemResource[]): Video[] {
+  return videos.map((item) => ({
+    id: item.snippet.resourceId.videoId,
+    title: item.snippet.title,
+    thumbnail: item.snippet.thumbnails.medium.url,
+  }));
+}
diff --git a/src/entities/mentor/index.ts b/src/entities/mentor/index.ts
@@ -1,2 +1,3 @@
 export type { MentorFeedback } from './types';
 export { MentorFeedbackCard } from './ui/mentor-feedback-card/mentor-feedback-card';
+export { mentorStore } from './model/store';
diff --git a/src/entities/mentor/model/store.ts b/src/entities/mentor/model/store.ts
@@ -0,0 +1,20 @@
+import { transformMentorVideos } from '@/entities/mentor/helpers/transform-mentor-videos';
+import { api } from '@/shared/api/api';
+import { filterYoutubePrivateVideos } from '@/shared/helpers/filter-youtube-private-videos';
+
+export class MentorStore {
+  public loadYoutubePlaylist = async () => {
+    const res = await api.mentor.queryMentorPlaylist();
+
+    if (res.isSuccess) {
+      const publicVideos = filterYoutubePrivateVideos(res.result);
+      const videoItems = transformMentorVideos(publicVideos);
+
+      return videoItems;
+    }
+
+    throw new Error('Error while loading mentor playlist.');
+  };
+}
+
+export const mentorStore = new MentorStore();
diff --git a/src/entities/mentor/types.ts b/src/entities/mentor/types.ts
@@ -6,3 +6,7 @@ export type MentorFeedback = {
   review: string;
   photo: StaticImageData;
 };
+
+export type MentorPlaylistResponse = {
+  items: GoogleApiYouTubePlaylistItemResource[];
+};
diff --git a/src/shared/api/api-base-class.ts b/src/shared/api/api-base-class.ts
@@ -122,6 +122,6 @@ export class ApiBaseClass {
     const urlWithQueryString = `${url}?${queryString}`;
     const baseUrl = !isValidUrl(url) ? this.baseUrl : '';
 
-    return new URL(urlWithQueryString, baseUrl).toString();
+    return `${baseUrl}${urlWithQueryString}`;
   }
 }
diff --git a/src/shared/api/api.ts b/src/shared/api/api.ts
@@ -1,3 +1,6 @@
 import { Api } from '@/core/api/app-api';
 
-export const api = new Api(process.env.API_BASE_URL);
+export const api = new Api(
+  process.env.NEXT_PUBLIC_API_BASE_URL,
+  process.env.NEXT_PUBLIC_YOUTUBE_API_BASE_URL,
+);
diff --git a/src/shared/constants.ts b/src/shared/constants.ts
@@ -7,6 +7,7 @@ export const TO_BE_DETERMINED = 'TBD';
 export const REGISTRATION_WILL_OPEN_SOON = 'Registration will open soon!';
 export const REGISTRATION_WILL_OPEN_SOON_RU = 'Регистрация откроется скоро!';
 export const UNKNOWN_API_ERROR = 'Unknown error, API request failed.';
+export const YOUTUBE_API_MAX_RESULTS_PER_PAGE = 50;
 
 /**
  * https://www.contentful.com/developers/docs/references/content-preview-api/#/reference/links
@@ -106,3 +107,5 @@ export const ROUTES = {
   DOCS_RU: 'docs/ru',
   NOT_FOUND: '*',
 } as const;
+
+export const SWR_CACHE_KEY = { MENTORS_PLAYLIST: 'MENTORS_PLAYLIST' };
diff --git a/src/shared/helpers/filter-youtube-private-videos.ts b/src/shared/helpers/filter-youtube-private-videos.ts
@@ -0,0 +1,6 @@
+import { MentorPlaylistResponse } from '@/entities/mentor/types';
+import { videoPrivacyStatus } from '@/shared/ui/video-playlist-with-player/constants';
+
+export function filterYoutubePrivateVideos(videos: MentorPlaylistResponse) {
+  return videos.items.filter((item) => item.status.privacyStatus === videoPrivacyStatus.public);
+}
diff --git a/src/shared/helpers/log-request.ts b/src/shared/helpers/log-request.ts
@@ -1,4 +1,4 @@
-import util from 'node:util';
+import { inspect } from 'util';
 
 import { stringifyJSONSafe } from '@/shared/helpers/stringify-json-safe';
 import { HttpMethod } from '@/shared/types';
@@ -26,7 +26,7 @@ export function logRequest({
   body,
   error,
 }: LogRequestParams) {
-  if (process.env.LOG_QUERY === 'false') {
+  if (process.env.NODE_ENV === 'production' || process.env.LOG_QUERY === 'false') {
     return;
   }
 
@@ -57,9 +57,9 @@ export function logRequest({
   }
 
   console.log(
-    util.inspect(logObject, {
+    inspect(logObject, {
       depth: null,
-      colors: process.env.NODE_ENV !== 'production',
+      colors: true,
     }),
   );
 }
diff --git a/src/shared/types/types.ts b/src/shared/types/types.ts
@@ -17,6 +17,7 @@ export type Video = {
 
 export type ApiServices = {
   rest: ApiBaseClass;
+  youtube: ApiBaseClass;
 };
 
 export type HttpMethod = (typeof HTTP_METHOD)[keyof typeof HTTP_METHOD];
diff --git a/src/shared/ui/video-playlist-with-player/video-playlist-with-player.test.tsx b/src/shared/ui/video-playlist-with-player/video-playlist-with-player.test.tsx
diff --git a/src/shared/ui/video-playlist-with-player/video-playlist-with-player.tsx b/src/shared/ui/video-playlist-with-player/video-playlist-with-player.tsx
diff --git a/src/widgets/mentors-feedback/ui/mentors-feedback.tsx b/src/widgets/mentors-feedback/ui/mentors-feedback.tsx
diff --git a/src/widgets/mentors-feedback/ui/mentors-playlist/mentor-talks-video-playlist-with-player.tsx b/src/widgets/mentors-feedback/ui/mentors-playlist/mentor-talks-video-playlist-with-player.tsx

Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,9 @@`
`1`	`1`	`declare namespace NodeJS {`
`2`	`2`	`interface ProcessEnv {`
`3`		`- YOUTUBE_API_KEY: string;`
`4`		`- API_BASE_URL: string;`
	`3`	`+ API_URL: string;`
	`4`	`+ NEXT_PUBLIC_API_BASE_URL: string;`
	`5`	`+ NEXT_PUBLIC_YOUTUBE_API_BASE_URL: string;`
	`6`	`+ NEXT_PUBLIC_YOUTUBE_API_KEY: string;`
`5`	`7`	`LOG_QUERY: string;`
`6`	`8`	`}`
`7`	`9`	`}`