⚡ Bolt: [performance improvement] avoid intermediate array allocation in string extractions

src/helpers/creator.helper.ts

@@ -2,7 +2,7 @@ import { HTMLElement } from 'node-html-parser';
 import { CSFDCreatorScreening } from '../dto/creator';
 import { CSFDColorRating } from '../dto/global';
 import { CSFDColors } from '../dto/user-ratings';
-import { addProtocol, parseColor, parseDate, parseIdFromUrl } from './global.helper';
+import { addProtocol, getFirstLine, parseColor, parseDate, parseIdFromUrl } from './global.helper';
 
 const getCreatorColorRating = (el: HTMLElement | null): CSFDColorRating => {
   const classes: string[] = el?.classNames.split(' ') ?? [];
@@ -42,7 +42,7 @@ export const getCreatorBirthdayInfo = (
 
 export const getCreatorBio = (el: HTMLElement | null): string | null => {
   const p = el?.querySelector('.article-content p');
-  const first = p?.text?.trim().split('\n')[0]?.trim();
+  const first = getFirstLine(p?.text?.trim())?.trim();
   return first || null;
 };
 

src/helpers/global.helper.ts

@@ -145,3 +145,23 @@ export const parseDate = (date: string): string | null => {
 
 // Sleep in loop
 export const sleep = (ms: number) => new Promise((res) => setTimeout(res, ms));
+
+/**
+ * Performance optimization: Extract the last word from a string efficiently
+ * without intermediate array allocations (avoids .split(' ').pop()).
+ */
+export const getLastWord = (text: string | null | undefined, delimiter: string = ' '): string => {
+  if (!text) return '';
+  const lastIndex = text.lastIndexOf(delimiter);
+  return lastIndex !== -1 ? text.substring(lastIndex + delimiter.length) : text;
+};
+
+/**
+ * Performance optimization: Extract the first line from a string efficiently
+ * without intermediate array allocations (avoids .split('\n')[0]).
+ */
+export const getFirstLine = (text: string | null | undefined): string => {
+  if (!text) return '';
+  const index = text.indexOf('\n');
+  return index !== -1 ? text.substring(0, index) : text;
+};

src/helpers/movie.helper.ts

@@ -21,6 +21,7 @@ import type { CSFDOptions } from '../types';
 import {
   addProtocol,
   getColor,
+  getFirstLine,
   parseDate,
   parseFilmType,
   parseISO8601Duration,
@@ -219,7 +220,7 @@ export const getMovieTitlesOther = (el: HTMLElement): CSFDTitlesOther[] => {
 
   const titlesOther = namesNode.map((el) => {
     const country = el.querySelector('img.flag').attributes.alt;
-    const title = el.textContent.trim().split('\n')[0];
+    const title = getFirstLine(el.textContent.trim());
 
     if (country && title) {
       return {
@@ -434,7 +435,7 @@ export const getMovieGroup = (
 
 export const getMovieType = (el: HTMLElement): CSFDFilmTypes => {
   const type = el.querySelector('.film-header-name .type');
-  const text = type?.innerText?.replace(/[{()}]/g, '').split('\n')[0].trim() || 'film';
+  const text = getFirstLine(type?.innerText?.replace(/[{()}]/g, ''))?.trim() || 'film';
   return parseFilmType(text);
 };
 

src/helpers/search.helper.ts

@@ -2,7 +2,7 @@ import { HTMLElement } from 'node-html-parser';
 import { CSFDColorRating, CSFDFilmTypes } from '../dto/global';
 import { CSFDMovieCreator } from '../dto/movie';
 import { CSFDColors } from '../dto/user-ratings';
-import { addProtocol, parseColor, parseFilmType, parseIdFromUrl } from './global.helper';
+import { addProtocol, getLastWord, parseColor, parseFilmType, parseIdFromUrl } from './global.helper';
 
 type Creator = 'Režie:' | 'Hrají:';
 
@@ -26,7 +26,7 @@ export const getSearchUrl = (el: HTMLElement): string => {
 
 export const getSearchColorRating = (el: HTMLElement): CSFDColorRating => {
   return parseColor(
-    el.querySelector('.article-header i.icon').classNames.split(' ').pop() as CSFDColors
+    getLastWord(el.querySelector('.article-header i.icon').classNames) as CSFDColors
   );
 };
 

src/helpers/user-ratings.helper.ts

@@ -1,17 +1,17 @@
 import { HTMLElement } from 'node-html-parser';
 import { CSFDColorRating, CSFDFilmTypes, CSFDStars } from '../dto/global';
 import { CSFDColors } from '../dto/user-ratings';
-import { parseColor, parseDate, parseFilmType, parseIdFromUrl } from './global.helper';
+import { getLastWord, parseColor, parseDate, parseFilmType, parseIdFromUrl } from './global.helper';
 
 export const getUserRatingId = (el: HTMLElement): number => {
   const url = el.querySelector('td.name .film-title-name').attributes.href;
   return parseIdFromUrl(url);
 };
 
 export const getUserRating = (el: HTMLElement): CSFDStars => {
-  const ratingText = el.querySelector('td.star-rating-only .stars').classNames.split(' ').pop();
+  const ratingText = getLastWord(el.querySelector('td.star-rating-only .stars').classNames);
 
-  const rating = ratingText.includes('stars-') ? +ratingText.split('-').pop() : 0;
+  const rating = ratingText.includes('stars-') ? +getLastWord(ratingText, '-') : 0;
   return rating as CSFDStars;
 };
 
@@ -31,7 +31,7 @@ export const getUserRatingYear = (el: HTMLElement): number | null => {
 
 export const getUserRatingColorRating = (el: HTMLElement): CSFDColorRating => {
   const color = parseColor(
-    el.querySelector('td.name .icon').classNames.split(' ').pop() as CSFDColors
+    getLastWord(el.querySelector('td.name .icon').classNames) as CSFDColors
   );
   return color;
 };

src/helpers/user-reviews.helper.ts

@@ -1,17 +1,17 @@
 import { HTMLElement } from 'node-html-parser';
 import { CSFDColorRating, CSFDFilmTypes, CSFDStars } from '../dto/global';
 import { CSFDColors } from '../dto/user-ratings';
-import { parseColor, parseDate, parseFilmType, parseIdFromUrl } from './global.helper';
+import { getLastWord, parseColor, parseDate, parseFilmType, parseIdFromUrl } from './global.helper';
 
 export const getUserReviewId = (el: HTMLElement): number => {
   const url = el.querySelector('.film-title-name').attributes.href;
   return parseIdFromUrl(url);
 };
 
 export const getUserReviewRating = (el: HTMLElement): CSFDStars => {
-  const ratingText = el.querySelector('.star-rating .stars').classNames.split(' ').pop();
+  const ratingText = getLastWord(el.querySelector('.star-rating .stars').classNames);
 
-  const rating = ratingText.includes('stars-') ? +ratingText.split('-').pop() : 0;
+  const rating = ratingText.includes('stars-') ? +getLastWord(ratingText, '-') : 0;
   return rating as CSFDStars;
 };
 
@@ -32,7 +32,7 @@ export const getUserReviewYear = (el: HTMLElement): number | null => {
 
 export const getUserReviewColorRating = (el: HTMLElement): CSFDColorRating => {
   const icon = el.querySelector('.film-title-inline i.icon');
-  const color = parseColor(icon?.classNames.split(' ').pop() as CSFDColors);
+  const color = parseColor(getLastWord(icon?.classNames) as CSFDColors);
   return color;
 };