fix(review): meaningful beyond-cutoff bands test, LogoChip escape/click-outside dismiss, drop orphaned useIsochrone

Final-review follow-ups. Remaining minors (degraded-mode double fetch and
reduce-time flicker on band-less providers, desktop-only empty hint)
recorded as acceptable follow-ups - they only affect ISOCHRONE_PROVIDER=ors.

Author: TomerBenda

apps/web/src/app/page.tsx

@@ -6,9 +6,9 @@
  * State flow:
  *   URL  ──parse──► state  ──serialize──► history.replaceState
  *                  │
- *                  ├──► <IlsochroneMap origin={...} polygon={...} />
- *                  ├──► <TimeSelector value={minutes} />
- *                  └──► useIsochrone(...) → SWR → /api/isochrone → ORS
+ *                  ├──► <IlsochroneMap origin={...} bands={...} />
+ *                  ├──► <TimeSelector value={minutes} />  (pure client state)
+ *                  └──► useIsochroneBands(...) → SWR → /api/isochrone?bands=1 → local engine
  */
 import { useCallback, useEffect, useMemo, useRef, useState } from 'react';
 import dynamic from 'next/dynamic';

apps/web/src/components/brand/LogoChip.tsx

@@ -1,12 +1,31 @@
 'use client';
 
-import { useState } from 'react';
+import { useEffect, useRef, useState } from 'react';
 import { Info } from 'lucide-react';
 
 export function LogoChip() {
   const [open, setOpen] = useState(false);
+  const rootRef = useRef<HTMLDivElement>(null);
+
+  // Escape and click-outside dismiss the info popover.
+  useEffect(() => {
+    if (!open) return;
+    const onKey = (e: KeyboardEvent) => {
+      if (e.key === 'Escape') setOpen(false);
+    };
+    const onDown = (e: MouseEvent) => {
+      if (rootRef.current && !rootRef.current.contains(e.target as Node)) setOpen(false);
+    };
+    document.addEventListener('keydown', onKey);
+    document.addEventListener('mousedown', onDown);
+    return () => {
+      document.removeEventListener('keydown', onKey);
+      document.removeEventListener('mousedown', onDown);
+    };
+  }, [open]);
+
   return (
-    <div className="relative">
+    <div ref={rootRef} className="relative">
       <button
         type="button"
         onClick={() => setOpen((v) => !v)}

apps/web/src/lib/hooks/useIsochrone.ts

@@ -18,8 +18,14 @@ const CENTER: [number, number] = [34.7821, 32.0818];
 
 describe('computeIsochroneBands', () => {
   it('one pass equals per-band computeIsochrone results', () => {
-    const { bands, snapDistanceM } = computeIsochroneBands(GRID, CENTER, [5, 10, 15]);
-    expect(bands.map((b) => b.minutes)).toEqual([5, 10, 15]);
+    // Bands chosen so the grid genuinely straddles cutoffs: at 2 min (120 s)
+    // the 144 s neighbors are beyond the band but finite in the shared
+    // max-cutoff times array — the exact window where the one-pass
+    // optimization could diverge from solo runs if the beyond-cutoff
+    // interpolation were wrong. (With all-nodes-inside bands the deep-equal
+    // would pass vacuously.)
+    const { bands, snapDistanceM } = computeIsochroneBands(GRID, CENTER, [2, 5, 10]);
+    expect(bands.map((b) => b.minutes)).toEqual([2, 5, 10]);
     expect(snapDistanceM).toBeLessThan(10);
     for (const band of bands) {
       const solo = computeIsochrone(GRID, CENTER, band.minutes);