fix: make legacy utilities safe by default

Author: sidineyr

.github/workflows/python-app.yml

@@ -1,40 +1,49 @@
-# Legacy PythonProjects workflow. SimpleDjIA has its own isolated compatibility matrix.
+# Curated checks for the maintained root exercises. Historical experiments and
+# standalone subprojects have their own validation paths.
 name: Python application
 
 on:
   push:
-    branches: [ "master" ]
-    paths-ignore:
-      - 'SimpleDjIA/**'
-      - '.github/workflows/simple-djia.yml'
-      - '.github/workflows/python-app.yml'
+    branches: ["master"]
+    paths:
+      - "music_composer/**"
+      - "tests/**"
+      - "calculador.py"
+      - "formatapendrivelinux.py"
+      - "requirements.txt"
+      - ".github/workflows/python-app.yml"
   pull_request:
-    branches: [ "master" ]
-    paths-ignore:
-      - 'SimpleDjIA/**'
-      - '.github/workflows/simple-djia.yml'
-      - '.github/workflows/python-app.yml'
+    branches: ["master"]
+    paths:
+      - "music_composer/**"
+      - "tests/**"
+      - "calculador.py"
+      - "formatapendrivelinux.py"
+      - "requirements.txt"
+      - ".github/workflows/python-app.yml"
 
 permissions:
   contents: read
 
 jobs:
   build:
     runs-on: ubuntu-latest
+    timeout-minutes: 10
     steps:
-    - uses: actions/checkout@v4
-    - name: Set up Python 3.10
-      uses: actions/setup-python@v3
-      with:
-        python-version: "3.10"
-    - name: Install dependencies
-      run: |
-        python -m pip install --upgrade pip
-        pip install flake8 pytest
-        if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
-    - name: Lint with flake8
-      run: |
-        flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
-        flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
-    - name: Test with pytest
-      run: pytest
+      - uses: actions/checkout@v4
+      - name: Set up Python 3.10
+        uses: actions/setup-python@v5
+        with:
+          python-version: "3.10"
+          cache: pip
+      - name: Install dependencies
+        run: |
+          python -m pip install --upgrade pip
+          python -m pip install -r requirements.txt
+          python -m pip install flake8
+      - name: Check maintained source
+        run: >-
+          flake8 music_composer tests calculador.py formatapendrivelinux.py
+          --count --select=E9,F63,F7,F82 --show-source --statistics
+      - name: Run maintained tests
+        run: python -m pytest -q tests

README.md

@@ -7,6 +7,15 @@ This initiative was dedicated to harnessing the power of Python programming and
 
 ---
 
+## Segurança dos experimentos
+
+Este repositório preserva exercícios produzidos em diferentes momentos de aprendizagem. Dois utilitários que exigem cuidado receberam proteções explícitas:
+
+- `calculador.py` aceita somente números, parênteses e as quatro operações exibidas; não executa texto como código;
+- `formatapendrivelinux.py` mostra apenas partições removíveis, graváveis e desmontadas, revalida o dispositivo e exige duas confirmações antes de apagar dados.
+
+Formatar uma partição continua sendo uma ação irreversível. Leia o caminho e o tamanho do dispositivo, mantenha cópias dos arquivos importantes e nunca confirme uma unidade que você não identificou fisicamente.
+
 ## Project Highlights  
 
 1. **Automation of Musical Structures**:  

calculador.py

@@ -1,52 +1,120 @@
+"""Calculadora gráfica com avaliação aritmética segura."""
+
+from __future__ import annotations
+
+import ast
+import math
+import operator
 import tkinter as tk
 
+
+_OPERADORES_BINARIOS = {
+    ast.Add: operator.add,
+    ast.Sub: operator.sub,
+    ast.Mult: operator.mul,
+    ast.Div: operator.truediv,
+}
+_OPERADORES_UNARIOS = {
+    ast.UAdd: operator.pos,
+    ast.USub: operator.neg,
+}
+
+
+def avaliar_expressao(expressao: str) -> int | float:
+    """Avalia somente números e os operadores exibidos pela calculadora."""
+    if not expressao or len(expressao) > 100:
+        raise ValueError("Expressão vazia ou longa demais")
+
+    try:
+        arvore = ast.parse(expressao, mode="eval")
+    except SyntaxError as erro:
+        raise ValueError("Expressão inválida") from erro
+
+    def avaliar(no: ast.AST, profundidade: int = 0) -> int | float:
+        if profundidade > 20:
+            raise ValueError("Expressão complexa demais")
+
+        if isinstance(no, ast.Expression):
+            return avaliar(no.body, profundidade + 1)
+
+        if isinstance(no, ast.Constant):
+            if isinstance(no.value, bool) or not isinstance(no.value, (int, float)):
+                raise ValueError("Somente números são permitidos")
+            return no.value
+
+        if isinstance(no, ast.BinOp) and type(no.op) in _OPERADORES_BINARIOS:
+            esquerda = avaliar(no.left, profundidade + 1)
+            direita = avaliar(no.right, profundidade + 1)
+            resultado = _OPERADORES_BINARIOS[type(no.op)](esquerda, direita)
+        elif isinstance(no, ast.UnaryOp) and type(no.op) in _OPERADORES_UNARIOS:
+            resultado = _OPERADORES_UNARIOS[type(no.op)](
+                avaliar(no.operand, profundidade + 1)
+            )
+        else:
+            raise ValueError("Operação não permitida")
+
+        if isinstance(resultado, float) and not math.isfinite(resultado):
+            raise ValueError("Resultado fora do intervalo permitido")
+        return resultado
+
+    return avaliar(arvore)
+
+
 class Calculadora(tk.Tk):
-    def __init__(self):
+    def __init__(self) -> None:
         super().__init__()
         self.title("Calculadora")
         self.geometry("300x400")
-
         self.resultado_var = tk.StringVar()
-        self.limpar_var = tk.StringVar()
+        self.criar_widgets()
 
-        self.create_widgets()
+    def criar_widgets(self) -> None:
+        entrada = tk.Entry(
+            self,
+            textvariable=self.resultado_var,
+            font=("Arial", 24),
+            bd=10,
+            relief=tk.GROOVE,
+            justify=tk.RIGHT,
+        )
+        entrada.grid(row=0, column=0, columnspan=4)
 
-    def create_widgets(self):
-        # Entry para exibir os resultados
-        entry = tk.Entry(self, textvariable=self.resultado_var, font=('Arial', 24), bd=10, relief=tk.GROOVE, justify=tk.RIGHT)
-        entry.grid(row=0, column=0, columnspan=4)
-
-        # Botões
         botoes = [
-            '7', '8', '9', '/',
-            '4', '5', '6', '*',
-            '1', '2', '3', '-',
-            '0', '.', '=', '+',
-            'C'  # Botão para limpar
+            "7", "8", "9", "/",
+            "4", "5", "6", "*",
+            "1", "2", "3", "-",
+            "0", ".", "=", "+",
+            "C",
         ]
 
-        row_val = 1
-        col_val = 0
-
-        for botao in botoes:
-            tk.Button(self, text=botao, padx=20, pady=20, font=('Arial', 18), bd=8, relief=tk.RIDGE, command=lambda b=botao: self.botao_click(b)).grid(row=row_val, column=col_val)
-            col_val += 1
-            if col_val > 3:
-                col_val = 0
-                row_val += 1
+        linha = 1
+        coluna = 0
+        for rotulo in botoes:
+            tk.Button(
+                self,
+                text=rotulo,
+                padx=20,
+                pady=20,
+                font=("Arial", 18),
+                bd=8,
+                relief=tk.RIDGE,
+                command=lambda valor=rotulo: self.botao_click(valor),
+            ).grid(row=linha, column=coluna)
+            coluna += 1
+            if coluna > 3:
+                coluna = 0
+                linha += 1
 
-    def botao_click(self, valor):
-        if valor == '=':
+    def botao_click(self, valor: str) -> None:
+        if valor == "=":
             try:
-                resultado = eval(self.resultado_var.get())
-                self.resultado_var.set(resultado)
-            except Exception as e:
+                self.resultado_var.set(avaliar_expressao(self.resultado_var.get()))
+            except (ArithmeticError, TypeError, ValueError):
                 self.resultado_var.set("Erro")
-        elif valor == 'C':
+        elif valor == "C":
             self.resultado_var.set("")
         else:
-            current_text = self.resultado_var.get()
-            self.resultado_var.set(current_text + valor)
+            self.resultado_var.set(self.resultado_var.get() + valor)
 
 
 if __name__ == "__main__":