Fix buy advice: Minimum and Comfortable are now distinct builds

engine/real_advisor.py

@@ -791,8 +791,16 @@ def min_specs(usecases, mode: str = "run") -> dict:
         return advise_real({"usecase": uc.key}, spec)
 
     def walk(tiers):
-        minimum = comfortable = None
-        for label, price, hw, spec_line, detail in tiers:
+        # Score every tier once, then choose two DISTINCT builds:
+        #   minimum     = cheapest tier that genuinely works (great or tight).
+        #   comfortable = the next step UP that adds real headroom (prefers a
+        #                 tier that runs everything great; falls back to the next
+        #                 one that at least works). It is always strictly above
+        #                 the minimum, so the two are never the same build -- the
+        #                 old logic collapsed them whenever the cheapest tier
+        #                 already ran everything great.
+        built = []
+        for i, (label, price, hw, spec_line, detail) in enumerate(tiers):
             spec = _spec_for_tier(hw)
             per_goal, all_meet, all_great = [], True, True
             for uc in ucs:
@@ -806,13 +814,21 @@ def min_specs(usecases, mode: str = "run") -> dict:
                     "goals": per_goal,
                     "runs": "; ".join(f"{g['goal']}: {g['model']}" for g in per_goal)
                             if len(per_goal) > 1 else per_goal[0]["model"]}
-            if minimum is None and all_meet:
-                minimum = tier
-            if comfortable is None and all_great:
-                comfortable = tier
-            if minimum and comfortable:
-                break
-        return minimum, comfortable
+            built.append({"idx": i, "tier": tier, "all_meet": all_meet,
+                          "all_great": all_great})
+
+        mn = next((b for b in built if b["all_meet"]), None)
+        if mn is None:
+            return None, None
+        # Comfortable: cheapest tier strictly above the minimum that runs all
+        # goals great; else the next one above that at least works.
+        cf = (next((b for b in built if b["idx"] > mn["idx"] and b["all_great"]), None)
+              or next((b for b in built if b["idx"] > mn["idx"] and b["all_meet"]), None))
+        if cf is None:
+            # Minimum is the top of this platform's ladder; say so honestly
+            # instead of repeating the same build as "comfortable".
+            return mn["tier"], {"ceiling": True, "label": mn["tier"]["label"]}
+        return mn["tier"], cf["tier"]
 
     platforms = []
     for p in _PLATFORMS:

static/app.js

@@ -429,6 +429,10 @@ function renderBuy(d) {
       <div class="bc-head"><span class="bc-kind ${kind}">${kind === "min" ? "Minimum" : "Comfortable"}</span></div>
       <p class="bc-none">No build on this platform ${ft ? "trains" : "runs"} ${multi ? "all of these" : "this"}${ft ? "" : " well"}, it wants more than this ladder offers.</p>
     </div>`;
+    if (t.ceiling) return `<div class="buy-card ceiling">
+      <div class="bc-head"><span class="bc-kind ${kind}">Comfortable</span><span class="ic" data-ic="check"></span></div>
+      <p class="bc-none">The minimum build is already the most capable ${ft ? "training " : ""}box on this platform for ${multi ? "these goals" : "this goal"}. Nothing bigger is needed here.</p>
+    </div>`;
     const goalsHtml = multi
       ? `<ul class="bc-goals">${t.goals.map(g => `<li><span>${esc(g.goal)}</span><b>${esc(g.model)}</b>${g.verdict === "tight" ? " <i>(tight)</i>" : ""}</li>`).join("")}</ul>`
       : `<div class="bc-runs">${ft ? "Can train up to" : "Runs"} <b>${esc(t.runs)}</b>${t.goals[0] && t.goals[0].verdict === "tight" ? " <i>(tight)</i>" : ""}</div>`;

static/style.css

@@ -496,6 +496,8 @@ details.disc > summary:hover { color: var(--text-primary); }
   border-radius: var(--r-md); padding: var(--s-3) var(--s-4);
 }
 .buy-card.empty { opacity: .7; }
+.buy-card.ceiling { border-color: var(--accent-strong); }
+.buy-card.ceiling .bc-head .ic { color: var(--accent); font-size: 16px; }
 .bc-head { display: flex; align-items: center; justify-content: space-between; }
 .bc-kind { font-family: var(--font-crt); font-size: 14px; font-weight: 400; text-transform: uppercase; letter-spacing: .08em; color: var(--text-muted); }
 .bc-kind.min { color: var(--warn); } .bc-kind.comfy { color: var(--ok); }