Skip to content

Commit 83d756b

Browse files
Toreburnclaude
andcommitted
Improve fetch log display clarity
- Show "X vendors scanned" and "Y with recent patches" instead of confusing "Vendors: 12" which didn't explain why only 12 of 56. - Always show whether new patches were found or "no new patches". - Summary pills now show "X with new patches", "Y unchanged", "Z failed" instead of generic "successful" which was misleading. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 26f68a7 commit 83d756b

2 files changed

Lines changed: 14 additions & 5 deletions

File tree

script.js

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -361,18 +361,24 @@ document.addEventListener('DOMContentLoaded', () => {
361361
return `<div class="log-line"><span class="status-icon"></span><span class="vendor-name">${escapeHtml(l)}</span></div>`;
362362
}).join('');
363363

364+
const scanned = log.vendorsScanned || log.vendors?.length || 0;
365+
const active = log.vendorsWithPatches || log.vendors?.length || 0;
366+
const info = (log.logs || []).filter(l => l.includes('[INFO]')).length;
367+
364368
return `<div class="patch-card">
365369
<div class="patch-header"><h3>Fetch at ${new Date(log.timestamp).toLocaleString()}</h3></div>
366370
<div class="patch-meta">
367-
<span class="tag">Vendors: ${log.vendors?.length || 0}</span>
368-
<span class="tag">${log.totalPatches || 0} patches</span>
369-
${log.newPatchCount > 0 ? `<span class="tag" style="color:var(--success)">${log.newPatchCount} new</span>` : ''}
371+
<span class="tag">${scanned} vendors scanned</span>
372+
<span class="tag">${active} with recent patches</span>
373+
<span class="tag">${log.totalPatches || 0} total patches</span>
374+
${log.newPatchCount > 0 ? `<span class="tag" style="color:var(--success)">+${log.newPatchCount} new</span>` : '<span class="tag" style="color:var(--text-dim)">no new patches</span>'}
370375
</div>
371376
${npSection}
372377
<div class="vendor-logs">
373378
<h4>Vendor Status</h4>
374379
<div class="log-summary">
375-
${ok > 0 ? `<div class="log-summary-item success-count">${ok} successful</div>` : ''}
380+
${ok > 0 ? `<div class="log-summary-item success-count">${ok} with new patches</div>` : ''}
381+
${info > 0 ? `<div class="log-summary-item" style="background:rgba(124,110,246,0.1);color:var(--accent);border:1px solid rgba(124,110,246,0.2)">${info} unchanged</div>` : ''}
376382
${fail > 0 ? `<div class="log-summary-item error-count">${fail} failed</div>` : ''}
377383
</div>
378384
<div class="log-grid">${vendorLines}</div>

scripts/fetch-patches.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,12 +91,15 @@ async function fetchPatches() {
9191
}
9292

9393
// Create new log entry
94+
const vendorsWithPatches = Array.from(new Set(allPatches.map(patch => patch.vendor)));
9495
const logEntry = {
9596
timestamp: new Date().toISOString(),
9697
logs,
9798
newPatchCount,
9899
totalPatches: allPatches.length,
99-
vendors: Array.from(new Set(allPatches.map(patch => patch.vendor))),
100+
vendorsScanned: vendorFiles.filter(f => f.endsWith('.json')).length,
101+
vendorsWithPatches: vendorsWithPatches.length,
102+
vendors: vendorsWithPatches,
100103
newPatches: allNewPatches.map(patch => ({
101104
vendor: patch.vendor,
102105
title: patch.title,

0 commit comments

Comments
 (0)