Merge pull request #18 from merendamattia/develop

Update financial assessment questions and advisor tools

Author: merendamattia

.env.example

@@ -1,6 +1,6 @@
 # Ollama Configuration
 OLLAMA_API_URL=http://localhost:11434/v1
-OLLAMA_MODEL=qwen3:0.6b
+OLLAMA_MODEL=llama3.2:3b
 
 # Google Configuration
 # Get API key from: https://aistudio.google.com/app/apikey
@@ -10,7 +10,7 @@ GOOGLE_MODEL=gemini-2.5-flash
 # OpenAI Configuration
 # Get API key from: https://platform.openai.com/api-keys
 OPENAI_API_KEY=your_openai_api_key_here
-OPENAI_MODEL=gpt-4o-mini
+OPENAI_MODEL=gpt-4.1-mini
 
 # Agent Configuration
 AGENT_NAME=PersonalFinancialAIAgent

prompts/questions.json

@@ -2,46 +2,50 @@
   "questions": [
     {
       "id": 1,
-      "text": "Ciao! 👋 Iniziamo con le informazioni personali. Quanti anni hai e che lavoro fai? (ad esempio: 30 anni, impiegato)"
+      "text": "Ciao! 👋 Iniziamo con le informazioni personali. Quanti anni hai e che lavoro fai? (ad esempio: ho 30 anni e sono un impiegato)"
     },
     {
       "id": 2,
-      "text": "Qual è il tuo reddito mensile netto approssimativo (considerando tutte le fonti di reddito aggiuntive) e come valuti la stabilità del tuo reddito? (ad esempio: 3000€, stabile)"
+      "text": "Com'è la tua situazione familiare? Sei single, fidanzato o sposato? Hai figli a carico? (ad esempio: sposato con due figli minorenni a carico)"
     },
     {
       "id": 3,
-      "text": "Quanto riesci a risparmiare al mese? (ad esempio: 200€)"
+      "text": "Spostandoci sul lato finanziario, qual è il tuo reddito mensile netto approssimativo (considerando tutte le fonti di reddito aggiuntive)? E come valuti la stabilità del tuo reddito, hai un contratto a tempo indeterminato o determinato? (ad esempio: 3000€, con contratto a tempo indeterminato)"
     },
     {
       "id": 4,
-      "text": "A quanto ammontano le tue spese mensili? Quali sono? (ad esempio: totale 1500€, di cui affitto 700€, cibo 300€, trasporti 200€, tempo libero 300€)"
+      "text": "Quanto riesci a risparmiare al mese? (ad esempio: 200€)"
     },
     {
       "id": 5,
-      "text": "Attualmente hai debiti? Se sì, a quanto ammontano? (ad esempio: 20000€ di mutuo)"
+      "text": "A quanto ammontano le tue spese mensili? Quali sono? (ad esempio: totale 1500€, di cui casa 700€, cibo 300€, trasporti 200€, tempo libero 300€)"
     },
     {
       "id": 6,
-      "text": "Hai familiari a carico? Hai inoltre coperture assicurative attive? (ad esempio: due figli e ho un'assicurazione sulla vita)"
+      "text": "Attualmente hai debiti? Se sì, a quanto ammontano? (ad esempio: 20000€ di mutuo)"
     },
     {
       "id": 7,
-      "text": "Hai investimenti attivi? Se sì, a quanto ammonta la cifra investita? (ad esempio: 20000€ in azioni e 10000€ in obbligazioni)"
+      "text": "Hai coperture assicurative attive? (ad esempio: assicurazione sulla vita)"
     },
     {
       "id": 8,
-      "text": "Quali sono i tuoi principali obiettivi finanziari? (ad esempio: comprare casa, pensione anticipata, accumulare ricchezza, FIRE)"
+      "text": "Hai investimenti attivi? Se sì, a quanto ammonta la cifra investita? (ad esempio: 20000€ in azioni e 10000€ in obbligazioni)"
     },
     {
       "id": 9,
-      "text": "Come definisci la tua propensione al rischio? Investiresti per preservare il capitale o per farlo crescere? (ad esempio: aggressivo, preferisco far crescere il capitale)"
+      "text": "Quali sono i tuoi principali obiettivi finanziari? (ad esempio: comprare casa, pensione anticipata, accumulare ricchezza, FIRE)"
     },
     {
       "id": 10,
-      "text": "Per quanto riguarda gli investimenti geografici, preferisci mercati emergenti, USA, Europa, oppure una distribuzione bilanciata globale?"
+      "text": "Come definisci la tua propensione al rischio? Investiresti per preservare il capitale o per farlo crescere? (ad esempio: aggressivo, preferisco far crescere il capitale)"
     },
     {
       "id": 11,
+      "text": "Per quanto riguarda gli investimenti geografici, preferisci mercati emergenti, USA, Europa, oppure una distribuzione bilanciata globale?"
+    },
+    {
+      "id": 12,
       "text": "C'è qualcos'altro di importante sulla tua situazione finanziaria che vorresti condividere?"
     }
   ]

src/core/financial_advisor.py

@@ -86,13 +86,13 @@ def _get_default_system_prompt(self) -> str:
 
     def _get_default_tools(self) -> list:
         """
-        Financial advisor has asset analysis tool.
+        Financial advisor has NO tools.
 
         Returns:
-            List with analyze_financial_asset tool
+            Empty list - no tools for Financial advisor
         """
-        logger.debug("Setting up financial advisor tools")
-        return [analyze_financial_asset]
+        logger.debug("Financial advisor initialized with no tools")
+        return []
 
     def _get_default_planning_interval(self) -> int:
         """

tests/unit/test_financial_profile.py

@@ -33,7 +33,6 @@ def test_optional_fields(self):
             investment_experience="beginner",
             risk_tolerance="conservative",
         )
-        assert profile.occupation == "None"
         assert profile.summary_notes == "None"
 
     def test_geographic_allocation_default(self):
@@ -73,7 +72,6 @@ def test_model_dump(self, sample_financial_profile):
         assert isinstance(profile_dict, dict)
         assert "age_range" in profile_dict
         assert profile_dict["age_range"] == "30-39"
-        assert profile_dict["occupation"] == "Software Engineer"
 
     def test_model_dump_json(self, sample_financial_profile):
         """Test converting profile to JSON."""