Merge pull request #1396 from pikasTech/fix/1385-dedupe-findings

修正 Web 哨兵 findings 回填去重
This commit is contained in:
Lyon
2026-07-01 17:46:16 +08:00
committed by GitHub
2 changed files with 3 additions and 2 deletions
+1 -1
View File
@@ -333,7 +333,7 @@ function mergeSentinelReportFindings(primary: readonly Record<string, unknown>[]
const seen = new Set<string>();
for (const item of [...primary, ...artifact]) {
const id = sentinelReportFindingIdentity(item);
const key = `${id ?? JSON.stringify(item).slice(0, 160)}:${stringAtNullable(item, "severity") ?? stringAtNullable(item, "level") ?? ""}`;
const key = id ?? `${JSON.stringify(item).slice(0, 160)}:${stringAtNullable(item, "severity") ?? stringAtNullable(item, "level") ?? ""}`;
if (seen.has(key)) continue;
seen.add(key);
merged.push(item);
@@ -1536,7 +1536,8 @@ function mergeVisibleFindings(primary: readonly Record<string, unknown>[], artif
const merged: Record<string, unknown>[] = [];
const seen = new Set<string>();
for (const item of [...primary, ...artifact]) {
const key = `${findingIdentity(item) ?? JSON.stringify(item).slice(0, 160)}:${stringOrNull(item.severity) ?? stringOrNull(item.level) ?? ""}`;
const id = findingIdentity(item);
const key = id ?? `${JSON.stringify(item).slice(0, 160)}:${stringOrNull(item.severity) ?? stringOrNull(item.level) ?? ""}`;
if (seen.has(key)) continue;
seen.add(key);
merged.push(item);