fix: don't re-show unlabeled price table when format is unknown

app/services/intent.py

@@ -103,6 +103,7 @@ _PRICE_INQUIRY_SIGNALS: tuple[str, ...] = (
     "ebook price", "ebook prices", "e-book price",
     "hardcover price", "hardcover prices", "paperback price", "paperback prices",
     "what about audio", "what about ebook", "what about hardcover",
+    "is this price", "is that price", "price of hard",
 )
 
 _PURCHASE_SIGNALS: tuple[str, ...] = (

app/services/price_catalog_service.py

@@ -133,7 +133,10 @@ def classify_price_query(message: str) -> PriceQuery:
         "ebook or", "e-book or", "paperback or",
         "which format", "what format", "are those", "are these",
         "is that the", "is that hardcover", "is that audiobook",
-        "is that ebook", "format are", "those prices",
+        "is that ebook", "is that hard", "is this hard",
+        "is this price", "is that price", "is this the",
+        "price of hard", "price of audio", "price of e-book", "price of ebook",
+        "format are", "those prices",
     )
     if any(s in q for s in clarify):
         return PriceQuery(kind="format_clarify", platform_ids=platforms, format_hint=fmt)
@@ -344,27 +347,36 @@ class PriceCatalogService:
         book_title: str,
         format_hint: str,
     ) -> dict[str, Any]:
-        """Honest reply when a named format has no stored prices; show what is stored."""
+        """Honest reply when a named format has no stored prices.
+
+        Only attach platform_offers when other formats are labeled — never re-show
+        an unlabeled price grid that contradicts 'I don't have that format'.
+        """
         available = await self.catalog_for_book(author_id, book_id, format_hint="")
-        formats = sorted({o.format for o in available if o.format})
+        labeled = [o for o in available if o.format]
+        formats = sorted({o.format for o in labeled})
         if formats:
             named = ", ".join(formats)
             text = (
                 f"I don't have {format_hint} prices stored for {book_title} yet. "
                 f"On file I have: {named}. Pick a Buy button for those."
             )
-        else:
-            text = (
-                f"I don't have {format_hint} prices stored for {book_title} yet. "
-                "I only list formats I have on file — nothing invented."
-            )
+            return {
+                "text": text[:380],
+                "links": [],
+                "has_links": False,
+                "platform_offers": [offer_to_platform_item(o) for o in labeled],
+            }
+        text = (
+            f"I can't confirm whether that price is {format_hint} for {book_title} — "
+            "I don't have format labels stored for these listings yet. "
+            "Check the retailer page for binding or edition details."
+        )
         return {
             "text": text[:380],
             "links": [],
             "has_links": False,
-            "platform_offers": [
-                offer_to_platform_item(o) for o in available
-            ] if available else None,
+            "platform_offers": None,
         }
 
     async def _build_empty_fallback(
@@ -418,17 +430,22 @@ class PriceCatalogService:
                 f"Here’s each listed price for {book_title} with its format "
                 f"({named}). I only show formats I have stored — nothing invented."
             )
-        else:
-            text = (
-                f"Here are the listed prices I have for {book_title}. "
-                "I don’t have separate format labels for these yet — "
-                "check the retailer page for binding details."
-            )
+            return {
+                "text": text[:380],
+                "links": [],
+                "has_links": False,
+                "platform_offers": [offer_to_platform_item(o) for o in offers],
+            }
+        text = (
+            f"I can't confirm the format for these prices on {book_title} — "
+            "I don't have format labels stored yet. "
+            "Check the retailer page for hardcover, ebook, or audiobook details."
+        )
         return {
             "text": text[:380],
             "links": [],
             "has_links": False,
-            "platform_offers": [offer_to_platform_item(o) for o in offers],
+            "platform_offers": None,
         }
 
     def _build_cheapest_answer(self, offers: list[Offer], book_title: str) -> dict[str, Any]:

tests/unit/test_price_catalog_service.py

@@ -322,6 +322,60 @@ async def test_missing_audiobook_is_honest_and_shows_available():
     assert "Buy Book" not in (result.get("links") or [])
 
 
+@pytest.mark.asyncio
+async def test_missing_hardcover_unlabeled_hides_table():
+    svc = PriceCatalogService(MagicMock())
+
+    async def _catalog(_a, _b, *, format_hint=""):
+        if format_hint == "Hardcover":
+            return []
+        return [
+            Offer("amazon", "Amazon", "$35.00", 3500, "https://a.com", True, ""),
+            Offer("apple_books", "Apple Books", "$14.99", 1499, "https://apple.com", False, ""),
+        ]
+
+    svc.catalog_for_book = _catalog
+    result = await svc.handle(
+        "hardcover prices", "a1", "b1", "All We Say",
+    )
+    assert result["platform_offers"] is None
+    assert "hardcover" in result["text"].lower()
+    assert "can't confirm" in result["text"].lower() or "cannot confirm" in result["text"].lower()
+
+
+def test_is_this_price_hard_copy_is_format_clarify():
+    pq = classify_price_query("is this price of hard copy")
+    assert pq.kind == "format_clarify"
+    assert pq.format_hint == "Hardcover"
+
+
+@pytest.mark.asyncio
+async def test_is_this_hard_copy_unlabeled_hides_table():
+    svc = PriceCatalogService(MagicMock())
+    svc.catalog_for_book = AsyncMock(return_value=[
+        Offer("amazon", "Amazon", "$35.00", 3500, "https://a.com", True, ""),
+        Offer("apple_books", "Apple Books", "$14.99", 1499, "https://apple.com", False, ""),
+    ])
+    result = await svc.handle(
+        "is this price of hard copy", "a1", "b1", "All We Say",
+    )
+    assert result["platform_offers"] is None
+    assert "format" in result["text"].lower()
+    assert _classify_by_rules("is this price of hard copy") == "price_inquiry"
+
+
+@pytest.mark.asyncio
+async def test_format_clarify_unlabeled_hides_table():
+    svc = PriceCatalogService.__new__(PriceCatalogService)
+    offers = [
+        Offer("amazon", "Amazon", "$35.00", 3500, "https://a.com", True, ""),
+        Offer("apple_books", "Apple Books", "$14.99", 1499, "https://apple.com", False, ""),
+    ]
+    result = svc._build_format_clarify_answer(offers, "All We Say")
+    assert result["platform_offers"] is None
+    assert "format" in result["text"].lower()
+
+
 @pytest.mark.asyncio
 async def test_catalog_uses_book_format_when_no_format_prices():
     svc = PriceCatalogService(MagicMock())