feat(inbox): overspend as a tile beside Fixes applied, and a $5 noise floor

Three founder calls from reviewing the page live.

1. THE FULL-WIDTH BAND IS GONE, from both the Dashboard and the Review inbox.
   It led each page with a large figure, three paragraphs of prose and a
   derivation accordion, and read as heavy. The figure now occupies the
   compact tile slot the old forward-looking 'est. savings' tile used to hold,
   beside 'Fixes applied' — which is the shape the page was asked for.

   What did NOT change, because it is honesty discipline and not layout: the
   number is still the server's own past_overspend block read verbatim (never
   reduced over rendered cards); it is still past tense, observed, and carries
   no recovery vocabulary; the total observed COST is still disclosed and still
   worded as cost rather than waste (rule 30). The long-form derivation moved
   into the figure's hover text rather than being dropped. The excluded-waste
   cross-reference moved BELOW the tile row: it used to hang off the band's
   note slot and would have disappeared with it, and leaving it unstated reads
   as 'the biggest figure the product computes doesn't exist'.

2. A $5 NOISE FLOOR on both tabs. A card worth less than that is not worth a
   row — acting on it costs more attention than the fix returns. Measured on a
   real corpus the effect is the whole complaint: 51 recurring mistakes -> 5,
   15 cost advisories -> 5.

   Two rules keep it a display filter and not a quiet write-off. ONLY A KNOWN
   FIGURE CAN BE JUDGED SMALL — an unpriced item is not a cheap one, so the
   floor never hides it. And WHAT IS HIDDEN IS STATED: each tab renders a line
   naming how many rows went and what they cost combined ('46 smaller
   recurring mistakes under $5.00 are hidden'). The headline is untouched and
   still counts them.

3. The floor is declared ABOVE the selection helpers so the tab count, 'select
   all' and 'dismiss checked' all operate on exactly the rows a reader can
   see. A bulk action that can reach a row nobody was shown is the failure
   mode that ordering exists to prevent. An analyzer-focused deep link skips
   the floor, or an Optimize page's 'review it ->' would land on an empty tab.

Tests moved with the decisions rather than being deleted: the band tests now
target the tile and additionally assert it sits IN the tile row, that the cost
disclosure survived the shrink, and that the excluded note is not nested inside
the tile that can disappear.

Co-Authored-By: Claude <noreply@anthropic.com>

Author: anshss

tests/unit/test_lens_ui_regression.py

@@ -2506,11 +2506,19 @@ def test_dollars_suppressed_reads_the_server_display_rule(html):
 # might return. A past figure is checkable against a bill the user already
 # paid; a forward one asks them to trust a projection of a month that has not
 # happened, and trust is the scarce resource.
-def test_inbox_headline_is_the_past_overspend_band(html):
+def test_inbox_headline_is_the_past_overspend_tile(html):
+    # 2026-07-26 founder call: the full-width band was removed and this figure
+    # now occupies the compact tile slot beside "Fixes applied". The TENSE
+    # decision above is unchanged -- only the shape moved.
     start = html.index("function InboxStatTiles")
     end = html.index("function ReviewInboxView", start)
     tile = html[start:end]
-    assert "<${PastOverspendBand} block=${pastOverspend}" in tile
+    assert "<${PastOverspendTile} block=${pastOverspend}" in tile
+    # It sits IN the tile row, beside the applied tile -- not on its own row
+    # above it, which is what made it read as a heavy banner.
+    row = tile[tile.index('display:flex;gap:14px'):]
+    assert "<${PastOverspendTile}" in row
+    assert "Fixes applied" in row
     # The old forward-looking headline and its vocabulary are gone from here.
     assert ">Estimated recoverable " not in tile
     assert "est./mo" not in tile
@@ -2529,17 +2537,21 @@ def test_inbox_headline_number_comes_from_the_payload_not_a_client_side_sum(html
     assert "priceable.length * 2 >= openItems.length" not in tile
 
 
-def test_past_overspend_band_hides_when_nothing_is_open(html):
-    # Nothing open means no observed figure to state, so the band hides rather
+def test_past_overspend_tile_hides_when_nothing_is_open(html):
+    # Nothing open means no observed figure to state, so the tile hides rather
     # than rendering a fabricated zero — but the excluded-waste line still has
-    # to render, so it gets a bare band of its own.
+    # to render, which is why it now lives BELOW the tile row rather than
+    # inside the tile that can disappear.
     start = html.index("function InboxStatTiles")
     end = html.index("function ReviewInboxView", start)
     tile = html[start:end]
     assert "hasOpenOverspend ? html`" in tile
-    band_start = html.index("function PastOverspendBand")
+    band_start = html.index("function PastOverspendTile")
     band = html[band_start:html.index("\n}", band_start)]
     assert "if (!causes && !toks) return null;" in band
+    # The note is not nested inside the conditional tile.
+    note_at = tile.index("<${ExcludedWasteNote}")
+    assert "hasOpenOverspend" not in tile[note_at - 200:note_at]
 
 
 # --- #326: excluded waste (summarize) is stated + linked, never summed ----- #
@@ -2549,7 +2561,10 @@ def test_inbox_stat_tiles_renders_the_excluded_cross_reference(html):
     block = html[start:end]
     # Rendered inside InboxStatTiles, not just defined standalone.
     assert "<${ExcludedWasteNote} excluded=${excluded} />" in block
-    assert block.count("<${ExcludedWasteNote} excluded=${excluded} />") == 2
+    # Once, not twice: it used to render both inside the band's `note` slot and
+    # again in a bare fallback band. With the band gone it has ONE home, below
+    # the tile row, where it renders whether or not the tile does.
+    assert block.count("<${ExcludedWasteNote} excluded=${excluded} />") == 1
     # Never folded into the blue tile's own dollar figure — only ever a
     # separate stated line with a link out.
     assert "not summed above" in block
@@ -2629,7 +2644,7 @@ def test_select_all_checkbox_sits_beside_the_bulk_dismiss_button(html):
     # checked" rather than inside a <thead><th>.
     assert "function SelectAllCheckbox" in html
     assert (
-        "<${SelectAllCheckbox} total=${visible.length} "
+        "<${SelectAllCheckbox} total=${shownRelearn.length} "
         "selected=${selectedCount} onToggle=${toggleAll} />"
     ) in html
     # The per-row checkbox is still present, just inside a flat row now.
@@ -2657,7 +2672,7 @@ def test_select_all_toggles_off_when_everything_is_selected(html):
     assert "else next.add(sig)" in fn
     # The component delegates to it over the RENDERED row set.
     assert (
-        "nextSelectAllSelection(visible.map(c => c.signature), prev)"
+        "nextSelectAllSelection(shownRelearn.map(c => c.signature), prev)"
     ) in html
 
 
@@ -2669,8 +2684,8 @@ def test_select_all_applies_only_to_the_rendered_rows(html):
     start = html.index("const selectedVisible =")
     end = html.index("const modalCluster =", start)
     block = html[start:end]
-    assert "visible.filter(c => checked.has(c.signature))" in block
-    assert "visible.map(c => c.signature)" in block
+    assert "shownRelearn.filter(c => checked.has(c.signature))" in block
+    assert "shownRelearn.map(c => c.signature)" in block
     assert "d.clusters" not in block
     # The filter that makes `visible` a strict subset is still in place. This
     # previously pinned the `!appliedSigs.has(...)` form verbatim, which meant
@@ -2714,7 +2729,7 @@ def test_dismiss_checked_cannot_reach_an_unlisted_row(html):
     start = html.index("const dismissChecked =")
     end = html.index("const modalCluster =", start)
     fn = html[start:end]
-    assert "visible.filter(c => checked.has(c.signature)).map(c => c.signature)" in fn
+    assert "shownRelearn.filter(c => checked.has(c.signature)).map(c => c.signature)" in fn
     assert "...checked]" not in fn
 
 

tests/unit/test_past_overspend.py

@@ -708,7 +708,7 @@ def test_ui_renders_the_observed_figure_at_all(ui):
     # token class, wrote an honesty basis for it, shipped it on the payload,
     # and handed it to a dashboard that referenced it zero times.
     assert ui.count("past_overspend_usd") > 0
-    assert "PastOverspendBand" in ui
+    assert "PastOverspendTile" in ui
 
 
 def test_ui_never_derives_a_past_overspend_figure_client_side(ui):
@@ -721,11 +721,14 @@ def test_ui_never_derives_a_past_overspend_figure_client_side(ui):
         assert forbidden not in ui, f"UI derives its own figure: {forbidden}"
 
 
-def test_both_headline_surfaces_read_the_same_server_block(ui):
-    # The Dashboard hero and the Review inbox headline render the SAME
-    # component over the SAME payload key, so they cannot disagree on basis,
-    # window, or number.
-    assert ui.count("<${PastOverspendBand}") == 2
+def test_the_observed_figure_renders_from_the_server_block_only(ui):
+    # 2026-07-26 founder call: the Dashboard hero band was removed and the
+    # inbox band became a compact tile, so this figure now has exactly ONE
+    # render site. The guarantee that survives is the one that mattered: it is
+    # read from the server's `past_overspend` block, never reduced client-side,
+    # so what renders cannot drift from what the endpoint computed.
+    assert ui.count("<${PastOverspendTile}") == 1
+    assert "PastOverspendBand" not in ui, "the removed band must not linger"
     assert "setCostPastOverspend(r.past_overspend || null)" in ui
     assert "setHeroPast((r && r.past_overspend) || null)" in ui
     # The hero fetches on its OWN effect rather than inside the Dashboard's
@@ -737,7 +740,7 @@ def test_both_headline_surfaces_read_the_same_server_block(ui):
 
 
 def test_ui_labels_are_past_tense_and_carry_no_recovery_vocabulary(ui):
-    band = ui[ui.index("function PastOverspendBand"):]
+    band = ui[ui.index("function PastOverspendTile"):]
     band = band[:band.index("\n}")]
     # The headline is the AVOIDABLE amount, and says so — it is no longer
     # labelled as everything the behaviour cost.
@@ -747,8 +750,12 @@ def test_ui_labels_are_past_tense_and_carry_no_recovery_vocabulary(ui):
     assert "could save" not in ui
     # No ratio framing ("recovering $X of a $Y problem") anywhere.
     assert "recovering $" not in ui
-    # The cost line is present, past tense, and worded as COST — never waste.
-    assert "cost ${fmtUsd(cost)} in total" in band
+    # The cost line survived the band's removal: still present, still past
+    # tense, still worded as COST rather than waste. It is terser and its long
+    # form moved into the figure's hover text, but rule 30's disclosure is not
+    # something a layout change is allowed to drop.
+    assert "total cost" in band
+    assert "that is cost, not waste" in band.lower()
     assert "That is cost, not waste" in band
     fn = ui[ui.index("function observedCostSentence"):]
     fn = fn[:fn.index("\n}")]