@curless/hotel-storefront 0.27.3 → 0.27.4

package/dist/tools.js

@@ -1,5 +1,5 @@
-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';
@@ -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;
 /**
@@ -154,7 +160,12 @@ const allTools = (n, v) => {
     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 [
         {
@@ -174,7 +185,9 @@ const allTools = (n, v) => {
                     },
                     ...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: {
@@ -231,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`,
                     },
                 },
@@ -262,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`,
                     },
                 },
@@ -306,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`,
                     },
                 },
@@ -331,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',
@@ -437,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: {
@@ -449,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: {
@@ -477,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: {
@@ -489,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: {
@@ -1019,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 } : {}),
@@ -1427,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.
@@ -1811,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);
@@ -2391,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)
@@ -2489,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.`,
@@ -2876,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)
@@ -2938,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)
@@ -2983,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)
@@ -3032,7 +3130,11 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 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.`}`
-                    : `${cancelled.confirmationCode} is cancelled. ${charged.minor === 0 ? 'No fee.' : `The hotel charges ${formatMoney(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)