[test-parallel] Add t.Parallel() to safe top-level Go tests in pkg/cli codemod suite

pkg/cli/codemod_safe_output_merge_pr_constraints_test.go

@@ -10,6 +10,7 @@ import (
 )
 
 func TestSafeOutputMergePRConstraintsCodemod(t *testing.T) {
+	t.Parallel()
 	codemod := getSafeOutputMergePRConstraintsCodemod()
 
 	t.Run("renames allowed-labels to required-labels, leaves allowed-branches unchanged", func(t *testing.T) {

pkg/cli/codemod_safe_output_require_title_prefix_test.go

@@ -10,6 +10,7 @@ import (
 )
 
 func TestSafeOutputRequireTitlePrefixCodemod(t *testing.T) {
+	t.Parallel()
 	codemod := getSafeOutputRequireTitlePrefixCodemod()
 
 	t.Run("renames close and push constraint keys", func(t *testing.T) {

pkg/cli/codemod_sandbox_agent_false_removal_test.go

@@ -10,6 +10,7 @@ import (
 )
 
 func TestGetSandboxAgentFalseRemovalCodemod(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxAgentFalseRemovalCodemod()
 
 	assert.Equal(t, "sandbox-agent-false-removal", codemod.ID)
@@ -20,6 +21,7 @@ func TestGetSandboxAgentFalseRemovalCodemod(t *testing.T) {
 }
 
 func TestSandboxAgentFalseRemoval_RemovesAgentFalse(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxAgentFalseRemovalCodemod()
 
 	content := `---
@@ -52,6 +54,7 @@ permissions:
 }
 
 func TestSandboxAgentFalseRemoval_PreservesOtherSandboxKeys(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxAgentFalseRemovalCodemod()
 
 	content := `---
@@ -84,6 +87,7 @@ sandbox:
 }
 
 func TestSandboxAgentFalseRemoval_NoSandboxKey(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxAgentFalseRemovalCodemod()
 
 	content := `---
@@ -109,6 +113,7 @@ permissions:
 }
 
 func TestSandboxAgentFalseRemoval_AgentNotFalse(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxAgentFalseRemovalCodemod()
 
 	content := `---
@@ -134,6 +139,7 @@ sandbox:
 }
 
 func TestSandboxAgentFalseRemoval_AgentObject(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxAgentFalseRemovalCodemod()
 
 	content := `---
@@ -162,6 +168,7 @@ sandbox:
 }
 
 func TestSandboxAgentFalseRemoval_AgentTrue(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxAgentFalseRemovalCodemod()
 
 	content := `---
@@ -187,6 +194,7 @@ sandbox:
 }
 
 func TestSandboxAgentFalseRemoval_SkipsWhenStrictFalse(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxAgentFalseRemovalCodemod()
 
 	content := `---
@@ -214,6 +222,7 @@ sandbox:
 }
 
 func TestSandboxAgentFalseRemoval_PreservesMarkdown(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxAgentFalseRemovalCodemod()
 
 	content := `---
@@ -242,6 +251,7 @@ This workflow was using the nosandbox escape hatch.`
 }
 
 func TestSandboxAgentFalseRemoval_NoAgentKey(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxAgentFalseRemovalCodemod()
 
 	content := `---

pkg/cli/codemod_sandbox_agent_test.go

@@ -10,6 +10,7 @@ import (
 )
 
 func TestGetSandboxFalseToAgentFalseCodemod(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxFalseToAgentFalseCodemod()
 
 	assert.Equal(t, "sandbox-false-to-agent-false", codemod.ID)
@@ -20,6 +21,7 @@ func TestGetSandboxFalseToAgentFalseCodemod(t *testing.T) {
 }
 
 func TestSandboxFalseToAgentFalseCodemod_ConvertsBooleanFalse(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxFalseToAgentFalseCodemod()
 
 	content := `---
@@ -49,6 +51,7 @@ permissions:
 }
 
 func TestSandboxFalseToAgentFalseCodemod_PreservesIndentation(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxFalseToAgentFalseCodemod()
 
 	content := `---
@@ -79,6 +82,7 @@ permissions:
 }
 
 func TestSandboxFalseToAgentFalseCodemod_NoSandboxField(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxFalseToAgentFalseCodemod()
 
 	content := `---
@@ -104,6 +108,7 @@ permissions:
 }
 
 func TestSandboxFalseToAgentFalseCodemod_SandboxTrue(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxFalseToAgentFalseCodemod()
 
 	content := `---
@@ -126,6 +131,7 @@ sandbox: true
 }
 
 func TestSandboxFalseToAgentFalseCodemod_SandboxObject(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxFalseToAgentFalseCodemod()
 
 	content := `---
@@ -151,6 +157,7 @@ sandbox:
 }
 
 func TestSandboxFalseToAgentFalseCodemod_PreservesMarkdown(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxFalseToAgentFalseCodemod()
 
 	content := `---
@@ -176,6 +183,7 @@ This workflow runs without a sandbox.`
 }
 
 func TestSandboxFalseToAgentFalseCodemod_WithStrictFalse(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxFalseToAgentFalseCodemod()
 
 	content := `---

pkg/cli/codemod_sandbox_mcp_internal_test.go

@@ -12,6 +12,7 @@ import (
 // ----- getSandboxMCPContainerRemovalCodemod tests -----
 
 func TestGetSandboxMCPContainerRemovalCodemod(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxMCPContainerRemovalCodemod()
 
 	assert.Equal(t, "sandbox-mcp-container-removal", codemod.ID)
@@ -22,6 +23,7 @@ func TestGetSandboxMCPContainerRemovalCodemod(t *testing.T) {
 }
 
 func TestSandboxMCPContainerRemoval_RemovesContainer(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxMCPContainerRemovalCodemod()
 
 	content := `---
@@ -59,6 +61,7 @@ permissions:
 }
 
 func TestSandboxMCPContainerRemoval_NoSandboxKey(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxMCPContainerRemovalCodemod()
 
 	content := `---
@@ -84,6 +87,7 @@ permissions:
 }
 
 func TestSandboxMCPContainerRemoval_NoMCPKey(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxMCPContainerRemovalCodemod()
 
 	content := `---
@@ -109,6 +113,7 @@ sandbox:
 }
 
 func TestSandboxMCPContainerRemoval_NoContainerField(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxMCPContainerRemovalCodemod()
 
 	content := `---
@@ -137,6 +142,7 @@ sandbox:
 }
 
 func TestSandboxMCPContainerRemoval_PreservesMarkdown(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxMCPContainerRemovalCodemod()
 
 	content := `---
@@ -171,6 +177,7 @@ This is a test workflow.`
 // ----- getSandboxMCPVersionRemovalCodemod tests -----
 
 func TestGetSandboxMCPVersionRemovalCodemod(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxMCPVersionRemovalCodemod()
 
 	assert.Equal(t, "sandbox-mcp-version-removal", codemod.ID)
@@ -181,6 +188,7 @@ func TestGetSandboxMCPVersionRemovalCodemod(t *testing.T) {
 }
 
 func TestSandboxMCPVersionRemoval_RemovesVersion(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxMCPVersionRemovalCodemod()
 
 	content := `---
@@ -218,6 +226,7 @@ permissions:
 }
 
 func TestSandboxMCPVersionRemoval_NoVersionField(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxMCPVersionRemovalCodemod()
 
 	content := `---
@@ -246,6 +255,7 @@ sandbox:
 }
 
 func TestSandboxMCPVersionRemoval_NoSandboxKey(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxMCPVersionRemovalCodemod()
 
 	content := `---
@@ -266,6 +276,7 @@ on: workflow_dispatch
 }
 
 func TestSandboxMCPContainerRemoval_SkipsWhenStrictFalse(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxMCPContainerRemovalCodemod()
 
 	content := `---
@@ -298,6 +309,7 @@ sandbox:
 }
 
 func TestSandboxMCPVersionRemoval_SkipsWhenStrictFalse(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxMCPVersionRemovalCodemod()
 
 	content := `---
@@ -330,6 +342,7 @@ sandbox:
 }
 
 func TestSandboxMCPVersionRemoval_BothContainerAndVersion(t *testing.T) {
+	t.Parallel()
 	// Verify that version removal does not affect the container key.
 	codemod := getSandboxMCPVersionRemovalCodemod()
 
@@ -365,6 +378,7 @@ sandbox:
 }
 
 func TestSandboxMCPContainerRemoval_RemovesEmptySandboxGrandparent(t *testing.T) {
+	t.Parallel()
 	// When container: is the only field under mcp:, and mcp: is the only field
 	// under sandbox:, the codemod must remove all three levels to avoid a
 	// dangling "sandbox:" key that YAML parses as null.
@@ -404,6 +418,7 @@ permissions:
 }
 
 func TestSandboxMCPVersionRemoval_RemovesEmptySandboxGrandparent(t *testing.T) {
+	t.Parallel()
 	// When version: is the only field under mcp:, and mcp: is the only field
 	// under sandbox:, the codemod must remove all three levels to avoid a
 	// dangling "sandbox:" key that YAML parses as null.
@@ -443,6 +458,7 @@ permissions:
 }
 
 func TestSandboxMCPContainerRemoval_KeepsSandboxWhenOtherChildrenRemain(t *testing.T) {
+	t.Parallel()
 	// When sandbox: has other children besides mcp:, it must be preserved even
 	// after mcp: becomes empty and is removed.
 	codemod := getSandboxMCPContainerRemovalCodemod()

pkg/cli/codemod_sandbox_runtime_profile_test.go

@@ -10,6 +10,7 @@ import (
 )
 
 func TestGetSandboxRuntimeProfileCodemod(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxRuntimeProfileCodemod()
 
 	assert.Equal(t, "sandbox-runtime-profiles", codemod.ID)
@@ -20,6 +21,7 @@ func TestGetSandboxRuntimeProfileCodemod(t *testing.T) {
 }
 
 func TestSandboxRuntimeProfileCodemod_Migrations(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxRuntimeProfileCodemod()
 
 	tests := []struct {
@@ -210,6 +212,7 @@ sandbox:
 // only key under sandbox.agent also removes the now-empty parent mappings, which would
 // otherwise fail schema validation as a null value.
 func TestSandboxRuntimeProfileCodemod_RemovesEmptySandboxBlock(t *testing.T) {
+	t.Parallel()
 	codemod := getSandboxRuntimeProfileCodemod()
 
 	content := `---

pkg/cli/codemod_schedule_test.go

@@ -10,6 +10,7 @@ import (
 )
 
 func TestGetScheduleAtToAroundCodemod(t *testing.T) {
+	t.Parallel()
 	codemod := getScheduleAtToAroundCodemod()
 
 	assert.Equal(t, "schedule-at-to-around-migration", codemod.ID)
@@ -20,6 +21,7 @@ func TestGetScheduleAtToAroundCodemod(t *testing.T) {
 }
 
 func TestScheduleCodemod_DailyAt(t *testing.T) {
+	t.Parallel()
 	codemod := getScheduleAtToAroundCodemod()
 
 	content := `---
@@ -47,6 +49,7 @@ on:
 }
 
 func TestScheduleCodemod_WeeklyOnAt(t *testing.T) {
+	t.Parallel()
 	codemod := getScheduleAtToAroundCodemod()
 
 	content := `---
@@ -74,6 +77,7 @@ on:
 }
 
 func TestScheduleCodemod_MonthlyOn(t *testing.T) {
+	t.Parallel()
 	codemod := getScheduleAtToAroundCodemod()
 
 	content := `---
@@ -101,6 +105,7 @@ on:
 }
 
 func TestScheduleCodemod_MonthlyOnAt(t *testing.T) {
+	t.Parallel()
 	codemod := getScheduleAtToAroundCodemod()
 
 	content := `---
@@ -128,6 +133,7 @@ on:
 }
 
 func TestScheduleCodemod_DailyAround_NoChange(t *testing.T) {
+	t.Parallel()
 	codemod := getScheduleAtToAroundCodemod()
 
 	content := `---
@@ -154,6 +160,7 @@ on:
 }
 
 func TestScheduleCodemod_StandardCron_NoChange(t *testing.T) {
+	t.Parallel()
 	codemod := getScheduleAtToAroundCodemod()
 
 	content := `---
@@ -180,6 +187,7 @@ on:
 }
 
 func TestScheduleCodemod_PreservesIndentation(t *testing.T) {
+	t.Parallel()
 	codemod := getScheduleAtToAroundCodemod()
 
 	content := `---
@@ -206,6 +214,7 @@ on:
 }
 
 func TestScheduleCodemod_MultipleSchedules(t *testing.T) {
+	t.Parallel()
 	codemod := getScheduleAtToAroundCodemod()
 
 	content := `---
@@ -235,6 +244,7 @@ on:
 }
 
 func TestScheduleCodemod_ScheduleField(t *testing.T) {
+	t.Parallel()
 	codemod := getScheduleAtToAroundCodemod()
 
 	content := `---
@@ -261,6 +271,7 @@ on:
 }
 
 func TestScheduleCodemod_PreservesMarkdown(t *testing.T) {
+	t.Parallel()
 	codemod := getScheduleAtToAroundCodemod()
 
 	content := `---