Fix logs that exceeds Vercel limit (#1648)

.changeset/fuzzy-pets-explode.md

@@ -0,0 +1,5 @@
+---
+"app-avatax": patch
+---
+
+Remove not needed information from AvaTax app logs to make them more readable and to be below Vercel log drain limit.

.changeset/new-hornets-build.md

@@ -0,0 +1,8 @@
+---
+"@saleor/webhook-utils": patch
+"@saleor/apps-logger": patch
+"app-avatax": patch
+"smtp": patch
+---
+
+Update modern-errors-serialize library so it supports excluding error property from serialization

apps/avatax/package.json

@@ -65,7 +65,7 @@
     "jotai": "^2.4.2",
     "jsdom": "^20.0.3",
     "modern-errors": "7.0.1",
-    "modern-errors-serialize": "6.0.0",
+    "modern-errors-serialize": "6.1.0",
     "neverthrow": "6.2.1",
     "next": "14.2.3",
     "react": "18.2.0",

apps/avatax/src/error.ts

@@ -9,6 +9,9 @@ export type CommonErrorProps = {
 export const BaseError = ModernError.subclass("BaseError", {
   props: {} as CommonErrorProps,
   plugins: [modernErrorsSerialize],
+  serialize: {
+    exclude: ["stack"],
+  },
 });
 
 // Critical errors are reported to Sentry.

apps/avatax/src/modules/avatax/extract-transaction-redacted-log-properties.ts

@@ -8,14 +8,7 @@ export const extractTransactionRedactedLogProperties = (model: CreateTransaction
   companyCode: model.companyCode,
   isAutocommit: model.commit,
   currencyCode: model.currencyCode,
-  lines: model.lines.map((line) => ({
-    amount: line.amount,
-    taxCode: line.taxCode,
-    quantity: line.quantity,
-    itemCode: line.itemCode,
-    description: line.description,
-    discounted: line.discounted,
-  })),
+  linesCount: model.lines.length,
   date: model.date.toISOString(),
   isTaxIncluded: model.lines[0]?.taxIncluded ?? false,
   discountAmount: model.discount,

apps/avatax/src/modules/calculate-taxes/use-case/calculate-taxes.use-case.ts

@@ -221,7 +221,7 @@ export class CalculateTaxesUseCase {
         });
       },
     ).map((results) => {
-      this.logger.info("Taxes calculated", { calculatedTaxes: results });
+      this.logger.info("Taxes calculated - returning response do Saleor");
 
       ClientLogStoreRequest.create({
         level: "info",

apps/avatax/src/pages/api/webhooks/checkout-calculate-taxes.ts

@@ -47,10 +47,6 @@ const handler = checkoutCalculateTaxesSyncWebhook.createHandler(async (req, res,
 
     subscriptionErrorChecker.checkPayload(payload);
 
-    logger.info("Tax base payload for checkout calculate taxes", {
-      payload: payload.taxBase,
-    });
-
     loggerContext.set(ObservabilityAttributes.CHANNEL_SLUG, ctx.payload.taxBase.channel.slug);
     loggerContext.set(ObservabilityAttributes.CHECKOUT_ID, ctx.payload.taxBase.sourceObject.id);
 

apps/avatax/src/pages/api/webhooks/order-calculate-taxes.ts

@@ -207,8 +207,7 @@ const handler = orderCalculateTaxesSyncWebhook.createHandler(async (req, res, ct
       discountStrategy,
     );
 
-    // eslint-disable-next-line @saleor/saleor-app/logger-leak
-    logger.info("Taxes calculated", { calculatedTaxes: JSON.stringify(calculatedTaxes) });
+    logger.info("Taxes calculated - returning response do Saleor");
 
     ClientLogStoreRequest.create({
       level: "info",

apps/smtp/package.json

@@ -55,7 +55,7 @@
     "jsdom": "^20.0.3",
     "mjml": "4.15.3",
     "modern-errors": "7.0.1",
-    "modern-errors-serialize": "6.0.0",
+    "modern-errors-serialize": "6.1.0",
     "neverthrow": "6.2.1",
     "next": "14.2.3",
     "nodemailer": "^6.9.1",

packages/logger/package.json

@@ -19,7 +19,7 @@
     "vite": "5.3.3",
     "vitest": "1.6.0",
     "modern-errors": "7.0.1",
-    "modern-errors-serialize": "6.0.0"
+    "modern-errors-serialize": "6.1.0"
   },
   "exports": {
     ".": {

packages/webhook-utils/package.json

@@ -16,7 +16,7 @@
     "graphql": "16.7.1",
     "graphql-tag": "2.12.6",
     "modern-errors": "7.0.1",
-    "modern-errors-serialize": "6.0.0",
+    "modern-errors-serialize": "6.1.0",
     "semver": "7.5.1",
     "urql": "4.0.4"
   },