review comments

mafredri · mafredri · commit 7e5ede0a9d99 · 2025-06-16T09:36:50.000Z
diff --git a/agent/agentcontainers/api.go b/agent/agentcontainers/api.go
@@ -1100,15 +1100,13 @@ func (api *API) maybeInjectSubAgentIntoContainerLocked(ctx context.Context, dc c
 		directory = DevcontainerDefaultContainerWorkspaceFolder
 	}
 
-	if proc.agent.ID != uuid.Nil {
-		if recreateSubAgent {
-			logger.Debug(ctx, "deleting existing subagent for recreation", slog.F("agent_id", proc.agent.ID))
-			err = api.subAgentClient.Delete(ctx, proc.agent.ID)
-			if err != nil {
-				return xerrors.Errorf("delete existing subagent failed: %w", err)
-			}
-			proc.agent = SubAgent{}
+	if proc.agent.ID != uuid.Nil && recreateSubAgent {
+		logger.Debug(ctx, "deleting existing subagent for recreation", slog.F("agent_id", proc.agent.ID))
+		err = api.subAgentClient.Delete(ctx, proc.agent.ID)
+		if err != nil {
+			return xerrors.Errorf("delete existing subagent failed: %w", err)
 		}
+		proc.agent = SubAgent{}
 	}
 	if proc.agent.ID == uuid.Nil {
 		displayAppsMap := map[codersdk.DisplayApp]bool{
