@voyant-travel/bookings-react 0.185.0 → 0.186.1

package/dist/journey/components/booking-journey.js

@@ -203,6 +203,19 @@ export function BookingJourney(props) {
         (quote.data.available === false ||
             (quote.data.invalidReason != null && quote.data.invalidReason !== ""));
     const quoteBlocked = hasQuoteError || quoteUnpriceable;
+    // The pristine baseline quote is intentionally un-priceable: a product has no
+    // price until its price driver — a room, or a traveler count — is chosen, and
+    // the engine reports that empty draft as `no_sell_amount_configured`. Suppress
+    // the un-priceable banner (and the side-panel error) for exactly that baseline
+    // so step 1 doesn't scream "can't be priced" before the buyer has picked
+    // anything. Any other reason (e.g. `rates_missing` on an already-picked stay,
+    // #2638), or the same code once a driver IS picked (a room that carries no
+    // price), still surfaces. Commit stays gated on `quoteBlocked` regardless, so
+    // an unpriced booking can never be submitted.
+    const priceDriversConfigured = (draft?.configure?.optionSelections?.length ?? 0) > 0 ||
+        Object.values(draft?.configure?.pax ?? {}).reduce((sum, n) => sum + (n ?? 0), 0) > 0;
+    const isPristineUnconfigured = quote.data?.invalidReason === "no_sell_amount_configured" && !priceDriversConfigured;
+    const showQuoteUnpriceable = quoteBlocked && !isPristineUnconfigured;
     const quoteReady = Boolean(quote.data?.quoteId);
     // Step navigation only hard-blocks on a thrown quote error (a transient fetch
     // failure that a retry fixes). An un-priceable quote (e.g. `rates_missing`
@@ -404,7 +417,7 @@ export function BookingJourney(props) {
         setConfirmError(null);
         void quote.refetch();
     };
-    const quoteErrorBanner = quoteBlocked ? (_jsxs("div", { role: "alert", "aria-live": "polite", className: "flex flex-col items-start gap-2 rounded-md border border-destructive/40 bg-destructive/5 p-3 text-destructive text-sm", children: [_jsx("span", { children: hasQuoteError
+    const quoteErrorBanner = showQuoteUnpriceable ? (_jsxs("div", { role: "alert", "aria-live": "polite", className: "flex flex-col items-start gap-2 rounded-md border border-destructive/40 bg-destructive/5 p-3 text-destructive text-sm", children: [_jsx("span", { children: hasQuoteError
                     ? messages.bookingJourney.validation.quoteFailed
                     : messages.bookingJourney.validation.pricingUnavailable }), hasQuoteError ? (_jsx(Button, { type: "button", variant: "outline", size: "sm", onClick: retryQuote, disabled: quote.isQuoting, children: messages.bookingJourney.validation.retryQuote })) : null] })) : null;
     // Renders one step's content. Shared by both layouts — the wizard shows

package/dist/journey/components/side-panel.js

@@ -31,7 +31,7 @@ export function PriceSidePanel({ pricing, isQuoting, invalidReason, entitySummar
     // Departure shows directly under the product title — it's the most-glanced
     // fact, so it doesn't belong buried in the recap accordion.
     const departureText = draft ? stepHeadline("departure", draft, messages, locale) : "";
-    return (_jsxs(Card, { className: className, children: [entitySummary?.heroImageUrl ? (_jsx("img", { src: entitySummary.heroImageUrl, alt: entitySummary.name, className: "aspect-video w-full object-cover" })) : null, entitySummary ? (_jsx(EntityHeader, { summary: entitySummary, departureText: departureText })) : null, _jsxs(CardContent, { className: "space-y-4", children: [steps && steps.length > 0 && currentStep && draft ? (_jsx(StepRecap, { steps: steps, currentStep: currentStep, draft: draft })) : null, invalidReason ? _jsx("p", { className: "text-destructive text-sm", children: invalidReason }) : null, !showPricing ? (_jsx("p", { className: "border-t pt-4 text-muted-foreground text-sm", children: pricingHint })) : isQuoting && !pricing ? (_jsxs("div", { className: "space-y-2 border-t pt-4", children: [_jsx(Skeleton, { className: "h-4 w-24" }), _jsx(Skeleton, { className: "h-4 w-32" }), _jsx(Skeleton, { className: "h-4 w-20" })] })) : null, showPricing && pricing ? (_jsxs("div", { className: "space-y-2 border-t pt-4", children: [_jsx("ul", { className: "space-y-1 text-sm", children: pricing.lines.map((line) => (_jsxs("li", { className: "flex justify-between", children: [_jsxs("span", { children: [line.label, line.quantity ? (_jsxs("span", { className: "text-muted-foreground", children: [" \u00D7 ", line.quantity] })) : null] }), _jsx("span", { children: formatMoney(line.totalAmount, pricing.currency, locale) })] }, `${line.kind}-${line.label}-${line.totalAmount}`))) }), pricing.taxes.length > 0 ? (_jsx("ul", { className: "space-y-1 border-t pt-2 text-sm text-muted-foreground", children: pricing.taxes.map((tax) => (_jsxs("li", { className: "flex justify-between", children: [_jsxs("span", { children: [tax.label, tax.rate > 0 ? ` (${formatTaxRate(tax.rate)})` : ""] }), _jsx("span", { children: formatMoney(tax.amount, pricing.currency, locale) })] }, tax.code))) })) : null, _jsxs("div", { className: "flex justify-between border-t pt-2 font-medium", children: [_jsx("span", { children: messages.bookingJourney.sidePanel.total }), _jsx("span", { children: formatMoney(pricing.total, pricing.currency, locale) })] })] })) : null, pricingExtras ? _jsx("div", { className: "border-t pt-4", children: pricingExtras }) : null] })] }));
+    return (_jsxs(Card, { className: className, children: [entitySummary?.heroImageUrl ? (_jsx("img", { src: entitySummary.heroImageUrl, alt: entitySummary.name, className: "aspect-video w-full object-cover" })) : null, entitySummary ? (_jsx(EntityHeader, { summary: entitySummary, departureText: departureText })) : null, _jsxs(CardContent, { className: "space-y-4", children: [steps && steps.length > 0 && currentStep && draft ? (_jsx(StepRecap, { steps: steps, currentStep: currentStep, draft: draft })) : null, invalidReason && showPricing ? (_jsx("p", { className: "text-destructive text-sm", children: messages.bookingJourney.validation.pricingUnavailable })) : null, !showPricing ? (_jsx("p", { className: "border-t pt-4 text-muted-foreground text-sm", children: pricingHint })) : isQuoting && !pricing ? (_jsxs("div", { className: "space-y-2 border-t pt-4", children: [_jsx(Skeleton, { className: "h-4 w-24" }), _jsx(Skeleton, { className: "h-4 w-32" }), _jsx(Skeleton, { className: "h-4 w-20" })] })) : null, showPricing && pricing ? (_jsxs("div", { className: "space-y-2 border-t pt-4", children: [_jsx("ul", { className: "space-y-1 text-sm", children: pricing.lines.map((line) => (_jsxs("li", { className: "flex justify-between", children: [_jsxs("span", { children: [line.label, line.quantity ? (_jsxs("span", { className: "text-muted-foreground", children: [" \u00D7 ", line.quantity] })) : null] }), _jsx("span", { children: formatMoney(line.totalAmount, pricing.currency, locale) })] }, `${line.kind}-${line.label}-${line.totalAmount}`))) }), pricing.taxes.length > 0 ? (_jsx("ul", { className: "space-y-1 border-t pt-2 text-sm text-muted-foreground", children: pricing.taxes.map((tax) => (_jsxs("li", { className: "flex justify-between", children: [_jsxs("span", { children: [tax.label, tax.rate > 0 ? ` (${formatTaxRate(tax.rate)})` : ""] }), _jsx("span", { children: formatMoney(tax.amount, pricing.currency, locale) })] }, tax.code))) })) : null, _jsxs("div", { className: "flex justify-between border-t pt-2 font-medium", children: [_jsx("span", { children: messages.bookingJourney.sidePanel.total }), _jsx("span", { children: formatMoney(pricing.total, pricing.currency, locale) })] })] })) : null, pricingExtras ? _jsx("div", { className: "border-t pt-4", children: pricingExtras }) : null] })] }));
 }
 function EntityHeader({ summary, departureText, }) {
     // Text block only — the hero image is rendered as a direct Card child above.

package/dist/requirements/hooks/use-booking-questions.d.ts

@@ -10,7 +10,7 @@ export declare function useBookingQuestions(options?: UseBookingQuestionsOptions
         label: string;
         description: string | null;
         target: "service" | "extra" | "traveler" | "booking" | "lead_traveler" | "booker";
-        fieldType: "number" | "boolean" | "date" | "other" | "file" | "email" | "phone" | "textarea" | "text" | "datetime" | "single_select" | "multi_select" | "country";
+        fieldType: "number" | "boolean" | "date" | "other" | "file" | "textarea" | "text" | "email" | "phone" | "datetime" | "single_select" | "multi_select" | "country";
         placeholder: string | null;
         helpText: string | null;
         isRequired: boolean;

package/dist/requirements/hooks/use-contact-requirements.d.ts

@@ -7,7 +7,7 @@ export declare function useContactRequirements(options?: UseContactRequirementsO
         id: string;
         productId: string;
         optionId: string | null;
-        fieldKey: "other" | "email" | "phone" | "address" | "nationality" | "first_name" | "last_name" | "date_of_birth" | "passport_number" | "passport_expiry" | "dietary_requirements" | "accessibility_needs" | "special_requests";
+        fieldKey: "other" | "address" | "email" | "phone" | "nationality" | "first_name" | "last_name" | "date_of_birth" | "passport_number" | "passport_expiry" | "dietary_requirements" | "accessibility_needs" | "special_requests";
         scope: "traveler" | "booking" | "lead_traveler" | "booker";
         isRequired: boolean;
         perTraveler: boolean;

package/dist/requirements/query-options.d.ts

@@ -57,7 +57,7 @@ export declare function getContactRequirementsQueryOptions(client: FetchWithVali
         id: string;
         productId: string;
         optionId: string | null;
-        fieldKey: "other" | "email" | "phone" | "address" | "nationality" | "first_name" | "last_name" | "date_of_birth" | "passport_number" | "passport_expiry" | "dietary_requirements" | "accessibility_needs" | "special_requests";
+        fieldKey: "other" | "address" | "email" | "phone" | "nationality" | "first_name" | "last_name" | "date_of_birth" | "passport_number" | "passport_expiry" | "dietary_requirements" | "accessibility_needs" | "special_requests";
         scope: "traveler" | "booking" | "lead_traveler" | "booker";
         isRequired: boolean;
         perTraveler: boolean;
@@ -73,7 +73,7 @@ export declare function getContactRequirementsQueryOptions(client: FetchWithVali
         id: string;
         productId: string;
         optionId: string | null;
-        fieldKey: "other" | "email" | "phone" | "address" | "nationality" | "first_name" | "last_name" | "date_of_birth" | "passport_number" | "passport_expiry" | "dietary_requirements" | "accessibility_needs" | "special_requests";
+        fieldKey: "other" | "address" | "email" | "phone" | "nationality" | "first_name" | "last_name" | "date_of_birth" | "passport_number" | "passport_expiry" | "dietary_requirements" | "accessibility_needs" | "special_requests";
         scope: "traveler" | "booking" | "lead_traveler" | "booker";
         isRequired: boolean;
         perTraveler: boolean;
@@ -90,7 +90,7 @@ export declare function getContactRequirementsQueryOptions(client: FetchWithVali
             id: string;
             productId: string;
             optionId: string | null;
-            fieldKey: "other" | "email" | "phone" | "address" | "nationality" | "first_name" | "last_name" | "date_of_birth" | "passport_number" | "passport_expiry" | "dietary_requirements" | "accessibility_needs" | "special_requests";
+            fieldKey: "other" | "address" | "email" | "phone" | "nationality" | "first_name" | "last_name" | "date_of_birth" | "passport_number" | "passport_expiry" | "dietary_requirements" | "accessibility_needs" | "special_requests";
             scope: "traveler" | "booking" | "lead_traveler" | "booker";
             isRequired: boolean;
             perTraveler: boolean;
@@ -109,7 +109,7 @@ export declare function getContactRequirementsQueryOptions(client: FetchWithVali
                 id: string;
                 productId: string;
                 optionId: string | null;
-                fieldKey: "other" | "email" | "phone" | "address" | "nationality" | "first_name" | "last_name" | "date_of_birth" | "passport_number" | "passport_expiry" | "dietary_requirements" | "accessibility_needs" | "special_requests";
+                fieldKey: "other" | "address" | "email" | "phone" | "nationality" | "first_name" | "last_name" | "date_of_birth" | "passport_number" | "passport_expiry" | "dietary_requirements" | "accessibility_needs" | "special_requests";
                 scope: "traveler" | "booking" | "lead_traveler" | "booker";
                 isRequired: boolean;
                 perTraveler: boolean;
@@ -132,7 +132,7 @@ export declare function getBookingQuestionsQueryOptions(client: FetchWithValidat
         label: string;
         description: string | null;
         target: "service" | "extra" | "traveler" | "booking" | "lead_traveler" | "booker";
-        fieldType: "number" | "boolean" | "date" | "other" | "file" | "email" | "phone" | "textarea" | "text" | "datetime" | "single_select" | "multi_select" | "country";
+        fieldType: "number" | "boolean" | "date" | "other" | "file" | "textarea" | "text" | "email" | "phone" | "datetime" | "single_select" | "multi_select" | "country";
         placeholder: string | null;
         helpText: string | null;
         isRequired: boolean;
@@ -150,7 +150,7 @@ export declare function getBookingQuestionsQueryOptions(client: FetchWithValidat
         label: string;
         description: string | null;
         target: "service" | "extra" | "traveler" | "booking" | "lead_traveler" | "booker";
-        fieldType: "number" | "boolean" | "date" | "other" | "file" | "email" | "phone" | "textarea" | "text" | "datetime" | "single_select" | "multi_select" | "country";
+        fieldType: "number" | "boolean" | "date" | "other" | "file" | "textarea" | "text" | "email" | "phone" | "datetime" | "single_select" | "multi_select" | "country";
         placeholder: string | null;
         helpText: string | null;
         isRequired: boolean;
@@ -169,7 +169,7 @@ export declare function getBookingQuestionsQueryOptions(client: FetchWithValidat
             label: string;
             description: string | null;
             target: "service" | "extra" | "traveler" | "booking" | "lead_traveler" | "booker";
-            fieldType: "number" | "boolean" | "date" | "other" | "file" | "email" | "phone" | "textarea" | "text" | "datetime" | "single_select" | "multi_select" | "country";
+            fieldType: "number" | "boolean" | "date" | "other" | "file" | "textarea" | "text" | "email" | "phone" | "datetime" | "single_select" | "multi_select" | "country";
             placeholder: string | null;
             helpText: string | null;
             isRequired: boolean;
@@ -190,7 +190,7 @@ export declare function getBookingQuestionsQueryOptions(client: FetchWithValidat
                 label: string;
                 description: string | null;
                 target: "service" | "extra" | "traveler" | "booking" | "lead_traveler" | "booker";
-                fieldType: "number" | "boolean" | "date" | "other" | "file" | "email" | "phone" | "textarea" | "text" | "datetime" | "single_select" | "multi_select" | "country";
+                fieldType: "number" | "boolean" | "date" | "other" | "file" | "textarea" | "text" | "email" | "phone" | "datetime" | "single_select" | "multi_select" | "country";
                 placeholder: string | null;
                 helpText: string | null;
                 isRequired: boolean;

package/dist/requirements/schemas.d.ts

@@ -22,10 +22,10 @@ export declare const questionFieldTypeSchema: z.ZodEnum<{
     date: "date";
     other: "other";
     file: "file";
-    email: "email";
-    phone: "phone";
     textarea: "textarea";
     text: "text";
+    email: "email";
+    phone: "phone";
     datetime: "datetime";
     single_select: "single_select";
     multi_select: "multi_select";
@@ -51,10 +51,10 @@ export declare const bookingQuestionSchema: z.ZodObject<{
         date: "date";
         other: "other";
         file: "file";
-        email: "email";
-        phone: "phone";
         textarea: "textarea";
         text: "text";
+        email: "email";
+        phone: "phone";
         datetime: "datetime";
         single_select: "single_select";
         multi_select: "multi_select";
@@ -79,9 +79,9 @@ export declare const bookingQuestionOptionSchema: z.ZodObject<{
 export type BookingQuestionOption = z.infer<typeof bookingQuestionOptionSchema>;
 export declare const contactFieldKeySchema: z.ZodEnum<{
     other: "other";
+    address: "address";
     email: "email";
     phone: "phone";
-    address: "address";
     nationality: "nationality";
     first_name: "first_name";
     last_name: "last_name";
@@ -104,9 +104,9 @@ export declare const contactRequirementSchema: z.ZodObject<{
     optionId: z.ZodNullable<z.ZodString>;
     fieldKey: z.ZodEnum<{
         other: "other";
+        address: "address";
         email: "email";
         phone: "phone";
-        address: "address";
         nationality: "nationality";
         first_name: "first_name";
         last_name: "last_name";
@@ -162,10 +162,10 @@ export declare const bookingQuestionListResponse: z.ZodObject<{
             date: "date";
             other: "other";
             file: "file";
-            email: "email";
-            phone: "phone";
             textarea: "textarea";
             text: "text";
+            email: "email";
+            phone: "phone";
             datetime: "datetime";
             single_select: "single_select";
             multi_select: "multi_select";
@@ -202,9 +202,9 @@ export declare const contactRequirementListResponse: z.ZodObject<{
         optionId: z.ZodNullable<z.ZodString>;
         fieldKey: z.ZodEnum<{
             other: "other";
+            address: "address";
             email: "email";
             phone: "phone";
-            address: "address";
             nationality: "nationality";
             first_name: "first_name";
             last_name: "last_name";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@voyant-travel/bookings-react",
-  "version": "0.185.0",
+  "version": "0.186.1",
   "license": "Apache-2.0",
   "repository": {
     "type": "git",
@@ -150,24 +150,24 @@
     "react-dom": "^19.0.0",
     "react-hook-form": "^7.80.0",
     "zod": "^4.0.0",
-    "@voyant-travel/accommodations": "^0.145.0",
     "@voyant-travel/admin": "^0.128.3",
-    "@voyant-travel/bookings": "^0.185.0",
-    "@voyant-travel/catalog": "^0.183.0",
-    "@voyant-travel/catalog-react": "^0.183.0",
-    "@voyant-travel/distribution-react": "^0.175.0",
-    "@voyant-travel/relationships-react": "^0.185.0",
-    "@voyant-travel/finance": "^0.185.0",
-    "@voyant-travel/finance-react": "^0.185.0",
-    "@voyant-travel/identity-react": "^0.185.0",
-    "@voyant-travel/legal-react": "^0.185.0",
-    "@voyant-travel/commerce-react": "^0.67.0",
-    "@voyant-travel/cruises": "^0.184.0",
-    "@voyant-travel/inventory-react": "^0.67.0",
-    "@voyant-travel/inventory": "^0.14.21",
-    "@voyant-travel/storefront-react": "^0.187.0",
+    "@voyant-travel/bookings": "^0.186.1",
+    "@voyant-travel/accommodations": "^0.146.0",
+    "@voyant-travel/catalog": "^0.184.0",
+    "@voyant-travel/catalog-react": "^0.184.0",
+    "@voyant-travel/distribution-react": "^0.176.0",
+    "@voyant-travel/relationships-react": "^0.186.0",
+    "@voyant-travel/finance-react": "^0.186.0",
+    "@voyant-travel/identity-react": "^0.186.0",
+    "@voyant-travel/legal-react": "^0.186.0",
+    "@voyant-travel/cruises": "^0.185.0",
+    "@voyant-travel/inventory-react": "^0.68.0",
+    "@voyant-travel/inventory": "^0.14.22",
     "@voyant-travel/ui": "^0.109.3",
-    "@voyant-travel/operations-react": "^0.66.0"
+    "@voyant-travel/storefront-react": "^0.188.0",
+    "@voyant-travel/operations-react": "^0.67.0",
+    "@voyant-travel/finance": "^0.186.0",
+    "@voyant-travel/commerce-react": "^0.68.0"
   },
   "peerDependenciesMeta": {
     "@tanstack/react-table": {
@@ -251,26 +251,26 @@
     "typescript": "^6.0.3",
     "vitest": "^4.1.9",
     "zod": "^4.4.3",
-    "@voyant-travel/accommodations": "^0.145.0",
+    "@voyant-travel/accommodations": "^0.146.0",
     "@voyant-travel/admin": "^0.128.3",
-    "@voyant-travel/bookings": "^0.185.0",
-    "@voyant-travel/catalog": "^0.183.0",
-    "@voyant-travel/catalog-react": "^0.183.0",
-    "@voyant-travel/distribution-react": "^0.175.0",
-    "@voyant-travel/relationships-react": "^0.185.0",
-    "@voyant-travel/finance": "^0.185.0",
-    "@voyant-travel/finance-react": "^0.185.0",
-    "@voyant-travel/identity-react": "^0.185.0",
-    "@voyant-travel/legal-react": "^0.185.0",
-    "@voyant-travel/commerce-react": "^0.67.0",
-    "@voyant-travel/cruises": "^0.184.0",
-    "@voyant-travel/inventory-react": "^0.67.0",
-    "@voyant-travel/inventory": "^0.14.21",
+    "@voyant-travel/bookings": "^0.186.1",
+    "@voyant-travel/catalog": "^0.184.0",
+    "@voyant-travel/catalog-react": "^0.184.0",
+    "@voyant-travel/distribution-react": "^0.176.0",
+    "@voyant-travel/relationships-react": "^0.186.0",
+    "@voyant-travel/finance": "^0.186.0",
+    "@voyant-travel/finance-react": "^0.186.0",
+    "@voyant-travel/identity-react": "^0.186.0",
+    "@voyant-travel/legal-react": "^0.186.0",
+    "@voyant-travel/commerce-react": "^0.68.0",
+    "@voyant-travel/cruises": "^0.185.0",
+    "@voyant-travel/inventory-react": "^0.68.0",
+    "@voyant-travel/inventory": "^0.14.22",
     "@voyant-travel/react": "^0.104.2",
-    "@voyant-travel/storefront-react": "^0.187.0",
+    "@voyant-travel/storefront-react": "^0.188.0",
     "@voyant-travel/ui": "^0.109.3",
     "@voyant-travel/voyant-typescript-config": "^0.1.0",
-    "@voyant-travel/operations-react": "^0.66.0"
+    "@voyant-travel/operations-react": "^0.67.0"
   },
   "files": [
     "dist",