mi/werkator
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Link index-less report pages from the artifact index (v0.9.8)

Browse Source 
Gradle's --profile report is archived but was unreachable: report discovery
only looked for index.html, while the profile page carries a timestamped
file name. Scan reports/ and its direct sub-directories for HTML pages that
no index covers, so a report tree cannot flood the index with inner pages.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
mhoennig
2026-08-10 21:11:15 +02:00
co-authored by Claude Fable 5
parent 632e4396e4
commit c1869ea427
4 changed files with 66 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/main/kotlin/de/hoennig/gittally/server/UiController.kt
+36 -3
View File
@@ -226,7 +226,7 @@ class UiController(
}
/**
* The browsable `index.html` pages under `reports/`, shallowest first; pages nested
* The browsable report pages under `reports/`, shallowest first; pages nested
* below an already-listed report index are skipped — like the legacy artifact index.
* Each entry carries the report's failures counter for the failed-badge.
*/
@@ -248,13 +248,46 @@ class UiController(
val topmost = mutableListOf<String>()
for (relativeIndex in allIndexes) {
val dir = relativeIndex.substringBeforeLast('/', "")
if (knownDirs.any { known -> known.isEmpty() || dir == known || dir.startsWith("$known/") }) {
if (dir.isCoveredBy(knownDirs)) {
continue
}
knownDirs += dir
topmost += relativeIndex
}
return topmost.map { ReportIndexView(path = it, failures = reportFailures(reportsDir.resolve(it))) }
return (topmost + indexLessReportPages(reportsDir, knownDirs))
.map { ReportIndexView(path = it, failures = reportFailures(reportsDir.resolve(it))) }
}
private fun String.isCoveredBy(knownDirs: List<String>): Boolean =
knownDirs.any { known -> known.isEmpty() || this == known || this.startsWith("$known/") }
/**
* Report pages of directories without an `index.html`, such as Gradle's `--profile` report
* with its timestamped file name. Only `reports/` itself and its direct sub-directories are
* scanned, so that a report tree cannot flood the artifact index with its inner pages.
*/
private fun indexLessReportPages(
reportsDir: Path,
knownDirs: List<String>,
): List<String> {
val candidateDirs =
buildList {
add(reportsDir)
Files.list(reportsDir).use { children ->
children.asSequence().filter { Files.isDirectory(it) }.forEach { add(it) }
}
}
return candidateDirs
.filterNot { reportsDir.relativize(it).toString().isCoveredBy(knownDirs) }
.flatMap { dir ->
Files.list(dir).use { pages ->
pages
.asSequence()
.filter { Files.isRegularFile(it) && it.name.endsWith(".html") }
.map { reportsDir.relativize(it).toString() }
.toList()
}
}.sorted()
}
/**