airev 1.1.1__tar.gz → 1.2.0__tar.gz

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: airev
-Version: 1.1.1
+Version: 1.2.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.2.0}/src/airev.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: airev
-Version: 1.1.1
+Version: 1.2.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.2.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.2.0"

{airev-1.1.1 → airev-1.2.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
@@ -88,13 +88,13 @@ def review(
 
     Exemplos:
 
-        code-reviewer review --base main
+        airev review --base main
 
-        code-reviewer review --base develop --runner copilot
+        airev review --base develop --runner copilot
 
-        code-reviewer review --base main --json-output
+        airev review --base main --json-output
 
-        code-reviewer review --base main --no-progress
+        airev review --base main --no-progress
     """
     workdir = workdir or Path.cwd()
     start_time = time.perf_counter()
@@ -232,7 +232,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.2.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.2.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.2.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.2.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.1.1 → airev-1.2.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