Merge pull request #68 from crux-bphc/chore/cleanup

Pre-release db and build changes

Author: CWSwastik

README.md

@@ -64,6 +64,37 @@ The official Competitive Coding website for BITS Pilani, Hyderabad Campus. Surge
    docker compose --profile prod down
    ```
 
+## Migrations
+**Make sure you have the dev profile up and running before running migrations.**
+
+
+The two containers required to run and generate migrations are:
+- `backend-dev`
+- `postgres`
+
+After the above mentioned containers are running you can use the following commands
+
+### Apply existing migrations
+```bash
+docker compose exec backend-dev npx drizzle-kit migrate
+```
+
+Runs migrations which haven't been applied yet. A list of migrations which have been applied is stored in a table in the database itself. Read more about this [here](https://github.com/jbranchaud/til/blob/master/drizzle/drizzle-tracks-migrations-in-a-log-table.md).
+
+### Apply current schema to the database
+```bash
+docker compose exec backend-dev npx drizzle-kit push
+```
+This command **does not** generate migrations! Make sure to generate migrations for any major changes to ensure that the history of the database schema is always preserved.
+
+
+### Generate migrations from schema
+```bash
+docker compose exec backend-dev npx drizzle-kit generate
+```
+This compares the current state of the database with the schema and generates migrations.
+
+
 ## Contribution
 
 Follow the [Contributing Guide](./CONTRIBUTING.md).

backend/docker/prod/Dockerfile

@@ -2,9 +2,12 @@ FROM node:22.17-bookworm
 
 WORKDIR /app
 
-COPY ./backend .
+COPY ./backend/package*.json ./
+COPY ./backend/tsconfig.json ./
 
 RUN npm install
+
+COPY ./backend .
 RUN npm run build
 
 CMD ["sh", "-c", "npm run start"]

…ckend/src/drizzle/0001_store_cf_data.sql backend/src/drizzle/0000_init.sqlbackend/src/drizzle/0001_store_cf_data.sql renamed to backend/src/drizzle/0000_init.sql backend/src/drizzle/0001_store_cf_data.sql renamed to backend/src/drizzle/0000_init.sql

@@ -1,4 +1,3 @@
-/*
 CREATE TYPE "public"."verdict" AS ENUM('AC', 'WA', 'TLE', 'RE', 'CE', 'other');--> statement-breakpoint
 CREATE TABLE "cf_user_stats" (
 	"id" serial PRIMARY KEY NOT NULL,
@@ -68,20 +67,26 @@ CREATE TABLE "user_contests" (
 	"update_time" timestamp(0)
 );
 --> statement-breakpoint
-ALTER TABLE "_prisma_migrations" DISABLE ROW LEVEL SECURITY;--> statement-breakpoint
-DROP TABLE "_prisma_migrations" CASCADE;--> statement-breakpoint
-ALTER TABLE "User" RENAME TO "users";--> statement-breakpoint
-ALTER TABLE "users" RENAME COLUMN "pfpUrl" TO "pfp_url";--> statement-breakpoint
-ALTER TABLE "users" RENAME COLUMN "cfHandle" TO "cf_handle";--> statement-breakpoint
-ALTER TABLE "users" RENAME COLUMN "cfRating" TO "cf_rating";--> statement-breakpoint
-ALTER TABLE "users" RENAME COLUMN "createdAt" TO "created_at";--> statement-breakpoint
-DROP INDEX "User_cfHandle_key";--> statement-breakpoint
-DROP INDEX "User_email_key";--> statement-breakpoint
-ALTER TABLE "users" ADD COLUMN "codechef_handle" text;--> statement-breakpoint
-ALTER TABLE "users" ADD COLUMN "codechef_rating" integer;--> statement-breakpoint
-ALTER TABLE "users" ADD COLUMN "atcoder_handle" text;--> statement-breakpoint
-ALTER TABLE "users" ADD COLUMN "atcoder_rating" integer;--> statement-breakpoint
-ALTER TABLE "users" ADD COLUMN "leetcode_handle" text;--> statement-breakpoint
+CREATE TABLE "users" (
+	"id" text PRIMARY KEY NOT NULL,
+	"email" text NOT NULL,
+	"name" text,
+	"pfp_url" text,
+	"cf_handle" text,
+	"cf_rating" integer,
+	"codechef_handle" text,
+	"codechef_rating" integer,
+	"atcoder_handle" text,
+	"atcoder_rating" integer,
+	"leetcode_handle" text,
+	"created_at" timestamp(3) DEFAULT now() NOT NULL,
+	CONSTRAINT "users_email_unique" UNIQUE("email"),
+	CONSTRAINT "users_cf_handle_unique" UNIQUE("cf_handle"),
+	CONSTRAINT "users_codechef_handle_unique" UNIQUE("codechef_handle"),
+	CONSTRAINT "users_atcoder_handle_unique" UNIQUE("atcoder_handle"),
+	CONSTRAINT "users_leetcode_handle_unique" UNIQUE("leetcode_handle")
+);
+--> statement-breakpoint
 ALTER TABLE "cf_user_stats" ADD CONSTRAINT "cf_user_stats_user_id_users_id_fk" FOREIGN KEY ("user_id") REFERENCES "public"."users"("id") ON DELETE no action ON UPDATE no action;--> statement-breakpoint
 ALTER TABLE "potd" ADD CONSTRAINT "potd_problem_id_problems_id_fk" FOREIGN KEY ("problem_id") REFERENCES "public"."problems"("id") ON DELETE no action ON UPDATE no action;--> statement-breakpoint
 ALTER TABLE "potd_solves" ADD CONSTRAINT "potd_solves_potd_id_potd_id_fk" FOREIGN KEY ("potd_id") REFERENCES "public"."potd"("id") ON DELETE no action ON UPDATE no action;--> statement-breakpoint
@@ -92,9 +97,4 @@ ALTER TABLE "submissions" ADD CONSTRAINT "submissions_problem_id_problems_id_fk"
 ALTER TABLE "user_contests" ADD CONSTRAINT "user_contests_user_id_users_id_fk" FOREIGN KEY ("user_id") REFERENCES "public"."users"("id") ON DELETE no action ON UPDATE no action;--> statement-breakpoint
 ALTER TABLE "user_contests" ADD CONSTRAINT "user_contests_contest_id_contests_external_id_fk" FOREIGN KEY ("contest_id") REFERENCES "public"."contests"("external_id") ON DELETE no action ON UPDATE no action;--> statement-breakpoint
 CREATE UNIQUE INDEX "problems_contest_index" ON "problems" USING btree ("contest_id","index");--> statement-breakpoint
-ALTER TABLE "users" ADD CONSTRAINT "users_email_unique" UNIQUE("email");--> statement-breakpoint
-ALTER TABLE "users" ADD CONSTRAINT "users_cf_handle_unique" UNIQUE("cf_handle");--> statement-breakpoint
-ALTER TABLE "users" ADD CONSTRAINT "users_codechef_handle_unique" UNIQUE("codechef_handle");--> statement-breakpoint
-ALTER TABLE "users" ADD CONSTRAINT "users_atcoder_handle_unique" UNIQUE("atcoder_handle");--> statement-breakpoint
-ALTER TABLE "users" ADD CONSTRAINT "users_leetcode_handle_unique" UNIQUE("leetcode_handle");
-*/
+CREATE UNIQUE INDEX "user_contest_unique" ON "user_contests" USING btree ("user_id","contest_id");