rio-cli 0.2.0__tar.gz → 0.2.2__tar.gz

{rio_cli-0.2.0 → rio_cli-0.2.2}/PKG-INFO

@@ -1,14 +1,15 @@
 Metadata-Version: 2.5
 Name: rio-cli
-Version: 0.2.0
+Version: 0.2.2
 Summary: Command-line client for Rio, an automated code review tool.
 Project-URL: Homepage, https://github.com/JayTheCoder77/rio
 Project-URL: Repository, https://github.com/JayTheCoder77/rio
 Author: JayTheCoder77
 License-Expression: ISC
 License-File: LICENSE
-Requires-Python: <3.13,>=3.12
+Requires-Python: >=3.12
 Requires-Dist: httpx>=0.28.1
+Requires-Dist: platformdirs>=4.0
 Requires-Dist: pyyaml>=6.0
 Requires-Dist: rich>=15.0.0
 Requires-Dist: rio-review-core<0.2.0,>=0.1.0
@@ -38,8 +39,17 @@ dashboard's API keys page):
 rio auth
 ```
 
-This prompts for your API key, validates it against your Rio instance, and saves it to
-`~/.config/rio/config.toml` with restricted (`0600`) file permissions.
+This prompts for your API key, validates it against your Rio instance, and saves it to the
+platform-specific config file shown below. On POSIX the file is written with restricted
+(`0600`) permissions; on Windows it lives under your already-private `%APPDATA%`.
+
+| OS | Config file |
+| --- | --- |
+| Linux | `~/.config/rio/config.toml` |
+| macOS | `~/Library/Application Support/rio/config.toml` |
+| Windows | `%APPDATA%\rio\config.toml` |
+
+The directory is created for you on first run, so you never need to mkdir by hand.
 
 ## Usage
 
@@ -62,9 +72,10 @@ rio review --staged --include-untracked
 
 ## Configuration
 
-`rio auth` manages `~/.config/rio/config.toml` for you. To point the CLI at a
-self-hosted `ai-engine` instance instead of the default (`http://localhost:8000`), edit
-the `[api] url` value in that file, or set it before running `rio auth`:
+`rio auth` manages the platform-specific config file for you (see the table above for the
+exact location per OS). To point the CLI at a self-hosted `ai-engine` instance instead of
+the default (`https://rio-ai-engine.onrender.com`), edit the `[api] url` value in that
+file, or set it before running `rio auth`:
 
 ```toml
 [api]

{rio_cli-0.2.0 → rio_cli-0.2.2}/README.md

@@ -21,8 +21,17 @@ dashboard's API keys page):
 rio auth
 ```
 
-This prompts for your API key, validates it against your Rio instance, and saves it to
-`~/.config/rio/config.toml` with restricted (`0600`) file permissions.
+This prompts for your API key, validates it against your Rio instance, and saves it to the
+platform-specific config file shown below. On POSIX the file is written with restricted
+(`0600`) permissions; on Windows it lives under your already-private `%APPDATA%`.
+
+| OS | Config file |
+| --- | --- |
+| Linux | `~/.config/rio/config.toml` |
+| macOS | `~/Library/Application Support/rio/config.toml` |
+| Windows | `%APPDATA%\rio\config.toml` |
+
+The directory is created for you on first run, so you never need to mkdir by hand.
 
 ## Usage
 
@@ -45,9 +54,10 @@ rio review --staged --include-untracked
 
 ## Configuration
 
-`rio auth` manages `~/.config/rio/config.toml` for you. To point the CLI at a
-self-hosted `ai-engine` instance instead of the default (`http://localhost:8000`), edit
-the `[api] url` value in that file, or set it before running `rio auth`:
+`rio auth` manages the platform-specific config file for you (see the table above for the
+exact location per OS). To point the CLI at a self-hosted `ai-engine` instance instead of
+the default (`https://rio-ai-engine.onrender.com`), edit the `[api] url` value in that
+file, or set it before running `rio auth`:
 
 ```toml
 [api]

{rio_cli-0.2.0 → rio_cli-0.2.2}/pyproject.toml

@@ -1,17 +1,18 @@
 [project]
 name = "rio-cli"
-version = "0.2.0"
+version = "0.2.2"
 description = "Command-line client for Rio, an automated code review tool."
 readme = "README.md"
 license = "ISC"
 authors = [{ name = "JayTheCoder77" }]
-requires-python = ">=3.12,<3.13"
+requires-python = ">=3.12"
 dependencies = [
     "rio-review-core>=0.1.0,<0.2.0",
     "httpx>=0.28.1",
     "rich>=15.0.0",
     "typer>=0.27.1",
-    "pyyaml>=6.0"
+    "pyyaml>=6.0",
+    "platformdirs>=4.0",
 ]
 
 [project.scripts]

{rio_cli-0.2.0 → rio_cli-0.2.2}/rio_cli/auth.py

@@ -29,8 +29,11 @@ def _write_config(data: dict) -> None:
     lines.append(f'api_key = "{api_section["api_key"]}"')
 
     CONFIG_PATH.write_text("\n".join(lines) + "\n")
-    # This file holds a real credential — don't rely on the default umask.
-    os.chmod(CONFIG_PATH, stat.S_IRUSR | stat.S_IWUSR)  # 0o600
+    # This file holds a real credential. On POSIX, lock it to the user via 0o600.
+    # That mode has no real meaning on Windows, where %APPDATA% is already
+    # per-user private by default.
+    if os.name != "nt":
+        os.chmod(CONFIG_PATH, stat.S_IRUSR | stat.S_IWUSR)  # 0o600
 
 
 def _validate_api_key(api_key: str, ai_engine_url: str) -> None:

{rio_cli-0.2.0 → rio_cli-0.2.2}/rio_cli/config.py

@@ -1,11 +1,15 @@
-from pathlib import Path
+import os
 
+import platformdirs
 import tomllib
 
 from rio_cli.utils import _fail
 
-DEFAULT_AI_ENGINE_URL = "http://localhost:8000"
-CONFIG_PATH = Path.home() / ".config" / "rio" / "config.toml"
+DEFAULT_AI_ENGINE_URL = os.environ.get(
+    "RIO_API_URL", "https://rio-ai-engine.onrender.com"
+)
+# DEFAULT_AI_ENGINE_URL = "http://localhost:8000"
+CONFIG_PATH = platformdirs.user_config_path("rio") / "config.toml"
 
 def get_ai_engine_url() -> str:
     if not CONFIG_PATH.exists():

{rio_cli-0.2.0 → rio_cli-0.2.2}/rio_cli/post.py

@@ -7,11 +7,12 @@ from rio_cli.utils import _fail
 
 
 def _post_review(diff_text : str) -> list[Finding]:
-    headers : dict = {}
     api_key = get_api_key()
-    if api_key:
-        headers["Authorization"] = f"Bearer {api_key}"
-    
+    if not api_key:
+        _fail("Error: not authenticated. Run `rio auth` first.")
+
+    headers = {"Authorization": f"Bearer {api_key}"}
+
     rio_config = load_rio_config()
     ai_engine_url = get_ai_engine_url()
     try:
@@ -26,7 +27,13 @@ def _post_review(diff_text : str) -> list[Finding]:
     
     except httpx.TimeoutException:
         _fail("Error: ai-engine request timed out.")
-    
+
+    if response.status_code == 401:
+        _fail("Error: not authenticated. Run `rio auth` first.")
+    if response.status_code == 412:
+        _fail(f"Error: {response.json().get('detail', response.text)}")
+    if response.status_code == 422:
+        _fail(f"Error: {response.json().get('detail', response.text)}")
     if response.status_code != 200:
         _fail(f"Error: ai-engine returned {response.status_code}: {response.text}")