fix: repair JaCoCo sourcefile paths before CodeScene upload

 Pester's JaCoCo output (after the source-mirrored coverage migration)
 emits package + sourcefilename pairs that double the relative path
 (package="src/private/build" + sourcefilename="private/build/Foo.ps1"),
 so CodeScene's cs-coverage rejects the upload with "The coverage data
 does not contains any records related to the current repo".

 Invoke-CodeSceneAnalysis.ps1 now normalizes <class sourcefilename> and
 <sourcefile name> attributes to bare filenames before upload so
 package + sourcefile resolves to real repository paths.

Author: stiwicourage

CHANGELOG.md

@@ -53,6 +53,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 ### Fixed
 - `Test-NovaBuild` and `% nova test` no longer emit `Remove-Item -Recurse` progress bars during the Pester run; `$global:ProgressPreference` is saved and restored around the test execution so CI logs stay clean.
 - Code coverage in `Test-NovaBuild` now runs against the source files, instead of the built `dist/<ModuleName>/<ModuleName>.psm1` file so measured coverage percentages and line numbers align with the deployed module.
+- `Invoke-CodeSceneAnalysis.ps1` now normalizes Pester's JaCoCo output before upload so the `<class sourcefilename>` and `<sourcefile name>` entries are bare filenames and `package + sourcefile` resolves to real repository paths. This fixes the CodeScene `cs-coverage upload` failure "The coverage data does not contains any records related to the current repo" that appeared after the source-mirrored coverage migration.
 
 
 ### Security

scripts/build/ci/Invoke-CodeSceneAnalysis.ps1

@@ -103,6 +103,36 @@ function Invoke-CodeSceneAnalysisTrigger {
     }
 }
 
+function Repair-PesterJaCoCoCoveragePath {
+    param([Parameter(Mandatory)][string]$Path)
+
+    [xml]$document = Get-Content -LiteralPath $Path -Raw
+    $changed = $false
+
+    foreach ($class in $document.SelectNodes('//class[@sourcefilename]')) {
+        $original = $class.GetAttribute('sourcefilename')
+        $normalized = [System.IO.Path]::GetFileName($original)
+        if ($normalized -ne $original) {
+            $class.SetAttribute('sourcefilename', $normalized)
+            $changed = $true
+        }
+    }
+
+    foreach ($sourcefile in $document.SelectNodes('//sourcefile[@name]')) {
+        $original = $sourcefile.GetAttribute('name')
+        $normalized = [System.IO.Path]::GetFileName($original)
+        if ($normalized -ne $original) {
+            $sourcefile.SetAttribute('name', $normalized)
+            $changed = $true
+        }
+    }
+
+    if ($changed) {
+        $document.Save($Path)
+        Write-Host "Normalized JaCoCo sourcefile paths in '$Path' so package + sourcefile resolve to repo files."
+    }
+}
+
 $shouldUploadCoverage = Test-CodeSceneCoverageUploadRequested -CoveragePath $CoveragePath -UploadCoverage:$UploadCoverage
 $shouldRunAnalysis = $TriggerAnalysis.IsPresent
 
@@ -117,6 +147,7 @@ $accessToken = Get-RequiredCodeSceneValue -Name 'CS_ACCESS_TOKEN'
 
 if ($shouldUploadCoverage) {
     $resolvedCoveragePath = Resolve-CodeSceneCoveragePath -CoveragePath $CoveragePath
+    Repair-PesterJaCoCoCoveragePath -Path $resolvedCoveragePath
 
     if (-not (Get-Command -Name 'cs-coverage' -ErrorAction SilentlyContinue)) {
         throw "The 'cs-coverage' CLI was not found on PATH. Install the CodeScene coverage upload tool before running this script."

tests/CodeSceneAnalysis.Tests.ps1

@@ -162,4 +162,42 @@ Set-Location '$emptyWorkingDir'
         $result.ExitCode | Should -Not -Be 0
         ($result.Output -join [Environment]::NewLine) | Should -Match 'No JaCoCo coverage file was found at'
     }
+
+    It 'normalizes Pester JaCoCo sourcefile paths so package + sourcefile resolves to repo files' {
+        $coverageDir = Join-Path $TestDrive 'pester-jacoco'
+        New-Item -ItemType Directory -Path $coverageDir -Force | Out-Null
+        $coveragePath = Join-Path $coverageDir 'coverage.xml'
+        Set-Content -LiteralPath $coveragePath -Encoding utf8 -Value @'
+<?xml version="1.0" encoding="UTF-8" standalone="no"?>
+<report name="Pester">
+  <package name="src/private/build">
+    <class name="src/private/build/Foo" sourcefilename="private/build/Foo.ps1" />
+    <sourcefile name="private/build/Foo.ps1" />
+  </package>
+</report>
+'@
+
+        $runnerContent = @"
+function cs-coverage {
+    `$global:LASTEXITCODE = 0
+    return
+}
+
+[Environment]::SetEnvironmentVariable('CS_URL', 'https://codescene.example.test')
+[Environment]::SetEnvironmentVariable('CS_PROJECT_ID', '123')
+[Environment]::SetEnvironmentVariable('CS_ACCESS_TOKEN', 'token')
+
+& '$codeSceneAnalysisScriptPath' -CoveragePath '$coveragePath'
+"@
+
+        $result = Invoke-CodeSceneAnalysisTestScript -RunnerContent $runnerContent
+
+        $result.ExitCode | Should -Be 0 -Because ($result.Output -join [Environment]::NewLine)
+
+        [xml]$rewritten = Get-Content -LiteralPath $coveragePath -Raw
+        $class = $rewritten.SelectSingleNode('//class[@sourcefilename]')
+        $class.GetAttribute('sourcefilename') | Should -Be 'Foo.ps1'
+        $sourcefile = $rewritten.SelectSingleNode('//sourcefile[@name]')
+        $sourcefile.GetAttribute('name') | Should -Be 'Foo.ps1'
+    }
 }