fix: logging

Author: gearonixx

src/commands/run.command.ts

@@ -1,7 +1,6 @@
 import { ensureFile } from '@neodx/fs'
 import { hasOwn, isObject } from '@neodx/std'
 import { Inject } from '@nestjs/common'
-import chalk from 'chalk'
 import { Command, CommandRunner, Option } from 'nest-commander'
 import { dirname, resolve } from 'node:path'
 import { LoggerService } from '@/logger'
@@ -33,16 +32,13 @@ export class RunCommand extends CommandRunner {
 
   public async run(params: string[], options: BaseInitOptions) {
     const [target, project = null] = params
-
-    invariant(target, 'Target is not provided.')
-
     const timeEnd = this.logger.time()
-
     let packageJsonPath = resolve(process.cwd(), 'package.json')
 
+    invariant(target, 'Target is not provided.')
+
     if (project) {
       const workspaces = await this.manager.getWorkspaces()
-
       const projectMeta = workspaces.find(
         (workspace) => workspace.name === project
       )
@@ -58,12 +54,11 @@ export class RunCommand extends CommandRunner {
     await ensureFile(packageJsonPath)
 
     const pkg = await readJson<PackageJson>(packageJsonPath)
-
     const projectName = project ?? pkg.name ?? 'root'
 
     invariant(
       isObject(pkg.scripts) && hasOwn(pkg.scripts, target),
-      `Could not find target ${chalk.cyan(target)} in project ${chalk.white(projectName)}.`
+      `Could not find target ${target} in project ${projectName}.`
     )
 
     const command = this.manager.createRunCommand({
@@ -86,6 +81,7 @@ export class RunCommand extends CommandRunner {
         `Error occurred while executing a command via ${this.manager.agent} agent.`
       )
       this.logger.error(error)
+      return
     }
 
     timeEnd(`Successfully ran target ${target} for project ${projectName}`)

src/logger/logger.service.ts

@@ -20,7 +20,7 @@ export class LoggerService {
 
   public error(msg: unknown) {
     if (isTypeOfString(msg)) {
-      console.error(`\n ${this.errorPrefix}  ${chalk.bold(chalk.red(msg))}\n`)
+      console.error(`\n ${this.errorPrefix} ${chalk.bold(chalk.red(msg))}`)
     } else if (msg instanceof Error && msg.stack) {
       console.error(chalk.bold(chalk.red(msg.stack)))
     } else {
@@ -30,19 +30,19 @@ export class LoggerService {
 
   public info(msg: unknown) {
     if (isTypeOfString(msg)) {
-      console.info(`\n${this.libraryPrefix}  ${chalk.bold(msg)}\n`)
+      console.info(`\n${this.libraryPrefix} ${chalk.bold(msg)}`)
     } else console.info(msg)
   }
 
-  public time(method = this.log) {
+  public time(method = this.info) {
     const startTime = Date.now()
 
     return (msg: string) => {
-      method(`${msg} (${Date.now() - startTime} ms)`)
+      method.call(this, `${msg} (${Date.now() - startTime} ms)`)
     }
   }
 
-  public serialize(msg: unknown, method = this.log) {
+  public serialize(msg: unknown, method = this.info) {
     method(serializeJson(msg))
   }
 

src/shared/misc.ts

@@ -6,7 +6,7 @@ export function invariant<T extends unknown>(
   message: string
 ): asserts condition is NonNullable<T> {
   if (!condition) {
-    console.error(`\n ${ERROR_PREFIX}  ${chalk.bold(chalk.red(message))}\n`)
+    console.error(`\n ${ERROR_PREFIX} ${chalk.bold(chalk.red(message))}`)
 
     process.exit(1)
   }