@unchainedshop/api 4.0.0-rc.15 → 4.0.0-rc.17

package/src/resolvers/mutations/assortments/removeAssortment.ts

@@ -11,10 +11,8 @@ export default async function removeAssortment(
 
   if (!assortmentId) throw new InvalidIdError({ assortmentId });
 
-  const assortment = await modules.assortments.findAssortment({ assortmentId });
-  if (!assortment) throw new AssortmentNotFoundError({ assortmentId });
+  const deletedAssortment = await modules.assortments.delete(assortmentId);
+  if (!deletedAssortment) throw new AssortmentNotFoundError({ assortmentId });
 
-  await modules.assortments.delete(assortmentId);
-
-  return assortment;
+  return deletedAssortment;
 }

package/src/resolvers/mutations/delivery/createDeliveryProvider.ts

@@ -6,7 +6,7 @@ import { DeliveryDirector } from '@unchainedshop/core';
 
 export default async function createDeliveryProvider(
   root: never,
-  { deliveryProvider }: { deliveryProvider: DeliveryProvider },
+  { deliveryProvider }: { deliveryProvider: Pick<DeliveryProvider, 'type' | 'adapterKey'> },
   { modules, userId }: Context,
 ) {
   log('mutation createDeliveryProvider', { userId });

package/src/resolvers/mutations/enrollments/createEnrollment.ts

@@ -45,7 +45,7 @@ export default async function createEnrollment(
     payment,
     productId,
     quantity,
-    userId,
+    userId: userId!,
   });
 
   return await services.enrollments.initializeEnrollment(enrollment, {

package/src/resolvers/mutations/enrollments/updateEnrollment.ts

@@ -38,23 +38,26 @@ export default async function updateEnrollment(
   }
 
   if (meta) {
-    enrollment = await modules.enrollments.updateContext(enrollmentId, meta);
+    enrollment = (await modules.enrollments.updateContext(enrollmentId, meta)) as Enrollment;
   }
 
   if (billingAddress) {
-    enrollment = await modules.enrollments.updateBillingAddress(enrollmentId, billingAddress);
+    enrollment = (await modules.enrollments.updateBillingAddress(
+      enrollmentId,
+      billingAddress,
+    )) as Enrollment;
   }
 
   if (contact) {
-    enrollment = await modules.enrollments.updateContact(enrollmentId, contact);
+    enrollment = (await modules.enrollments.updateContact(enrollmentId, contact)) as Enrollment;
   }
 
   if (payment) {
-    enrollment = await modules.enrollments.updatePayment(enrollmentId, payment);
+    enrollment = (await modules.enrollments.updatePayment(enrollmentId, payment)) as Enrollment;
   }
 
   if (delivery) {
-    enrollment = await modules.enrollments.updateDelivery(enrollmentId, delivery);
+    enrollment = (await modules.enrollments.updateDelivery(enrollmentId, delivery)) as Enrollment;
   }
 
   if (plan) {
@@ -64,7 +67,7 @@ export default async function updateEnrollment(
         'TODO: Unchained currently does not support order splitting for enrollments, therefore updates to quantity, product and configuration of a enrollment is forbidden for non initial enrollments',
       );
     }
-    enrollment = await modules.enrollments.updatePlan(enrollmentId, plan);
+    enrollment = (await modules.enrollments.updatePlan(enrollmentId, plan)) as Enrollment;
     enrollment = await services.enrollments.initializeEnrollment(enrollment, { reason: 'updated_plan' });
   }
 

package/src/resolvers/mutations/filters/createFilterOption.ts

@@ -18,7 +18,7 @@ export default async function createFilterOption(
   if (!(await modules.filters.filterExists({ filterId }))) throw new FilterNotFoundError({ filterId });
 
   const filter = await modules.filters.createFilterOption(filterId, { value: option });
-  await FilterDirector.invalidateProductIdCache(filter, context);
+  await FilterDirector.invalidateProductIdCache(filter!, context);
 
   if (texts) {
     await modules.filters.texts.updateTexts({ filterId, filterOptionValue: option }, texts);

package/src/resolvers/mutations/filters/removeFilterOption.ts

@@ -19,7 +19,7 @@ export default async function removeFilterOption(
     filterId,
     filterOptionValue,
   });
-  await FilterDirector.invalidateProductIdCache(filter, context);
+  await FilterDirector.invalidateProductIdCache(filter!, context);
 
   return filter;
 }

package/src/resolvers/mutations/filters/updateFilter.ts

@@ -18,7 +18,7 @@ export default async function updateFilter(
   if (!(await modules.filters.filterExists({ filterId }))) throw new FilterNotFoundError({ filterId });
 
   const updatedFilter = await modules.filters.update(filterId, filter);
-  await FilterDirector.invalidateProductIdCache(updatedFilter, context);
+  await FilterDirector.invalidateProductIdCache(updatedFilter!, context);
 
   return updatedFilter;
 }

package/src/resolvers/mutations/orders/addCartDiscount.ts

@@ -18,7 +18,7 @@ export default async function addCartDiscount(
 
   const order = await services.orders.findOrInitCart({
     orderId,
-    user,
+    user: user!,
     countryCode: context.countryCode,
   });
   if (!order) throw new OrderNotFoundError({ orderId });

package/src/resolvers/mutations/orders/addCartProduct.ts

@@ -31,7 +31,7 @@ export default async function addCartProduct(
 
   const order = await services.orders.findOrInitCart({
     orderId,
-    user,
+    user: user!,
     countryCode: context.countryCode,
   });
   if (!order) throw new OrderNotFoundError({ orderId });

package/src/resolvers/mutations/orders/addCartQuotation.ts

@@ -9,6 +9,7 @@ import {
   OrderWrongStatusError,
   ProductNotFoundError,
   OrderNotFoundError,
+  QuotationItemConfigurationError,
 } from '../../../errors.js';
 import { QuotationDirector } from '@unchainedshop/core';
 
@@ -45,7 +46,7 @@ export default async function addCartQuotation(
 
   const order = await services.orders.findOrInitCart({
     orderId,
-    user,
+    user: user!,
     countryCode: context.countryCode,
   });
   if (!order) throw new OrderNotFoundError({ orderId });
@@ -64,8 +65,10 @@ export default async function addCartQuotation(
     configuration,
   });
 
+  if (!quotationConfiguration) throw new QuotationItemConfigurationError({ configuration });
+
   const updatedOrderPosition = await modules.orders.positions.addProductItem({
-    quantity: quotationConfiguration.quantity,
+    quantity: quotationConfiguration.quantity || 1,
     configuration: quotationConfiguration.configuration,
     quotationId,
     productId: quotation.productId,

package/src/resolvers/mutations/orders/addMultipleCartProducts.ts

@@ -6,7 +6,7 @@ import {
   OrderWrongStatusError,
   OrderNotFoundError,
 } from '../../../errors.js';
-import { ordersSettings } from '@unchainedshop/core-orders';
+import { OrderPosition, ordersSettings } from '@unchainedshop/core-orders';
 
 export default async function addMultipleCartProducts(
   root: never,
@@ -43,7 +43,7 @@ export default async function addMultipleCartProducts(
 
   const order = await services.orders.findOrInitCart({
     orderId,
-    user,
+    user: user!,
     countryCode: context.countryCode,
   });
   if (!order) throw new OrderNotFoundError({ orderId });
@@ -52,7 +52,7 @@ export default async function addMultipleCartProducts(
   // Reduce is used to wait for each product to be added before processing the next (sequential processing)
   await itemsWithProducts.reduce(
     async (positionsPromise, { originalProduct, quantity, configuration }) => {
-      const positions = await positionsPromise;
+      const positions: OrderPosition[] = await positionsPromise;
       if (quantity < 1)
         throw new OrderQuantityTooLowError({
           quantity,

package/src/resolvers/mutations/orders/checkoutCart.ts

@@ -21,7 +21,7 @@ export default async function checkoutCart(
   // Do not check for order status here! The checkout method will act accordingly
   const order = await services.orders.findOrInitCart({
     orderId: forceOrderId,
-    user,
+    user: user!,
     countryCode: context.countryCode,
   });
   if (!order) throw new OrderNotFoundError({ orderId: forceOrderId });
@@ -30,7 +30,7 @@ export default async function checkoutCart(
     const checkedOutOrder = await services.orders.checkoutOrder(order._id, transactionContext);
     return checkedOutOrder;
   } catch (error) {
-    logger.error(error.message, { userId, orderId: order._id });
+    logger.error(error, { userId, orderId: order._id });
     throw new OrderCheckoutError({
       userId,
       orderId: order._id,

package/src/resolvers/mutations/orders/createCart.ts

@@ -14,7 +14,7 @@ export default async function createCart(
   if (order) throw new OrderNumberAlreadyExistsError({ orderNumber });
 
   return services.orders.nextUserCart({
-    user,
+    user: user!,
     orderNumber,
     countryCode: context.countryCode,
     forceCartCreation: true,

package/src/resolvers/mutations/orders/deliverOrder.ts

@@ -6,6 +6,7 @@ import {
   OrderWrongDeliveryStatusError,
   OrderWrongStatusError,
   InvalidIdError,
+  OrderDeliveryNotFoundError,
 } from '../../../errors.js';
 
 export default async function deliverOrder(
@@ -25,9 +26,13 @@ export default async function deliverOrder(
     throw new OrderWrongStatusError({ status: order.status });
   }
 
-  const orderDelivery = await modules.orders.deliveries.findDelivery({
-    orderDeliveryId: order.deliveryId,
-  });
+  const orderDelivery =
+    order.deliveryId &&
+    (await modules.orders.deliveries.findDelivery({
+      orderDeliveryId: order.deliveryId,
+    }));
+
+  if (!orderDelivery) throw new OrderDeliveryNotFoundError({ orderDeliveryId: order.deliveryId });
 
   if (
     modules.orders.deliveries.normalizedStatus(orderDelivery) !== OrderDeliveryStatus.OPEN &&

package/src/resolvers/mutations/orders/emptyCart.ts

@@ -13,7 +13,7 @@ export default async function emptyCart(
 
   const order = await services.orders.findOrInitCart({
     orderId,
-    user,
+    user: user!,
     countryCode: context.countryCode,
   });
   if (!order) throw new OrderNotFoundError({ orderId });

package/src/resolvers/mutations/orders/payOrder.ts

@@ -6,6 +6,7 @@ import {
   OrderWrongPaymentStatusError,
   OrderWrongStatusError,
   InvalidIdError,
+  OrderPaymentNotFoundError,
 } from '../../../errors.js';
 
 export default async function payOrder(root: never, { orderId }: { orderId: string }, context: Context) {
@@ -22,9 +23,13 @@ export default async function payOrder(root: never, { orderId }: { orderId: stri
     throw new OrderWrongStatusError({ status: order.status });
   }
 
-  const payment = await modules.orders.payments.findOrderPayment({
-    orderPaymentId: order.paymentId,
-  });
+  const payment =
+    order.paymentId &&
+    (await modules.orders.payments.findOrderPayment({
+      orderPaymentId: order.paymentId,
+    }));
+
+  if (!payment) throw new OrderPaymentNotFoundError({ orderPaymentId: order.paymentId });
 
   if (modules.orders.payments.normalizedStatus(payment) !== OrderPaymentStatus.OPEN) {
     throw new OrderWrongPaymentStatusError({

package/src/resolvers/mutations/orders/removeCartDiscount.ts

@@ -1,7 +1,12 @@
 import { log } from '@unchainedshop/logger';
 import { Context } from '../../../context.js';
 
-import { OrderDiscountNotFoundError, OrderWrongStatusError, InvalidIdError } from '../../../errors.js';
+import {
+  OrderDiscountNotFoundError,
+  OrderWrongStatusError,
+  InvalidIdError,
+  OrderNotFoundError,
+} from '../../../errors.js';
 import { OrderDiscountTrigger } from '@unchainedshop/core-orders';
 import { OrderDiscountDirector } from '@unchainedshop/core';
 
@@ -24,8 +29,11 @@ export default async function removeCartDiscount(
   const order = await modules.orders.findOrder({
     orderId: orderDiscount.orderId,
   });
+
+  if (!order) throw new OrderNotFoundError({ orderId: orderDiscount.orderId });
+
   if (!modules.orders.isCart(order)) {
-    throw new OrderWrongStatusError({ status: order.status });
+    throw new OrderWrongStatusError({ status: order!.status });
   }
 
   if (orderDiscount.trigger === OrderDiscountTrigger.USER) {

package/src/resolvers/mutations/orders/removeCartItem.ts

@@ -1,6 +1,11 @@
 import { log } from '@unchainedshop/logger';
 import { Context } from '../../../context.js';
-import { OrderItemNotFoundError, OrderWrongStatusError, InvalidIdError } from '../../../errors.js';
+import {
+  OrderItemNotFoundError,
+  OrderWrongStatusError,
+  InvalidIdError,
+  OrderNotFoundError,
+} from '../../../errors.js';
 
 export default async function removeCartItem(
   root: never,
@@ -20,6 +25,8 @@ export default async function removeCartItem(
 
   const order = await modules.orders.findOrder({ orderId: orderItem.orderId });
 
+  if (!order) throw new OrderNotFoundError({ orderId: orderItem.orderId });
+
   if (!modules.orders.isCart(order)) {
     throw new OrderWrongStatusError({ status: order.status });
   }

package/src/resolvers/mutations/orders/signPaymentProviderForCheckout.ts

@@ -7,6 +7,7 @@ import {
   OrderPaymentNotFoundError,
   OrderPaymentTypeError,
   OrderWrongPaymentStatusError,
+  PaymentProviderNotFoundError,
   UserNoCartError,
 } from '../../../errors.js';
 import { PaymentDirector } from '@unchainedshop/core';
@@ -33,18 +34,23 @@ export default async function signPaymentProviderForCheckout(
   } else {
     const order = await modules.orders.cart({
       countryCode: context.countryCode,
-      userId,
+      userId: userId!,
     });
     if (!order) throw new UserNoCartError({ userId });
-    orderPayment = await modules.orders.payments.findOrderPayment({
-      orderPaymentId: order.paymentId,
-    });
+    orderPayment =
+      order.paymentId &&
+      (await modules.orders.payments.findOrderPayment({
+        orderPaymentId: order.paymentId,
+      }));
     if (!orderPayment) throw new OrderPaymentNotFoundError({ orderPaymentId });
   }
 
   const provider = await modules.payment.paymentProviders.findProvider({
     paymentProviderId: orderPayment.paymentProviderId,
   });
+
+  if (!provider) throw new PaymentProviderNotFoundError({ id: orderPayment.paymentProviderId });
+
   const providerType = provider?.type;
 
   if (providerType !== PaymentProviderType.GENERIC)

package/src/resolvers/mutations/orders/updateCart.ts

@@ -21,7 +21,7 @@ export default async function updateCart(root: never, params: UpdateCartParams,
 
   const order = await services.orders.findOrInitCart({
     orderId,
-    user,
+    user: user!,
     countryCode: context.countryCode,
   });
   if (!order) throw new OrderNotFoundError({ orderId });

package/src/resolvers/mutations/orders/updateCartDeliveryPickUp.ts

@@ -3,6 +3,7 @@ import { DeliveryProviderType } from '@unchainedshop/core-delivery';
 import { log } from '@unchainedshop/logger';
 import {
   InvalidIdError,
+  OrderDeliveryNotFoundError,
   OrderDeliveryTypeError,
   OrderNotFoundError,
   OrderWrongStatusError,
@@ -23,7 +24,7 @@ export default async function updateCartDeliveryPickUp(
 
   let order = await services.orders.findOrInitCart({
     orderId,
-    user,
+    user: user!,
     countryCode: context.countryCode,
   });
   if (!order) throw new OrderNotFoundError({ orderId });
@@ -43,6 +44,8 @@ export default async function updateCartDeliveryPickUp(
 
   order = (await modules.orders.setDeliveryProvider(order._id, deliveryProviderId)) || order;
 
+  if (!order.deliveryId) throw new OrderDeliveryNotFoundError({ orderId: order._id });
+
   await modules.orders.deliveries.updateContext(order.deliveryId, {
     orderPickUpLocationId,
     meta,

package/src/resolvers/mutations/orders/updateCartDeliveryShipping.ts

@@ -3,6 +3,7 @@ import { DeliveryProviderType } from '@unchainedshop/core-delivery';
 import { log } from '@unchainedshop/logger';
 import {
   InvalidIdError,
+  OrderDeliveryNotFoundError,
   OrderDeliveryTypeError,
   OrderNotFoundError,
   OrderWrongStatusError,
@@ -23,7 +24,7 @@ export default async function updateCartDeliveryShipping(
 
   let order = await services.orders.findOrInitCart({
     orderId,
-    user,
+    user: user!,
     countryCode: context.countryCode,
   });
   if (!order) throw new OrderNotFoundError({ orderId });
@@ -43,6 +44,8 @@ export default async function updateCartDeliveryShipping(
 
   order = (await modules.orders.setDeliveryProvider(order._id, deliveryProviderId)) || order;
 
+  if (!order.deliveryId) throw new OrderDeliveryNotFoundError({ orderId: order._id });
+
   await modules.orders.deliveries.updateContext(order.deliveryId, {
     address,
     meta,

package/src/resolvers/mutations/orders/updateCartItem.ts

@@ -6,6 +6,7 @@ import {
   OrderWrongStatusError,
   ProductNotFoundError,
   InvalidIdError,
+  OrderNotFoundError,
 } from '../../../errors.js';
 import { ordersSettings } from '@unchainedshop/core-orders';
 
@@ -19,7 +20,7 @@ export default async function updateCartItem(
   context: Context,
 ) {
   const { modules, services, userId } = context;
-  const { itemId, configuration = null, quantity = null } = params;
+  const { itemId, configuration, quantity = 1 } = params;
 
   log(`mutation updateCartItem ${itemId} ${quantity} ${JSON.stringify(configuration)}`, { userId });
 
@@ -29,6 +30,9 @@ export default async function updateCartItem(
   if (!item) throw new OrderItemNotFoundError({ itemId });
 
   const order = await modules.orders.findOrder({ orderId: item.orderId });
+
+  if (!order) throw new OrderNotFoundError({ orderId: item.orderId });
+
   if (!modules.orders.isCart(order)) {
     throw new OrderWrongStatusError({ status: order.status });
   }
@@ -50,11 +54,12 @@ export default async function updateCartItem(
     context,
   );
 
-  const updatedOrderPosition = await modules.orders.positions.updateProductItem({
+  await modules.orders.positions.updateProductItem({
     orderPositionId: item._id,
     quantity,
     configuration,
   });
+
   await services.orders.updateCalculation(order._id);
-  return modules.orders.positions.findOrderPosition({ itemId: updatedOrderPosition._id });
+  return modules.orders.positions.findOrderPosition({ itemId: item._id });
 }

package/src/resolvers/mutations/orders/updateCartPaymentGeneric.ts

@@ -4,6 +4,7 @@ import { log } from '@unchainedshop/logger';
 import {
   InvalidIdError,
   OrderNotFoundError,
+  OrderPaymentNotFoundError,
   OrderPaymentTypeError,
   OrderWrongStatusError,
 } from '../../../errors.js';
@@ -23,7 +24,7 @@ export default async function updateCartPaymentGeneric(
 
   let order = await services.orders.findOrInitCart({
     orderId,
-    user,
+    user: user!,
     countryCode: context.countryCode,
   });
   if (!order) throw new OrderNotFoundError({ orderId });
@@ -43,6 +44,8 @@ export default async function updateCartPaymentGeneric(
 
   order = (await modules.orders.setPaymentProvider(order._id, paymentProviderId)) || order;
 
+  if (!order.paymentId) throw new OrderPaymentNotFoundError({ orderId: order._id });
+
   await modules.orders.payments.updateContext(order.paymentId, {
     meta,
   });

package/src/resolvers/mutations/orders/updateCartPaymentInvoice.ts

@@ -4,6 +4,7 @@ import { log } from '@unchainedshop/logger';
 import {
   InvalidIdError,
   OrderNotFoundError,
+  OrderPaymentNotFoundError,
   OrderPaymentTypeError,
   OrderWrongStatusError,
 } from '../../../errors.js';
@@ -23,7 +24,7 @@ export default async function updateCartPaymentInvoice(
 
   let order = await services.orders.findOrInitCart({
     orderId,
-    user,
+    user: user!,
     countryCode: context.countryCode,
   });
   if (!order) throw new OrderNotFoundError({ orderId });
@@ -43,6 +44,8 @@ export default async function updateCartPaymentInvoice(
 
   order = (await modules.orders.setPaymentProvider(order._id, paymentProviderId)) || order;
 
+  if (!order.paymentId) throw new OrderPaymentNotFoundError({ orderId: order._id });
+
   await modules.orders.payments.updateContext(order.paymentId, {
     meta,
   });

package/src/resolvers/mutations/payment/createPaymentProvider.ts

@@ -6,7 +6,7 @@ import { PaymentDirector } from '@unchainedshop/core';
 
 export default async (
   root: never,
-  { paymentProvider }: { paymentProvider: PaymentProvider },
+  { paymentProvider }: { paymentProvider: Pick<PaymentProvider, 'type' | 'adapterKey'> },
   { modules, userId }: Context,
 ) => {
   log('mutation createPaymentProvider', { userId });

package/src/resolvers/mutations/payment/markPaymentCredentialsPreferred.ts

@@ -22,7 +22,7 @@ export default async function makePaymentCredentialsPreferred(
 
   await modules.payment.paymentCredentials.markPreferred({
     paymentCredentialsId,
-    userId,
+    userId: userId!,
   });
 
   return modules.payment.paymentCredentials.findPaymentCredential({

package/src/resolvers/mutations/products/addProductAssignment.ts

@@ -1,5 +1,5 @@
 import { log } from '@unchainedshop/logger';
-import { ProductTypes } from '@unchainedshop/core-products';
+import { ProductTypes, ProductVariation } from '@unchainedshop/core-products';
 
 import { Context } from '../../../context.js';
 import { ProductConfiguration } from '@unchainedshop/core-products';
@@ -11,7 +11,7 @@ import {
   ProductVariationVectorAlreadySet,
   ProductVariationInfinityLoop,
 } from '../../../errors.js';
-const extractVariationMatrix = (variations = []) => {
+const extractVariationMatrix = (variations: ProductVariation[] = []) => {
   const cartesianProduct = (arrays) => {
     return arrays.reduce(
       (acc, array) => acc.flatMap((item) => array.map((value) => [...item, value])),

package/src/resolvers/mutations/products/addProductReviewVote.ts

@@ -18,5 +18,5 @@ export default async function addProductReviewVote(
   });
   if (!productReview) throw new ProductReviewNotFoundError({ productReviewId });
 
-  return modules.products.reviews.votes.addVote(productReview, { meta, type, userId });
+  return modules.products.reviews.votes.addVote(productReview, { meta, type, userId: userId! });
 }

package/src/resolvers/mutations/products/createProductReview.ts

@@ -6,7 +6,10 @@ import { InvalidIdError, ProductNotFoundError } from '../../../errors.js';
 
 export default async function createProductReview(
   root: never,
-  { productId, productReview }: { productId: string; productReview: ProductReview },
+  {
+    productId,
+    productReview,
+  }: { productId: string; productReview: Pick<ProductReview, 'title' | 'rating' | 'meta'> },
   { modules, userId }: Context,
 ) {
   log('mutation createProductReview', { userId, productId });
@@ -17,8 +20,8 @@ export default async function createProductReview(
     throw new ProductNotFoundError({ productId });
 
   return modules.products.reviews.create({
-    productId,
-    authorId: userId,
     ...productReview,
+    productId,
+    authorId: userId!,
   });
 }

package/src/resolvers/mutations/quotations/requestQuotation.ts

@@ -20,7 +20,7 @@ export default async function requestQuotation(
     throw new ProductNotFoundError({ productId });
 
   const newQuotation = await modules.quotations.create({
-    userId,
+    userId: userId!,
     productId,
     countryCode,
     currencyCode,

package/src/resolvers/mutations/users/addPushSubscription.ts

@@ -10,12 +10,12 @@ const addPushSubscription = async (
   const userAgent = context.getHeader('user-agent');
   log(`mutation addPushSubscription ${userAgent}`, { userId });
 
-  await modules.users.addPushSubscription(userId, subscription, {
+  await modules.users.addPushSubscription(userId!, subscription, {
     userAgent,
     unsubscribeFromOtherUsers,
   });
 
-  return modules.users.findUserById(userId);
+  return modules.users.findUserById(userId!);
 };
 
 export default addPushSubscription;

package/src/resolvers/mutations/users/removePushSubscription.ts

@@ -7,9 +7,9 @@ const removePushSubscription = async (_, { p256dh }, context: Context) => {
 
   log(`mutation removePushSubscription ${userId} ${userAgent}`, { userId });
 
-  await modules.users.removePushSubscription(userId, p256dh);
+  await modules.users.removePushSubscription(userId!, p256dh);
 
-  return modules.users.findUserById(userId);
+  return modules.users.findUserById(userId!);
 };
 
 export default removePushSubscription;

package/src/resolvers/mutations/users/removeUser.ts

@@ -13,11 +13,11 @@ export default async function removeUser(
 
   log(`mutation removeUser ${normalizedUserId}`, { userId: currentUserId });
 
-  if (!(await modules.users.userExists({ userId: normalizedUserId })))
+  if (!(await modules.users.userExists({ userId: normalizedUserId! })))
     throw new UserNotFoundError({ userId: normalizedUserId });
 
   if (removeUserReviews) {
-    await modules.products.reviews.deleteMany({ authorId: normalizedUserId });
+    await modules.products.reviews.deleteMany({ authorId: normalizedUserId! });
   }
-  return services.users.deleteUser({ userId: normalizedUserId });
+  return services.users.deleteUser({ userId: normalizedUserId! });
 }

package/src/resolvers/mutations/users/updateUserProfile.ts

@@ -13,8 +13,8 @@ export default async function updateUserProfile(
 
   log(`mutation updateUserProfile ${normalizedUserId}`, { userId });
 
-  if (!(await modules.users.userExists({ userId: normalizedUserId })))
+  if (!(await modules.users.userExists({ userId: normalizedUserId! })))
     throw UserNotFoundError({ userId: normalizedUserId });
 
-  return modules.users.updateProfile(normalizedUserId, profile);
+  return modules.users.updateProfile(normalizedUserId!, profile);
 }