From ac542f4d728e9bc7f024dfeba1f7681d9fc4a88d Mon Sep 17 00:00:00 2001
From: Josh Mabry <mabry1985@gmail.com>
Date: Sat, 12 Sep 2026 19:30:41 -0700
Subject: [PATCH] feat(view): multi-select items and copy as a Markdown list
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Josh: "as a user I want to be able to multiselect items and copy them to clipboard as md
list item category price".

- A checkbox column with a select-all for the visible rows; selection survives filter and
  tab changes (kept by id, with the item records cached), so a list can span lots.
- A selection bar with the count, "Copy as Markdown" and "Clear". ⌘C / Ctrl+C copies too
  when rows are selected and focus is not in a field or on a text selection.
- One line per item: `- Name — Category — $price` (category omitted when blank; the price
  is the target, "—" when unpriced; whitespace in names collapsed).
- Clipboard: navigator.clipboard first, execCommand("copy") fallback, and when the frame
  is denied both a dialog with the text pre-selected.
- Tests: the node harness pins the exact list format; presence checks for the controls.
  Smoke-tested in the local harness with a stubbed clipboard (button and shortcut).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011UQ33xXfygwEkzbbJx3YwZ
---
 README.md              |  2 +-
 protoagent.plugin.yaml |  2 +-
 pyproject.toml         |  2 +-
 tests/test_view.py     | 21 ++++++++++++++++-
 view.py                | 51 ++++++++++++++++++++++++++++++++++++++----
 5 files changed, 70 insertions(+), 8 deletions(-)

diff --git a/README.md b/README.md
index ad536e6..f20afc5 100644
--- a/README.md
+++ b/README.md
@@ -15,7 +15,7 @@ inventory_summary()
 | | |
 |---|---|
 | **Tools** | `inventory_summary` · `inventory_list` · `inventory_get` · `inventory_upsert_lot` · `inventory_upsert_item` · `inventory_delete_item` · `inventory_set_price` · `inventory_mark_sold` · `inventory_listing` · `inventory_import_csv` · `inventory_export_csv` · `inventory_reprice_plan` · `inventory_stale` |
-| **View** | a rail panel: the item grid (double-click to edit name/category/condition/qty; status select; click a price for the Price dialog), Price / Sold / Listing / Edit / Delete per row, lots with their P&L, sales, an activity log, CSV import (file or paste, with the mapping report) and export |
+| **View** | a rail panel: the item grid (double-click to edit name/category/condition/qty; status select; click a price for the Price dialog), Price / Sold / Listing / Edit / Delete per row, multi-select with **Copy as Markdown** (`- Name — Category — $price`, also ⌘C), lots with their P&L, sales, an activity log, CSV import (file or paste, with the mapping report) and export |
 | **API** | bearer-gated JSON under `/api/plugins/inventory` — `summary`, `lots`, `items`, `items/{id}/price`, `items/{id}/sold`, `items/{id}/listings`, `listings/{id}/end`, `sales`, `stale`, `audit`, `import`, `export` |
 | **Automations** | `weekly_review: true` arms a plugin-owned recurring turn (`weekly_review_cron`, default Monday 09:00 in `review_timezone`) that re-prices stale evidence from eBay sold comps through the same tools, reports stale listings with a recommendation (it never changes a listing itself), and posts the per-lot P&L. Cancelled when the plugin is disabled. |
 | **Events** | `inventory.item.changed`, `inventory.lot.changed`, `inventory.sale.recorded`, `inventory.imported` |
diff --git a/protoagent.plugin.yaml b/protoagent.plugin.yaml
index 1eef71d..d53bb26 100644
--- a/protoagent.plugin.yaml
+++ b/protoagent.plugin.yaml
@@ -1,6 +1,6 @@
 id: inventory
 name: Inventory (resale source of truth)
-version: 0.3.0
+version: 0.3.1
 description: >-
   The resale inventory as a source of truth: lots (what you bought and for how much),
   items (what each lot split into, its condition and status), listings (where each item is
diff --git a/pyproject.toml b/pyproject.toml
index fb8fad0..f640574 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -1,6 +1,6 @@
 [project]
 name = "inventory-plugin"
-version = "0.3.0"
+version = "0.3.1"
 description = "Resale inventory as a source of truth for protoAgent — lots, items, listings, sales, price evidence, audit"
 requires-python = ">=3.11"
 
diff --git a/tests/test_view.py b/tests/test_view.py
index fada9d1..afcb125 100644
--- a/tests/test_view.py
+++ b/tests/test_view.py
@@ -108,7 +108,8 @@ def test_the_pure_helpers_under_node():
     )
     probe = (
         "\nconsole.log(JSON.stringify({ neg: fmt(-12.5), pos: fmt(1234.5), nul: fmt(null), zero: fmt(0), sneg: signed(-3), spos: signed(3), snul: signed(null),"
-        " esc: esc('<a href=\"x\">&\\'</a>'), field: field('name', 'Name', '\"><img src=x onerror=alert(1)>'), opt: field('lot_id', 'Lot', '', { type: 'select', options: [['\"><b>', 'x</option><script>']] }) }));\n"
+        " esc: esc('<a href=\"x\">&\\'</a>'), field: field('name', 'Name', '\"><img src=x onerror=alert(1)>'), opt: field('lot_id', 'Lot', '', { type: 'select', options: [['\"><b>', 'x</option><script>']] }),"
+        " md: mdList([{ name: 'Reikland  Reavers\\nHuman Team', category: 'Blood Bowl 2016 Split', target: 57.69 }, { name: 'Loose dice', category: '', target: null }, { name: 'Bundle', category: 'Misc', target: 1234.5 }]) }));\n"
     )
     r = subprocess.run(
         [node, "--input-type=module"], input=harness + module + probe, capture_output=True, text=True, timeout=60
@@ -120,3 +121,21 @@ def test_the_pure_helpers_under_node():
     assert out["esc"] == "&lt;a href=&quot;x&quot;&gt;&amp;&#39;&lt;/a&gt;"
     assert "<img" not in out["field"] and "&quot;&gt;&lt;img" in out["field"]
     assert "<b>" not in out["opt"] and "<script>" not in out["opt"]
+    assert (
+        out["md"]
+        == "- Reikland Reavers Human Team — Blood Bowl 2016 Split — $57.69\n- Loose dice — —\n- Bundle — Misc — $1,234.50"
+    )
+
+
+def test_the_page_has_multiselect_and_copy():
+    from inventory_plugin.view import PAGE
+
+    for needle in (
+        'id="sel-all"',
+        'data-sel="',
+        'id="copy-md"',
+        "navigator.clipboard.writeText",
+        'execCommand("copy")',
+        "copySelected()",
+    ):
+        assert needle in PAGE, needle
diff --git a/view.py b/view.py
index 517c517..080a52f 100644
--- a/view.py
+++ b/view.py
@@ -44,6 +44,10 @@
   .basis{max-width:28ch;overflow:hidden;text-overflow:ellipsis}
   .pl-toast-stack{bottom:var(--pl-space-4);right:var(--pl-space-4)}
   .kicker{font-family:var(--pl-font-mono);font-size:11px;text-transform:uppercase;letter-spacing:.06em;color:var(--pl-color-fg-muted)}
+  .pl-table td.sel,.pl-table th.sel{width:28px;padding-right:0}
+  .selbar{display:flex;align-items:center;gap:var(--pl-space-2);padding:var(--pl-space-2) 0;font-size:12px;color:var(--pl-color-fg-muted)}
+  .selbar .pl-kbd{font-family:var(--pl-font-mono);font-size:11px;padding:1px 5px;border:var(--pl-border-width) solid var(--pl-color-border);border-radius:4px}
+  textarea.copytext{width:100%;min-height:160px;font-family:var(--pl-font-mono);font-size:12px}
   @media (max-width:640px){.form{grid-template-columns:1fr}.bar .pl-input,.bar .pl-select{min-width:100px}}
 </style>
 <script>
@@ -97,12 +101,22 @@
 
   const API = "/api/plugins/inventory";
   const STATUSES = ["planned","available","listed","pending","sold","kept","withdrawn"];
-  const state = { tab: "items", lots: [], items: [], summary: null, sales: [], audit: [], filters: { lot: "", status: "", q: "" } };
+  const state = { tab: "items", lots: [], items: [], itemsById: {}, selected: new Set(), summary: null, sales: [], audit: [], filters: { lot: "", status: "", q: "" } };
   const $ = (s, r = document) => r.querySelector(s);
   const esc = (v) => String(v ?? "").replace(/[&<>"']/g, (c) => ({"&":"&amp;","<":"&lt;",">":"&gt;",'"':"&quot;","'":"&#39;"}[c]));
   const fmt = (v) => (v === null || v === undefined || v === "" ? "—" : (Number(v) < 0 ? "-" : "") + "$" + Math.abs(Number(v)).toLocaleString(undefined, { minimumFractionDigits: 2, maximumFractionDigits: 2 }));
   const signed = (v) => (v === null || v === undefined || v === "" ? "—" : (Number(v) >= 0 ? "+" : "") + fmt(v));
   const num = (v) => (v === "" || v === null || v === undefined ? null : Number(v));
+  // One markdown list line per item: "- Name — Category — $price" (category omitted when blank).
+  const mdLine = (it) => "- " + [String(it.name || "").replace(/\s+/g, " ").trim(), String(it.category || "").trim(), fmt(it.target)].filter((x) => x).join(" — ");
+  const mdList = (items) => items.map(mdLine).join("\n");
+  async function copyText(text) {
+    try { await navigator.clipboard.writeText(text); return true; } catch (e) { /* denied in this frame, or no secure context */ }
+    try {
+      const ta = document.createElement("textarea"); ta.value = text; ta.setAttribute("readonly", ""); ta.style.position = "fixed"; ta.style.opacity = "0";
+      document.body.appendChild(ta); ta.select(); const ok = document.execCommand("copy"); ta.remove(); return ok;
+    } catch (e) { return false; }
+  }
 
   // RULES 2+3 — every data call goes through the kit's slug-aware, bearer-carrying fetch with a bare path.
   async function api(path, method = "GET", body) {
@@ -164,6 +178,7 @@
         finally { if (ok) { ok.disabled = false; ok.classList.remove("pl-btn--loading"); } }
       });
       const first = form.querySelector("input:not([readonly]),select,textarea"); (first || root.querySelector(".pl-dialog")).focus();
+      if (first && first.tagName === "TEXTAREA" && first.readOnly) first.select();
     });
   }
   const confirm = (title, text, submit = "Confirm") => dialog({ title, body: '<div class="span2">' + esc(text) + "</div>", submit, danger: true, onSubmit: async () => true });
@@ -174,6 +189,7 @@
     const qs = new URLSearchParams(); if (f.lot) qs.set("lot_id", f.lot); if (f.status) qs.set("status", f.status); if (f.q) qs.set("q", f.q);
     const [lots, summary, items] = await Promise.all([api("/lots"), api("/summary"), api("/items?" + qs.toString())]);
     state.lots = lots.lots; state.summary = summary; state.items = items.items;
+    for (const it of state.items) state.itemsById[it.id] = it;
   }
   let inflight = 0;
   async function refresh() {
@@ -215,12 +231,23 @@
       tile(signed(t.projected_net_at_target), "Projected net at target", "if the rest sells at target") +
       tile(String(t.items), "Items", t.unpriced_items ? t.unpriced_items + " unpriced" : "all priced");
   }
+  function selectedItems() { return [...state.selected].map((id) => state.itemsById[id]).filter(Boolean); }
+  function renderSelBar() {
+    const n = state.selected.size; if (!n) return "";
+    const visible = state.items.filter((it) => state.selected.has(it.id)).length;
+    return '<div class="selbar" id="selbar"><span>' + n + " selected" + (visible !== n ? " (" + visible + " visible)" : "") + "</span>" +
+      '<button class="pl-btn pl-btn--xs pl-btn--primary" id="copy-md">Copy as Markdown</button>' +
+      '<button class="pl-btn pl-btn--xs pl-btn--ghost" id="sel-clear">Clear</button>' +
+      '<span class="pl-kbd">⌘C</span></div>';
+  }
   function renderItems() {
-    if (!state.items.length) return '<div class="pl-empty pl-empty--slotted"><div class="pl-empty__title">No items</div><div class="pl-empty__desc">Add one, or import a CSV — headers like inventory_id, item, lot_id, target_price_usd, status are recognised.</div></div>';
+    if (!state.items.length) return renderSelBar() + '<div class="pl-empty pl-empty--slotted"><div class="pl-empty__title">No items</div><div class="pl-empty__desc">Add one, or import a CSV — headers like inventory_id, item, lot_id, target_price_usd, status are recognised.</div></div>';
+    const allVisible = state.items.length > 0 && state.items.every((it) => state.selected.has(it.id));
     const rows = state.items.map((it) => {
       const ed = (f, v, cls = "") => '<span class="edit ' + cls + '" data-edit="' + f + '" data-id="' + esc(it.id) + '" title="Double-click to edit">' + (v === "" || v === null ? '<span class="muted">—</span>' : esc(v)) + "</span>";
       const pr = (f, v) => '<td class="num price" data-price="' + esc(it.id) + '" title="Click to set the price band">' + fmt(v) + "</td>";
-      return "<tr>" +
+      return "<tr" + (state.selected.has(it.id) ? ' class="pl-tr--selected"' : "") + ">" +
+        '<td class="sel"><input type="checkbox" data-sel="' + esc(it.id) + '"' + (state.selected.has(it.id) ? " checked" : "") + ' aria-label="Select ' + esc(it.name) + '"></td>' +
         '<td class="muted">' + esc(it.id) + "</td>" +
         '<td class="wrap-cell">' + ed("name", it.name) + '<span class="sub">' + ed("category", it.category) + " · " + ed("condition", it.condition) + "</span></td>" +
         '<td class="num">' + ed("quantity", it.quantity) + "</td>" +
@@ -235,7 +262,13 @@
           '<button class="pl-btn pl-btn--xs pl-btn--ghost" data-act="del" data-id="' + esc(it.id) + '" aria-label="Delete">✕</button>' +
         "</td></tr>";
     }).join("");
-    return '<table class="pl-table"><thead><tr><th>ID</th><th>Item</th><th class="num">Qty</th><th>Status</th><th class="num">Low</th><th class="num">Target</th><th class="num">High</th><th>Basis</th><th></th></tr></thead><tbody>' + rows + "</tbody></table>";
+    return renderSelBar() + '<table class="pl-table"><thead><tr><th class="sel"><input type="checkbox" id="sel-all" aria-label="Select all visible"' + (allVisible ? " checked" : "") + '></th><th>ID</th><th>Item</th><th class="num">Qty</th><th>Status</th><th class="num">Low</th><th class="num">Target</th><th class="num">High</th><th>Basis</th><th></th></tr></thead><tbody>' + rows + "</tbody></table>";
+  }
+  async function copySelected() {
+    const items = selectedItems(); if (!items.length) return;
+    const text = mdList(items);
+    if (await copyText(text)) { toast("Copied " + items.length + " item" + (items.length === 1 ? "" : "s") + " as Markdown", "success"); return; }
+    await dialog({ title: "Copy " + items.length + " items", body: '<label class="pl-field span2"><span class="pl-field__label">The clipboard is blocked in this frame — select all and copy</span><textarea class="pl-field__input copytext" id="copytext" readonly>' + esc(text) + "</textarea></label>", onSubmit: null });
   }
   function renderLots() {
     const s = state.summary; if (!s || !s.lots.length) return '<div class="pl-empty">No lots yet.</div>';
@@ -384,6 +417,10 @@
   }
   function wireTable() {
     const out = $("#out");
+    out.querySelectorAll("[data-sel]").forEach((cb) => cb.addEventListener("change", () => { if (cb.checked) state.selected.add(cb.dataset.sel); else state.selected.delete(cb.dataset.sel); render(); }));
+    const all = $("#sel-all", out); if (all) all.addEventListener("change", () => { for (const it of state.items) { if (all.checked) state.selected.add(it.id); else state.selected.delete(it.id); } render(); });
+    const copy = $("#copy-md", out); if (copy) copy.addEventListener("click", copySelected);
+    const clear = $("#sel-clear", out); if (clear) clear.addEventListener("click", () => { state.selected.clear(); render(); });
     out.querySelectorAll("[data-edit]").forEach((s) => s.addEventListener("dblclick", () => inlineEdit(s)));
     out.querySelectorAll("[data-price]").forEach((td) => td.addEventListener("click", () => priceDialog(td.dataset.price)));
     out.querySelectorAll("[data-status]").forEach((sel) => sel.addEventListener("change", async () => {
@@ -414,6 +451,12 @@
   $("#f-status").addEventListener("change", (e) => { state.filters.status = e.target.value; refresh(); });
   let qt; $("#f-q").addEventListener("input", (e) => { clearTimeout(qt); qt = setTimeout(() => { state.filters.q = e.target.value.trim(); refresh(); }, 250); });
   $("#tabs").addEventListener("click", (e) => { const b = e.target.closest("[data-tab]"); if (!b) return; state.tab = b.dataset.tab; refresh(); });
+  document.addEventListener("keydown", (e) => {
+    if (!(e.metaKey || e.ctrlKey) || e.key.toLowerCase() !== "c" || !state.selected.size) return;
+    const el = document.activeElement; if (el && /^(INPUT|TEXTAREA|SELECT)$/.test(el.tagName)) return;
+    if (String(window.getSelection && window.getSelection()).length) return;  // a real text selection wins
+    e.preventDefault(); copySelected();
+  });
 
   // Boot once, on whichever fires first: the kit's protoagent:init handshake (bearer + theme) or a short timer.
   let booted = false;