Add a warning on invalid authorization (#58)

* Add a warning on invalid authorization

* Integrate auth warning into auth.require_auth

* Bump version to 0.1.8

Author: ab-10

.pre-commit-config.yaml

@@ -1,6 +1,6 @@
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.12.2
+    rev: v0.12.3
     hooks:
       - id: ruff
         types_or: [ python, pyi ]

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "rml"
-version = "0.1.7"
+version = "0.1.8"
 description = "CLI for Recurse ML"
 readme = "README.md"
 authors = [

src/rml/__init__.py

@@ -227,7 +227,10 @@ def make_tar(
     ),
 )
 def post_check(
-    archive_filename: str, archive_path: Path, target_filenames: list[str], **kwargs
+    archive_filename: str,
+    archive_path: Path,
+    target_filenames: list[str],
+    **kwargs,
 ) -> dict[str, Any]:
     api_key = get_env_value(RECURSE_API_KEY_NAME)
 
@@ -311,15 +314,14 @@ def analyze(
                 console.print(Text("✨ No changes found in the specified files! ✨"))
             return
 
-        if len(changed_target_filenames) != len(target_paths):
-            target_file_paths = filter(lambda path: path.is_file(), target_paths)
-            skipped_target_paths = list(
-                filter(
-                    lambda path: str(path) not in changed_target_filenames,
-                    target_file_paths,
-                )
+        target_file_paths = filter(lambda path: path.is_file(), target_paths)
+        skipped_target_paths = list(
+            filter(
+                lambda path: str(path) not in changed_target_filenames,
+                target_file_paths,
             )
-
+        )
+        if len(skipped_target_paths) > 0:
             if markdown:
                 print(
                     f"Skipping {len(skipped_target_paths)} unchanged files: {', '.join(str(p) for p in skipped_target_paths)}"

src/rml/auth.py

@@ -1,5 +1,4 @@
 import asyncio
-import sys
 from functools import wraps
 from typing import Optional
 
@@ -196,8 +195,10 @@ def wrapper(*args, **kwargs):
         if not (SKIP_AUTH or is_authenticated()):
             auth_result = asyncio.run(authenticate_with_github(console=console))
             render_auth_result(auth_result, console=console)
+            """ TODO (Armin): Uncomment this on 21/07/2025
             if auth_result.status != AuthStatus.SUCCESS:
                 sys.exit(1)
+            """
 
         return f(*args, **kwargs)
 

src/rml/package_config.py

@@ -16,4 +16,4 @@
 ENV_FILE_PATH = PROJECT_ROOT / ".env.rml"
 RECURSE_API_KEY_NAME = "RECURSE_API_KEY"
 
-SKIP_AUTH = True
+SKIP_AUTH = False

src/rml/ui.py

@@ -350,9 +350,18 @@ def render_auth_result(result: AuthResult, console: Console) -> None:
     if result.status == AuthStatus.SUCCESS:
         console.print("[bold green]✅ Authentication successful![/bold green]")
     elif result.status == AuthStatus.PLAN_REQUIRED:
+        console.print("[bold] ⚠️ You need to purchase a plan on the marketplace[/bold]")
+        console.print("   📦 https://github.com/marketplace/recurse-ml")
+        console.print(
+            "[dim]Free CLI access will terminate on 21/07/2025.[/dim]"
+            "[dim]Our GH App will remain free for open source projects.[/dim]"
+        )
+        console.print()
+        """ TODO (Armin): Uncomment this on 21/07/2025
         console.print(
             "[bold yellow]⚠️ To use rml, please purchase a plan at https://github.com/marketplace/recurse-ml and run `rml auth login` again.[/bold yellow]"
         )
+        """
     else:
         console.print(
             f"[bold red]❌ Authentication failed {'(' + result.message + ')' if result.message else ''}[/bold red]"