Objective
Split pkg/cli/audit_report_render.go (1,139 lines) into focused sub-files following the audit_report_render_{domain}.go naming pattern.
Context
Discussion: #26221
This file contains 10+ logically independent render functions and directly violates the documented 300-line hard limit. The compile pipeline already demonstrates the correct pattern.
Approach
Move domain-specific render functions to new files:
audit_report_render_overview.go — renderOverview, renderMetrics
audit_report_render_jobs.go — renderJobsTable
audit_report_render_tools.go — renderToolUsageTable, renderMCPToolUsageTable
audit_report_render_firewall.go — renderFirewallAnalysis, renderRedactedDomainsAnalysis
audit_report_render_guard.go — renderGuardPolicySummary
audit_report_render_findings.go — renderKeyFindings, renderRecommendations, renderCreatedItemsTable
- Keep
renderConsole, renderJSON, and renderAuditComparison in the original file (entry points)
Files to Modify
- Split:
pkg/cli/audit_report_render.go
- Create:
pkg/cli/audit_report_render_overview.go
- Create:
pkg/cli/audit_report_render_jobs.go
- Create:
pkg/cli/audit_report_render_tools.go
- Create:
pkg/cli/audit_report_render_firewall.go
- Create:
pkg/cli/audit_report_render_guard.go
- Create:
pkg/cli/audit_report_render_findings.go
Acceptance Criteria
Generated by Plan Command for issue #discussion #26221 · ● 150.1K · ◷
Objective
Split
pkg/cli/audit_report_render.go(1,139 lines) into focused sub-files following theaudit_report_render_{domain}.gonaming pattern.Context
Discussion: #26221
This file contains 10+ logically independent render functions and directly violates the documented 300-line hard limit. The compile pipeline already demonstrates the correct pattern.
Approach
Move domain-specific render functions to new files:
audit_report_render_overview.go—renderOverview,renderMetricsaudit_report_render_jobs.go—renderJobsTableaudit_report_render_tools.go—renderToolUsageTable,renderMCPToolUsageTableaudit_report_render_firewall.go—renderFirewallAnalysis,renderRedactedDomainsAnalysisaudit_report_render_guard.go—renderGuardPolicySummaryaudit_report_render_findings.go—renderKeyFindings,renderRecommendations,renderCreatedItemsTablerenderConsole,renderJSON, andrenderAuditComparisonin the original file (entry points)Files to Modify
pkg/cli/audit_report_render.gopkg/cli/audit_report_render_overview.gopkg/cli/audit_report_render_jobs.gopkg/cli/audit_report_render_tools.gopkg/cli/audit_report_render_firewall.gopkg/cli/audit_report_render_guard.gopkg/cli/audit_report_render_findings.goAcceptance Criteria
audit_report_render.gois ≤ 300 lines after the splitmake build && make test-unitpass without errorsmake fmt && make lintpass without errors