Deploy a9d45ca

server/deckbuilder.py

@@ -74,6 +74,14 @@ class DeckCard:
     category: str
     oracle_text: str
     locked: bool = False  # kept from a previous completion at the user's request
+    # Real-deck-frequency classification ("staple" | "mid" | "long_tail"),
+    # None for basics (outside the tokenizer vocab, so untiered). Distinct
+    # from `source`: the synergy phase runs unrestricted and routinely
+    # picks real staples (Sol Ring, Command Tower) on their own merit, so
+    # a card can be source="synergy" and tier="staple" at once -- without
+    # this field the frontend had no way to show that, and a staple could
+    # read as "just a synergy pick" with nothing marking it as universal.
+    tier: str | None = None
     # Per-face power/toughness, loyalty, or defense ("4/3", "Loyalty 3", None), see server/card_stats.py.
     stats: list[str | None] = field(default_factory=list)
     # Cheapest paper printing: {"usd": 0.25, "set": "M21", "finish": "foil"}, see server/card_prices.py.
@@ -291,6 +299,7 @@ class DeckBuilder:
             type_line=card.type_line, mana_cost=card.mana_cost, cmc=card.cmc,
             category=category_of(card.type_line), oracle_text=card.oracle_text,
             stats=self.card_stats.get(card.oracle_id, []), price=self.prices.get(card.oracle_id),
+            tier=self.tiers.get(self.tok.token_to_id[self.tok.oracle_id_to_token[card.oracle_id]]),
         )
 
     def _basic_card(self, name: str, quantity: int, source: str) -> DeckCard:

web/src/api.ts

@@ -2,10 +2,17 @@
 
 export type CardSource = 'input' | 'synergy' | 'staple' | 'basic'
 
+/** Real-deck-frequency classification, independent of `source` -- the
+ * synergy phase is unrestricted and routinely picks real staples on their
+ * own merit, so a card can be source: 'synergy' and tier: 'staple' at
+ * once. null for basics (outside the tokenizer vocab, so untiered). */
+export type CardTier = 'staple' | 'mid' | 'long_tail'
+
 export interface DeckCard {
   name: string
   quantity: number
   source: CardSource
+  tier: CardTier | null
   score: number | null
   type_line: string
   mana_cost: string

web/src/components/DeckView.tsx

@@ -132,9 +132,18 @@ function CardRow({
   const adjustable = !isCommander && isAdjustable(card)
   const isBanned = adjustable && banned.has(card.name)
   const locked = adjustable && !isBanned && lockedNames.has(card.name)
-  const title = card.source === 'input' || isCommander
-    ? SOURCE_LABELS.input
-    : card.score != null ? `${SOURCE_LABELS[card.source]} · model score ${card.score.toFixed(3)}` : SOURCE_LABELS[card.source]
+  // A card picked by the synergy phase can still be a real staple by
+  // real-deck frequency -- the synergy phase is unrestricted and often
+  // picks staples (Sol Ring, Command Tower) on their own merit. Call that
+  // out specifically: it's the one case where `source` alone would read
+  // as "just a synergy pick" when the card is actually universal.
+  const isUnlabeledStaple = card.source === 'synergy' && card.tier === 'staple'
+  const sourceLabel = card.source === 'input' || isCommander ? SOURCE_LABELS.input : SOURCE_LABELS[card.source]
+  const title = [
+    sourceLabel,
+    isUnlabeledStaple && 'also a staple',
+    !isCommander && card.score != null && `model score ${card.score.toFixed(3)}`,
+  ].filter(Boolean).join(' · ')
   const classes = [
     'card-row',
     `source-${isCommander ? 'commander' : card.source}`,
@@ -171,6 +180,7 @@ function CardRow({
         </span>
       )}
       {isCommander && <span className="tag">Commander</span>}
+      {isUnlabeledStaple && <span className="tag tag-staple" title="Also a staple by real-deck frequency">Staple</span>}
       <ManaCost cost={card.mana_cost} />
       {/* Fixed-width, right-aligned last column so prices line up down each
           group; an empty cell keeps the column when a card has no price. */}

web/src/index.css

@@ -740,6 +740,15 @@ textarea {
   color: var(--warn-text);
 }
 
+/* Outlined, not filled like .tag/.tag-warn -- this flags a secondary fact
+   about the card (it's also a staple) alongside its primary source dot,
+   not a replacement for it. */
+.tag-staple {
+  background: transparent;
+  border: 1px solid var(--src-staple);
+  color: var(--src-staple);
+}
+
 /* ---------- mana symbols ---------- */
 
 .mana-cost {