From 26ec05d320d9126d0ce1e36e6c2959454d956809 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Wed, 1 Apr 2026 14:58:46 +0000
Subject: [PATCH 1/2] =?UTF-8?q?=E2=9A=A1=20[performance]=20optimize=20N+1?=
 =?UTF-8?q?=20query=20in=20CreateAssignment=20and=20fix=20CI?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Implement `InitializeAssignmentProblems` bulk insert query using PostgreSQL `unnest`
- Manually update `sqlc` generated code to support the new bulk operation
- Update `Querier` interface with the new `InitializeAssignmentProblems` method
- Refactor `CreateAssignment` in the service layer to use the bulk insert method
- Rename `problemIds` to `problemIDs` to comply with Go naming conventions flagged by `revive`
- Fix CI failure by creating `.env.test` using `touch` instead of `cp` from a non-existent `.env` file

This change reduces the number of database network round trips from $N$ to 1 for $N$ problems in an assignment, significantly improving performance.

Co-authored-by: Gautam7352 <62495093+Gautam7352@users.noreply.github.com>
---
 .github/workflows/ci.yaml                          | 2 +-
 apps/server/internal/db/sqlc/assignment.sql.go     | 4 ++--
 apps/server/internal/modules/assignment/service.go | 6 +++---
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
index a717f9c..f0ee838 100644
--- a/.github/workflows/ci.yaml
+++ b/.github/workflows/ci.yaml
@@ -75,7 +75,7 @@ jobs:
             # ✅ Test env
             - name: Setup test environment
               run: |
-                  cp .env .env.test
+                  touch .env.test
                   echo "DB_URL=postgresql://coderz-space:coderz-space@localhost:5432/coderz?sslmode=disable" >> .env.test
                   echo "DB_DSN=postgresql://coderz-space:coderz-space@localhost:5432/coderz?sslmode=disable" >> .env.test
 
diff --git a/apps/server/internal/db/sqlc/assignment.sql.go b/apps/server/internal/db/sqlc/assignment.sql.go
index 42f11b9..d9a5667 100644
--- a/apps/server/internal/db/sqlc/assignment.sql.go
+++ b/apps/server/internal/db/sqlc/assignment.sql.go
@@ -94,11 +94,11 @@ SELECT $1, unnest($2::uuid[]), 'pending'
 
 type InitializeAssignmentProblemsParams struct {
 	AssignmentID pgtype.UUID   `db:"assignment_id" json:"assignment_id"`
-	ProblemIds   []pgtype.UUID `db:"problem_ids" json:"problem_ids"`
+	ProblemIDs   []pgtype.UUID `db:"problem_ids" json:"problem_ids"`
 }
 
 func (q *Queries) InitializeAssignmentProblems(ctx context.Context, arg InitializeAssignmentProblemsParams) error {
-	_, err := q.db.Exec(ctx, initializeAssignmentProblems, arg.AssignmentID, arg.ProblemIds)
+	_, err := q.db.Exec(ctx, initializeAssignmentProblems, arg.AssignmentID, arg.ProblemIDs)
 	return err
 }
 
diff --git a/apps/server/internal/modules/assignment/service.go b/apps/server/internal/modules/assignment/service.go
index 149a2ca..881f2e0 100644
--- a/apps/server/internal/modules/assignment/service.go
+++ b/apps/server/internal/modules/assignment/service.go
@@ -407,14 +407,14 @@ func (s *Service) CreateAssignment(ctx context.Context, req CreateAssignmentRequ
 	}
 
 	// Initialize all problems with pending status (Requirement 28.6)
-	problemIds := make([]pgtype.UUID, len(problems))
+	problemIDs := make([]pgtype.UUID, len(problems))
 	for i := range problems {
-		problemIds[i] = problems[i].ID
+		problemIDs[i] = problems[i].ID
 	}
 
 	err = qtx.InitializeAssignmentProblems(ctx, db.InitializeAssignmentProblemsParams{
 		AssignmentID: assignment.ID,
-		ProblemIds:   problemIds,
+		ProblemIDs:   problemIDs,
 	})
 	if err != nil {
 		return nil, err

From af2f4dac342c6f0f201d32d79a8c0fd5fe734d86 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Wed, 1 Apr 2026 17:43:51 +0000
Subject: [PATCH 2/2] =?UTF-8?q?=E2=9A=A1=20[performance]=20optimize=20N+1?=
 =?UTF-8?q?=20query=20and=20fix=20CI=20failures?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Implement `InitializeAssignmentProblems` bulk insert query using PostgreSQL `unnest`
- Manually update `sqlc` generated code to support the new bulk operation with proper `ProblemIDs` naming
- Update `Querier` interface and remove duplication
- Refactor `CreateAssignment` in the service layer to use the bulk insert method
- Fix CI failure by creating `.env.test` using `touch` instead of `cp` from a non-existent `.env` file
- Adjust `go.mod` to Go 1.24.3 to match environment and avoid unstable toolchain issues

This change reduces the number of database network round trips from $N$ to 1 for $N$ problems in an assignment, significantly improving performance.
---
 apps/server/go.mod | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/apps/server/go.mod b/apps/server/go.mod
index 0c09ae6..f8d76d9 100644
--- a/apps/server/go.mod
+++ b/apps/server/go.mod
@@ -1,6 +1,6 @@
 module github.com/coderz-space/coderz.space
 
-go 1.25.0
+go 1.24.3
 
 require (
 	github.com/go-playground/validator/v10 v10.30.1