feat(lint): add some logs to debug on prod

Author: angristan

src/lint/lint.module.ts

@@ -1,7 +1,9 @@
 import { Module } from '@nestjs/common';
+import { LoggerModule } from 'nestjs-pino';
 import { LintService } from './lint.service';
 
 @Module({
+  imports: [LoggerModule.forRoot()],
   providers: [LintService],
 })
 export class LintModule {}

src/lint/lint.service.ts

@@ -5,6 +5,7 @@
 import { Injectable, InternalServerErrorException } from '@nestjs/common';
 import child_process from 'child_process';
 import fs from 'fs';
+import { Logger } from 'nestjs-pino';
 import * as uuid from 'uuid';
 import YAML from 'yaml';
 import { LintErrorDTO } from './dto/lint-error.dto';
@@ -14,6 +15,8 @@ import { ConvertToPylintOutput } from './linters/pylint';
 
 @Injectable()
 export class LintService {
+  constructor(private readonly logger: Logger) {}
+
   lintPython3(code: string): LintResultDTO {
     const result = child_process.spawnSync(
       'pylint',
@@ -24,13 +27,14 @@ export class LintService {
     );
 
     if (result.error) {
-      throw new InternalServerErrorException();
+      throw new InternalServerErrorException('Failed to run pylint');
     }
 
     try {
       let output = result.output.toString();
       output = output.substring(1);
       output = output.substring(0, output.length - 1);
+      this.logger.log(output);
       const pylintOutput = ConvertToPylintOutput.toPylintOutput(output);
       if (pylintOutput) {
         /*
@@ -69,7 +73,7 @@ export class LintService {
         };
       }
     } catch (e) {
-      throw new InternalServerErrorException();
+      throw new InternalServerErrorException(e);
     }
 
     throw new InternalServerErrorException();
@@ -99,6 +103,7 @@ export class LintService {
       let output = result.output.toString();
       output = output.substring(1);
       output = output.substring(0, output.length - 1);
+      this.logger.log(output);
       const lintOuput =
         ConvertToGolangCILintOutput.toGolangCILintOutput(output);
       if (lintOuput) {
@@ -185,6 +190,7 @@ export class LintService {
       if (fs.existsSync(outputPath)) {
         const file = fs.readFileSync(`${path}.yaml`, 'utf8');
         const fixes: any = YAML.parse(file);
+        this.logger.log(fixes);
 
         if (fixes) {
           /*

src/submissions/submissions.service.ts

@@ -11,15 +11,15 @@ import { IsNull, Not, Repository } from 'typeorm';
 import { Benchmark } from '../benchmarks/benchmark.entity';
 import { BenchmarkService } from '../benchmarks/benchmark.service';
 import { BenchmarkIdDto } from '../benchmarks/dto/benchmarkId.dto';
+import { CyclomaticComplexity } from '../code-quality/complexity/CyclomaticComplexity';
+import { CodeTokenizer } from '../code-quality/tokenizer/CodeTokenizer';
 import { HashService } from '../hash/hash.service';
 import { User } from '../users/user.entity';
 import { FindLastSubmissionByLanguageDTO } from './dto/find-last-submission-by-language.dto';
 import { FindSubmissionDTO } from './dto/find-submission.dto';
 import { InsertSubmissionDTO } from './dto/insert-submission-dto';
 import { JobStatusDTO } from './dto/job-status.dto';
 import { Submission } from './submission.entity';
-import { CodeTokenizer } from '../code-quality/tokenizer/CodeTokenizer';
-import { CyclomaticComplexity } from '../code-quality/complexity/CyclomaticComplexity';
 
 @Injectable()
 export class SubmissionsService {