feat: respect logger=false in environment variables (#838)

.changeset/rude-dodos-collect.md

@@ -0,0 +1,5 @@
+---
+"@bigcommerce/catalyst-core": patch
+---
+
+Respects when `CLIENT_LOGGER="false"` or `KV_LOGGER="false"` is set in .env.local regardless of environment.

apps/core/client/index.ts

@@ -8,5 +8,7 @@ export const client = createClient({
   storeHash: process.env.BIGCOMMERCE_STORE_HASH ?? '',
   channelId: process.env.BIGCOMMERCE_CHANNEL_ID,
   backendUserAgentExtensions: backendUserAgent,
-  logger: process.env.NODE_ENV !== 'production' || process.env.CLIENT_LOGGER === 'true',
+  logger:
+    (process.env.NODE_ENV !== 'production' && process.env.CLIENT_LOGGER !== 'false') ||
+    process.env.CLIENT_LOGGER === 'true',
 });

apps/core/lib/kv/index.ts

@@ -83,7 +83,9 @@ async function createKVAdapter() {
 }
 
 const adapterInstance = new KV(createKVAdapter, {
-  logger: process.env.NODE_ENV !== 'production' || process.env.KV_LOGGER === 'true',
+  logger:
+    (process.env.NODE_ENV !== 'production' && process.env.KV_LOGGER !== 'false') ||
+    process.env.KV_LOGGER === 'true',
 });
 
 export { adapterInstance as kv };