chore: allow prebuilds user to be added to group

dannykopping · dannykopping · commit 992727b338e5 · 2025-04-28T14:38:46.000+02:00
Signed-off-by: Danny Kopping &lt;dannykopping@gmail.com&gt;
diff --git a/enterprise/coderd/groups.go b/enterprise/coderd/groups.go
@@ -14,6 +14,7 @@ import (
 	"github.com/coder/coder/v2/coderd/database/db2sdk"
 	"github.com/coder/coder/v2/coderd/httpapi"
 	"github.com/coder/coder/v2/coderd/httpmw"
+	"github.com/coder/coder/v2/coderd/prebuilds"
 	"github.com/coder/coder/v2/codersdk"
 )
 
@@ -165,16 +166,19 @@ func (api *API) patchGroup(rw http.ResponseWriter, r *http.Request) {
 	aReq.Old = group.Auditable(currentMembers)
 
 	for _, id := range users {
-		if _, err := uuid.Parse(id); err != nil {
+		uid, err := uuid.Parse(id)
+		if err != nil {
 			httpapi.Write(ctx, rw, http.StatusBadRequest, codersdk.Response{
 				Message: fmt.Sprintf("ID %q must be a valid user UUID.", id),
 			})
 			return
 		}
-		_, err := database.ExpectOne(api.Database.OrganizationMembers(ctx, database.OrganizationMembersParams{
+
+		isSystemUser := uid == prebuilds.SystemUserID
+		_, err = database.ExpectOne(api.Database.OrganizationMembers(ctx, database.OrganizationMembersParams{
 			OrganizationID: group.OrganizationID,
-			UserID:         uuid.MustParse(id),
-			IncludeSystem:  false,
+			UserID:         uid,
+			IncludeSystem:  isSystemUser,
 		}))
 		if errors.Is(err, sql.ErrNoRows) {
 			httpapi.Write(ctx, rw, http.StatusBadRequest, codersdk.Response{
