@curless/hotel-storefront 0.27.2 → 0.27.4

package/dist/tools.js

@@ -1,11 +1,11 @@
-import { DEFAULT_PAGE, FULL_CAPABILITIES, LASTING_REQUESTS, MAX_PAGE, REQUEST_KINDS, REQUEST_NOTE_MAX, guestTooLong, } from './backend.js';
-import { isDay, nights as nightsBetween } from './dates.js';
+import { DEFAULT_PAGE, FULL_CAPABILITIES, LASTING_REQUESTS, MAX_PAGE, REQUEST_KINDS, REQUEST_NOTE_MAX, guestTooLong, unitWords, } from './backend.js';
+import { MAX_GUESTS, isDay, nights as nightsBetween } from './dates.js';
 import { FACILITIES } from './facilities.js';
 import { DESCRIPTION_MAX, POLICY_MAX, 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 { regionOf } from './regions.js';
-import { MAX_EXTRA_QUANTITY, MAX_ROOMS_PER_LINE, TRIP_TTL_MS, clampCount, tripKey, } from './trip.js';
+import { MAX_EXTRA_QUANTITY, MAX_ROOMS_PER_LINE, TRIP_TTL_MS, clampCount, shownOf, tripKey, } from './trip.js';
 import { DEMO_NOTE } 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
@@ -60,7 +60,13 @@ const cancels = (title, openWorld) => ({
     idempotentHint: true,
     openWorldHint: openWorld,
 });
-const DATE = (what) => ({ type: 'string', description: `${what}, as YYYY-MM-DD` });
+// ⚠️ THE SHAPE IS IN THE SCHEMA, not only in the words (2026-10-05 audit, P2):
+// a date, a count of rooms and a number of guests each say what they may be.
+const DATE = (what) => ({
+    type: 'string',
+    pattern: '^\\d{4}-\\d{2}-\\d{2}$',
+    description: `${what}, as YYYY-MM-DD`,
+});
 /** How many ${nounPlural} one look-up reads - a city's worth, and what a comparison can hold and then some. */
 export const LOOK_UP_MAX = 6;
 /**
@@ -150,10 +156,16 @@ const forCapabilities = (tools, n, v, caps) => {
 };
 const allTools = (n, v) => {
     const open = !v.demo;
+    const places = v.hints.places ?? [];
     const stayProps = {
         checkIn: DATE('The arrival date'),
         checkOut: DATE('The departure date - the night before it is the last night'),
-        guests: { type: 'number', description: 'People in the room. Default 2' },
+        guests: {
+            type: 'integer',
+            minimum: 1,
+            maximum: MAX_GUESTS,
+            description: 'People in the room. Default 2',
+        },
     };
     return [
         {
@@ -169,11 +181,13 @@ const allTools = (n, v) => {
                     },
                     city: {
                         type: 'string',
-                        description: 'One city or one country, e.g. "Hong Kong" or "Japan"',
+                        description: `One city or one country${places.length ? `, e.g. ${places.map((p) => `"${p}"`).join(' or ')}` : ''}`,
                     },
                     ...stayProps,
                     rooms: {
-                        type: 'number',
+                        type: 'integer',
+                        minimum: 1,
+                        maximum: MAX_ROOMS_PER_LINE,
                         description: `With dates: how many rooms of one type the party needs, 1 to ${MAX_ROOMS_PER_LINE}. Default 1`,
                     },
                     sort: {
@@ -230,7 +244,9 @@ const allTools = (n, v) => {
                     },
                     ...stayProps,
                     rooms: {
-                        type: 'number',
+                        type: 'integer',
+                        minimum: 1,
+                        maximum: MAX_ROOMS_PER_LINE,
                         description: `How many rooms of one type, 1 to ${MAX_ROOMS_PER_LINE}. Default 1`,
                     },
                 },
@@ -261,7 +277,9 @@ const allTools = (n, v) => {
                     },
                     ...stayProps,
                     rooms: {
-                        type: 'number',
+                        type: 'integer',
+                        minimum: 1,
+                        maximum: MAX_ROOMS_PER_LINE,
                         description: `How many rooms of one type, 1 to ${MAX_ROOMS_PER_LINE}. Default 1`,
                     },
                 },
@@ -305,11 +323,15 @@ const allTools = (n, v) => {
                     },
                     ...stayProps,
                     rooms: {
-                        type: 'number',
+                        type: 'integer',
+                        minimum: 0,
+                        maximum: MAX_ROOMS_PER_LINE,
                         description: `How many of that room type, 1 to ${MAX_ROOMS_PER_LINE}. Default 1; 0 removes the line`,
                     },
                     quantity: {
-                        type: 'number',
+                        type: 'integer',
+                        minimum: 0,
+                        maximum: MAX_EXTRA_QUANTITY,
                         description: `For an add-on: how many, 1 to ${MAX_EXTRA_QUANTITY}. Default 1; 0 removes it`,
                     },
                 },
@@ -330,7 +352,12 @@ const allTools = (n, v) => {
         },
         {
             name: n.requestBooking,
-            description: `Asks ${v.brand} to hold the trip's rooms for a named guest and returns a confirmation code and the moment the hold lapses. No payment is taken and no card is collected; the rooms are held, not paid for. Preferences - a bed type, a floor, a late arrival - ride along on the booking as requests the desk tries to meet.`,
+            // ⚠️ A DEMONSTRATION BOOKING IS SAID AS ONE (2026-10-05 audit, P2):
+            // "asks Shangri-La to hold the rooms" read as a real hold, and a guest
+            // could take the code to the real hotel.
+            description: v.demo
+                ? `Makes a demonstration booking request for the trip's rooms, for a named guest: it returns a confirmation code and the moment the demonstration hold lapses on its own. ${v.brand} does not receive it and no real room is held. No payment is taken and no card is collected. Preferences - a bed type, a floor, a late arrival - ride along on the request.`
+                : `Asks ${v.brand} to hold the trip's rooms for a named guest and returns a confirmation code and the moment the hold lapses. No payment is taken and no card is collected; the rooms are held, not paid for. Preferences - a bed type, a floor, a late arrival - ride along on the booking as requests the desk tries to meet.`,
             annotations: books(`Request a ${v.brand} booking`, open),
             inputSchema: {
                 type: 'object',
@@ -436,7 +463,9 @@ const allTools = (n, v) => {
                         description: 'The new check-out, YYYY-MM-DD. Leave out to keep it',
                     },
                     rooms: {
-                        type: 'number',
+                        type: 'integer',
+                        minimum: 1,
+                        maximum: MAX_ROOMS_PER_LINE,
                         description: `How many of that room type, 1 to ${MAX_ROOMS_PER_LINE}. Leave out to keep it`,
                     },
                     room: {
@@ -448,7 +477,9 @@ const allTools = (n, v) => {
                         description: `An add-on at this ${v.noun}, by id or name, to put on the booking, take off or change - with \`quantity\``,
                     },
                     quantity: {
-                        type: 'number',
+                        type: 'integer',
+                        minimum: 0,
+                        maximum: MAX_EXTRA_QUANTITY,
                         description: `How many of that add-on the booking should hold, 0 to ${MAX_EXTRA_QUANTITY}; 0 takes it off`,
                     },
                     email: {
@@ -476,7 +507,9 @@ const allTools = (n, v) => {
                         description: 'The new check-out, YYYY-MM-DD. Leave out to keep it',
                     },
                     rooms: {
-                        type: 'number',
+                        type: 'integer',
+                        minimum: 1,
+                        maximum: MAX_ROOMS_PER_LINE,
                         description: `How many of that room type, 1 to ${MAX_ROOMS_PER_LINE}. Leave out to keep it`,
                     },
                     room: {
@@ -488,7 +521,9 @@ const allTools = (n, v) => {
                         description: `An add-on at this ${v.noun}, by id or name, to put on the booking, take off or change - with \`quantity\``,
                     },
                     quantity: {
-                        type: 'number',
+                        type: 'integer',
+                        minimum: 0,
+                        maximum: MAX_EXTRA_QUANTITY,
                         description: `How many of that add-on the booking should hold, 0 to ${MAX_EXTRA_QUANTITY}; 0 takes it off`,
                     },
                     email: {
@@ -555,7 +590,10 @@ const allTools = (n, v) => {
                         items: {
                             type: 'object',
                             properties: {
-                                label: { type: 'string', description: 'e.g. "Day 1 — Arrive in Hong Kong"' },
+                                label: {
+                                    type: 'string',
+                                    description: `e.g. "Day 1 — Arrive${places[0] ? ` in ${places[0]}` : ''}"`,
+                                },
                                 note: {
                                     type: 'string',
                                     description: 'One or two lines on the day, in your own words',
@@ -722,6 +760,31 @@ const CHANGE_SCHEMA = {
     },
     required: ['view'],
 };
+/**
+ * The output schemas with what this backend never fills taken out.
+ *
+ * ⚠️ A SCHEMA PROMISES. A Huazhu hotel's still promised its add-ons and
+ * policies, and every trip a member badge and saving (2026-10-05 audit) -
+ * fields no answer there ever carries.
+ */
+export const outputSchemasFor = (schemas, caps) => {
+    const gone = [
+        ...(caps.members ? [] : ['member', 'memberSaving']),
+        ...(caps.extras ? [] : ['extras']),
+        ...(caps.policies === false ? ['policies'] : []),
+    ];
+    if (!gone.length)
+        return schemas;
+    return Object.fromEntries(Object.entries(schemas).map(([name, schema]) => {
+        const s = schema;
+        if (!s?.properties)
+            return [name, schema];
+        const properties = { ...s.properties };
+        for (const k of gone)
+            delete properties[k];
+        return [name, { ...s, properties }];
+    }));
+};
 export const buildOutputSchemas = (n) => ({
     [n.listHotels]: {
         type: 'object',
@@ -990,8 +1053,25 @@ const bookedLine = (deps, b) => {
             ? `${total} all in, to be paid at the hotel. ${deps.vocab.brand} has not confirmed this booking yet - that takes up to eight minutes, and ${deps.tools.listBookings} shows when it has. Nothing has been charged.`
             : `${total} all in, paid at the hotel. **Nothing has been charged here** - this is ${deps.vocab.brand}'s own reservation, kept until it is cancelled, and ${b.guest.name} pays at check-in.`;
     }
-    return `${total} all in. **Nothing has been charged** - the rooms are held until ${b.holdUntil?.slice(0, 16).replace('T', ' ')} UTC, and ${b.guest.name} presents a card at check-in.`;
+    const lapses = b.holdUntilLocal
+        ? `${b.holdUntilLocal} hotel time`
+        : `${b.holdUntil?.slice(0, 16).replace('T', ' ')} UTC`;
+    // ⚠️ NOT "PRESENTS A CARD AT CHECK-IN" (2026-10-05 audit, P2): a demo
+    // booking has no check-in, and saying so sent a guest towards a real desk.
+    if (deps.vocab.demo)
+        return `${total} all in. **${demoBookingWords(deps)}** Nothing has been charged and no card is needed. The demonstration hold lapses on its own at ${lapses}.`;
+    return `${total} all in. **Nothing has been charged** - the rooms are held until ${lapses}, and ${b.guest.name} presents a card at check-in.`;
 };
+/** What a demonstration booking is, in one sentence - see `demoBookingNote`. */
+const demoBookingWords = (deps) => `This is a demonstration booking: ${deps.vocab.brand} has not received it and is not holding any room.`;
+/**
+ * Said on every answer about a demonstration booking - made, changed or
+ * cancelled - and on nothing else: a hold of the demo's that a guest took for
+ * the hotel's would send them to a desk with a code nobody there has.
+ */
+const demoBookingNote = (deps) => deps.vocab.demo && (deps.backend.capabilities ?? FULL_CAPABILITIES).booking === 'hold'
+    ? demoBookingWords(deps)
+    : '';
 const common = (deps, key) => ({
     brand: deps.vocab.brand,
     ...(deps.logoUrl ? { logoUrl: deps.logoUrl } : {}),
@@ -1038,7 +1118,7 @@ const quoteForCard = (quote) => ({
         day: l.day,
     })),
     subtotal: formatMoney(quote.subtotal),
-    taxesAndFees: formatMoney(quote.taxesAndFees),
+    taxesAndFees: quote.taxesIncluded ? 'Included' : formatMoney(quote.taxesAndFees),
     total: formatMoney(quote.total),
     // ⚠️ NOT `member`: that key means "who is signed in" on every payload, and
     // the quote's block is a different shape (a tier NAME, a saving, points).
@@ -1057,7 +1137,7 @@ const tripText = (quote, demo) => {
         : `- ${l.ordered ?? l.quantity}× ${l.title}${l.ordered !== undefined && l.quantity !== undefined && l.quantity !== l.ordered ? ` (${l.quantity} in all, over the stay)` : ''} — ${formatMoney(l.total)}`);
     return [
         ...lines,
-        `Taxes and fees ${formatMoney(quote.taxesAndFees)} · **Total ${formatMoney(quote.total)}**`,
+        `Taxes and fees ${quote.taxesIncluded ? 'included in the prices' : formatMoney(quote.taxesAndFees)} · **Total ${formatMoney(quote.total)}**`,
         ...(quote.dropped ?? []).map((d) => `Left out: ${d.title} — ${d.why}`),
         demo ? DEMO_NOTE : '',
     ]
@@ -1265,7 +1345,11 @@ const planGap = (plan) => {
 const bookingLine = (b) => {
     const kinds = b.requests?.kinds ?? [];
     return [
-        `${b.confirmationCode} · ${b.hotelName} · ${b.stay.checkIn} to ${b.stay.checkOut} · ${b.status}`,
+        `${b.confirmationCode} · ${b.hotelName} · ${b.stay.checkIn} to ${b.stay.checkOut} · ${
+        // ⚠️ SAID AS WHAT IT IS - see BookingStatus. Neither booked nor failed.
+        b.status === 'unconfirmed'
+            ? 'unconfirmed: whether it was made is not known yet, and a person is confirming it'
+            : b.status}`,
         kinds.length ? ` · asked for ${kinds.map(bare).join(', ')}` : '',
     ].join('');
 };
@@ -1394,6 +1478,42 @@ const remember = (deps, k, amount) => {
         m.delete(oldest);
     }
 };
+/** 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. */
+const countsAsNumbers = (args) => {
+    const out = { ...args };
+    for (const f of COUNTS) {
+        const v = out[f];
+        if (typeof v === 'string' && /^\s*-?\d+\s*$/.test(v))
+            out[f] = Number(v);
+    }
+    return out;
+};
+/**
+ * Why a count in this call is not one this connector takes - or nothing.
+ *
+ * ⚠️ REFUSED, NOT QUIETLY MADE INTO SOMETHING ELSE (2026-10-05 audit, P2):
+ * six rooms became five, -1 became 1 and a billion became five, and the answer
+ * spoke as if that was what had been asked. A guest who asked for six rooms
+ * has to hear that five is the most, not be booked five.
+ */
+const countProblem = (t, name, args) => {
+    const check = (field, min, max) => {
+        const v = args[field];
+        if (v === undefined || v === null)
+            return undefined;
+        if (typeof v === 'number' && Number.isInteger(v) && v >= min && v <= max)
+            return undefined;
+        return `\`${field}\` has to be a whole number from ${min} to ${max}, and ${JSON.stringify(v)} is not - nothing was changed. Ask the guest, or use a number in that range.`;
+    };
+    // 0 means something here: take the line out of the trip, or - on a change -
+    // the backend's own "a booking keeps at least one room; cancel it instead".
+    const removes = name === t.addToTrip || name === t.previewChange || name === t.changeBooking;
+    return (check('guests', 1, MAX_GUESTS) ??
+        check('rooms', removes ? 0 : 1, MAX_ROOMS_PER_LINE) ??
+        check('quantity', 0, MAX_EXTRA_QUANTITY));
+};
 /**
  * What a backend's refusal says, for the guest - or, when the failure is the
  * code's own, a sentence that says nothing about the code.
@@ -1499,6 +1619,11 @@ const needsRoomIn = (t, lines, parts) => {
 const currentTrip = async (deps, key, who) => {
     const lines = deps.trips.read(key);
     const parts = await quoteParts(deps, lines, who);
+    // ⚠️ WHAT THE GUEST IS SHOWN IS WHAT IS BOOKED - see `ShownQuote`. Every
+    // priced trip answer goes through here: add_to_trip, view_trip, and the
+    // card's own trip view.
+    if (lines.length)
+        deps.trips.show(key, shownOf(parts));
     const [only] = parts;
     const orphans = needsRoomIn(deps.tools, lines, parts);
     const needs = orphans.needsRoom.length ? { needsRoom: orphans.needsRoom } : {};
@@ -1773,9 +1898,17 @@ export const handleTool = async (deps, name, rawArgs, ctx) => {
         : answer;
 };
 const answerTool = async (deps, name, rawArgs, ctx) => {
-    const args = (rawArgs ?? {});
+    const args = countsAsNumbers((rawArgs ?? {}));
     const t = deps.tools;
     const key = ctx.tripKey ?? tripKey(ctx.session, deps.vocab.brand);
+    const badCount = countProblem(t, name, args);
+    if (badCount) {
+        return {
+            structured: { view: 'error', ...common(deps, key), error: 'bad-count', detail: badCount },
+            text: badCount,
+            isError: true,
+        };
+    }
     if (name === t.listHotels) {
         const query = str(args.query);
         const city = str(args.city);
@@ -2353,7 +2486,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                         ].join('\n')
                         : (detail.noRooms ?? `No room type here has ${roomsWanted} left for those nights.`),
                 extras.length
-                    ? `Add-ons: ${extras.map((e) => `${e.name} (id \`${e.id}\`, ${formatMoney(e.price)} per ${e.per}${e.when && e.when !== 'any' ? `, on ${e.when}` : ''})`).join('; ')}`
+                    ? `Add-ons: ${extras.map((e) => `${e.name} (id \`${e.id}\`, ${formatMoney(e.price)} ${unitWords(e.per)}${e.when && e.when !== 'any' ? `, on ${e.when}` : ''})`).join('; ')}`
                     : '',
             ]
                 .filter(Boolean)
@@ -2451,7 +2584,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
             },
             text: extras.length
                 ? extras
-                    .map((e) => `- ${e.name} — ${formatMoney(e.price)} per ${e.per}${held[e.id] ? ` (${booked?.confirmationCode} holds ${held[e.id]})` : ''}`)
+                    .map((e) => `- ${e.name} — ${formatMoney(e.price)} ${unitWords(e.per)}${held[e.id] ? ` (${booked?.confirmationCode} holds ${held[e.id]})` : ''}`)
                     .concat(deps.vocab.demo ? [DEMO_NOTE] : [])
                     .join('\n')
                 : `${detail.hotel.name} has no add-ons listed here.`,
@@ -2599,7 +2732,8 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
         }
         const { requests, dropped } = readRequests(args);
         try {
-            const made = await deps.backend.requestBooking({ lines }, {
+            const shown = deps.trips.shown(key);
+            const made = await deps.backend.requestBooking({ lines, ...(shown ? { shown } : {}) }, {
                 ...guest,
                 ...(str(args.phone) ? { phone: str(args.phone) } : {}),
                 ...(requests?.note ? { notes: requests.note } : {}),
@@ -2714,6 +2848,13 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
         }
         catch (err) {
             const words = guestWords(err, BOOKING_FAULT);
+            // The price or terms moved since the guest saw them: nothing was booked,
+            // and the trip is shown again at today's - which is then what the guest
+            // has seen, and what a second ask books.
+            if (err.code === 'price-changed') {
+                const again = await currentTrip(deps, key, identity(deps, ctx, key));
+                return { ...again, text: [words, again.text].join('\n\n') };
+            }
             return {
                 structured: {
                     view: 'error',
@@ -2830,6 +2971,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                         : 'The total is unchanged.',
                     preview.rule,
                     `Say the word and it moves (${t.changeBooking} with \`confirmed\`).`,
+                    demoBookingNote(deps),
                     deps.vocab.demo ? DEMO_NOTE : '',
                 ]
                     .filter(Boolean)
@@ -2892,6 +3034,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 text: [
                     `${changed.confirmationCode} now holds ${changed.stay.checkIn} to ${changed.stay.checkOut}, ${formatMoney(changed.quote.total)} all in.`,
                     'The confirmation code has not changed. **Nothing has been charged.**',
+                    demoBookingNote(deps),
                     deps.vocab.demo ? DEMO_NOTE : '',
                 ]
                     .filter(Boolean)
@@ -2937,6 +3080,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 preview.cancellable
                     ? `Say the word and it is cancelled (${t.cancelBooking} with \`confirmed\`).`
                     : '',
+                demoBookingNote(deps),
                 deps.vocab.demo ? DEMO_NOTE : '',
             ]
                 .filter(Boolean)
@@ -2983,7 +3127,14 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 cancellable: false,
             },
             text: [
-                `${cancelled.confirmationCode} is cancelled. ${charged.minor === 0 ? 'No fee.' : `The hotel charges ${formatMoney(charged)}.`}`,
+                deps.vocab.sandbox
+                    ? // A test booking cancelled in the simulation: no hotel, no charge (2026-10-05 audit, P2).
+                        `${cancelled.confirmationCode} is cancelled in the simulated system. ${charged.minor === 0 ? 'No fee.' : `A real booking on this rate would be charged ${formatMoney(charged)}; this one is a test, and nothing is charged.`}`
+                    : deps.vocab.demo
+                        ? // Nobody charges a demonstration anything (2026-10-05 audit, P2).
+                            `${cancelled.confirmationCode} is cancelled. ${charged.minor === 0 ? 'No fee.' : `A real booking on this rate would be charged ${formatMoney(charged)}; this is a demonstration, and nothing is charged.`}`
+                        : `${cancelled.confirmationCode} is cancelled. ${charged.minor === 0 ? 'No fee.' : `The hotel charges ${formatMoney(charged)}.`}`,
+                demoBookingNote(deps),
                 deps.vocab.demo ? DEMO_NOTE : '',
             ]
                 .filter(Boolean)