diff --git a/src/services/claudeService.js b/src/services/claudeService.js index 01af369..09ca991 100644 --- a/src/services/claudeService.js +++ b/src/services/claudeService.js @@ -96,16 +96,16 @@ For real functionality, please configure valid GitHub and Claude API tokens.`; // Select appropriate entrypoint script based on operation type let entrypointScript; switch (operationType) { - case 'auto-tagging': - entrypointScript = '/scripts/runtime/claudecode-tagging-entrypoint.sh'; - logger.info({ operationType }, 'Using minimal tools for auto-tagging operation'); - break; - case 'pr-review': - case 'default': - default: - entrypointScript = '/scripts/runtime/claudecode-entrypoint.sh'; - logger.info({ operationType }, 'Using full tool set for standard operation'); - break; + case 'auto-tagging': + entrypointScript = '/scripts/runtime/claudecode-tagging-entrypoint.sh'; + logger.info({ operationType }, 'Using minimal tools for auto-tagging operation'); + break; + case 'pr-review': + case 'default': + default: + entrypointScript = '/scripts/runtime/claudecode-entrypoint.sh'; + logger.info({ operationType }, 'Using full tool set for standard operation'); + break; } // Create unique container name (sanitized to prevent command injection) diff --git a/src/utils/logger.js b/src/utils/logger.js index 137a3ee..cd9f8be 100644 --- a/src/utils/logger.js +++ b/src/utils/logger.js @@ -19,33 +19,33 @@ const logFileName = path.join(logsDir, 'app.log'); // Configure different transports based on environment const transport = isProduction ? { - targets: [ - // File transport for production - { - target: 'pino/file', - options: { destination: logFileName, mkdir: true } + targets: [ + // File transport for production + { + target: 'pino/file', + options: { destination: logFileName, mkdir: true } + }, + // Console pretty transport + { + target: 'pino-pretty', + options: { + colorize: true, + levelFirst: true, + translateTime: 'SYS:standard' }, - // Console pretty transport - { - target: 'pino-pretty', - options: { - colorize: true, - levelFirst: true, - translateTime: 'SYS:standard' - }, - level: 'info' - } - ] - } - : { - // Just use pretty logs in development - target: 'pino-pretty', - options: { - colorize: true, - levelFirst: true, - translateTime: 'SYS:standard' + level: 'info' } - }; + ] + } + : { + // Just use pretty logs in development + target: 'pino-pretty', + options: { + colorize: true, + levelFirst: true, + translateTime: 'SYS:standard' + } + }; // Configure the logger const logger = pino({ diff --git a/test/unit/services/claudeService.test.js b/test/unit/services/claudeService.test.js index dc7f663..8c18523 100644 --- a/test/unit/services/claudeService.test.js +++ b/test/unit/services/claudeService.test.js @@ -73,7 +73,7 @@ describe('Claude Service', () => { }); // Verify test mode response - expect(result).toContain("Hello! I'm Claude responding to your request."); + expect(result).toContain('Hello! I\'m Claude responding to your request.'); expect(result).toContain('test/repo'); expect(sanitizeBotMentions).toHaveBeenCalled(); diff --git a/test/unit/utils/awsCredentialProvider.test.js b/test/unit/utils/awsCredentialProvider.test.js index 852d4b5..d760c47 100644 --- a/test/unit/utils/awsCredentialProvider.test.js +++ b/test/unit/utils/awsCredentialProvider.test.js @@ -138,7 +138,7 @@ region = us-west-2 process.env.AWS_PROFILE = 'non-existent-profile'; await expect(awsCredentialProvider.getCredentials()).rejects.toThrow( - "Profile 'non-existent-profile' not found" + 'Profile \'non-existent-profile\' not found' ); // Restore AWS_PROFILE @@ -156,7 +156,7 @@ aws_access_key_id = test-access-key fs.promises.readFile.mockImplementationOnce(() => Promise.resolve(mockConfigFile)); await expect(awsCredentialProvider.getCredentials()).rejects.toThrow( - "Incomplete credentials for profile 'test-profile'" + 'Incomplete credentials for profile \'test-profile\'' ); }); });