codeberg-cli 0.4.1__tar.gz → 0.4.2__tar.gz

{codeberg_cli-0.4.1 → codeberg_cli-0.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codeberg-cli
-Version: 0.4.1
+Version: 0.4.2
 Summary: A Forgejo CLI — works with Codeberg and any Forgejo instance
 Project-URL: Homepage, https://codeberg.org/ThatXliner/codeberg-cli
 Project-URL: Repository, https://codeberg.org/ThatXliner/codeberg-cli

{codeberg_cli-0.4.1 → codeberg_cli-0.4.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "codeberg-cli"
-version = "0.4.1"
+version = "0.4.2"
 description = "A Forgejo CLI — works with Codeberg and any Forgejo instance"
 readme = "README.md"
 license = "MIT"

{codeberg_cli-0.4.1 → codeberg_cli-0.4.2}/src/codeberg_cli/client.py

@@ -16,6 +16,10 @@ _VERBS = {
 class ClientError(RuntimeError):
     """Raised when the API returns a non-2xx status."""
 
+    def __init__(self, message: str, status_code: int | None = None) -> None:
+        super().__init__(message)
+        self.status_code = status_code
+
 
 def _response_error_message(response: httpx.Response) -> str:
     if not response.text:
@@ -52,7 +56,12 @@ class Client:
         raw = raw.rstrip("/")
         if not raw.endswith("/api/v1"):
             raw += "/api/v1"
-        self._client = httpx.Client(base_url=raw, headers=headers, timeout=30.0)
+        self._client = httpx.Client(
+            base_url=raw,
+            headers=headers,
+            timeout=30.0,
+            follow_redirects=True,
+        )
 
     @property
     def base_url(self) -> str:
@@ -99,7 +108,9 @@ class Client:
             response = self._client.request(method, path, **kwargs)
         if not response.is_success:
             msg = _response_error_message(response)
-            raise ClientError(f"{response.status_code} {msg}")
+            raise ClientError(
+                f"{response.status_code} {msg}", status_code=response.status_code
+            )
         if response.status_code == 204:
             return None
         return response.json()

{codeberg_cli-0.4.1 → codeberg_cli-0.4.2}/src/codeberg_cli/routes/actions/_format.py

@@ -1,4 +1,27 @@
-from typing import Any
+from typing import Any, Callable, TypeVar
+
+from codeberg_cli.client import ClientError
+from xclif.errors import UsageError
+
+T = TypeVar("T")
+
+
+def actions_request(repo: str, call: Callable[[], T]) -> T:
+    """Run an Actions API *call*, turning a 404 into a clear message.
+
+    Forgejo returns 404 for Actions endpoints when Actions is not enabled on
+    the repository (or the repo does not exist), which is otherwise surfaced as
+    an opaque ``404 The target couldn't be found.`` traceback.
+    """
+    try:
+        return call()
+    except ClientError as exc:
+        if exc.status_code == 404:
+            raise UsageError(
+                f"No Actions found for {repo}.",
+                hint="Actions may be disabled for this repository, or the repository may not exist.",
+            ) from exc
+        raise
 
 
 def _first(run: dict[str, Any], *keys: str, default: str = "") -> Any:

{codeberg_cli-0.4.1 → codeberg_cli-0.4.2}/src/codeberg_cli/routes/actions/dispatch.py

@@ -23,9 +23,20 @@ def _(
             return 1
         repo = inferred
 
-    client.post(
-        f"/repos/{repo}/actions/workflows/{workflow}/dispatches",
-        data={"ref": ref},
-        action="Dispatching workflow",
-    )
+    from codeberg_cli.client import ClientError
+    from xclif.errors import UsageError
+
+    try:
+        client.post(
+            f"/repos/{repo}/actions/workflows/{workflow}/dispatches",
+            data={"ref": ref},
+            action="Dispatching workflow",
+        )
+    except ClientError as exc:
+        if exc.status_code == 404:
+            raise UsageError(
+                f"No workflow {workflow!r} found in {repo}.",
+                hint="List workflows with 'cb actions workflows', or Actions may be disabled for this repository.",
+            ) from exc
+        raise
     rich.print(f"[green]Dispatched {workflow} on {ref} in {repo}.[/green]")

{codeberg_cli-0.4.1 → codeberg_cli-0.4.2}/src/codeberg_cli/routes/actions/run.py

@@ -24,7 +24,18 @@ def _(
             return 1
         repo = inferred
 
-    run = client.get(f"/repos/{repo}/actions/runs/{run_id}")
+    from codeberg_cli.client import ClientError
+    from xclif.errors import UsageError
+
+    try:
+        run = client.get(f"/repos/{repo}/actions/runs/{run_id}")
+    except ClientError as exc:
+        if exc.status_code == 404:
+            raise UsageError(
+                f"No run #{run_id} found in {repo}.",
+                hint="Check the run ID with 'cb actions runs', or Actions may be disabled for this repository.",
+            ) from exc
+        raise
 
     if is_json_mode():
         output(run)

{codeberg_cli-0.4.1 → codeberg_cli-0.4.2}/src/codeberg_cli/routes/actions/runs.py

@@ -6,7 +6,7 @@ from codeberg_cli.git import infer_repo
 from codeberg_cli.helpers import print_table, require_client
 from xclif import Option, command
 
-from codeberg_cli.routes.actions._format import list_runs, normalize_run
+from codeberg_cli.routes.actions._format import actions_request, list_runs, normalize_run
 
 
 @command("runs")
@@ -24,7 +24,10 @@ def _(
             return 1
         repo = inferred
 
-    response = client.get(f"/repos/{repo}/actions/runs", params={"limit": limit, "page": 1})
+    response = actions_request(
+        repo,
+        lambda: client.get(f"/repos/{repo}/actions/runs", params={"limit": limit, "page": 1}),
+    )
     runs = list_runs(response)
 
     if not runs:

{codeberg_cli-0.4.1 → codeberg_cli-0.4.2}/src/codeberg_cli/routes/actions/workflows.py

@@ -34,7 +34,7 @@ def _(
             action="Fetching workflow files",
         )
     except ClientError as exc:
-        if str(exc).startswith("404 "):
+        if exc.status_code == 404:
             rich.print(f"[dim]No workflows in {repo}.[/dim]")
             return
         raise

{codeberg_cli-0.4.1 → codeberg_cli-0.4.2}/tests/test_actions.py

@@ -143,3 +143,54 @@ def test_actions_workflows_handles_missing_workflow_dir(httpx_mock, capsys, tmp_
     cmd.execute(["--repo", "owner/repo"])
     captured = capsys.readouterr()
     assert "No workflows in owner/repo." in captured.out
+
+
+def test_actions_runs_404_raises_clear_error(httpx_mock, capsys, tmp_path, monkeypatch):
+    """A 404 (Actions disabled) becomes a clean message, not a traceback."""
+    _login(monkeypatch, tmp_path)
+    httpx_mock.add_response(
+        url="https://codeberg.org/api/v1/repos/owner/repo/actions/runs?limit=20&page=1",
+        status_code=404,
+        json={"message": "The target couldn't be found."},
+    )
+
+    from codeberg_cli.routes.actions.runs import _ as cmd
+
+    rc = cmd.execute(["--repo", "owner/repo"])
+    assert rc == 2
+    captured = capsys.readouterr()
+    assert "No Actions found for owner/repo." in captured.err
+    assert "Actions may be disabled" in captured.err
+
+
+def test_actions_run_404_raises_clear_error(httpx_mock, capsys, tmp_path, monkeypatch):
+    _login(monkeypatch, tmp_path)
+    httpx_mock.add_response(
+        url="https://codeberg.org/api/v1/repos/owner/repo/actions/runs/999",
+        status_code=404,
+        json={"message": "The target couldn't be found."},
+    )
+
+    from codeberg_cli.routes.actions.run import _ as cmd
+
+    rc = cmd.execute(["999", "--repo", "owner/repo"])
+    assert rc == 2
+    captured = capsys.readouterr()
+    assert "No run #999 found in owner/repo." in captured.err
+
+
+def test_actions_dispatch_404_raises_clear_error(httpx_mock, capsys, tmp_path, monkeypatch):
+    _login(monkeypatch, tmp_path)
+    httpx_mock.add_response(
+        url="https://codeberg.org/api/v1/repos/owner/repo/actions/workflows/ci.yml/dispatches",
+        status_code=404,
+        json={"message": "The target couldn't be found."},
+        method="POST",
+    )
+
+    from codeberg_cli.routes.actions.dispatch import _ as cmd
+
+    rc = cmd.execute(["ci.yml", "--repo", "owner/repo"])
+    assert rc == 2
+    captured = capsys.readouterr()
+    assert "No workflow 'ci.yml' found in owner/repo." in captured.err

{codeberg_cli-0.4.1 → codeberg_cli-0.4.2}/tests/test_client.py

@@ -49,6 +49,34 @@ def test_client_error_with_non_json_body(httpx_mock):
     assert str(exc.value) == "502 Bad Gateway"
 
 
+def test_client_error_exposes_status_code(httpx_mock):
+    httpx_mock.add_response(
+        url="https://codeberg.org/api/v1/user",
+        status_code=404,
+        json={"message": "not found"},
+    )
+    client = Client(token="t")
+    with pytest.raises(ClientError) as exc:
+        client.get("/user")
+    assert exc.value.status_code == 404
+
+
+def test_client_follows_redirects(httpx_mock):
+    """A 301 (e.g. a renamed repo) is followed to the final location."""
+    httpx_mock.add_response(
+        url="https://codeberg.org/api/v1/repos/owner/old-name",
+        status_code=301,
+        headers={"Location": "https://codeberg.org/api/v1/repos/owner/new-name"},
+    )
+    httpx_mock.add_response(
+        url="https://codeberg.org/api/v1/repos/owner/new-name",
+        json={"full_name": "owner/new-name"},
+    )
+    client = Client(token="t")
+    result = client.get("/repos/owner/old-name")
+    assert result == {"full_name": "owner/new-name"}
+
+
 def test_client_no_auth(httpx_mock):
     """Client without token can still be created."""
     httpx_mock.add_response(url="https://codeberg.org/api/v1/version", json={"version": "1.22"})

{codeberg_cli-0.4.1 → codeberg_cli-0.4.2}/uv.lock

@@ -26,7 +26,7 @@ wheels = [
 
 [[package]]
 name = "codeberg-cli"
-version = "0.4.1"
+version = "0.4.2"
 source = { editable = "." }
 dependencies = [
     { name = "httpx" },