operx-cli 0.9.0__tar.gz → 0.10.0__tar.gz

{operx_cli-0.9.0 → operx_cli-0.10.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: operx-cli
-Version: 0.9.0
+Version: 0.10.0
 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
@@ -184,6 +184,10 @@ operx campos listar                                         # la información ex
 operx campos crear "Hijos" --tipo numero
 operx campos guardar "Ana Ruiz" --valor "Hijos=2" --valor "Fecha de nacimiento=17/05/1990"
 operx campos ver "Ana Ruiz"
+operx campos listar --de equipos                            # la información extra de los equipos
+operx campos crear "Voltaje" --tipo numero --de equipos
+operx campos guardar BOM-01 --valor "Voltaje=220" --de equipos
+operx campos ver BOM-01 --de equipos
 
 operx sst acciones --estado abiertas
 operx sst nueva-accion --titulo "Señalizar la salida" --para 15/11/2026 --responsable "Ana Ruiz"

{operx_cli-0.9.0 → operx_cli-0.10.0}/README.md

@@ -170,6 +170,10 @@ operx campos listar                                         # la información ex
 operx campos crear "Hijos" --tipo numero
 operx campos guardar "Ana Ruiz" --valor "Hijos=2" --valor "Fecha de nacimiento=17/05/1990"
 operx campos ver "Ana Ruiz"
+operx campos listar --de equipos                            # la información extra de los equipos
+operx campos crear "Voltaje" --tipo numero --de equipos
+operx campos guardar BOM-01 --valor "Voltaje=220" --de equipos
+operx campos ver BOM-01 --de equipos
 
 operx sst acciones --estado abiertas
 operx sst nueva-accion --titulo "Señalizar la salida" --para 15/11/2026 --responsable "Ana Ruiz"

{operx_cli-0.9.0 → operx_cli-0.10.0}/operx_cli/AGENTES.md

@@ -110,6 +110,12 @@ llamada queda en el historial del token, marcada como hecha desde la CLI.
   los campos que digas; `--vaciar Hijos` deja uno vacío) y `campos ver`. Antes
   de crear un campo, mira que no exista ya. `campos quitar` lo oculta de todas
   las personas y deja los valores guardados.
+- Lo que la ficha de un equipo no tiene (voltaje, capacidad, última
+  inspección…) va igual, con `--de equipos` en cada comando: `campos listar
+  --de equipos`, `campos crear "Voltaje" --tipo numero --de equipos`, `campos
+  guardar BOM-01 --valor "Voltaje=220" --de equipos`. Personas y equipos tienen
+  campos aparte. Los clientes ven estos valores en su portal: no guardes ahí lo
+  que no deban leer.
 - SG-SST y Ambiental van por empresa: sin `--empresa`, la de la cuenta. Una
   cuenta que lleva el SG-SST o lo ambiental de sus clientes ve las demás con
   `operx sst empresas` o `operx ambiental empresas`.

{operx_cli-0.9.0 → operx_cli-0.10.0}/operx_cli/__init__.py

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

{operx_cli-0.9.0 → operx_cli-0.10.0}/operx_cli/commands/campos.py

@@ -1,9 +1,12 @@
 """`operx campos`: Información extra, the fields a company adds to the record
-of its people (a birthday, the children, the marital status...) for what the
-record doesn't have, and their values on each person
-(backend/extra_fields, backend/people/extra_info.py). The fields are
-company-wide; each person has a value or none for every one."""
-
+of its people (a birthday, the children, the marital status...) or of its
+machines (the voltage, the last inspection...) for what the record doesn't
+have, and their values on each one (backend/extra_fields,
+backend/people/extra_info.py, backend/assets/extra_info.py). The fields are
+company-wide; each record has a value or none for every one. `--de` says which
+kind of record: people unless it says otherwise."""
+
+from dataclasses import dataclass
 from decimal import Decimal
 from enum import Enum
 from typing import Annotated, Optional
@@ -16,12 +19,42 @@ from ..values import decimal_text, iso_date
 from .common import CliError, CsvOption, JsonOption, YesOption, client_of, confirm_write, output_format, print_result
 
 app = typer.Typer(
-    help="Información extra: los campos que tu empresa agrega a la ficha de las personas, y sus valores.",
+    help="Información extra: los campos que tu empresa agrega a personas y equipos, y sus valores.",
     no_args_is_help=True,
 )
 
-# The kinds of record that can have extra fields; for now, people.
-ENTITY = "employee"
+
+class Registro(str, Enum):
+    personas = "personas"
+    equipos = "equipos"
+
+
+@dataclass(frozen=True)
+class RecordKind:
+    """How a kind of record is found and where its values are."""
+
+    # A key of backend/extra_fields/entities.py.
+    entity: str
+    everyone: str
+    list_path: str
+    lookup_fields: tuple
+    what: str
+    name_field: str
+    values_path: str
+
+
+KINDS = {
+    Registro.personas: RecordKind(
+        entity="employee", everyone="todas las personas", list_path="employees",
+        lookup_fields=("user_name", "employee_code"), what="persona", name_field="user_name",
+        values_path="people/employees/{id}/extra-fields",
+    ),
+    Registro.equipos: RecordKind(
+        entity="equipment", everyone="todos los equipos", list_path="assets",
+        lookup_fields=("code", "name"), what="equipo", name_field="name",
+        values_path="assets/{id}/extra-fields",
+    ),
+}
 
 FIELD_COLUMNS = [("id", "Id"), ("name", "Campo"), ("type", "Tipo"), ("options", "Opciones")]
 VALUE_COLUMNS = [("name", "Campo"), ("value", "Valor")]
@@ -50,10 +83,14 @@ OpcionOption = Annotated[
     Optional[list[str]],
     typer.Option("--opcion", help="Una opción de la lista; repítela para varias. Solo para el tipo lista."),
 ]
+DeOption = Annotated[Registro, typer.Option("--de", help="De qué registro son los campos: personas o equipos.")]
+RegistroArgument = Annotated[
+    str, typer.Argument(help="El nombre exacto, el código o el id de la persona o del equipo."),
+]
 
 
-def _fields(client):
-    return client.get("extra-fields", {"entity": ENTITY})
+def _fields(client, kind):
+    return client.get("extra-fields", {"entity": kind.entity})
 
 
 def _find_field(fields, text):
@@ -64,8 +101,8 @@ def _find_field(fields, text):
         raise CliError(str(error)) from None
 
 
-def _person(client, text):
-    return find_one(client, "employees", text, ["user_name", "employee_code"], "persona")
+def _record(client, kind, text):
+    return find_one(client, kind.list_path, text, list(kind.lookup_fields), kind.what)
 
 
 def _split_pair(pair):
@@ -123,12 +160,13 @@ def _shown(field):
 @app.command()
 def listar(
     ctx: typer.Context,
+    de: DeOption = Registro.personas,
     as_json: JsonOption = False,
     as_csv: CsvOption = False,
 ):
-    """Los campos de información extra de las personas: su nombre, su tipo y las opciones de las listas."""
+    """Los campos de información extra de las personas o de los equipos: su nombre, su tipo y las opciones de las listas."""
     output = output_format(as_json, as_csv)
-    fields = _fields(client_of(ctx))
+    fields = _fields(client_of(ctx), KINDS[de])
     if output != "json":
         fields = [
             {**field, "type": TYPE_LABELS.get(field["field_type"], field["field_type"]), "options": ", ".join(field["options"])}
@@ -143,16 +181,18 @@ def crear(
     nombre: Annotated[str, typer.Argument(help="El nombre del campo: «Fecha de nacimiento».")],
     tipo: Annotated[Tipo, typer.Option("--tipo", help="Qué guarda el campo; no se cambia después.")] = Tipo.texto,
     opcion: OpcionOption = None,
+    de: DeOption = Registro.personas,
     si: YesOption = False,
     as_json: JsonOption = False,
 ):
-    """Agrega un campo a la ficha de las personas. Todas las personas lo tienen, vacío hasta que se llene."""
+    """Agrega un campo a la ficha de las personas o de los equipos. Todos lo tienen, vacío hasta que se llene."""
     if tipo != Tipo.lista and opcion:
         raise CliError("--opcion solo va con --tipo lista.")
     if tipo == Tipo.lista and not opcion:
         raise CliError("Una lista necesita sus opciones: --opcion «Soltero/a» --opcion «Casado/a».")
-    body = {"entity": ENTITY, "name": nombre, "field_type": TYPE_KEYS[tipo.value], "options": opcion or []}
-    confirm_write(f"Agregar a las personas el campo «{nombre}» ({TYPE_LABELS[body['field_type']]}).", si)
+    kind = KINDS[de]
+    body = {"entity": kind.entity, "name": nombre, "field_type": TYPE_KEYS[tipo.value], "options": opcion or []}
+    confirm_write(f"Agregar a {kind.everyone} el campo «{nombre}» ({TYPE_LABELS[body['field_type']]}).", si)
     created = client_of(ctx).send("POST", "extra-fields", json=body).json()
     print_result(created, as_json, f"Campo {created['id']} creado: {created['name']}.")
 
@@ -163,6 +203,7 @@ def cambiar(
     campo: Annotated[str, typer.Argument(help="El id o el nombre exacto del campo.")],
     nombre: Annotated[Optional[str], typer.Option("--nombre", help="El nombre nuevo.")] = None,
     opcion: OpcionOption = None,
+    de: DeOption = Registro.personas,
     si: YesOption = False,
     as_json: JsonOption = False,
 ):
@@ -170,7 +211,7 @@ def cambiar(
     if not (nombre or opcion):
         raise CliError("Di qué cambiar: --nombre o --opcion.")
     client = client_of(ctx)
-    found = _find_field(_fields(client), campo)
+    found = _find_field(_fields(client, KINDS[de]), campo)
     body = {}
     if nombre:
         body["name"] = nombre
@@ -185,29 +226,35 @@ def cambiar(
 def quitar(
     ctx: typer.Context,
     campo: Annotated[str, typer.Argument(help="El id o el nombre exacto del campo.")],
+    de: DeOption = Registro.personas,
     si: YesOption = False,
     as_json: JsonOption = False,
 ):
-    """Quita un campo de la ficha de todas las personas. Los valores ya guardados quedan en la base de datos."""
+    """Quita un campo de la ficha de todas las personas o todos los equipos. Los valores ya guardados quedan en la base de datos."""
     client = client_of(ctx)
-    found = _find_field(_fields(client), campo)
-    confirm_write(f"Quitar el campo «{found['name']}» de todas las personas.", si)
+    kind = KINDS[de]
+    found = _find_field(_fields(client, kind), campo)
+    confirm_write(f"Quitar el campo «{found['name']}» de {kind.everyone}.", si)
     client.send("DELETE", f"extra-fields/{found['id']}")
-    print_result({"id": found["id"], "removed": True}, as_json, f"Campo {found['id']} quitado: {found['name']}.")
+    print_result(
+        {"id": found["id"], "removed": True}, as_json, f"Campo {found['id']} quitado de {kind.everyone}: {found['name']}.",
+    )
 
 
 @app.command()
 def ver(
     ctx: typer.Context,
-    persona: Annotated[str, typer.Argument(help="El nombre exacto, el código o el id de la persona.")],
+    registro: RegistroArgument,
+    de: DeOption = Registro.personas,
     as_json: JsonOption = False,
     as_csv: CsvOption = False,
 ):
-    """La información extra de una persona: cada campo con su valor."""
+    """La información extra de una persona o de un equipo: cada campo con su valor."""
     output = output_format(as_json, as_csv)
     client = client_of(ctx)
-    person = _person(client, persona)
-    answer = client.get(f"people/employees/{person['id']}/extra-fields")
+    kind = KINDS[de]
+    found = _record(client, kind, registro)
+    answer = client.get(kind.values_path.format(id=found["id"]))
     if output == "json":
         print_record(answer, "json")
         return
@@ -218,7 +265,7 @@ def ver(
 @app.command()
 def guardar(
     ctx: typer.Context,
-    persona: Annotated[str, typer.Argument(help="El nombre exacto, el código o el id de la persona.")],
+    registro: RegistroArgument,
     valor: Annotated[
         Optional[list[str]],
         typer.Option("--valor", help="Campo=valor, por ejemplo «Hijos=2» o «Fecha de nacimiento=17/05/1990»; repítelo para varios."),
@@ -226,14 +273,16 @@ def guardar(
     vaciar: Annotated[
         Optional[list[str]], typer.Option("--vaciar", help="El campo que se deja vacío; repítelo para varios."),
     ] = None,
+    de: DeOption = Registro.personas,
     si: YesOption = False,
     as_json: JsonOption = False,
 ):
-    """Guarda valores en la información extra de una persona. Solo cambian los campos que digas."""
+    """Guarda valores en la información extra de una persona o de un equipo. Solo cambian los campos que digas."""
     if not (valor or vaciar):
         raise CliError("Di qué guardar: --valor Campo=valor o --vaciar Campo.")
     client = client_of(ctx)
-    fields = _fields(client)
+    kind = KINDS[de]
+    fields = _fields(client, kind)
     values = {}
     for pair in valor or []:
         name, text = _split_pair(pair)
@@ -241,9 +290,9 @@ def guardar(
         values[str(field["id"])] = _value_for(field, text)
     for name in vaciar or []:
         values[str(_find_field(fields, name)["id"])] = None
-    person = _person(client, persona)
-    who = person.get("user_name") or persona
+    found = _record(client, kind, registro)
+    who = found.get(kind.name_field) or registro
     changes = list(valor or []) + [f"vaciar {name}" for name in vaciar or []]
     confirm_write(f"Guardar en la información extra de {who}: {'; '.join(changes)}.", si)
-    saved = client.send("PATCH", f"people/employees/{person['id']}/extra-fields", json={"values": values}).json()
+    saved = client.send("PATCH", kind.values_path.format(id=found["id"]), json={"values": values}).json()
     print_result(saved, as_json, f"Información extra de {who} guardada.")

{operx_cli-0.9.0 → operx_cli-0.10.0}/operx_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: operx-cli
-Version: 0.9.0
+Version: 0.10.0
 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
@@ -184,6 +184,10 @@ operx campos listar                                         # la información ex
 operx campos crear "Hijos" --tipo numero
 operx campos guardar "Ana Ruiz" --valor "Hijos=2" --valor "Fecha de nacimiento=17/05/1990"
 operx campos ver "Ana Ruiz"
+operx campos listar --de equipos                            # la información extra de los equipos
+operx campos crear "Voltaje" --tipo numero --de equipos
+operx campos guardar BOM-01 --valor "Voltaje=220" --de equipos
+operx campos ver BOM-01 --de equipos
 
 operx sst acciones --estado abiertas
 operx sst nueva-accion --titulo "Señalizar la salida" --para 15/11/2026 --responsable "Ana Ruiz"

{operx_cli-0.9.0 → operx_cli-0.10.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "operx-cli"
-version = "0.9.0"
+version = "0.10.0"
 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.9.0 → operx_cli-0.10.0}/tests/test_campos.py

@@ -163,3 +163,70 @@ def test_guardar_with_nothing_to_save_says_so():
     fake = FakeApi({})
     result = run(fake, "campos", "guardar", "María Torres", "--si")
     assert result.exit_code == 1 and "Di qué guardar" in result.output
+
+
+# --de equipos: the same commands on a machine's record.
+# extra_fields/serializers.py ExtraFieldSerializer
+VOLTAJE = {"id": 11, "entity": "equipment", "name": "Voltaje", "field_type": "number", "options": [], "position": 0}
+MACHINE_FIELDS = [VOLTAJE]
+# assets/serializers.py AssetListSerializer
+BOMBA = {"id": 40, "name": "Bomba 1", "code": "BOM-01"}
+# assets/extra_info.py extra_payload
+BOMBA_VALUES = {"fields": [{"id": 11, "name": "Voltaje", "field_type": "number", "options": [], "value": 220}]}
+
+
+def test_listar_de_equipos_asks_for_the_fields_of_the_machines():
+    fake = FakeApi({("GET", "extra-fields/"): MACHINE_FIELDS})
+    result = run(fake, "campos", "listar", "--de", "equipos")
+    assert result.exit_code == 0, result.output
+    assert "Voltaje" in result.output
+    assert fake.requests[0].url.params["entity"] == "equipment"
+
+
+def test_crear_de_equipos_adds_the_field_to_the_machines():
+    fake = FakeApi({("POST", "extra-fields/"): (201, VOLTAJE)})
+    result = run(fake, "campos", "crear", "Voltaje", "--tipo", "numero", "--de", "equipos", "--si")
+    assert result.exit_code == 0, result.output
+    assert fake.sent("POST", "extra-fields/") == {"entity": "equipment", "name": "Voltaje", "field_type": "number", "options": []}
+
+
+def test_cambiar_de_equipos_finds_the_field_among_those_of_the_machines():
+    fake = FakeApi({("GET", "extra-fields/"): MACHINE_FIELDS, ("PATCH", "extra-fields/11/"): VOLTAJE})
+    result = run(fake, "campos", "cambiar", "Voltaje", "--nombre", "Voltaje nominal", "--de", "equipos", "--si")
+    assert result.exit_code == 0, result.output
+    assert fake.requests[0].url.params["entity"] == "equipment"
+    assert fake.sent("PATCH", "extra-fields/11/") == {"name": "Voltaje nominal"}
+
+
+def test_quitar_de_equipos_says_it_leaves_every_machine():
+    fake = FakeApi({("GET", "extra-fields/"): MACHINE_FIELDS, ("DELETE", "extra-fields/11/"): (204, None)})
+    result = run(fake, "campos", "quitar", "Voltaje", "--de", "equipos", "--si")
+    assert result.exit_code == 0, result.output
+    assert "todos los equipos" in result.output
+    assert fake.called("DELETE", "extra-fields/11/")
+
+
+def test_ver_de_equipos_reads_the_machine_by_its_code():
+    fake = FakeApi({("GET", "assets/"): page(BOMBA), ("GET", "assets/40/extra-fields/"): BOMBA_VALUES})
+    result = run(fake, "campos", "ver", "BOM-01", "--de", "equipos")
+    assert result.exit_code == 0, result.output
+    assert "Voltaje" in result.output and "220" in result.output
+
+
+def test_guardar_de_equipos_saves_on_the_machine():
+    fake = FakeApi({
+        ("GET", "extra-fields/"): MACHINE_FIELDS,
+        ("GET", "assets/"): page(BOMBA),
+        ("PATCH", "assets/40/extra-fields/"): BOMBA_VALUES,
+    })
+    result = run(fake, "campos", "guardar", "Bomba 1", "--valor", "Voltaje=220", "--de", "equipos", "--si")
+    assert result.exit_code == 0, result.output
+    assert "Bomba 1" in result.output
+    assert fake.sent("PATCH", "assets/40/extra-fields/") == {"values": {"11": 220}}
+
+
+def test_de_refuses_a_kind_of_record_that_has_no_fields():
+    fake = FakeApi({})
+    result = run(fake, "campos", "listar", "--de", "ordenes")
+    assert result.exit_code != 0
+    assert fake.requests == []