feat: script should support more than 1 local git repository gf-589 (#589)

- moved helpers into src/libs/helpers
- changed functions to arrow functions

Author: Artem Zahorodniuk

scripts/analytics/src/modules/analytics/analytics.service.ts

@@ -8,6 +8,7 @@ import {
 	EMPTY_LENGTH,
 	FIRST_ARRAY_INDEX,
 } from "./libs/constants/constants.js";
+import { mergeStats } from "./libs/helpers/helpers.js";
 import {
 	type ActivityLogCreateItemRequestDto,
 	type CommitStatistics,
@@ -122,50 +123,4 @@ class AnalyticsService {
 	}
 }
 
-function mergeStats(
-	statsAll: ActivityLogCreateItemRequestDto[],
-): ActivityLogCreateItemRequestDto[] {
-	return mergeByCriteria(
-		statsAll,
-		(item1, item2) => item1.date === item2.date,
-		(mergedItem, item) =>
-			(mergedItem.items = mergeStatsItems([
-				...mergedItem.items,
-				...item.items,
-			])),
-	);
-}
-
-function mergeStatsItems(items: CommitStatistics[]): CommitStatistics[] {
-	return mergeByCriteria(
-		items,
-		(item1, item2) =>
-			item1.authorEmail === item2.authorEmail &&
-			item1.authorName === item2.authorName,
-		(mergedItem, item) => (mergedItem.commitsNumber += item.commitsNumber),
-	);
-}
-
-function mergeByCriteria<T>(
-	items: T[],
-	compareFunction: (item1: T, item2: T) => boolean,
-	mergeFunction: (mergedItem: T, item: T) => void,
-): T[] {
-	const mergedItems: T[] = [];
-
-	for (const item of items) {
-		const mergedItem = mergedItems.find((mergedItem) =>
-			compareFunction(mergedItem, item),
-		);
-
-		if (mergedItem) {
-			mergeFunction(mergedItem, item);
-		} else {
-			mergedItems.push(item);
-		}
-	}
-
-	return mergedItems;
-}
-
 export { AnalyticsService };

scripts/analytics/src/modules/analytics/libs/helpers/helpers.ts

@@ -0,0 +1 @@
+export { mergeStats } from "./merge-stats/merge-stats.helper.js";

scripts/analytics/src/modules/analytics/libs/helpers/merge-array-items/merge-array-items.helper.ts

@@ -0,0 +1,23 @@
+const mergeArrayItems = <T>(
+	items: T[],
+	compareFunction: (item1: T, item2: T) => boolean,
+	mergeFunction: (mergedItem: T, item: T) => void,
+): T[] => {
+	const mergedItems: T[] = [];
+
+	for (const item of items) {
+		const mergedItem = mergedItems.find((mergedItem) =>
+			compareFunction(mergedItem, item),
+		);
+
+		if (mergedItem) {
+			mergeFunction(mergedItem, item);
+		} else {
+			mergedItems.push(item);
+		}
+	}
+
+	return mergedItems;
+};
+
+export { mergeArrayItems };

scripts/analytics/src/modules/analytics/libs/helpers/merge-stats-items/merge-stats-items.helper.ts

@@ -0,0 +1,16 @@
+import { type ActivityLogCreateItemRequestDto } from "../../types/types.js";
+
+import { mergeArrayItems } from "../merge-array-items/merge-array-items.helper.js";
+
+const mergeStatsItems = (
+	items: ActivityLogCreateItemRequestDto["items"],
+): ActivityLogCreateItemRequestDto["items"] =>
+	mergeArrayItems(
+		items,
+		(item1, item2) =>
+			item1.authorEmail === item2.authorEmail &&
+			item1.authorName === item2.authorName,
+		(mergedItem, item) => (mergedItem.commitsNumber += item.commitsNumber),
+	);
+
+export { mergeStatsItems };

scripts/analytics/src/modules/analytics/libs/helpers/merge-stats/merge-stats.helper.ts

@@ -0,0 +1,19 @@
+import { type ActivityLogCreateItemRequestDto } from "../../types/types.js";
+
+import { mergeArrayItems } from "../merge-array-items/merge-array-items.helper.js";
+import { mergeStatsItems } from "../merge-stats-items/merge-stats-items.helper.js";
+
+const mergeStats = (
+	statsAll: ActivityLogCreateItemRequestDto[],
+): ActivityLogCreateItemRequestDto[] =>
+	mergeArrayItems(
+		statsAll,
+		(item1, item2) => item1.date === item2.date,
+		(mergedItem, item) =>
+			(mergedItem.items = mergeStatsItems([
+				...mergedItem.items,
+				...item.items,
+			])),
+	);
+
+export { mergeStats };