makea-cli 0.2.26__tar.gz → 0.2.27__tar.gz

{makea_cli-0.2.26 → makea_cli-0.2.27}/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## 0.2.27
+
+Notes and misfires on the Makea task queue.
+
+- `admin update task <id> --note "…"` alone adds a note (who was asked, what
+  the team is waiting on) and leaves the task open.
+- `admin update task <id> --dismiss --wrong --note "…"`: the task should never
+  have been raised, and the note says why. On a lesson Beeboo held, its next
+  reading of the chats is told and raises nothing on the same message again.
+- `admin list task --wrong`: every task marked wrong, newest first, with why.
+
+The same queue is on the admin page under Makea tasks.
+
 ## 0.2.26
 
 The admin page is now the CLI's companion. Every `makea-cli admin …` result

{makea_cli-0.2.26 → makea_cli-0.2.27}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: makea-cli
-Version: 0.2.26
+Version: 0.2.27
 Summary: Makea admin CLI (browser login + Makea admin HTTP API)
 Requires-Python: >=3.10
 Requires-Dist: click<9,>=8.1
@@ -152,7 +152,7 @@ exit code：`0` 成功 · `2` 用法错 · `3` 未登录或身份与 persona 不
 | `makea-cli admin list shipping-order` | read | `GET /admin/shipping_orders` | paginated list of all shipping orders (admin). | `list-all-shipping-orders` |
 | `makea-cli admin list supplier` | read | `GET /admin/supplier/get_all_suppliers` | list all suppliers (admin). | `list-suppliers` |
 | `makea-cli admin list supplier-link` | read | `GET /admin/product/linked_suppliers/by_supplier/{supplier_id}` | direct backend query. | `get-supplier-links-by-supplier-id` |
-| `makea-cli admin list task` | read | `GET /admin/tasks` | the Makea team's open tasks (Beeboo's notes, held lessons, sample price flags, PP samples, brand asks) by kind; `--user` adds that account's own `account.*` cards. | — |
+| `makea-cli admin list task` | read | `GET /admin/tasks` | the Makea team's open tasks (Beeboo's notes, held lessons, sample price flags, PP samples, brand asks) by kind, with notes; `--user` adds that account's own `account.*` cards; `--wrong` lists the tasks marked wrong instead. | — |
 | `makea-cli admin list user` | read | `GET /admin/users` | list registered users (admin, optional pagination). | `list-users` |
 | `makea-cli admin send email` | write（ask） | `POST /admin/email/send` | queue an HTML email (To required, Cc optional). From must be @makea.co; HTML from --html-file or stdin ('-'). Require… | `send-email` |
 | `makea-cli admin set pricing-groups` | write（ask） | `POST /admin/production_orders/{id}/details/update` | Set a production order's pricing groups from a JSON file, copy them from the order's quote, or clear them (--clear) t… | `production-groups-set` |
@@ -175,7 +175,7 @@ exit code：`0` 成功 · `2` 用法错 · `3` 未登录或身份与 persona 不
 | `makea-cli admin update sampling-order` | write（ask） | `PUT /admin/sampling_orders/{sampling_order_id}/status` | move a sampling order to another status (DRAFT, IN_PROGRESS, SAMPLING_COMPLETED, APPROVAL, SHIPPED, DELIVERED, CANCELLED). PAID is recorded by the payment and is refused here. Prefer admin update product-step, which moves the same statuses through the step model. | — |
 | `makea-cli admin update shipping-order` | write（ask） | `PUT /admin/shipping_orders/{shipping_order_id}/status` | Change a shipment: --status (PUT .../status), tracking flags (POST .../tracking), --address (PUT .../shipping-address… | — |
 | `makea-cli admin update supplier` | write（ask） | `PUT /admin/supplier/update_supplier` | update an existing supplier from a JSON file (admin). Run `makea-cli get-supplier-profile-schema` FIRST to get the fi… | `update-supplier` |
-| `makea-cli admin update task` | write（ask） | `POST /admin/tasks/{task_id}/{done\|dismiss}` | close a Makea task with a note: `--done` once handled (a held lesson's `--reason` goes to Beeboo's next reading), or `--dismiss`. | — |
+| `makea-cli admin update task` | write（ask） | `POST /admin/tasks/{task_id}/{done\|dismiss\|note}` | close a Makea task with a note: `--done` once handled (a held lesson's `--reason` goes to Beeboo's next reading), `--dismiss`, or `--dismiss --wrong` when it should never have been raised; `--note` alone adds a note and leaves it open. | — |
 | `makea-cli admin update tech-spec` | write（ask） | `POST /admin/products/{id}/technical-specifications` | persist specs from a JSON file. | `save-product-tech-spec` |
 | `makea-cli admin update tracking-status` | write（ask） | `POST /admin/tracking/carrier_status` | apply a carrier status looked up by hand (tracking-sweep skill) to one order: status, fan-out, delivered emails (--silent: state only). | — |
 | `makea-cli admin update user` | write（ask） | `PUT /admin/user-profiles/{user_id}/update` | admin updates DynamoDB user profile fields and/or Cognito custom:userType. Allowed --user-type values: "Fashion brand… | `update-user-profile` |

{makea_cli-0.2.26 → makea_cli-0.2.27}/README.md

@@ -140,7 +140,7 @@ exit code：`0` 成功 · `2` 用法错 · `3` 未登录或身份与 persona 不
 | `makea-cli admin list shipping-order` | read | `GET /admin/shipping_orders` | paginated list of all shipping orders (admin). | `list-all-shipping-orders` |
 | `makea-cli admin list supplier` | read | `GET /admin/supplier/get_all_suppliers` | list all suppliers (admin). | `list-suppliers` |
 | `makea-cli admin list supplier-link` | read | `GET /admin/product/linked_suppliers/by_supplier/{supplier_id}` | direct backend query. | `get-supplier-links-by-supplier-id` |
-| `makea-cli admin list task` | read | `GET /admin/tasks` | the Makea team's open tasks (Beeboo's notes, held lessons, sample price flags, PP samples, brand asks) by kind; `--user` adds that account's own `account.*` cards. | — |
+| `makea-cli admin list task` | read | `GET /admin/tasks` | the Makea team's open tasks (Beeboo's notes, held lessons, sample price flags, PP samples, brand asks) by kind, with notes; `--user` adds that account's own `account.*` cards; `--wrong` lists the tasks marked wrong instead. | — |
 | `makea-cli admin list user` | read | `GET /admin/users` | list registered users (admin, optional pagination). | `list-users` |
 | `makea-cli admin send email` | write（ask） | `POST /admin/email/send` | queue an HTML email (To required, Cc optional). From must be @makea.co; HTML from --html-file or stdin ('-'). Require… | `send-email` |
 | `makea-cli admin set pricing-groups` | write（ask） | `POST /admin/production_orders/{id}/details/update` | Set a production order's pricing groups from a JSON file, copy them from the order's quote, or clear them (--clear) t… | `production-groups-set` |
@@ -163,7 +163,7 @@ exit code：`0` 成功 · `2` 用法错 · `3` 未登录或身份与 persona 不
 | `makea-cli admin update sampling-order` | write（ask） | `PUT /admin/sampling_orders/{sampling_order_id}/status` | move a sampling order to another status (DRAFT, IN_PROGRESS, SAMPLING_COMPLETED, APPROVAL, SHIPPED, DELIVERED, CANCELLED). PAID is recorded by the payment and is refused here. Prefer admin update product-step, which moves the same statuses through the step model. | — |
 | `makea-cli admin update shipping-order` | write（ask） | `PUT /admin/shipping_orders/{shipping_order_id}/status` | Change a shipment: --status (PUT .../status), tracking flags (POST .../tracking), --address (PUT .../shipping-address… | — |
 | `makea-cli admin update supplier` | write（ask） | `PUT /admin/supplier/update_supplier` | update an existing supplier from a JSON file (admin). Run `makea-cli get-supplier-profile-schema` FIRST to get the fi… | `update-supplier` |
-| `makea-cli admin update task` | write（ask） | `POST /admin/tasks/{task_id}/{done\|dismiss}` | close a Makea task with a note: `--done` once handled (a held lesson's `--reason` goes to Beeboo's next reading), or `--dismiss`. | — |
+| `makea-cli admin update task` | write（ask） | `POST /admin/tasks/{task_id}/{done\|dismiss\|note}` | close a Makea task with a note: `--done` once handled (a held lesson's `--reason` goes to Beeboo's next reading), `--dismiss`, or `--dismiss --wrong` when it should never have been raised; `--note` alone adds a note and leaves it open. | — |
 | `makea-cli admin update tech-spec` | write（ask） | `POST /admin/products/{id}/technical-specifications` | persist specs from a JSON file. | `save-product-tech-spec` |
 | `makea-cli admin update tracking-status` | write（ask） | `POST /admin/tracking/carrier_status` | apply a carrier status looked up by hand (tracking-sweep skill) to one order: status, fan-out, delivered emails (--silent: state only). | — |
 | `makea-cli admin update user` | write（ask） | `PUT /admin/user-profiles/{user_id}/update` | admin updates DynamoDB user profile fields and/or Cognito custom:userType. Allowed --user-type values: "Fashion brand… | `update-user-profile` |

makea_cli-0.2.27/makea_cli/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.2.27"

{makea_cli-0.2.26 → makea_cli-0.2.27}/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-0.2.26 → makea_cli-0.2.27}/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")

{makea_cli-0.2.26 → makea_cli-0.2.27}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "makea-cli"
-version = "0.2.26"
+version = "0.2.27"
 description = "Makea admin CLI (browser login + Makea admin HTTP API)"
 readme = "README.md"
 requires-python = ">=3.10"

{makea_cli-0.2.26 → makea_cli-0.2.27}/tests/golden_manifest.json

@@ -3874,7 +3874,7 @@
       "idempotent": null,
       "limit": "client",
       "legacy": [],
-      "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.*).",
+      "help": "GET /admin/tasks — the Makea team's open tasks: task_id, type, kind, title, 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.",
       "function": "makea_cli.commands.tasks.cmd.register.<locals>.list_tasks",
       "flags": [
         {
@@ -3927,6 +3927,18 @@
           "multiple": false,
           "default": null,
           "help": "One product id."
+        },
+        {
+          "name": "wrong",
+          "kind": "option",
+          "opts": [
+            "--wrong"
+          ],
+          "type": "flag",
+          "required": false,
+          "multiple": false,
+          "default": false,
+          "help": "The tasks marked wrong (closed), with why, newest first."
         }
       ]
     },
@@ -6692,18 +6704,20 @@
       "verb": "update",
       "noun": "task",
       "access": "write",
-      "api": "POST /admin/tasks/{task_id}/{done|dismiss}",
+      "api": "POST /admin/tasks/{task_id}/{done|dismiss|note}",
       "external": null,
       "side_effects": [
-        "closes the task for the whole Makea team",
-        "with --reason, Beeboo's next reading of the chats uses that reason"
+        "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,
       "limit": null,
       "legacy": [],
-      "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.",
+      "help": "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.",
       "function": "makea_cli.commands.tasks.cmd.register.<locals>.update_task",
       "flags": [
         {
@@ -6742,6 +6756,18 @@
           "default": false,
           "help": "It will not be done."
         },
+        {
+          "name": "wrong",
+          "kind": "option",
+          "opts": [
+            "--wrong"
+          ],
+          "type": "flag",
+          "required": false,
+          "multiple": false,
+          "default": false,
+          "help": "With --dismiss: it should never have been raised; --note says why."
+        },
         {
           "name": "note",
           "kind": "option",
@@ -6753,7 +6779,7 @@
           "required": true,
           "multiple": false,
           "default": null,
-          "help": "What was done, or why not (required)."
+          "help": "What was done, or why not (required). Alone: a note; the task stays open."
         },
         {
           "name": "reason",

{makea_cli-0.2.26 → makea_cli-0.2.27}/tests/test_persona_commands.py

@@ -677,8 +677,27 @@ class TaskCommandTests(_PersonaCase):
         self.run_cli("admin", "update", "task", "t1", "--dismiss", "--note", "keeps GBP")
         self.assertEqual(self.rec.last["path"], "/admin/tasks/t1/dismiss")
 
+    def test_a_note_alone_leaves_the_task_open(self):
+        self.run_cli("admin", "update", "task", "t1", "--note", " asked,  waiting ")
+        self.assertEqual(
+            (self.rec.last["path"], self.rec.last["body"]),
+            ("/admin/tasks/t1/note", {"note": "asked, waiting"}),
+        )
+
+    def test_wrong_dismisses_with_why_and_lists(self):
+        self.run_cli("admin", "update", "task", "t1", "--dismiss", "--wrong", "-n", "misread")
+        self.assertEqual(
+            (self.rec.last["path"], self.rec.last["body"]),
+            ("/admin/tasks/t1/dismiss", {"note": "misread", "wrong": True}),
+        )
+        self.run_cli("admin", "list", "task", "--wrong")
+        self.assertEqual(self.rec.last["params"], {"wrong": "1"})
+
     def test_refused_before_the_backend(self):
-        self.run_cli("admin", "update", "task", "t1", "--note", "x", expect=2)
+        self.run_cli("admin", "update", "task", "t1", "--wrong", "-n", "x", expect=2)
+        self.run_cli("admin", "update", "task", "t1", "--done", "--wrong", "-n", "x", expect=2)
+        self.run_cli("admin", "update", "task", "t1", "-n", "x", "--reason", "r", expect=2)
+        self.run_cli("admin", "list", "task", "--wrong", "--kind", "once", expect=2)
         self.run_cli("admin", "update", "task", "t1", "--done", "--dismiss", "-n", "x", expect=2)
         self.run_cli("admin", "update", "task", "t1", "--done", expect=2)
         self.run_cli(

makea_cli-0.2.26/makea_cli/__init__.py

@@ -1 +0,0 @@
-__version__ = "0.2.26"