airev 1.1.1__tar.gz → 1.3.0__tar.gz

{airev-1.1.1 → airev-1.3.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: airev
-Version: 1.1.1
+Version: 1.3.0
 Summary: CLI tool for AI-powered code review with context backtracking
 Author-email: Tarcisio <tarcisiojunior@gmail.com>
 License: MIT

{airev-1.1.1 → airev-1.3.0}/src/airev.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: airev
-Version: 1.1.1
+Version: 1.3.0
 Summary: CLI tool for AI-powered code review with context backtracking
 Author-email: Tarcisio <tarcisiojunior@gmail.com>
 License: MIT

{airev-1.1.1 → airev-1.3.0}/src/airev.egg-info/SOURCES.txt

@@ -29,6 +29,7 @@ src/code_reviewer/updater/http_client.py
 src/code_reviewer/updater/notifier.py
 src/code_reviewer/updater/upgrade.py
 src/code_reviewer/updater/version_check.py
+tests/test_cli.py
 tests/test_context_builder.py
 tests/test_copilot_runner.py
 tests/test_diff_parser.py

{airev-1.1.1 → airev-1.3.0}/src/code_reviewer/__init__.py

@@ -1,3 +1,3 @@
 """Code Reviewer - CLI para revisão de código com IA."""
 
-__version__ = "1.1.1"
+__version__ = "1.3.0"

{airev-1.1.1 → airev-1.3.0}/src/code_reviewer/cli.py

@@ -1,4 +1,4 @@
-"""CLI Entry Point - Comando principal do code-reviewer."""
+"""CLI Entry Point - Comando principal do airev."""
 
 import sys
 import time
@@ -75,6 +75,13 @@ def main():
     type=click.Choice(get_available_languages()),
     help="Idioma das mensagens (default: pt-br)",
 )
+@click.option(
+    "--text-quality",
+    "-t",
+    is_flag=True,
+    default=False,
+    help="Ativa verificação de ortografia e clareza em mensagens de usuário",
+)
 def review(
     base: str,
     runner: str,
@@ -83,18 +90,21 @@ def review(
     no_progress: bool,
     progress: bool,
     lang: str,
+    text_quality: bool,
 ):
     """Analisa o diff da branch atual contra a branch base.
 
     Exemplos:
 
-        code-reviewer review --base main
+        airev review --base main
+
+        airev review --base develop --runner copilot
 
-        code-reviewer review --base develop --runner copilot
+        airev review --base main --json-output
 
-        code-reviewer review --base main --json-output
+        airev review --base main --no-progress
 
-        code-reviewer review --base main --no-progress
+        airev review --base main --text-quality
     """
     workdir = workdir or Path.cwd()
     start_time = time.perf_counter()
@@ -173,7 +183,9 @@ def review(
 
     # Monta o prompt
     with reporter.status(t("cli.building_prompt")):
-        prompt = build_prompt(diff_files, context_graphs, current_branch, base)
+        prompt = build_prompt(
+            diff_files, context_graphs, current_branch, base, text_quality=text_quality
+        )
 
     # Obtém o runner
     try:
@@ -232,7 +244,7 @@ def runners():
 
 @main.command()
 def upgrade():
-    """Atualiza o code-reviewer para a versão mais recente."""
+    """Atualiza o airev para a versão mais recente."""
     from rich.console import Console
 
     console = Console()

{airev-1.1.1 → airev-1.3.0}/src/code_reviewer/formatters/terminal.py

@@ -5,7 +5,7 @@ from typing import TextIO
 import sys
 
 from ..i18n import t
-from ..models import Finding, ReviewResult, Severity
+from ..models import Category, Finding, ReviewResult, Severity
 
 
 # Códigos ANSI para cores
@@ -71,6 +71,28 @@ def format_severity(severity: Severity) -> str:
         return _colorize(f"[{severity.value}]", Colors.BOLD, Colors.BLUE)
 
 
+def format_category_badge(category: Category) -> str:
+    """Formata o badge da categoria.
+
+    Args:
+        category: Categoria do finding
+
+    Returns:
+        String formatada com ícone e cor para a categoria
+    """
+    # Ícones e cores por categoria
+    category_styles = {
+        Category.SECURITY: ("🔒", Colors.RED),
+        Category.PERFORMANCE: ("⚡", Colors.YELLOW),
+        Category.BUG: ("🐛", Colors.MAGENTA),
+        Category.RESOURCE_LEAK: ("💧", Colors.CYAN),
+        Category.TEXT_QUALITY: ("✏️", Colors.CYAN),
+    }
+
+    icon, color = category_styles.get(category, ("•", Colors.WHITE))
+    return f"{icon} {_colorize(category.value, color)}"
+
+
 def format_finding(finding: Finding) -> str:
     """Formata um finding para exibição.
 
@@ -84,10 +106,11 @@ def format_finding(finding: Finding) -> str:
 
     # Header: [SEVERITY] arquivo:linha - Título
     severity_str = format_severity(finding.severity)
+    category_badge = format_category_badge(finding.category)
     location = _colorize(f"{finding.file}:{finding.line}", Colors.CYAN)
     title = _colorize(finding.title, Colors.BOLD)
 
-    lines.append(f"  {severity_str} {location} - {title}")
+    lines.append(f"  {severity_str} {category_badge} {location} - {title}")
 
     # Descrição
     if finding.description:

{airev-1.1.1 → airev-1.3.0}/src/code_reviewer/locales/en.yaml

@@ -47,6 +47,13 @@ terminal:
   # Findings
   suggestion: "Suggestion:"
 
+  # Categories
+  category_security: "security"
+  category_performance: "performance"
+  category_bug: "bug"
+  category_resource_leak: "resource leak"
+  category_text_quality: "text quality"
+
   # Summary
   summary: "SUMMARY:"
   findings_count: "{count} finding(s)"

{airev-1.1.1 → airev-1.3.0}/src/code_reviewer/locales/pt-br.yaml

@@ -47,6 +47,13 @@ terminal:
   # Findings
   suggestion: "Sugestão:"
 
+  # Categorias
+  category_security: "segurança"
+  category_performance: "performance"
+  category_bug: "bug"
+  category_resource_leak: "vazamento de recurso"
+  category_text_quality: "qualidade de texto"
+
   # Resumo
   summary: "RESUMO:"
   findings_count: "{count} finding(s)"

{airev-1.1.1 → airev-1.3.0}/src/code_reviewer/models.py

@@ -21,6 +21,7 @@ class Category(str, Enum):
     PERFORMANCE = "performance"
     BUG = "bug"
     RESOURCE_LEAK = "resource-leak"
+    TEXT_QUALITY = "text-quality"
 
 
 class DiffLine(BaseModel):
@@ -90,7 +91,7 @@ class Finding(BaseModel):
     line: int = Field(description="Número da linha")
     severity: Severity = Field(description="Severidade: CRITICAL, WARNING, INFO")
     category: Category = Field(
-        description="Categoria: security, performance, bug, resource-leak"
+        description="Categoria: security, performance, bug, resource-leak, text-quality"
     )
     title: str = Field(description="Título curto do problema")
     description: str = Field(description="Descrição detalhada do problema")

{airev-1.1.1 → airev-1.3.0}/src/code_reviewer/prompt_builder.py

@@ -152,11 +152,54 @@ def format_references_for_prompt(context_graphs: list[ContextGraph]) -> str:
     return "\n".join(parts)
 
 
+def get_text_quality_section(language_name: str) -> str:
+    """Retorna a seção de instruções para verificação de qualidade de texto.
+
+    Args:
+        language_name: Nome do idioma para verificação
+
+    Returns:
+        String com instruções de verificação de texto
+    """
+    return f"""
+## QUALIDADE DE TEXTO
+
+Verifique ortografia e clareza semântica em mensagens voltadas ao usuário, no idioma **{language_name}**.
+
+### O que verificar:
+
+**Padrões de código:**
+- `raise *Error("...")` e `raise *Exception("...")`
+- `print("...")` e `console.log("...")`
+- Parâmetros nomeados: `message=`, `label=`, `title=`, `description=`, `text=`
+- Funções de UI: `flash("...")`, `toast("...")`, `alert("...")`
+
+**Arquivos de i18n:**
+- Arquivos em `locales/**/*`
+- Arquivos em `i18n/**/*`
+- Arquivos `messages.*` e `strings.*`
+
+### O que ignorar:
+
+- Identificadores: snake_case, camelCase, PascalCase
+- Termos técnicos: HTTP, JSON, API, SQL, URL, etc.
+- Nomes próprios e termos de domínio específico
+- Chaves de configuração e variáveis de ambiente
+
+### Formato dos findings:
+
+- Categoria: `text-quality`
+- Severidade: sempre `INFO`
+- Inclua a correção sugerida no campo `suggestion`
+"""
+
+
 def build_prompt(
     diff_files: list[DiffFile],
     context_graphs: list[ContextGraph],
     branch: str,
     base: str,
+    text_quality: bool = False,
 ) -> str:
     """Monta o prompt completo para a IA.
 
@@ -165,6 +208,7 @@ def build_prompt(
         context_graphs: Grafos de contexto com backtracking
         branch: Nome da branch sendo analisada
         base: Nome da branch base
+        text_quality: Se True, inclui verificação de ortografia e clareza
 
     Returns:
         Prompt completo pronto para enviar à IA
@@ -183,11 +227,15 @@ def build_prompt(
     lang_code = get_language()
     language_name = LANGUAGE_NAMES.get(lang_code, lang_code)
 
+    # Seção de text-quality (condicional)
+    text_quality_section = get_text_quality_section(language_name) if text_quality else ""
+
     # Substitui placeholders
     prompt = template.replace("{diff}", diff_section)
     prompt = prompt.replace("{context}", context_section)
     prompt = prompt.replace("{references}", references_section)
     prompt = prompt.replace("{json_schema}", json_schema)
     prompt = prompt.replace("{language}", language_name)
+    prompt = prompt.replace("{text_quality_section}", text_quality_section)
 
     return prompt

{airev-1.1.1 → airev-1.3.0}/src/code_reviewer/prompts/review_system.md

@@ -25,7 +25,7 @@ O DIFF abaixo mostra APENAS as linhas alteradas, não o arquivo completo. Códig
 - **Performance**: N+1 queries, loops desnecessários, operações O(n²), falta de índices, carregamento excessivo
 - **Bugs potenciais**: null pointer, race conditions, off-by-one, divisão por zero, exceções não tratadas
 - **Recursos não fechados**: conexões de banco, arquivos, sockets, locks não liberados
-
+{text_quality_section}
 ## FORMATO DE SAÍDA
 
 Retorne APENAS um JSON válido no formato abaixo. Não inclua explicações fora do JSON.

{airev-1.1.1 → airev-1.3.0}/src/code_reviewer/response_parser.py

@@ -121,6 +121,11 @@ def normalize_category(value: str) -> Category:
         "resource_leak": Category.RESOURCE_LEAK,
         "leak": Category.RESOURCE_LEAK,
         "memory": Category.RESOURCE_LEAK,
+        "text-quality": Category.TEXT_QUALITY,
+        "text_quality": Category.TEXT_QUALITY,
+        "spelling": Category.TEXT_QUALITY,
+        "grammar": Category.TEXT_QUALITY,
+        "typo": Category.TEXT_QUALITY,
     }
 
     return mapping.get(value_lower, Category.BUG)

{airev-1.1.1 → airev-1.3.0}/src/code_reviewer/updater/__init__.py

@@ -1,11 +1,12 @@
 """Módulo de auto-update para verificação e atualização de versões."""
 
-from .version_check import check_for_update, UpdateInfo
+from .version_check import check_for_update, clear_cache, UpdateInfo
 from .notifier import notify_update
 from .upgrade import detect_installer, run_upgrade
 
 __all__ = [
     "check_for_update",
+    "clear_cache",
     "UpdateInfo",
     "notify_update",
     "detect_installer",

{airev-1.1.1 → airev-1.3.0}/src/code_reviewer/updater/http_client.py

@@ -30,7 +30,7 @@ class UrllibClient:
         try:
             request = urllib.request.Request(
                 url,
-                headers={"Accept": "application/json", "User-Agent": "code-reviewer"},
+                headers={"Accept": "application/json", "User-Agent": "airev"},
             )
             with urllib.request.urlopen(request, timeout=timeout) as response:
                 data = response.read().decode("utf-8")

airev-1.3.0/src/code_reviewer/updater/upgrade.py

@@ -0,0 +1,182 @@
+"""Funcionalidades de upgrade do pacote."""
+
+import json
+import subprocess
+import sys
+from pathlib import Path
+
+from rich.console import Console
+
+from .. import __version__
+from .version_check import check_for_update, clear_cache
+
+
+def detect_installer() -> str:
+    """Detecta qual instalador foi usado (pipx ou pip).
+
+    Returns:
+        "pipx" se instalado via pipx, "pip" caso contrário
+    """
+    exe_path = Path(sys.executable)
+
+    # pipx instala em ~/.local/pipx/venvs/
+    if "pipx" in exe_path.parts:
+        return "pipx"
+
+    return "pip"
+
+
+def get_installed_version(installer: str) -> str | None:
+    """Obtém a versão real instalada do airev consultando o instalador.
+
+    Args:
+        installer: "pipx" ou "pip"
+
+    Returns:
+        String com a versão instalada ou None se não conseguir obter
+    """
+    try:
+        if installer == "pipx":
+            # Tenta usar pipx list --json
+            result = subprocess.run(
+                ["pipx", "list", "--json"],
+                capture_output=True,
+                text=True,
+                check=False,
+            )
+            if result.returncode == 0:
+                data = json.loads(result.stdout)
+                venvs = data.get("venvs", {})
+                if "airev" in venvs:
+                    metadata = venvs["airev"].get("metadata", {})
+                    return metadata.get("main_package", {}).get("package_version")
+            # Fallback: tenta sem --json (versões antigas do pipx)
+            return _get_version_from_pipx_list_text()
+        else:
+            # pip show airev
+            result = subprocess.run(
+                [sys.executable, "-m", "pip", "show", "airev"],
+                capture_output=True,
+                text=True,
+                check=False,
+            )
+            if result.returncode == 0:
+                for line in result.stdout.splitlines():
+                    if line.startswith("Version:"):
+                        return line.split(":", 1)[1].strip()
+    except (subprocess.SubprocessError, json.JSONDecodeError, KeyError):
+        pass
+    return None
+
+
+def _get_version_from_pipx_list_text() -> str | None:
+    """Fallback para obter versão do pipx list sem --json.
+
+    Returns:
+        String com a versão ou None se não encontrar
+    """
+    try:
+        result = subprocess.run(
+            ["pipx", "list"],
+            capture_output=True,
+            text=True,
+            check=False,
+        )
+        if result.returncode == 0:
+            # Procura por linha como "airev 1.0.0" ou "package airev 1.0.0"
+            for line in result.stdout.splitlines():
+                if "airev" in line.lower():
+                    parts = line.split()
+                    for i, part in enumerate(parts):
+                        if "airev" in part.lower() and i + 1 < len(parts):
+                            version = parts[i + 1].strip(",")
+                            # Verifica se parece uma versão
+                            if version and version[0].isdigit():
+                                return version
+    except subprocess.SubprocessError:
+        pass
+    return None
+
+
+def run_upgrade(console: Console | None = None) -> bool:
+    """Executa upgrade do pacote.
+
+    Args:
+        console: Console Rich para output (opcional)
+
+    Returns:
+        True se upgrade foi executado com sucesso, False caso contrário
+    """
+    if console is None:
+        console = Console()
+
+    # Verifica se há atualização disponível
+    console.print("[dim]Verificando atualizações...[/]")
+    update_info = check_for_update()
+
+    if update_info is None:
+        # Limpa cache mesmo quando não há atualização (garante consistência)
+        clear_cache()
+        console.print(
+            f"[green]✓[/] Você já está na versão mais recente ([cyan]{__version__}[/])"
+        )
+        return False
+
+    installer = detect_installer()
+
+    # Obtém versão antes do upgrade para comparação
+    version_before = get_installed_version(installer) or __version__
+
+    console.print(
+        f"[yellow]→[/] Atualizando de {version_before} "
+        f"para {update_info.latest_version}..."
+    )
+
+    # Monta comando de upgrade
+    if installer == "pipx":
+        cmd = ["pipx", "upgrade", "airev"]
+    else:
+        cmd = [sys.executable, "-m", "pip", "install", "--upgrade", "airev"]
+
+    try:
+        result = subprocess.run(
+            cmd,
+            capture_output=True,
+            text=True,
+            check=False,
+        )
+
+        # Sempre limpa o cache após tentativa de upgrade
+        clear_cache()
+
+        if result.returncode == 0:
+            # Verifica se a versão realmente mudou
+            version_after = get_installed_version(installer)
+
+            if version_after and version_after != version_before:
+                console.print(
+                    f"[green]✓[/] Atualizado de {version_before} para {version_after}"
+                )
+                return True
+            else:
+                # Comando executou mas versão não mudou
+                console.print(
+                    f"[green]✓[/] Você já está na versão mais recente "
+                    f"([cyan]{version_after or version_before}[/])"
+                )
+                return False
+        else:
+            console.print(f"[red]✗[/] Falha ao atualizar: {result.stderr.strip()}")
+            console.print(f"[dim]Tente manualmente: {' '.join(cmd)}[/]")
+            return False
+
+    except FileNotFoundError:
+        clear_cache()
+        console.print(f"[red]✗[/] Comando '{cmd[0]}' não encontrado")
+        if installer == "pipx":
+            console.print("[dim]Tente: pip install --upgrade airev[/]")
+        return False
+    except Exception as e:
+        clear_cache()
+        console.print(f"[red]✗[/] Erro inesperado: {e}")
+        return False

{airev-1.1.1 → airev-1.3.0}/src/code_reviewer/updater/version_check.py

@@ -110,6 +110,24 @@ def _write_cache(latest_version: str) -> None:
         pass
 
 
+def clear_cache() -> bool:
+    """Remove o arquivo de cache de verificação de versão.
+
+    Deve ser chamada após upgrade para garantir que a próxima
+    verificação consulte o PyPI novamente.
+
+    Returns:
+        True se o cache foi removido ou não existia, False se falhou
+    """
+    try:
+        if CACHE_FILE.exists():
+            CACHE_FILE.unlink()
+        return True
+    except OSError:
+        # Falha ao remover cache - ignora silenciosamente
+        return False
+
+
 def get_latest_version(timeout: float = DEFAULT_TIMEOUT) -> str | None:
     """Consulta PyPI para obter a versão mais recente.
 

airev-1.3.0/tests/test_cli.py

@@ -0,0 +1,29 @@
+"""Testes para o CLI."""
+
+from click.testing import CliRunner
+
+from code_reviewer.cli import review
+
+
+class TestReviewCommand:
+    """Testes para o comando review."""
+
+    def test_flag_text_quality_reconhecida(self):
+        """Verifica que a flag --text-quality é aceita pelo CLI."""
+        runner = CliRunner()
+
+        # Invoca com --help para verificar que a flag existe
+        result = runner.invoke(review, ["--help"])
+
+        assert result.exit_code == 0
+        assert "--text-quality" in result.output
+        assert "ortografia" in result.output or "spelling" in result.output.lower()
+
+    def test_flag_text_quality_short_form(self):
+        """Verifica que a forma curta -t funciona."""
+        runner = CliRunner()
+
+        result = runner.invoke(review, ["--help"])
+
+        assert result.exit_code == 0
+        assert "-t" in result.output

{airev-1.1.1 → airev-1.3.0}/tests/test_prompt_builder.py

@@ -188,3 +188,70 @@ class TestBuildPrompt:
         assert "REGRAS" in prompt
         assert "test.py" in prompt
         assert "review" in prompt  # Schema JSON
+
+    def test_prompt_sem_text_quality_por_padrao(self):
+        """Verifica que seção text-quality não aparece quando flag desativada."""
+        diff_files = [
+            DiffFile(
+                path="test.py",
+                hunks=[
+                    DiffHunk(
+                        function_name="test",
+                        start_line_old=1,
+                        start_line_new=1,
+                        added_lines=[],
+                        removed_lines=[],
+                    )
+                ],
+            )
+        ]
+        context_graphs = []
+
+        prompt = build_prompt(diff_files, context_graphs, "feature/test", "main")
+
+        # Não deve conter seção de text-quality
+        assert "QUALIDADE DE TEXTO" not in prompt
+        assert "text-quality" not in prompt
+
+    def test_prompt_com_text_quality_ativo(self):
+        """Verifica que seção text-quality aparece quando flag ativada."""
+        diff_files = [
+            DiffFile(
+                path="test.py",
+                hunks=[
+                    DiffHunk(
+                        function_name="test",
+                        start_line_old=1,
+                        start_line_new=1,
+                        added_lines=[],
+                        removed_lines=[],
+                    )
+                ],
+            )
+        ]
+        context_graphs = []
+
+        prompt = build_prompt(
+            diff_files, context_graphs, "feature/test", "main", text_quality=True
+        )
+
+        # Deve conter seção de text-quality
+        assert "QUALIDADE DE TEXTO" in prompt
+        assert "text-quality" in prompt
+        assert "ortografia" in prompt
+        assert "raise *Error" in prompt
+        assert "locales/" in prompt
+
+    def test_prompt_text_quality_menciona_ignorar_identificadores(self):
+        """Verifica que instruções de exclusão estão presentes."""
+        diff_files = []
+        context_graphs = []
+
+        prompt = build_prompt(
+            diff_files, context_graphs, "feature/test", "main", text_quality=True
+        )
+
+        # Deve instruir a ignorar identificadores
+        assert "snake_case" in prompt
+        assert "camelCase" in prompt
+        assert "termos técnicos" in prompt or "Termos técnicos" in prompt

{airev-1.1.1 → airev-1.3.0}/tests/test_response_parser.py

@@ -117,6 +117,13 @@ class TestNormalizeCategory:
         assert normalize_category("resource-leak") == Category.RESOURCE_LEAK
         assert normalize_category("memory") == Category.RESOURCE_LEAK
 
+    def test_text_quality(self):
+        assert normalize_category("text-quality") == Category.TEXT_QUALITY
+        assert normalize_category("text_quality") == Category.TEXT_QUALITY
+        assert normalize_category("spelling") == Category.TEXT_QUALITY
+        assert normalize_category("grammar") == Category.TEXT_QUALITY
+        assert normalize_category("typo") == Category.TEXT_QUALITY
+
     def test_desconhecido_para_bug(self):
         assert normalize_category("unknown") == Category.BUG
 

{airev-1.1.1 → airev-1.3.0}/tests/test_updater.py

@@ -1,6 +1,7 @@
 """Testes para o módulo de auto-update."""
 
 import json
+import subprocess
 import sys
 from datetime import datetime, timedelta, timezone
 from io import StringIO
@@ -17,11 +18,16 @@ from code_reviewer.updater.version_check import (
     _read_cache,
     _write_cache,
     check_for_update,
+    clear_cache,
     compare_versions,
     get_latest_version,
 )
 from code_reviewer.updater.notifier import notify_update
-from code_reviewer.updater.upgrade import detect_installer, run_upgrade
+from code_reviewer.updater.upgrade import (
+    detect_installer,
+    get_installed_version,
+    run_upgrade,
+)
 
 
 class TestHttpClient:
@@ -159,6 +165,48 @@ class TestCache:
         assert cache is None
 
 
+class TestClearCache:
+    """Testes para clear_cache."""
+
+    def test_clear_existing_cache(self, tmp_path):
+        """Deve remover arquivo de cache existente."""
+        cache_file = tmp_path / "update-check.json"
+        cache_file.write_text('{"test": true}')
+
+        with patch(
+            "code_reviewer.updater.version_check.CACHE_FILE", cache_file
+        ):
+            result = clear_cache()
+
+        assert result is True
+        assert not cache_file.exists()
+
+    def test_clear_nonexistent_cache(self, tmp_path):
+        """Deve retornar True quando cache não existe."""
+        cache_file = tmp_path / "nonexistent.json"
+
+        with patch(
+            "code_reviewer.updater.version_check.CACHE_FILE", cache_file
+        ):
+            result = clear_cache()
+
+        assert result is True
+
+    def test_clear_cache_permission_error(self, tmp_path):
+        """Deve retornar False quando não consegue remover."""
+        cache_file = tmp_path / "update-check.json"
+        cache_file.write_text('{"test": true}')
+
+        with patch(
+            "code_reviewer.updater.version_check.CACHE_FILE", cache_file
+        ), patch(
+            "pathlib.Path.unlink", side_effect=OSError("Permission denied")
+        ):
+            result = clear_cache()
+
+        assert result is False
+
+
 class TestCheckForUpdate:
     """Testes para check_for_update."""
 
@@ -252,7 +300,7 @@ class TestDetectInstaller:
 
     def test_detect_pipx(self):
         """Deve detectar pipx quando no path do executável."""
-        fake_path = Path("/home/user/.local/pipx/venvs/code-reviewer/bin/python")
+        fake_path = Path("/home/user/.local/pipx/venvs/airev/bin/python")
         with patch.object(sys, "executable", str(fake_path)):
             result = detect_installer()
             assert result == "pipx"
@@ -272,59 +320,167 @@ class TestDetectInstaller:
             assert result == "pip"
 
 
+class TestGetInstalledVersion:
+    """Testes para get_installed_version."""
+
+    def test_get_version_pipx_json(self):
+        """Deve obter versão do pipx list --json."""
+        pipx_json = {
+            "venvs": {
+                "airev": {
+                    "metadata": {
+                        "main_package": {
+                            "package_version": "1.2.3"
+                        }
+                    }
+                }
+            }
+        }
+        mock_result = MagicMock(returncode=0, stdout=json.dumps(pipx_json))
+
+        with patch("subprocess.run", return_value=mock_result):
+            result = get_installed_version("pipx")
+
+        assert result == "1.2.3"
+
+    def test_get_version_pipx_not_installed(self):
+        """Deve retornar None quando airev não está instalado no pipx."""
+        pipx_json = {"venvs": {}}
+        mock_result = MagicMock(returncode=0, stdout=json.dumps(pipx_json))
+
+        with patch("subprocess.run", return_value=mock_result):
+            result = get_installed_version("pipx")
+
+        assert result is None
+
+    def test_get_version_pip(self):
+        """Deve obter versão do pip show."""
+        pip_output = """Name: airev
+Version: 1.0.0
+Summary: Code reviewer CLI
+"""
+        mock_result = MagicMock(returncode=0, stdout=pip_output)
+
+        with patch("subprocess.run", return_value=mock_result):
+            result = get_installed_version("pip")
+
+        assert result == "1.0.0"
+
+    def test_get_version_pip_not_installed(self):
+        """Deve retornar None quando pip show falha."""
+        mock_result = MagicMock(returncode=1, stdout="")
+
+        with patch("subprocess.run", return_value=mock_result):
+            result = get_installed_version("pip")
+
+        assert result is None
+
+    def test_get_version_subprocess_error(self):
+        """Deve retornar None em erro de subprocess."""
+        with patch("subprocess.run", side_effect=subprocess.SubprocessError):
+            result = get_installed_version("pipx")
+
+        assert result is None
+
+
 class TestRunUpgrade:
     """Testes para run_upgrade."""
 
     def test_upgrade_no_update_available(self):
-        """Deve retornar False quando não há update."""
+        """Deve retornar False e limpar cache quando não há update."""
         from rich.console import Console
 
         console = Console(file=StringIO(), force_terminal=True)
 
         with patch(
             "code_reviewer.updater.upgrade.check_for_update", return_value=None
-        ):
+        ), patch(
+            "code_reviewer.updater.upgrade.clear_cache", return_value=True
+        ) as mock_clear:
             result = run_upgrade(console)
 
             assert result is False
+            mock_clear.assert_called_once()
 
-    def test_upgrade_success_pipx(self):
-        """Deve executar pipx upgrade com sucesso."""
+    def test_upgrade_success_pipx_version_changed(self):
+        """Deve executar pipx upgrade e verificar mudança de versão."""
         from rich.console import Console
 
         console = Console(file=StringIO(), force_terminal=True)
         update_info = UpdateInfo(current_version="0.1.0", latest_version="0.2.0")
         mock_result = MagicMock(returncode=0, stderr="")
 
+        # Simula versão mudando de 0.1.0 para 0.2.0 após upgrade
+        version_calls = iter(["0.1.0", "0.2.0"])
+
         with patch(
             "code_reviewer.updater.upgrade.check_for_update",
             return_value=update_info,
         ), patch(
             "code_reviewer.updater.upgrade.detect_installer", return_value="pipx"
         ), patch(
+            "code_reviewer.updater.upgrade.get_installed_version",
+            side_effect=lambda _: next(version_calls),
+        ), patch(
+            "code_reviewer.updater.upgrade.clear_cache", return_value=True
+        ) as mock_clear, patch(
             "subprocess.run", return_value=mock_result
         ) as mock_run:
             result = run_upgrade(console)
 
             assert result is True
             mock_run.assert_called_once()
+            mock_clear.assert_called_once()
             cmd = mock_run.call_args[0][0]
             assert "pipx" in cmd
             assert "upgrade" in cmd
 
-    def test_upgrade_success_pip(self):
-        """Deve executar pip install --upgrade com sucesso."""
+    def test_upgrade_success_but_version_unchanged(self):
+        """Deve retornar False quando versão não muda após upgrade."""
+        from rich.console import Console
+
+        console = Console(file=StringIO(), force_terminal=True)
+        update_info = UpdateInfo(current_version="0.1.0", latest_version="0.2.0")
+        mock_result = MagicMock(returncode=0, stderr="")
+
+        with patch(
+            "code_reviewer.updater.upgrade.check_for_update",
+            return_value=update_info,
+        ), patch(
+            "code_reviewer.updater.upgrade.detect_installer", return_value="pipx"
+        ), patch(
+            "code_reviewer.updater.upgrade.get_installed_version",
+            return_value="0.1.0",  # Versão não muda
+        ), patch(
+            "code_reviewer.updater.upgrade.clear_cache", return_value=True
+        ) as mock_clear, patch(
+            "subprocess.run", return_value=mock_result
+        ):
+            result = run_upgrade(console)
+
+            assert result is False
+            mock_clear.assert_called_once()
+
+    def test_upgrade_success_pip_version_changed(self):
+        """Deve executar pip install --upgrade e verificar mudança."""
         from rich.console import Console
 
         console = Console(file=StringIO(), force_terminal=True)
         update_info = UpdateInfo(current_version="0.1.0", latest_version="0.2.0")
         mock_result = MagicMock(returncode=0, stderr="")
 
+        version_calls = iter(["0.1.0", "0.2.0"])
+
         with patch(
             "code_reviewer.updater.upgrade.check_for_update",
             return_value=update_info,
         ), patch(
             "code_reviewer.updater.upgrade.detect_installer", return_value="pip"
+        ), patch(
+            "code_reviewer.updater.upgrade.get_installed_version",
+            side_effect=lambda _: next(version_calls),
+        ), patch(
+            "code_reviewer.updater.upgrade.clear_cache", return_value=True
         ), patch(
             "subprocess.run", return_value=mock_result
         ) as mock_run:
@@ -335,3 +491,54 @@ class TestRunUpgrade:
             cmd = mock_run.call_args[0][0]
             assert "--upgrade" in cmd
             assert "airev" in cmd
+
+    def test_upgrade_clears_cache_on_failure(self):
+        """Deve limpar cache mesmo quando upgrade falha."""
+        from rich.console import Console
+
+        console = Console(file=StringIO(), force_terminal=True)
+        update_info = UpdateInfo(current_version="0.1.0", latest_version="0.2.0")
+        mock_result = MagicMock(returncode=1, stderr="Error")
+
+        with patch(
+            "code_reviewer.updater.upgrade.check_for_update",
+            return_value=update_info,
+        ), patch(
+            "code_reviewer.updater.upgrade.detect_installer", return_value="pipx"
+        ), patch(
+            "code_reviewer.updater.upgrade.get_installed_version",
+            return_value="0.1.0",
+        ), patch(
+            "code_reviewer.updater.upgrade.clear_cache", return_value=True
+        ) as mock_clear, patch(
+            "subprocess.run", return_value=mock_result
+        ):
+            result = run_upgrade(console)
+
+            assert result is False
+            mock_clear.assert_called_once()
+
+    def test_upgrade_clears_cache_on_exception(self):
+        """Deve limpar cache quando ocorre exceção."""
+        from rich.console import Console
+
+        console = Console(file=StringIO(), force_terminal=True)
+        update_info = UpdateInfo(current_version="0.1.0", latest_version="0.2.0")
+
+        with patch(
+            "code_reviewer.updater.upgrade.check_for_update",
+            return_value=update_info,
+        ), patch(
+            "code_reviewer.updater.upgrade.detect_installer", return_value="pipx"
+        ), patch(
+            "code_reviewer.updater.upgrade.get_installed_version",
+            return_value="0.1.0",
+        ), patch(
+            "code_reviewer.updater.upgrade.clear_cache", return_value=True
+        ) as mock_clear, patch(
+            "subprocess.run", side_effect=FileNotFoundError
+        ):
+            result = run_upgrade(console)
+
+            assert result is False
+            mock_clear.assert_called_once()

airev-1.1.1/src/code_reviewer/updater/upgrade.py

@@ -1,87 +0,0 @@
-"""Funcionalidades de upgrade do pacote."""
-
-import subprocess
-import sys
-from pathlib import Path
-
-from rich.console import Console
-
-from .. import __version__
-from .version_check import check_for_update
-
-
-def detect_installer() -> str:
-    """Detecta qual instalador foi usado (pipx ou pip).
-
-    Returns:
-        "pipx" se instalado via pipx, "pip" caso contrário
-    """
-    exe_path = Path(sys.executable)
-
-    # pipx instala em ~/.local/pipx/venvs/
-    if "pipx" in exe_path.parts:
-        return "pipx"
-
-    return "pip"
-
-
-def run_upgrade(console: Console | None = None) -> bool:
-    """Executa upgrade do pacote.
-
-    Args:
-        console: Console Rich para output (opcional)
-
-    Returns:
-        True se upgrade foi executado, False se já está atualizado ou falhou
-    """
-    if console is None:
-        console = Console()
-
-    # Verifica se há atualização disponível
-    console.print("[dim]Verificando atualizações...[/]")
-    update_info = check_for_update()
-
-    if update_info is None:
-        console.print(
-            f"[green]✓[/] Você já está na versão mais recente ([cyan]{__version__}[/])"
-        )
-        return False
-
-    installer = detect_installer()
-    console.print(
-        f"[yellow]→[/] Atualizando de {update_info.current_version} "
-        f"para {update_info.latest_version}..."
-    )
-
-    # Monta comando de upgrade
-    if installer == "pipx":
-        cmd = ["pipx", "upgrade", "airev"]
-    else:
-        cmd = [sys.executable, "-m", "pip", "install", "--upgrade", "airev"]
-
-    try:
-        result = subprocess.run(
-            cmd,
-            capture_output=True,
-            text=True,
-            check=False,
-        )
-
-        if result.returncode == 0:
-            console.print(
-                f"[green]✓[/] Atualizado para versão {update_info.latest_version}"
-            )
-            return True
-        else:
-            console.print(f"[red]✗[/] Falha ao atualizar: {result.stderr.strip()}")
-            console.print(f"[dim]Tente manualmente: {' '.join(cmd)}[/]")
-            return False
-
-    except FileNotFoundError:
-        console.print(f"[red]✗[/] Comando '{cmd[0]}' não encontrado")
-        if installer == "pipx":
-            console.print("[dim]Tente: pip install --upgrade airev[/]")
-        return False
-    except Exception as e:
-        console.print(f"[red]✗[/] Erro inesperado: {e}")
-        return False