@curless/hotel-storefront 0.49.1 → 0.50.0

package/dist/tools.js

@@ -8,7 +8,8 @@ import { namedExactly, roomNamesOf, roomTypeNamed, withTraditional } from './nam
 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';
-import { MAX_EXTRA_QUANTITY, MAX_ROOMS_PER_LINE, clampCount, pickedKey, shownOf, tripKey, } from './trip.js';
+import { SAME_BOOKING_MS } from './same-booking.js';
+import { MAX_EXTRA_QUANTITY, MAX_ROOMS_PER_LINE, clampCount, pickedKey, shownOf, stampOf, tripKey, } from './trip.js';
 import { DEMO_NOTE, branded, catalogue, noteFor, upFirst } from './vocab.js';
 // The eleven things a guest can do: find hotels, look at one on dates, see what
 // else it offers, build a trip, ask for the rooms, and deal with a booking they
@@ -993,6 +994,8 @@ const TRIP_SCHEMA = {
         dropped: ARR,
         /** A name that fits more than one hotel: the ones it fits (`namesLike`). */
         candidates: ARR,
+        /** The trip as this answer drew it - the card sends it back with its booking form (`stampOf`). */
+        stamp: STR,
     },
     required: ['view'],
 };
@@ -1200,6 +1203,12 @@ const BOOKING_SCHEMA = {
         repeat: BOOL,
         /** A form whose room is no longer the one picked: the pick that would put it back (`pickChanged`). */
         again: OBJ,
+        /** Of `bookings`, the ids asked for minutes before, when the others are new (`BookingResult.reused`). */
+        repeated: STRS,
+        /** `trip`, drawn again: see `stampOf`. */
+        stamp: STR,
+        /** The trip changed after the card that sent the form was drawn, so nothing was booked (`stampOf`). */
+        tripChanged: BOOL,
     },
     required: ['view'],
 };
@@ -1236,6 +1245,8 @@ export const buildOutputSchemas = (n) => ({
             widened: OBJ,
             /** The deployment's only hotel, answered with its page - see HOTEL_PAGE. */
             only: BOOL,
+            /** Only where the client shows no cards: nothing was drawn - see `LIST_NOT_DRAWN`. */
+            notDrawn: STR,
             ...HOTEL_PAGE,
         },
         required: ['view'],
@@ -1377,7 +1388,7 @@ export const buildOutputSchemas = (n) => ({
             inTrip: { type: 'number' },
             planned: { type: 'number' },
             dropped: ARR,
-            /** The stays not booked yet, together - only when all are priced in one currency. */
+            /** The stays not in the trip or booked yet - what "add them" adds - together, only when all are priced in one currency. */
             staysTotal: STR,
             /** What the text says of a long stay at one hotel, in the same words. */
             nudges: STRS,
@@ -2646,6 +2657,8 @@ const currentTrip = async (deps, key, who, store = key) => {
     // card's own trip view.
     if (lines.length)
         deps.trips.show(store, shownOf(parts));
+    // What this answer drew, for the card's form to send back - see `stampOf`.
+    const stamp = lines.length ? { stamp: stampOf(lines, deps.trips.shown(store)) } : {};
     const [only] = parts;
     const orphans = needsRoomIn(deps.tools, lines, parts);
     const needs = orphans.needsRoom.length ? { needsRoom: orphans.needsRoom } : {};
@@ -2659,6 +2672,7 @@ const currentTrip = async (deps, key, who, store = key) => {
                 ...common(deps, key),
                 ...quoteForCard(only, groupOf),
                 ...needs,
+                ...stamp,
                 count: lines.length,
             },
             text: [
@@ -2700,6 +2714,7 @@ const currentTrip = async (deps, key, who, store = key) => {
                 .join(' · '),
             ...needs,
             ...(shaped.some((p) => p.dropped) ? { dropped: shaped.flatMap((p) => p.dropped ?? []) } : {}),
+            ...stamp,
             count: lines.length,
         },
         text: [
@@ -3014,6 +3029,29 @@ export const PLAN_DRAWN_ONCE = 'The plan on the card is this one. Something to c
 export const PLAN_NOT_DRAWN = 'Nothing was drawn: this app shows no cards, so the guest has seen none of this plan. Tell it to them in your reply, day by day - where each day is spent, and each stay with its hotel, dates, price and terms as above. Something to change that you notice now - a sight closed on its day, a day too far from its hotel - is said in that reply, not planned again: the plan is made again only when the guest asks for the change.';
 /** A comparison's answer, for a client that shows no cards - see `PLAN_NOT_DRAWN`. */
 export const COMPARISON_NOT_DRAWN = "Nothing was drawn: this app shows no cards, so the guest has seen none of this comparison. Compare the hotels in your reply, with each one's price and terms as given here - each is its cheapest room that sleeps the party.";
+/**
+ * A list's answer, for a client that shows no cards - see `PLAN_NOT_DRAWN`.
+ *
+ * ⚠️ A LIST WITH NO CARD IS THE HOTELS IN WORDS (2026-10-29 audit, H4). The
+ * answer said "12 Huazhu hotels in 上海." and nothing else - the hotels were on
+ * the card - so a client that draws none had a count to tell the guest.
+ */
+export const LIST_NOT_DRAWN = "Nothing was drawn: this app shows no cards, so the guest has seen none of these hotels. Tell them in your reply - each one's name, area and price as given here:";
+/** One hotel of a list as a line, for a client that shows no cards. */
+const listedInWords = (h) => {
+    const where = [h.area, h.city]
+        .filter((x) => !!x)
+        .map((x) => hotelName(x))
+        .join(', ');
+    const price = h.forStay
+        ? `${formatMoney(h.forStay.total)} for the stay incl. taxes and fees, ${hotelName(h.forStay.room)}${h.forStay.rooms > 1 ? ` × ${h.forStay.rooms}` : ''}`
+        : h.noRoomsForStay
+            ? 'no room for those nights'
+            : h.fromNightly
+                ? `from ${formatMoney(h.fromNightly)} a night`
+                : '';
+    return `- ${hotelName(h.name)} (\`${h.id}\`)${where ? ` · ${where}` : ''}${price ? `: ${price}` : ''}`;
+};
 /**
  * One column of a comparison as a line, for a client that shows no cards:
  * what the card's rows say - where, the rating, the room, the price - and the
@@ -3022,7 +3060,11 @@ export const COMPARISON_NOT_DRAWN = "Nothing was drawn: this app shows no cards,
 const comparedInWords = (c) => {
     const h = c.hotel;
     const about = [
-        [h.area, h.city].filter(Boolean).join(', '),
+        // The group's own words, as the name is (2026-10-29 audit, H9).
+        [h.area, h.city]
+            .filter((x) => !!x)
+            .map((x) => hotelName(x))
+            .join(', '),
         h.rating ? `${h.rating.value}/${h.rating.scaleMax} from ${h.rating.count} reviews` : '',
     ]
         .filter(Boolean)
@@ -3226,6 +3268,73 @@ const pickChanged = (deps, key, args) => {
         text: words,
     };
 };
+/** A booking that is still one - what `bookedFromThisForm` may answer with. */
+const LIVE_STATUSES = new Set([
+    'requested',
+    'confirmed',
+    'unconfirmed',
+    'awaiting-payment',
+]);
+/**
+ * The booking this very form made a few minutes ago, when the slot is empty
+ * because of it - or undefined.
+ *
+ * ⚠️ THE GUEST'S OWN BOOKING IS NOT "ANOTHER CARD'S" (2026-10-29 audit, H2c).
+ * A booking empties the pick; when its answer is lost on the way (a host that
+ * gives up before 华住 has answered), the guest presses again on the same form
+ * and was told another card had booked or let go of the room - about the
+ * booking they had just made. A live booking of this conversation, for this
+ * email, this room, these nights and this count, made within the minutes a
+ * repeat is recognised in (`SAME_BOOKING_MS`), is that booking: said as one,
+ * and nothing is booked again.
+ */
+const bookedFromThisForm = async (deps, ctx, key, args) => {
+    const email = str(args.email).trim().toLowerCase();
+    const room = str(args.room);
+    if (!email || !room)
+        return undefined;
+    const said = (v) => v !== undefined && v !== null && v !== '';
+    const rooms = said(args.rooms) ? Number(args.rooms) : 1;
+    const at = deps.trips.now();
+    let mine;
+    try {
+        mine = await deps.backend.listBookings(identity(deps, ctx, key, { email }));
+    }
+    catch {
+        return undefined;
+    }
+    const b = mine.find((x) => LIVE_STATUSES.has(x.status) &&
+        at - Date.parse(x.createdAt) < SAME_BOOKING_MS &&
+        x.hotelId === str(args.hotel) &&
+        x.guest.email.trim().toLowerCase() === email &&
+        (!said(args.guests) || Number(args.guests) === x.stay.guests) &&
+        (said(args.startTime)
+            ? x.hourly?.start === str(args.startTime)
+            : (!said(args.checkIn) || str(args.checkIn) === x.stay.checkIn) &&
+                (!said(args.checkOut) || str(args.checkOut) === x.stay.checkOut)) &&
+        x.quote.lines.some((l) => l.kind === 'room' && l.roomRef === room && (l.quantity ?? 1) === rooms));
+    if (!b)
+        return undefined;
+    return {
+        structured: {
+            view: 'booking',
+            ...common(deps, key),
+            booking: madeForCard(deps, b),
+            paid: false,
+            holdUntil: b.holdUntil,
+            total: formatMoney(b.quote.total),
+            repeat: true,
+        },
+        text: [
+            'NOTHING NEW WAS BOOKED: this booking form already booked this room a few minutes ago, and that answer did not reach the card - this is the booking made then.',
+            `**${confirming(deps, b) ? 'Being confirmed' : b.confirmationCode}** · ${b.hotelName} · ${stayWords(b.stay, b.hourly)}`,
+            bookedLine(deps, b) ?? '',
+            deps.vocab.demo ? noteFor(deps.vocab) : '',
+        ]
+            .filter(Boolean)
+            .join('\n'),
+    };
+};
 /**
  * A room line, validated and priced against the hotel's calendar and put into
  * one slot of the trip store - THE ONE ROOM-LINE CODE, for `add_to_trip` (`store`
@@ -3460,6 +3569,20 @@ const bookTripLines = async (deps, t, ctx, key, store, args, nothingThere) => {
             text: nothingThere,
         };
     }
+    // ⚠️ THE TRIP THE CARD DREW, OR NOTHING - see `stampOf`. The assistant books
+    // the trip as it is now (no stamp); a card's form books the one it showed.
+    const drawn = str(args.stamp);
+    if (drawn && drawn !== stampOf(inTrip, deps.trips.shown(store))) {
+        const again = await currentTrip(deps, key, identity(deps, ctx, key), store);
+        return {
+            ...again,
+            structured: { ...again.structured, tripChanged: true },
+            text: [
+                'Nothing was booked: the trip changed after the card the guest pressed Book on was drawn - a stay was added, taken out or changed in this conversation, or its price moved. It is shown again as it is now; the guest checks it and presses Book again.',
+                again.text,
+            ].join('\n\n'),
+        };
+    }
     // ⚠️ ONE GROUP AT A TIME, in a market: a booking cannot be all-or-nothing
     // across two groups' systems (Marketplace, §6-1). `group` books one
     // group's stays and leaves the rest in the trip.
@@ -3506,7 +3629,7 @@ const bookTripLines = async (deps, t, ctx, key, store, args, nothingThere) => {
     const { requests, dropped } = readRequests(args);
     try {
         const shown = deps.trips.shown(store);
-        const { booked: made, failed, repeat, } = await deps.backend.requestBooking({ lines, ...(shown ? { shown } : {}) }, {
+        const { booked: made, failed, repeat, reused, } = await deps.backend.requestBooking({ lines, ...(shown ? { shown } : {}) }, {
             ...guest,
             ...(str(args.phone) ? { phone: str(args.phone) } : {}),
             ...(requests?.note ? { notes: requests.note } : {}),
@@ -3590,6 +3713,8 @@ const bookTripLines = async (deps, t, ctx, key, store, args, nothingThere) => {
         }));
         const [first] = booked;
         const several = booked.length > 1;
+        // The stays asked for minutes before, beside new ones - see `reused`.
+        const earlier = booked.filter(({ booking }) => reused?.includes(booking.id));
         const waiting = booked.filter(({ booking }) => confirming(deps, booking)).length;
         const notBooked = failed.map((f, i) => ({
             hotelId: f.hotelId,
@@ -3683,13 +3808,16 @@ const bookTripLines = async (deps, t, ctx, key, store, args, nothingThere) => {
                     : {}),
                 ...(notBooked.length ? { failed: notBooked } : {}),
                 ...(repeat ? { repeat: true } : {}),
+                ...(earlier.length ? { repeated: earlier.map(({ booking }) => booking.id) } : {}),
             },
             text: [
                 // ⚠️ THE SAME BOOKING ASKED FOR AGAIN IS SAID TO BE THAT, never a new
                 // one (2026-10-27 audit, F2): it was answered as if just booked.
                 repeat
                     ? `NOTHING NEW WAS BOOKED: the same ${booked.length > 1 ? 'stays' : 'stay'} for the same guest ${booked.length > 1 ? 'were' : 'was'} asked for a few minutes ago, and ${booked.length > 1 ? 'these are the bookings' : 'this is the booking'} made then. A booking for someone else is made under their name.`
-                    : '',
+                    : earlier.length
+                        ? `NOT ALL NEW: ${listAnd(earlier.map(({ booking }) => booking.confirmationCode || booking.hotelName))} ${earlier.length > 1 ? 'were' : 'was'} asked for a few minutes ago for the same guest and ${earlier.length > 1 ? 'are the bookings' : 'is the booking'} made then; the rest were booked now.`
+                        : '',
                 // ⚠️ BOOKED IS CONFIRMED: a stay still being confirmed is said as
                 // that, and not counted (总线程, 2026-10-14).
                 notBooked.length || (booked.length > 1 && waiting)
@@ -3736,7 +3864,10 @@ const bookTripLines = async (deps, t, ctx, key, store, args, nothingThere) => {
             structured: {
                 view: 'error',
                 ...common(deps, key),
-                error: 'not-bookable',
+                // ⚠️ NOT KNOWN IS NOT "NOT BOOKED" (2026-10-29 audit, H2): the card
+                // said a fault as "could not be booked, try again later" - the one
+                // thing a guest must not do after a booking that may have gone through.
+                error: words === BOOKING_FAULT ? 'uncertain' : 'not-bookable',
                 detail: words,
             },
             text: words,
@@ -4177,6 +4308,9 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 ...(answer.overBudget ? { overBudget: answer.overBudget } : {}),
                 ...(widened?.found ? { widened } : {}),
                 ...(answer.currencies.length > 1 ? { currencies: answer.currencies } : {}),
+                // 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 } : {}),
             },
             text: [
                 `${total} ${branded(deps.vocab, total === 1 ? deps.vocab.noun : deps.vocab.nounPlural)}${where}.`,
@@ -4189,6 +4323,9 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                     : '',
                 ...priceNotes,
                 widened?.found ? `Nothing matched ${widened.from}, so this is ${widened.to}.` : '',
+                ctx.noCards && hotels.length
+                    ? `\n${LIST_NOT_DRAWN}\n${hotels.map(listedInWords).join('\n')}\n`
+                    : '',
                 forPlans,
             ]
                 .filter(Boolean)
@@ -4648,8 +4785,10 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
     // a trip is - and the trip left as it is.
     if (name === t.bookPicked) {
         const changed = pickChanged(deps, key, args);
-        if (changed)
-            return direct(changed);
+        if (changed) {
+            const emptied = !deps.trips.read(pickedKey(key)).some((l) => l.kind === 'room');
+            return direct((emptied && (await bookedFromThisForm(deps, ctx, key, args))) || changed);
+        }
         return direct(await bookTripLines(deps, t, ctx, key, pickedKey(key), args, 'Nothing is picked: the guest presses Reserve on a room first.'));
     }
     if (name === t.listBookings) {
@@ -5267,7 +5406,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                         `Priced stays (${plan.stay?.guests ?? 2} guests; ${t.addToTrip} takes each one's hotel id, room id and dates):`,
                         ...stays,
                         ...(plan.staysTotal
-                            ? [`Together, the stays not booked yet: ${plan.staysTotal}.`]
+                            ? [`Together, the stays not in the trip or booked yet: ${plan.staysTotal}.`]
                             : []),
                         ...nudges,
                     ]