@bash-app/bash-common 30.423.0 → 30.425.0

package/src/utils/__tests__/badgeUtils.test.ts

@@ -1,5 +1,5 @@
 /**
- * badgeUtils — host level / temp-badge helpers.
+ * badgeUtils. Host level / temp-badge helpers.
  *
  * The level-badge calculation is the public reputation signal on every host
  * profile. A bug here either over- or under-credits a host (= unfair / loss
@@ -22,7 +22,7 @@ import {
 import type { UserTemporaryBadge } from "../../definitions.js";
 
 // ============================================================================
-// getLevelBadgeById / getTemporaryBadgeById — registry lookups
+// getLevelBadgeById / getTemporaryBadgeById. Registry lookups
 // ============================================================================
 
 describe("getLevelBadgeById", () => {
@@ -88,7 +88,7 @@ describe("isTemporaryBadgeExpired", () => {
 
   it("returns true when expiresOn is RIGHT NOW (boundary: not strictly in the future)", () => {
     // Source uses `expiryDate < currentDate`. A simultaneous tick = false (not expired),
-    // but Date.now() inside the function is sampled later than ours — which is enough
+    // but Date.now() inside the function is sampled later than ours. Which is enough
     // to flip <. We pin the practical observable: "earlier than now → expired".
     const past = new Date(Date.now() - 1).toISOString();
     const badge: UserTemporaryBadge = {
@@ -130,10 +130,10 @@ describe("filterExpiredTemporaryBadges", () => {
 });
 
 // ============================================================================
-// determineLevelBadge — the reputation ladder
+// determineLevelBadge. The reputation ladder
 // ============================================================================
 
-describe("determineLevelBadge — host reputation ladder", () => {
+describe("determineLevelBadge. Host reputation ladder", () => {
   it("returns null when the host has no events", () => {
     expect(determineLevelBadge(0, 5)).toBeNull();
   });

package/src/utils/__tests__/bashPointsPaymentUtils.test.ts

@@ -1,5 +1,5 @@
 /**
- * bashPointsPaymentUtils — earning math + display + deprecated stubs.
+ * bashPointsPaymentUtils. Earning math + display + deprecated stubs.
  *
  * BashPoints are loyalty cashback. The active code paths are:
  *   - calculateBashPointsCashback(amountCents, rate) → integer points
@@ -8,7 +8,7 @@
  * The "service booking" overloads were intentionally deprecated because
  * BashPoints have no fixed dollar value and cannot pay for services. We pin
  * the deprecated stubs here so they stay benign (return safe defaults, log a
- * warning) — accidentally re-enabling them would cause real money loss.
+ * warning). Accidentally re-enabling them would cause real money loss.
  */
 import {
   formatBashPoints,
@@ -39,7 +39,7 @@ describe("formatBashPoints", () => {
     expect(formatBashPoints(1_000_000)).toBe("1,000,000 points");
   });
 
-  it("formats negative values (defensive — should never happen but no throw)", () => {
+  it("formats negative values (defensive. Should never happen but no throw)", () => {
     // -1 !== 1 so it gets the plural suffix
     expect(formatBashPoints(-5)).toBe("-5 points");
   });
@@ -78,7 +78,7 @@ describe("calculateBashPointsCashback", () => {
   });
 });
 
-describe("DEPRECATED stubs — must remain safe defaults", () => {
+describe("DEPRECATED stubs. Must remain safe defaults", () => {
   let warnSpy: jest.SpyInstance;
 
   beforeEach(() => {

package/src/utils/__tests__/contentFilterUtils.test.ts

@@ -1,5 +1,5 @@
 /**
- * contentFilterUtils — anti-fee-evasion content moderation.
+ * contentFilterUtils. Anti-fee-evasion content moderation.
  *
  * Strategic context: hosts can lose Bash money (and break ToS) by telling
  * attendees "DM me on Venmo to pay" instead of using the platform's payment
@@ -27,10 +27,10 @@ import {
 } from "../contentFilterUtils.js";
 
 // ============================================================================
-// filterContent — defensive input handling
+// filterContent. Defensive input handling
 // ============================================================================
 
-describe("filterContent — defensive input handling", () => {
+describe("filterContent. Defensive input handling", () => {
   it("returns clean for empty / null / non-string input without throwing", () => {
     expect(filterContent("")).toEqual({
       isClean: true,
@@ -69,10 +69,10 @@ describe("filterContent — defensive input handling", () => {
 });
 
 // ============================================================================
-// filterContent — P2P payment service detection (the core money use case)
+// filterContent. P2P payment service detection (the core money use case)
 // ============================================================================
 
-describe("filterContent — direct P2P payment service mentions", () => {
+describe("filterContent. Direct P2P payment service mentions", () => {
   it.each([
     ["venmo", "Venmo"],
     ["zelle", "Zelle"],
@@ -111,10 +111,10 @@ describe("filterContent — direct P2P payment service mentions", () => {
 });
 
 // ============================================================================
-// filterContent — leetspeak / dot-separator bypass attempts
+// filterContent. Leetspeak / dot-separator bypass attempts
 // ============================================================================
 
-describe("filterContent — bypass attempts (leetspeak, dot-separators)", () => {
+describe("filterContent. Bypass attempts (leetspeak, dot-separators)", () => {
   it("catches dot-separated 'v.e.n.m.o'", () => {
     const result = filterContent("Send to v.e.n.m.o please");
     expect(result.isClean).toBe(false);
@@ -136,7 +136,7 @@ describe("filterContent — bypass attempts (leetspeak, dot-separators)", () =>
     expect(result.isClean).toBe(false);
     // 'v3nm0' matches both the literal P2P pattern (recorded as p2p_payment / low)
     // AND the phonetic pattern (recorded as bypass_attempt / high). We assert the
-    // phonetic / high entry exists — that's the one that drives enforcement.
+    // phonetic / high entry exists. That's the one that drives enforcement.
     const phoneticHigh = result.violations.find(
       (v) =>
         v.match === "v3nm0" &&
@@ -157,10 +157,10 @@ describe("filterContent — bypass attempts (leetspeak, dot-separators)", () =>
 });
 
 // ============================================================================
-// filterContent — high-severity contact info (handles, phones, emails)
+// filterContent. High-severity contact info (handles, phones, emails)
 // ============================================================================
 
-describe("filterContent — high-severity contact info patterns", () => {
+describe("filterContent. High-severity contact info patterns", () => {
   it("flags '@handle' style mentions as high severity (Venmo handle)", () => {
     const result = filterContent("DM @cooluser123");
     const handleMatch = result.violations.find((v) =>
@@ -199,7 +199,7 @@ describe("filterContent — high-severity contact info patterns", () => {
   it("DOES detect parenthesized US phone numbers, but classifies them as 'low' (known gap in getSeverity)", () => {
     // The pattern /\(\d{3}\)\s*\d{3}[-.]?\d{4}/ matches "(415) 555-0123",
     // but getSeverity's high-severity heuristic uses /\d{3}[-.]?\d{3}[-.]?\d{4}/
-    // which doesn't allow ')' between the area code and prefix — so this gets 'low'.
+    // which doesn't allow ')' between the area code and prefix. So this gets 'low'.
     // This test pins that behavior so a future tightening of getSeverity is intentional.
     const result = filterContent("(415) 555-0123");
     const phoneViolation = result.violations.find((v) =>
@@ -227,10 +227,10 @@ describe("filterContent — high-severity contact info patterns", () => {
 });
 
 // ============================================================================
-// filterContent — bypass phrasing (DM me, contact me directly, etc.)
+// filterContent. Bypass phrasing (DM me, contact me directly, etc.)
 // ============================================================================
 
-describe("filterContent — bypass phrasing", () => {
+describe("filterContent. Bypass phrasing", () => {
   it.each([
     "dm me for payment",
     "message me to pay",
@@ -266,10 +266,10 @@ describe("filterContent — bypass phrasing", () => {
 });
 
 // ============================================================================
-// filterContent — overall severity bubbling
+// filterContent. Overall severity bubbling
 // ============================================================================
 
-describe("filterContent — overall severity rollup", () => {
+describe("filterContent. Overall severity rollup", () => {
   it("returns 'low' overall severity for clean content", () => {
     expect(filterContent("Have a great party").severity).toBe("low");
   });
@@ -287,7 +287,7 @@ describe("filterContent — overall severity rollup", () => {
 
   it("escalates to 'high' overall when a brand mention also trips the phonetic pattern", () => {
     // 'venmo' is matched by BOTH /venmo/gi (low p2p_payment)
-    // AND /v[e3][n][m][o0]/gi (high bypass_attempt) — so the rollup is high.
+    // AND /v[e3][n][m][o0]/gi (high bypass_attempt). So the rollup is high.
     // This documents that even a casual "we accept venmo" is treated as high.
     const result = filterContent("we accept venmo");
     expect(result.severity).toBe("high");
@@ -295,10 +295,10 @@ describe("filterContent — overall severity rollup", () => {
 });
 
 // ============================================================================
-// filterContent — cleanedContent masking
+// filterContent. CleanedContent masking
 // ============================================================================
 
-describe("filterContent — cleanedContent masking", () => {
+describe("filterContent. CleanedContent masking", () => {
   it("replaces every matched substring with the same number of asterisks", () => {
     const result = filterContent("Hit me on venmo");
     expect(result.cleanedContent).toBe("Hit me on *****");
@@ -317,7 +317,7 @@ describe("filterContent — cleanedContent masking", () => {
 });
 
 // ============================================================================
-// isContentClean — fast-path predicate
+// isContentClean. Fast-path predicate
 // ============================================================================
 
 describe("isContentClean", () => {
@@ -349,7 +349,7 @@ describe("isContentClean", () => {
 });
 
 // ============================================================================
-// filterMultipleFields — bulk validation
+// filterMultipleFields. Bulk validation
 // ============================================================================
 
 describe("filterMultipleFields", () => {
@@ -396,7 +396,7 @@ describe("filterMultipleFields", () => {
 });
 
 // ============================================================================
-// getViolationMessage — user-facing copy
+// getViolationMessage. User-facing copy
 // ============================================================================
 
 describe("getViolationMessage", () => {
@@ -458,10 +458,10 @@ describe("getViolationMessage", () => {
 });
 
 // ============================================================================
-// shouldWarnUser — tier-aware action mapping (the actual gate)
+// shouldWarnUser. Tier-aware action mapping (the actual gate)
 // ============================================================================
 
-describe("shouldWarnUser — tier-aware enforcement", () => {
+describe("shouldWarnUser. Tier-aware enforcement", () => {
   it("returns 'allow' with empty message when there are no violations", () => {
     expect(shouldWarnUser([], "Basic")).toEqual({
       action: "allow",

package/src/utils/__tests__/dateTimeUtils.test.ts

@@ -1,5 +1,5 @@
 /**
- * dateTimeUtils — parsing, comparison, and normalization helpers.
+ * dateTimeUtils. Parsing, comparison, and normalization helpers.
  * Pin UTC normalization and comparison semantics with fixed instants.
  */
 import { DateTime } from "luxon";

package/src/utils/__tests__/displayDateUtils.test.ts

@@ -1,5 +1,5 @@
 /**
- * displayDateUtils — UI date formatting (browser local timezone).
+ * displayDateUtils. UI date formatting (browser local timezone).
  *
  * These two helpers feed every "card date" / "long form date" string in the
  * app. Any regression here ripples across feed cards, ticket emails (preview),

package/src/utils/__tests__/flyerUtils.test.ts

@@ -149,7 +149,7 @@ describe("flyerUtils", () => {
     expect(distanceMiles(40.7, -112, 40.7, -112)).toBe(0);
   });
 
-  it("distanceMiles SLC to Provo is in plausible range (~35–50 mi)", () => {
+  it("distanceMiles SLC to Provo is in plausible range (~35 to 50 mi)", () => {
     const mi = distanceMiles(40.7608, -111.891, 40.2338, -111.6585);
     expect(mi).toBeGreaterThan(30);
     expect(mi).toBeLessThan(55);

package/src/utils/__tests__/getBackendHost.test.ts

@@ -1,5 +1,5 @@
 /**
- * getBackendHost — chooses the API origin (api.bash.community vs Vite dev same-origin
+ * getBackendHost. Chooses the API origin (api.bash.community vs Vite dev same-origin
  * vs the bundled API_HOST for native Capacitor builds).
  *
  * This file pins the contract so a future "let's tidy up env handling" change
@@ -42,7 +42,7 @@ afterEach(() => {
   clearWindow();
 });
 
-describe("getBackendHost — local dev (Vite NODE_ENV=development)", () => {
+describe("getBackendHost. Local dev (Vite NODE_ENV=development)", () => {
   let origNodeEnv: string | undefined;
 
   beforeAll(() => {
@@ -88,7 +88,7 @@ describe("getBackendHost — local dev (Vite NODE_ENV=development)", () => {
     expect(getBackendHost()).toBe("http://192.168.1.42:3000");
   });
 
-  it("matches page origin when Vite listens on PORT=3500 — set VITE_API_PROXY_TARGET so /api proxies to Express (not the same TCP port)", () => {
+  it("matches page origin when Vite listens on PORT=3500. Set VITE_API_PROXY_TARGET so /api proxies to Express (not the same TCP port)", () => {
     setWindow({
       protocol: "http:",
       hostname: "localhost",
@@ -99,7 +99,7 @@ describe("getBackendHost — local dev (Vite NODE_ENV=development)", () => {
   });
 });
 
-describe("getBackendHost — local non-dev (e.g. jest NODE_ENV=test)", () => {
+describe("getBackendHost. Local non-dev (e.g. jest NODE_ENV=test)", () => {
   it("falls back to canonical API port :3500 when not in Vite development bundle", () => {
     const orig = process.env.NODE_ENV;
     process.env.NODE_ENV = "test";
@@ -118,11 +118,11 @@ describe("getBackendHost — local non-dev (e.g. jest NODE_ENV=test)", () => {
   });
 });
 
-describe("getBackendHost — deployed web", () => {
+describe("getBackendHost. Deployed web", () => {
   // For deployed web, getBackendHost falls back to API_HOST (compiled in from
   // process.env.REACT_APP_API at bundle time; defaults to http://localhost:3500
   // when unset, which is what jest sees here). The contract we pin is "for any
-  // non-local hostname, do NOT derive the API URL from window.location" — that
+  // non-local hostname, do NOT derive the API URL from window.location". That
   // would silently route prod browsers at api.bash.community… instead of the
   // configured API_HOST.
   it("does not derive the backend host from window.location for deployed sites", () => {
@@ -149,7 +149,7 @@ describe("getBackendHost — deployed web", () => {
   });
 });
 
-describe("getBackendHost — Capacitor native (iOS / Android)", () => {
+describe("getBackendHost. Capacitor native (iOS / Android)", () => {
   it("uses the bundled API_HOST when running inside a Capacitor WebView (hostname is 'localhost' but it's not really local dev)", () => {
     // Capacitor loads from a localhost-like origin inside the WebView. If we
     // routed to the localhost dev API based on hostname alone, the app would
@@ -171,7 +171,7 @@ describe("getBackendHost — Capacitor native (iOS / Android)", () => {
   });
 });
 
-describe("getFrontendHost — Capacitor native (iOS / Android)", () => {
+describe("getFrontendHost. Capacitor native (iOS / Android)", () => {
   it("uses the public web host instead of the WebView localhost origin", () => {
     setWindow(
       {

package/src/utils/__tests__/getFrontendHost.native.test.ts

@@ -39,7 +39,7 @@ afterEach(() => {
   }
 });
 
-describe("getFrontendHost — native public URL contract", () => {
+describe("getFrontendHost. Native public URL contract", () => {
   it("returns bash.community for a Capacitor localhost WebView", async () => {
     setWindow(
       {

package/src/utils/__tests__/getFrontendHost.test.ts

@@ -1,5 +1,5 @@
 /**
- * getFrontendHost — the single source of truth for the URL prefix that goes
+ * getFrontendHost. The single source of truth for the URL prefix that goes
  * into every shareable Bash link (SharePopup, embed snippets, LinkedIn
  * Announce, Facebook event helper, QR codes…).
  *
@@ -7,7 +7,7 @@
  * ----------------------
  * Pre-fix, getFrontendHost() called isProduction() which inspects
  * process.env.ENVIRONMENT / BASH_ENV / NODE_ENV. Those env vars are server-
- * only — Vite's envPlugin only inlines REACT_APP_* / VITE_* / NODE_ENV /
+ * only. Vite's envPlugin only inlines REACT_APP_* / VITE_* / NODE_ENV /
  * PUBLIC_URL into the browser bundle. As a result, in the production browser
  * `process.env.ENVIRONMENT` is undefined → isProduction() returns false →
  * getFrontendHost() falls into the dev branch → appends ":3000" to whatever
@@ -60,7 +60,7 @@ afterEach(() => {
 // PRODUCTION (the regression we're fixing)
 // ============================================================================
 
-describe("getFrontendHost — production (the share-link bug)", () => {
+describe("getFrontendHost. Production (the share-link bug)", () => {
   it("returns 'https://bash.community' for the production domain (NO :3000)", () => {
     setLocation({
       protocol: "https:",
@@ -132,7 +132,7 @@ describe("getFrontendHost — production (the share-link bug)", () => {
   });
 
   it("preserves an EXPLICIT non-default port if the deployed site is running on one", () => {
-    // (uncommon but legal — e.g. preview deploy on :8080)
+    // (uncommon but legal. E.g. preview deploy on :8080)
     setLocation({
       protocol: "https:",
       hostname: "preview.bash.community",
@@ -147,7 +147,7 @@ describe("getFrontendHost — production (the share-link bug)", () => {
 // LOCAL DEV (must keep working)
 // ============================================================================
 
-describe("getFrontendHost — local development", () => {
+describe("getFrontendHost. Local development", () => {
   it("returns http://localhost:3000 for hostname=localhost", () => {
     setLocation({
       protocol: "http:",
@@ -180,10 +180,10 @@ describe("getFrontendHost — local development", () => {
 });
 
 // ============================================================================
-// NGROK / TUNNELS (deployed-equivalent — should NOT get :3000)
+// NGROK / TUNNELS (deployed-equivalent. Should NOT get :3000)
 // ============================================================================
 
-describe("getFrontendHost — ngrok / public tunnels", () => {
+describe("getFrontendHost. Ngrok / public tunnels", () => {
   it("does NOT append :3000 to an ngrok-free.dev tunnel hostname", () => {
     // ngrok forwards port 443 publicly to the local :3000; the link Facebook
     // crawls must hit the public 443, never the local 3000.
@@ -211,7 +211,7 @@ describe("getFrontendHost — ngrok / public tunnels", () => {
 // SHARE-URL ASSEMBLY (the exact concatenation SharePopup performs)
 // ============================================================================
 
-describe("getFrontendHost — composed share URL (SharePopup integration)", () => {
+describe("getFrontendHost. Composed share URL (SharePopup integration)", () => {
   it("composes the canonical share URL Facebook can crawl", () => {
     setLocation({
       protocol: "https:",
@@ -226,7 +226,7 @@ describe("getFrontendHost — composed share URL (SharePopup integration)", () =
     expect(fullShareUrl).toBe(
       "https://bash.community/bash/evt_123-empanada-fest-slc"
     );
-    // Hard guard — this is the literal regression we're preventing.
+    // Hard guard. This is the literal regression we're preventing.
     expect(fullShareUrl).not.toMatch(/bash\.community:3000/);
   });
 });

package/src/utils/__tests__/organizationEntitlements.test.ts

@@ -77,7 +77,7 @@ describe("organizationHasCustomDomainAccess", () => {
     expect(organizationHasCustomDomainAccess(undefined)).toBe(false);
   });
 
-  test("Growth is not entitled — custom domains stay Enterprise-only", () => {
+  test("Growth is not entitled. Custom domains stay Enterprise-only", () => {
     expect(
       organizationHasCustomDomainAccess({
         subscriptionTier: "Growth",

package/src/utils/__tests__/paymentUtils.test.ts

@@ -159,7 +159,7 @@ describe("PaymentUtils - Core Functions", () => {
       expect(r.performerCommissionCents).toBe(0);
     });
 
-    test("computeTicketApplicationFeeAmountCents — explicit isVerifiedNonprofitHost: false matches default", () => {
+    test("computeTicketApplicationFeeAmountCents. Explicit isVerifiedNonprofitHost: false matches default", () => {
       const baseArgs = {
         discountedSubtotalCents: 10000,
         platformFeeEnabled: true,
@@ -175,7 +175,7 @@ describe("PaymentUtils - Core Functions", () => {
       expect(explicitFalse).toEqual(defaulted);
     });
 
-    test("computeTicketApplicationFeeAmountCents — verified nonprofit host: zero Bash platform, guest may pay processing when GuestPays", () => {
+    test("computeTicketApplicationFeeAmountCents. Verified nonprofit host: zero Bash platform, guest may pay processing when GuestPays", () => {
       const r = computeTicketApplicationFeeAmountCents({
         discountedSubtotalCents: 10000,
         platformFeeEnabled: true,
@@ -198,7 +198,7 @@ describe("PaymentUtils - Core Functions", () => {
       expect(r.applicationFeeAmountCents).toBe(r.stripeProcessingCents);
     });
 
-    test("computeTicketApplicationFeeAmountCents — verified nonprofit waiver overrides platform on GuestPays", () => {
+    test("computeTicketApplicationFeeAmountCents. Verified nonprofit waiver overrides platform on GuestPays", () => {
       const nonNonprofit = computeTicketApplicationFeeAmountCents({
         discountedSubtotalCents: 5000,
         platformFeeEnabled: true,
@@ -236,7 +236,7 @@ describe("PaymentUtils - Core Functions", () => {
       );
     });
 
-    test("computeTicketApplicationFeeAmountCents — verified nonprofit waiver beats any membership-tier math", () => {
+    test("computeTicketApplicationFeeAmountCents. Verified nonprofit waiver beats any membership-tier math", () => {
       const tierCases = [
         { feeHandling: "HostAbsorbs" as const, bashPassWaivesGuestFee: false },
         { feeHandling: "Split" as const, bashPassWaivesGuestFee: false },
@@ -268,7 +268,7 @@ describe("PaymentUtils - Core Functions", () => {
       }
     });
 
-    test("computeTicketApplicationFeeAmountCents — verified nonprofit + platformFeeEnabled=false still zeroes the take", () => {
+    test("computeTicketApplicationFeeAmountCents. Verified nonprofit + platformFeeEnabled=false still zeroes the take", () => {
       const r = computeTicketApplicationFeeAmountCents({
         discountedSubtotalCents: 2500,
         platformFeeEnabled: false,
@@ -460,7 +460,7 @@ describe("PaymentUtils - Core Functions", () => {
   });
 
   describe("guest ticket checkout breakdown (No Platform Fees. Free for Everyone.)", () => {
-    test("HostAbsorbs — $20 ticket → $20 due, no processing line", () => {
+    test("HostAbsorbs. $20 ticket → $20 due, no processing line", () => {
       const r = computeGuestTicketCheckoutBreakdown({
         discountedSubtotalCents: 2000,
         platformFeeEnabled: false,
@@ -473,7 +473,7 @@ describe("PaymentUtils - Core Functions", () => {
       expect(r.totalDueCents).toBe(2000);
     });
 
-    test("GuestPays — $20 ticket → $20.88 all-in with processing line on subtotal", () => {
+    test("GuestPays. $20 ticket → $20.88 all-in with processing line on subtotal", () => {
       const processing = calculateStripeProcessingFeeCents(2000);
       expect(processing).toBe(88);
       const r = computeGuestTicketCheckoutBreakdown({

package/src/utils/__tests__/promoCodesUtils.test.ts

@@ -1,5 +1,5 @@
 /**
- * promoCodesUtils — locate / aggregate / re-distribute promo codes across a
+ * promoCodesUtils. Locate / aggregate / re-distribute promo codes across a
  * BashEvent's ticket tiers.
  *
  * These three helpers are tiny but they sit on the path between checkout
@@ -101,7 +101,7 @@ describe("findPromoCodeFromBashEvent", () => {
     expect(findPromoCodeFromBashEvent(bash, "p2")).toBe(promo);
   });
 
-  it("returns the FIRST match if the same id appears on multiple tiers (defensive — shouldn't happen but guard the order)", () => {
+  it("returns the FIRST match if the same id appears on multiple tiers (defensive. Shouldn't happen but guard the order)", () => {
     const dupOnT1 = makePromo("dup", "t1");
     const dupOnT2 = makePromo("dup", "t2");
     const bash = makeBashEvent([

package/src/utils/__tests__/qrCodeUtils.test.ts

@@ -1,5 +1,5 @@
 /**
- * qrCodeUtils — URL builders for the *navigable* `/ticket-details` deep
+ * qrCodeUtils. URL builders for the *navigable* `/ticket-details` deep
  * links that get encoded into ticket QR codes (and createQrCodeImage that
  * renders them as PNG data URLs).
  *
@@ -31,7 +31,7 @@ jest.mock("qrcode", () => ({
 }));
 
 // ============================================================================
-// bashEventPrismaCheckoutQrCodeUrl — checkout-id deep link
+// bashEventPrismaCheckoutQrCodeUrl. Checkout-id deep link
 // ============================================================================
 
 describe("bashEventPrismaCheckoutQrCodeUrl", () => {
@@ -54,7 +54,7 @@ describe("bashEventPrismaCheckoutQrCodeUrl", () => {
 });
 
 // ============================================================================
-// bashEventQrCodeUrl — base / tier / ticket / slug variants
+// bashEventQrCodeUrl. Base / tier / ticket / slug variants
 // ============================================================================
 
 describe("bashEventQrCodeUrl", () => {
@@ -122,7 +122,7 @@ describe("bashEventQrCodeUrl", () => {
 });
 
 // ============================================================================
-// createQrCodeImage — PNG data URL
+// createQrCodeImage. PNG data URL
 // ============================================================================
 
 describe("createQrCodeImage", () => {

package/src/utils/__tests__/reviewDimensionUtils.test.ts

@@ -7,7 +7,7 @@ import {
 
 describe("reviewDimensionUtils", () => {
   describe("parseRequiredOverallRating", () => {
-    it("accepts integers 1–5", () => {
+    it("accepts integers 1 to 5", () => {
       expect(parseRequiredOverallRating(4)).toBe(4);
     });
 

package/src/utils/__tests__/stripeAccountUtils.test.ts

@@ -1,5 +1,5 @@
 /**
- * stripeAccountUtils — the gate that decides whether a host can accept money.
+ * stripeAccountUtils. The gate that decides whether a host can accept money.
  *
  * `stripeAccountFullyLinked` returns `true` ONLY when ALL FOUR of these are true:
  *   - linkingStatus is non-null

package/src/utils/__tests__/ticketPrintPostageUtils.test.ts

@@ -59,7 +59,7 @@ describe("ticketPrintPostageUtils", () => {
           deliveryMethod: TicketDeliveryMethod.PhysicalMail,
           status: TicketPrintOrderStatus.Failed,
         })
-      ).toBe("Physical ticket delivery failed — contact the host");
+      ).toBe("Physical ticket delivery failed. Contact the host");
     });
 
     it.each([

package/src/utils/addressUtils.ts

@@ -244,7 +244,7 @@ export async function getAddressFromCoordinates( lat: number, lng: number ): Pro
         }
       }
 
-      // Place is intentionally empty here — only filled when user picks a Place in Maps UI or types it.
+      // Place is intentionally empty here. Only filled when user picks a Place in Maps UI or types it.
       return { place: "", street, city, state, zipCode, country: "USA" };
     } else {
       throw new Error("No address found");

package/src/utils/apiUtils.ts

@@ -61,7 +61,7 @@ export function getUserImage(
     return uploaded;
   }
 
-  // If only one image exists, return it (never return "" — that breaks <img src="">)
+  // If only one image exists, return it (never return "". That breaks <img src="">)
   return uploaded ?? googleImg ?? noProfilePicture;
 }
 

package/src/utils/birthdayExperienceQuality.ts

@@ -1,6 +1,6 @@
 /**
  * Shared Quality of Experience math for birthday freebie catalog offers.
- * Vote tiers map to star bands; the displayed score is a weighted average of band midpoints (0–5).
+ * Vote tiers map to star bands; the displayed score is a weighted average of band midpoints (0 to 5).
  */
 
 export type BirthdayExperienceVoteTier = "Awesome" | "Okay" | "Lame";
@@ -15,9 +15,9 @@ export const EXPERIENCE_STAR_BANDS: Record<
   BirthdayExperienceVoteTier,
   { min: number; max: number; label: string }
 > = {
-  Lame: { min: 0, max: 1.999, label: "0–2.0" },
-  Okay: { min: 2.0, max: 3.499, label: "2.0–3.5" },
-  Awesome: { min: 3.5, max: 5.0, label: "3.5–5.0" },
+  Lame: { min: 0, max: 1.999, label: "0 to 2.0" },
+  Okay: { min: 2.0, max: 3.499, label: "2.0 to 3.5" },
+  Awesome: { min: 3.5, max: 5.0, label: "3.5 to 5.0" },
 };
 
 const MIDPOINT: Record<BirthdayExperienceVoteTier, number> = {

package/src/utils/blogUtils.ts

@@ -22,7 +22,7 @@ export function getBlogDetailUrl(blogId: string, slug?: string, title?: string):
   }
 }
 
-/** Prisma default IDs are CUID / CUID2-like: start with `c`, 21–33 chars. */
+/** Prisma default IDs are CUID / CUID2-like: start with `c`, 21 to 33 chars. */
 export function looksLikePrismaCuid(id: string): boolean {
   return /^c[a-z0-9]{20,32}$/i.test(id);
 }
@@ -41,7 +41,7 @@ export function parseBlogUrlParams(blogIdSlug: string): { id: string; slug?: str
   const slug = blogIdSlug.substring(hyphenIndex + 1);
 
   // Sitemap and SPA links use `/blog/:slug`. Only split id-slug when the
-  // prefix is a Prisma CUID — otherwise "bash-community-vs-eventbrite"
+  // prefix is a Prisma CUID. Otherwise "bash-community-vs-eventbrite"
   // would be parsed as id "bash".
   if (looksLikePrismaCuid(id)) {
     return { id, slug };

package/src/utils/discountEngine/discountCalculator.ts

@@ -58,7 +58,7 @@ export const calculatePromoCodeDiscount = (
     amountDiscounted,
     originalPrice: originalTotal,
     finalPrice,
-    // Attribution = promoter's User id only (never Promoter row id — breaks dashboards).
+    // Attribution = promoter's User id only (never Promoter row id. Breaks dashboards).
     // Omit if promoter pending signup (no userId yet) or relation not loaded.
     ...(promoCode.promoterId &&
       promoCode.promoter?.userId && {