@autobusal/routes-search 1.3.1 → 1.3.3

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # 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.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autobusal/routes-search",
-  "version": "1.3.1",
+  "version": "1.3.3",
   "author": "Ferjolt Ozuni",
   "type": "module",
   "main": "index.ts"

package/utilities.ts

@@ -6,15 +6,69 @@ export const getCities = (cities: CityData[], t: TFunction<'common'>): Autocompl
   const data: AutocompleteData[] = [];
 
   cities.forEach(city => {
+    /*
+     * Claude - Date: 2026-08-21
+     *
+     * Every spelling this place answers to, so the box matches what the
+     * visitor TYPES rather than what their interface language happens to
+     * call the place. `aliases` comes from /api/cities/browse and holds the
+     * name in all 15 languages plus historical and colloquial variants -
+     * "Solun", "Selanik", "Θεσσαλονίκη" and "Salonic" all reach
+     * Thessaloniki. The country name and the slug go in too: the slug is
+     * what appears in the URL, so somebody pasting or half-remembering one
+     * finds the city.
+     *
+     * These are NEVER displayed - `name` is what shows, resolved to the
+     * visitor's own language server-side.
+     */
+    const search = [
+      ...(city.aliases ?? []),
+      city.country?.name ?? '',
+      city.slug
+    ].filter(Boolean);
+
+    const stops = city.stops ?? [];
+
+    /*
+     * Edited: Claude - Date: 2026-08-29
+     *
+     * A CITY WITH ONE STATION OFFERS NO CHOICE. Listing both "(All
+     * Stations)" and the station itself puts two entries in the box for
+     * one place, and the visitor has to read both to discover they are the
+     * same - so the station is named and the pair collapses to a single
+     * line.
+     *
+     * The value stays the CITY's, not the station's: with one station the
+     * search is identical either way, and keeping the city slug keeps the
+     * URL (and everything keyed off it - canonicals, the prerendered pair
+     * pages, the date strip) exactly as it is. The day a second station
+     * appears, this reverts to "All Stations" plus one entry each, on its
+     * own.
+     */
+    if (stops.length === 1) {
+      data.push({
+        name: `${ city.name }, ${ city.country?.name ?? '' } (${ stops[0].name })`,
+        value: `${ city.slug }`,
+        search: [ ...search, stops[0].name ]
+      });
+
+      return;
+    }
+
     data.push({
       name: `${ city.name }, ${ city.country?.name ?? '' } (${ t('all_stations', { ns: 'common' }) })`,
-      value: `${ city.slug }`
+      value: `${ city.slug }`,
+      search
     });
 
-    city.stops?.forEach(stop => {
+    stops.forEach(stop => {
       data.push({
         name: `${ city.name }, ${ city.country?.name ?? '' } (${ stop.name })`,
-        value: `${ city.slug }:${ stop.id }`
+        value: `${ city.slug }:${ stop.id }`,
+        // the stop's own name as well - station names are not translated
+        // (they are what is written on the building) but they are certainly
+        // searched for
+        search: [ ...search, stop.name ]
       });
     });
   });