fix(coderd): ignore sub agents when converting a task to workspace (coder#19624)

DanielleMaywood · web-flow · commit 75b38f12d8aa · 2025-08-28T18:27:31.000+01:00
Addresses comment raised on previous PR coder#19619 (comment) We know we can skip sub agents when searching for which agent is related to the task, as this is not an explicitly supported feature at the moment. When we come to properly setting up a Task -> Agent relationship this limitation will be dropped.
diff --git a/coderd/aitasks.go b/coderd/aitasks.go
@@ -217,11 +217,19 @@ func (api *API) tasksFromWorkspaces(ctx context.Context, apiWorkspaces []codersd
 		// This just picks up the first agent it discovers.
 		// This approach _might_ break when a task has multiple agents,
 		// depending on which agent was found first.
+		//
+		// We explicitly do not have support for running tasks
+		// inside of a sub agent at the moment, so we can be sure
+		// that any sub agents are not the agent we're looking for.
 		var taskAgentID uuid.NullUUID
 		var taskAgentLifecycle *codersdk.WorkspaceAgentLifecycle
 		var taskAgentHealth *codersdk.WorkspaceAgentHealth
 		for _, resource := range ws.LatestBuild.Resources {
 			for _, agent := range resource.Agents {
+				if agent.ParentID.Valid {
+					continue
+				}
+
 				taskAgentID = uuid.NullUUID{Valid: true, UUID: agent.ID}
 				taskAgentLifecycle = &agent.LifecycleState
 				taskAgentHealth = &agent.Health
