refactor: update percentage validation logic in Portfolio model

Author: merendamattia

src/models/portfolio.py

@@ -5,7 +5,6 @@
 a structured investment portfolio recommendation.
 """
 
-import math
 from enum import Enum
 from typing import List
 
@@ -98,55 +97,12 @@ class Portfolio(BaseModel):
         },
     )
 
-    # @model_validator(mode="after")
-    # def validate_total_percentage(self):
-    #     """Validate that asset percentages sum to approximately 100."""
-    #     total = sum(asset.percentage for asset in self.assets)
-    #     if not (99 <= total <= 101):
-    #         raise ValueError(
-    #             f"Asset percentages must sum to 100%, got {total}% instead"
-    #         )
-    #     return self
-
-    # TODO: completare nuova versione del model validator e eliminare la vecchia qui sopra
     @model_validator(mode="after")
-    def normalize_total_percentage(self):
-        """
-        Auto-correct asset percentages to ensure they sum to exactly 100%.
-        Instead of raising an error, specifically re-proportions the values.
-        """
-        if not self.assets:
-            return self
-
+    def validate_total_percentage(self):
+        """Validate that asset percentages sum to approximately 100."""
         total = sum(asset.percentage for asset in self.assets)
-
-        # Se il totale è 0, non possiamo normalizzare (evitiamo divisione per zero)
-        if total == 0:
-            # In questo caso estremo, assegniamo tutto al primo asset o lanciamo errore
-            # Qui scegliamo di distribuire equamente per non rompere l'app
-            share = 100.0 / len(self.assets)
-            for asset in self.assets:
-                asset.percentage = round(share, 2)
-            return self
-
-        # Se la somma è già corretta (con tolleranza), non facciamo nulla
-        if 99.0 <= total <= 101.0:
-            return self
-
-        # LOGICA DI NORMALIZZAZIONE
-        # Esempio: Se il totale è 60%, il fattore è 100/60 = 1.666...
-        factor = 100.0 / total
-
-        for asset in self.assets:
-            # Scaliamo ogni asset
-            asset.percentage = round(asset.percentage * factor, 2)
-
-        # Controllo finale per arrotondamenti (es. somma 99.99 o 100.01)
-        new_total = sum(asset.percentage for asset in self.assets)
-        diff = 100.0 - new_total
-
-        if diff != 0:
-            # Aggiungiamo/togliamo la piccola differenza al primo asset (spesso il più grande)
-            self.assets[0].percentage = round(self.assets[0].percentage + diff, 2)
-
+        if not (99 <= total <= 101):
+            raise ValueError(
+                f"Asset percentages must sum to 100%, got {total}% instead"
+            )
         return self