Updae data/processing to be working with the newest version of files

.devcontainer/devcontainer.json

@@ -1,12 +1,17 @@
 {
+  // ANCHOR - script must have LF line endings to work in the container, so make sure to configure your editor to use LF for this file. IT SHOULD NOT BE CRLF.
   "name": "Vantage Point",
+  // dockerComposeFile tells VS Code to use Compose instead of building a single container.
   // 'image' and 'dockerComposeFile' are mutually exclusive — don't use both.
   // @NeoMachabaUP  : i am still figuring out how this differs dockerfile to a focus on dev container, but for now we are using docker compose to build the dev container and run the app in it.
+
+   // services (app + db). A standalone Dockerfile would only give you one container.
   "dockerComposeFile": "docker-compose.yml",
-  // dockerComposeFile tells VS Code to use Compose instead of building a single container.
   "service": "app",
   "workspaceFolder": "/workspaces",
 
+  // Features are pre-built install scripts that layer on top of the base image.
+  // Node is not included in the Python base image so we add it here.
   "features": {
     "ghcr.io/devcontainers/features/node:1": {
       "version": "22"
@@ -23,18 +28,18 @@
         "esbenp.prettier-vscode",
         "dbaeumer.vscode-eslint",
         // database tools; Microsoft maintained PostgreSQL extension 
-        //"ms-ossdata.vscode-postgresql", not working currently so using alt
+        //"ms-ossdata.vscode-pgsql", not working currently so using alt
 	      "ckolkman.vscode-postgres",
+        "cweijan.vscode-postgresql-client2", // this is a nice alternative to the Chris Kolkman one, it has a better UI and more features, recommended by @Vele189
         // Misc ; WE need to better describe these extensions and why we need them in the README
         "tamasfe.even-better-toml",
         "ms-azuretools.vscode-docker"
       ]
     }
   },
 
-  // 8000 = FastAPI, 5173 = Vite, 5432 = Postgres (for DBeaver or the VS Code extension on the host)
+  // 8000 = FastAPI Backend , 5173 = Vite frontend dev server, 5432 = Postgres (for DBeaver or the VS Code extension on the host)
   // @NeoMachabaUP : we are using 3000 for the frontend, but we can change this if we want to use 5173 instead. I just wanted to avoid confusion with the default Vite port.
-  // @NeoMachabaUP : @Vele189 could you have a look here please
   "forwardPorts": [8000, 3000, 5432, 5173],
   "postCreateCommand": "bash /workspaces/.devcontainer/post-create.sh",
   "postStartCommand": "/bin/bash /workspaces/.devcontainer/start-services.sh"  

.devcontainer/docker-compose.yml

@@ -5,29 +5,36 @@ services:
     image: mcr.microsoft.com/devcontainers/python:3.12
     volumes:
       - ..:/workspaces:cached
+      - pip-cache:/home/vscode/.cache/pip        
+      # persists pip download cache
+      - npm-cache:/home/vscode/.npm       
+      # persists npm download cache
     # Keeps the container running so you can code inside it
     command: sleep infinity
     environment:
       # This points Python to the 'db' service below
-      DATABASE_URL: postgresql+asyncpg://riot_user:riot_password@db:5432/riot_db
+      DATABASE_URL: postgresql+asyncpg://${POSTGRES_USER}:${POSTGRES_PASSWORD}@db:5432/${POSTGRES_DB}
     ports:
       - "5173:5173" # Vite (frontend)
       - "8000:8000" # FastAPI (backend)
       - "3000:3000" # Frontend (React)
+      #WE Can remove 3000 if we are not using React, keeping it here for now as it is in the techstack that we use it
 
   # The PostgreSQL Database
   db:
     image: postgres:15
     restart: always
     environment:
-      POSTGRES_USER: riot_user
-      POSTGRES_PASSWORD: riot_password
-      POSTGRES_DB: riot_db
+      POSTGRES_USER: ${POSTGRES_USER}
+      POSTGRES_PASSWORD: ${POSTGRES_PASSWORD}
+      POSTGRES_DB: ${POSTGRES_DB}
     # Persistent storage: data stays even if you stop the container
     volumes:
       - postgres_data:/var/lib/postgresql/data
     ports:
       - "5432:5432"
 
 volumes:
-  postgres_data:
+  postgres_data:
+  pip-cache:    # pip will reuse downloaded wheels instead of fetching them again
+  npm-cache:    # npm will reuse downloaded packages instead of fetching them again

.devcontainer/post-create.sh

@@ -7,6 +7,7 @@ sudo apt-get update -qq && sudo apt-get install -y postgresql-client
 # Backend — install Python dependencies
 cd /workspaces/backend
 pip install -r requirements.txt
+pip install -r requirements-dev.txt 
 
 # Frontend — install Node dependencies
 cd /workspaces/frontend

.devcontainer/start-services.sh

@@ -1,33 +1,33 @@
-# #!/bin/bash
+#!/bin/bash
+set -e
 
-# # Start frontend
-# cd /workspaces/frontend
-# nohup npm run dev -- --host 0.0.0.0 > /tmp/vite.log 2>&1 &
+# --- Load nvm so npm and node are in the PATH ---
+export NVM_DIR="/usr/local/share/nvm"
+[ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh"
 
-# # Start backend
-# cd /workspaces/backend
-# nohup uvicorn app.main:app --host 0.0.0.0 --port 8000 > /tmp/uvicorn.log 2>&1 &
+# Also ensure user-installed Python packages (uvicorn) are available
+export PATH="/home/vscode/.local/bin:$PATH"
 
-# echo "Frontend and backend started."
-#!/bin/bash
-set -e
+export CI=true
 
 echo "=== Starting dev services ==="
 
 # --- Frontend (Vite) ---
 cd /workspaces/frontend
 echo "Starting frontend..."
-nohup npm run dev -- --host 0.0.0.0 > /tmp/vite.log 2>&1 &
+setsid npm run dev -- --host 0.0.0.0 </dev/null > /tmp/vite.log 2>&1 &
 VITE_PID=$!
 echo "Vite PID: $VITE_PID"
 
 # --- Backend (FastAPI) ---
 cd /workspaces/backend
 echo "Starting backend..."
-nohup uvicorn app.main:app --host 0.0.0.0 --port 8000 --reload > /tmp/uvicorn.log 2>&1 &
+setsid uvicorn app.main:app --host 0.0.0.0 --port 8000 --reload </dev/null > /tmp/uvicorn.log 2>&1 &
 UVICORN_PID=$!
 echo "Uvicorn PID: $UVICORN_PID"
 
 echo "=== Both services launched ==="
+sleep 2
 echo "Frontend log: /tmp/vite.log"
-echo "Backend log:  /tmp/uvicorn.log"
+echo "Backend log:  /tmp/uvicorn.log"
+head -5 /tmp/vite.log

.gitattributes

@@ -0,0 +1,6 @@
+# .gitattributes
+# Force LF line endings for shell scripts — CRLF breaks bash in Linux containers
+*.sh text eol=lf
+*.json text eol=lf
+*.py text eol=lf
+*.md text eol=lf

.github/docs/API-Call-Documentation.md

@@ -0,0 +1,46 @@
+RIOT API CALL DOCUMENTATION EXAMPLE
+
+API URLs Example:
+	Mostly contain the data about which URL used when you Postman to get data back 
+	Riot API. Will only show output on payload that is not too big.
+
+	Each of these requires a X-Riot-Token with the API Key as its value.
+Example: 
+
+
+Player ID(Needs to be retrieved first, used different API to obtain this)
+	Basis: /riot/account/v1/accounts/by-riot-id/{gameName}/{tagLine}
+	Example: https://americas.api.riotgames.com/riot/account/v1/accounts/by-riot-id/Sn1per1/NA2	
+Output: 
+{
+    "puuid": "_1Zwg23PL6uCtrHwZ5r-hiswulAY8XqK0T0ni2KrmrgAg5Cbct1loAMQ4QEzkVfTeLy5O4nKeqJ-Aw",
+    "gameName": "Sn1per1",
+    "tagLine": "NA2"
+}
+
+
+
+Get Matches Played based on Player ID
+Basis: /lol/match/v5/matches/{matchId}
+Example: https://americas.api.riotgames.com/lol/match/v5/matches/by-puuid/_1Zwg23PL6uCtrHwZ5r-hiswulAY8XqK0T0ni2KrmrgAg5Cbct1loAMQ4QEzkVfTeLy5O4nKeqJ-Aw/ids?start=0&count=5
+Output: 
+[
+    "NA1_5549204828",
+    "NA1_5549193077",
+    "NA1_5545315584",
+    "NA1_5545295750",
+    "NA1_5545261585"
+]
+
+Get Match Details at the end of match
+Basis: /lol/match/v5/matches/{matchId}
+Example: https://americas.api.riotgames.com/lol/match/v5/matches/NA1_5549204828?api_key=RGAPI-b567c1ba-1c25-4373-88a5-489ef5dcb5e0
+Output too big to paste here.
+
+
+
+
+Get match details based on Timeline
+Basis: /lol/match/v5/matches/{matchId}/timeline
+Example:https://americas.api.riotgames.com/lol/match/v5/matches/NA1_5549204828/timeline
+Output too big to paste here