@curless/hotel-storefront 0.27.26 → 0.27.27

package/dist/tools.js

@@ -3,7 +3,7 @@ import { MAX_GUESTS, isDay, nights as nightsBetween } from './dates.js';
 import { FACILITIES } from './facilities.js';
 import { DESCRIPTION_MAX, POLICY_MAX, guestText, hotelText } from './fence.js';
 import { formatMoney, money, multiplyMoney, toMinor } from './money.js';
-import { MAX_COMPARED, MAX_CONS, MAX_DAYS, MAX_ITEMS_PER_DAY, MAX_MOMENTS, MAX_PROS, buildItinerary, compareHotels, } from './present.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 { MAX_EXTRA_QUANTITY, MAX_ROOMS_PER_LINE, TRIP_TTL_MS, clampCount, shownOf, tripKey, } from './trip.js';
 import { DEMO_NOTE, branded, catalogue, upFirst } from './vocab.js';
@@ -770,6 +770,8 @@ const TRIP_SCHEMA = {
         memberSaving: OBJ,
         days: ARR,
         dropped: ARR,
+        /** A name that fits more than one hotel: the ones it fits (`namesLike`). */
+        candidates: ARR,
     },
     required: ['view'],
 };
@@ -891,6 +893,8 @@ export const buildOutputSchemas = (n) => ({
             policies: ARR,
             /** Arrivals a few days either side, each re-quoted for the same nights. */
             dates: ARR,
+            /** A name that fits more than one hotel: the ones it fits (`namesLike`). */
+            candidates: ARR,
         },
         required: ['view'],
     },
@@ -905,6 +909,8 @@ export const buildOutputSchemas = (n) => ({
             demoNote: STR,
             hotel: OBJ,
             extras: ARR,
+            /** A name that fits more than one hotel: the ones it fits (`namesLike`). */
+            candidates: ARR,
         },
         required: ['view'],
     },
@@ -1032,7 +1038,10 @@ export const buildOutputSchemas = (n) => ({
             hotels: ARR,
             stay: OBJ,
             total: { type: 'number' },
-            dropped: ARR,
+            /** The names that did not open one hotel - none answered, or more than one did. */
+            dropped: STRS,
+            /** For a name that fits more than one hotel, the ones it fits - each with \`for\`, the name. */
+            candidates: ARR,
         },
         required: ['view'],
     },
@@ -1045,6 +1054,8 @@ export const buildOutputSchemas = (n) => ({
             bookingMode: STR,
             demo: BOOL,
             hotel: OBJ,
+            /** A name that fits more than one hotel: the ones it fits (`namesLike`). */
+            candidates: ARR,
             topic: STR,
             title: STR,
             text: STR,
@@ -2021,6 +2032,14 @@ export const handleTool = async (deps, name, rawArgs, ctx) => {
         ? { ...answer, structured: { ...answer.structured, suggestions } }
         : answer;
 };
+/**
+ * For each left-out name that fits more than one hotel, the sentence that says
+ * which - for the model, in the tool's text, never on the card (see
+ * `MORE_THAN_ONE`).
+ */
+const fitsManyOf = (deps, dropped) => dropped.flatMap((d) => d.candidates?.length
+    ? [`"${d.name}" ${namesMoreThanOne(d.candidates, branded(deps.vocab, deps.vocab.noun))}`]
+    : []);
 const answerTool = async (deps, name, rawArgs, ctx) => {
     const args = countsAsNumbers((rawArgs ?? {}));
     const t = deps.tools;
@@ -2422,9 +2441,17 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
         const who = identity(deps, ctx, key);
         const detail = await deps.backend.getHotel(wanted, stay, who);
         if (!detail) {
+            const candidates = candidatesFor(deps.backend, wanted);
             return {
-                structured: { view: 'hotel-not-found', ...common(deps, key), query: wanted },
-                text: `No ${branded(deps.vocab, deps.vocab.noun)} by that name. ${t.listHotels} lists them.`,
+                structured: {
+                    view: 'hotel-not-found',
+                    ...common(deps, key),
+                    query: wanted,
+                    ...(candidates.length ? { candidates } : {}),
+                },
+                text: candidates.length
+                    ? `"${wanted}" ${namesMoreThanOne(candidates, branded(deps.vocab, deps.vocab.noun))}`
+                    : `No ${branded(deps.vocab, deps.vocab.noun)} by that name. ${t.listHotels} lists them.`,
             };
         }
         const h = detail.hotel;
@@ -2556,14 +2583,21 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
         const city = str(args.city);
         const ids = [];
         const dropped = [];
+        /** For each name that fits more than one, which ones - see `namesLike`. */
+        const candidates = [];
         let total = 0;
         if (named.length) {
             for (const ref of named) {
                 const found = await deps.backend.getHotel(ref);
                 if (found)
                     ids.push(found.hotel.id);
-                else
+                else {
                     dropped.push(ref);
+                    // The same name asked twice is named once.
+                    if (!candidates.some((c) => c.for === ref))
+                        for (const c of candidatesFor(deps.backend, ref))
+                            candidates.push({ for: ref, ...c });
+                }
             }
             total = ids.length;
         }
@@ -2573,17 +2607,28 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
             total = found.total;
         }
         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));
+        const fitsMany = [...new Set(candidates.map((c) => c.for))].map((ref) => `"${ref}" ${namesMoreThanOne(candidates.filter((c) => c.for === ref), branded(deps.vocab, deps.vocab.noun))}`);
         if (unique.length === 0) {
             return {
                 structured: {
                     view: 'nothing-found',
                     ...common(deps, key),
                     ...(dropped.length ? { dropped } : {}),
+                    ...(candidates.length ? { candidates } : {}),
                 },
                 text: city
                     ? `No ${branded(deps.vocab, deps.vocab.nounPlural)} in "${city}". ${t.listHotels} with no place shows the places there are.`
                     : named.length
-                        ? `No ${branded(deps.vocab, deps.vocab.noun)} answers to ${dropped.map((d) => `"${d}"`).join(', ')}.`
+                        ? [
+                            notFound.length
+                                ? `No ${branded(deps.vocab, deps.vocab.noun)} answers to ${notFound.map((d) => `"${d}"`).join(', ')}.`
+                                : '',
+                            ...fitsMany,
+                        ]
+                            .filter(Boolean)
+                            .join('\n')
                         : `Name a \`city\`, or up to ${LOOK_UP_MAX} ${deps.vocab.nounPlural} in \`${deps.vocab.nounPlural}\`.`,
             };
         }
@@ -2635,11 +2680,13 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 ...(stay ? { stay } : {}),
                 total,
                 ...(dropped.length ? { dropped } : {}),
+                ...(candidates.length ? { candidates } : {}),
             },
             text: [
                 `Read ${read.length}${total > read.length ? ` of ${total}` : ''} ${read.length === 1 ? deps.vocab.noun : deps.vocab.nounPlural}. Nothing was shown to the guest.`,
                 ...blocks,
-                dropped.length ? `Not found: ${dropped.map((d) => `"${d}"`).join(', ')}.` : '',
+                notFound.length ? `Not found: ${notFound.map((d) => `"${d}"`).join(', ')}.` : '',
+                ...fitsMany,
                 total > read.length
                     ? `${total - read.length} more ${total - read.length === 1 ? 'is' : 'are'} there; name ${total - read.length === 1 ? 'it' : 'them'} in \`${deps.vocab.nounPlural}\` to read ${total - read.length === 1 ? 'it' : 'them'}.`
                     : '',
@@ -2697,9 +2744,17 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
         const wanted = booked ? booked.hotelId : str(args.hotel);
         const detail = await deps.backend.getHotel(wanted);
         if (!detail) {
+            const candidates = candidatesFor(deps.backend, wanted);
             return {
-                structured: { view: 'hotel-not-found', ...common(deps, key), query: wanted },
-                text: `No ${branded(deps.vocab, deps.vocab.noun)} by that name.`,
+                structured: {
+                    view: 'hotel-not-found',
+                    ...common(deps, key),
+                    query: wanted,
+                    ...(candidates.length ? { candidates } : {}),
+                },
+                text: candidates.length
+                    ? `"${wanted}" ${namesMoreThanOne(candidates, branded(deps.vocab, deps.vocab.noun))}`
+                    : `No ${branded(deps.vocab, deps.vocab.noun)} by that name.`,
             };
         }
         const extras = await deps.backend.extrasFor(detail.hotel.id, booked?.stay ?? asStay(args));
@@ -2736,14 +2791,18 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
         const wanted = str(args.hotel);
         const detail = await deps.backend.getHotel(wanted);
         if (!detail) {
+            const candidates = candidatesFor(deps.backend, wanted);
             return {
                 structured: {
                     view: 'error',
                     ...common(deps, key),
                     error: 'hotel-not-found',
                     query: wanted,
+                    ...(candidates.length ? { candidates } : {}),
                 },
-                text: `No ${branded(deps.vocab, deps.vocab.noun)} by that name.`,
+                text: candidates.length
+                    ? `"${wanted}" ${namesMoreThanOne(candidates, branded(deps.vocab, deps.vocab.noun))}`
+                    : `No ${branded(deps.vocab, deps.vocab.noun)} by that name.`,
             };
         }
         const hotelId = detail.hotel.id;
@@ -3345,13 +3404,23 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
             who: identity(deps, ctx, key),
         });
         if (comparison.columns.length === 0) {
+            const unknown = comparison.dropped.filter((d) => !d.candidates?.length);
+            const fitsMany = fitsManyOf(deps, comparison.dropped);
             return {
                 structured: {
                     view: 'nothing-to-compare',
                     ...common(deps, key),
                     dropped: comparison.dropped,
                 },
-                text: `None of those are ${catalogue(deps.vocab)}: ${comparison.dropped.map((d) => d.name).join(', ')}.`,
+                // A name that fits several IS one of ours - only the rest are "none".
+                text: [
+                    ...(unknown.length || !fitsMany.length
+                        ? [
+                            `None of those are ${catalogue(deps.vocab)}: ${unknown.map((d) => d.name).join(', ')}.`,
+                        ]
+                        : []),
+                    ...fitsMany,
+                ].join('\n'),
             };
         }
         return {
@@ -3376,6 +3445,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                     ? `These are priced in different currencies (${comparison.currencies.join(', ')}) and nothing here converts them — do not call one cheaper than another.`
                     : '',
                 ...comparison.dropped.map((d) => `Left out: ${d.name} — ${d.why}`),
+                ...fitsManyOf(deps, comparison.dropped),
                 deps.vocab.demo ? DEMO_NOTE : '',
             ]
                 .filter(Boolean)
@@ -3419,6 +3489,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 `**${plan.title}** · ${plan.days.length} ${plan.days.length === 1 ? 'day' : 'days'}. ${gap}`,
                 stays.length ? `Stays, priced on: ${stays.join('; ')}.` : '',
                 ...plan.dropped.map((d) => `Left out: ${d.name} — ${d.why}`),
+                ...fitsManyOf(deps, plan.dropped),
                 deps.vocab.demo ? DEMO_NOTE : '',
             ]
                 .filter(Boolean)
@@ -3505,9 +3576,17 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
         const wanted = str(args.hotel);
         const detail = await deps.backend.getHotel(wanted);
         if (!detail) {
+            const candidates = candidatesFor(deps.backend, wanted);
             return {
-                structured: { view: 'hotel-not-found', ...common(deps, key), query: wanted },
-                text: `No ${branded(deps.vocab, deps.vocab.noun)} by that name.`,
+                structured: {
+                    view: 'hotel-not-found',
+                    ...common(deps, key),
+                    query: wanted,
+                    ...(candidates.length ? { candidates } : {}),
+                },
+                text: candidates.length
+                    ? `"${wanted}" ${namesMoreThanOne(candidates, branded(deps.vocab, deps.vocab.noun))}`
+                    : `No ${branded(deps.vocab, deps.vocab.noun)} by that name.`,
             };
         }
         const policies = await deps.backend.policies(detail.hotel.id);