refactor(api): update Namespace interface to use entity objects

- Change NamespaceUpdate to accept *models.Namespace instead of
  tenant/changes parameters
- Change NamespaceDelete to accept *models.Namespace instead of tenantID
- Update service layer to resolve namespaces before update/delete
  operations
- Refactor EditNamespace to modify and persist full Namespace entity
- Refactor EditSessionRecordStatus to update Namespace entity directly
- Update mocks to use new NamespaceUpdate/NamespaceDelete signatures
- Update mongo store implementation with new NamespaceUpdate and
  NamespaceDelete behavior
- Update CLI service and tests to use entity-based operations
- Add tests for new EditSessionRecordStatus flow
- Remove models.NamespaceChanges usage in favor of direct entity updates

BREAKING CHANGE: NamespaceUpdate and NamespaceDelete method signatures
changed to accept *models.Namespace

Author: heiytor

api/services/namespace.go

@@ -132,7 +132,7 @@ func (s *service) GetNamespace(ctx context.Context, tenantID string) (*models.Na
 // When cloud and billing is enabled, it will try to delete the namespace's billing information from the billing
 // service if it exists.
 func (s *service) DeleteNamespace(ctx context.Context, tenantID string) error {
-	ns, err := s.store.NamespaceResolve(ctx, store.NamespaceTenantIDResolver, tenantID)
+	n, err := s.store.NamespaceResolve(ctx, store.NamespaceTenantIDResolver, tenantID)
 	if err != nil {
 		return NewErrNamespaceNotFound(tenantID, err)
 	}
@@ -141,29 +141,35 @@ func (s *service) DeleteNamespace(ctx context.Context, tenantID string) error {
 		return !ns.Billing.IsNil() && ns.Billing.HasCutomer() && ns.Billing.HasSubscription()
 	}
 
-	if envs.IsCloud() && ableToReportDeleteNamespace(ns) {
+	if envs.IsCloud() && ableToReportDeleteNamespace(n) {
 		if err := s.BillingReport(ctx, s.client, tenantID, ReportNamespaceDelete); err != nil {
 			return NewErrBillingReportNamespaceDelete(err)
 		}
 	}
 
-	return s.store.NamespaceDelete(ctx, tenantID)
+	return s.store.NamespaceDelete(ctx, n)
 }
 
 func (s *service) EditNamespace(ctx context.Context, req *requests.NamespaceEdit) (*models.Namespace, error) {
-	changes := &models.NamespaceChanges{
-		Name:                   strings.ToLower(req.Name),
-		SessionRecord:          req.Settings.SessionRecord,
-		ConnectionAnnouncement: req.Settings.ConnectionAnnouncement,
+	namespace, err := s.store.NamespaceResolve(ctx, store.NamespaceTenantIDResolver, req.Tenant)
+	if err != nil {
+		return nil, NewErrNamespaceNotFound(req.Tenant, err)
 	}
 
-	if err := s.store.NamespaceUpdate(ctx, req.Tenant, changes); err != nil {
-		switch {
-		case errors.Is(err, store.ErrNoDocuments):
-			return nil, NewErrNamespaceNotFound(req.Tenant, err)
-		default:
-			return nil, err
-		}
+	if req.Name != "" && !strings.EqualFold(req.Name, namespace.Name) {
+		namespace.Name = strings.ToLower(req.Name)
+	}
+
+	if req.Settings.SessionRecord != nil {
+		namespace.Settings.SessionRecord = *req.Settings.SessionRecord
+	}
+
+	if req.Settings.ConnectionAnnouncement != nil {
+		namespace.Settings.ConnectionAnnouncement = *req.Settings.ConnectionAnnouncement
+	}
+
+	if err := s.store.NamespaceUpdate(ctx, namespace); err != nil {
+		return nil, err
 	}
 
 	return s.store.NamespaceResolve(ctx, store.NamespaceTenantIDResolver, req.Tenant)
@@ -176,7 +182,8 @@ func (s *service) EditNamespace(ctx context.Context, req *requests.NamespaceEdit
 //
 // This method is deprecated, use [NamespaceService#EditNamespace] instead.
 func (s *service) EditSessionRecordStatus(ctx context.Context, sessionRecord bool, tenantID string) error {
-	if err := s.store.NamespaceUpdate(ctx, tenantID, &models.NamespaceChanges{SessionRecord: &sessionRecord}); err != nil {
+	n, err := s.store.NamespaceResolve(ctx, store.NamespaceTenantIDResolver, tenantID)
+	if err != nil {
 		switch {
 		case errors.Is(err, store.ErrNoDocuments):
 			return NewErrNamespaceNotFound(tenantID, err)
@@ -185,6 +192,11 @@ func (s *service) EditSessionRecordStatus(ctx context.Context, sessionRecord boo
 		}
 	}
 
+	n.Settings.SessionRecord = sessionRecord
+	if err := s.store.NamespaceUpdate(ctx, n); err != nil { // nolint:revive
+		return err
+	}
+
 	return nil
 }