chore: feedback from another PR

DanielleMaywood · DanielleMaywood · commit 59db86e0352c · 2025-06-26T15:46:31.000Z
diff --git a/agent/agentcontainers/api.go b/agent/agentcontainers/api.go
@@ -1442,7 +1442,7 @@ func (api *API) maybeInjectSubAgentIntoContainerLocked(ctx context.Context, dc c
 		return xerrors.Errorf("set agent binary executable: %w", err)
 	}
 
-	// Make sure the agent binary is executable so we can run it.
+	// Make sure the agent binary is owned by a valid user so we can run it.
 	if _, err := api.ccli.ExecAs(ctx, container.ID, "root", "/bin/sh", "-c", fmt.Sprintf("chown $(id -u):$(id -g) %s", coderPathInsideContainer)); err != nil {
 		return xerrors.Errorf("set agent binary ownership: %w", err)
 	}

Original file line number	Diff line number	Diff line change
`@@ -1442,7 +1442,7 @@ func (api *API) maybeInjectSubAgentIntoContainerLocked(ctx context.Context, dc c`
`1442`	`1442`	`return xerrors.Errorf("set agent binary executable: %w", err)`
`1443`	`1443`	`}`
`1444`	`1444`
`1445`		`- // Make sure the agent binary is executable so we can run it.`
	`1445`	`+ // Make sure the agent binary is owned by a valid user so we can run it.`
`1446`	`1446`	`if _, err := api.ccli.ExecAs(ctx, container.ID, "root", "/bin/sh", "-c", fmt.Sprintf("chown $(id -u):$(id -g) %s", coderPathInsideContainer)); err != nil {`
`1447`	`1447`	`return xerrors.Errorf("set agent binary ownership: %w", err)`
`1448`	`1448`	`}`