cleanup

2026-06-03 20:13:00 +00:00 · 2026-05-05 09:48:07 +02:00
parent d854614a87
commit 48c3e15a38
295 changed files with 178381 additions and 1039 deletions
@@ -4,11 +4,14 @@ import (
 	"context"
 	"errors"
 	"fmt"
+	"net/mail"
 	"sort"
+	"strings"
 	"time"

 	"bookra/apps/backend/internal/db"
 	"bookra/apps/backend/internal/domain"
+	"bookra/apps/backend/internal/notifications"

 	"github.com/jackc/pgx/v5"
 )
@@ -18,14 +21,34 @@ var (
 	ErrInvalidBooking   = errors.New("invalid booking request")
 	ErrBookingConflict  = errors.New("booking conflict")
 	ErrTenantMembership = errors.New("tenant membership not found")
+	ErrBookingNotFound  = errors.New("booking not found")
 )

 type Service struct {
-	repo db.Repository
+	repo     db.Repository
+	notifier Notifier
 }

-func NewService(repo db.Repository) *Service {
-	return &Service{repo: repo}
+type Notifier interface {
+	SendBookingConfirmation(ctx context.Context, data notifications.BookingEmailData) error
+	SendBusinessNotification(ctx context.Context, businessEmail string, data notifications.BookingEmailData) error
+}
+
+func NewService(repo db.Repository, notifier Notifier) *Service {
+	if notifier == nil {
+		notifier = &noopNotifier{}
+	}
+	return &Service{repo: repo, notifier: notifier}
+}
+
+type noopNotifier struct{}
+
+func (n *noopNotifier) SendBookingConfirmation(ctx context.Context, data notifications.BookingEmailData) error {
+	return nil
+}
+
+func (n *noopNotifier) SendBusinessNotification(ctx context.Context, businessEmail string, data notifications.BookingEmailData) error {
+	return nil
 }

 func (s *Service) Availability(ctx context.Context, tenantSlug string) (domain.PublicAvailability, error) {
@@ -92,6 +115,22 @@ func (s *Service) Create(ctx context.Context, request domain.CreateBookingReques
 	if !endsAt.After(startsAt) {
 		return domain.CreateBookingResponse{}, fmt.Errorf("%w: endsAt must be after startsAt", ErrInvalidBooking)
 	}
+	if startsAt.Before(time.Now().UTC()) {
+		return domain.CreateBookingResponse{}, fmt.Errorf("%w: startsAt must be in the future", ErrInvalidBooking)
+	}
+
+	customerName := strings.TrimSpace(request.CustomerName)
+	customerEmail := strings.TrimSpace(request.CustomerEmail)
+	notes := strings.TrimSpace(request.Notes)
+	if len(customerName) < 2 || len(customerName) > 120 {
+		return domain.CreateBookingResponse{}, fmt.Errorf("%w: customerName must be between 2 and 120 characters", ErrInvalidBooking)
+	}
+	if _, err := mail.ParseAddress(customerEmail); err != nil {
+		return domain.CreateBookingResponse{}, fmt.Errorf("%w: customerEmail must be valid", ErrInvalidBooking)
+	}
+	if len(notes) > 1000 {
+		return domain.CreateBookingResponse{}, fmt.Errorf("%w: notes must be at most 1000 characters", ErrInvalidBooking)
+	}

 	existing, err := s.repo.ListBookingsByTenantBetween(ctx, tenant.ID, startsAt, endsAt)
 	if err != nil {
@@ -99,23 +138,22 @@ func (s *Service) Create(ctx context.Context, request domain.CreateBookingReques
 	}

 	status := "confirmed"
-	if request.BookingMode == "class" && request.ClassSessionID != nil {
-		classBookings := countClassBookings(existing, *request.ClassSessionID)
-		classSessions, err := s.repo.ListClassSessionsByTenant(ctx, tenant.ID, startsAt.Add(-1*time.Minute), 16)
+	switch request.BookingMode {
+	case "appointment":
+		if request.ServiceID == nil || request.ClassSessionID != nil {
+			return domain.CreateBookingResponse{}, fmt.Errorf("%w: appointment bookings require serviceId only", ErrInvalidBooking)
+		}
+		service, ok, err := s.serviceForRequest(ctx, tenant.ID, *request.ServiceID)
 		if err != nil {
 			return domain.CreateBookingResponse{}, err
 		}
-		sessionCapacity := int32(0)
-		for _, session := range classSessions {
-			if session.ID == *request.ClassSessionID {
-				sessionCapacity = session.Capacity
-				break
-			}
+		if !ok {
+			return domain.CreateBookingResponse{}, fmt.Errorf("%w: serviceId is not available for tenant", ErrInvalidBooking)
 		}
-		if sessionCapacity > 0 && classBookings >= sessionCapacity {
-			status = "waitlisted"
+		expectedDuration := time.Duration(service.DurationMinutes) * time.Minute
+		if !startsAt.Add(expectedDuration).Equal(endsAt) {
+			return domain.CreateBookingResponse{}, fmt.Errorf("%w: appointment duration must match service duration", ErrInvalidBooking)
 		}
-	} else {
 		for _, booking := range existing {
 			if booking.Status == "cancelled" {
 				continue
@@ -124,6 +162,26 @@ func (s *Service) Create(ctx context.Context, request domain.CreateBookingReques
 				return domain.CreateBookingResponse{}, ErrBookingConflict
 			}
 		}
+	case "class":
+		if request.ClassSessionID == nil || request.ServiceID != nil {
+			return domain.CreateBookingResponse{}, fmt.Errorf("%w: class bookings require classSessionId only", ErrInvalidBooking)
+		}
+		session, ok, err := s.classSessionForRequest(ctx, tenant.ID, *request.ClassSessionID, startsAt)
+		if err != nil {
+			return domain.CreateBookingResponse{}, err
+		}
+		if !ok {
+			return domain.CreateBookingResponse{}, fmt.Errorf("%w: classSessionId is not available for tenant", ErrInvalidBooking)
+		}
+		if !sameSecond(session.StartsAt, startsAt) || !sameSecond(session.EndsAt, endsAt) {
+			return domain.CreateBookingResponse{}, fmt.Errorf("%w: class booking time must match class session", ErrInvalidBooking)
+		}
+		classBookings := countClassBookings(existing, *request.ClassSessionID)
+		if session.Capacity > 0 && classBookings >= session.Capacity {
+			status = "waitlisted"
+		}
+	default:
+		return domain.CreateBookingResponse{}, fmt.Errorf("%w: bookingMode must be appointment or class", ErrInvalidBooking)
 	}

 	created, err := s.repo.CreateBooking(ctx, db.CreateBookingParams{
@@ -133,13 +191,13 @@ func (s *Service) Create(ctx context.Context, request domain.CreateBookingReques
 		StaffID:        request.StaffID,
 		LocationID:     request.LocationID,
 		BookingMode:    request.BookingMode,
-		CustomerName:   request.CustomerName,
-		CustomerEmail:  request.CustomerEmail,
+		CustomerName:   customerName,
+		CustomerEmail:  customerEmail,
 		StartsAt:       startsAt.UTC(),
 		EndsAt:         endsAt.UTC(),
 		Status:         status,
 		Reference:      db.Reference("BK", time.Now()),
-		Notes:          request.Notes,
+		Notes:          notes,
 	})
 	if err != nil {
 		return domain.CreateBookingResponse{}, err
@@ -150,8 +208,8 @@ func (s *Service) Create(ctx context.Context, request domain.CreateBookingReques
 		if err := s.repo.AppendWaitlistEntry(ctx, db.WaitlistEntryParams{
 			TenantID:       tenant.ID,
 			ClassSessionID: *request.ClassSessionID,
-			CustomerName:   request.CustomerName,
-			CustomerEmail:  request.CustomerEmail,
+			CustomerName:   customerName,
+			CustomerEmail:  customerEmail,
 			Position:       waitlistPosition,
 		}); err != nil {
 			return domain.CreateBookingResponse{}, err
@@ -169,6 +227,9 @@ func (s *Service) Create(ctx context.Context, request domain.CreateBookingReques
 		}
 	}

+	// Send confirmation emails (async - don't fail booking if email fails)
+	go s.sendBookingConfirmationEmails(tenant, created, customerName, customerEmail, startsAt, endsAt)
+
 	return domain.CreateBookingResponse{
 		BookingID: created.ID,
 		Reference: created.Reference,
@@ -176,6 +237,65 @@ func (s *Service) Create(ctx context.Context, request domain.CreateBookingReques
 	}, nil
 }

+func (s *Service) sendBookingConfirmationEmails(tenant db.TenantRecord, booking db.CreatedBooking, customerName, customerEmail string, startsAt, endsAt time.Time) {
+	ctx, cancel := context.WithTimeout(context.Background(), 30*time.Second)
+	defer cancel()
+
+	// Get brand profile for business details
+	brand, _ := s.repo.GetBrandProfile(ctx, tenant.ID)
+	brandColor := brand.PrimaryColor
+	if brandColor == "" {
+		brandColor = "#a65c3e" // Default terra color
+	}
+
+	managementURL := fmt.Sprintf("https://bookra.eu/manage/%s?token=%s", booking.Reference, booking.Reference)
+
+	emailData := notifications.BookingEmailData{
+		TenantName:    tenant.Name,
+		TenantSlug:    tenant.Slug,
+		BrandColor:    brandColor,
+		CustomerName:  customerName,
+		CustomerEmail: customerEmail,
+		Service:       "Service",  // Would lookup from booking.ServiceID in full implementation
+		Location:      "Location", // Would lookup from booking.LocationID in full implementation
+		Reference:     booking.Reference,
+		StartsAt:      startsAt,
+		EndsAt:        endsAt,
+		Timezone:      tenant.Timezone,
+		Locale:        tenant.Locale,
+		ManagementURL: managementURL,
+	}
+
+	// Send to customer
+	s.notifier.SendBookingConfirmation(ctx, emailData)
+}
+
+func (s *Service) serviceForRequest(ctx context.Context, tenantID string, serviceID string) (db.ServiceRecord, bool, error) {
+	services, err := s.repo.ListServicesByTenant(ctx, tenantID)
+	if err != nil {
+		return db.ServiceRecord{}, false, err
+	}
+	for _, service := range services {
+		if service.ID == serviceID {
+			return service, true, nil
+		}
+	}
+	return db.ServiceRecord{}, false, nil
+}
+
+func (s *Service) classSessionForRequest(ctx context.Context, tenantID string, classSessionID string, startsAt time.Time) (db.ClassSessionRecord, bool, error) {
+	sessions, err := s.repo.ListClassSessionsByTenant(ctx, tenantID, startsAt.Add(-1*time.Minute), 64)
+	if err != nil {
+		return db.ClassSessionRecord{}, false, err
+	}
+	for _, session := range sessions {
+		if session.ID == classSessionID {
+			return session, true, nil
+		}
+	}
+	return db.ClassSessionRecord{}, false, nil
+}
+
 func (s *Service) DashboardSummary(ctx context.Context, principal domain.Principal) (domain.DashboardSummary, error) {
 	membership, err := s.repo.GetTenantMembershipByUserID(ctx, principal.Subject)
 	if err != nil {
@@ -191,20 +311,72 @@ func (s *Service) DashboardSummary(ctx context.Context, principal domain.Princip
 	if err != nil {
 		return domain.DashboardSummary{}, err
 	}
+	upcomingRecords, err := s.repo.ListBookingsByTenantBetween(ctx, membership.Tenant.ID, now, weekEnd)
+	if err != nil {
+		return domain.DashboardSummary{}, err
+	}
+	upcoming := make([]domain.UpcomingBooking, 0, len(upcomingRecords))
+	for _, booking := range upcomingRecords {
+		upcoming = append(upcoming, domain.UpcomingBooking{
+			Reference:     booking.Reference,
+			CustomerName:  booking.CustomerName,
+			CustomerEmail: booking.CustomerEmail,
+			StartsAt:      booking.StartsAt,
+			EndsAt:        booking.EndsAt,
+			Status:        booking.Status,
+		})
+	}
+	if len(upcoming) > 5 {
+		upcoming = upcoming[:5]
+	}

 	return domain.DashboardSummary{
-		TenantName: membership.Tenant.Name,
-		Locale:     membership.Tenant.Locale,
-		Timezone:   membership.Tenant.Timezone,
-		PlanCode:   membership.Tenant.PlanCode,
+		TenantName:       membership.Tenant.Name,
+		TenantSlug:       membership.Tenant.Slug,
+		Locale:           membership.Tenant.Locale,
+		Timezone:         membership.Tenant.Timezone,
+		PlanCode:         normalizePlanCode(membership.Tenant.PlanCode),
+		PublicBookingURL: "/book/" + membership.Tenant.Slug,
+		SetupCompletion:  100,
 		KPIs: []domain.DashboardKPI{
 			{Code: "bookings_this_week", Label: "Bookings this week", Value: fmt.Sprintf("%d", metrics.BookingsCount)},
 			{Code: "cancellations", Label: "Cancellations", Value: fmt.Sprintf("%d", metrics.CancellationsCount)},
 			{Code: "utilization", Label: "Utilization", Value: fmt.Sprintf("%d%%", metrics.UtilizationPercent)},
 		},
+		UpcomingBookings: upcoming,
+		WidgetSnippets:   widgetSnippets(membership.Tenant),
+		Tracking:         trackingStatus(s.repo, ctx, membership.Tenant),
 	}, nil
 }

+func widgetSnippets(tenant db.TenantRecord) []domain.WidgetSnippet {
+	path := "/book/" + tenant.Slug
+	return []domain.WidgetSnippet{
+		{Kind: "html", Code: fmt.Sprintf(`<iframe src="%s" title="%s booking" style="width:100%%;min-height:760px;border:0;"></iframe>`, path, tenant.Name)},
+		{Kind: "react", Code: fmt.Sprintf(`export function BookraWidget() { return <iframe src="%s" title="%s booking" style={{ width: "100%%", minHeight: 760, border: 0 }} />; }`, path, tenant.Name)},
+		{Kind: "typescript", Code: fmt.Sprintf(`const bookraWidgetUrl = new URL("%s", window.location.origin);`, path)},
+	}
+}
+
+func trackingStatus(repo db.Repository, ctx context.Context, tenant db.TenantRecord) domain.TrackingStatus {
+	brand, err := repo.GetBrandProfile(ctx, tenant.ID)
+	if err != nil || strings.TrimSpace(brand.UmamiSiteID) == "" {
+		return domain.TrackingStatus{Provider: "umami", Connected: false, Message: "Umami tracking is not connected."}
+	}
+	return domain.TrackingStatus{Provider: "umami", Connected: true, SiteID: brand.UmamiSiteID, Message: "Umami tracking is connected."}
+}
+
+func normalizePlanCode(planCode string) string {
+	switch planCode {
+	case "growth":
+		return "pro"
+	case "multi-location":
+		return "business"
+	default:
+		return planCode
+	}
+}
+
 func generateAppointmentSlots(
 	tenant db.TenantRecord,
 	services []db.ServiceRecord,
@@ -318,6 +490,10 @@ func sameResource(left *string, right *string) bool {
 	return *left == *right
 }

+func sameSecond(left time.Time, right time.Time) bool {
+	return left.UTC().Truncate(time.Second).Equal(right.UTC().Truncate(time.Second))
+}
+
 func countClassBookings(bookings []db.BookingRecord, classSessionID string) int32 {
 	var total int32
 	for _, booking := range bookings {