operx-cli 0.7.7__tar.gz → 0.8.1__tar.gz

{operx_cli-0.7.7 → operx_cli-0.8.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: operx-cli
-Version: 0.7.7
+Version: 0.8.1
 Summary: La línea de comandos de OperX: tu empresa desde la terminal, con tu token de API.
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown

{operx_cli-0.7.7 → operx_cli-0.8.1}/operx_cli/AGENTES.md

@@ -57,6 +57,13 @@ llamada queda en el historial del token, marcada como hecha desde la CLI.
   (la empresa usa Personas y no ha iniciado su jornada), `assigned` (está en
   un trabajo abierto sin estar en sitio) o `available`. Filtra con
   `operx api GET employees/ -q status=on_site`.
+- Una orden o un plan preventivo que hace un contratista vinculado no tiene a
+  nadie de la empresa: «Asignada a» muestra el contratista, y en `--json` va
+  en `assigned_company` (`{"name": "Climas del Valle", "kind": "contractor"}`;
+  `null` si no lo tiene un contratista). Mientras lo tenga, la API rechaza
+  cambiar la orden desde aquí, y el plan no cambia de responsable ni de
+  equipo: el 400 dice por qué. Díselo a la persona; no lo intentes de otra
+  forma.
 - Los resultados van a stdout y los avisos a stderr:
   `operx equipos listar --todas --json > equipos.json` guarda solo los datos.
 - `--csv` da un CSV para Excel (separado por punto y coma, con decimales con
@@ -79,7 +86,7 @@ llamada queda en el historial del token, marcada como hecha desde la CLI.
     `rutinas crear`, `cambiar` y `quitar`; `campos crear`, `cambiar`,
     `quitar` y `guardar`.
   - SG-SST y Ambiental: `sst nueva-accion` y `accion-hecha`;
-    `ambiental nueva-accion`, `accion-hecha`, `residuo` y `lectura`.
+    `ambiental nueva-accion`, `accion-hecha` y `residuo`.
   - `avisos --marcar-leidos`, y `operx api` con POST, PUT, PATCH o DELETE.
 - Un error se corrige con su contrario, nunca borrando: `inventario revertir`
   deshace un movimiento, `lecturas anular` una lectura. La excepción es

{operx_cli-0.7.7 → operx_cli-0.8.1}/operx_cli/__init__.py

@@ -2,4 +2,4 @@
 API (backend/users/api_allowlist.py) from a terminal, with a personal API
 token."""
 
-__version__ = "0.7.7"
+__version__ = "0.8.1"

{operx_cli-0.7.7 → operx_cli-0.8.1}/operx_cli/commands/ambiental.py

@@ -1,6 +1,7 @@
 """`operx ambiental`: Ambiental, per company: its action plan, permits and
-obligations, and the two records made most often, waste and meter
-readings. A wrong record is voided in the app, never edited."""
+obligations, and the record made most often, waste. Consumption is kept
+by the month in the app (no meter readings). A wrong record is voided in
+the app, never edited."""
 
 from datetime import date
 from typing import Annotated, Optional
@@ -158,23 +159,3 @@ def _stream(client, path, corriente, sede):
     if not streams:
         raise CliError("Esta empresa aún no tiene corrientes de residuos: créalas en la app, en Ambiental › Residuos.")
     return find_in(streams, corriente, ["name"], "corriente")
-
-
-@app.command()
-def lectura(
-    ctx: typer.Context,
-    medidor: Annotated[str, typer.Option("--medidor", help="El nombre del medidor o de lo que mide: Acueducto.")],
-    valor: Annotated[str, typer.Option("--valor", help="Lo que marca: 1.234,5.")],
-    fecha: FechaOption = None,
-    empresa: EmpresaOption = None,
-    si: YesOption = False,
-    as_json: JsonOption = False,
-):
-    """Registra la lectura de un medidor (agua, energía, gas)."""
-    client = client_of(ctx)
-    path = company_path(client, ENVIRONMENT, empresa)
-    meter = find_in(client.get(f"{path}/consumption/meters"), medidor, ["name", "meter_name"], "medidor")
-    reading = {"source": meter["id"], "read_on": day(fecha) or date.today().isoformat(), "value": number(valor)}
-    confirm_write(f"Lectura {valor} en «{meter['name']}» el {shown_moment(reading['read_on'])}.", si)
-    recorded = answer_of(client.send("POST", f"{path}/consumption/readings", json=reading))
-    print_result(recorded, as_json, f"Lectura registrada: {valor} en {meter['name']}.")

{operx_cli-0.7.7 → operx_cli-0.8.1}/operx_cli/commands/ordenes.py

@@ -72,11 +72,18 @@ PersonOption = Annotated[
 ]
 
 
+def assignee(record):
+    """The person doing an order or a plan, or the linked contractor doing it
+    instead."""
+    return record.get("assigned_to_name") or (record.get("assigned_company") or {}).get("name")
+
+
 def _in_spanish(order):
     return {
         **order,
         "status": STATUS_LABELS.get(order.get("status"), order.get("status")),
         "priority": PRIORITY_LABELS.get(order.get("priority"), order.get("priority")),
+        "assigned_to_name": assignee(order),
         "due_date": shown_moment(order.get("due_date")),
     }
 
@@ -127,7 +134,7 @@ def ver(ctx: typer.Context, orden: Annotated[int, typer.Argument(help="El númer
         ("Estado", STATUS_LABELS.get(order.get("status"), order.get("status"))),
         ("Prioridad", PRIORITY_LABELS.get(order.get("priority"), order.get("priority"))),
         ("Tipo", TYPE_LABELS.get(order.get("work_type"), order.get("work_type"))),
-        ("Asignada a", order.get("assigned_to_name")),
+        ("Asignada a", assignee(order)),
         ("También", order.get("extra_assignee_names")),
         ("Equipo", order.get("asset_name")),
         ("Lugar", order.get("site_name")),

{operx_cli-0.7.7 → operx_cli-0.8.1}/operx_cli/commands/preventivos.py

@@ -24,7 +24,7 @@ from .common import (
     print_result,
     without_empty,
 )
-from .ordenes import PRIORITY_KEYS, PRIORITY_LABELS, Prioridad, account_id
+from .ordenes import PRIORITY_KEYS, PRIORITY_LABELS, Prioridad, account_id, assignee
 
 app = typer.Typer(help="Trabajos recurrentes: los planes que crean una orden cada tanto.", no_args_is_help=True)
 
@@ -69,7 +69,9 @@ def every(plan):
 
 
 def _shown(plan):
-    return {**plan, "every": every(plan), "next": shown_moment(plan.get("next_run"))}
+    return {
+        **plan, "every": every(plan), "next": shown_moment(plan.get("next_run")), "assigned_to_name": assignee(plan),
+    }
 
 
 @app.command()
@@ -106,7 +108,7 @@ def ver(ctx: typer.Context, plan: PlanArgument, as_json: JsonOption = False):
         ("Cliente", found.get("customer_name")),
         ("Orden que crea", found.get("wo_title")),
         ("Prioridad", PRIORITY_LABELS.get(found.get("wo_priority"), found.get("wo_priority"))),
-        ("Asignado a", found.get("assigned_to_name")),
+        ("Asignado a", assignee(found)),
         ("Horas estimadas", found.get("estimated_hours")),
         ("Lista de chequeo", f"{len(checklist)} puntos" if checklist else ""),
         ("Activo", found.get("is_active")),

{operx_cli-0.7.7 → operx_cli-0.8.1}/operx_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: operx-cli
-Version: 0.7.7
+Version: 0.8.1
 Summary: La línea de comandos de OperX: tu empresa desde la terminal, con tu token de API.
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown

{operx_cli-0.7.7 → operx_cli-0.8.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "operx-cli"
-version = "0.7.7"
+version = "0.8.1"
 description = "La línea de comandos de OperX: tu empresa desde la terminal, con tu token de API."
 readme = "README.md"
 requires-python = ">=3.10"

{operx_cli-0.7.7 → operx_cli-0.8.1}/tests/test_cards.py

@@ -17,7 +17,7 @@ ORDER = {
     "site_name": "Planta Norte",
     "site_arrival": {"address": "", "city": "", "latitude": None, "longitude": None, "directions": ""},
     "customer_id": None, "customer_name": None, "assigned_to_name": None, "assigned_to_avatar": None,
-    "extra_assignee_names": [], "location_description": "", "custom_fields": {}, "signature": None,
+    "assigned_company": None, "extra_assignee_names": [], "location_description": "", "custom_fields": {}, "signature": None,
     "signed_by_name": "", "signed_at": None, "created_at": "2026-10-23T02:19:42.253068-05:00",
     "updated_at": "2026-10-23T02:19:42.253068-05:00",
     "comments": [{

{operx_cli-0.7.7 → operx_cli-0.8.1}/tests/test_json_writes.py

@@ -12,7 +12,7 @@ from operx_cli.app import app
 from .fake_api import FakeApi, page, run
 from .test_more_of_the_api import FINDING, MOVEMENT, REASONS, STOP
 from .test_preventivos import ASSET, NOTES, PLAN
-from .test_sst_ambiental import EMPLOYEES, ENV_ACTION, ENV_COMPANIES, METERS, SST_ACTION, SST_COMPANIES, STREAMS
+from .test_sst_ambiental import EMPLOYEES, ENV_ACTION, ENV_COMPANIES, SST_ACTION, SST_COMPANIES, STREAMS
 
 FILTER = {"id": 3, "code": "FIL-010", "name": "Filtro de aceite"}
 STORE = {"id": 2, "name": "Bodega central"}
@@ -74,9 +74,6 @@ CASES = [
     (["ambiental", "residuo", "--corriente", "aceite usado", "--kg", "12,5"],
      {("GET", "environment/companies/"): ENV_COMPANIES, ("GET", "environment/companies/1/waste/streams/"): STREAMS},
      ("POST", "environment/companies/1/waste/entries/"), (201, {"id": 9, "kg": "12.500"})),
-    (["ambiental", "lectura", "--medidor", "Acueducto", "--valor", "1.234,5"],
-     {("GET", "environment/companies/"): ENV_COMPANIES, ("GET", "environment/companies/1/consumption/meters/"): METERS},
-     ("POST", "environment/companies/1/consumption/readings/"), (201, {"id": 3})),
     (["avisos", "--marcar-leidos"], {}, ("POST", "notifications/mark-read/"), {"marked": 4}),
 ]
 

{operx_cli-0.7.7 → operx_cli-0.8.1}/tests/test_ordenes.py

@@ -6,7 +6,12 @@ from .fake_api import FakeApi, page, run
 
 ORDER = {
     "id": 15, "title": "Cambio de rodamiento", "status": "in_progress", "priority": "high",
-    "assigned_to_name": "Pedro Pérez", "asset_name": "Bomba 4", "site_name": "Planta", "due_date": "2026-10-30T00:00:00-05:00",
+    "assigned_to_name": "Pedro Pérez", "assigned_company": None, "asset_name": "Bomba 4", "site_name": "Planta",
+    "due_date": "2026-10-30T00:00:00-05:00",
+}
+# The same order while a linked contractor does it: nobody of the company is on it.
+HELD_BY_CONTRACTOR = {
+    **ORDER, "assigned_to_name": None, "assigned_company": {"name": "Climas del Valle", "kind": "contractor"},
 }
 PEOPLE = page({"id": 3, "user_id": 21, "user_name": "Pedro Pérez", "employee_code": "T-01"})
 
@@ -102,3 +107,11 @@ def test_ver_shows_one_order():
     result = run(FakeApi({("GET", "work-orders/15/"): ORDER}), "ordenes", "ver", "15")
     assert result.exit_code == 0, result.output
     assert "Cambio de rodamiento" in result.output
+
+
+def test_an_order_a_contractor_has_names_the_contractor():
+    listed = run(FakeApi({("GET", "work-orders/"): page(HELD_BY_CONTRACTOR)}), "ordenes", "listar")
+    shown = run(FakeApi({("GET", "work-orders/15/"): HELD_BY_CONTRACTOR}), "ordenes", "ver", "15")
+    assert listed.exit_code == 0, listed.output
+    assert shown.exit_code == 0, shown.output
+    assert "Climas del Valle" in listed.output and "Climas del Valle" in shown.output

{operx_cli-0.7.7 → operx_cli-0.8.1}/tests/test_preventivos.py

@@ -14,11 +14,13 @@ PLAN = {
     "site_name": "Planta Norte", "customer_id": None, "customer_name": None, "location_description": "",
     "interval": 3, "unit": "months", "last_run": None, "next_run": "2026-11-01T08:00:00-05:00",
     "wo_title": "Lubricar rodamientos Bomba 4", "wo_description": "", "wo_priority": "medium",
-    "assigned_to_id": None, "assigned_to_name": None, "assigned_to_avatar": None, "estimated_hours": None,
-    "checklist": [], "images": [], "is_active": True, "created_by_id": 1,
+    "assigned_to_id": None, "assigned_to_name": None, "assigned_to_avatar": None, "assigned_company": None,
+    "estimated_hours": None, "checklist": [], "images": [], "is_active": True, "created_by_id": 1,
     "created_at": "2026-10-23T02:34:02.203781-05:00", "updated_at": "2026-10-23T02:34:02.203781-05:00",
     "last_execution": None,
 }
+# The same plan while a linked contractor has it: nobody of the company is on it.
+HELD_BY_CONTRACTOR = {**PLAN, "assigned_company": {"name": "Climas del Valle", "kind": "contractor"}}
 # GET /v1/pm/schedules/1/notes/ (preventive_maintenance/views.py PMScheduleNoteView)
 NOTES = [{"id": 1, "body": "Usar grasa EP2.", "author_type": "staff", "author_name": "Admin Smoke",
           "created_at": "2026-10-23T07:34:02.284143Z"}]
@@ -70,6 +72,15 @@ def test_ver_json_is_the_plan_as_it_came():
     assert json.loads(run(fake, "preventivos", "ver", "1", "--json").output) == PLAN
 
 
+def test_a_plan_a_contractor_has_names_the_contractor():
+    listed = run(FakeApi({("GET", "pm/schedules/"): page(HELD_BY_CONTRACTOR)}), "preventivos", "listar")
+    shown = run(FakeApi({("GET", "pm/schedules/1/"): HELD_BY_CONTRACTOR, ("GET", "pm/schedules/1/notes/"): []}),
+                "preventivos", "ver", "1")
+    assert listed.exit_code == 0, listed.output
+    assert shown.exit_code == 0, shown.output
+    assert "Climas del Valle" in listed.output and "Climas del Valle" in shown.output
+
+
 def test_crear_sends_the_plan_at_the_machine_s_place():
     fake = FakeApi({
         ("GET", "assets/"): page(ASSET),

{operx_cli-0.7.7 → operx_cli-0.8.1}/tests/test_sst_ambiental.py

@@ -52,9 +52,6 @@ STREAMS = {
     "moving_average_kg": None, "storage_alert_months": [], "certificate_days": 30,
     "streams": [{"id": 11, "name": "Aceite usado", "kind": "hazardous", "hazardous": True}],
 }
-METERS = [{"id": 21, "name": "Acueducto", "meter_name": "Contador principal", "qr_code": "x", "establishment": "Planta Norte"}]
-
-
 @pytest.fixture(autouse=True)
 def signed_in(monkeypatch):
     monkeypatch.setenv("OPERX_TOKEN", "opx_test")
@@ -224,20 +221,6 @@ def test_ambiental_residuo_finds_the_stream_by_its_name():
     assert fake.sent("POST", "environment/companies/1/waste/entries/") == {"stream": 11, "date": "2026-10-22", "kg": "12.5"}
 
 
-def test_ambiental_lectura_of_a_meter():
-    fake = FakeApi({
-        ("GET", "environment/companies/"): ENV_COMPANIES,
-        ("GET", "environment/companies/1/consumption/meters/"): METERS,
-        ("POST", "environment/companies/1/consumption/readings/"): (201, {"id": 3}),
-    })
-    result = run(fake, "ambiental", "lectura", "--medidor", "Acueducto", "--valor", "1.234,5", "--fecha", "22/10/2026", "--si")
-
-    assert result.exit_code == 0, result.output
-    assert fake.sent("POST", "environment/companies/1/consumption/readings/") == {
-        "source": 21, "read_on": "2026-10-22", "value": "1234.5",
-    }
-
-
 # ── Personas ──────────────────────────────────────────────────────────────