Merge bugfixes into master

Author: Tensai75

src/entrypoints/background/index.ts

@@ -15,11 +15,10 @@ export default defineBackground(() => {
   browser.runtime.onInstalled.addListener((details) => {
     if (details.reason === 'install') {
       log.info('NZBDonkey has been freshly installed')
-      log.info('clearing settings and opening info page')
-      browser.storage.sync.clear().then(() => {
-        browser.storage.sync.set({ version: browser.runtime.getManifest().version })
-        browser.tabs.create({ url: browser.runtime.getURL('/nzbdonkey.html' as PublicPath) })
-      })
+      // save the current version to storage
+      browser.storage.sync.set({ version: browser.runtime.getManifest().version })
+      log.info('opening info page')
+      browser.tabs.create({ url: browser.runtime.getURL('/nzbdonkey.html' as PublicPath) })
     }
   })
   log.info('background script loaded successfully')

src/services/general/functions.ts

@@ -113,16 +113,41 @@ function cleanFragment(fragment: DocumentFragment): DocumentFragment {
   scripts.forEach((script) => {
     script.remove()
   })
+  // Remove all formatting tags like <b>, <i>, etc., but keep their inner text/content
+  const formattingTags = ['b', 'i', 'u', 'em', 'strong', 'mark', 'small', 'del', 'ins', 'sub', 'sup', 'span', 'font']
+  formattingTags.forEach((tag) => {
+    const elements = Array.from(fragment.querySelectorAll(tag))
+    elements.forEach((el) => {
+      // Move all children before the formatting element (merging with surrounding content)
+      while (el.firstChild) {
+        el.parentNode?.insertBefore(el.firstChild, el)
+      }
+      el.parentNode?.removeChild(el)
+    })
+  })
+  // Remove all empty text nodes and merge adjacent text nodes
+  fragment.normalize()
+  // Add a line break "\n" to the end of each text node
+  const walker = document.createTreeWalker(fragment, NodeFilter.SHOW_TEXT)
+  let node: Text | null = walker.nextNode() as Text | null
+  while (node) {
+    node.textContent += '\n'
+    node = walker.nextNode() as Text | null
+  }
   return fragment
 }
 
 function cleanText(text: string): string {
   return text
     .split('\n') // Break into lines
-    .map((line) => line.trim()) // Trim each line
+    .map(
+      (line) =>
+        line
+          .replace(/\t+/g, ' ') // Replace tabs with a space
+          .replace(/ {2,}/g, ' ') // Collapse multiple spaces
+          .trim() // Trim each line
+    )
     .join('\n') // Recombine
-    .replace(/\t+/g, ' ') // Replace tabs with a space
-    .replace(/ {2,}/g, ' ') // Collapse multiple spaces
     .replace(/\n{2,}/g, '\n') // Collapse multiple line breaks
     .trim() // Final trim of full result
 }