@curless/hotel-storefront 0.27.5 → 0.27.6

package/dist/tools.js

@@ -1,7 +1,7 @@
 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 { 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 { regionOf } from './regions.js';
@@ -45,6 +45,18 @@ const edits = (title, openWorld) => ({
     idempotentHint: true,
     openWorldHint: openWorld,
 });
+/**
+ * Writes over something that was there - the whole trip, or a booking's nights.
+ * Repeating it changes nothing more, but what it replaced is gone (2026-10-05
+ * audit, P3: these two said `destructiveHint: false`).
+ */
+const replaces = (title, openWorld) => ({
+    title,
+    readOnlyHint: false,
+    destructiveHint: true,
+    idempotentHint: true,
+    openWorldHint: openWorld,
+});
 /** Asks the hotel to hold rooms. Takes no money, and undoing it is what the cancel tools are for. */
 const books = (title, openWorld) => ({
     title,
@@ -347,7 +359,7 @@ const allTools = (n, v) => {
         {
             name: n.clearTrip,
             description: 'Empties the trip.',
-            annotations: edits(`Clear the ${v.brand} trip`, open),
+            annotations: replaces(`Clear the ${v.brand} trip`, open),
             inputSchema: { type: 'object', properties: {} },
         },
         {
@@ -407,7 +419,7 @@ const allTools = (n, v) => {
         },
         {
             name: n.previewCancellation,
-            description: 'What cancelling a booking would cost and what would be refunded, with the rule it was worked out from and the free-cancellation deadline where the rate has one. Nothing is cancelled.',
+            description: 'What cancelling a booking would cost, with the rule it was worked out from and the free-cancellation deadline where the rate has one. Nothing is cancelled.',
             annotations: reads('Preview a cancellation', open),
             inputSchema: {
                 type: 'object',
@@ -427,7 +439,7 @@ const allTools = (n, v) => {
             // directory attestation covers descriptions that instruct about other
             // tools. The preview it returns is the same payload the preview tool
             // returns, which is a fact about THIS tool's answer.
-            description: 'Cancels a booking. Called without `confirmed` it returns the cancellation preview - the fee, the refund and the rule behind them - and cancels nothing; it cancels only when `confirmed` is true.',
+            description: 'Cancels a booking. Called without `confirmed` it returns the cancellation preview - the fee and the rule behind it - and cancels nothing; it cancels only when `confirmed` is true.',
             annotations: cancels('Cancel a booking', open),
             inputSchema: {
                 type: 'object',
@@ -493,7 +505,7 @@ const allTools = (n, v) => {
         {
             name: n.changeBooking,
             description: 'Moves a booking to other nights, another room type or a different number of rooms, or sets one of its add-ons (`extra` and `quantity`; 0 takes it off). Called without `confirmed` it returns the same preview - the two stays and the difference - and changes nothing; it changes only when `confirmed` is true. The confirmation code stays the same.',
-            annotations: edits('Change a booking', open),
+            annotations: replaces('Change a booking', open),
             inputSchema: {
                 type: 'object',
                 properties: {
@@ -1225,7 +1237,7 @@ const requestLines = (b, dropped, checkInFrom) => {
         out.push(`Asked for on the booking: ${words.join(', ')}. The desk meets what it can on the day; none of it is held.`);
     }
     if (r.note)
-        out.push(`Passed to the hotel in the guest's words: "${r.note}"`);
+        out.push(`Passed to the hotel:\n${guestText('note for the hotel', r.note, REQUEST_NOTE_MAX)}`);
     if (r.kinds.includes('accessible-room')) {
         out.push(`An accessible room is a room type, and this booking does not hold one - ${b.hotelName} confirms accessible rooms directly, quoting ${b.confirmationCode}.`);
     }
@@ -3068,7 +3080,9 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 ...common(deps, key),
                 booking: { id: preview.bookingId, confirmationCode: preview.confirmationCode },
                 fee: formatMoney(preview.fee),
-                refund: formatMoney(preview.refund),
+                // ⚠️ NO "REFUND": nothing was ever paid here - a demonstration takes no
+                // money and a hotel's own booking is paid at the hotel - so a figure
+                // called a refund is money that is not coming back (2026-10-05 audit, P3).
                 rule: preview.rule,
                 freeUntil: preview.freeUntil,
                 cancellable: preview.cancellable,
@@ -3122,7 +3136,6 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 ...common(deps, key),
                 booking: { ...cancelled, quote: quoteForCard(cancelled.quote) },
                 fee: formatMoney(charged),
-                refund: formatMoney(preview.refund),
                 rule: preview.rule,
                 cancellable: false,
             },