From 69ecf1012e2d82a7467e0a5bbdef497bd6653aea Mon Sep 17 00:00:00 2001
From: jtricerolph <info@tricerolph.com>
Date: Tue, 14 Jul 2026 17:49:27 +0000
Subject: [PATCH] Fix breakfast detection on hotel-page scraper + add update
 banner

Breakfast was keyed off cells[2] specifically; hotel page templates vary and
the conditions column can appear at a different index, causing breakfast_included
to always be false for own/competitor rates. Now scans full row innerText and
also checks data-fltrs.mealplan/breakfast_included as a secondary signal.

Free cancellation extraction similarly updated to search all cells rather than
assuming a fixed column.

Also ships the update-available banner (polls /health every 2 min, prompts
reload when the version hash changes after a deploy).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../scraper_backends/playwright_hotel_page.py | 21 +++++++--
 frontend/src/App.tsx                          |  6 +++
 frontend/src/components/UpdateBanner.tsx      | 44 +++++++++++++++++++
 frontend/src/hooks/useVersionCheck.ts         | 43 ++++++++++++++++++
 4 files changed, 110 insertions(+), 4 deletions(-)
 create mode 100644 frontend/src/components/UpdateBanner.tsx
 create mode 100644 frontend/src/hooks/useVersionCheck.ts

diff --git a/backend/services/scraper_backends/playwright_hotel_page.py b/backend/services/scraper_backends/playwright_hotel_page.py
index 93d7328..64cd04b 100644
--- a/backend/services/scraper_backends/playwright_hotel_page.py
+++ b/backend/services/scraper_backends/playwright_hotel_page.py
@@ -107,12 +107,25 @@ _EXTRACT_RATES_JS = """
         let fltrs = {};
         try { fltrs = JSON.parse(tr.getAttribute('data-fltrs') || '{}'); } catch(e) {}
 
-        // Conditions cell (3rd <td>) holds meal plan + cancel info
         const cells = tr.querySelectorAll('td');
-        const condCell = cells.length >= 3 ? cells[2].innerText || '' : '';
 
-        const breakfastIncluded = condCell.toLowerCase().includes('breakfast');
-        const nonRefundable     = (fltrs.non_refundable === 1);
+        // Scan ALL cells for breakfast — the conditions column index varies
+        // across hotel page templates (some hotels swap occupancy + conditions).
+        // Also check data-fltrs which carries a mealplan flag on some properties.
+        const rowText = (tr.innerText || '').toLowerCase();
+        const breakfastIncluded = rowText.includes('breakfast')
+            || fltrs.mealplan === 1
+            || fltrs.breakfast_included === 1;
+
+        const nonRefundable = (fltrs.non_refundable === 1);
+
+        // Cancellation text: find whichever cell mentions it.
+        let condCell = '';
+        for (const cell of cells) {
+            const t = cell.innerText || '';
+            if (t.toLowerCase().includes('free cancellation')) { condCell = t; break; }
+        }
+        if (!condCell && cells.length >= 3) condCell = cells[2].innerText || '';
 
         const cancelMatch = condCell.match(/free cancellation before ([\\w\\s]+)/i);
         const freeCancelText = cancelMatch ? cancelMatch[1].trim() : null;
diff --git a/frontend/src/App.tsx b/frontend/src/App.tsx
index e683936..182dc72 100644
--- a/frontend/src/App.tsx
+++ b/frontend/src/App.tsx
@@ -1,5 +1,7 @@
 import { Routes, Route, Navigate } from 'react-router-dom'
 import AuthGate from './components/AuthGate'
+import { UpdateBanner } from './components/UpdateBanner'
+import { useVersionCheck } from './hooks/useVersionCheck'
 import Layout from './components/Layout'
 import Bookability from './pages/Bookability'
 import MarketView from './pages/MarketView'
@@ -8,7 +10,9 @@ import RateAnalysis from './pages/RateAnalysis'
 import Settings from './pages/Settings'
 
 export default function App() {
+  const updateAvailable = useVersionCheck('/rates/health')
   return (
+    <>
     <AuthGate>
       <Layout>
         <Routes>
@@ -24,5 +28,7 @@ export default function App() {
         </Routes>
       </Layout>
     </AuthGate>
+    <UpdateBanner visible={updateAvailable} />
+    </>
   )
 }
diff --git a/frontend/src/components/UpdateBanner.tsx b/frontend/src/components/UpdateBanner.tsx
new file mode 100644
index 0000000..b492c67
--- /dev/null
+++ b/frontend/src/components/UpdateBanner.tsx
@@ -0,0 +1,44 @@
+import { RefreshCw } from 'lucide-react'
+
+export function UpdateBanner({ visible }: { visible: boolean }) {
+  if (!visible) return null
+  return (
+    <div style={{
+      position: 'fixed',
+      bottom: 0,
+      left: 0,
+      right: 0,
+      zIndex: 9999,
+      background: 'var(--sidebar)',
+      color: 'var(--text-light)',
+      display: 'flex',
+      alignItems: 'center',
+      justifyContent: 'center',
+      gap: '12px',
+      padding: '10px 16px',
+      fontSize: '14px',
+      boxShadow: '0 -2px 8px rgba(0,0,0,0.3)',
+    }}>
+      <span>A new version is available.</span>
+      <button
+        onClick={() => window.location.reload()}
+        style={{
+          display: 'flex',
+          alignItems: 'center',
+          gap: '6px',
+          background: 'var(--accent)',
+          color: 'var(--sidebar)',
+          border: 'none',
+          borderRadius: '4px',
+          padding: '6px 14px',
+          fontWeight: 600,
+          cursor: 'pointer',
+          fontSize: '13px',
+        }}
+      >
+        <RefreshCw size={14} strokeWidth={1.75} />
+        Reload
+      </button>
+    </div>
+  )
+}
diff --git a/frontend/src/hooks/useVersionCheck.ts b/frontend/src/hooks/useVersionCheck.ts
new file mode 100644
index 0000000..e4fed28
--- /dev/null
+++ b/frontend/src/hooks/useVersionCheck.ts
@@ -0,0 +1,43 @@
+import { useEffect, useState } from 'react'
+
+const POLL_MS = 2 * 60 * 1000
+
+export function useVersionCheck(healthUrl: string) {
+  const [updateAvailable, setUpdateAvailable] = useState(false)
+
+  useEffect(() => {
+    let seenVersion: string | null = null
+
+    async function check() {
+      try {
+        const res = await fetch(healthUrl, { cache: 'no-store' })
+        if (!res.ok) return
+        const data = await res.json()
+        const v: string | undefined = data.version
+        if (!v) return
+        if (seenVersion === null) {
+          seenVersion = v
+        } else if (v !== seenVersion) {
+          setUpdateAvailable(true)
+        }
+      } catch {
+        // network error — skip silently
+      }
+    }
+
+    check()
+    const interval = setInterval(check, POLL_MS)
+
+    function onVisible() {
+      if (document.visibilityState === 'visible') check()
+    }
+    document.addEventListener('visibilitychange', onVisible)
+
+    return () => {
+      clearInterval(interval)
+      document.removeEventListener('visibilitychange', onVisible)
+    }
+  }, [healthUrl])
+
+  return updateAvailable
+}