@curless/hotel-storefront 0.51.6 → 0.52.0

package/dist/tools.js

@@ -7,6 +7,7 @@ import { amountsIn, formatMoney, keepingAmounts, money, multiplyMoney, toMinor }
 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 { withReference } from './reference.js';
 import { regionOf } from './regions.js';
 import { activeKeysOf, byRoomType, facetsOf, keyWords, matchesTerms, optionsOf, roomTermsOf, termWords, } from './room-filters.js';
 import { SAME_BOOKING_MS } from './same-booking.js';
@@ -1337,6 +1338,10 @@ export const buildOutputSchemas = (n) => ({
             stayDefault: STR,
             /** Only where the client shows no cards: nothing was drawn - see `LIST_NOT_DRAWN`. */
             notDrawn: STR,
+            /** In a market: the groups that did not answer in time - none of their hotels is in this answer (`degradedOf`). */
+            degraded: ARR,
+            /** What `degraded` means, as the text says it: not that those groups have none here. */
+            degradedSaid: STR,
             ...HOTEL_PAGE,
         },
         required: ['view'],
@@ -1535,6 +1540,9 @@ export const buildOutputSchemas = (n) => ({
             dropped: STRS,
             /** For a name that fits more than one hotel, the ones it fits - each with \`for\`, the name. */
             candidates: ARR,
+            /** A \`city\` read in a market: the groups that did not answer in time (\`degradedOf\`). */
+            degraded: ARR,
+            degradedSaid: STR,
         },
         required: ['view'],
     },
@@ -2833,6 +2841,30 @@ const remember = (deps, k, amount) => {
 };
 /** "a, b and c" - "a and b" - "a". */
 const listAnd = (xs) => xs.length > 1 ? `${xs.slice(0, -1).join(', ')} and ${xs.at(-1)}` : (xs[0] ?? '');
+/**
+ * The groups of a market that did not answer a search, as the sentence the
+ * model is told and the fields the card draws its own line from - the same
+ * list in both (`HotelSearchResult.degraded`).
+ *
+ * ⚠️ "DID NOT ANSWER" IS NOT "HAS NONE". A list without Atour's hotels, from
+ * an Atour that timed out, reads to a guest as a city where Atour has no
+ * hotel. The sentence says the first and denies the second, in the text and in
+ * `structuredContent` alike - a host with no card has only the words.
+ */
+const degradedOf = (deps, answer) => {
+    const down = answer.degraded ?? [];
+    if (down.length === 0)
+        return { structured: {}, said: '' };
+    const one = down.length === 1;
+    const said = `${listAnd(down.map((d) => d.name))} did not answer in time, so none of ${one ? 'its' : 'their'} ${deps.vocab.nounPlural} is in this list - this does not mean ${one ? 'it has' : 'they have'} none here; asking again in a moment may bring them.`;
+    return {
+        structured: {
+            degraded: down.map((d) => ({ supplier: d.supplier, name: d.name })),
+            degradedSaid: said,
+        },
+        said,
+    };
+};
 /** The counts a call can carry, and what each may be. */
 const COUNTS = ['guests', 'rooms', 'quantity'];
 /** "2" is 2: a host that sends a count as a string has not asked for something else. */
@@ -3002,6 +3034,25 @@ const needsRoomIn = (t, lines, parts) => {
             .join('\n'),
     };
 };
+/**
+ * What a booking of these stays takes of the guest, in the words of the
+ * sentence that asks for it: "name and email", and a phone number where a group
+ * needs one - for all of them, or, in a market where only some of the trip's
+ * groups do, for the stays with those.
+ */
+const guestDetailsFor = (deps, rooms) => {
+    if (!deps.backend.suppliers)
+        return (deps.backend.capabilities ?? FULL_CAPABILITIES).requiresPhone
+            ? 'name, email and phone number'
+            : 'name and email';
+    const groups = [...new Set(rooms.map((l) => deps.backend.supplierOf?.(l.hotelId)))];
+    const needing = groups.filter((id) => !!holderOf(deps, id).caps.requiresPhone);
+    if (needing.length === 0)
+        return 'name and email';
+    if (needing.length === groups.length)
+        return 'name, email and phone number';
+    return `name and email, and a phone number for the stays with ${listAnd(needing.map((id) => holderOf(deps, id).name))}`;
+};
 /**
  * The lines in one slot of the trip store, priced - the trip itself, or (with
  * `store`) the picked room's slot. `key` stays the conversation's trip key: it
@@ -3040,8 +3091,14 @@ const currentTrip = async (deps, key, who, store = key, coupon, ctx) => {
     const overlapping = overlaps.length ? { overlaps } : {};
     // ⚠️ WHAT COMES NEXT, IN WORDS (2026-11-02 audit, P2-4): the card has a
     // Book button under the trip, and the words had nothing in its place.
+    //
+    // ⚠️ AND A PHONE NUMBER WHERE THE TRIP'S GROUP NEEDS ONE (the 0.52.0 gap): the
+    // card's form asks for it, and this sentence - the whole of the card for a
+    // client that draws none - said only "name and email", so the first booking
+    // came back "needs a phone number". Whose need it is is the trip's groups',
+    // as `requestBooking` judges it.
     const nextStep = ctx?.noCards && parts.some((p) => p.lines.some((l) => l.kind === 'room'))
-        ? `Nothing in the trip is booked yet. To book it, ${deps.tools.requestBooking} takes the guest's own name and email - ask for them if the guest has not typed them in this conversation.`
+        ? `Nothing in the trip is booked yet. To book it, ${deps.tools.requestBooking} takes the guest's own ${guestDetailsFor(deps, lines.filter((l) => l.kind === 'room'))} - ask for them if the guest has not typed them in this conversation.`
         : '';
     if (parts.length === 1 && only) {
         return {
@@ -3389,6 +3446,14 @@ export const handleTool = async (deps, name, rawArgs, ctx) => {
     // a card from before - its minor units and currency, for the card to write
     // in its reader's language. The card does no sums: every figure is ours.
     const amounts = amountsIn(named, written);
+    // ⚠️ AND, WHERE THE DEPLOYMENT HAS REFERENCE RATES, A SMALL "≈" BESIDE EACH
+    // AMOUNT IN ANOTHER CURRENCY (reference.ts): the hotel's own currency is what
+    // is quoted and charged, and the figure is only ever beside it.
+    if (deps.backend.reference)
+        return {
+            ...answer,
+            ...withReference(deps.backend.reference, { structured: named, text: answer.text }, amounts, written),
+        };
     const structured = Object.keys(amounts).length ? { ...named, amounts } : named;
     return { ...answer, structured };
 };
@@ -4672,6 +4737,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 : `${c.name} (${c.count})`)
                 .join(', ');
             const n = deps.vocab.nounPlural;
+            const degraded = degradedOf(deps, answer);
             if (namedCountry) {
                 const [only] = countries;
                 return {
@@ -4683,8 +4749,9 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                         ask: { city: asked.city, ...stayArgs },
                         ...(ask.stay ? { stay: ask.stay } : {}),
                         ...(today ? { today } : {}),
+                        ...degraded.structured,
                     },
-                    text: `${total} ${branded(deps.vocab, n)} in ${only?.name}, in ${only?.cities.length} cities to choose from: ${every(only?.cities ?? [])}. A city lists its ${n}; \`all\` lists every one in ${only?.name}. ${forPlans}`,
+                    text: `${total} ${branded(deps.vocab, n)} in ${only?.name}, in ${only?.cities.length} cities to choose from: ${every(only?.cities ?? [])}. A city lists its ${n}; \`all\` lists every one in ${only?.name}. ${degraded.said ? `${degraded.said} ` : ''}${forPlans}`,
                 };
             }
             const regions = [
@@ -4707,11 +4774,13 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                     ask: { ...stayArgs },
                     ...(ask.stay ? { stay: ask.stay } : {}),
                     ...(today ? { today } : {}),
+                    ...degraded.structured,
                 },
-                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}: ${ctx.noCards ? everyWithCities(r.countries) : 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}: ${ctx.noCards ? everyWithCities(r.countries) : every(r.countries)}`).join('; ')}. A city or a country lists its ${n}. ${degraded.said ? `${degraded.said} ` : ''}${forPlans}`,
             };
         }
         if (hotels.length === 0) {
+            const degraded = degradedOf(deps, answer);
             const nearest = (answer.nearest ?? []).map((h) => ({
                 ...h,
                 summary: hotelText('description', h.name, h.summary, DESCRIPTION_MAX).text,
@@ -4759,6 +4828,8 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                     ...(nearest.length ? { nearest, overBudget: answer.overBudget } : {}),
                     ...(ask.stay ? { stay: ask.stay } : {}),
                     ...(widened?.found ? { widened } : {}),
+                    // Nothing found while a group did not answer is not "none here".
+                    ...degraded.structured,
                     // Why it is empty, in the words the model is told, so the card says
                     // the same thing rather than guessing at it.
                     why,
@@ -4776,6 +4847,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 text: [
                     widened?.found ? `Nothing matched ${widened.from}, so this looked in ${widened.to}.` : '',
                     why,
+                    degraded.said,
                 ]
                     .filter(Boolean)
                     .join(' '),
@@ -4851,6 +4923,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
             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 degraded = degradedOf(deps, answer);
         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}` : ''}`)
@@ -4903,6 +4976,8 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 ...(answer.overBudget ? { overBudget: answer.overBudget } : {}),
                 ...(widened?.found ? { widened } : {}),
                 ...(answer.currencies.length > 1 ? { currencies: answer.currencies } : {}),
+                // A group that did not answer, in both: the card draws a line from it.
+                ...degraded.structured,
                 // 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 } : {}),
@@ -4920,6 +4995,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                     : '',
                 ...priceNotes,
                 widened?.found ? `Nothing matched ${widened.from}, so this is ${widened.to}.` : '',
+                degraded.said,
                 ctx.noCards && hotels.length
                     ? `\n${LIST_NOT_DRAWN}\n${hotels.map(listedInWords).join('\n')}\n`
                     : '',
@@ -5120,6 +5196,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
         /** For each name that fits more than one, which ones - see `namesLike`. */
         const candidates = [];
         let total = 0;
+        let degradedFound = {};
         if (named.length) {
             for (const ref of named) {
                 const found = await deps.backend.getHotel(ref);
@@ -5139,7 +5216,9 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
             const found = await deps.backend.searchHotels({ city, limit: LOOK_UP_MAX }, who);
             ids.push(...found.hotels.map((h) => h.id));
             total = found.total;
+            degradedFound = found;
         }
+        const degraded = degradedOf(deps, degradedFound);
         const unique = [...new Set(ids)];
         // A name that fits more than one is said with the ones it fits; the rest are not found.
         const notFound = dropped.filter((d) => !candidates.some((c) => c.for === d));
@@ -5151,9 +5230,10 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                     ...common(deps, key),
                     ...(dropped.length ? { dropped } : {}),
                     ...(candidates.length ? { candidates } : {}),
+                    ...degraded.structured,
                 },
                 text: city
-                    ? `No ${branded(deps.vocab, deps.vocab.nounPlural)} in "${city}". ${t.listHotels} with no place shows the places there are.`
+                    ? `No ${branded(deps.vocab, deps.vocab.nounPlural)} in "${city}". ${degraded.said ? `${degraded.said} ` : ''}${t.listHotels} with no place shows the places there are.`
                     : named.length
                         ? [
                             notFound.length
@@ -5222,9 +5302,11 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 total,
                 ...(dropped.length ? { dropped } : {}),
                 ...(candidates.length ? { candidates } : {}),
+                ...degraded.structured,
             },
             text: [
                 `Read ${read.length}${total > read.length ? ` of ${total}` : ''} ${read.length === 1 ? deps.vocab.noun : deps.vocab.nounPlural}. Nothing was shown to the guest.`,
+                degraded.said,
                 ...blocks,
                 notFound.length ? `Not found: ${notFound.map((d) => `"${d}"`).join(', ')}.` : '',
                 ...fitsMany,