@autobusal/routes-order 1.37.4 → 1.37.5

package/Facts/questions.ts

@@ -170,17 +170,40 @@ const questions = (facts: FactsData, t: TFunction<'common'>): Question[] => {
     });
   }
 
-  // Asked only where the answer is a real count. `direct` is meaningless
-  // without a total to read it against, and `departures` is that total.
+  /**
+   * DIRECT MEANS NO CHANGE OF BUS. That is the question being asked.
+   *
+   * Edited: Claude - Date: 2026-09-16
+   *
+   * This was answered out of `facts.direct` alone, which counts services
+   * with no intermediate STOPS - so it said "No. Every scheduled service
+   * calls at stops along the way" on 201 of the 202 published pairs, under
+   * a timetable showing a single coach running the whole route. Every bus
+   * in the Balkans calls somewhere, so the answer was structurally always
+   * no. A traveller reads that as "you have to change buses", and so does
+   * an assistant: ChatGPT quoted the contradiction back to us against the
+   * Thessaloniki-Tirana page.
+   *
+   * Nothing in Libraries\Routes\Search ever joins two routes into one
+   * journey, so every service that can be booked here is one bus end to
+   * end, and yes is not a generous reading - it is the fact.
+   *
+   * The stop count keeps its own word, NON-STOP, and its own sentence. It
+   * is the rarer and more valuable fact, still counted out of the total
+   * rather than claimed for the whole pair, but it no longer decides
+   * whether the pair has a direct bus at all.
+   */
   if (facts.departures > 0 && !dead) {
     all.push({
       id: 'direct',
       q: t('routes_order.schema.direct_q', cities),
-      a: facts.direct === 0
-        ? t('routes_order.schema.direct_a_none', cities)
-        : (facts.direct === facts.departures
-            ? t('routes_order.schema.direct_a_all', cities)
-            : t('routes_order.schema.direct_a', { ...cities, count: facts.direct, total: facts.departures }))
+      a: facts.direct === facts.departures
+        ? t('routes_order.schema.direct_a_nonstop_all', cities)
+        : (facts.direct > 0
+            ? t('routes_order.schema.direct_a_nonstop_some', { ...cities, count: facts.direct, total: facts.departures })
+            : (facts.departures === 1
+                ? t('routes_order.schema.direct_a_through_single', cities)
+                : t('routes_order.schema.direct_a_through', { ...cities, total: facts.departures })))
     });
   }
 
@@ -190,9 +213,17 @@ const questions = (facts: FactsData, t: TFunction<'common'>): Question[] => {
     all.push({
       id: 'overnight',
       q: t('routes_order.schema.overnight_q', cities),
+      // Edited: Claude - Date: 2026-09-16
+      // "1 of the 2 scheduled services ... travel overnight" was published
+      // on 40 pairs, in the visible FAQ and in the JSON-LD both. Its own
+      // key rather than an i18next plural, for the reason spelled out
+      // above departures_a_single: the eight Slavic locales here do not
+      // share English's one rule.
       a: facts.overnight === facts.departures
         ? t('routes_order.schema.overnight_a_all', cities)
-        : t('routes_order.schema.overnight_a', { ...cities, count: facts.overnight, total: facts.departures })
+        : (facts.overnight === 1
+            ? t('routes_order.schema.overnight_a_single', { ...cities, total: facts.departures })
+            : t('routes_order.schema.overnight_a', { ...cities, count: facts.overnight, total: facts.departures }))
     });
   }
 

package/Found/Route/Route.tsx

@@ -8,9 +8,20 @@ import { formatDuration, splitPrice } from '../refine';
 import { Button, RouteFeature, Information, Rating } from '@autobusal/common';
 import { useGetSettings } from '@autobusal/providers/services';
 import { createDate, getDayParts } from '@autobusal/utilities';
-import { Container, ButtonDetails, SpecialOffer, FavoriteStop, Trip, Company, Location, Stop, Iso, Time, Price, Amount, Decimals, Currency, PriceNotice, LinkCompany, CodeCompany, Details, Detail, ButtonPolicy, TitleDetail, FeaturesItems, Leaves, Day, NextDay, Standard, Stops, StopsItems, StopRow, StopTime, StopDay, StopBorder } from './styles';
+import { Container, ButtonDetails, SpecialOffer, FavoriteStop, Trip, Company, Location, Stop, Iso, Time, Price, Amount, Decimals, Currency, PriceNotice, SeatsLeft, LinkCompany, CodeCompany, Details, Detail, ButtonPolicy, TitleDetail, FeaturesItems, Leaves, Day, NextDay, Standard, Stops, StopsItems, StopRow, StopTime, StopDay, StopBorder } from './styles';
 import { FoundData } from '@autobusal/providers/types/routes';
 
+/**
+ * At or below this many seats, the count is emphasised.
+ *
+ * Edited: Claude - Date: 2026-09-16
+ *
+ * A coach carries roughly fifty, so single figures are the point at which
+ * the number changes what somebody does about it rather than merely
+ * describing the bus.
+ */
+const LOW_SEATS = 5;
+
 // neutral placeholder so a missing/broken operator logo doesn't render the
 // browser's broken-image icon in the route results
 const LOGO_FALLBACK = "data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/svg' width='120' height='60' viewBox='0 0 120 60'%3E%3Crect width='120' height='60' rx='6' fill='%23e5e7eb'/%3E%3Cpath d='M42 22h36v14a2 2 0 0 1-2 2h-2a3 3 0 0 1-6 0H52a3 3 0 0 1-6 0h-2a2 2 0 0 1-2-2z' fill='%239ca3af'/%3E%3C/svg%3E";
@@ -78,6 +89,21 @@ const Route = ({ data, type, date, passengers, t, onSave, children }: Props): JS
     ? `${ import.meta.env.VITE_API_OBT }/api/external/go/${ data.external.offer_id }`
     : undefined;
 
+  /**
+   * Seats still on sale on this coach.
+   *
+   * Edited: Claude - Date: 2026-09-16
+   *
+   * Typed rather than truthy-checked: 0 is a real answer from obtapi and it
+   * means a full bus, which never reaches a result list - and a `!` test
+   * would have hidden the field on exactly the coach it mattered most on if
+   * one ever did. An external-provider offer (Mode A) carries no trip of
+   * ours to count, so it simply shows nothing.
+   */
+  const seats = typeof data.trip?.available_seats === 'number' && data.trip.available_seats > 0
+    ? data.trip.available_seats
+    : null;
+
   // Edited: Ferjolt Ozuni - Date: 2026-08-03
   // Only today's results carry this - obtapi sends null on every other date,
   // so there is no "in 26 days" badge to suppress here.
@@ -296,6 +322,19 @@ const Route = ({ data, type, date, passengers, t, onSave, children }: Props): JS
             { details ? <HiChevronUp /> : <HiChevronDown /> }
             { t('routes_order.step2.route.details') }
           </ButtonDetails>
+
+          { /* Claude - 2026-09-16 (Ferjolt: "show somewhere on each search
+               result the number of available/bookable seats per route").
+
+               Last in source, and ordered by class rather than by position -
+               see Price in ./styles. It sits inside the fare cell because
+               how many are left is a fact about this fare on this coach, and
+               it is read at the same moment as the price. */ }
+          { seats !== null && (
+            <SeatsLeft className="seats-left" $low={ seats <= LOW_SEATS }>
+              { t('routes_order.step2.route.seats_left', { count: seats }) }
+            </SeatsLeft>
+          ) }
         </Price>
       </Trip>
 

package/Found/Route/styles.ts

@@ -510,6 +510,17 @@ export const Price = styled.div`
     flex: 0 0 auto;
   }
 
+  /*
+   * Claude - 2026-09-16: last line of the fare cell on a phone, on a row of
+   * its own under the note and the toggle. By class for the same reason as
+   * the two above - a dynamic-price drop renders an extra child and shifts
+   * every nth-child index in this block by one.
+   */
+  & > .seats-left {
+    order: 5;
+    flex: 1 1 100%;
+  }
+
   @media (min-width: 768px) {
     flex: 0 0 165px;
     flex-direction: column;
@@ -530,7 +541,8 @@ export const Price = styled.div`
      * just now inside the price cell rather than under the whole card.
      */
     & > .price-notice,
-    & > .details-toggle {
+    & > .details-toggle,
+    & > .seats-left {
       order: initial;
       flex-basis: auto;
     }
@@ -970,3 +982,33 @@ export const Day = styled.span`
   white-space: nowrap;
   color: ${ props => props.theme.primary.normal };
 `;
+
+/**
+ * How many seats are still on sale, under the fare.
+ *
+ * Edited: Claude - Date: 2026-09-16
+ *
+ * Ferjolt asked for the seat count on every result. obtapi has always sent
+ * it (trip.available_seats) and nothing on the card ever showed it, so the
+ * only way to find out how full a coach was, was to try to book it.
+ *
+ * The string is "Seats left: 49", a label and a number, rather than
+ * "49 seats left" - this codebase carries no i18next plural rules, and a
+ * counted noun that has to agree in fifteen languages is how "1 stops"
+ * happened in the timetable. A colon sidesteps the agreement entirely and
+ * translates cleanly into all of them.
+ *
+ * Low stock is emphasised rather than coloured red on its own: it is a real
+ * fact about the bus, not a countdown we invented to hurry anyone.
+ */
+export const SeatsLeft = styled.span<{ $low: boolean }>`
+  font-size: ${ props => props.theme.size.xxs };
+  line-height: 1.25;
+  text-align: left;
+  font-weight: ${ props => (props.$low ? 700 : 400) };
+  color: ${ props => (props.$low ? props.theme.font.error : props.theme.font.faded) };
+
+  @media (min-width: 768px) {
+    text-align: center;
+  }
+`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autobusal/routes-order",
-  "version": "1.37.4",
+  "version": "1.37.5",
   "author": "Ferjolt Ozuni",
   "type": "module",
   "main": "index.ts",