Update app.py

app.py

@@ -1,7 +1,5 @@
 import logging
 import os
-from logging.handlers import RotatingFileHandler
-
 from flask import (
     Flask, render_template, request, redirect,
     url_for, flash, jsonify, Response
@@ -15,33 +13,27 @@ from dotenv import load_dotenv
 from openai import OpenAI
 from datetime import datetime, timedelta
 
-# ─── Load environment variables ────────────────────────────────────────────────
+# ─── 1) Load env vars ──────────────────────────────────────────────────────────
 load_dotenv()
 
-# ─── Logging configuration ────────────────────────────────────────────────────
-import logging
-import os
-
-# ─── Logging to stdout only ───────────────────────────────────────────────────
+# ─── 2) Logging to stdout ─────────────────────────────────────────────────────
 logging.basicConfig(
     level=logging.INFO,
     format="[%(asctime)s] %(levelname)s in %(module)s: %(message)s"
 )
-logger = logging.getLogger()  # root logger
+logger = logging.getLogger()
 
-# ─── Flask setup ───────────────────────────────────────────────────────────────
-from flask import Flask
+# ─── 3) Flask setup ───────────────────────────────────────────────────────────
 app = Flask(__name__)
 app.logger.handlers = logger.handlers
 app.logger.setLevel(logger.level)
 
-# ─── OpenAI client setup ──────────────────────────────────────────────────────
-# client = OpenAI(api_key=os.getenv("OPENAI_API_KEY"))  # if you prefer v1 client
+# ─── 4) OpenAI client setup ──────────────────────────────────────────────────
 OPENAI_API_KEY = os.getenv("OPENAI_API_KEY")
 if not OPENAI_API_KEY:
     app.logger.warning("OPENAI_API_KEY is not set in the environment")
 
-# ─── Database configuration ───────────────────────────────────────────────────
+# ─── 5) Database config ───────────────────────────────────────────────────────
 app.config["SQLALCHEMY_DATABASE_URI"] = "sqlite:///jobs.db"
 app.config["SQLALCHEMY_TRACK_MODIFICATIONS"] = False
 app.secret_key = os.getenv("SECRET_KEY", "your_secret_key_here")
@@ -49,7 +41,7 @@ app.secret_key = os.getenv("SECRET_KEY", "your_secret_key_here")
 db = SQLAlchemy(app)
 migrate = Migrate(app, db)
 
-# ─── Models ───────────────────────────────────────────────────────────────────
+# ─── 6) Models ────────────────────────────────────────────────────────────────
 class Job(db.Model):
     id = db.Column(db.Integer, primary_key=True)
     company = db.Column(db.String(100), nullable=False)
@@ -65,7 +57,7 @@ class Job(db.Model):
     def __repr__(self):
         return f"<Job {self.company} - {self.position}>"
 
-# ─── Helpers ──────────────────────────────────────────────────────────────────
+# ─── 7) Helpers ───────────────────────────────────────────────────────────────
 def convert_excel_serial_dates():
     updated = False
     for job in Job.query.all():
@@ -77,7 +69,7 @@ def convert_excel_serial_dates():
                 job.date_applied = dt.strftime("%Y-%m-%d")
                 updated = True
             except ValueError:
-                continue
+                pass
     if updated:
         db.session.commit()
         app.logger.info("Converted Excel serial dates to ISO strings")
@@ -91,7 +83,7 @@ def generate_interview_plan(job_description: str) -> str:
                 {"role": "system", "content": "You are a helpful assistant."},
                 {"role": "user",   "content":
                     f"Create an interview plan for the following job description. "
-                    f"Highlight key skills and requirements:\n{job_description}"
+                    f"Highlight key skills and requirements:\n\n{job_description}"
                 },
             ],
             max_tokens=150
@@ -101,16 +93,14 @@ def generate_interview_plan(job_description: str) -> str:
         app.logger.exception("Error generating interview plan")
         return "Error generating interview plan. Please try again later."
 
-# ─── Auto-initialize DB ────────────────────────────────────────────────────────
-
-@app.before_serving
-def initialize_database():
+# ─── 8) Immediate DB init ────────────────────────────────────────────────────
+with app.app_context():
     db.create_all()
     convert_excel_serial_dates()
     app.logger.info("Database initialized")
 
+# ─── 9) Routes ────────────────────────────────────────────────────────────────
 
-# ─── Routes ───────────────────────────────────────────────────────────────────
 @app.route("/")
 def index():
     return redirect(url_for("jobs"))
@@ -135,8 +125,8 @@ def jobs():
             return redirect(url_for("jobs"))
         except SQLAlchemyError:
             app.logger.exception("Error adding job")
-            flash("Error adding job. Check logs for details.")
-    # GET
+            flash("Error adding job. See logs for details.")
+    # GET handling
     search = request.args.get("search", "").strip()
     status_filter = request.args.get("status_filter", "").strip()
     sort_by = request.args.get("sort", "")
@@ -159,8 +149,8 @@ def jobs():
 
     jobs_list = query.all()
     counts = {
-        "Applied":    Job.query.filter_by(status="Applied").count(),
-        "Rejected":   Job.query.filter_by(status="Rejected").count(),
+        "Applied":      Job.query.filter_by(status="Applied").count(),
+        "Rejected":     Job.query.filter_by(status="Rejected").count(),
         "Interviewing": Job.query.filter_by(status="Interviewing").count()
     }
     return render_template(
@@ -191,7 +181,7 @@ def edit_job(job_id):
             return redirect(url_for("jobs"))
         except SQLAlchemyError:
             app.logger.exception("Error updating job")
-            flash("Error updating job. Check logs for details.")
+            flash("Error updating job. See logs for details.")
             db.session.rollback()
     return render_template("jobs.html", jobs=Job.query.all(), edit_job=job)
 
@@ -203,7 +193,7 @@ def delete_job(job_id):
         db.session.commit()
     except SQLAlchemyError:
         app.logger.exception("Error deleting job")
-        flash("Error deleting job. Check logs for details.")
+        flash("Error deleting job. See logs for details.")
         db.session.rollback()
     return redirect(url_for("jobs"))
 
@@ -244,7 +234,7 @@ def prepme(job_id):
             model="gpt-4o-mini",
             messages=[
                 {"role": "system", "content": "You are a career coach."},
-                {"role": "user",   "content": context},
+                {"role": "user",   "content": context}
             ],
             max_tokens=150
         )
@@ -294,7 +284,8 @@ def download_jobs():
         headers={"Content-Disposition": "attachment;filename=jobs.csv"}
     )
 
-# ─── Application entry point ─────────────────────────────────────────────────
+# ─── 10) Entry point ─────────────────────────────────────────────────────────
 if __name__ == "__main__":
     port = int(os.environ.get("PORT", 5000))
     app.run(host="0.0.0.0", port=port, debug=False)
+