From e1c6052e04a0173be2ef0d45d50c2299da1248b4 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 26 Mar 2026 09:00:29 +0000 Subject: [PATCH] fix: derive writeCount from single source to prevent double-counting in agentic analysis Agent-Logs-Url: https://github.com/github/gh-aw/sessions/e0315538-7b99-4c78-9bf3-2dd647e93fdf Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com> --- pkg/cli/audit_agentic_analysis.go | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/pkg/cli/audit_agentic_analysis.go b/pkg/cli/audit_agentic_analysis.go index 8cf5a1eb66..c80d6148f3 100644 --- a/pkg/cli/audit_agentic_analysis.go +++ b/pkg/cli/audit_agentic_analysis.go @@ -164,7 +164,10 @@ func detectTaskDomain(processedRun ProcessedRun, createdItems []CreatedItemRepor func buildBehaviorFingerprint(processedRun ProcessedRun, metrics MetricsData, toolUsage []ToolUsageInfo, createdItems []CreatedItemReport, awContext *AwContext) *BehaviorFingerprint { toolTypes := len(toolUsage) - writeCount := len(createdItems) + processedRun.Run.SafeItemsCount + writeCount := processedRun.Run.SafeItemsCount + if writeCount == 0 { + writeCount = len(createdItems) + } executionStyle := "directed" switch { @@ -223,7 +226,10 @@ func buildAgenticAssessments(processedRun ProcessedRun, metrics MetricsData, too assessments := make([]AgenticAssessment, 0, 4) toolTypes := len(toolUsage) frictionEvents := len(processedRun.MissingTools) + len(processedRun.MCPFailures) + len(processedRun.MissingData) - writeCount := len(createdItems) + processedRun.Run.SafeItemsCount + writeCount := processedRun.Run.SafeItemsCount + if writeCount == 0 { + writeCount = len(createdItems) + } if fingerprint.ResourceProfile == "heavy" { severity := "medium"