@autobusal/routes-search 1.3.3 → 1.4.1

package/RoutesSearch.tsx

@@ -1,4 +1,4 @@
-import { useState, useEffect, useCallback } from 'react';
+import { useState, useEffect, useCallback, useRef } from 'react';
 import { TFunction } from 'i18next';
 import { useForm } from 'react-hook-form';
 import { FaArrowUp, FaHistory } from 'react-icons/fa';
@@ -9,7 +9,7 @@ import { getCities } from './utilities';
 import Picker from './Picker';
 import { Options, Columns, Column, Item, Title, Action, PickerField } from './styles';
 import { RoutesSearchForm } from './types';
-import { useGetSuggestions } from './services';
+import { useGetSuggestions, useSuggestedOrigin } from './services';
 
 interface Props {
   type: 'departure' | 'return'
@@ -20,11 +20,21 @@ interface Props {
   adultsNo: number
   childrenNo: number
   babiesNo: number
+  /**
+   * Edited: Claude - Date: 2026-08-29
+   *
+   * Open "From" on a departure city near the visitor instead of the
+   * brand's configured default. Only for a FRESH form (the homepage, the
+   * bare /bus-lines page) - never for the collapsed form on a results
+   * page, where the pair on screen is the one being searched and must not
+   * be quietly rewritten under the visitor.
+   */
+  suggest?: boolean
   t: TFunction<'common'>
   onSearch: (data: RoutesSearchForm) => void
 }
 
-const RoutesSearch = ({ type, from, to, departure, _return, adultsNo, childrenNo, babiesNo, t, onSearch }: Props): JSX.Element => {
+const RoutesSearch = ({ type, from, to, departure, _return, adultsNo, childrenNo, babiesNo, suggest = false, t, onSearch }: Props): JSX.Element => {
   const [ selected, setSelected ] = useState<'departure' | 'return'>(type);
 
   // Edited: Ferjolt Ozuni - Date: 2026-08-01
@@ -54,6 +64,41 @@ const RoutesSearch = ({ type, from, to, departure, _return, adultsNo, childrenNo
 
   const { data: CitiesData } = useGetSuggestions();
 
+  /**
+   * Where the visitor most likely wants to leave from.
+   *
+   * Edited: Claude - Date: 2026-08-29
+   *
+   * Applied ONCE and only while the field is untouched: it is a better
+   * opening guess than the brand default, not a correction of anybody's
+   * typing. The remount-to-reseed trick is the map picker's, above -
+   * Autocomplete is uncontrolled, so changing its defaultValue needs a new
+   * key.
+   */
+  const { data: suggestedOrigin } = useSuggestedOrigin(suggest);
+
+  const touchedFrom = useRef<boolean>(false);
+  const appliedSuggestion = useRef<boolean>(false);
+
+  useEffect(() => {
+    if (!suggest || appliedSuggestion.current || touchedFrom.current) {
+      return;
+    }
+
+    if (!suggestedOrigin || suggestedOrigin === from) {
+      return;
+    }
+
+    appliedSuggestion.current = true;
+
+    setPicked(previous => ({ ...previous, from: suggestedOrigin }));
+
+    setValue('from', suggestedOrigin);
+
+    onDeparture(suggestedOrigin);
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [ suggestedOrigin ]);
+
   /**
    * Tell the date picker which clock this journey starts on.
    *
@@ -158,7 +203,12 @@ const RoutesSearch = ({ type, from, to, departure, _return, adultsNo, childrenNo
                 validation="required"
                 t={ t }
                 refs={ register }
-                onUpdate={ (_, value) => { setValue('from', value); onDeparture(String(value)); } }
+                onUpdate={ (_, value) => {
+                  touchedFrom.current = true;
+
+                  setValue('from', value);
+                  onDeparture(String(value));
+                } }
               />
 
               <Picker t={ t } onPick={ (slug) => onPick('from', slug) } />

package/package.json

@@ -1,7 +1,10 @@
 {
   "name": "@autobusal/routes-search",
-  "version": "1.3.3",
+  "version": "1.4.1",
   "author": "Ferjolt Ozuni",
   "type": "module",
-  "main": "index.ts"
+  "main": "index.ts",
+  "publishConfig": {
+    "access": "public"
+  }
 }

package/services.ts

@@ -1,4 +1,4 @@
-import { useSuspenseQuery, UseSuspenseQueryResult } from '@tanstack/react-query';
+import { useQuery, useSuspenseQuery, UseQueryResult, UseSuspenseQueryResult } from '@tanstack/react-query';
 import { apiClient } from '@autobusal/providers';
 import { CityData } from '@autobusal/providers/types/locations';
 
@@ -13,4 +13,37 @@ export const useGetSuggestions = (): UseSuspenseQueryResult<CityData[]> => (
         ))
     )
   })
-);
+);
+
+/**
+ * The departure city to open the form on, for THIS visitor.
+ *
+ * Claude - Date: 2026-08-29
+ *
+ * obtapi answers with the origin serving the most destinations in the
+ * visitor's own country - see Libraries\Geo\Origin, and note the
+ * precision is the COUNTRY: the geo database is country-level, so this is
+ * "somewhere you can actually leave from near you", not "your city".
+ *
+ * A plain useQuery, not a suspense one: the form must render immediately
+ * with the brand's default and improve on it when the answer arrives -
+ * suspending the whole search box on a geolocation lookup would be a far
+ * worse trade than a slightly-late pre-fill. Null (no database, a private
+ * address, a country we sell nothing from) simply leaves the default.
+ */
+export const useSuggestedOrigin = (enabled: boolean): UseQueryResult<string | null> => (
+  useQuery({
+    queryKey: ['search-suggested-origin'],
+    enabled,
+    // the answer only changes with the brand's inventory, and the visitor's
+    // country does not change mid-session
+    staleTime: 1000 * 60 * 30,
+    retry: false,
+    queryFn: async () => (
+      await apiClient
+        .get('/api/geo/origin')
+        .then(response => response?.data?.from ?? null)
+        .catch(() => null)
+    )
+  })
+);

package/styles.ts

@@ -4,12 +4,32 @@ export const Options = styled.div`
   display: flex;
   gap: 15px;
   margin-bottom: 20px;
+
+  /*
+   * Claude - 2026-09-16 (audit): 44px controls under a thumb. The trip-type
+   * buttons are the shared 32px "medium", the fields below are the theme's
+   * 38px and the search button its 38px default; common is not this
+   * package's to edit, so the phone sizes are set from the containers.
+   * min-height beats the global height, and the desktop form is untouched.
+   */
+  @media (max-width: 639px) {
+    & > button {
+      height: 44px;
+    }
+  }
 `;
 
 export const Columns = styled.div`
   display: flex;
   flex-wrap: wrap;
   gap: 15px;
+
+  // the 44px phone fields - see Options above
+  @media (max-width: 639px) {
+    input:not([type='checkbox']):not([type='radio']), select {
+      min-height: 44px;
+    }
+  }
 `;
 
 export const Column = styled.div`
@@ -43,6 +63,13 @@ export const Action = styled.div`
   justify-content: center;
   align-items: end;
 
+  // the 44px phone search button - see Options above
+  @media (max-width: 639px) {
+    & > button {
+      height: 44px;
+    }
+  }
+
   @media (min-width: 640px) {
     justify-content: right;
   }
@@ -60,6 +87,11 @@ export const PickerContainer = styled.div`
   height: 38px;
   display: flex;
   align-items: center;
+
+  // the pin stays centred in the taller phone field - see Options above
+  @media (max-width: 639px) {
+    height: 44px;
+  }
 `;
 
 export const PickerField = styled.div`

package/CHANGELOG.md

@@ -1,41 +0,0 @@
-# Changelog
-
-## 1.3.3 (2026-08-29)
-
-- A city with a single station lists that station instead of an identical "All Stations" entry beside it; the submitted value stays the city's slug, so URLs are unchanged.
-
-## 1.2.0
-
-**The date picker follows the departure city.** Choosing a "from" city now sets the picker's floor to that city's timezone, and a pre-filled form (a shared link, or the search form kept on the results page) sets it on mount rather than waiting for a keystroke.
-
-An unknown city *clears* the zone rather than keeping the last one — silently flooring a new search on the previous journey's clock is the subtler version of the same bug.
-
-All notable changes to `@autobusal/routes-search` are documented here. This
-project adheres to [Keep a Changelog](https://keepachangelog.com/en/1.1.0/) and
-[Semantic Versioning](https://semver.org/spec/v2.0.0.html).
-
-## [1.1.1] - 2026-08-01
-
-### Fixed
-
-- **The return date could be set before the departure date.** The picker
-  floored at today, so an invalid pair was selectable and only refused by
-  an alert after pressing search. It now floors at whatever departure
-  currently holds, and follows it when that changes.
-
-## [1.1.0] - 2026-07-25
-
-### Fixed
-
-- **Recovered as canonical source.** This package was published to npm and
-  in active use, but had no source anywhere in this monorepo (no
-  `packages/routes-search` directory, no git history) - the only copy of its
-  code existed as the published npm artifact. Recreated here,
-  byte-for-byte identical to the published 1.1.0 tarball (verified via
-  `diff` against the installed `node_modules/@autobusal/routes-search`), so it
-  can be edited and republished like every other package instead of
-  requiring reverse-engineering from `node_modules` or the npm tarball.
-  History prior to this point is not recoverable - no git log,
-  changelog, or other record of it exists.
-
-Authored by Ferjolt Ozuni.