loxo-cli 0.2.2__tar.gz → 0.2.3__tar.gz

{loxo_cli-0.2.2 → loxo_cli-0.2.3}/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## [0.2.3]
+
+### Changed
+
+- `loxo activities list` drops the `--job-id` filter and gains `--company-id`.
+  Loxo's `person_events` endpoint rejects `job_id` as a query parameter (HTTP
+  422 `Invalid parameters: [:job_id]`), so the flag never worked. `person_id`
+  and `company_id` are the only server-side filters the endpoint accepts.
+  (`activities add --job-id` is unaffected — there `job_id` is a request-body
+  field, not a query parameter.)
+
 ## [0.2.2]
 
 ### Fixed

{loxo_cli-0.2.2 → loxo_cli-0.2.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: loxo-cli
-Version: 0.2.2
+Version: 0.2.3
 Summary: Unofficial command-line interface for the Loxo recruiting ATS/CRM API
 Project-URL: Homepage, https://github.com/alphaomegateam/loxo-cli
 License: MIT

{loxo_cli-0.2.2 → loxo_cli-0.2.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "loxo-cli"
-version = "0.2.2"
+version = "0.2.3"
 description = "Unofficial command-line interface for the Loxo recruiting ATS/CRM API"
 readme = "README.md"
 license = { text = "MIT" }

{loxo_cli-0.2.2 → loxo_cli-0.2.3}/src/loxo_cli/commands/activities.py

@@ -16,16 +16,19 @@ activities_app = typer.Typer(
 def list_activities(
     ctx: typer.Context,
     person_id: Optional[int] = typer.Option(None, "--person-id"),
-    job_id: Optional[int] = typer.Option(None, "--job-id"),
+    company_id: Optional[int] = typer.Option(None, "--company-id"),
     all_pages: bool = typer.Option(False, "--all"),
     per_page: int = typer.Option(50, "--per-page"),
 ) -> None:
+    # Loxo's person_events endpoint only accepts person_id / company_id as
+    # server-side filters. job_id is not a valid query param (returns 422), so
+    # it is intentionally not exposed here — filter by person or company instead.
     state = ctx.obj
     params: dict = {}
     if person_id is not None:
         params["person_id"] = person_id
-    if job_id is not None:
-        params["job_id"] = job_id
+    if company_id is not None:
+        params["company_id"] = company_id
     client = state.client()
     if all_pages:
         rows = list(

{loxo_cli-0.2.2 → loxo_cli-0.2.3}/tests/test_cmd_activities.py

@@ -21,6 +21,31 @@ def test_activities_list():
     assert json.loads(result.stdout)[0]["notes"] == "called"
 
 
+@respx.mock
+def test_activities_list_filters_by_person_and_company():
+    route = respx.get("https://app.loxo.co/api/acme/person_events").mock(
+        return_value=httpx.Response(200, json={"scroll_id": None, "person_events": []})
+    )
+    result = runner.invoke(
+        app,
+        ["--json", "activities", "list", "--person-id", "50", "--company-id", "7"],
+        env=ENV,
+    )
+    assert result.exit_code == 0
+    assert dict(route.calls.last.request.url.params) == {
+        "person_id": "50",
+        "company_id": "7",
+        "per_page": "50",
+    }
+
+
+def test_activities_list_has_no_job_id_option():
+    # job_id is not a valid person_events query param (Loxo returns 422), so the
+    # list command must not advertise a --job-id filter.
+    result = runner.invoke(app, ["activities", "list", "--help"], env=ENV)
+    assert "--job-id" not in result.stdout
+
+
 @respx.mock
 def test_activities_add_wraps_person_event():
     captured = {}

{loxo_cli-0.2.2 → loxo_cli-0.2.3}/uv.lock

@@ -274,7 +274,7 @@ wheels = [
 
 [[package]]
 name = "loxo-cli"
-version = "0.2.2"
+version = "0.2.3"
 source = { editable = "." }
 dependencies = [
     { name = "click" },