Fix linter issues in agent-run packages

Replace the single-case switch in the agent-run handler with an if,
and add the blank lines wsl_v5 requires around statements in the
approval checkpoint, worker tests, and policy-set registration.

Signed-off-by: Bryan Frimin <bryan@probo.com>

pkg/agent/approval.go

@@ -80,6 +80,7 @@ func MergeApprovalDecisions(
 	if cp.ApprovalInput == nil {
 		cp.ApprovalInput = make(map[string]ApprovalResult, len(decisions))
 	}
+
 	maps.Copy(cp.ApprovalInput, decisions)
 
 	data, err := json.Marshal(&cp)

pkg/agentrun/handler.go

@@ -177,8 +177,7 @@ func (h *handler) executeRun(ctx context.Context, run *coredata.AgentRun) error
 	run.Result = nil
 	run.ErrorMessage = nil
 
-	switch {
+	if runErr == nil {
-	case runErr == nil:
 		run.Status = coredata.AgentRunStatusCompleted
 
 		if result != nil {

pkg/agentrun/worker_test.go

@@ -151,6 +151,7 @@ func TestWorker_StopAndResume(t *testing.T) {
 		t,
 		func() bool {
 			r, err := tryLoadAgentRun(client, run.ID)
+
 			return err == nil &&
 				r.Status == coredata.AgentRunStatusPending &&
 				r.Checkpoint != nil
@@ -558,6 +559,7 @@ func TestWorker_StopAndResumeAcrossHandoff(t *testing.T) {
 		t,
 		func() bool {
 			r, err := tryLoadAgentRun(client, run.ID)
+
 			return err == nil &&
 				r.Status == coredata.AgentRunStatusPending &&
 				r.Checkpoint != nil
@@ -690,6 +692,7 @@ func TestWorker_StopAndResumeNestedSubAgent(t *testing.T) {
 		t,
 		func() bool {
 			r, err := tryLoadAgentRun(client, run.ID)
+
 			return err == nil &&
 				r.Status == coredata.AgentRunStatusPending &&
 				r.Checkpoint != nil

pkg/probod/probod.go

@@ -595,6 +595,7 @@ func (impl *Implm) Run(
 	)
 
 	agentRunService := agentrun.NewService(pgClient)
+
 	iamService.Authorizer.RegisterPolicySet(agentrun.PolicySet())
 
 	thirdPartyService := thirdparty.NewService(pgClient, fileService, thirdPartyVetter)