Review venal values per SIAT exercise

extractors/pdf_extractor.py

@@ -22,18 +22,18 @@ def anos_venais_ausentes(anos):
 
 
 def anos_venais_para_revisao(registros):
-    """Identifica anos repetidos ou com quantidade inesperada de valores monetários."""
+    """Identifica anos repetidos ou com mais de uma linha de lançamento venal."""
     contagem_por_ano = {}
-    anos_com_linhas_incorretas = set()
-    for ano, quantidade_valores, _ in registros:
+    anos_com_linhas_adicionais = set()
+    for ano, quantidade_linhas, _ in registros:
         contagem_por_ano[ano] = contagem_por_ano.get(ano, 0) + 1
-        if quantidade_valores != 3:
-            anos_com_linhas_incorretas.add(ano)
+        if quantidade_linhas > 1:
+            anos_com_linhas_adicionais.add(ano)
 
     return [
         int(ano)
         for ano in contagem_por_ano
-        if contagem_por_ano[ano] > 1 or ano in anos_com_linhas_incorretas
+        if contagem_por_ano[ano] > 1 or ano in anos_com_linhas_adicionais
     ]
 
 
@@ -175,7 +175,11 @@ def extrair_dados_pdf(pdf_file) -> Optional[Dict]:
             try:
                 ano = re.search(r'(\d{4})', m).group(1)
                 valores = re.findall(r'(\d{1,3}(?:\.\d{3})*,\d{2})', m)
-                registros_venais.append((ano, len(valores), valores[2] if len(valores) >= 3 else None))
+                # Cada lançamento traz seis valores monetários: construção, terreno,
+                # total, alíquota, IPTU e TCL/multa. Mais de seis indica outra linha
+                # de lançamento para o mesmo exercício e exige conferência manual.
+                linhas_venais = len(valores) // 6
+                registros_venais.append((ano, linhas_venais, valores[2] if len(valores) >= 3 else None))
             except (AttributeError, IndexError):
                 continue
 

frontend/src/App.jsx

@@ -182,7 +182,7 @@ function App() {
   const [databaseColumns, setDatabaseColumns] = useState([]);
 
   const [error, setError] = useState("");
-  const [pdfWarning, setPdfWarning] = useState("");
+  const [pdfWarnings, setPdfWarnings] = useState([]);
   const [result, setResult] = useState(null);
 
   useEffect(() => {
@@ -406,7 +406,7 @@ function App() {
 
     setLoadingPdf(true);
     setError("");
-    setPdfWarning("");
+    setPdfWarnings([]);
 
     try {
       const formData = new FormData();
@@ -439,12 +439,12 @@ function App() {
         avisos.push(
           `${anosRevisao.length === 1 ? "O ano" : "Os anos"} ${anosRevisao.join(", ")} ${
             anosRevisao.length === 1 ? "aparece" : "aparecem"
-          } mais de uma vez ou possui${anosRevisao.length === 1 ? "" : "em"} uma quantidade inesperada de valores venais. ` +
+          } mais de uma vez ou ${anosRevisao.length === 1 ? "possui" : "possuem"} mais de uma linha de valor venal. ` +
             "Foi inserido como XXX.XXX,XX para revisão.",
         );
       }
       if (avisos.length) {
-        setPdfWarning(`${avisos.join(" ")} Complete manualmente o campo “Valores Venais”.`);
+        setPdfWarnings(avisos.map((aviso) => `${aviso} Complete manualmente o campo “Valores Venais”.`));
       }
     } catch (err) {
       setError(String(err));
@@ -478,7 +478,7 @@ function App() {
     setForm(buildEmptyForm(options));
     setDatabaseColumns([]);
     setError("");
-    setPdfWarning("");
+    setPdfWarnings([]);
     setResult(null);
   };
 
@@ -787,7 +787,11 @@ function App() {
               </button>
             </div>
 
-            {pdfWarning && <div className="warning-box">{pdfWarning}</div>}
+            {pdfWarnings.map((warning) => (
+              <div className="warning-box" key={warning}>
+                {warning}
+              </div>
+            ))}
 
             <label>
               Inscrição

tests/test_pdf_extractor.py

@@ -44,11 +44,15 @@ Documento de Origem
     def test_marca_anos_repetidos_ou_com_linhas_extras_para_revisao(self):
         registros = [
             ("2023", 3, "300.000,00"),
-            ("2024", 4, "320.000,00"),
+            ("2024", 2, "320.000,00"),
             ("2023", 3, "300.000,00"),
         ]
         self.assertEqual(anos_venais_para_revisao(registros), [2023, 2024])
 
+    def test_uma_linha_venal_por_ano_nao_exige_revisao(self):
+        registros = [("2021", 1, "293.882,68"), ("2022", 1, "325.240,88")]
+        self.assertEqual(anos_venais_para_revisao(registros), [])
+
 
 if __name__ == "__main__":
     unittest.main()