Skip to content

Commit b9f1aeb

Browse files
committed
fix: remove empty APK placeholder rows and dedupe APK tabs
Filter parsed apkx rows that only contain placeholder values (target '-' with autoar/apkx finding) and suppress module tab duplication when the apkx dataset tab already exists. Made-with: Cursor
1 parent b87128e commit b9f1aeb

2 files changed

Lines changed: 14 additions & 0 deletions

File tree

internal/modules/gobot/scan_results_api.go

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1138,6 +1138,15 @@ func apiScanParsedResults(c *gin.Context) {
11381138
if len(rows) >= limit {
11391139
return
11401140
}
1141+
if scanType == "apkx" {
1142+
// Drop placeholder rows produced by summary objects with no concrete finding fields.
1143+
f := strings.ToLower(strings.TrimSpace(r.Finding))
1144+
t := strings.TrimSpace(r.Target)
1145+
if (f == "" || f == "—" || f == "autoar" || f == "apkx") &&
1146+
(t == "" || t == "—" || t == "-") {
1147+
continue
1148+
}
1149+
}
11411150
r.File = e.FileName
11421151
r.Module = module
11431152
r.Category = category

internal/modules/gobot/ui/app.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4309,6 +4309,11 @@ async function loadReconUnifiedTable(scanId, allFiles, containerId, scanRecord)
43094309
if (hasUrlsDatasetTab) {
43104310
excludedModuleTabs.add('url-collection');
43114311
}
4312+
// APK dataset tab already exists; skip duplicate module tab.
4313+
const hasApkxDatasetTab = UNIQUE_TABS.some((t) => t[0] === 'apkx');
4314+
if (hasApkxDatasetTab) {
4315+
excludedModuleTabs.add('apkx');
4316+
}
43124317
const moduleTabs = usedModules.filter((mod) => !excludedModuleTabs.has(mod)).map((mod) => {
43134318
const info = getModuleDisplayInfo(mod);
43144319
return [`mod:${mod}`, `${info.icon} ${info.name}`];

0 commit comments

Comments
 (0)