@curless/hotel-storefront 0.50.2 → 0.50.4

package/dist/tools.js

@@ -5,6 +5,7 @@ import { DESCRIPTION_MAX, POLICY_MAX, hotelName, hotelText } from './fence.js';
 import { clockOf, minutesOf, startProblem, stayWords, windowWords } from './hourly.js';
 import { amountsIn, formatMoney, keepingAmounts, money, multiplyMoney, toMinor } from './money.js';
 import { namedExactly, roomNamesOf, roomTypeNamed, withTraditional } from './names.js';
+import { overlapsOf } from './overlap.js';
 import { MAX_COMPARED, MAX_CONS, MAX_DAYS, MAX_ITEMS_PER_DAY, MAX_MOMENTS, MAX_PROS, buildItinerary, candidatesFor, compareHotels, namesMoreThanOne, } from './present.js';
 import { regionOf } from './regions.js';
 import { activeKeysOf, byRoomType, facetsOf, keyWords, matchesTerms, optionsOf, roomTermsOf, termWords, } from './room-filters.js';
@@ -96,13 +97,17 @@ export const buildTools = (n, v, caps = FULL_CAPABILITIES) => forCapabilities(al
  * The list tool where the deployment is one hotel in all (2026-10-15, Huazhu's
  * test system): asked with no place it answers with that hotel's page, not
  * with places to choose from - and says so, or the model offers to look in
- * another city (three smoke runs did). Never in a market.
+ * another city (three smoke runs did). Where it is a few (`HOME_LIST_MAX`,
+ * Huazhu's production, 2026-10-30), asked with no place it lists all of them.
+ * Never in a market.
  */
-export const oneHotelListing = (t, brand) => {
+export const homeListing = (t, brand, count) => {
     const v = brand.vocab;
-    if (t.name !== brand.tools.listHotels || v.market)
+    if (t.name !== brand.tools.listHotels || v.market || count < 1 || count > HOME_LIST_MAX)
         return t;
-    const one = `This deployment has ONE ${v.noun} in all: asked with no place, it answers with that ${v.noun}'s own page, and there is no other ${v.noun} or city to list or search. `;
+    const one = count === 1
+        ? `This deployment has ONE ${v.noun} in all: asked with no place, it answers with that ${v.noun}'s own page, and there is no other ${v.noun} or city to list or search. `
+        : `This deployment has ${count} ${v.nounPlural} in all: asked with no place, it lists every one of them, and there is no other ${v.noun} or city to list or search. `;
     const places = /Asked with no place[\s\S]*?themselves\. /;
     return {
         ...t,
@@ -1161,6 +1166,12 @@ const HOTEL_PAGE = {
      * none - and never in what the assistant is told.
      */
     soldOut: ARR,
+    /**
+     * Where some rates are listed but not booked here (`Room.notSoldHere`, paid
+     * in advance on a deployment that takes no payment): what that means and
+     * what to offer - the same sentence as the text.
+     */
+    notSoldHere: STR,
 };
 /**
  * What request_booking answers - and book_picked_room, which books the same way
@@ -1209,6 +1220,8 @@ const BOOKING_SCHEMA = {
         stamp: STR,
         /** The trip changed after the card that sent the form was drawn, so nothing was booked (`stampOf`). */
         tripChanged: BOOL,
+        /** Two stays at two hotels on the same nights, each with what the text says of it (`overlapsOf`). */
+        overlaps: ARR,
     },
     required: ['view'],
 };
@@ -1245,6 +1258,10 @@ export const buildOutputSchemas = (n) => ({
             widened: OBJ,
             /** The deployment's only hotel, answered with its page - see HOTEL_PAGE. */
             only: BOOL,
+            /** A deployment of a few, answered with the list of all of them (`HOME_LIST_MAX`). */
+            home: BOOL,
+            /** With `only` or `home`: every hotel there is, and that there are no others - as the text says it. */
+            inAll: STR,
             /** Only where the client shows no cards: nothing was drawn - see `LIST_NOT_DRAWN`. */
             notDrawn: STR,
             ...HOTEL_PAGE,
@@ -1392,6 +1409,8 @@ export const buildOutputSchemas = (n) => ({
             staysTotal: STR,
             /** What the text says of a long stay at one hotel, in the same words. */
             nudges: STRS,
+            /** Two stays at two hotels on the same nights, each with what the text says of it (`overlapsOf`). */
+            overlaps: ARR,
             /** Only for a client that shows no cards: that nothing was drawn - `PLAN_NOT_DRAWN`. */
             notDrawn: STR,
         },
@@ -2160,10 +2179,22 @@ const LABEL_WORDS = {
 const labelWords = (r) => {
     const words = [
         ...(r.labels ?? []).map((l) => LABEL_WORDS[l]),
-        ...(r.payment === 'prepaid' ? ['paid in advance'] : []),
+        ...(r.payment === 'prepaid'
+            ? [r.notSoldHere ? 'paid in advance - not booked here' : 'paid in advance']
+            : []),
     ];
     return words.length ? ` [${words.join(', ')}]` : '';
 };
+/**
+ * What is said of rates listed but not booked here (`Room.notSoldHere`) - in
+ * the text and, word for word, in the structure (`notSoldHere`): the model reads
+ * the structure (2026-10-28). Listed and marked as the group sells them; the
+ * card says no more than 预付 on them (robin, 2026-10-30), so the assistant is
+ * the one who knows - and says it when the guest wants one.
+ */
+export const notSoldHereWords = (group) => `Rates marked paid in advance are listed as ${group} sells them, but a stay paid in advance cannot be booked here yet: this connector books rates paid at the hotel. If the guest wants one, say so plainly - nothing is booked or charged - and offer a rate paid at the hotel, other dates, or booking it with ${group} directly.`;
+/** The group a hotel is with, by name - the deployment's own brand outside a market. */
+const groupNameOf = (deps, hotelId) => holderOf(deps, deps.backend.supplierOf?.(hotelId)).name;
 /** What a rate says about cancelling, as the text puts it. */
 const cancelWords = (r) => r.freeCancellationUntil
     ? `free to cancel until ${freeUntilText(r)}`
@@ -2370,7 +2401,13 @@ const dateFlex = (dates, stay) => {
         dates: dates.map((d) => ({
             checkIn: d.checkIn,
             checkOut: d.checkOut,
-            ...(d.from ? { from: formatMoney(d.from) } : { soldOut: true }),
+            // "Sold out" only when it is (D2, robin 2026-10-30): a day with rooms
+            // that cannot be booked here says why instead.
+            ...(d.from
+                ? { from: formatMoney(d.from) }
+                : !d.why || d.why === 'sold-out'
+                    ? { soldOut: true }
+                    : { noRoomsCode: d.why }),
             ...(d.chosen ? { chosen: true } : {}),
             // The chosen date cannot match: `beatsBy` is a price strictly below it.
             ...(beatsBy !== undefined && d.from?.minor === beatsBy ? { best: true } : {}),
@@ -2665,6 +2702,19 @@ const currentTrip = async (deps, key, who, store = key) => {
     const groupOf = deps.backend.supplierOf
         ? (id) => deps.backend.supplierOf?.(id)
         : undefined;
+    // Two hotels on the same nights - said, never refused (overlap.ts). In
+    // both, in the same words.
+    const overlaps = overlapsOf(parts.flatMap((p) => p.lines.flatMap((l) => l.kind === 'room' && l.stay && !l.hourly
+        ? [
+            {
+                hotelId: l.hotelId,
+                name: hotelName(l.hotelName),
+                checkIn: l.stay.checkIn,
+                checkOut: l.stay.checkOut,
+            },
+        ]
+        : [])));
+    const overlapping = overlaps.length ? { overlaps } : {};
     if (parts.length === 1 && only) {
         return {
             structured: {
@@ -2673,10 +2723,12 @@ const currentTrip = async (deps, key, who, store = key) => {
                 ...quoteForCard(only, groupOf),
                 ...needs,
                 ...stamp,
+                ...overlapping,
                 count: lines.length,
             },
             text: [
                 tripText(only, false),
+                ...overlaps.map((o) => o.said),
                 orphans.line,
                 deps.vocab.demo && lines.length ? noteFor(deps.vocab) : '',
             ]
@@ -2715,11 +2767,13 @@ const currentTrip = async (deps, key, who, store = key) => {
             ...needs,
             ...(shaped.some((p) => p.dropped) ? { dropped: shaped.flatMap((p) => p.dropped ?? []) } : {}),
             ...stamp,
+            ...overlapping,
             count: lines.length,
         },
         text: [
             `This trip is in ${parts.length} currencies, priced separately and not converted - each hotel is booked and paid in its own.`,
             ...parts.map((q) => tripText(q, false)),
+            ...overlaps.map((o) => o.said),
             orphans.line,
             deps.vocab.demo ? noteFor(deps.vocab) : '',
         ]
@@ -2824,6 +2878,17 @@ dated = false) => {
  * read better as hotels, grouped by city.
  */
 export const CITY_PICKER_MIN = 4;
+/**
+ * Up to how many hotels a whole deployment is a list rather than places to
+ * choose from. Huazhu's production had two, in two cities, and "where to?"
+ * over two city chips and "browse all 2" had the model list them all again -
+ * two cards for one question (robin, 2026-10-30). Six is what the card shows
+ * whole at every width, with no "Next" (its inline page is five rows on a
+ * phone and lets two more in - see the card's `pageSize`): a place picker
+ * over that many saves no one a tap. A group of nine in four cities still
+ * chooses a city first.
+ */
+export const HOME_LIST_MAX = 6;
 const suggestionsFor = async (deps, d) => {
     const view = String(d.view ?? '');
     const out = [];
@@ -3078,7 +3143,11 @@ const comparedInWords = (c) => {
                 : p.refundable
                     ? 'no free cancellation'
                     : 'non-refundable',
-            p.payment === 'prepaid' ? 'paid in advance' : '',
+            p.payment === 'prepaid'
+                ? p.notSoldHere
+                    ? 'paid in advance - not booked here'
+                    : 'paid in advance'
+                : '',
             p.breakfast ? 'breakfast included' : 'no breakfast',
         ]
             .filter(Boolean)
@@ -3493,7 +3562,10 @@ const putRoomLine = async (deps, t, ctx, key, store, args, detail) => {
     let took = '';
     const type = room ? undefined : roomTypeNamed(dated.rooms, roomRef);
     if (type) {
-        const fits = type.rooms.filter((r) => r.sleeps >= stay.guests && r.roomsLeft >= wanted);
+        const fitting = type.rooms.filter((r) => r.sleeps >= stay.guests && r.roomsLeft >= wanted);
+        // A rate booked here before one only shown (`notSoldHere`), where the type has both.
+        const sold = fitting.filter((r) => !r.notSoldHere);
+        const fits = sold.length ? sold : fitting;
         const [first, ...rest] = fits;
         const cheapest = first
             ? rest.reduce((a, b) => (b.stayTotal.minor < a.stayTotal.minor ? b : a), first)
@@ -3535,6 +3607,45 @@ const putRoomLine = async (deps, t, ctx, key, store, args, detail) => {
             text: `${type ? `${detail.hotel.name} has no ${type.name} that sleeps ${stay.guests} and has ${wanted === 1 ? 'a room' : `${wanted} rooms`} left for those nights.` : `${detail.hotel.name} has no room type by that name for those nights.`} It has: ${dated.rooms.map((r) => `${r.name} (${han && r.names?.zh ? `${r.names.zh}, ` : ''}\`${r.id}\`)`).join(', ')}. Call again with the id.`,
         };
     }
+    // ⚠️ LISTED, NOT BOOKED HERE (robin, 2026-10-30): a rate paid in advance
+    // where this deployment takes no payment goes in no trip and no pick. The
+    // press on its Reserve - or the assistant asking for it - is answered with
+    // what is true and what is left: nothing booked, nothing charged; the rates
+    // here paid at the hotel, a press each; other dates; or the group itself.
+    // Never a review that fails at Book, after the guest typed their name.
+    if (room.notSoldHere) {
+        const open = dated.rooms.filter((r) => !r.notSoldHere && r.sleeps >= stay.guests && r.roomsLeft >= wanted);
+        const group = groupNameOf(deps, hotelId);
+        const said = [
+            `Nothing was booked or charged: ${room.name} is paid in advance, and a stay paid in advance cannot be booked here yet - this connector books rates paid at the hotel.`,
+            open.length
+                ? `Paid at the hotel at ${detail.hotel.name} for those nights: ${open.map((r) => `${r.name} (\`${r.id}\`)`).join(', ')}.`
+                : `No rate at ${detail.hotel.name} for those nights is paid at the hotel.`,
+            `Other dates, or booking this rate with ${group} directly, are the other ways on.`,
+        ].join(' ');
+        return {
+            structured: {
+                view: 'error',
+                ...common(deps, key),
+                error: 'prepaid-not-here',
+                room: { id: room.id, name: room.name, ...(room.names ? { names: room.names } : {}) },
+                rooms: open.map((r) => ({
+                    id: r.id,
+                    name: r.name,
+                    ...(r.names ? { names: r.names } : {}),
+                })),
+                hotel: {
+                    id: detail.hotel.id,
+                    name: detail.hotel.name,
+                    ...(detail.hotel.names ? { names: detail.hotel.names } : {}),
+                },
+                stay,
+                group,
+                said,
+            },
+            text: said,
+        };
+    }
     deps.trips.set(store, {
         kind: 'room',
         hotelId,
@@ -4091,17 +4202,33 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
         const [sole] = hotels;
         if (choosing && total === 1 && sole && !deps.backend.suppliers) {
             const page = await answerTool(deps, t.getHotel, { hotel: sole.id, ...stayArgs }, ctx);
+            // ⚠️ NOT "THE ONLY ONE HERE": read as "in this list", it had the
+            // assistant ask which city to list the rest in (2026-10-15 smoke run).
+            // And not "tell me another city": the second run said there was one
+            // hotel, then offered to look in another city (2026-10-15).
+            const inAll = `This connector has one ${branded(deps.vocab, deps.vocab.noun)} in all, ${sole.name}${sole.city ? ` in ${sole.city}` : ''}. There is no other hotel and no other city here - nothing else to search for or to offer.`;
             return {
                 ...page,
-                structured: { ...page.structured, only: true },
-                // ⚠️ NOT "THE ONLY ONE HERE": read as "in this list", it had the
-                // assistant ask which city to list the rest in (2026-10-15 smoke run).
-                // And not "tell me another city": the second run said there was one
-                // hotel, then offered to look in another city (2026-10-15).
-                text: `This connector has one ${branded(deps.vocab, deps.vocab.noun)} in all, ${sole.name}${sole.city ? ` in ${sole.city}` : ''}. There is no other hotel and no other city here - nothing else to search for or to offer. Its page:\n\n${page.text}`,
+                // In both, in the same words: a host may hand its model this rather
+                // than the text.
+                structured: { ...page.structured, only: true, inAll },
+                text: `${inAll} Its page:\n\n${page.text}`,
             };
         }
-        if (choosing && facets) {
+        // ⚠️ A FEW HOTELS ARE NO CHOICE OF PLACES EITHER (robin, 2026-10-30):
+        // Huazhu's production had two, in two cities, and its home was "where to?"
+        // over two city chips and "browse all 2" - the model then listed them
+        // again, a second card for the same question. Where the whole deployment
+        // fits on one page - never a market, whose home is its groups - the home
+        // is the list of all of them, marked `home` so the card keeps the home's
+        // own ways in on it.
+        const homeList = !!choosing &&
+            !asked.city &&
+            !deps.backend.suppliers &&
+            total > 1 &&
+            total <= HOME_LIST_MAX &&
+            hotels.length === total;
+        if (choosing && facets && !homeList) {
             const countries = facets.countries.map((c) => ({
                 name: c.name,
                 count: c.count,
@@ -4151,7 +4278,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                     ...(ask.stay ? { stay: ask.stay } : {}),
                     ...(today ? { today } : {}),
                 },
-                text: `${total} ${branded(deps.vocab, n)} in ${facets.cities.length} cities across ${countries.length} countries, to choose from by region, country and city - ${regions.map((r) => `${r.name}: ${every(r.countries)}`).join('; ')}. A city or a country lists its ${n}. ${forPlans}`,
+                text: `${total} ${branded(deps.vocab, n)} in ${facets.cities.length} cities across ${countries.length} ${countries.length === 1 ? 'country' : 'countries'}, to choose from by region, country and city - ${regions.map((r) => `${r.name}: ${every(r.countries)}`).join('; ')}. A city or a country lists its ${n}. ${forPlans}`,
             };
         }
         if (hotels.length === 0) {
@@ -4286,10 +4413,25 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                         : ''}.`
                     : '',
         ].filter(Boolean);
+        // A price on the list that is not booked here (`notSoldHere`): said as on
+        // the hotel's page, in both - see `notSoldHereWords`.
+        const notSoldOn = hotels.find((h) => h.forStay?.notSoldHere);
+        const notSold = notSoldOn ? notSoldHereWords(groupNameOf(deps, notSoldOn.id)) : '';
+        if (notSold)
+            priceNotes.push(notSold);
+        // Every one by name and city, and that there are no others - or, told
+        // only "2 hotels", the model asked for the list again to learn which.
+        const inAll = homeList
+            ? `This connector has ${total} ${branded(deps.vocab, deps.vocab.nounPlural)} in all, and this answer lists every one: ${hotels
+                .map((h) => `${hotelName(h.name)}${h.city ? ` in ${h.city}` : ''}`)
+                .join('; ')}. There is no other ${deps.vocab.noun} and no other city here - nothing else to list, search for or offer.`
+            : '';
         return {
             structured: {
                 view: 'hotels',
                 ...common(deps, key),
+                // The home, as a list - see `homeList`. The same sentence as the text.
+                ...(homeList ? { home: true, inAll } : {}),
                 hotels: forCard,
                 total,
                 shown: hotels.length,
@@ -4311,9 +4453,11 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 // In both, like a plan's: a host with no card may hand its model this
                 // rather than the text.
                 ...(ctx.noCards ? { notDrawn: LIST_NOT_DRAWN } : {}),
+                ...(notSold ? { notSoldHere: notSold } : {}),
             },
             text: [
-                `${total} ${branded(deps.vocab, total === 1 ? deps.vocab.noun : deps.vocab.nounPlural)}${where}.`,
+                inAll ||
+                    `${total} ${branded(deps.vocab, total === 1 ? deps.vocab.noun : deps.vocab.nounPlural)}${where}.`,
                 // ⚠️ NO "BELOW", NO "TAP". This sentence is read out by a voice
                 // assistant as often as it is drawn next to a card, and a guest whose
                 // host has no card at all cannot tap the one it tells them to tap. The
@@ -4445,6 +4589,9 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
         // card needs no call to the server.
         const terms = roomTermsOf(args);
         const soldOut = soldOutTypesOf(detail, enough, roomsWanted);
+        const notSold = enough.some((r) => r.notSoldHere)
+            ? notSoldHereWords(groupNameOf(deps, h.id))
+            : '';
         const rooms = enough.map((r) => ({
             ...r,
             facets: facetsOf(r),
@@ -4484,6 +4631,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 activeFilters: activeKeysOf(terms),
                 // Types with no room to book for these nights, said and not left out.
                 ...(byTheHour || !soldOut.length ? {} : { soldOut }),
+                ...(notSold ? { notSoldHere: notSold } : {}),
                 ...flex.structured,
             },
             text: [
@@ -4494,6 +4642,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 terms && flex.text
                     ? `${flex.text} (That is the cheapest room of any kind, not only one with ${termWords(terms)}.)`
                     : flex.text,
+                notSold,
                 deps.vocab.demo ? noteFor(deps.vocab) : '',
             ]
                 .filter(Boolean)
@@ -5376,7 +5525,11 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
         // Each stay's rate says its terms where they bind - see `ItineraryItem.refundable`.
         const termsOf = (x) => [
             x.refundable === false ? 'non-refundable' : '',
-            x.payment === 'prepaid' ? 'paid in advance' : '',
+            x.payment === 'prepaid'
+                ? x.notSoldHere
+                    ? 'paid in advance - not booked here'
+                    : 'paid in advance'
+                : '',
         ]
             .filter(Boolean)
             .join(', ');
@@ -5388,6 +5541,10 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
         // text said of a long stay. The model's part of the answer is the same on
         // either side.
         const nudges = stays.length ? await longStayNudges(deps, stayItems, plan.days) : [];
+        // Two hotels on the same nights of the plan - said, never refused (overlap.ts).
+        const overlaps = overlapsOf(stayItems.flatMap((x) => x.checkIn && x.checkOut
+            ? [{ hotelId: x.id, name: hotelName(x.name), checkIn: x.checkIn, checkOut: x.checkOut }]
+            : []));
         return {
             structured: {
                 view: 'itinerary',
@@ -5395,6 +5552,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 ...plan,
                 gap,
                 ...(nudges.length ? { nudges } : {}),
+                ...(overlaps.length ? { overlaps } : {}),
                 // In both, like the nudges: a host with no card may hand its model
                 // this rather than the text.
                 ...(ctx.noCards ? { notDrawn: PLAN_NOT_DRAWN } : {}),
@@ -5409,6 +5567,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                             ? [`Together, the stays not in the trip or booked yet: ${plan.staysTotal}.`]
                             : []),
                         ...nudges,
+                        ...overlaps.map((o) => o.said),
                     ]
                     : []),
                 ...plan.dropped.map((d) => `Left out: ${d.name} — ${d.why}`),