refactor: extract CHANGELOG into a separated file. Should avoid characters issue

- replace unusued throw new Error by logger.error
- remove unusued variables isWindows, err
- remove unusued logFormat
- remove unusued imports logSessionEnd

.github/workflows/codeql.yml

@@ -0,0 +1,46 @@
+name: "CodeQL PR"
+
+on:
+  pull_request:
+    branches: [ "main" ]
+  schedule:
+    - cron: '00 00 * * 1'
+
+jobs:
+  analyze:
+    name: Analyze (${{ matrix.language }})
+    runs-on: ${{ (matrix.language == 'swift' && 'macos-latest') || 'ubuntu-latest' }}
+    permissions:
+      security-events: write
+      packages: read
+      actions: read
+      contents: read
+
+    strategy:
+      fail-fast: false
+      matrix:
+        include:
+          - language: javascript-typescript
+            build-mode: none
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Initialize CodeQL
+        uses: github/codeql-action/init@v4
+        with:
+          languages: ${{ matrix.language }}
+          build-mode: ${{ matrix.build-mode }}
+
+      - name: Run manual build steps
+        if: matrix.build-mode == 'manual'
+        shell: bash
+        run: |
+          echo 'Manual build required'
+          exit 1
+
+      - name: Perform CodeQL Analysis
+        uses: github/codeql-action/analyze@v4
+        with:
+          category: "/language:${{matrix.language}}"

README.md

@@ -15,8 +15,6 @@
 
 </div>
 
----
-
 ## Overview
 
 Freedom Loader is a desktop application built with Electron that provides a straightforward way to download video and audio content with full metadata support. The application focuses on reliability, transparency, and user privacy-no ads, no tracking, no questionable third-party services.
@@ -25,7 +23,7 @@ The primary goal is to make media downloading accessible to users who want offli
 
 ### See related repo:
 
-- **[Website](https://masteracnolo.github.io/Freedom-Loader-Site/)** - Official project website and documentation
+- **[Website](https://github.com/MasterAcnolo/Freedom-Loader-Site/)** - Official project website and documentation
 - **[Workshop](https://github.com/MasterAcnolo/Freedom-Loader-Workshop)** - Theme creation tool and theme browser
 
 ## Table of Contents
@@ -412,7 +410,6 @@ You are free to use, modify, and redistribute this software under the terms of t
 - [@SpicyFire21](https://github.com/SpicyFire21) to be the spiciest one
 - [@AureCX](https://github.com/AureCX) for being the Official Windows Tester!
 - Zakaria for the website icon 
-- Jacques Chirac to love Apples
 - All users who test, report issues, and help improve the application
 
 ---

app/discordRPC.js

@@ -102,7 +102,7 @@ function startRPC() {
 
   });
 
-  rpc.login({ clientId }).catch(err => {
+  rpc.login({ clientId }).catch(() => {
     // Since 1.6.2, it no longer print the error. This is because if discord is not opened, it will send an error.
     // logger.error("Unable to connect to the RPC:", err);
     logger.info("Unable to connect to the Discord RPC. Discord is maybe not launched");
@@ -123,14 +123,6 @@ async function stopRPC() {
     try {
       if (intervalId) clearInterval(intervalId);
 
-      /**
-       * Ensures RPC connection AND the underlying socket exist
-       * before attempting to clear activity to prevent crash.
-       */
-      if (rpc.transport && rpc.transport.socket) {
-        await rpc.clearActivity();
-      }
-
       await rpc.destroy();
 
     } catch (err) {

app/ipcHandlers.js

@@ -13,7 +13,7 @@
 const { ipcMain, dialog, shell } = require("electron");
 const fs = require("fs");
 const { logger, logDir } = require("../server/logger");
-const { configFeatures, featuresPath } = require("../config");
+const {configFeatures, featuresPath, devMode} = require("../config");
 const { getThemes, reloadThemes } = require("./themeManager");
 const config = require("../config");
 const { validateDownloadPath, getDefaultDownloadPath } = require("./pathValidator");
@@ -114,7 +114,18 @@ function registerIpcHandlers(getMainWindow) {
   /**
    * Window minimize request from renderer.
    */
-  ipcMain.on("window-minimize", () => getMainWindow()?.minimize());
+  ipcMain.on("window-minimize", () => {
+        // Minimize to tray
+        if (configFeatures.systemTray) {
+          getMainWindow()?.hide()
+          logger.info("Window Minimized in SystemTray (Using Minimize Button)");
+        } else {
+          // Native minimize
+          getMainWindow()?.minimize()
+          logger.info("Window minimized normally");
+        }
+      }
+  );
 
   /**
    * Toggles maximize/unmaximize state of main window.
@@ -223,7 +234,7 @@ function registerIpcHandlers(getMainWindow) {
           if (key === "systemTray") {
             if (value === true) {
               logger.info("System Tray enabled dynamically.");
-              createSystemTray(getMainWindow(), config.devMode);
+              createSystemTray(devMode);
             } else {
               logger.info("System Tray disabled dynamically.");
               destroyTray();

app/pathValidator.js

@@ -66,7 +66,7 @@ function validateDownloadPath(userPath) {
       const real = fs.realpathSync(absolutePath);
 
       if (!isSafePath(real)) {
-        throw new Error("Path not allowed: system folders are blocked!");
+          logger.error("Path not allowed: system folders are blocked!");
       }
 
       return real;

app/tray.js

@@ -2,6 +2,7 @@ const {app, Menu, Tray, nativeImage} = require("electron");
 const path = require("path");
 const {logger} = require("../server/logger"); // Ajuste le chemin si besoin
 const fs = require("fs");
+const {getMainWindow} = require("./windowManager");
 
 /**
  * Global reference to the Tray instance to prevent garbage collection.
@@ -17,11 +18,10 @@ let tray = null;
  * - Builds the right-click context menu.
  * - Handles left-click behavior to toggle main window visibility.
  *
- * @param {import('electron').BrowserWindow} mainWindow - The main application window.
  * @param {boolean} devMode - Indicates whether the application is running in development mode.
  * @returns {Tray} The created Tray instance.
  */
-function createSystemTray(mainWindow, devMode) {
+function createSystemTray(devMode) {
     // Prevent creating multiple tray instances
     if (tray) return tray;
 
@@ -58,7 +58,12 @@ function createSystemTray(mainWindow, devMode) {
         {
             label: "Show Freedom Loader",
             click: () => {
-                mainWindow.show();
+                const window = getMainWindow();
+                if (window && !window.isDestroyed()) {
+                    window.show();
+                    window.focus();
+                    logger.info("Window Restored from SystemTray (Using Show Action)");
+                }
             },
         },
         { type: "separator" },
@@ -82,11 +87,15 @@ function createSystemTray(mainWindow, devMode) {
      * but it remains standard practice for Windows and Linux environments.
      */
     tray.on("click", () => {
-        if (mainWindow.isVisible()) {
-            mainWindow.hide();
+        const window = getMainWindow();
+        if (!window || window.isDestroyed()) return;
+
+        if (window.isVisible()) {
+            window.hide();
         } else {
-            mainWindow.show();
-            mainWindow.focus();
+            window.show();
+            window.focus();
+            logger.info("Window Restored from SystemTray (Double Click on Tray Icon)")
         }
     });
 

app/windowManager.js

@@ -87,9 +87,11 @@ async function createMainWindow() {
    * cancels the destruction and hides the window instead.
    */
   mainWindow.on('close', (event) => {
-    if (!app.isQuitting && config.systemTray ) {
+
+    if (!app.isQuitting && config.configFeatures.systemTray) {
       event.preventDefault();
       mainWindow.hide();
+      logger.info("Window Minimized in SystemTray (Using Close Button)");
       return false;
     }
   });

config.js

@@ -7,7 +7,7 @@ const path = require("path");
  * Indicates whether the application is running in development mode.
  * Determined by Electron's packaging state.
  */
-const devMode = process.env.NODE_ENV === "test" || !app?.isPackaged;
+const devMode = !app.isPackaged || process.env.NODE_ENV === "test";
 
 /**
  * Resolves the configuration file path depending on runtime environment.

main.js

@@ -38,7 +38,7 @@ app.setAppUserModelId("com.masteracnolo.freedomloader");
  * Load the app dependencies for hardware choice
  */
 const { logger, logSessionStart, logSessionEnd, logDir } = require("./server/logger");
-const { configFeatures } = require("./config");
+const { configFeatures, devMode} = require("./config");
 
 /**
  * In-memory snapshot of application feature flags.
@@ -53,7 +53,7 @@ if (!configFeatures.enableHardwareAcceleration){
   logger.info("Enable Hardware Acceleration")
 }
 
-if(configFeatures.devMode){
+if(devMode){
   /**
    * Start devTron extensions - @see https://github.com/electron/devtron
    */
@@ -74,7 +74,7 @@ const { updateYtDlp } = require("./app/ytDlpUpdater");
 const { createMainWindow, getMainWindow } = require("./app/windowManager");
 const { registerIpcHandlers } = require("./app/ipcHandlers");
 const { createSplashWindow, closeSplashWindow, setSplashProgress } = require("./app/splashManager");
-const { userThemesPath, initUserThemes, isWindows, validateBinaries, defaultDownloadFolder } = require("./server/helpers/path.helpers");
+const { userThemesPath, initUserThemes, validateBinaries, defaultDownloadFolder } = require("./server/helpers/path.helpers");
 const {createSystemTray, destroyTray} = require("./app/tray");
 const { stopServer } = require("./server/server");
 
@@ -135,7 +135,7 @@ app.whenReady().then(async () => {
 
   createSplashWindow();
 
-  if (!configFeatures.devMode && !checkNativeDependencies()) return;
+  if (!devMode && !checkNativeDependencies()) return;
 
   const { userYtDlp } = require("./server/helpers/path.helpers");
   updateYtDlp(userYtDlp);
@@ -161,7 +161,7 @@ app.whenReady().then(async () => {
     getMainWindow().show();
 
     if (configFeatures.systemTray) {
-      createSystemTray(getMainWindow(), configFeatures.devMode);
+      createSystemTray(devMode);
     }
 
     if (configFeatures.discordRPC) startRPC();

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "freedom-loader",
-  "version": "1.6.2-preview",
+  "version": "1.6.3-preview",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "freedom-loader",
-      "version": "1.6.2-preview",
+      "version": "1.6.3-preview",
       "license": "GPL-3.0-only",
       "dependencies": {
         "chalk": "^4.1.2",
@@ -24,7 +24,7 @@
         "@electron/devtron": "^2.1.1",
         "@jest/globals": "^30.4.1",
         "@playwright/test": "^1.62.1",
-        "electron": "^43.3.0",
+        "electron": "^44.3.0",
         "electron-builder": "^26.15.3",
         "jest": "^30.4.2",
         "nodemon": "^3.1.14",
@@ -3910,9 +3910,9 @@
       }
     },
     "node_modules/electron": {
-      "version": "43.3.0",
-      "resolved": "https://registry.npmjs.org/electron/-/electron-43.3.0.tgz",
-      "integrity": "sha512-nLlvu0WFjftWsSaTkV2B/c4NDuJBspTyXu8vKSQ6vLvFt8uG3NgN49LLKcXddwX0GqVvAQDhciWp+4xOdTdhew==",
+      "version": "44.3.0",
+      "resolved": "https://registry.npmjs.org/electron/-/electron-44.3.0.tgz",
+      "integrity": "sha512-St9EV7F2VtYaYWD2qaAjBwUgKxx39eJOUsUJ5+/1113sqbVfNqv4Dbm/W1rN7qmYSPa+mWwR6yr+b7MfgjgVfQ==",
       "dev": true,
       "license": "MIT",
       "dependencies": {

package.json

@@ -1,7 +1,8 @@
 {
   "name": "freedom-loader",
+  "desktopName": "com.masteracnolo.freedomloader",
   "productName": "Freedom Loader",
-  "version": "1.6.3-preview",
+  "version": "1.6.3",
   "author": "MasterAcnolo <MasterAcnolo@users.noreply.github.com>",
   "description": "Free and open-source GUI for yt-dlp — download video and audio from hundreds of platforms",
   "homepage": "https://masteracnolo.github.io/Freedom-Loader-Site/",
@@ -21,7 +22,6 @@
     "dev": "nodemon --watch server --watch app --watch main.js --watch config.js --ext js,json --exec \"electron --no-warnings .\"",
     "dev:warn": "nodemon --watch server --watch app --watch main.js --watch config.js --ext js,json --exec \"electron --trace-warnings .\"",
     "dev:debug": "nodemon --watch server --watch app --watch main.js --watch config.js --ext js,json --exec \"WAYLAND_DEBUG=1 ELECTRON_ENABLE_LOGGING=1 ELECTRON_ENABLE_STACK_DUMPING=1 electron --enable-logging --v=1 .\"",
-
     "build": "electron-builder",
     "build:win": "electron-builder --win",
     "build:linux": "electron-builder --linux",
@@ -30,10 +30,8 @@
     "release": "bash scripts/release.sh",
     "release:publish": "bash scripts/release.sh --publish",
     "release:dry-run": "bash scripts/release.sh --dry-run",
-
     "test": "jest",
     "test:unit": "jest test/unit",
-
     "update": "npm update"
   },
   "dependencies": {
@@ -52,7 +50,7 @@
     "@electron/devtron": "^2.1.1",
     "@jest/globals": "^30.4.1",
     "@playwright/test": "^1.62.1",
-    "electron": "^43.3.0",
+    "electron": "^44.3.0",
     "electron-builder": "^26.15.3",
     "jest": "^30.4.2",
     "nodemon": "^3.1.14",
@@ -136,6 +134,8 @@
         "snap"
       ],
       "category": "Utility",
+      "synopsis": "GUI for yt-dlp to download video and audio with metadata",
+      "maintainer": "MasterAcnolo",
       "icon": "build/app-icon.png",
       "executableName": "freedom-loader",
       "extraResources": [

scripts/CHANGELOG.md

@@ -0,0 +1 @@
+[//]: # (This file is used for the release CHANGELOG. Please use Markdown to describe changes.)

scripts/release.sh

@@ -5,7 +5,7 @@ SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 ROOT_DIR="$(cd "$SCRIPT_DIR/.." && pwd)"
 VERSION=$(node -p "require('$ROOT_DIR/package.json').version")
 
-f [[ "$VERSION" == *"-preview"* ]]; then
+if [[ "$VERSION" == *"-preview"* ]]; then
     echo "Error: The current version ($VERSION) is a 'preview"
     echo "Unable to launch release pipeline. Please update the package.json with a stable version.."
     exit 1
@@ -64,6 +64,16 @@ echo "[0/5] Cleaning workspace..."
 rm -rf "$ROOT_DIR/dist" "$ROOT_DIR/srpm-out"
 echo "Workspace cleaned (dist/ and srpm-out/ removed)"
 
+# ------------------------------------------------------------
+# Step 0.3 - Retrieve Changelog
+# ------------------------------------------------------------
+if [ ! -f "$ROOT_DIR/CHANGELOG.md" ]; then
+    echo "Warning: CHANGELOG.md not found, release notes will be empty."
+    CHANGELOG=""
+else
+    CHANGELOG="$(cat "$ROOT_DIR/CHANGELOG.md")"
+    echo "Changelog retrieved successfully"
+fi
 # ------------------------------------------------------------
 # Step 1 - Build Linux packages (AppImage, deb, snap)
 # ------------------------------------------------------------
@@ -105,19 +115,13 @@ PREV_TAG=$(git rev-list --tags --skip=1 --max-count=1 | xargs git describe --tag
 
 RELEASE_NOTES="# Freedom Loader - $VERSION
 
+## Changelog
+
+$CHANGELOG
+
 ## Found a bug or issue?
 Please report it in the [GitHub Issues](https://github.com/MasterAcnolo/Freedom-Loader/issues) section.
 
-## Next Release (non-exhaustive roadmap)
-- More format options
-- Subtitle support
-- Improved UI / UX
-- Language selection
-- Download specific parts of videos
-- File renaming options
-- Parallel downloads
-- Skip sponsored parts automatically
-
 **Full Changelog**: https://github.com/MasterAcnolo/Freedom-Loader/compare/${PREV_TAG}...${TAG}"
 
 if [ "$DRY_RUN" = false ]; then

server/controller/info.controller.js

@@ -2,6 +2,7 @@ const { fetchInfo } = require("../services/info.services");
 const { parseVideo, parsePlaylist } = require("../helpers/parseInfo.helpers");
 const { logger } = require("../logger");
 const { isValidUrl } = require("../helpers/validation.helpers");
+const {isUrlPlaylist} = require("../helpers/playlist.helpers");
 
 /**
  * Handles metadata retrieval for a video or playlist.
@@ -36,7 +37,7 @@ async function infoController(req, res) {
     logger.info(`Info request received. ENCODED: ${encodedUrl}`);
 
     // Lightweight heuristic to detect playlist URLs. It works for Youtube.
-    const isPlaylistUrl = url.includes("&list") || url.includes("?list");
+    const isPlaylistUrl = isUrlPlaylist(url);
 
     logger.info(
         isPlaylistUrl
@@ -76,11 +77,11 @@ async function infoController(req, res) {
                     );
 
                     // Preserve original error context
-                    throw err;
+                    logger.error(err);
                 }
 
             } else {
-                throw err;
+                logger.error(err);
             }
         }
 

server/helpers/buildArgs.helpers.js

@@ -1,6 +1,6 @@
 const path = require("path");
 
-const getUserBrowser = require("./getBrowser.helpers.js");
+const {getUserBrowser} = require("./getBrowser.helpers.js");
 const { ffmpegPath, denoPath} = require("./path.helpers.js");
 const { configFeatures } = require("../../config.js");
 const { logger } = require("../logger.js");

server/helpers/getBrowser.helpers.js

@@ -1,7 +1,6 @@
 const fs = require("fs");
 const os = require("os");
-const path = require("path");
-const notify = require("./notify.helpers")
+const notify = require("./notify.helpers");
 const { logger } = require("../logger");
 
 /**
@@ -22,41 +21,38 @@ const { logger } = require("../logger");
  * @returns {string} The detected browser identifier.
  */
 function getUserBrowser() {
-  //const userProfile = os.homedir();
-  const firefoxPath = getFirefoxProfilePath();
+  const platform = process.platform;
+  let isFirefoxHere = false;
 
-  /**
-   * Browsers supported by yt-dlp cookie extraction.
-   *
-   * Only Firefox is currently enabled and tested.
-   * Additional browsers can be enabled in the future
-   * once compatibility has been verified.
-   */
-  const browsers = [
-    { name: "firefox", path: firefoxPath },
-    // { name: "chrome", path: path.join(userProfile, "AppData", "Local", "Google", "Chrome", "User Data", "Default") },
-    // { name: "brave", path: path.join(userProfile, "AppData", "Local", "BraveSoftware", "Brave-Browser", "User Data", "Default") },
-    // { name: "edge", path: path.join(userProfile, "AppData", "Local", "Microsoft", "Edge", "User Data", "Default") },
-    // { name: "opera", path: path.join(userProfile, "AppData", "Roaming", "Opera Software", "Opera Stable") },
-    // { name: "vivaldi", path: path.join(userProfile, "AppData", "Local", "Vivaldi", "User Data", "Default") },
-    // { name: "safari", path: path.join(userProfile, "AppData", "Local", "Apple Computer", "Safari") },
-    // { name: "whale", path: path.join(userProfile, "AppData", "Local", "Naver", "Naver Whale", "User Data", "Default") }
-  ];
+  if (platform === "win32") {
+    const paths = [
+      "C:\\Program Files\\Mozilla Firefox\\firefox.exe",
+      "C:\\Program Files (x86)\\Mozilla Firefox\\firefox.exe",
+      `${process.env.LOCALAPPDATA}\\Mozilla Firefox\\firefox.exe`,
+    ];
 
-  // Search for the first available browser profile.
-  for (const browser of browsers) {
-    if (browser.path && fs.existsSync(browser.path)) {
-      logger.info(`Browser found: ${browser.name}`);
-      return browser.name;
-    }
+    isFirefoxHere = paths.some(p => fs.existsSync(p));
   }
 
-  // No supported browser found => Notify User
-  logger.warn("No supported browser found on the system");
+  if (platform === "linux") {
+    const paths = [
+      "/usr/bin/firefox",
+      "/usr/bin/firefox-esr",
+      "/usr/local/bin/firefox",
+      "/snap/bin/firefox",
+      "/var/lib/flatpak/exports/bin/org.mozilla.firefox",
+      `${os.homedir()}/.local/share/flatpak/exports/bin/org.mozilla.firefox`,
+    ];
+    isFirefoxHere = paths.some(p => fs.existsSync(p));
+  }
 
-  // If you somehow managed to live without Firefox and need help installing it. - Don't applied to my Linux chad
-  if (process.platform === "win32") {
+  if (!isFirefoxHere) {
+    // If you somehow managed to live without Firefox and need help installing it. - Don't applied to my Linux chad
+    // No supported browser found => Notify User
+    logger.warn("No supported browser found on the system");
     notify.notifyFirefoxBrowserMissing();
+  } else {
+    logger.info("Browser found: firefox");
   }
 
   // Fallback to Firefox and let yt-dlp handle the error gracefully.
@@ -64,39 +60,4 @@ function getUserBrowser() {
   return "firefox";
 }
 
-function getFirefoxProfilePath() {
-  const home = os.homedir();
-
-  if (process.platform === "win32") {
-    return path.join(
-      home,
-      "AppData",
-      "Roaming",
-      "Mozilla",
-      "Firefox",
-      "Profiles"
-    );
-  }
-
-  if (process.platform === "linux") {
-    return path.join(
-      "usr",
-      "bin",
-      "firefox"
-    );
-  }
-
-  if (process.platform === "darwin") {
-    return path.join(
-      home,
-      "Library",
-      "Application Support",
-      "Firefox",
-      "Profiles"
-    );
-  }
-
-  return null;
-}
-
-module.exports = getUserBrowser
+module.exports = {getUserBrowser};

server/helpers/path.helpers.js

@@ -260,7 +260,7 @@ function validateBinaries() {
 
 /**
  * Helper to lazy load logger, avoid circular import
- * @returns {winston.Logger}
+ * @returns {import("winston").Logger}
  */
 function getLogger() {
     return require("../logger.js").logger;

server/helpers/playlist.helpers.js

@@ -0,0 +1,12 @@
+/**
+ * Determines whether a URL targets a playlist.
+ *
+ * The detection is based on the presence of the YouTube
+ * playlist query parameter (`list`).
+ *
+ * @param {string} url - URL to inspect.
+ * @returns {boolean} True if the URL appears to be a playlist.
+ */
+export function isUrlPlaylist(url) {
+    return url.includes("?list=") || url.includes("&list=") || url.includes("@");
+}

server/logger.js

@@ -17,11 +17,6 @@ try {
   console.error(`Failed to create log directory: ${error.message}`);
 }
 
-const logFormat = format.combine(
-  format.timestamp({ format: "YYYY-MM-DD HH:mm:ss" }),
-  format.printf(({ timestamp, level, message }) => `${timestamp} | ${level.toUpperCase()} |  ${message}`)
-);
-
 /**
  * Format used by the saved file
  * @type {Format}

server/server.js

@@ -2,7 +2,7 @@ const express = require("express");
 const path = require("path");
 const config = require("../config");
 
-const { logger, logSessionEnd } = require("./logger");
+const {logger} = require("./logger");
 const { rateLimit } = require("./helpers/rateLimit.helpers");
 
 const app = express();
@@ -53,11 +53,14 @@ async function startServer() {
  */
 function stopServer() {
   if (serverInstance) {
-    serverInstance.close();
-    logger.info("Express server closed cleanly.");
-
+    try{
+      serverInstance.close();
+      logger.info("Express server closed cleanly");
+    } catch (err){
+      logger.error("Express server was not closed cleanly:", err)
+    }
   } else {
-    logger.error("Express server was not closed cleanly")
+    logger.info("Express server was already closed");
   }
 }
 

server/services/download.services.js

@@ -7,6 +7,7 @@ const notify = require("../helpers/notify.helpers");
 const path = require("path");
 const { isSafePath } = require("../helpers/validation.helpers");
 const {reloadFeatures} = require("../../config");
+const {isUrlPlaylist} = require("../helpers/playlist.helpers");
 
 /**
  * Reference to the currently running yt-dlp process.
@@ -18,19 +19,6 @@ const {reloadFeatures} = require("../../config");
  */
 let currentDownloadProcess = null;
 
-/**
- * Determines whether a URL targets a playlist.
- *
- * The detection is based on the presence of the YouTube
- * playlist query parameter (`list`).
- *
- * @param {string} url - URL to inspect.
- * @returns {boolean} True if the URL appears to be a playlist.
- */
-function isPlaylistUrl(url) {
-  return url.includes("?list=") || url.includes("&list=");
-}
-
 /**
  * Creates a dedicated download folder for a playlist.
  *
@@ -49,8 +37,8 @@ function isPlaylistUrl(url) {
  */
 function createPlaylistFolder(basePath, playlistTitle) {
   const sanitizedTitle = playlistTitle
-    .replace(/[<>:"|?*]/g, '') 
-    .replace(/\s+/g, ' ') 
+      .replace(/[<>:"|?*]/g, '')
+      .replace(/\s+/g, ' ')
     .trim()
     .substring(0, 200);
 
@@ -77,7 +65,7 @@ function createPlaylistFolder(basePath, playlistTitle) {
     }
 
     logger.error(`Could not find available playlist folder after 1000 attempts`);
-    throw new Error("Unable to create playlist folder");
+    logger.error("Unable to create playlist folder");
 
   } catch (err) {
     logger.warn(`Failed to create playlist folder with title "${sanitizedTitle}": ${err.message}`);
@@ -160,7 +148,7 @@ function fetchDownload(options, progressListeners, speedListeners, stageListener
     }
 
     // Détecte si c'est une playlist et crée un dossier approprié
-    const isPlaylist = options.playlistTitle || isPlaylistUrl(options.url);
+    const isPlaylist = options.playlistTitle || isUrlPlaylist(options.url);
 
     if (isPlaylist && userConfig.createPlaylistFolders) {
       try {

server/services/info.services.js

@@ -1,6 +1,6 @@
 const { execFile } = require("child_process");
 const { userYtDlp, denoPath } = require("../helpers/path.helpers");
-const getUserBrowser = require("../helpers/getBrowser.helpers");
+const {getUserBrowser} = require("../helpers/getBrowser.helpers");
 const { logger } = require("../logger");
 
 /**
@@ -61,6 +61,7 @@ function fetchInfo(url) {
 
             try {
                 const data = JSON.parse(stdout);
+
                 resolve(data);
             } catch (e) {
                 logger.error(`JSON Parsing Error: ${e.message}`);