makea-cli 0.2.26__py3-none-any.whl → 0.2.28__py3-none-any.whl

makea_cli/__init__.py

@@ -1 +1 @@
-__version__ = "0.2.26"
+__version__ = "0.2.28"

makea_cli/api_client.py

@@ -1195,8 +1195,12 @@ def admin_list_tasks(
     type_: str | None = None,
     user: str | None = None,
     product: str | None = None,
+    wrong: bool = False,
 ) -> Any:
-    """GET /admin/tasks — the team's open tasks; filters are comma-separated."""
+    """GET /admin/tasks — the team's open tasks; filters are comma-separated.
+    ``wrong``: the tasks the team marked wrong instead, newest first."""
+    if wrong:
+        return get_json("/admin/tasks", params={"wrong": "1"})
     params = {
         k: v
         for k, v in (("kind", kind), ("type", type_), ("user", user), ("product", product))
@@ -1211,7 +1215,8 @@ def admin_get_task(task_id: str) -> Any:
 
 
 def admin_close_task(task_id: str, verb: str, body: dict[str, Any]) -> Any:
-    """POST /admin/tasks/{task_id}/{done|dismiss} — {note, reason?, caused_by?, recurs?}."""
+    """POST /admin/tasks/{task_id}/{done|dismiss|note} — {note, reason?, caused_by?,
+    recurs?, wrong?}. ``note`` adds a note and leaves the task open."""
     return post_json(f"/admin/tasks/{quote(task_id, safe='')}/{verb}", body)
 
 

makea_cli/commands/__init__.py

@@ -11,6 +11,7 @@ from makea_cli.commands.directory.cmd import register as register_directory
 from makea_cli.commands.document.cmd import register as register_document
 from makea_cli.commands.email.cmd import register as register_email
 from makea_cli.commands.factory.cmd import register as register_factory
+from makea_cli.commands.files.cmd import register as register_files
 from makea_cli.commands.flags.cmd import register as register_flags
 from makea_cli.commands.library.cmd import register as register_library
 from makea_cli.commands.metrics.cmd import register as register_metrics
@@ -38,6 +39,7 @@ def register_all(app: typer.Typer) -> None:
     register_email(app)
     register_library(app)
     register_factory(app)
+    register_files(app)
     register_brand(app)
     register_playbook(app)
     register_about(app)

makea_cli/commands/files/cmd.py

@@ -0,0 +1,84 @@
+"""``makea-cli <persona> get file PATH``: read a file a brand sent, locally.
+
+Pattern DXF/RUL, Illustrator, EPS, Photoshop, embroidery (PES/DST/EMB), ZIP,
+AVIF/TIFF/SVG. No API call: the file never leaves the machine. Pictures of what
+the file draws are saved next to it so the caller can look at them (an agent
+reads them with its image tool).
+"""
+
+from __future__ import annotations
+
+import re
+from pathlib import Path
+
+import typer
+from rich.console import Console
+
+from makea_cli.commands._util import print_json_or_exit
+from makea_cli.file_reader import read_path
+from makea_cli.registry import command
+
+console = Console()
+
+_HELP = (
+    "Read a pattern (DXF/RUL), artwork (AI/EPS/PSD/SVG), embroidery (PES/DST/EMB), ZIP or "
+    "AVIF/TIFF file on this machine: what it states plus PNG pictures of what it draws."
+)
+
+
+def _slug(text: str) -> str:
+    return re.sub(r"[^A-Za-z0-9]+", "-", text).strip("-").lower()[:40] or "picture"
+
+
+def read_file_payload(path: Path, pictures_dir: Path | None) -> dict:
+    if not path.is_file():
+        raise FileNotFoundError(f"{path} is not a file")
+    reading = read_path(str(path))
+    out_dir = pictures_dir or path.parent
+    saved = []
+    pictures = reading.all_pictures()
+    if pictures:
+        out_dir.mkdir(parents=True, exist_ok=True)
+    for i, pic in enumerate(pictures, 1):
+        target = out_dir / f"{path.stem}.{i}-{_slug(pic.label)}.png"
+        target.write_bytes(pic.png)
+        saved.append({"label": pic.label, "path": str(target)})
+    payload = {
+        "file": str(path),
+        "kind": reading.kind,
+        "format": reading.format,
+        "readable": reading.readable,
+        "readout": reading.markdown(),
+        "pictures": saved,
+    }
+    if saved:
+        payload["next"] = (
+            "Look at each picture to read what it shows; the readout lists only stated facts."
+        )
+    return payload
+
+
+def register(app: typer.Typer) -> None:
+    for persona in ("designer", "supplier", "admin"):
+        _register_for(app, persona)
+
+
+def _register_for(app: typer.Typer, persona: str) -> None:
+    @command(
+        app,
+        persona,
+        "get",
+        "file",
+        api="LOCAL file_reader",
+        external="local",
+        help=_HELP,
+    )
+    def get_file(
+        path: Path = typer.Argument(..., help="The file to read."),
+        pictures_dir: Path | None = typer.Option(
+            None,
+            "--pictures-dir",
+            help="Where to save the PNG pictures (default: next to the file).",
+        ),
+    ) -> None:
+        print_json_or_exit(console, lambda: read_file_payload(path, pictures_dir))

makea_cli/commands/tasks/cmd.py

@@ -12,7 +12,10 @@ read as one order's problem), ``note`` (a slip of Makea's or the platform's),
 
 The change a task asks for is made with the command that owns it (``admin
 create rule``, ``admin update user``, ``admin update about`` …); ``update
-task --done`` then records that it was handled. Only notes the team handles
+task --done`` then records that it was handled. ``update task --note`` alone
+adds a note and leaves the task open; ``--dismiss --wrong`` says the task
+should never have been raised (Beeboo's next reading is told, and ``list
+task --wrong`` lists every such task). Only notes the team handles
 off the platform can be marked done; the others close by themselves when the
 fact behind them lands, and ``get task`` says which fact.
 """
@@ -44,6 +47,10 @@ Examples
   makea-cli admin update task <task_id> --done --note "Asked the brand on WhatsApp" \\
       --reason "The brand changed the cuff after approval" --caused-by brand --once
   makea-cli admin update task <task_id> --dismiss --note "Brand keeps GBP"
+  makea-cli admin update task <task_id> --note "Asked the brand, waiting"
+  makea-cli admin update task <task_id> --dismiss --wrong \
+      --note "The factory was confirming the file, not reporting a mistake"
+  makea-cli admin list task --wrong
 """.strip()
 
 
@@ -56,8 +63,9 @@ def register(app: typer.Typer) -> None:
         api="GET /admin/tasks",
         help=(
             "GET /admin/tasks — the Makea team's open tasks: task_id, type, kind, title, "
-            "owner (the account), product, due date, can_mark_done; counts by kind. "
-            "--user adds that account's own cards the team may settle (account.*)."
+            "owner (the account), product, due date, can_mark_done, notes; counts by kind. "
+            "--user adds that account's own cards the team may settle (account.*). "
+            "--wrong lists the tasks the team marked wrong instead, newest first."
         ),
         epilog=_TASKS_EPILOG,
     )
@@ -72,11 +80,18 @@ def register(app: typer.Typer) -> None:
             None, "--user", "-u", help="One account: user_id or email."
         ),
         product: str | None = typer.Option(None, "--product", "-p", help="One product id."),
+        wrong: bool = typer.Option(
+            False, "--wrong", help="The tasks marked wrong (closed), with why, newest first."
+        ),
     ) -> None:
-        """List the Makea team's open tasks."""
+        """List the Makea team's open tasks, or the ones marked wrong."""
+        if wrong and (kind or type_ or user or product):
+            raise typer.BadParameter("--wrong takes no filters")
         print_json_or_exit(
             console,
-            lambda: api_client.admin_list_tasks(kind=kind, type_=type_, user=user, product=product),
+            lambda: api_client.admin_list_tasks(
+                kind=kind, type_=type_, user=user, product=product, wrong=wrong
+            ),
         )
 
     @command(
@@ -101,19 +116,23 @@ def register(app: typer.Typer) -> None:
         "admin",
         "update",
         "task",
-        api="POST /admin/tasks/{task_id}/{done|dismiss}",
+        api="POST /admin/tasks/{task_id}/{done|dismiss|note}",
         side_effects=[
-            "closes the task for the whole Makea team",
+            "with --done or --dismiss, closes the task for the whole Makea team",
+            "with --note alone, adds a note the team sees; the task stays open",
             "with --reason, Beeboo's next reading of the chats uses that reason",
+            "with --wrong, Beeboo's next reading is told the finding was wrong",
         ],
         approval="ask",
         reversible=False,
         idempotent=False,
         help=(
-            "POST /admin/tasks/{task_id}/{done|dismiss} — close a task: --done once it is "
-            "handled (notes the team handles off the platform only), --dismiss when it "
-            "will not be done. --note required. On a held lesson, --reason records why "
-            "it happened for Beeboo's next reading."
+            "POST /admin/tasks/{task_id}/{done|dismiss|note} — close a task: --done once "
+            "it is handled (notes the team handles off the platform only), --dismiss when "
+            "it will not be done, --dismiss --wrong when it should never have been raised "
+            "(the note says why). --note required; --note alone adds a note and leaves the "
+            "task open. On a held lesson, --reason records why it happened for Beeboo's "
+            "next reading."
         ),
         epilog=_TASKS_EPILOG,
     )
@@ -121,7 +140,17 @@ def register(app: typer.Typer) -> None:
         task_id: str = typer.Argument(..., help="Task id (from list task)."),
         done: bool = typer.Option(False, "--done", help="It was handled."),
         dismiss: bool = typer.Option(False, "--dismiss", help="It will not be done."),
-        note: str = typer.Option(..., "--note", "-n", help="What was done, or why not (required)."),
+        wrong: bool = typer.Option(
+            False,
+            "--wrong",
+            help="With --dismiss: it should never have been raised; --note says why.",
+        ),
+        note: str = typer.Option(
+            ...,
+            "--note",
+            "-n",
+            help="What was done, or why not (required). Alone: a note; the task stays open.",
+        ),
         reason: str | None = typer.Option(
             None,
             "--reason",
@@ -136,13 +165,22 @@ def register(app: typer.Typer) -> None:
             help="With --reason: it will happen again, or it was this order only.",
         ),
     ) -> None:
-        """Mark a task done, or dismiss it."""
-        if done == dismiss:
-            raise typer.BadParameter("pass exactly one of --done, --dismiss")
+        """Mark a task done, dismiss it, or add a note."""
+        if done and dismiss:
+            raise typer.BadParameter("pass at most one of --done, --dismiss")
+        if wrong and not dismiss:
+            raise typer.BadParameter("--wrong goes with --dismiss")
         clean_note = " ".join(note.split())
         if not clean_note:
             raise typer.BadParameter("--note must not be empty")
         body: dict[str, Any] = {"note": clean_note}
+        if not (done or dismiss):
+            if reason is not None or caused_by is not None or recurs is not None:
+                raise typer.BadParameter("--reason, --caused-by, --recurs/--once go with --done")
+            print_json_or_exit(console, lambda: api_client.admin_close_task(task_id, "note", body))
+            return
+        if wrong:
+            body["wrong"] = True
         if reason is not None:
             if not done:
                 raise typer.BadParameter("--reason goes with --done")