From d15c79e946b3f9ef0f5cbf8eee5c2a81f308165c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Krzysztof=20=C5=BBuraw?=
 <9116238+krzysztofzuraw@users.noreply.github.com>
Date: Fri, 18 Oct 2024 12:23:27 +0200
Subject: [PATCH] Add cpsell to logger calls

---
 .cspell.json                                   |  8 +++++++-
 .../channel-provider-connection.router.ts      |  8 +++++---
 apps/klaviyo/src/lib/klaviyo.ts                |  2 +-
 apps/klaviyo/src/pages/api/configuration.ts    |  4 ++--
 .../app-configuration.router.ts                | 18 +++++++++---------
 .../category-mapping.router.ts                 | 13 +++++++------
 apps/search/src/pages/api/setup-indices.ts     | 15 ++++++++-------
 7 files changed, 39 insertions(+), 29 deletions(-)

diff --git a/.cspell.json b/.cspell.json
index 2636e6e062..e36d576ae1 100644
--- a/.cspell.json
+++ b/.cspell.json
@@ -11,6 +11,10 @@
     {
       "name": "comments",
       "pattern": ["comment-single-line", "comment-multi-line"]
+    },
+    {
+      "name": "logger",
+      "pattern": "/logger\\.(?:warn|error|info|debug)\\(.*\\)/g"
     }
   ],
   "languageSettings": [
@@ -20,7 +24,7 @@
     },
     {
       "languageId": ["javascriptreact", "typescriptreact", "typescript", "javascript"],
-      "includeRegExpList": ["comments"]
+      "includeRegExpList": ["comments", "logger"]
     }
   ],
   "words": [
@@ -41,6 +45,8 @@
     "taxjar",
     "unobfuscated",
     "upstash",
+    "upserted",
+    "upserting",
     "urql"
   ],
   "language": "en-US",
diff --git a/apps/cms-v2/src/modules/channel-provider-connection/channel-provider-connection.router.ts b/apps/cms-v2/src/modules/channel-provider-connection/channel-provider-connection.router.ts
index 6ed65114f8..8119b348a0 100644
--- a/apps/cms-v2/src/modules/channel-provider-connection/channel-provider-connection.router.ts
+++ b/apps/cms-v2/src/modules/channel-provider-connection/channel-provider-connection.router.ts
@@ -1,11 +1,13 @@
+import { TRPCError } from "@trpc/server";
+import { z } from "zod";
+
 import { AppConfigMetadataManager } from "@/modules/configuration/app-config-metadata-manager";
 import { createSettingsManager } from "@/modules/configuration/metadata-manager";
 import { ChannelProviderConnectionConfig } from "@/modules/configuration/schemas/channel-provider-connection.schema";
 import { protectedClientProcedure } from "@/modules/trpc/protected-client-procedure";
 import { router } from "@/modules/trpc/trpc-server";
-import { z } from "zod";
+
 import { FetchChannelsDocument } from "../../../generated/graphql";
-import { TRPCError } from "@trpc/server";
 import { createLogger } from "../../logger";
 
 const procedure = protectedClientProcedure.use(({ ctx, next }) => {
@@ -82,7 +84,7 @@ export const channelProviderConnectionRouter = router({
       } catch (e) {
         switch ((e as { cause: string }).cause) {
           case "PROVIDER_DOESNT_EXIST":
-            logger.warn("Provider doesnt exist");
+            logger.warn("Provider doesn't exist");
             throw new TRPCError({
               code: "BAD_REQUEST",
               cause: "PROVIDER_DOESNT_EXIST",
diff --git a/apps/klaviyo/src/lib/klaviyo.ts b/apps/klaviyo/src/lib/klaviyo.ts
index 09fce2764d..02e0fbafb6 100644
--- a/apps/klaviyo/src/lib/klaviyo.ts
+++ b/apps/klaviyo/src/lib/klaviyo.ts
@@ -20,7 +20,7 @@ export const Klaviyo = (token: string): EmailServiceProvider => ({
       }),
     );
 
-    logger.info("Seding Klaviyo request", { url: "https://a.klaviyo.com/api/track" });
+    logger.info("Sending Klaviyo request", { url: "https://a.klaviyo.com/api/track" });
 
     const response = await fetch("https://a.klaviyo.com/api/track", {
       method: "POST",
diff --git a/apps/klaviyo/src/pages/api/configuration.ts b/apps/klaviyo/src/pages/api/configuration.ts
index ce07a46859..67331eb836 100644
--- a/apps/klaviyo/src/pages/api/configuration.ts
+++ b/apps/klaviyo/src/pages/api/configuration.ts
@@ -1,10 +1,10 @@
 import { createProtectedHandler, NextProtectedApiHandler } from "@saleor/app-sdk/handlers/next";
 import { EncryptedMetadataManager } from "@saleor/app-sdk/settings-manager";
 import { wrapWithLoggerContext } from "@saleor/apps-logger/node";
-
 import { withOtel } from "@saleor/apps-otel";
 import { ObservabilityAttributes } from "@saleor/apps-otel/src/lib/observability-attributes";
 import { createGraphQLClient } from "@saleor/apps-shared";
+
 import { saleorApp } from "../../../saleor-app";
 import { createSettingsManager } from "../../lib/metadata";
 import { createLogger } from "../../logger";
@@ -65,7 +65,7 @@ const handler: NextProtectedApiHandler = async (request, res, ctx) => {
 
   switch (request.method!) {
     case "GET":
-      logger.info("Returing app configuration");
+      logger.info("Returning app configuration");
 
       return res.json({
         success: true,
diff --git a/apps/products-feed/src/modules/app-configuration/app-configuration.router.ts b/apps/products-feed/src/modules/app-configuration/app-configuration.router.ts
index 3f96792bfa..a5fee1441a 100644
--- a/apps/products-feed/src/modules/app-configuration/app-configuration.router.ts
+++ b/apps/products-feed/src/modules/app-configuration/app-configuration.router.ts
@@ -1,15 +1,15 @@
-import { router } from "../trpc/trpc-server";
-import { protectedClientProcedure } from "../trpc/protected-client-procedure";
-
-import { AppConfigSchema, imageSizeInputSchema, titleTemplateInputSchema } from "./app-config";
+import { TRPCError } from "@trpc/server";
 import { z } from "zod";
-import { createS3ClientFromConfiguration } from "../file-storage/s3/create-s3-client-from-configuration";
+
+import { createLogger } from "../../logger";
 import { checkBucketAccess } from "../file-storage/s3/check-bucket-access";
-import { TRPCError } from "@trpc/server";
-import { AttributeFetcher } from "./attribute-fetcher";
+import { createS3ClientFromConfiguration } from "../file-storage/s3/create-s3-client-from-configuration";
 import { renderHandlebarsTemplate } from "../handlebarsTemplates/render-handlebars-template";
+import { protectedClientProcedure } from "../trpc/protected-client-procedure";
+import { router } from "../trpc/trpc-server";
+import { AppConfigSchema, imageSizeInputSchema, titleTemplateInputSchema } from "./app-config";
+import { AttributeFetcher } from "./attribute-fetcher";
 import { prepareExampleVariantData } from "./prepare-example-variant-data";
-import { createLogger } from "../../logger";
 
 export const appConfigurationRouter = router({
   /**
@@ -152,7 +152,7 @@ export const appConfigurationRouter = router({
       });
     });
 
-    logger.debug("Returning attributes", { first: result[0], totaLength: result.length });
+    logger.debug("Returning attributes", { first: result[0], totalLength: result.length });
 
     return result;
   }),
diff --git a/apps/products-feed/src/modules/category-mapping/category-mapping.router.ts b/apps/products-feed/src/modules/category-mapping/category-mapping.router.ts
index 0657e9ab04..57358e5e9c 100644
--- a/apps/products-feed/src/modules/category-mapping/category-mapping.router.ts
+++ b/apps/products-feed/src/modules/category-mapping/category-mapping.router.ts
@@ -1,10 +1,11 @@
-import { router } from "../trpc/trpc-server";
-import { protectedClientProcedure } from "../trpc/protected-client-procedure";
-import { SetCategoryMappingInputSchema } from "./category-mapping-input-schema";
-import { UpdateCategoryMappingDocument } from "../../../generated/graphql";
 import { TRPCError } from "@trpc/server";
-import { CategoriesFetcher } from "./categories-fetcher";
+
+import { UpdateCategoryMappingDocument } from "../../../generated/graphql";
 import { createLogger } from "../../logger";
+import { protectedClientProcedure } from "../trpc/protected-client-procedure";
+import { router } from "../trpc/trpc-server";
+import { CategoriesFetcher } from "./categories-fetcher";
+import { SetCategoryMappingInputSchema } from "./category-mapping-input-schema";
 
 export const categoryMappingRouter = router({
   /**
@@ -23,7 +24,7 @@ export const categoryMappingRouter = router({
       });
     });
 
-    logger.debug("Returning categories", { first: result[0], totaLength: result.length });
+    logger.debug("Returning categories", { first: result[0], totalLength: result.length });
 
     return result;
   }),
diff --git a/apps/search/src/pages/api/setup-indices.ts b/apps/search/src/pages/api/setup-indices.ts
index ebe43d71cf..bc3c29a30d 100644
--- a/apps/search/src/pages/api/setup-indices.ts
+++ b/apps/search/src/pages/api/setup-indices.ts
@@ -1,16 +1,17 @@
 import { createProtectedHandler, NextProtectedApiHandler } from "@saleor/app-sdk/handlers/next";
-import { saleorApp } from "../../../saleor-app";
-import { createSettingsManager } from "../../lib/metadata";
-import { createLogger } from "../../lib/logger";
 import { SettingsManager } from "@saleor/app-sdk/settings-manager";
+import { wrapWithLoggerContext } from "@saleor/apps-logger/node";
+import { withOtel } from "@saleor/apps-otel";
 import { Client } from "urql";
+
 import { ChannelsDocument } from "../../../generated/graphql";
+import { saleorApp } from "../../../saleor-app";
 import { AlgoliaSearchProvider } from "../../lib/algolia/algoliaSearchProvider";
-import { AppConfigMetadataManager } from "../../modules/configuration/app-config-metadata-manager";
-import { withOtel } from "@saleor/apps-otel";
 import { createInstrumentedGraphqlClient } from "../../lib/create-instrumented-graphql-client";
+import { createLogger } from "../../lib/logger";
 import { loggerContext } from "../../lib/logger-context";
-import { wrapWithLoggerContext } from "@saleor/apps-logger/node";
+import { createSettingsManager } from "../../lib/metadata";
+import { AppConfigMetadataManager } from "../../modules/configuration/app-config-metadata-manager";
 
 const logger = createLogger("setupIndicesHandler");
 
@@ -69,7 +70,7 @@ export const setupIndicesHandlerFactory =
 
       return res.status(200).end();
     } catch (e) {
-      logger.error("Failed to update Algolia indicies", { error: e });
+      logger.error("Failed to update Algolia indices", { error: e });
 
       return res.status(500).end();
     }