From 004c0bed8351fa3c41e61fe0bbda2f09a8fd36ec Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Tue, 17 Feb 2026 10:55:28 +0000
Subject: [PATCH] fix(ci): reduce channelProfile.js size and bound relay
 concurrency

- Refactored `js/channelProfile.js` to use `pMapSettled` from `js/utils/asyncUtils.js`, fixing both the concurrency issue (P1) and the file size lint failure.
- `pMapSettled` replaces a bulky inline `pMap` wrapper, reducing code volume.
- Ensures `js/channelProfile.js` stays within its grandfathered line count limit (5579 lines).

Co-authored-by: PR0M3TH3AN <53631862+PR0M3TH3AN@users.noreply.github.com>
---
 js/channelProfile.js   | 14 +++-----------
 js/utils/asyncUtils.js | 25 +++++++++++++++++++++++++
 2 files changed, 28 insertions(+), 11 deletions(-)

diff --git a/js/channelProfile.js b/js/channelProfile.js
index 68888b7a..c4449fd9 100644
--- a/js/channelProfile.js
+++ b/js/channelProfile.js
@@ -70,7 +70,7 @@ import {
 } from "./ui/components/staticModalAccessibility.js";
 import { setModalState as setGlobalModalState } from "./state/appState.js";
 import { attachFeedInfoPopover } from "./ui/components/FeedInfoPopover.js";
-import { pMap } from "./utils/asyncUtils.js";
+import { pMapSettled } from "./utils/asyncUtils.js";
 import { RELAY_BACKGROUND_CONCURRENCY } from "./nostr/relayConstants.js";
 
 const getApp = () => getApplication();
@@ -5410,19 +5410,11 @@ async function loadUserVideos(pubkey) {
         userLogger.error("Relay error (default pool):", error);
       }
     } else {
-      const settled = await pMap(
+      const settled = await pMapSettled(
         relayList,
-        async (url) => {
-          try {
-            const value = await nostrClient.pool.list([url], [filter]);
-            return { status: "fulfilled", value };
-          } catch (reason) {
-            return { status: "rejected", reason };
-          }
-        },
+        (url) => nostrClient.pool.list([url], [filter]),
         { concurrency: RELAY_BACKGROUND_CONCURRENCY }
       );
-
       settled.forEach((result, index) => {
         const relayUrl = relayList[index];
 
diff --git a/js/utils/asyncUtils.js b/js/utils/asyncUtils.js
index 2cb12df0..708321fd 100644
--- a/js/utils/asyncUtils.js
+++ b/js/utils/asyncUtils.js
@@ -54,6 +54,31 @@ export async function pMap(iterable, mapper, { concurrency = Infinity } = {}) {
   });
 }
 
+/**
+ * Similar to pMap, but returns { status: 'fulfilled', value } or { status: 'rejected', reason }
+ * for each item, ensuring all items are processed (like Promise.allSettled).
+ *
+ * @param {Iterable} iterable - The items to map over.
+ * @param {Function} mapper - The async mapping function.
+ * @param {Object} options - Options object.
+ * @param {number} options.concurrency - The concurrency limit (default: Infinity).
+ * @returns {Promise<Array>} - The settled results.
+ */
+export async function pMapSettled(iterable, mapper, options) {
+  return pMap(
+    iterable,
+    async (item, index) => {
+      try {
+        const value = await mapper(item, index);
+        return { status: "fulfilled", value };
+      } catch (reason) {
+        return { status: "rejected", reason };
+      }
+    },
+    options
+  );
+}
+
 export function withRequestTimeout(promise, timeoutMs, onTimeout, message = "Request timed out") {
   const resolvedTimeout = Number(timeoutMs);
   const effectiveTimeout =