[deep-report] [Code Quality] Extract shared base for AccessLogSummary/FirewallLogSummary in logs_report_firewall.go

[github-actions]

Description

pkg/cli/logs_report_firewall.go declares two ~80%-overlapping summary structs:

• AccessLogSummary (line ~11)
• FirewallLogSummary (line ~21)

Both carry TotalRequests, allowed/blocked counters, AllowedDomains, BlockedDomains, and ByWorkflow. They differ only in counter field naming (AllowedCount/BlockedCount vs AllowedRequests/BlockedRequests) and one extra RequestsByDomain map on the firewall variant. Extract a shared base struct, align the counter field names, and embed it in both.

Expected Impact

Removes parallel-maintenance drift between two firewall summary types. Bounded, single-file refactor.

Suggested Agent

Code Quality agent (Go refactor).

Estimated Effort

Medium (1-2 hours)

Data Source

DeepReport Intelligence Briefing 2026-07-16, from the [typist] Go Type Consistency report (Cluster 4). Verified un-filed as of this run.

Generated by 🔬 Deep Report · 247.5 AIC · ⌖ 12.4 AIC · ⊞ 10K · ◷

• expires on Jul 18, 2026, 7:30 AM UTC-08:00

[github-actions]

🍪 Issue Monster selected this for Copilot

I've identified this issue as a good candidate for automated resolution and requested assignment to the Copilot coding agent.

If assignment succeeds, the Copilot coding agent will analyze the issue and create a pull request with the fix.

Om nom nom! 🍪

🍪 Om nom nom by Issue Monster · 0.835 AIC · ⌖ 2.97 AIC · ⊞ 6K · ◷