From d1fc823778f6de719d720c2481656c0b8fccaa94 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Fri, 3 Apr 2026 11:25:18 +0000
Subject: [PATCH 1/2] Initial plan


From 4ce0ce24a3ef7fab6d75caf181f2716bb2cd0a32 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Fri, 3 Apr 2026 11:30:29 +0000
Subject: [PATCH 2/2] rename isEmptyDiff to isEmptyFirewallDiff for clarity

Agent-Logs-Url: https://github.com/github/gh-aw/sessions/d8a40454-719a-4040-8489-4e70e336adb7

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
---
 pkg/cli/audit_diff_render.go | 12 ++++++------
 pkg/cli/audit_diff_test.go   |  4 ++--
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/pkg/cli/audit_diff_render.go b/pkg/cli/audit_diff_render.go
index c7f7c81f0cb..9c318ce2cb3 100644
--- a/pkg/cli/audit_diff_render.go
+++ b/pkg/cli/audit_diff_render.go
@@ -50,7 +50,7 @@ func renderAuditDiffPretty(diff *AuditDiff) {
 	var summaryParts []string
 	anomalyCount := 0
 
-	if diff.FirewallDiff != nil && !isEmptyDiff(diff.FirewallDiff) {
+	if diff.FirewallDiff != nil && !isEmptyFirewallDiff(diff.FirewallDiff) {
 		fwParts := []string{}
 		if len(diff.FirewallDiff.NewDomains) > 0 {
 			fwParts = append(fwParts, fmt.Sprintf("%d new domains", len(diff.FirewallDiff.NewDomains)))
@@ -102,7 +102,7 @@ func renderAuditDiffPretty(diff *AuditDiff) {
 
 // renderFirewallDiffMarkdownSection renders the firewall diff sub-section as markdown
 func renderFirewallDiffMarkdownSection(diff *FirewallDiff) {
-	if diff == nil || isEmptyDiff(diff) {
+	if diff == nil || isEmptyFirewallDiff(diff) {
 		return
 	}
 
@@ -232,7 +232,7 @@ func renderRunMetricsDiffMarkdownSection(run1ID, run2ID int64, diff *RunMetricsD
 
 // renderFirewallDiffPrettySection renders the firewall diff as a pretty console sub-section
 func renderFirewallDiffPrettySection(diff *FirewallDiff) {
-	if diff == nil || isEmptyDiff(diff) {
+	if diff == nil || isEmptyFirewallDiff(diff) {
 		return
 	}
 
@@ -370,8 +370,8 @@ func statusEmoji(status string) string {
 	}
 }
 
-// isEmptyDiff returns true if the firewall diff contains no changes
-func isEmptyDiff(diff *FirewallDiff) bool {
+// isEmptyFirewallDiff returns true if the firewall diff contains no changes
+func isEmptyFirewallDiff(diff *FirewallDiff) bool {
 	return len(diff.NewDomains) == 0 &&
 		len(diff.RemovedDomains) == 0 &&
 		len(diff.StatusChanges) == 0 &&
@@ -387,7 +387,7 @@ func isEmptyMCPToolsDiff(diff *MCPToolsDiff) bool {
 
 // isEmptyAuditDiff returns true if the audit diff contains no changes across all sections
 func isEmptyAuditDiff(diff *AuditDiff) bool {
-	fwEmpty := diff.FirewallDiff == nil || isEmptyDiff(diff.FirewallDiff)
+	fwEmpty := diff.FirewallDiff == nil || isEmptyFirewallDiff(diff.FirewallDiff)
 	mcpEmpty := diff.MCPToolsDiff == nil || isEmptyMCPToolsDiff(diff.MCPToolsDiff)
 	metricsEmpty := diff.RunMetricsDiff == nil
 	return fwEmpty && mcpEmpty && metricsEmpty
diff --git a/pkg/cli/audit_diff_test.go b/pkg/cli/audit_diff_test.go
index a8cff83ea54..774dcb12dbb 100644
--- a/pkg/cli/audit_diff_test.go
+++ b/pkg/cli/audit_diff_test.go
@@ -306,12 +306,12 @@ func TestStatusEmoji(t *testing.T) {
 
 func TestIsEmptyDiff(t *testing.T) {
 	emptyDiff := &FirewallDiff{}
-	assert.True(t, isEmptyDiff(emptyDiff), "Empty diff should be detected")
+	assert.True(t, isEmptyFirewallDiff(emptyDiff), "Empty diff should be detected")
 
 	nonEmptyDiff := &FirewallDiff{
 		NewDomains: []DomainDiffEntry{{Domain: "test.com"}},
 	}
-	assert.False(t, isEmptyDiff(nonEmptyDiff), "Non-empty diff should not be detected as empty")
+	assert.False(t, isEmptyFirewallDiff(nonEmptyDiff), "Non-empty diff should not be detected as empty")
 }
 
 // findDiffEntry is a test helper to find a domain in a list of diff entries