Add expansion/collapse animations for cards

Key changes: - Add animation for card opening/collapse. Other supporting changes: - Remove card expansion panel to its own component for easier maintainability and better separation of concerns. - Use real DOM element instead of &:before pseudo class for showing expansion arrow. This increases by maintainability by separating its code and concerns. - TODO: When one card is expanded and others is also expanded then the transition sucks.
Improve URL checks to reduce false-negatives
2024-03-31 20:07:09 +02:00 · 2024-03-13 18:26:16 +01:00
24 changed files with 566 additions and 1027 deletions
--- a/.github/workflows/checks.external-urls.yaml
+++ b/.github/workflows/checks.external-urls.yaml
@@ -1,9 +1,11 @@
 name: checks.external-urls

 on:
-  push:
  schedule:
    - cron: '0 0 * * 0' # at 00:00 on every Sunday
+  push:
+    paths:
+      - tests/checks/external-urls/**

 jobs:
  run-check:
@@ -21,7 +23,3 @@ jobs:
      -
        name: Test
        run: npm run check:external-urls
-        env:
-          RANDOMIZED_URL_CHECK_LIMIT: "${{ github.event_name == 'push' && '10' || '' }}"
-          # - Scheduled checks has no limits, ensuring thorough testing.
-          # - For push events, triggered by code changes, the amount of URLs are limited to provide quick feedback.
--- a/package-lock.json
+++ b/package-lock.json
--- a/package.json
+++ b/package.json
@@ -82,7 +82,7 @@
    "tslib": "^2.6.2",
    "typescript": "^5.3.3",
    "vite": "^5.1.6",
-    "vitest": "^1.3.1",
+    "vitest": "^0.34.6",
    "vue-tsc": "^1.8.19",
    "yaml-lint": "^1.7.0"
  },
--- a/src/application/Common/Shuffle.ts
+++ b/src/application/Common/Shuffle.ts
@@ -1,12 +0,0 @@
-/*
-  Shuffle an array of strings, returning a new array with elements in random order.
-  Uses the Fisher-Yates (or Durstenfeld) algorithm.
-*/
-export function shuffle<T>(array: readonly T[]): T[] {
-  const shuffledArray = [...array];
-  for (let i = array.length - 1; i > 0; i--) {
-    const j = Math.floor(Math.random() * (i + 1));
-    [shuffledArray[i], shuffledArray[j]] = [shuffledArray[j], shuffledArray[i]];
-  }
-  return shuffledArray;
-}
--- a/src/application/collections/windows.yaml
+++ b/src/application/collections/windows.yaml
@@ -1174,7 +1174,7 @@ actions:
                                        parameters:
                                            fileGlob: '%SYSTEMROOT%\Logs\DISM\DISM.log'
                            -
-                                name: Clear Windows update files
+                                name: Clear Windows update files # Marked: stop-service-do-stuff-restart-service
                                docs: |-
                                    This script clears the contents of the `%SYSTEMROOT%\SoftwareDistribution\` directory.
                                    This action is sometimes called *resetting the Windows Update Agent* or *resetting Windows Update components* by Microsoft [1].
@@ -1203,22 +1203,18 @@ actions:
                                    [8]: https://web.archive.org/web/20231027190213/https://support.microsoft.com/en-us/windows/troubleshoot-problems-updating-windows-188c2b0f-10a7-d72f-65b8-32d177eb136c#WindowsVersion=Windows_11 "Troubleshoot problems updating Windows - Microsoft Support | support.microsoft.com"
                                    [9]: https://web.archive.org/web/20231027190503/https://learn.microsoft.com/en-us/troubleshoot/mem/configmgr/update-management/troubleshoot-software-update-scan-failures "Troubleshoot software update scan failures - Configuration Manager | Microsoft Learn | learn.microsoft.com"
                                    [10]: https://web.archive.org/web/20231029172022/https://support.microsoft.com/en-us/topic/you-receive-an-administrators-only-error-message-in-windows-xp-when-you-try-to-visit-the-windows-update-web-site-or-the-microsoft-update-web-site-d2c732b6-21e0-a2ce-8d18-303ed71736c9 'You receive an "Administrators only" error message in Windows XP when you try to visit the Windows Update Web site or the Microsoft Update Web site - Microsoft Support | support.microsoft.com'
-                                call:
-                                    -
-                                        function: StopService
-                                        parameters:
-                                            serviceName: wuauserv
-                                            waitUntilStopped: true
-                                            serviceRestartStateFile: '%APPDATA%\privacy.sexy-wuauserv' # Marked: refactor-with-variables (app dir should be unified, not using %TEMP% as it can be cleaned during operation)
-                                    -
-                                        function: ClearDirectoryContents
-                                        parameters:
-                                            directoryGlob: '%SYSTEMROOT%\SoftwareDistribution'
-                                    -
-                                        function: StartService
-                                        parameters:
-                                            serviceName: wuauserv
-                                            serviceRestartStateFile: '%APPDATA%\privacy.sexy-wuauserv' # Marked: refactor-with-variables (app dir should be unified, not using %TEMP% as it can be cleaned during operation)
+                                code: |- # `sc queryex` output is the same in every OS language
+                                    setlocal EnableDelayedExpansion
+                                        SET /A wuau_service_running=0
+                                        SC queryex "wuauserv"|Find "STATE"|Find /v "RUNNING">Nul||(
+                                            SET /A wuau_service_running=1
+                                            net stop wuauserv
+                                        )
+                                        del /q /s /f "%SYSTEMROOT%\SoftwareDistribution\*"
+                                        IF !wuau_service_running! == 1 (
+                                            net start wuauserv
+                                        )
+                                    endlocal
                            -
                                name: Clear Common Language Runtime system logs
                                recommend: standard
@@ -1255,7 +1251,7 @@ actions:
                                    parameters:
                                        directoryGlob: '%SYSTEMROOT%\System32\LogFiles\setupcln'
                    -
-                        name: Clear diagnostics tracking logs
+                        name: Clear diagnostics tracking logs # Marked: stop-service-do-stuff-restart-service ("DiagTrack")
                        recommend: standard
                        docs: |-
                            This script deletes primary telemetry files in Windows.
@@ -1290,12 +1286,6 @@ actions:
                            [6]: https://web.archive.org/web/20231027164510/https://learn.microsoft.com/en-us/windows/win32/etw/configuring-and-starting-an-autologger-session "Configuring and Starting an AutoLogger Session - Win32 apps | Microsoft Learn | learn.microsoft.com"
                            [7]: https://web.archive.org/web/20240217185108/https://learn.microsoft.com/en-us/windows/privacy/configure-windows-diagnostic-data-in-your-organization "Configure Windows diagnostic data in your organization (Windows 10 and Windows 11) - Windows Privacy | Microsoft Learn | learn.microsoft.com"
                        call:
-                            -
-                                function: StopService
-                                parameters:
-                                    serviceName: DiagTrack
-                                    waitUntilStopped: true
-                                    serviceRestartStateFile: '%APPDATA%\privacy.sexy-DiagTrack' # Marked: refactor-with-variables (app dir should be unified, not using %TEMP% as it can be cleaned during operation)
                            -
                                function: DeleteFiles
                                parameters:
@@ -1306,11 +1296,6 @@ actions:
                                parameters:
                                    fileGlob: '%PROGRAMDATA%\Microsoft\Diagnosis\ETLLogs\ShutdownLogger\AutoLogger-Diagtrack-Listener.etl'
                                    grantPermissions: true
-                            -
-                                function: StartService
-                                parameters:
-                                    serviceName: DiagTrack
-                                    serviceRestartStateFile: '%APPDATA%\privacy.sexy-DiagTrack' # Marked: refactor-with-variables (app dir should be unified, not using %TEMP% as it can be cleaned during operation)
                    -
                        name: Clear event logs in Event Viewer application
                        docs: https://serverfault.com/questions/407838/do-windows-events-from-the-windows-event-log-have-sensitive-information
@@ -1467,7 +1452,7 @@ actions:
                recommend: standard
                code: dism /online /Remove-DefaultAppAssociations
            -
-                name: Clear System Resource Usage Monitor (SRUM) data
+                name: Clear System Resource Usage Monitor (SRUM) data # Marked: stop-service-do-stuff-restart-service
                recommend: standard
                docs: |-
                    This script deletes the Windows System Resource Usage Monitor (SRUM) database file.
@@ -1487,25 +1472,47 @@ actions:
                    [5]: https://web.archive.org/web/20231008135321/https://devblogs.microsoft.com/sustainable-software/measuring-your-application-power-and-carbon-impact-part-1/ "Measuring Your Application Power and Carbon Impact (Part 1) - Sustainable Software | devblogs.microsoft.com"
                    [6]: https://web.archive.org/web/20231008135333/https://www.sciencedirect.com/science/article/abs/pii/S1742287615000031 "Forensic implications of System Resource Usage Monitor (SRUM) data in Windows 8 | Yogesh Khatri | sciencedirect.com"
                call:
-                    -
+                    function: RunPowerShell
+                    parameters:
                        # If the service is not stopped, following error is thrown:
                        #   Failed to delete SRUM database file at: "C:\Windows\System32\sru\SRUDB.dat". Error Details: The process cannot access
-                        #   the file 'C:\Windows\System32\sru\SRUDB.dat' because it is being used by another proces
-                        function: StopService
-                        parameters:
-                            serviceName: DPS
-                            waitUntilStopped: true
-                            serviceRestartStateFile: '%APPDATA%\privacy.sexy-DPS' # Marked: refactor-with-variables (app dir should be unified, not using %TEMP% as it can be cleaned during operation)
-                    -
-                        function: DeleteFiles
-                        parameters:
-                            fileGlob: '%WINDIR%\System32\sru\SRUDB.dat'
-                            grantPermissions: true
-                    -
-                        function: StartService
-                        parameters:
-                            serviceName: DPS
-                            serviceRestartStateFile: '%APPDATA%\privacy.sexy-DPS' # Marked: refactor-with-variables (app dir should be unified, not using %TEMP% as it can be cleaned during operation)
+                        #   the file 'C:\Windows\System32\sru\SRUDB.dat' because it is being used by another process.
+                        code: |-
+                            $srumDatabaseFilePath = "$env:WINDIR\System32\sru\SRUDB.dat"
+                            if (!(Test-Path -Path $srumDatabaseFilePath)) {
+                                Write-Output "Skipping, SRUM database file not found at `"$srumDatabaseFilePath`". No actions are required."
+                                exit 0
+                            }
+                            $dps = Get-Service -Name 'DPS' -ErrorAction Ignore
+                            $isDpsInitiallyRunning = $false
+                            if ($dps) {
+                                $isDpsInitiallyRunning = $dps.Status -eq [System.ServiceProcess.ServiceControllerStatus]::Running
+                                if ($isDpsInitiallyRunning) {
+                                    Write-Output "Stopping the Diagnostic Policy Service (DPS) to delete the SRUM database file."
+                                    $dps | Stop-Service -Force
+                                    $dps.WaitForStatus([System.ServiceProcess.ServiceControllerStatus]::Stopped)
+                                    Write-Output "Successfully stopped Diagnostic Policy Service (DPS)."
+                                }
+                            } else {
+                                Write-Output "Diagnostic Policy Service (DPS) not found. Proceeding without stopping the service."
+                            }
+                            try {
+                                Remove-Item -Path $srumDatabaseFilePath -Force -ErrorAction Stop
+                                Write-Output "Successfully deleted the SRUM database file at `"$srumDatabaseFilePath`"."
+                            } catch {
+                                throw "Failed to delete SRUM database file at: `"$srumDatabaseFilePath`". Error Details: $($_.Exception.Message)"
+                            } finally {
+                                if ($isDpsInitiallyRunning) {
+                                    try {
+                                        if ((Get-Service -Name 'DPS').Status -ne [System.ServiceProcess.ServiceControllerStatus]::Running) {
+                                            Write-Output "Restarting the Diagnostic Policy Service (DPS)."
+                                            $dps | Start-Service
+                                        }
+                                    } catch {
+                                        throw "Failed to restart the Diagnostic Policy Service (DPS). Error Details: $($_.Exception.Message)"
+                                    }
+                                }
+                            }
            -
                name: Clear previous Windows installations
                call:   
@@ -15875,12 +15882,10 @@ actions:
        category: Advanced settings
        children:
            -
-                name: Set NTP (time) server to `pool.ntp.org`
+                name: Set NTP (time) server to `pool.ntp.org` # Marked: stop-service-do-stuff-restart-service
                docs: https://www.pool.ntp.org/en/use.html
                recommend: strict
                 # `sc queryex` output is same in every OS language
-                 # Marked: refactor-with-revert-call, refactor-with-variables
-                 #  This would allow re-using `StartService` and `StopService`
                code: |-
                    :: Configure time source
                    w32tm /config /syncfromflags:manual /manualpeerlist:"0.pool.ntp.org 1.pool.ntp.org 2.pool.ntp.org 3.pool.ntp.org"
@@ -15899,7 +15904,7 @@ actions:
                    SC queryex "w32time"|Find "STATE"|Find /v "RUNNING">Nul||(
                        net stop w32time
                    )
-                    :: Start time service and sync now
+                    :: Start time servie and sync now
                    net start w32time
                    w32tm /config /update
                    w32tm /resync
@@ -16712,8 +16717,6 @@ functions:
            - name: defaultStartupMode # Allowed values: Boot | System | Automatic | Manual
        call:
            function: RunPowerShell
-            # Marked: refactor-with-revert-call, refactor-with-variables
-            #   Implementation of those should share similar code: `DisableService`, `StopService`, `StartService`, `DisableServiceInRegistry`
            parameters:
                code: |- # We do registry way because GUI, "sc config" or "Set-Service" won't not work
                    $serviceQuery = '{{ $serviceName }}'
@@ -16935,123 +16938,6 @@ functions:
                        }
                        exit 1
                    }
-    -
-        name: StopService
-        parameters:
-            - name: serviceName
-            - name: serviceRestartStateFile     # This file is created only if the service is successfully stopped.
-              optional: true
-            - name: waitUntilStopped            # Makes the script wait until the service is stopped
-              optional: true
-        call:
-            -
-                function: Comment
-                parameters:
-                    codeComment: >-
-                        Stop service: {{ $serviceName }}
-                        {{ with $serviceRestartStateFile }}(with state flag){{ end }}
-                        {{ with $waitUntilStopped }}(wait until stopped){{ end }}
-            -
-                function: RunPowerShell
-                parameters:
-                    # Marked: refactor-with-variables
-                    #   Implementation of those should share similar code: `DisableService`, `StopService`, `StartService`, `DisableServiceInRegistry`
-                    code: |-
-                        $serviceName = '{{ $serviceName }}'
-                        Write-Host "Stopping service: `"$serviceName`"."
-                        $service = Get-Service -Name $serviceName -ErrorAction SilentlyContinue
-                        if (!$service) {
-                            Write-Host "Skipping, service `"$serviceName`" could not be not found, no need to stop it."
-                            exit 0
-                        }
-                        if ($service.Status -ne [System.ServiceProcess.ServiceControllerStatus]::Running) {
-                            Write-Host "Skipping, `"$serviceName`" is not running, no need to stop."
-                            exit 0
-                        }
-                        Write-Host "`"$serviceName`" is running, stopping it."
-                        try {
-                            $service | Stop-Service -Force -ErrorAction Stop
-                            {{ with $waitUntilStopped }}
-                                $service.WaitForStatus([System.ServiceProcess.ServiceControllerStatus]::Stopped)
-                            {{ end }}
-                        } catch {
-                            throw "Failed to stop the service `"$serviceName`": $_"
-                        }
-                        Write-Host "Successfully stopped the service: `"$serviceName`"."
-                        {{ with $serviceRestartStateFile }}
-                            $stateFilePath = '{{ . }}'
-                            $expandedStateFilePath = [System.Environment]::ExpandEnvironmentVariables($stateFilePath)
-                            if (Test-Path -Path $expandedStateFilePath) {
-                                Write-Host "Skipping creating a service state file, it already exists: `"$expandedStateFilePath`"."
-                            } else {
-                                # Ensure the directory exists
-                                $parentDirectory = [System.IO.Path]::GetDirectoryName($expandedStateFilePath)
-                                if (-not (Test-Path $parentDirectory -PathType Container)) {
-                                    try {
-                                        New-Item -ItemType Directory -Path $parentDirectory -Force -ErrorAction Stop | Out-Null
-                                    }  catch {
-                                        Write-Warning "Failed to create parent directory of service state file `"$parentDirectory`": $_"
-                                    }
-                                }
-                                # Create the state file
-                                try {
-                                    New-Item -ItemType File -Path $expandedStateFilePath -Force -ErrorAction Stop | Out-Null
-                                    Write-Host 'The service will be started again.'
-                                } catch {
-                                    Write-Warning "Failed to create service state file `"$expandedStateFilePath`": $_"
-                                }
-                            }
-                        {{ end }}
-    -
-        name: StartService
-        parameters:
-            - name: serviceName
-            - name: serviceRestartStateFile     # Used for "check and delete": Starts the service only if file exists, always deletes the file.
-              optional: true
-        call:
-            -
-                function: Comment
-                parameters:
-                    codeComment: >-
-                        Start service: {{ $serviceName }}
-                        {{ with $serviceRestartStateFile }}(with state flag){{ end }}
-            -
-                function: RunPowerShell
-                parameters:
-                    # Marked: refactor-with-variables
-                    #   Implementation of those should share similar code: `DisableService`, `StopService`, `StartService`, `DisableServiceInRegistry`
-                    code: |-
-                        $serviceName = '{{ $serviceName }}'
-                        {{ with $serviceRestartStateFile }}
-                            $stateFilePath = '{{ . }}'
-                            $expandedStateFilePath = [System.Environment]::ExpandEnvironmentVariables($stateFilePath)
-                            if (-not (Test-Path -Path $expandedStateFilePath)) {
-                                Write-Host "Skipping starting the service: It was not running before."
-                            } else {
-                                try {
-                                    Remove-Item -Path $expandedStateFilePath -Force -ErrorAction Stop
-                                    Write-Host 'The service is expected to be started.'
-                                } catch {
-                                    Write-Warning "Failed to delete the service state file `"$expandedStateFilePath`": $_"
-                                }
-                            }
-                        {{ end }}
-                        $service = Get-Service -Name $serviceName -ErrorAction SilentlyContinue
-                        if (!$service) {
-                            throw "Failed to start service `"$serviceName`": Service not found."
-                        }
-                        if ($service.Status -eq [System.ServiceProcess.ServiceControllerStatus]::Running) {
-                            Write-Host "Skipping, `"$serviceName`" is already running, no need to start."
-                            exit 0
-                        }
-                        Write-Host "`"$serviceName`" is not running, starting it."
-                        try {
-                            $service | Start-Service -ErrorAction Stop
-                            Write-Host "Successfully started the service: `"$serviceName`"."
-                        } catch {
-                            Write-Warning "Failed to start the service: `"$serviceName`"."
-                            exit 1
-                        }
    -
        name: DisableService
        parameters:
@@ -17064,8 +16950,6 @@ functions:
                    codeComment: "Disable service(s): `{{ $serviceName }}`"
                    revertCodeComment: "Restore service(s) to default state: `{{ $serviceName }}`"
            -
-                # Marked: refactor-with-revert-call, refactor-with-variables
-                #   Implementation of those should share similar code: `DisableService`, `StopService`, `StartService`, `DisableServiceInRegistry`
                function: RunPowerShell
                # Careful with Set-Service cmdlet:
                #   1. It exits with positive code even if service is disabled
@@ -17075,7 +16959,7 @@ functions:
                #           So "Disabled", "Automatic" and "Manual" are only consistent ones.
                #       Read more:
                #           https://github.com/PowerShell/PowerShell/blob/v7.2.0/src/Microsoft.PowerShell.Commands.Management/commands/management/Service.cs#L2966-L2978
-                #           https://learn.microsoft.com/en-us/powershell/module/microsoft.powershell.management/set-service?view=powershell-7.4
+                #           https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.management/set-service?view=powershell-7.1
                parameters:
                    code: |-
                        $serviceName = '{{ $serviceName }}'
@@ -17098,6 +16982,7 @@ functions:
                        } else {
                            Write-Host "`"$serviceName`" is not running, no need to stop."
                        }
+
                        # -- 3. Skip if already disabled
                        $startupType = $service.StartType # Does not work before .NET 4.6.1
                        if(!$startupType) {
--- a/src/presentation/components/Scripts/View/Cards/CardExpansionPanel.vue
+++ b/src/presentation/components/Scripts/View/Cards/CardExpansionPanel.vue
@@ -0,0 +1,90 @@
+<template>
+  <div class="card__expander">
+    <div class="card__expander__close-button">
+      <FlatButton
+        icon="xmark"
+        @click="collapse()"
+      />
+    </div>
+    <div class="card__expander__content">
+      <ScriptsTree
+        :category-id="categoryId"
+        :has-top-padding="false"
+      />
+    </div>
+  </div>
+</template>
+
+<script lang="ts">
+import {
+  defineComponent,
+} from 'vue';
+import FlatButton from '@/presentation/components/Shared/FlatButton.vue';
+import ScriptsTree from '@/presentation/components/Scripts/View/Tree/ScriptsTree.vue';
+
+export default defineComponent({
+  components: {
+    ScriptsTree,
+    FlatButton,
+  },
+  props: {
+    categoryId: {
+      type: Number,
+      required: true,
+    },
+  },
+  emits: {
+    /* eslint-disable @typescript-eslint/no-unused-vars */
+    onCollapse: () => true,
+    /* eslint-enable @typescript-eslint/no-unused-vars */
+  },
+  setup(_, { emit }) {
+    function collapse() {
+      emit('onCollapse');
+    }
+
+    return {
+      collapse,
+    };
+  },
+});
+
+</script>
+
+<style scoped lang="scss">
+@use "@/presentation/assets/styles/main" as *;
+
+$expanded-margin-top    : 30px;
+
+.card__expander {
+  transition: all 0.2s ease-in-out;
+  position: relative;
+  background-color: $color-primary-darker;
+  color: $color-on-primary;
+
+  display: flex;
+  align-items: center;
+  flex-direction: column;
+
+  margin-top: $expanded-margin-top;
+
+  .card__expander__content {
+    display: flex;
+    justify-content: center;
+    word-break: break-word;
+    max-width: 100%; // Prevents horizontal expansion of inner content (e.g., when a code block is shown)
+    width: 100%; // Expands the container to fill available horizontal space, enabling alignment of child items.
+  }
+
+  .card__expander__close-button {
+    font-size: $font-size-absolute-large;
+    align-self: flex-end;
+    margin-right: 0.25em;
+    @include clickable;
+    color: $color-primary-light;
+    @include hover-or-touch {
+      color: $color-primary;
+    }
+  }
+}
+</style>
--- a/src/presentation/components/Scripts/View/Cards/CardExpansionPanelArrow.vue
+++ b/src/presentation/components/Scripts/View/Cards/CardExpansionPanelArrow.vue
@@ -0,0 +1,24 @@
+<template>
+  <div class="arrow-container">
+    <div class="arrow" />
+  </div>
+</template>
+
+<style scoped lang="scss">
+@use "@/presentation/assets/styles/main" as *;
+
+$arrow-size             : 15px;
+
+.arrow-container {
+  position: relative;
+  .arrow {
+    position: absolute;
+    left: calc(50% - $arrow-size * 1.5);
+    top: calc(1.5 * $arrow-size);
+    border: solid $color-primary-darker;
+    border-width: 0 $arrow-size $arrow-size 0;
+    padding: $arrow-size;
+    transform: rotate(-135deg);
+  }
+}
+</style>
--- a/src/presentation/components/Scripts/View/Cards/CardList.vue
+++ b/src/presentation/components/Scripts/View/Cards/CardList.vue
@@ -19,6 +19,7 @@
            v-for="categoryId of categoryIds"
            :key="categoryId"
            class="card"
+            :total-cards-per-row="cardsPerRow"
            :class="{
              'small-screen': width <= 500,
              'medium-screen': width > 500 && width < 750,
@@ -62,6 +63,19 @@ export default defineComponent({
    );
    const activeCategoryId = ref<number | undefined>(undefined);

+    const cardsPerRow = computed<number>(() => {
+      if (width.value === undefined) {
+        throw new Error('Unknown width, total cards should not be calculated');
+      }
+      if (width.value <= 500) {
+        return 1;
+      }
+      if (width.value < 750) {
+        return 2;
+      }
+      return 3;
+    });
+
    function onSelected(categoryId: number, isExpanded: boolean) {
      activeCategoryId.value = isExpanded ? categoryId : undefined;
    }
@@ -108,6 +122,7 @@ export default defineComponent({
      width,
      categoryIds,
      activeCategoryId,
+      cardsPerRow,
      onSelected,
    };
  },
--- a/src/presentation/components/Scripts/View/Cards/CardListItem.vue
+++ b/src/presentation/components/Scripts/View/Cards/CardListItem.vue
@@ -29,20 +29,15 @@
        :category-id="categoryId"
      />
    </div>
-    <div class="card__expander" @click.stop>
-      <div class="card__expander__close-button">
-        <FlatButton
-          icon="xmark"
-          @click="collapse()"
-        />
-      </div>
-      <div class="card__expander__content">
-        <ScriptsTree
-          :category-id="categoryId"
-          :has-top-padding="false"
-        />
-      </div>
-    </div>
+    <CardExpansionPanelArrow v-show="isExpanded" />
+    <ExpandCollapseTransition>
+      <CardExpansionPanel
+        v-show="isExpanded"
+        :category-id="categoryId"
+        @on-collapse="collapse"
+        @click.stop
+      />
+    </ExpandCollapseTransition>
  </div>
 </template>

@@ -51,24 +46,30 @@ import {
  defineComponent, computed, shallowRef,
 } from 'vue';
 import AppIcon from '@/presentation/components/Shared/Icon/AppIcon.vue';
-import FlatButton from '@/presentation/components/Shared/FlatButton.vue';
+import ExpandCollapseTransition from '@/presentation/components/Shared/ExpandCollapse/ExpandCollapseTransition.vue';
 import { injectKey } from '@/presentation/injectionSymbols';
-import ScriptsTree from '@/presentation/components/Scripts/View/Tree/ScriptsTree.vue';
 import { sleep } from '@/infrastructure/Threading/AsyncSleep';
 import CardSelectionIndicator from './CardSelectionIndicator.vue';
+import CardExpansionPanel from './CardExpansionPanel.vue';
+import CardExpansionPanelArrow from './CardExpansionPanelArrow.vue';

 export default defineComponent({
  components: {
-    ScriptsTree,
    AppIcon,
    CardSelectionIndicator,
-    FlatButton,
+    CardExpansionPanel,
+    ExpandCollapseTransition,
+    CardExpansionPanelArrow,
  },
  props: {
    categoryId: {
      type: Number,
      required: true,
    },
+    totalCardsPerRow: {
+      type: Number,
+      required: true,
+    },
    activeCategoryId: {
      type: Number,
      default: undefined,
@@ -94,6 +95,14 @@ export default defineComponent({
      },
    });

+    const cardWidth = computed<string>(() => {
+      const totalTimesGapIsUsedInRow = props.totalCardsPerRow - 1;
+      const totalGapWidthInRow = `calc(${totalTimesGapIsUsedInRow} * 15px)`; // TODO: 15px is hardcoded, $card-gap variable should be used
+      const availableRowWidthForCards = `calc(100% - (${totalGapWidthInRow}))`;
+      const availableWidthPerCard = `calc((${availableRowWidthForCards}) / ${totalTimesGapIsUsedInRow})`;
+      return availableWidthPerCard;
+    });
+
    const cardElement = shallowRef<HTMLElement>();

    const cardTitle = computed<string>(() => {
@@ -118,6 +127,7 @@ export default defineComponent({
      cardTitle,
      isExpanded,
      cardElement,
+      cardWidth,
      collapse,
    };
  },
@@ -131,11 +141,22 @@ export default defineComponent({
 $card-inner-padding     : 30px;
 $arrow-size             : 15px;
 $expanded-margin-top    : 30px;
-$card-horizontal-gap    : $card-gap;
+
+.expansion__arrow {
+  position: relative;
+  .expansion__arrow__inner {
+    position: absolute;
+    left: calc(50% - $arrow-size * 1.5);
+    top: calc(1.5 * $arrow-size);
+    border: solid $color-primary-darker;
+    border-width: 0 $arrow-size $arrow-size 0;
+    padding: $arrow-size;
+    transform: rotate(-135deg);
+  }
+}

 .card {
-  transition: all 0.2s ease-in-out;
-
+  width: v-bind(cardWidth);
  &__inner {
    padding-top: $card-inner-padding;
    padding-right: $card-inner-padding;
@@ -160,9 +181,6 @@ $card-horizontal-gap    : $card-gap;
      color: $color-on-secondary;
      transform: scale(1.05);
    }
-    &:after {
-      transition: all 0.3s ease-in-out;
-    }
    .card__inner__title {
      display: flex;
      flex-direction: column;
@@ -184,73 +202,12 @@ $card-horizontal-gap    : $card-gap;
      font-size: $font-size-absolute-normal;
    }
  }
-  .card__expander {
-    transition: all 0.2s ease-in-out;
-    position: relative;
-    background-color: $color-primary-darker;
-    color: $color-on-primary;
-
-    display: flex;
-    align-items: center;
-    flex-direction: column;
-
-    .card__expander__content {
-      display: flex;
-      justify-content: center;
-      word-break: break-word;
-      max-width: 100%; // Prevents horizontal expansion of inner content (e.g., when a code block is shown)
-      width: 100%; // Expands the container to fill available horizontal space, enabling alignment of child items.
-    }
-
-    .card__expander__close-button {
-      font-size: $font-size-absolute-large;
-      align-self: flex-end;
-      margin-right: 0.25em;
-      @include clickable;
-      color: $color-primary-light;
-      @include hover-or-touch {
-        color: $color-primary;
-      }
-    }
-  }
-
-  &.is-collapsed {
-    .card__inner {
-      &:after {
-        content: "";
-        opacity: 0;
-      }
-    }
-
-    .card__expander {
-      max-height: 0;
-      min-height: 0;
-      overflow: hidden;
-      opacity: 0;
-    }
-  }

  &.is-expanded {
    .card__inner {
      height: auto;
      background-color: $color-secondary;
      color: $color-on-secondary;
-      &:after { // arrow
-        content: "";
-        display: block;
-        position: absolute;
-        bottom: calc(-1 * #{$expanded-margin-top});
-        left: calc(50% - #{$arrow-size});
-        border-left: #{$arrow-size} solid transparent;
-        border-right: #{$arrow-size} solid transparent;
-        border-bottom: #{$arrow-size} solid $color-primary-darker;
-      }
-    }
-
-    .card__expander {
-      min-height: 200px;
-      margin-top: $expanded-margin-top;
-      opacity: 1;
    }

    @include hover-or-touch {
@@ -277,26 +234,26 @@ $card-horizontal-gap    : $card-gap;
  }
 }
@mixin adaptive-card($cards-in-row) {
-  &.card {
+  .card {
    $total-times-gap-is-used-in-row: $cards-in-row - 1;
    $total-gap-width-in-row: $total-times-gap-is-used-in-row * $card-horizontal-gap;
    $available-row-width-for-cards: calc(100% - #{$total-gap-width-in-row});
    $available-width-per-card: calc(#{$available-row-width-for-cards} / #{$cards-in-row});
    width:$available-width-per-card;
-    .card__expander {
-      $all-cards-width: 100% * $cards-in-row;
-      $additional-padding-width: $card-horizontal-gap * ($cards-in-row - 1);
-      width: calc(#{$all-cards-width} + #{$additional-padding-width});
-    }
-    @for $nth-card from 2 through $cards-in-row { // From second card to rest
-      &:nth-of-type(#{$cards-in-row}n+#{$nth-card}) {
-        .card__expander {
-          $card-left: -100% * ($nth-card - 1);
-          $additional-space: $card-horizontal-gap * ($nth-card - 1);
-          margin-left: calc(#{$card-left} - #{$additional-space});
-        }
-      }
-    }
+    // .card__expander {
+    //   $all-cards-width: 100% * $cards-in-row;
+    //   $additional-padding-width: $card-horizontal-gap * ($cards-in-row - 1);
+    //   width: calc(#{$all-cards-width} + #{$additional-padding-width});
+    // }
+    // @for $nth-card from 2 through $cards-in-row { // From second card to rest
+    //   &:nth-of-type(#{$cards-in-row}n+#{$nth-card}) {
+    //     .card__expander {
+    //       $card-left: -100% * ($nth-card - 1);
+    //       $additional-space: $card-horizontal-gap * ($nth-card - 1);
+    //       margin-left: calc(#{$card-left} - #{$additional-space});
+    //     }
+    //   }
+    // }
    // Ensure new line after last row
    $card-after-last: $cards-in-row + 1;
    &:nth-of-type(#{$cards-in-row}n+#{$card-after-last}) {
@@ -304,8 +261,4 @@ $card-horizontal-gap    : $card-gap;
    }
  }
 }
-
-.big-screen     {   @include adaptive-card(3);  }
-.medium-screen  {   @include adaptive-card(2);  }
-.small-screen   {   @include adaptive-card(1);  }
 </style>
--- a/src/presentation/components/Scripts/View/Cards/CardRow.vue
+++ b/src/presentation/components/Scripts/View/Cards/CardRow.vue
--- a/tests/checks/external-urls/DocumentationUrlExtractor.ts
+++ b/tests/checks/external-urls/DocumentationUrlExtractor.ts
@@ -1,69 +0,0 @@
-import type { IApplication } from '@/domain/IApplication';
-import type { TestExecutionDetailsLogger } from './TestExecutionDetailsLogger';
-
-interface UrlExtractionContext {
-  readonly logger: TestExecutionDetailsLogger;
-  readonly application: IApplication;
-  readonly urlExclusionPatterns: readonly RegExp[];
-}
-
-export function extractDocumentationUrls(
-  context: UrlExtractionContext,
-): string[] {
-  const urlsInApplication = extractUrlsFromApplication(context.application);
-  context.logger.logLabeledInformation(
-    'Extracted URLs from application',
-    urlsInApplication.length.toString(),
-  );
-  const uniqueUrls = filterDuplicateUrls(urlsInApplication);
-  context.logger.logLabeledInformation(
-    'Unique URLs after deduplication',
-    `${uniqueUrls.length} (duplicates removed)`,
-  );
-  context.logger.logLabeledInformation(
-    'Exclusion patterns for URLs',
-    context.urlExclusionPatterns.length === 0
-      ? 'None (all URLs included)'
-      : context.urlExclusionPatterns.map((pattern, index) => `${index + 1}) ${pattern.toString()}`).join('\n'),
-  );
-  const includedUrls = filterUrlsExcludingPatterns(uniqueUrls, context.urlExclusionPatterns);
-  context.logger.logLabeledInformation(
-    'URLs extracted for testing',
-    `${includedUrls.length} (after applying exclusion patterns; ${uniqueUrls.length - includedUrls.length} URLs ignored)`,
-  );
-  return includedUrls;
-}
-
-function extractUrlsFromApplication(application: IApplication): string[] {
-  return [ // Get all executables
-    ...application.collections.flatMap((c) => c.getAllCategories()),
-    ...application.collections.flatMap((c) => c.getAllScripts()),
-  ]
-    // Get all docs
-    .flatMap((documentable) => documentable.docs)
-    // Parse all URLs
-    .flatMap((docString) => extractUrlsExcludingCodeBlocks(docString));
-}
-
-function filterDuplicateUrls(urls: readonly string[]): string[] {
-  return urls.filter((url, index, array) => array.indexOf(url) === index);
-}
-
-function filterUrlsExcludingPatterns(
-  urls: readonly string[],
-  patterns: readonly RegExp[],
-): string[] {
-  return urls.filter((url) => !patterns.some((pattern) => pattern.test(url)));
-}
-
-function extractUrlsExcludingCodeBlocks(textWithInlineCode: string): string[] {
-  /*
-    Matches URLs:
-    - Excludes inline code blocks as they may contain URLs not intended for user interaction
-      and not guaranteed to support expected HTTP methods, leading to false-negatives.
-    - Supports URLs containing parentheses, avoiding matches within code that might not represent
-      actual links.
-  */
-  const nonCodeBlockUrlRegex = /(?<!`)(https?:\/\/[^\s`"<>()]+(?:\([^\s`"<>()]*\))?[^\s`"<>()]*)/g;
-  return textWithInlineCode.match(nonCodeBlockUrlRegex) || [];
-}
--- a/tests/checks/external-urls/StatusChecker/BatchStatusChecker.ts
+++ b/tests/checks/external-urls/StatusChecker/BatchStatusChecker.ts
@@ -10,10 +10,7 @@ export async function getUrlStatusesInParallel(
 ): Promise<UrlStatus[]> {
  // urls = ['https://privacy.sexy']; // Comment out this line to use a hardcoded URL for testing.
  const uniqueUrls = Array.from(new Set(urls));
-  const defaultedDomainOptions: Required<DomainOptions> = {
-    ...DefaultDomainOptions,
-    ...options?.domainOptions,
-  };
+  const defaultedDomainOptions = { ...DefaultDomainOptions, ...options?.domainOptions };
  console.log('Batch request options applied:', defaultedDomainOptions);
  const results = await request(uniqueUrls, defaultedDomainOptions, options);
  return results;
--- a/tests/checks/external-urls/StatusChecker/FetchFollow.ts
+++ b/tests/checks/external-urls/StatusChecker/FetchFollow.ts
@@ -1,4 +1,3 @@
-import { indentText } from '@tests/shared/Text';
 import { fetchWithTimeout } from './FetchWithTimeout';
 import { getDomainFromUrl } from './UrlDomainProcessing';

@@ -8,12 +7,8 @@ export function fetchFollow(
  fetchOptions?: Partial<RequestInit>,
  followOptions?: Partial<FollowOptions>,
 ): Promise<Response> {
-  const defaultedFollowOptions: Required<FollowOptions> = {
-    ...DefaultFollowOptions,
-    ...followOptions,
-  };
-  console.log(indentText(`Follow options: ${JSON.stringify(defaultedFollowOptions)}`));
-  if (!followRedirects(defaultedFollowOptions)) {
+  const defaultedFollowOptions = { ...DefaultFollowOptions, ...followOptions };
+  if (followRedirects(defaultedFollowOptions)) {
    return fetchWithTimeout(url, timeoutInMs, fetchOptions);
  }
  fetchOptions = { ...fetchOptions, redirect: 'manual' /* handled manually */, mode: 'cors' };
@@ -27,6 +22,8 @@ export function fetchFollow(
  );
 }

+// "cors" | "navigate" | "no-cors" | "same-origin";
+
 export interface FollowOptions {
  readonly followRedirects?: boolean;
  readonly maximumRedirectFollowDepth?: number;
@@ -101,11 +98,11 @@ class CookieStorage {
 }

 function followRedirects(options: FollowOptions): boolean {
-  if (options.followRedirects !== true) {
+  if (!options.followRedirects) {
    return false;
  }
-  if (options.maximumRedirectFollowDepth === undefined || options.maximumRedirectFollowDepth <= 0) {
-    throw new Error('Invalid followRedirects configuration: maximumRedirectFollowDepth must be a positive integer');
+  if (options.maximumRedirectFollowDepth === 0) {
+    return false;
  }
  return true;
 }
--- a/tests/checks/external-urls/StatusChecker/README.md
+++ b/tests/checks/external-urls/StatusChecker/README.md
@@ -100,10 +100,3 @@ console.log(`Status code: ${status.code}`);
 - **`enableCookies`** (*boolean*), default: `true`
  - Enables cookie storage to facilitate seamless navigation through login or other authentication challenges.
  - 💡 Helps to over-come sign-in challenges with callbacks.
- **`forceHttpGetForUrlPatterns`** (*array*), default: `[]`
-  - Specifies URL patterns that should always use an HTTP GET request instead of the default HTTP HEAD.
-  - This is useful for websites that do not respond to HEAD requests, such as those behind certain CDN or web application firewalls.
-  - Provide patterns as regular expressions (`RegExp`), allowing them to match any part of a URL.
-  - Examples:
-    - To match any URL starting with "https://example.com/api": `/^https:\/\/example\.com\/api/`
-    - To match any domain ending with "cloudflare.com": `/^https:\/\/.*\.cloudflare\.com\//`
--- a/tests/checks/external-urls/StatusChecker/Requestor.ts
+++ b/tests/checks/external-urls/StatusChecker/Requestor.ts
@@ -24,7 +24,6 @@ export interface RequestOptions {
  readonly additionalHeadersUrlIgnore?: string[];
  readonly requestTimeoutInMs: number;
  readonly randomizeTlsFingerprint: boolean;
-  readonly forceHttpGetForUrlPatterns: RegExp[];
 }

 const DefaultOptions: Required<RequestOptions> = {
@@ -33,7 +32,6 @@ const DefaultOptions: Required<RequestOptions> = {
  additionalHeadersUrlIgnore: [],
  requestTimeoutInMs: 60 /* seconds */ * 1000,
  randomizeTlsFingerprint: true,
-  forceHttpGetForUrlPatterns: [],
 };

 function fetchUrlStatusWithRetry(
@@ -43,11 +41,7 @@ function fetchUrlStatusWithRetry(
 ): Promise<UrlStatus> {
  const fetchOptions = getFetchOptions(url, requestOptions);
  return retryWithExponentialBackOff(async () => {
-    console.log(`🚀 Initiating request for URL: ${url}`);
-    console.log(indentText([
-      `HTTP method: ${fetchOptions.method}`,
-      `Request options: ${JSON.stringify(requestOptions)}`,
-    ].join('\n')));
+    console.log(`Initiating request for URL: ${url}`);
    let result: UrlStatus;
    try {
      const response = await fetchFollow(
@@ -62,8 +56,7 @@ function fetchUrlStatusWithRetry(
        url,
        error: [
          'Error:', indentText(JSON.stringify(err, null, '\t') || err.toString()),
-          'Fetch options:', indentText(JSON.stringify(fetchOptions, null, '\t')),
-          'Request options:', indentText(JSON.stringify(requestOptions, null, '\t')),
+          'Options:', indentText(JSON.stringify(fetchOptions, null, '\t')),
          'TLS:', indentText(getTlsContextInfo()),
        ].join('\n'),
      };
@@ -78,7 +71,7 @@ function getFetchOptions(url: string, options: Required<RequestOptions>): Reques
    ? {}
    : options.additionalHeaders;
  return {
-    method: getHttpMethod(url, options),
+    method: 'GET', // Fetch only headers without the full response body for better speed
    headers: {
      ...getDefaultHeaders(url),
      ...additionalHeaders,
@@ -87,14 +80,6 @@ function getFetchOptions(url: string, options: Required<RequestOptions>): Reques
  };
 }

-function getHttpMethod(url: string, options: Required<RequestOptions>): 'HEAD' | 'GET' {
-  if (options.forceHttpGetForUrlPatterns.some((pattern) => url.match(pattern))) {
-    return 'GET';
-  }
-  // By default fetch only headers without the full response body for better speed
-  return 'HEAD';
-}
-
 function getDefaultHeaders(url: string): Record<string, string> {
  return {
    // Needed for websites that filter out non-browser user agents.
--- a/tests/checks/external-urls/StatusChecker/TlsFingerprintRandomizer.ts
+++ b/tests/checks/external-urls/StatusChecker/TlsFingerprintRandomizer.ts
@@ -23,12 +23,12 @@ import { indentText } from '@tests/shared/Text';

 export function randomizeTlsFingerprint() {
  tls.DEFAULT_CIPHERS = getShuffledCiphers().join(':');
-  console.log(indentText(
-    `TLS context:\n${indentText([
+  console.log(
+    [
      'Original ciphers:', indentText(constants.defaultCipherList),
-      'Current ciphers:', indentText(getTlsContextInfo()),
-    ].join('\n'))}`,
-  ));
+      'Current context', indentText(getTlsContextInfo()),
+    ].join('\n'),
+  );
 }

 export function getTlsContextInfo(): string {
--- a/tests/checks/external-urls/TestExecutionDetailsLogger.ts
+++ b/tests/checks/external-urls/TestExecutionDetailsLogger.ts
@@ -1,26 +0,0 @@
-import { indentText } from '@tests/shared/Text';
-
-export class TestExecutionDetailsLogger {
-  public logTestSectionStartDelimiter(): void {
-    this.logSectionDelimiterLine();
-  }
-
-  public logTestSectionEndDelimiter(): void {
-    this.logSectionDelimiterLine();
-  }
-
-  public logLabeledInformation(
-    label: string,
-    detailedInformation: string,
-  ): void {
-    console.log([
-      `${label}:`,
-      indentText(detailedInformation),
-    ].join('\n'));
-  }
-
-  private logSectionDelimiterLine(): void {
-    const horizontalLine = '─'.repeat(40);
-    console.log(horizontalLine);
-  }
-}
--- a/tests/checks/external-urls/main.spec.ts
+++ b/tests/checks/external-urls/main.spec.ts
@@ -1,26 +1,19 @@
 import { test, expect } from 'vitest';
 import { parseApplication } from '@/application/Parser/ApplicationParser';
+import type { IApplication } from '@/domain/IApplication';
 import { indentText } from '@tests/shared/Text';
 import { formatAssertionMessage } from '@tests/shared/FormatAssertionMessage';
-import { shuffle } from '@/application/Common/Shuffle';
 import { type UrlStatus, formatUrlStatus } from './StatusChecker/UrlStatus';
 import { getUrlStatusesInParallel, type BatchRequestOptions } from './StatusChecker/BatchStatusChecker';
-import { TestExecutionDetailsLogger } from './TestExecutionDetailsLogger';
-import { extractDocumentationUrls } from './DocumentationUrlExtractor';

 // arrange
-const logger = new TestExecutionDetailsLogger();
-logger.logTestSectionStartDelimiter();
 const app = parseApplication();
-let urls = extractDocumentationUrls({
-  logger,
-  urlExclusionPatterns: [
+const urls = collectUniqueUrls({
+  application: app,
+  excludePatterns: [
    /^https:\/\/archive\.ph/, // Drops HEAD/GET requests via fetch/curl, responding to Postman/Chromium.
  ],
-  application: app,
 });
-urls = filterUrlsToEnvironmentCheckLimit(urls);
-logger.logLabeledInformation('URLs submitted for testing', urls.length.toString());
 const requestOptions: BatchRequestOptions = {
  domainOptions: {
    sameDomainParallelize: false, // be nice to our third-party servers
@@ -37,65 +30,53 @@ const requestOptions: BatchRequestOptions = {
    enableCookies: true,
  },
 };
-logger.logLabeledInformation('HTTP request options', JSON.stringify(requestOptions, null, 2));
 const testTimeoutInMs = urls.length * 60 /* seconds */ * 1000;
-logger.logLabeledInformation('Scheduled test duration', convertMillisecondsToHumanReadableFormat(testTimeoutInMs));
-logger.logTestSectionEndDelimiter();
 test(`all URLs (${urls.length}) should be alive`, async () => {
  // act
-  console.log('URLS', urls); // TODO: Delete
  const results = await getUrlStatusesInParallel(urls, requestOptions);
  // assert
  const deadUrls = results.filter((r) => r.code === undefined || !isOkStatusCode(r.code));
-  expect(deadUrls).to.have.lengthOf(
-    0,
-    formatAssertionMessage([createReportForDeadUrlStatuses(deadUrls)]),
-  );
+  expect(deadUrls).to.have.lengthOf(0, formatAssertionMessage([formatUrlStatusReport(deadUrls)]));
 }, testTimeoutInMs);

 function isOkStatusCode(statusCode: number): boolean {
  return statusCode >= 200 && statusCode < 300;
 }

-function createReportForDeadUrlStatuses(deadUrlStatuses: readonly UrlStatus[]): string {
+function collectUniqueUrls(
+  options: {
+    readonly application: IApplication,
+    readonly excludePatterns?: readonly RegExp[],
+  },
+): string[] {
+  return [ // Get all nodes
+    ...options.application.collections.flatMap((c) => c.getAllCategories()),
+    ...options.application.collections.flatMap((c) => c.getAllScripts()),
+  ]
+    // Get all docs
+    .flatMap((documentable) => documentable.docs)
+    // Parse all URLs
+    .flatMap((docString) => extractUrls(docString))
+    // Remove duplicates
+    .filter((url, index, array) => array.indexOf(url) === index)
+    // Exclude certain URLs based on patterns
+    .filter((url) => !shouldExcludeUrl(url, options.excludePatterns ?? []));
+}
+
+function shouldExcludeUrl(url: string, patterns: readonly RegExp[]): boolean {
+  return patterns.some((pattern) => pattern.test(url));
+}
+
+function formatUrlStatusReport(deadUrlStatuses: readonly UrlStatus[]): string {
  return `\n${deadUrlStatuses.map((status) => indentText(formatUrlStatus(status))).join('\n---\n')}\n`;
 }

-function filterUrlsToEnvironmentCheckLimit(originalUrls: string[]): string[] {
-  const { RANDOMIZED_URL_CHECK_LIMIT } = process.env;
-  logger.logLabeledInformation('URL check limit', RANDOMIZED_URL_CHECK_LIMIT || 'Unlimited');
-  if (RANDOMIZED_URL_CHECK_LIMIT !== undefined && RANDOMIZED_URL_CHECK_LIMIT !== '') {
-    const maxUrlsInTest = parseInt(RANDOMIZED_URL_CHECK_LIMIT, 10);
-    if (Number.isNaN(maxUrlsInTest)) {
-      throw new Error(`Invalid URL limit: ${RANDOMIZED_URL_CHECK_LIMIT}`);
-    }
-    if (maxUrlsInTest < originalUrls.length) {
-      return shuffle(originalUrls).slice(0, maxUrlsInTest);
-    }
-  }
-  return originalUrls;
-}
-
-function convertMillisecondsToHumanReadableFormat(milliseconds: number): string {
-  const timeParts: string[] = [];
-  const addTimePart = (amount: number, label: string) => {
-    if (amount === 0) {
-      return;
-    }
-    timeParts.push(`${amount} ${label}`);
-  };
-
-  const hours = milliseconds / (1000 * 60 * 60);
-  const absoluteHours = Math.floor(hours);
-  addTimePart(absoluteHours, 'hours');
-
-  const minutes = (hours - absoluteHours) * 60;
-  const absoluteMinutes = Math.floor(minutes);
-  addTimePart(absoluteMinutes, 'minutes');
-
-  const seconds = (minutes - absoluteMinutes) * 60;
-  const absoluteSeconds = Math.floor(seconds);
-  addTimePart(absoluteSeconds, 'seconds');
-
-  return timeParts.join(', ');
+function extractUrls(textWithInlineCode: string): string[] {
+  /*
+    Matches all URLs.
+    Inline code blocks contain URLs not intended for user interaction and not
+    guaranteed to support expected HTTP methods, leading to false-negatives.
+  */
+  const nonCodeBlockUrlRegex = /(?<!`)(https?:\/\/[^\s`"<>()]+)/g;
+  return textWithInlineCode.match(nonCodeBlockUrlRegex) || [];
 }
--- a/tests/unit/application/Common/Shuffle.spec.ts
+++ b/tests/unit/application/Common/Shuffle.spec.ts
@@ -1,52 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { shuffle } from '@/application/Common/Shuffle';
-
-describe('Shuffle', () => {
-  describe('shuffle', () => {
-    it('returns a new array', () => {
-      // arrange
-      const inputArray = ['a', 'b', 'c', 'd'];
-      // act
-      const result = shuffle(inputArray);
-      // assert
-      expect(result).not.to.equal(inputArray);
-    });
-
-    it('returns an array of the same length', () => {
-      // arrange
-      const inputArray = ['a', 'b', 'c', 'd'];
-      // act
-      const result = shuffle(inputArray);
-      // assert
-      expect(result.length).toBe(inputArray.length);
-    });
-
-    it('contains the same elements', () => {
-      // arrange
-      const inputArray = ['a', 'b', 'c', 'd'];
-      // act
-      const result = shuffle(inputArray);
-      // assert
-      expect(result).to.have.members(inputArray);
-    });
-
-    it('does not modify the input array', () => {
-      // arrange
-      const inputArray = ['a', 'b', 'c', 'd'];
-      const inputArrayCopy = [...inputArray];
-      // act
-      shuffle(inputArray);
-      // assert
-      expect(inputArray).to.deep.equal(inputArrayCopy);
-    });
-
-    it('handles an empty array correctly', () => {
-      // arrange
-      const inputArray: string[] = [];
-      // act
-      const result = shuffle(inputArray);
-      // assert
-      expect(result).have.lengthOf(0);
-    });
-  });
-});
--- a/tests/unit/application/Parser/Script/Compiler/Function/Call/Argument/FunctionCallArgumentCollection.spec.ts
+++ b/tests/unit/application/Parser/Script/Compiler/Function/Call/Argument/FunctionCallArgumentCollection.spec.ts
@@ -2,7 +2,6 @@ import { describe, it, expect } from 'vitest';
 import { FunctionCallArgumentCollection } from '@/application/Parser/Script/Compiler/Function/Call/Argument/FunctionCallArgumentCollection';
 import { FunctionCallArgumentStub } from '@tests/unit/shared/Stubs/FunctionCallArgumentStub';
 import { itEachAbsentStringValue } from '@tests/unit/shared/TestCases/AbsentTests';
-import type { IFunctionCallArgument } from '@/application/Parser/Script/Compiler/Function/Call/Argument/IFunctionCallArgument';

 describe('FunctionCallArgumentCollection', () => {
  describe('addArgument', () => {
@@ -21,25 +20,21 @@ describe('FunctionCallArgumentCollection', () => {
    });
  });
  describe('getAllParameterNames', () => {
-    describe('returns as expected', () => {
+    it('returns as expected', () => {
      // arrange
-      const testCases: ReadonlyArray<{
-        readonly description: string;
-        readonly args: readonly IFunctionCallArgument[];
-        readonly expectedParameterNames: string[];
-      }> = [{
-        description: 'no args',
+      const testCases = [{
+        name: 'no args',
        args: [],
-        expectedParameterNames: [],
+        expected: [],
      }, {
-        description: 'with some args',
+        name: 'with some args',
        args: [
          new FunctionCallArgumentStub().withParameterName('a-param-name'),
          new FunctionCallArgumentStub().withParameterName('b-param-name')],
-        expectedParameterNames: ['a-param-name', 'b-param-name'],
+        expected: ['a-param-name', 'b-param-name'],
      }];
      for (const testCase of testCases) {
-        it(testCase.description, () => {
+        it(testCase.name, () => {
          const sut = new FunctionCallArgumentCollection();
          // act
          for (const arg of testCase.args) {
@@ -47,7 +42,7 @@ describe('FunctionCallArgumentCollection', () => {
          }
          const actual = sut.getAllParameterNames();
          // assert
-          expect(actual).to.deep.equal(testCase.expectedParameterNames);
+          expect(actual).to.equal(testCase.expected);
        });
      }
    });
--- a/tests/unit/application/Parser/Script/Compiler/Function/SharedFunction.spec.ts
+++ b/tests/unit/application/Parser/Script/Compiler/Function/SharedFunction.spec.ts
@@ -25,7 +25,7 @@ describe('SharedFunction', () => {
          // assert
          expect(sut.name).equal(expected);
        });
-        describe('throws when absent', () => {
+        it('throws when absent', () => {
          itEachAbsentStringValue((absentValue) => {
            // arrange
            const expectedError = 'missing function name';
--- a/tests/unit/presentation/bootstrapping/DependencyProvider.spec.ts
+++ b/tests/unit/presentation/bootstrapping/DependencyProvider.spec.ts
@@ -4,7 +4,6 @@ import { InjectionKeys } from '@/presentation/injectionSymbols';
 import { provideDependencies, type VueDependencyInjectionApi } from '@/presentation/bootstrapping/DependencyProvider';
 import { ApplicationContextStub } from '@tests/unit/shared/Stubs/ApplicationContextStub';
 import { itIsSingleton } from '@tests/unit/shared/TestCases/SingletonTests';
-import type { IApplicationContext } from '@/application/Context/IApplicationContext';

 describe('DependencyProvider', () => {
  describe('provideDependencies', () => {
@@ -75,25 +74,25 @@ function createSingletonTests() {
      const registeredObject = api.inject(injectionKey);
      expect(registeredObject).to.be.instanceOf(Object);
    });
-    describe('should return the same instance for singleton dependency', () => {
-      // arrange
-      const singletonContext = new ApplicationContextStub();
-      const api = new VueDependencyInjectionApiStub();
-      new ProvideDependenciesBuilder()
-        .withContext(singletonContext)
-        .withApi(api)
-        .provideDependencies();
-      // act
-      const getRegisteredInstance = () => api.inject(injectionKey);
-      // assert
+    it('should return the same instance for singleton dependency', () => {
      itIsSingleton({
-        getter: getRegisteredInstance,
+        getter: () => {
+          // arrange
+          const api = new VueDependencyInjectionApiStub();
+          // act
+          new ProvideDependenciesBuilder()
+            .withApi(api)
+            .provideDependencies();
+          // expect
+          const registeredObject = api.inject(injectionKey);
+          return registeredObject;
+        },
      });
    });
  };
 }
 class ProvideDependenciesBuilder {
-  private context: IApplicationContext = new ApplicationContextStub();
+  private context = new ApplicationContextStub();

  private api: VueDependencyInjectionApi = new VueDependencyInjectionApiStub();

@@ -102,11 +101,6 @@ class ProvideDependenciesBuilder {
    return this;
  }

-  public withContext(context: IApplicationContext): this {
-    this.context = context;
-    return this;
-  }
-
  public provideDependencies() {
    return provideDependencies(this.context, this.api);
  }
--- a/tests/unit/presentation/components/Scripts/View/Tree/TreeView/TreeRoot/NodeCollection/TreeNodeInitializerAndUpdater.spec.ts
+++ b/tests/unit/presentation/components/Scripts/View/Tree/TreeView/TreeRoot/NodeCollection/TreeNodeInitializerAndUpdater.spec.ts
@@ -10,7 +10,7 @@ import type { TreeInputNodeData } from '@/presentation/components/Scripts/View/T

 describe('TreeNodeInitializerAndUpdater', () => {
  describe('updateRootNodes', () => {
-    describe('should throw an error if no data is provided', () => {
+    it('should throw an error if no data is provided', () => {
      itEachAbsentCollectionValue<TreeInputNodeData>((absentValue) => {
        // arrange
        const expectedError = 'missing data';
--- a/tests/unit/presentation/electron/main/IpcRegistration.spec.ts
+++ b/tests/unit/presentation/electron/main/IpcRegistration.spec.ts
@@ -13,7 +13,7 @@ import { ScriptDiagnosticsCollectorStub } from '../../../shared/Stubs/ScriptDiag

 describe('IpcRegistration', () => {
  describe('registerAllIpcChannels', () => {
-    describe('registers all defined IPC channels', () => {
+    it('registers all defined IPC channels', () => {
      Object.entries(IpcChannelDefinitions).forEach(([key, expectedChannel]) => {
        it(key, () => {
          // arrange