fix: gofumpt formatting + go mod tidy

Author: Patel230

.github/dependabot.yml

@@ -454,8 +454,14 @@ func renderBrailleChart(w io.Writer, sty activityStyles, hourly []hourlyPoint, r
 			//   dot3 dot6     bit2 bit5
 			//   dot7 dot8     bit6 bit7
 			dotBits := [8][2]int{
-				{0, 0}, {1, 0}, {2, 0}, {0, 1},
-				{1, 1}, {2, 1}, {3, 0}, {3, 1},
+				{0, 0},
+				{1, 0},
+				{2, 0},
+				{0, 1},
+				{1, 1},
+				{2, 1},
+				{3, 0},
+				{3, 1},
 			}
 
 			for bit, offset := range dotBits {

cmd/trace/cli/activity_render.go

@@ -8,8 +8,10 @@ import (
 	"github.com/GrayCodeAI/trace/cmd/trace/cli/agent/types"
 )
 
-const mockAgentName types.AgentName = "mock" // Used by mock implementations
-const mockAgentType types.AgentType = "Mock Agent"
+const (
+	mockAgentName types.AgentName = "mock" // Used by mock implementations
+	mockAgentType types.AgentType = "Mock Agent"
+)
 
 // mockAgent is a minimal implementation of Agent for testing interface compliance.
 type mockAgent struct{}
@@ -29,6 +31,7 @@ func (m *mockAgent) ReadTranscript(_ string) ([]byte, error) { return nil, nil }
 func (m *mockAgent) ChunkTranscript(_ context.Context, content []byte, _ int) ([][]byte, error) {
 	return [][]byte{content}, nil
 }
+
 func (m *mockAgent) ReassembleTranscript(chunks [][]byte) ([]byte, error) {
 	var result []byte
 	for _, c := range chunks {

cmd/trace/cli/agent/agent_test.go

@@ -40,6 +40,7 @@ func (m *mockBuiltinHookAgent) HookNames() []string { return nil }
 func (m *mockBuiltinHookAgent) ParseHookEvent(context.Context, string, io.Reader) (*Event, error) {
 	return nil, nil //nolint:nilnil // test mock
 }
+
 func (m *mockBuiltinHookAgent) InstallHooks(context.Context, bool, bool) (int, error) {
 	return 0, nil
 }
@@ -57,6 +58,7 @@ func (m *mockFullAgent) DeclaredCapabilities() DeclaredCaps { return m.caps }
 
 // HookSupport
 func (m *mockFullAgent) HookNames() []string { return nil }
+
 func (m *mockFullAgent) ParseHookEvent(context.Context, string, io.Reader) (*Event, error) {
 	return nil, nil //nolint:nilnil // test mock
 }
@@ -66,6 +68,7 @@ func (m *mockFullAgent) AreHooksInstalled(context.Context) bool                {
 
 // TranscriptAnalyzer
 func (m *mockFullAgent) GetTranscriptPosition(string) (int, error) { return 0, nil }
+
 func (m *mockFullAgent) ExtractModifiedFilesFromOffset(string, int) ([]string, int, error) {
 	return nil, 0, nil
 }
@@ -95,6 +98,7 @@ func (m *mockFullAgent) WriteHookResponse(string) error { return nil }
 func (m *mockFullAgent) ExtractAllModifiedFiles([]byte, int, string) ([]string, error) {
 	return nil, nil
 }
+
 func (m *mockFullAgent) CalculateTotalTokenUsage([]byte, int, string) (*TokenUsage, error) {
 	return nil, nil //nolint:nilnil // test mock
 }

cmd/trace/cli/agent/capabilities_test.go

@@ -218,7 +218,8 @@ func waitForTranscriptFlush(ctx context.Context, transcriptPath string, hookStar
 	}
 	fileAge := time.Since(info.ModTime())
 	if fileAge > staleThreshold {
-		logging.Debug(logCtx, "transcript file is stale, skipping sentinel wait",
+		logging.Debug(
+			logCtx, "transcript file is stale, skipping sentinel wait",
 			slog.Duration("file_age", fileAge),
 		)
 		return
@@ -227,14 +228,16 @@ func waitForTranscriptFlush(ctx context.Context, transcriptPath string, hookStar
 	deadline := time.Now().Add(maxWait)
 	for time.Now().Before(deadline) {
 		if checkStopSentinel(transcriptPath, tailBytes, hookStartTime, maxSkew) {
-			logging.Debug(logCtx, "transcript flush sentinel found",
+			logging.Debug(
+				logCtx, "transcript flush sentinel found",
 				slog.Duration("wait", time.Since(hookStartTime)),
 			)
 			return
 		}
 		time.Sleep(pollInterval)
 	}
-	logging.Warn(logCtx, "transcript flush sentinel not found within timeout, proceeding",
+	logging.Warn(
+		logCtx, "transcript flush sentinel not found within timeout, proceeding",
 		slog.Duration("timeout", maxWait),
 	)
 }

cmd/trace/cli/agent/claudecode/lifecycle.go

@@ -20,7 +20,6 @@ func TestParseHookEvent_SessionStart(t *testing.T) {
 	input := `{"session_id": "test-session-123", "transcript_path": "/tmp/transcript.jsonl"}`
 
 	event, err := ag.ParseHookEvent(context.Background(), HookNameSessionStart, strings.NewReader(input))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -46,7 +45,6 @@ func TestParseHookEvent_SessionStart_IncludesModel(t *testing.T) {
 	input := `{"session_id": "model-session", "transcript_path": "/tmp/t.jsonl", "model": "claude-sonnet-4-20250514"}`
 
 	event, err := ag.ParseHookEvent(context.Background(), HookNameSessionStart, strings.NewReader(input))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -66,7 +64,6 @@ func TestParseHookEvent_SessionStart_EmptyModel(t *testing.T) {
 	input := `{"session_id": "no-model-session", "transcript_path": "/tmp/t.jsonl"}`
 
 	event, err := ag.ParseHookEvent(context.Background(), HookNameSessionStart, strings.NewReader(input))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -83,7 +80,6 @@ func TestParseHookEvent_TurnStart(t *testing.T) {
 	input := `{"session_id": "sess-456", "transcript_path": "/tmp/t.jsonl", "prompt": "Hello world"}`
 
 	event, err := ag.ParseHookEvent(context.Background(), HookNameUserPromptSubmit, strings.NewReader(input))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -106,7 +102,6 @@ func TestParseHookEvent_TurnEnd(t *testing.T) {
 	input := `{"session_id": "sess-789", "transcript_path": "/tmp/stop.jsonl"}`
 
 	event, err := ag.ParseHookEvent(context.Background(), HookNameStop, strings.NewReader(input))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -126,7 +121,6 @@ func TestParseHookEvent_TurnEnd_IncludesModel(t *testing.T) {
 	input := `{"session_id": "sess-stop-model", "transcript_path": "/tmp/stop.jsonl", "model": "claude-opus-4-6"}`
 
 	event, err := ag.ParseHookEvent(context.Background(), HookNameStop, strings.NewReader(input))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -143,7 +137,6 @@ func TestParseHookEvent_SessionEnd(t *testing.T) {
 	input := `{"session_id": "ending-session", "transcript_path": "/tmp/end.jsonl"}`
 
 	event, err := ag.ParseHookEvent(context.Background(), HookNameSessionEnd, strings.NewReader(input))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -163,7 +156,6 @@ func TestParseHookEvent_SessionEnd_IncludesModel(t *testing.T) {
 	input := `{"session_id": "end-model", "transcript_path": "/tmp/end.jsonl", "model": "claude-sonnet-4-20250514"}`
 
 	event, err := ag.ParseHookEvent(context.Background(), HookNameSessionEnd, strings.NewReader(input))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -190,7 +182,6 @@ func TestParseHookEvent_SubagentStart(t *testing.T) {
 	}
 
 	event, err := ag.ParseHookEvent(context.Background(), HookNamePreTask, strings.NewReader(string(inputBytes)))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -228,7 +219,6 @@ func TestParseHookEvent_SubagentEnd(t *testing.T) {
 	}
 
 	event, err := ag.ParseHookEvent(context.Background(), HookNamePostTask, strings.NewReader(string(inputBytes)))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -261,7 +251,6 @@ func TestParseHookEvent_SubagentEnd_NoAgentID(t *testing.T) {
 	}
 
 	event, err := ag.ParseHookEvent(context.Background(), HookNamePostTask, strings.NewReader(string(inputBytes)))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -278,7 +267,6 @@ func TestParseHookEvent_PostTodo_ReturnsNil(t *testing.T) {
 	input := `{"session_id": "todo-session", "transcript_path": "/tmp/todo.jsonl"}`
 
 	event, err := ag.ParseHookEvent(context.Background(), HookNamePostTodo, strings.NewReader(input))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -294,7 +282,6 @@ func TestParseHookEvent_UnknownHook_ReturnsNil(t *testing.T) {
 	input := `{"session_id": "unknown", "transcript_path": "/tmp/unknown.jsonl"}`
 
 	event, err := ag.ParseHookEvent(context.Background(), "unknown-hook-name", strings.NewReader(input))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -386,7 +373,6 @@ func TestParseHookEvent_AllHookTypes(t *testing.T) {
 
 			ag := &ClaudeCodeAgent{}
 			event, err := ag.ParseHookEvent(context.Background(), tc.hookName, strings.NewReader(tc.inputTemplate))
-
 			if err != nil {
 				t.Fatalf("unexpected error: %v", err)
 			}
@@ -412,7 +398,6 @@ func TestReadAndParse_ValidInput(t *testing.T) {
 	input := `{"session_id": "test-123", "transcript_path": "/path/to/transcript"}`
 
 	result, err := agent.ReadAndParseHookInput[sessionInfoRaw](strings.NewReader(input))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -458,7 +443,6 @@ func TestReadAndParse_PartialJSON(t *testing.T) {
 	input := `{"session_id": "partial-only"}`
 
 	result, err := agent.ReadAndParseHookInput[sessionInfoRaw](strings.NewReader(input))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -477,7 +461,6 @@ func TestReadAndParse_ExtraFields(t *testing.T) {
 	input := `{"session_id": "test", "transcript_path": "/t", "extra_field": "ignored", "another": 123}`
 
 	result, err := agent.ReadAndParseHookInput[sessionInfoRaw](strings.NewReader(input))
-
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}

cmd/trace/cli/agent/claudecode/lifecycle_test.go

@@ -699,7 +699,8 @@ func TestExtractAllModifiedFiles_IncludesSubagentFiles(t *testing.T) {
 	)
 
 	// Subagent transcript: Write to helper.go + Edit to utils.go (still on disk)
-	writeJSONLFile(t, subagentsDir+"/agent-sub1.jsonl",
+	writeJSONLFile(
+		t, subagentsDir+"/agent-sub1.jsonl",
 		makeWriteToolLine(t, "sa1", "/repo/helper.go"),
 		makeEditToolLine(t, "sa2", "/repo/utils.go"),
 	)
@@ -748,7 +749,8 @@ func TestExtractAllModifiedFiles_DeduplicatesAcrossAgents(t *testing.T) {
 	)
 
 	// Subagent transcript: Also modifies shared.go (same file as main)
-	writeJSONLFile(t, subagentsDir+"/agent-sub1.jsonl",
+	writeJSONLFile(
+		t, subagentsDir+"/agent-sub1.jsonl",
 		makeEditToolLine(t, "sa1", "/repo/shared.go"),
 	)
 
@@ -809,7 +811,8 @@ func TestExtractAllModifiedFiles_SubagentOnlyChanges(t *testing.T) {
 	)
 
 	// Subagent transcript: Write to two files (still on disk)
-	writeJSONLFile(t, subagentsDir+"/agent-sub1.jsonl",
+	writeJSONLFile(
+		t, subagentsDir+"/agent-sub1.jsonl",
 		makeWriteToolLine(t, "sa1", "/repo/subagent_file1.go"),
 		makeWriteToolLine(t, "sa2", "/repo/subagent_file2.go"),
 	)

cmd/trace/cli/agent/claudecode/transcript_test.go

@@ -101,7 +101,8 @@ func TestCodexAgent_ResolveRestoredSessionFile(t *testing.T) {
 
 	path, err := ag.ResolveRestoredSessionFile(dir, "019d24c3-1111-2222-3333-444444444444", []byte(sampleRollout))
 	require.NoError(t, err)
-	require.Equal(t,
+	require.Equal(
+		t,
 		filepath.Join(dir, "2026", "03", "25", "rollout-2026-03-25T11-31-10-019d24c3-1111-2222-3333-444444444444.jsonl"),
 		path,
 	)