BUG: Prevent ai.py import crash when env vars are missing
I maintainer di solito rispondono entro 1 giorno
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 52/100
Direzione di ricerca
Inizia da Backend/app/routes/ai.py e riproduci il fallimento eseguendo uvicorn app.main:app --reload con SUPABASE_URL, SUPABASE_KEY e GEMINI_API_KEY non impostate. Traccia la convalida al momento dell’importazione e i gestori delle route AI; il lavoro è completo quando l’app FastAPI si avvia, le route non correlate rimangono disponibili e le richieste AI restituiscono un 500 chiaro in caso di configurazione mancante.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Is there an existing issue for this?
- I have searched the existing issues
What happened?
📌 Issue Overview
Importing Backend/app/routes/ai.py currently raises ValueError at module import if SUPABASE_URL, SUPABASE_KEY, or GEMINI_API_KEY are unset. This aborts FastAPI startup, so the entire API is down when any of those env vars are missing—blocking unrelated routes.
🔍 Steps to Reproduce
- Unset SUPABASE_URL/SUPABASE_KEY/GEMINI_API_KEY in the backend environment.
- Start the FastAPI app (e.g.,
uvicorn app.main:app --reload). - Observe startup failure.
🎯 Expected Behavior
The app should start; only the AI endpoints should fail gracefully per-request, returning a clear 500 explaining missing config.
🚨 Actual Behavior
FastAPI fails at import time with ValueError("Missing required environment variables: SUPABASE_URL, SUPABASE_KEY, GEMINI_API_KEY"), preventing any route from serving.
📷 Screenshot
N/A (startup crash)
💡 Suggested Improvements
- Move env validation from module import to within the AI route handlers.
- On missing keys, return 500 JSON with a clear message; log a startup warning instead of raising.
- Add a lightweight check/test to cover the missing-env path.
Record
- I agree to follow this project's Code of Conduct
- I want to work on this issue
- Lingua principale
- TypeScript
- Stelle
- 101
- Fork
- 145
- Merge medio
- 32m
- PR unite (30g)
- 2
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di AOSSIE-Org/InPactAI
-
UI Bug: Low contrast makes input text unreadable in Sign Up and Login formForse già presa @Abhik-Mudi l’ha presa 312 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
AOSSIE-Org/InPactAI#218 ·
I maintainer di solito rispondono entro 1 giorno
-
DOCS UPDATE: Fix incorrect .env file location in Backend Setup instructionsForse già presa @PulkitDadwal l’ha presa 326 giorni fa. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 68/100
AOSSIE-Org/InPactAI#201 ·
I maintainer di solito rispondono entro 1 giorno
-
Maintainer note: v1 wrap-upApertaenhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 12/100
AOSSIE-Org/InPactAI#315 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 42/100
AOSSIE-Org/InPactAI#314 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
AOSSIE-Org/InPactAI#312 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di AOSSIE-Org/InPactAI
Issue simili
-
ai-driven-qa bug claude Minor
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
linagora/twake-contacts-frontend#241 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
ui(ListToolbar): search takes the full row on a phone instead of sharing it with the result countAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
I maintainer di solito rispondono entro 1 giorno
-
priority:high
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
ahmedkhaleel2004/gitdiagram#228 ·