@aglyn/plugins-bookings 1.0.0-beta.218 → 1.0.0-beta.220

package/src/lib/server.js

@@ -22,7 +22,8 @@ import { checkEntitlement, registerPluginConfigSchema, registerPluginJob, resolv
 // reach the real verdict.
 import { siteLockdownJobGate } from "@aglyn/tenant-data-admin/server/tenant-write-lockdown";
 import { loadHostEmailWithTokens, renderLoadedHostEmailWithTokens } from "@aglyn/tenant-data-admin/server/host-email-tokens";
-import { BOOKING_MAX_DAYS_AHEAD, computeOpenSlots, isBookingReminderDue, isSlotOpen, REMINDER_WINDOW_END_HOURS, REMINDER_WINDOW_START_HOURS } from "./model/index.js";
+import { BOOKING_MAX_DAYS_AHEAD, bookingTimeZone, computeOpenSlots, formatBookingWhen, isBookingReminderDue, isSlotOpen, REMINDER_WINDOW_END_HOURS, REMINDER_WINDOW_START_HOURS } from "./model/index.js";
+import { bookingTimeZoneFor } from "./server/booking-time-zone.js";
 import { registerBillingWebhookHandler, registerPluginApiRoute } from "@aglyn/aglyn/server";
 import { pluginTaxProfile } from "@aglyn/aglyn/plugin-manager/plugin-tax-profile";
 import { BUNDLE_ID } from "./constants/bundle-common.js";
@@ -31,12 +32,10 @@ import { bookingsBillingWebhookHandler } from "./server/billing-webhook.js";
 import { bookingAnalyticsHandler } from "./server/booking-analytics.js";
 import { registerBookingFigureReader } from "./server/booking-figures.js";
 import { bookingRefundHandler } from "./server/refund.js";
-// The booking's way back to the CRM record (AGL-2660): the reference a
-// booking link carried, and the meeting a free booking files on landing.
-// The leaf, not the barrel: this library's specs substitute the barrel
-// wholesale, and a parser that vanished under them would fail every
-// booking they take.
-import { formatCrmBookingRef, parseCrmBookingRef } from "@aglyn/aglyn/app-utils/crm-booking";
+// The booking's way back to the record it was booked from (AGL-2660): the
+// reference a booking link carried, and the meeting a free booking files on
+// landing.
+import { formatBookingRecordRef, parseBookingRecordRef } from "./model/booking-record.js";
 import { fileBookingOnCrm } from "./server/booking-crm.js";
 // Settings schema (AGL-428): registered here too so server-only loads
 // (API dispatchers) get defaults-merged getPluginConfig reads.
@@ -87,7 +86,7 @@ registerPluginJob({
     }
 });
 import { firebaseAdmin, getOrgForHost, meterHostEmail, notifyHostManagers, attributeCampaignConversion, resolveCampaignTouch, getPluginConfig, resolveOrgIdForHost, renderHostEmailWithTokens, hostSendingIdentity } from "@aglyn/tenant-data-admin";
-import { connectLinkageIsReady } from "@aglyn/tenant-data-admin/server/stripe-account-mode";
+import { merchantAccountIsReady } from "@aglyn/tenant-data-admin/server/payment-provider";
 import { checkoutSessionCardAuthenticationParams } from "@aglyn/tenant-data-admin/server/stripe-card-authentication";
 import { emitHostEvent } from "@aglyn/tenant-runtime";
 import recordCapturedContact from "@aglyn/aglyn/plugin-manager/record-captured-contact";
@@ -236,10 +235,10 @@ export const slotsHandler = async (req, res)=>{
     // consent to be emailed marketing, so this is only set when the visitor
     // checked it.
     const marketingConsent = ((_req_body5 = req.body) == null ? void 0 : _req_body5.marketingConsent) === true;
-    // The CRM record the booking link named (AGL-2660), kept on the row so
-    // the booking can be attributed to it even when the visitor books with
-    // a different address. Parsed, never stored raw: this is a public door.
-    const crmRef = parseCrmBookingRef((_req_body6 = req.body) == null ? void 0 : _req_body6.crmRef);
+    // The record the booking link named (AGL-2660), kept on the row so the
+    // booking can be attributed to it even when the visitor books with a
+    // different address. Parsed, never stored raw: this is a public door.
+    const crmRef = parseBookingRecordRef((_req_body6 = req.body) == null ? void 0 : _req_body6.crmRef);
     if (!hostId || !serviceId || !Number.isFinite(startsAtMs) || !startsAtMs) {
         return res.status(400).json({
             error: 'Invalid booking request'
@@ -270,7 +269,7 @@ export const slotsHandler = async (req, res)=>{
         });
     }
     try {
-        var _service_priceUsd, _service_name, _service_name1, _service_name2;
+        var _service_priceUsd, _service_name, _service_name1, _service_name2, _service_name3;
         var _this, _req_body7;
         const firestore = firebaseAdmin.app().firestore();
         const hostRef = firestore.collection('hosts').doc(hostId);
@@ -302,6 +301,16 @@ export const slotsHandler = async (req, res)=>{
         const durationMs = Math.max(5, Math.round(service.durationMinutes || 30)) * 60000;
         const endsAtMs = startsAtMs + durationMs;
         const bookingsRef = hostRef.collection('bookings');
+        // The zone this booking's time is told in (AGL-3432), stored on the
+        // booking so the paid confirmation and the reminder — which read the
+        // booking and not the service — state the same wall-clock time as the
+        // free confirmation below, and name the zone it is in.
+        const timezone = bookingTimeZone({
+            service,
+            host: hostSnapshot.data == null ? void 0 : hostSnapshot.data.call(hostSnapshot),
+            org: ownerOrg
+        });
+        const when = formatBookingWhen(startsAtMs, timezone);
         // Paid services (AGL-170): the slot is HELD pending payment — the
         // booking lands as `pendingPayment` with a 15-minute expiry (expired
         // holds release the slot in the collision filters), and the visitor
@@ -337,17 +346,18 @@ export const slotsHandler = async (req, res)=>{
             pct: 0
         };
         if (paid) {
-            var _ref5, _ref6;
-            var _storeSnapshot_data;
+            var _ref5;
             const ownerUid = ownerOrg == null ? void 0 : ownerOrg.ownerUid;
-            const [ownerProfile, storeSnapshot] = await Promise.all([
+            const [ownerProfile, serviceRate] = await Promise.all([
                 ownerUid ? firestore.collection('profiles').doc(String(ownerUid)).get() : Promise.resolve(null),
                 // The merchant's own service rate (AGL-2028), in the SAME round trip
-                // as the connected-account read this path already makes.
-                hostRef.collection('settings').doc('store').get()
+                // as the connected-account read this path already makes — asked of
+                // the plugin that keeps the merchant's tax settings, which reads them
+                // where it stores them.
+                pluginTaxProfile().flatRate(hostId, 'service')
             ]);
             chargeAccountId = String((_ref5 = ownerProfile == null ? void 0 : ownerProfile.get('stripeAccountId')) != null ? _ref5 : '');
-            if (!connectLinkageIsReady({
+            if (!merchantAccountIsReady({
                 accountId: chargeAccountId,
                 chargesEnabled: ownerProfile == null ? void 0 : ownerProfile.get('stripeChargesEnabled'),
                 accountLivemode: ownerProfile == null ? void 0 : ownerProfile.get('stripeAccountLivemode')
@@ -426,12 +436,11 @@ export const slotsHandler = async (req, res)=>{
             // DEFAULT OFF, and load-bearing: an absent, zero, negative or
             // out-of-range rate resolves to zero, so no existing merchant's charge
             // moves because this shipped.
-            const taxSettings = (_ref6 = storeSnapshot == null ? void 0 : (_storeSnapshot_data = storeSnapshot.data()) == null ? void 0 : _storeSnapshot_data.tax) != null ? _ref6 : {};
             const chargeCents = Math.round(priceUsd * 100);
             // Asked of the plugin that owns the merchant's tax rule, which throws
             // rather than answer zero if none is loaded: an untaxed charge recorded
             // as untaxed is the failure nobody sees until the merchant owes it.
-            serviceTax = pluginTaxProfile().flatTax(taxSettings.service, chargeCents, 'Service tax');
+            serviceTax = pluginTaxProfile().flatTax(serviceRate, chargeCents, 'Service tax');
             // THE TWO FEE BASES ARE NOW DIFFERENT NUMBERS (AGL-2028). The take is
             // computed on the SERVICE, never on the state's money — AGL-2317's rule
             // — while Stripe's card cost is debited from the platform balance on
@@ -470,11 +479,12 @@ export const slotsHandler = async (req, res)=>{
                 email,
                 startsAtMs,
                 endsAtMs,
+                timezone,
                 status: paid ? 'pendingPayment' : 'confirmed'
             }, paid && {
                 expiresAtMs: Date.now() + 15 * 60000
             }, crmRef ? {
-                crmRef: formatCrmBookingRef(crmRef)
+                crmRef: formatBookingRecordRef(crmRef)
             } : {}, {
                 createdAt: FieldValue.serverTimestamp()
             }));
@@ -548,7 +558,7 @@ export const slotsHandler = async (req, res)=>{
             }
         } : {}));
         if (paid) {
-            var _req_headers_host, _service_name3;
+            var _req_headers_host, _service_name4;
             // The site's own origin, never a header's say-so (AGL-3363): a
             // caller writes `Origin` and `Host`, and Stripe sends the payer here.
             const origin = siteReturnOrigin({
@@ -582,7 +592,7 @@ export const slotsHandler = async (req, res)=>{
                 'line_items[0][quantity]': '1',
                 'line_items[0][price_data][currency]': 'usd',
                 'line_items[0][price_data][unit_amount]': String(Math.round(priceUsd * 100)),
-                'line_items[0][price_data][product_data][name]': String((_service_name3 = service.name) != null ? _service_name3 : 'Booking').slice(0, 120)
+                'line_items[0][price_data][product_data][name]': String((_service_name4 = service.name) != null ? _service_name4 : 'Booking').slice(0, 120)
             }, serviceTax.taxCents > 0 ? {
                 'line_items[1][quantity]': '1',
                 'line_items[1][price_data][currency]': 'usd',
@@ -672,15 +682,16 @@ export const slotsHandler = async (req, res)=>{
             });
         }
         // Event trigger (AGL-128/148/159).
-        // In-app notification to the site's managers (AGL-259).
+        // In-app notification to the site's managers (AGL-259): who booked what,
+        // and when, in the booking's own zone and naming it (AGL-3432).
         void notifyHostManagers(hostId, {
             type: 'content.booking',
-            title: 'New booking',
-            body: new Date(startsAtMs).toLocaleString(),
+            title: 'New booking on {site}',
+            body: `${name} (${email}) booked ${String((_service_name1 = service.name) != null ? _service_name1 : 'a service')} ` + `on {site} for ${when} (${timezone}).`,
             link: `/${hostId}/bookings`
         });
         const { alerts } = await emitHostEvent(hostId, 'booking', {
-            serviceName: (_service_name1 = service.name) != null ? _service_name1 : '',
+            serviceName: (_service_name2 = service.name) != null ? _service_name2 : '',
             email,
             startsAtMs
         }, // The visitor who booked (AGL-3376).
@@ -692,22 +703,18 @@ export const slotsHandler = async (req, res)=>{
         });
         // Env-gated confirmation email (same provider as AGL-98).
         if (isEmailConfigured()) {
-            var _service_name4, _ref7;
+            var _service_name5, _ref6;
             var _this1;
-            const timezone = service.timezone || 'UTC';
-            const when = new Intl.DateTimeFormat('en-US', {
-                timeZone: timezone,
-                dateStyle: 'full',
-                timeStyle: 'short'
-            }).format(new Date(startsAtMs));
             const fallbackText = `Hi ${name},\n\nYour booking for "${service.name}" is ` + `confirmed for ${when} (${timezone}).\n\n` + `Reference: ${bookingId}`;
             // Site-owner-designed template when published (AGL-770); null keeps the
             // built-in copy above.
             const designed = await renderHostEmailWithTokens(firestore, hostId, 'booking-confirmed', {
                 name: String(name != null ? name : ''),
-                'service.name': String((_service_name4 = service.name) != null ? _service_name4 : ''),
+                'service.name': String((_service_name5 = service.name) != null ? _service_name5 : ''),
                 when,
                 timezone,
+                // A free booking charged nothing: empty, so the line is left out.
+                'booking.payment': '',
                 'booking.ref': String(bookingId)
             });
             // White-label sender identity (White-Label Phase 3): the store's brand
@@ -715,7 +722,7 @@ export const slotsHandler = async (req, res)=>{
             const branding = resolveBrandingProfile((_this1 = await getOrgForHost(hostId).catch(()=>null)) == null ? void 0 : _this1.org);
             await sendEmail(_extends({
                 to: email,
-                subject: (_ref7 = designed == null ? void 0 : designed.subject) != null ? _ref7 : `Booking confirmed: ${service.name}`,
+                subject: (_ref6 = designed == null ? void 0 : designed.subject) != null ? _ref6 : `Booking confirmed: ${service.name}`,
                 text: (designed == null ? void 0 : designed.text) || fallbackText
             }, (designed == null ? void 0 : designed.html) ? {
                 html: designed.html
@@ -745,12 +752,12 @@ export const slotsHandler = async (req, res)=>{
             bookingId,
             booking: _extends({
                 serviceId,
-                serviceName: (_service_name2 = service.name) != null ? _service_name2 : '',
+                serviceName: (_service_name3 = service.name) != null ? _service_name3 : '',
                 email,
                 startsAtMs,
                 endsAtMs
             }, crmRef ? {
-                crmRef: formatCrmBookingRef(crmRef)
+                crmRef: formatBookingRecordRef(crmRef)
             } : {}),
             service
         });
@@ -813,6 +820,9 @@ export const slotsHandler = async (req, res)=>{
    * reads per reminder rather than per host. Scoped to the run, so a domain
    * un-verified between runs is picked up on the next one.
    */ const identityByHost = new Map();
+    // The zone for a booking that predates storing one (AGL-3432), resolved
+    // once per service for the whole run.
+    const zoneByService = new Map();
     for (const doc of upcoming.docs){
         var _ref, _data_serviceName, _data_name, _ref1, _data_name1;
         var _doc_ref_parent_parent, _brandingByHost_get;
@@ -824,10 +834,6 @@ export const slotsHandler = async (req, res)=>{
             skipped += 1;
             continue;
         }
-        const when = new Date(Number(data['startsAtMs'])).toLocaleString('en-US', {
-            dateStyle: 'full',
-            timeStyle: 'short'
-        });
         // bookings live at hosts/{hostId}/bookings/{id}, so the grandparent is
         // the host.
         const hostId = (_ref = (_doc_ref_parent_parent = doc.ref.parent.parent) == null ? void 0 : _doc_ref_parent_parent.id) != null ? _ref : '';
@@ -850,6 +856,10 @@ export const slotsHandler = async (req, res)=>{
             skippedLocked += 1;
             continue;
         }
+        // The booking's own zone, named beside the time (AGL-3432). A formatter
+        // with no zone reads in the server's — UTC — and states the wrong hour.
+        const timezone = await bookingTimeZoneFor(firestore, hostId, data, zoneByService);
+        const when = formatBookingWhen(Number(data['startsAtMs']), timezone);
         let loaded = templateCache.get(hostId);
         if (loaded === undefined) {
             var _this;
@@ -862,12 +872,13 @@ export const slotsHandler = async (req, res)=>{
         const designed = loaded ? renderLoadedHostEmailWithTokens(loaded, {
             name: String((_data_name = data['name']) != null ? _data_name : ''),
             'service.name': serviceName,
-            when
+            when,
+            timezone
         }) : null;
         const result = await sendEmail(_extends({
             to: String(data['email']),
             subject: (_ref1 = designed == null ? void 0 : designed.subject) != null ? _ref1 : `Reminder: ${serviceName} tomorrow`,
-            text: (designed == null ? void 0 : designed.text) || `Hi ${(_data_name1 = data['name']) != null ? _data_name1 : ''},\n\nA reminder that "${serviceName}" is ` + `scheduled for ${when}.\n\nReference: ${doc.id}`
+            text: (designed == null ? void 0 : designed.text) || `Hi ${(_data_name1 = data['name']) != null ? _data_name1 : ''},\n\nA reminder that "${serviceName}" is ` + `scheduled for ${when} (${timezone}).\n\nReference: ${doc.id}`
         }, (designed == null ? void 0 : designed.html) ? {
             html: designed.html
         } : {}, {