slab-cli 0.9.0__tar.gz → 0.11.0__tar.gz

{slab_cli-0.9.0 → slab_cli-0.11.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: slab-cli
-Version: 0.9.0
+Version: 0.11.0
 Summary: CLI for the slab trading-card API — catalog, collect, and track your cards from the terminal.
 Author: dev_jeb
 Requires-Dist: slab-schemas>=0.1.0

{slab_cli-0.9.0 → slab_cli-0.11.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "slab-cli"
-version = "0.9.0"
+version = "0.11.0"
 description = "CLI for the slab trading-card API — catalog, collect, and track your cards from the terminal."
 readme = "README.md"
 authors = [{name = "dev_jeb"}]

{slab_cli-0.9.0 → slab_cli-0.11.0}/src/slab_cli/client.py

@@ -336,7 +336,14 @@ class SlabClient:
         resp = self._request("POST", "/custom-sets/search", json=body)
         return CustomSetSearchResult.model_validate(resp.json())
 
-    def get_custom_set(self, set_uuid: str, collector_uuid: str | None = None) -> CustomSetDetail:
-        params = {"collector_uuid": collector_uuid} if collector_uuid else {}
+    def get_custom_set(
+        self, set_uuid: str, collector_uuid: str | None = None,
+        limit: int | None = None, offset: int | None = None,
+    ) -> CustomSetDetail:
+        params: dict = {"collector_uuid": collector_uuid} if collector_uuid else {}
+        if limit is not None:
+            params["limit"] = limit
+        if offset is not None:
+            params["offset"] = offset
         resp = self._request("GET", f"/custom-sets/{set_uuid}", params=params)
         return CustomSetDetail.model_validate(resp.json())

{slab_cli-0.9.0 → slab_cli-0.11.0}/src/slab_cli/commands/breaks.py

@@ -6,7 +6,6 @@ from __future__ import annotations
 
 from slab_schemas.cards import SetSearchQuery
 from slab_schemas.collection import BreakCreate, BreakOut, BreakSearchQuery, CollectionSearchQuery
-from slab_schemas.enums import BreakType
 
 from ..client import SlabClient
 from ..context import ctx
@@ -59,7 +58,13 @@ def create_break_interactive(client: SlabClient, collector: str) -> BreakOut | N
     if set_choice is None:
         return None
 
-    break_type = select_break_type()
+    # Offer the formats this set actually comes in (its sealed SKUs); a set with no SKUs — or a
+    # failed lookup — falls back to the generic picker. Never let the extra lookup kill the flow.
+    try:
+        formats = [p.format.value for p in client.get_set_sealed(set_choice.uuid)]
+    except Exception:
+        formats = []
+    break_type = select_break_type(formats)
     break_date = ask_date("When did you open it?")
     total_cost = ask_decimal("Total cost:")
     # Offer the collector's previously-used sources for arrow-key reuse (falls back to a plain
@@ -70,7 +75,7 @@ def create_break_interactive(client: SlabClient, collector: str) -> BreakOut | N
         collector,
         BreakCreate(
             set_uuid=set_choice.uuid,
-            break_type=BreakType(break_type),
+            break_type=break_type,
             break_date=break_date,
             total_cost=total_cost,
             source=source,

{slab_cli-0.9.0 → slab_cli-0.11.0}/src/slab_cli/commands/collection.py

@@ -4,6 +4,7 @@ from __future__ import annotations
 
 import re
 from dataclasses import dataclass, field
+from datetime import date
 from decimal import Decimal
 
 from slab_schemas.cards import CardOut, CardSearchQuery
@@ -68,6 +69,7 @@ class AddSession:
     lot_uuid: str | None = None          # the purchase this batch belongs to; its cost splits across cards
     release_set_uuid: str | None = None  # the break's product — scopes every card search to that box
     acquired_source: str | None = None   # where these came from (purchases/trades/gifts), set once
+    acquired_date: date | None = None    # when you got them; None for break/lot cards (theirs wins)
     cards_added: int = 0
 
 
@@ -119,6 +121,14 @@ def _ensure_acquisition_context(session: AddSession) -> None:
         # Trade / gift / other → ask where these came from, once for the batch.
         session.acquired_source = pick_source(session.client, session.collector)
 
+    # When you got them — asked once, and ONLY for standalone cards: a break/lot card takes its
+    # date from the container (which already asked), and storing a second copy of it would just be
+    # something else to keep in sync. Enter defaults to today, so the common "adding what I bought
+    # today" case is one keystroke — but an old pickup can be dated correctly, which matters
+    # because the portfolio chart only counts a card from the day you got it.
+    if session.break_uuid is None and session.lot_uuid is None:
+        session.acquired_date = ask_date("When did you get it?")
+
 
 def _per_card_cost(session: AddSession) -> Decimal | None:
     """The cost that varies per card. Pack pulls derive cost from the break, and a purchase attached
@@ -164,6 +174,8 @@ def _show_review(card: CardOut, session: AddSession, acq_cost: Decimal | None,
         console.print(f"    Purchase:    {session.lot_uuid[:8]}  [dim](cost split across the lot)[/dim]")
     if acq_cost is not None:
         console.print(f"    Cost:        ${acq_cost:.2f}")
+    if session.acquired_date:
+        console.print(f"    Date:        {session.acquired_date}")
     if session.acquired_source:
         console.print(f"    Source:      {session.acquired_source}")
     if self_grade:
@@ -197,6 +209,7 @@ def _submit_and_report(session: AddSession, card: CardOut, acq_cost: Decimal | N
         lot_uuid=session.lot_uuid,
         acquisition_type=session.acq_type,
         acquisition_cost=acq_cost,
+        acquired_date=session.acquired_date,
         acquired_source=session.acquired_source,
         self_grade=self_grade,
         grading_company=grading_company,

{slab_cli-0.9.0 → slab_cli-0.11.0}/src/slab_cli/commands/custom_sets.py

@@ -5,13 +5,16 @@ from __future__ import annotations
 from InquirerPy import inquirer
 from InquirerPy.base.control import Choice
 
+from types import SimpleNamespace
+
 from slab_schemas.custom_sets import (
     CustomSetCardAdd,
     CustomSetCreate,
+    CustomSetDetail,
     CustomSetOut,
     CustomSetSearchQuery,
 )
-from slab_schemas.enums import CustomSetType, MatchMode, Visibility
+from slab_schemas.enums import CustomSetType, DynamicMatch, MatchMode, Visibility
 
 from ..client import SlabClient
 from ..context import ctx
@@ -21,6 +24,7 @@ from ..display import (
     format_card_summary,
     print_custom_set_detail,
 )
+from ..paging import fetch_all_pages
 from ..picker import search_and_pick_subject
 from ..prompts import (
     ask_int_optional,
@@ -28,6 +32,7 @@ from ..prompts import (
     ask_text_optional,
     confirm,
     select_custom_set_type,
+    select_dynamic_match,
     select_match_mode,
     select_printing,
     select_visibility,
@@ -38,6 +43,25 @@ from ..prompts import (
 # Helpers
 # ---------------------------------------------------------------------------
 
+def _fetch_full_detail(
+    client: SlabClient, set_uuid: str, collector: str | None
+) -> CustomSetDetail:
+    """The set with its WHOLE card list — the detail endpoint pages `cards` (completion stats
+    already cover the full set), so flows that show or iterate every entry stitch the pages back
+    together here. Capped at 5000 entries as a runaway bound (a `year`-only dynamic filter can
+    match a whole catalog); past the cap, `card_count` > len(cards) and the display says so."""
+    first: list = []
+
+    def fetch(limit: int, offset: int):
+        d = client.get_custom_set(set_uuid, collector, limit=limit, offset=offset)
+        if not first:
+            first.append(d)
+        return SimpleNamespace(items=d.cards, total=d.card_count)
+
+    cards, _ = fetch_all_pages(fetch, page=500, cap=5000)
+    return first[0].model_copy(update={"cards": cards, "offset": 0})
+
+
 def _select_custom_set(client: SlabClient, collector: str, message: str = "Select a set:") -> CustomSetOut | None:
     """Browse the collector's custom sets and pick one."""
     sets = client.collector_custom_sets(collector)
@@ -98,6 +122,7 @@ def cmd_chase_create(args: list[str]) -> None:
     visibility = select_visibility()
 
     filter_json = None
+    dynamic_match = "any_printing"
     if set_type == "dynamic":
         console.print("\n[dim]A dynamic set is defined by a search filter. Enter the filter criteria.[/dim]")
         console.print("[dim]Cards matching this filter will automatically be in the set.[/dim]\n")
@@ -106,6 +131,15 @@ def cmd_chase_create(args: list[str]) -> None:
         if filter_json is None:
             console.print("[dim]Cancelled.[/dim]")
             return
+        # One set-wide granularity. Default follows the filter's intent: a filter that names a
+        # finish/parallel dimension reads as a per-printing chase; anything else (team, player,
+        # season) reads as "one of each card — base or any parallel counts".
+        chasing_printings = any(
+            k in filter_json for k in ("finish", "parallel_only", "is_numbered", "is_1of1")
+        )
+        dynamic_match = select_dynamic_match(
+            default="exact" if chasing_printings else "any_printing"
+        )
 
     cs = ctx.client.create_custom_set(collector, CustomSetCreate(
         name=name,
@@ -113,6 +147,7 @@ def cmd_chase_create(args: list[str]) -> None:
         visibility=Visibility(visibility),
         set_type=CustomSetType(set_type),
         filter_json=filter_json,
+        dynamic_match=DynamicMatch(dynamic_match),
     ))
 
     console.print(f"\n[green]Created![/green]  {cs.name}  [dim]({cs.uuid[:8]})[/dim]")
@@ -277,12 +312,12 @@ def cmd_chase_view(args: list[str]) -> None:
     client = ctx.client
 
     if args:
-        detail = client.get_custom_set(args[0], collector)
+        detail = _fetch_full_detail(client, args[0], collector)
     else:
         cs = _select_custom_set(client, collector, message="Which set to view?")
         if cs is None:
             return
-        detail = client.get_custom_set(cs.uuid, collector)
+        detail = _fetch_full_detail(client, cs.uuid, collector)
 
     print_custom_set_detail(detail)
 
@@ -311,8 +346,8 @@ def cmd_chase_edit(args: list[str]) -> None:
     if cs is None:
         return
 
-    # Check ownership
-    detail = client.get_custom_set(cs.uuid, collector)
+    # Check ownership — metadata only, skip the card list
+    detail = client.get_custom_set(cs.uuid, collector, limit=1)
     if detail.creator_uuid != collector:
         console.print("[yellow]You can only edit sets you created.[/yellow]")
         return
@@ -350,7 +385,7 @@ def cmd_chase_edit(args: list[str]) -> None:
 
 def _chase_detail(s) -> None:
     """The drill-in for a browsed chase set: its full card list + completion, owner-aware."""
-    print_custom_set_detail(ctx.client.get_custom_set(s.uuid, ctx.collector))
+    print_custom_set_detail(_fetch_full_detail(ctx.client, s.uuid, ctx.collector))
 
 
 def cmd_chase_list(args: list[str]) -> None:
@@ -436,7 +471,7 @@ def cmd_chase_remove(args: list[str]) -> None:
         console.print("[yellow]Can only remove cards from curated sets.[/yellow]")
         return
 
-    detail = client.get_custom_set(cs.uuid, collector)
+    detail = _fetch_full_detail(client, cs.uuid, collector)
     if not detail.cards:
         console.print("[dim]This set has no cards.[/dim]")
         return
@@ -475,8 +510,8 @@ def cmd_chase_delete(args: list[str]) -> None:
     if cs is None:
         return
 
-    # Check ownership
-    detail = client.get_custom_set(cs.uuid, collector)
+    # Check ownership — metadata only, skip the card list
+    detail = client.get_custom_set(cs.uuid, collector, limit=1)
     if detail.creator_uuid != collector:
         console.print("[yellow]You can only delete sets you created.[/yellow]")
         return

{slab_cli-0.9.0 → slab_cli-0.11.0}/src/slab_cli/display.py

@@ -261,7 +261,7 @@ BREAK_COLUMNS = [
 def break_cells(b) -> list[str]:
     """One break as its BREAK_COLUMNS cells."""
     cpc = money(b.cost_per_card) if b.cost_per_card is not None else "—"
-    return [b.set_name or "", b.break_type, money(b.total_cost), str(b.copy_count), cpc,
+    return [b.set_name or "", _format_label(b.break_type), money(b.total_cost), str(b.copy_count), cpc,
             str(b.break_date or "—"), b.source or "—", b.uuid[:8]]
 
 
@@ -314,7 +314,7 @@ def render_break_pulls(brk, result: CollectionResult) -> None:
     console.print()
     t = Text()
     t.append(brk.set_name or "Break", style="slab.value")
-    t.append(f"  —  {brk.break_type} · {brk.break_date or '—'}"
+    t.append(f"  —  {brk.break_type.replace('_', ' ')} · {brk.break_date or '—'}"
              + (f" · {brk.source}" if brk.source else ""), style="slab.dim")
     console.print(t)
 
@@ -530,10 +530,15 @@ def render_dashboard(d: DashboardStats) -> None:
         if d.priced_coverage is not None:
             rows.append(("Coverage", f"{pct_coverage(float(d.priced_coverage) * 100)} [slab.dim]priced[/]"))
         # Portfolio value over time — the CLI stand-in for the portal's chart, from the same payload.
+        # Each point values only the copies owned that day, so the net includes cards you ADDED in
+        # the window, not just price movement — said plainly so it isn't read as a 90d gain.
         if len(d.portfolio_series) >= 2:
             vals = [p.portfolio_value for p in d.portfolio_series]
             net = float(vals[-1]) - float(vals[0])
-            rows.append(("Trend 90d", f"{sparkline(vals)}   {signed_money(net)}"))
+            rows.append((
+                "Trend 90d",
+                f"{sparkline(vals)}   {signed_money(net)} [slab.dim]incl. cards added[/]",
+            ))
     value = section("Value", kv_grid(rows), foil=True)
 
     comp = _composition_panel(
@@ -555,7 +560,8 @@ def render_dashboard(d: DashboardStats) -> None:
             ("$/card", {"justify": "right", "style": "slab.dim"}),
         ])
         for b in d.break_stats:
-            table.add_row(str(b.break_date or "—"), b.set_name or "—", b.break_type or "—",
+            table.add_row(str(b.break_date or "—"), b.set_name or "—",
+                          _format_label(b.break_type) if b.break_type else "—",
                           money(b.total_cost), str(b.cards), money(b.cost_per_card))
         breaks = titled("Breaks", table)
 
@@ -804,7 +810,11 @@ def print_custom_set_detail(detail: CustomSetDetail) -> None:
     vis = "Public" if detail.visibility == "public" else "Private"
     sub = "subscribed" if detail.is_subscribed else "not subscribed"
 
-    head = [f"[slab.dim]{detail.set_type} · {vis} · {sub}[/]"]
+    kind = detail.set_type
+    if detail.dynamic_match:
+        grain = "any printing counts" if detail.dynamic_match == "any_printing" else "exact printings"
+        kind = f"{detail.set_type} ({grain})"
+    head = [f"[slab.dim]{kind} · {vis} · {sub}[/]"]
     if detail.description:
         head.append(detail.description)
     head.append(f"[slab.label]Creator[/] {detail.creator_name or '—'}   "
@@ -847,6 +857,13 @@ def print_custom_set_detail(detail: CustomSetDetail) -> None:
             table.add_row(str(entry.position), label, match, owned)
 
         console.print(table)
+        if detail.card_count > len(detail.cards):
+            first = detail.offset + 1
+            last = detail.offset + len(detail.cards)
+            console.print(
+                f"  [slab.dim]Showing entries {first}–{last} of {detail.card_count} "
+                f"(completion covers the full set).[/]"
+            )
     console.print()
 
 
@@ -1073,7 +1090,7 @@ def _render_most_valuable_break(mvb: dict | None) -> None:
     date_str = f"  ·  {mvb['break_date']}" if mvb.get("break_date") else ""
     title = Text()
     title.append(mvb["set_name"], style="slab.value")
-    title.append(f"   {mvb['break_type']}{date_str}", style="slab.dim")
+    title.append(f"   {mvb['break_type'].replace('_', ' ')}{date_str}", style="slab.dim")
 
     g = mvb["gain"]
     verdict = "paid off" if g >= 0 else "underwater"

{slab_cli-0.9.0 → slab_cli-0.11.0}/src/slab_cli/prompts.py

@@ -62,7 +62,7 @@ def _break_label(b: BreakOut) -> str:
     when = b.break_date.isoformat() if b.break_date else "—"
     src = f" · {b.source}" if b.source else ""
     cpc = f" @ ${b.cost_per_card:.2f}/card" if b.cost_per_card else ""
-    return (f"{when} · {b.set_name} · {b.break_type} · ${b.total_cost:.2f}{src} · "
+    return (f"{when} · {b.set_name} · {b.break_type.replace('_', ' ')} · ${b.total_cost:.2f}{src} · "
             f"{b.copy_count} cards{cpc}  [{b.uuid[:8]}]")
 
 
@@ -388,7 +388,16 @@ def select_acquisition_type() -> str:
     return inquirer.select(message="How did you acquire this card?", choices=choices).execute()
 
 
-def select_break_type() -> str:
+def select_break_type(formats: list[str] | None = None) -> str:
+    """What was opened. When the set's sealed SKUs are known, offer those formats first (the
+    things actually purchasable for THIS set) with an Other… escape to the generic list; a set
+    with no SKUs goes straight to the generic pack/box/case/other."""
+    if formats:
+        choices = [Choice(value=f, name=f.replace("_", " ").title()) for f in formats]
+        choices.append(Choice(value=None, name="Other…"))
+        picked = inquirer.select(message="What did you open?", choices=choices).execute()
+        if picked is not None:
+            return picked
     choices = [Choice(value=t.value, name=_enum_label(t, _BREAK_TYPE_LABELS)) for t in BreakType]
     return inquirer.select(message="What did you open?", choices=choices).execute()
 
@@ -443,6 +452,19 @@ def select_custom_set_type() -> str:
     return inquirer.select(message="Set type:", choices=choices).execute()
 
 
+def select_dynamic_match(default: str = "any_printing") -> str:
+    """The set-wide granularity of a dynamic set — what counts as completing a card."""
+    choices = [
+        Choice(value="any_printing",
+               name="Any printing — one entry per card; base or any parallel completes it"),
+        Choice(value="exact",
+               name="Exact — every matching parallel is its own entry to chase"),
+    ]
+    return inquirer.select(
+        message="What counts as having a card?", choices=choices, default=default,
+    ).execute()
+
+
 def select_cost_category() -> str:
     choices = [
         Choice(value="grading", name="Grading fee"),