refactor: reduce logging severity (#35)

callstack · Dec 16, 2024 · d6b3e3a · d6b3e3a
1 parent 122dfd2
commit d6b3e3a
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 7 deletions.
diff --git a/.changeset/lemon-coats-fry.md b/.changeset/lemon-coats-fry.md
@@ -0,0 +1,6 @@
+---
+'@callstack/byorg-utils': patch
+'@callstack/byorg-core': patch
+---
+
+core: reduce some of the startup logs severity to debug
diff --git a/examples/discord/src/index.ts b/examples/discord/src/index.ts
@@ -34,6 +34,6 @@ void (async () => {
     await discord.login(DISCORD_BOT_TOKEN);
     logger.info('Discord ready.');
   } catch (error) {
-    logger.error('Dev Slack start error:', error);
+    logger.error('Dev Discord start error:', error);
   }
 })();
diff --git a/packages/core/src/application.ts b/packages/core/src/application.ts
@@ -56,7 +56,7 @@ export function createApp(config: ApplicationConfig): Application {
   const { plugins = [], chatModel, errorHandler = defaultErrorHandler } = config;
 
   plugins.forEach((plugin) => {
-    logger.info(`Plugin "${plugin.name}" registered`);
+    logger.debug(`Plugin "${plugin.name}" registered`);
   });
 
   const middlewares: Middleware[] = [
@@ -68,7 +68,7 @@ export function createApp(config: ApplicationConfig): Application {
 
   // TODO: Validate
   tools.forEach((tool) => {
-    logger.info(`Tool "${tool.name}" registered`);
+    logger.debug(`Tool "${tool.name}" registered`);
   });
 
   const middlewareExecutor = new MiddlewareHandler();

diff --git a/packages/utils/src/env.ts b/packages/utils/src/env.ts
@@ -1,10 +1,7 @@
-import * as logger from './logger/logger.js';
-
 export function requireEnv(key: string) {
   const value = process.env[key];
   if (!value) {
-    logger.error(`Missing environment variable: ${key}`);
-    process.exit(1);
+    throw new Error(`Missing environment variable: ${key}`);
   }
 
   return value;