@curless/hotel-storefront 0.55.1 → 0.55.2

package/dist/tools.js

@@ -2312,10 +2312,15 @@ const couponRefusedLine = (q) => q.couponRefused
     ? `The coupon ${q.couponRefused.name ? couponName({ name: q.couponRefused.name, names: q.couponRefused.names }) : q.couponRefused.id} was NOT taken off: ${q.couponRefused.why} The total is without it.`
     : '';
 const tripText = (quote, noCards) => {
+    // A part not priced because its group did not answer just now: said once, and
+    // left to the guest to ask for again (`askLater`; the trip is drawn as a card).
+    const later = quote.dropped?.some((d) => d.code === 'unreachable') ? askLater(!noCards) : '';
     // ⚠️ NOT "EMPTY" WHEN SOMETHING WAS LEFT OUT: a trip whose only group did not
     // answer is not an empty trip (2026-11-03 review, C5).
     if (quote.lines.length === 0 && quote.dropped?.length)
-        return quote.dropped.map((d) => `Left out: ${d.title} — ${d.why}`).join('\n');
+        return [...quote.dropped.map((d) => `Left out: ${d.title} — ${d.why}`), later]
+            .filter(Boolean)
+            .join('\n');
     if (quote.lines.length === 0)
         return 'The trip is empty.';
     const lines = quote.lines.map((l) => l.kind === 'room'
@@ -2327,6 +2332,7 @@ const tripText = (quote, noCards) => {
         couponRefusedLine(quote),
         ...memberTripLines(quote),
         ...(quote.dropped ?? []).map((d) => `Left out: ${d.title} — ${d.why}`),
+        later,
     ]
         .filter(Boolean)
         .join('\n');
@@ -2994,6 +3000,20 @@ 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] ?? '');
+/**
+ * What the model does when a hotel group did not answer: tells the guest, and
+ * leaves asking again to them - said after the sentence that names the groups,
+ * in the text and in `structuredContent` alike.
+ *
+ * ⚠️ NEVER "ASKING AGAIN IN A MOMENT MAY BRING THEM" (robin 11-04). Read as
+ * "ask again now", it had the model call the list a second time in the same
+ * turn: two of the same card under 「打开 Otaless」 on claude.ai, with every
+ * group of the backend silent. The guest asks again: with the card's 「重试」
+ * where a card was drawn, in words where none was.
+ */
+const askLater = (card) => card
+    ? 'Do not look again in this turn - no second call for it: tell the guest, who can press Try again (重试) on the card when they want to look again.'
+    : 'Do not look again in this turn - no second call for it: tell the guest, and that they can ask you to look again later.';
 /**
  * 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
@@ -3013,12 +3033,14 @@ noun = deps.vocab.nounPlural,
  * card called it 锦江之星, another group (11-03, text-only eval). A card host
  * is told the English name alone; the card says the group in its own words.
  */
-noCards = false) => {
+noCards = false, 
+/** This answer draws a card, with its 「重试」 (`askLater`) - not so for `look_up`, which draws none. */
+card = !noCards) => {
     const down = answer.degraded ?? [];
     if (down.length === 0)
         return { structured: {}, said: '' };
     const one = down.length === 1;
-    const said = `${listAnd(down.map((d) => withZh(d.name, d.names, noCards)))} did not answer in time, so none of ${one ? 'its' : 'their'} ${noun} is in this list - this does not mean ${one ? 'it has' : 'they have'} none here; asking again in a moment may bring them.`;
+    const said = `${listAnd(down.map((d) => withZh(d.name, d.names, noCards)))} did not answer in time, so none of ${one ? 'its' : 'their'} ${noun} is in this list - this does not mean ${one ? 'it has' : 'they have'} none here. ${askLater(card)}`;
     return {
         structured: {
             degraded: down.map((d) => ({ supplier: d.supplier, name: d.name })),
@@ -3036,13 +3058,15 @@ noCards = false) => {
  * time"), not the name of whichever was asked first; and in every case it
  * denies the reading a guest would make of silence, that there is nothing.
  */
-const notAnswered = (deps, key, down) => {
+const notAnswered = (deps, key, down, 
+/** The answer draws a card, with its 「重试」 - see `askLater`. */
+card) => {
     const groups = deps.backend.suppliers?.().length ?? 0;
     const names = listAnd(down.map((d) => d.name));
     const one = down.length === 1;
     const said = groups > 1 && down.length >= groups
-        ? `No hotel group answered this time: ${names} did not answer in time - this does not mean there is nothing here; asking again in a moment may bring them.`
-        : `${names} did not answer in time, so nothing from ${one ? 'it' : 'them'} could be read just now - this does not mean ${one ? 'it has' : 'they have'} none here; asking again in a moment may bring ${one ? 'it' : 'them'}.`;
+        ? `No hotel group answered this time: ${names} did not answer in time - this does not mean there is nothing here. ${askLater(card)}`
+        : `${names} did not answer in time, so nothing from ${one ? 'it' : 'them'} could be read just now - this does not mean ${one ? 'it has' : 'they have'} none here. ${askLater(card)}`;
     return {
         structured: {
             view: 'error',
@@ -3107,7 +3131,9 @@ const countProblem = (t, name, args) => {
  * properties of null" - were read back too (2026-10-05 audit); they go to the
  * log now.
  */
-const guestWords = (err, fault = 'That could not be done just now - nothing was changed. Try again in a moment.') => {
+const guestWords = (err, 
+// Not "try again in a moment" (robin 11-04): the guest asks for it again.
+fault = 'That could not be done just now - nothing was changed. Tell the guest; they can ask for it again later.') => {
     const e = err;
     // ⚠️ A READ THAT TIMED OUT IS NOT A REFUSAL (2026-11-03 review, C5). After a
     // booking was made, "Atour did not answer within 8000 ms" would be read
@@ -3666,7 +3692,9 @@ export const handleTool = async (deps, name, rawArgs, ctx) => {
             const down = unansweredBy(err);
             if (!down)
                 throw err;
-            return notAnswered(deps, ctx.tripKey ?? tripKey(ctx.session, deps.vocab.brand), down);
+            // `look_up` and the policies draw no card, so there is no 「重试」 to press.
+            const card = !ctx.noCards && name !== deps.tools.lookUp && name !== deps.tools.policies;
+            return notAnswered(deps, ctx.tripKey ?? tripKey(ctx.session, deps.vocab.brand), down, card);
         });
         const suggestions = await suggestionsFor(deps, answer.structured).catch(() => []);
         return { answer, suggestions };
@@ -5386,7 +5414,13 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
         const description = hotelText('description', h.name, 
         // Its 【…】 parts a line each, the words as they came (`aboutLines`).
         aboutLines(h.summary), DESCRIPTION_MAX, deps.vocab.demo);
-        const policies = await deps.backend.policies(h.id);
+        // ⚠️ NO POLICIES WHERE THE POLICIES TOOL IS NOT OFFERED (总线程 11-04): Atour's
+        // single store (`policies: false`) still listed its topics on every hotel, and
+        // the card drew six rows of 「入住须知」 that each opened on "could not load
+        // this policy". Not read, not in the answer - the model is not told of a
+        // policy nobody can open either.
+        const offersPolicies = (deps.backend.capabilities ?? FULL_CAPABILITIES).policies !== false;
+        const policies = offersPolicies ? await deps.backend.policies(h.id) : [];
         const extras = await deps.backend.extrasFor(h.id, stay);
         const base = {
             ...common(deps, key),
@@ -5396,7 +5430,9 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                 fromNightly: h.fromNightly ? formatMoney(h.fromNightly) : undefined,
             },
             extras: extras.map((e) => ({ ...e, price: formatMoney(e.price) })),
-            policies: policies.map((p) => ({ topic: p.topic, title: p.title })),
+            ...(offersPolicies
+                ? { policies: policies.map((p) => ({ topic: p.topic, title: p.title })) }
+                : {}),
         };
         if (!stay) {
             return {
@@ -5437,7 +5473,10 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
             // (it says it in the guest's language from `noRoomsCode`).
             const said = detail.noRooms && detail.noRoomsCode === 'none-on-sale'
                 ? notOnSaleWords(groupNameOf(deps, h.id), detail.noRooms)
-                : detail.noRooms;
+                : // Rooms that could not be read just now: asked for again by the guest.
+                    detail.noRooms && detail.noRoomsCode === 'unreachable'
+                        ? `${detail.noRooms} ${askLater(!ctx.noCards)}`
+                        : detail.noRooms;
             return {
                 structured: {
                     view: 'no-rooms',
@@ -5588,7 +5627,8 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
             total = found.total;
             degradedFound = found;
         }
-        const degraded = degradedOf(deps, degradedFound, undefined, ctx?.noCards);
+        // `look_up` draws no card: the guest asks again in words (`askLater`).
+        const degraded = degradedOf(deps, degradedFound, undefined, ctx?.noCards, false);
         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));
@@ -6398,7 +6438,10 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                         : []),
                     ...fitsMany,
                     ...silent.map((d) => `Left out: ${d.name} — ${d.why}`),
-                ].join('\n'),
+                    silent.length ? askLater(!ctx.noCards) : '',
+                ]
+                    .filter(Boolean)
+                    .join('\n'),
             };
         }
         const stayWords = comparison.stay
@@ -6440,6 +6483,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                     : '',
                 ...comparison.dropped.map((d) => `Left out: ${d.name} — ${d.why}`),
                 ...fitsManyOf(deps, comparison.dropped),
+                comparison.dropped.some((d) => d.code === 'supplier-timeout') ? askLater(!ctx.noCards) : '',
             ]
                 .filter(Boolean)
                 .join('\n'),
@@ -6464,7 +6508,7 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
             unchecked = down;
         }
         const uncheckedSaid = unchecked.length
-            ? `${listAnd(unchecked.map((d) => d.name))} did not answer in time, so ${unchecked.length === 1 ? 'its' : 'their'} bookings were not checked against this plan - a stay there may already be booked without the plan saying so; asking again in a moment may check them.`
+            ? `${listAnd(unchecked.map((d) => d.name))} did not answer in time, so ${unchecked.length === 1 ? 'its' : 'their'} bookings were not checked against this plan - a stay there may already be booked without the plan saying so. ${askLater(!ctx.noCards)}`
             : '';
         const plan = await buildItinerary(deps.backend, days.map((d) => ({
             label: str(d.label),
@@ -6565,6 +6609,10 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
                     : []),
                 ...plan.dropped.map((d) => `Left out: ${d.name} — ${d.why}`),
                 ...fitsManyOf(deps, plan.dropped),
+                // Once, for every group left silent - the bookings' sentence has its own.
+                !uncheckedSaid && plan.dropped.some((d) => d.code === 'supplier-timeout')
+                    ? askLater(!ctx.noCards)
+                    : '',
                 uncheckedSaid,
                 ctx.noCards ? PLAN_NOT_DRAWN : PLAN_DRAWN_ONCE,
             ]