Merge branch 'refactor/backend' of github.com:Agamya-Samuel/wikicontest into ft/add-alembic-support

Author: Agamya-Samuel

.github/workflows/pylint.yml

@@ -34,14 +34,14 @@ jobs:
           if [ -f requirements.txt ]; then python -m pip install -r requirements.txt; fi
           python -m pip install pylint
 
-      - name: Run pylint on app.py
+      - name: Run pylint on main.py
         working-directory: backend
-        run: python -m pylint --fail-under=9 app.py
+        run: python -m pylint --fail-under=9 main.py
 
       - name: Run pylint on models
         working-directory: backend
-        run: python -m pylint --fail-under=9 models/
+        run: python -m pylint --fail-under=9 app/models/
 
       - name: Run pylint on routes
         working-directory: backend
-        run: python -m pylint --fail-under=9 routes/
+        run: python -m pylint --fail-under=9 app/routes/

backend/README.md

@@ -60,7 +60,7 @@ backend/
 ├── logs/                        # Application logs
 ├── docs/                        # Documentation
 │   ├── ALEMBIC_USAGE_GUIDE.md  # Detailed Alembic usage guide
-├── run.py                       # Main entry point for running the app
+├── main.py                      # Main entry point for running the app
 ├── alembic.ini                  # Alembic configuration file
 ├── Makefile                     # Makefile for common commands
 ├── requirements.txt             # Python dependencies
@@ -144,7 +144,7 @@ make help
 
 1. **Start the Flask development server:**
    ```bash
-   python run.py
+   python main.py
    ```
 
    Or using Flask directly:

backend/app/routes/contest_routes.py

@@ -177,7 +177,7 @@ def create_contest():
             marks_setting_accepted=marks_accepted,
             marks_setting_rejected=marks_rejected,
             jury_members=jury_members,
-            allowed_submission_type=allowed_submission_type  
+            allowed_submission_type=allowed_submission_type
         )
 
         contest.save()
@@ -367,16 +367,16 @@ def update_contest(contest_id):  # pylint: disable=too-many-return-statements
                 contest.set_rules(rules_payload)
             else:
                 contest.set_rules({'text': ''})
-        
+
         if "allowed_submission_type" in data:
-           new_type = data.get("allowed_submission_type", "both")
+            new_type = data.get("allowed_submission_type", "both")
 
-          # Validate only allowed values
-        if new_type not in ["new", "expansion", "both"]:
+            # Validate only allowed values
+            if new_type not in ["new", "expansion", "both"]:
                 return jsonify({"error": "Invalid allowed_submission_type"}), 400
 
-        contest.allowed_submission_type = new_type
-    
+            contest.allowed_submission_type = new_type
+
         # --- Dates ---
         if 'start_date' in data:
             parsed = parse_date_or_none(data.get('start_date'))

backend/run.py backend/main.pybackend/run.py renamed to backend/main.py

@@ -6,13 +6,16 @@
 For production, use a WSGI server like gunicorn or uwsgi.
 
 Usage:
-    python run.py
+    python main.py
 """
 
-from app import app, db
+# Third-party imports should come before local imports
 from sqlalchemy import inspect, text
 from sqlalchemy.exc import SQLAlchemyError, ProgrammingError, OperationalError
 
+# Local application imports
+from app import app, db
+
 def migrate_database():
     """
     Run database migrations to add new columns if they don't exist.

backend/migrations/README.md

@@ -49,7 +49,7 @@ python migrations/add_size_at_start_to_submissions.py
 
 ```bash
 # The application runs all migrations automatically on startup
-python run.py
+python main.py
 ```
 
 ## Migration Safety

docs/RUN_DEVELOPMENT.md

@@ -12,7 +12,7 @@ Open a terminal and run:
 
 ```bash
 cd backend
-python app.py
+python main.py
 ```
 
 Flask will start on `http://localhost:5000`
@@ -63,7 +63,7 @@ This creates a `dist/` directory with production files.
 
 ```bash
 cd backend
-python app.py
+python main.py
 ```
 
 Flask will automatically detect and serve the built Vue.js files from `frontend/dist/`.
@@ -96,7 +96,7 @@ Create `start-dev.ps1`:
 
 ```powershell
 # Start Flask backend
-Start-Process powershell -ArgumentList "-NoExit", "-Command", "cd backend; python app.py"
+Start-Process powershell -ArgumentList "-NoExit", "-Command", "cd backend; python main.py"
 
 # Wait a moment
 Start-Sleep -Seconds 2
@@ -116,7 +116,7 @@ Create `start-dev.sh`:
 
 # Start Flask backend in background
 cd backend
-python app.py &
+python main.py &
 BACKEND_PID=$!
 
 # Wait a moment
@@ -140,15 +140,15 @@ Run with: `./start-dev.sh`
 ### Port Already in Use
 
 **Flask (port 5000):**
-- Change port in `backend/app.py`: `app.run(port=5001)`
+- Change port in `backend/main.py`: `app.run(port=5001)`
 
 **Vue.js (port 5173):**
 - Change port in `frontend/vite.config.js`: `server: { port: 5174 }`
 
 ### API Requests Failing
 
 - Ensure Flask is running on port 5000
-- Check CORS configuration in `backend/app.py`
+- Check CORS configuration in `backend/app/__init__.py`
 - Verify proxy settings in `frontend/vite.config.js`
 
 ### Module Not Found Errors
@@ -169,7 +169,7 @@ npm install
 
 For daily development, use **Option 1** (separate servers):
 
-1. Terminal 1: `cd backend && python app.py`
+1. Terminal 1: `cd backend && python main.py`
 2. Terminal 2: `cd frontend && npm run dev`
 3. Open browser: http://localhost:5173