@curless/hotel-storefront 0.27.14 → 0.27.16

package/dist/tools.js

@@ -729,6 +729,13 @@ const allTools = (n, v) => {
 // model deciding what to tell a guest.
 const OBJ = { type: 'object' };
 const ARR = { type: 'array', items: { type: 'object' } };
+/**
+ * A list of names - cities, currencies, a comparison's row labels. ⚠️ Declared
+ * as ARR (objects) since the first build while the answers carried strings: a
+ * host validating `structuredContent` against `outputSchema`, as the MCP spec
+ * has a client do, would refuse every city list (2026-10-08).
+ */
+const STRS = { type: 'array', items: { type: 'string' } };
 const STR = { type: 'string' };
 const BOOL = { type: 'boolean' };
 const TRIP_SCHEMA = {
@@ -836,7 +843,7 @@ export const buildOutputSchemas = (n) => ({
             /** How many MATCHED. `shown` is how many are in `hotels`. */
             total: { type: 'number' },
             shown: { type: 'number' },
-            cities: ARR,
+            cities: STRS,
             query: STR,
             /** The order actually used, which is not always the one asked for. */
             sort: STR,
@@ -845,7 +852,7 @@ export const buildOutputSchemas = (n) => ({
             /** The cheapest of those, ONLY when the ceiling left nothing. Never matches. */
             nearest: ARR,
             /** Present only when the matches span more than one, which drops the price knobs. */
-            currencies: ARR,
+            currencies: STRS,
             /** Set when nothing matched and one wider search was tried instead. */
             widened: OBJ,
         },
@@ -896,7 +903,9 @@ export const buildOutputSchemas = (n) => ({
     [n.requestBooking]: {
         type: 'object',
         properties: {
-            view: { enum: ['booking', 'needs-guest', 'empty-trip', 'error'] },
+            // `trip`: nothing was booked and the trip is shown again - its price
+            // moved, or (in a market) it holds two groups' stays.
+            view: { enum: ['booking', 'needs-guest', 'empty-trip', 'error', 'trip'] },
             brand: STR,
             toolNames: OBJ,
             bookingMode: STR,
@@ -946,8 +955,8 @@ export const buildOutputSchemas = (n) => ({
             columns: ARR,
             dropped: ARR,
             /** Present only when the columns are priced in more than one currency. */
-            currencies: ARR,
-            rowLabels: ARR,
+            currencies: STRS,
+            rowLabels: STRS,
         },
         required: ['view'],
     },
@@ -2811,7 +2820,8 @@ const answerTool = async (deps, name, rawArgs, ctx) => {
     if (name === t.clearTrip) {
         deps.trips.clear(key);
         return {
-            structured: { view: 'trip', ...common(deps, key), lines: [], count: 0, total: null },
+            // No `total` at all: an empty trip has none, and `null` broke the schema.
+            structured: { view: 'trip', ...common(deps, key), lines: [], count: 0 },
             text: 'The trip is empty.',
         };
     }