Fix linting errors in MCP Slack module

- Remove unused SearchMessagesDto import
- Remove unused processedCount variable
- Remove unused since parameter from destructuring
- Remove unused SearchMessagesOptions import
- Prefix unused accessToken parameter with underscore
- Apply linter formatting fixes to MCP core modules

Author: claude

backend/src/modules/mcp-slack/controllers/mcp-slack.controller.ts

@@ -13,7 +13,6 @@ import { McpSlackService } from '../services/mcp-slack.service';
 import {
   SyncChannelDto,
   GetMessagesDto,
-  SearchMessagesDto,
   ProcessAudioDto,
   SyncAudioDto,
 } from '../dto/slack-mcp.dto';
@@ -190,10 +189,7 @@ export class McpSlackController {
   @Get('audio/:channelId')
   @ApiOperation({ summary: 'Get audio recordings from a Slack channel' })
   @ApiResponse({ status: 200, description: 'Audio recordings retrieved' })
-  async getAudioRecordings(
-    @Param('channelId') channelId: string,
-    @Query('since') since?: string,
-  ) {
+  async getAudioRecordings(@Param('channelId') channelId: string, @Query('since') since?: string) {
     try {
       const sinceDate = since ? new Date(since) : undefined;
       const recordings = await this.mcpSlackService.getAudioRecordings(channelId, sinceDate);
@@ -252,14 +248,10 @@ export class McpSlackController {
   async syncAudio(@Body() dto: SyncAudioDto) {
     try {
       const sinceDate = dto.since ? new Date(dto.since) : undefined;
-      const recordings = await this.mcpSlackService.getAudioRecordings(
-        dto.channelId,
-        sinceDate,
-      );
+      const recordings = await this.mcpSlackService.getAudioRecordings(dto.channelId, sinceDate);
 
       // Process each recording (limit to dto.limit if specified)
       const toProcess = dto.limit ? recordings.slice(0, dto.limit) : recordings;
-      const processedCount = 0;
 
       for (const recording of toProcess) {
         try {

backend/src/modules/mcp-slack/processors/slack-audio.processor.ts

@@ -87,10 +87,8 @@ export class SlackAudioProcessor {
    * Process bulk audio sync
    */
   @Process('bulk-sync')
-  async handleBulkSync(
-    job: Job<{ channelId: string; fileIds: string[]; since?: Date }>,
-  ) {
-    const { channelId, fileIds, since } = job.data;
+  async handleBulkSync(job: Job<{ channelId: string; fileIds: string[]; since?: Date }>) {
+    const { channelId, fileIds } = job.data;
 
     this.logger.log(`Processing bulk audio sync for channel ${channelId}: ${fileIds.length} files`);
 

backend/src/modules/mcp-slack/services/mcp-slack.service.ts

@@ -12,7 +12,6 @@ import {
   SlackFile,
   SlackAudioRecording,
   GetChannelMessagesOptions,
-  SearchMessagesOptions,
   SlackSyncResult,
 } from '../interfaces/slack-mcp.interface';
 
@@ -210,7 +209,7 @@ export class McpSlackService implements OnModuleInit {
   /**
    * Download file
    */
-  async downloadFile(fileUrl: string, accessToken?: string): Promise<Buffer> {
+  async downloadFile(fileUrl: string, _accessToken?: string): Promise<Buffer> {
     this.logger.debug(`Downloading file from ${fileUrl}`);
 
     try {

backend/src/modules/mcp/exceptions/mcp.exceptions.ts

@@ -48,10 +48,7 @@ export class MCPToolException extends MCPException {
  */
 export class MCPToolNotFoundException extends MCPException {
   constructor(toolName: string, serverName: string) {
-    super(
-      `MCP tool '${toolName}' not found on server '${serverName}'`,
-      HttpStatus.NOT_FOUND,
-    );
+    super(`MCP tool '${toolName}' not found on server '${serverName}'`, HttpStatus.NOT_FOUND);
     this.name = 'MCPToolNotFoundException';
   }
 }

backend/src/modules/mcp/mcp.module.ts

@@ -10,15 +10,7 @@ import { MCPToolInvokerService } from './services/mcp-tool-invoker.service';
  */
 @Module({
   imports: [ConfigModule],
-  providers: [
-    MCPClientService,
-    MCPConnectionManagerService,
-    MCPToolInvokerService,
-  ],
-  exports: [
-    MCPClientService,
-    MCPConnectionManagerService,
-    MCPToolInvokerService,
-  ],
+  providers: [MCPClientService, MCPConnectionManagerService, MCPToolInvokerService],
+  exports: [MCPClientService, MCPConnectionManagerService, MCPToolInvokerService],
 })
 export class MCPModule {}

backend/src/modules/mcp/services/mcp-connection-manager.service.ts

@@ -1,11 +1,7 @@
 import { Injectable, Logger, OnModuleInit } from '@nestjs/common';
 import { ConfigService } from '@nestjs/config';
 import { MCPClientService } from './mcp-client.service';
-import {
-  getMCPServersConfig,
-  isMCPEnabled,
-  validateMCPServerConfig,
-} from '../config/mcp.config';
+import { getMCPServersConfig, isMCPEnabled, validateMCPServerConfig } from '../config/mcp.config';
 import { MCPServerConfig, MCPConnectionInfo } from '../interfaces/mcp.interface';
 import { MCPConfigurationException } from '../exceptions/mcp.exceptions';
 
@@ -68,9 +64,7 @@ export class MCPConnectionManagerService implements OnModuleInit {
       this.logger.log('MCP Connection Manager initialized successfully');
     } catch (error) {
       this.logger.error('Failed to initialize MCP Connection Manager', error);
-      throw new MCPConfigurationException(
-        error instanceof Error ? error.message : 'Unknown error',
-      );
+      throw new MCPConfigurationException(error instanceof Error ? error.message : 'Unknown error');
     }
   }
 
@@ -80,9 +74,7 @@ export class MCPConnectionManagerService implements OnModuleInit {
   async connectAll(): Promise<void> {
     this.logger.log(`Connecting to ${this.serverConfigs.length} MCP servers`);
 
-    const connectionPromises = this.serverConfigs.map((config) =>
-      this.connectWithRetry(config),
-    );
+    const connectionPromises = this.serverConfigs.map((config) => this.connectWithRetry(config));
 
     const results = await Promise.allSettled(connectionPromises);
 
@@ -112,9 +104,7 @@ export class MCPConnectionManagerService implements OnModuleInit {
         await this.mcpClient.connect(config);
         return; // Success
       } catch (error) {
-        this.logger.warn(
-          `Connection attempt ${attempt}/${maxAttempts} failed for ${config.name}`,
-        );
+        this.logger.warn(`Connection attempt ${attempt}/${maxAttempts} failed for ${config.name}`);
 
         if (attempt === maxAttempts) {
           throw error; // All attempts failed

backend/src/modules/mcp/services/mcp-tool-invoker.service.ts

@@ -87,9 +87,7 @@ export class MCPToolInvokerService {
 
       // Stop if any tool fails
       if (!result.success) {
-        this.logger.warn(
-          `Tool ${request.toolName} failed, stopping sequential execution`,
-        );
+        this.logger.warn(`Tool ${request.toolName} failed, stopping sequential execution`);
         break;
       }
     }
@@ -129,10 +127,7 @@ export class MCPToolInvokerService {
     if (schema.required) {
       for (const requiredParam of schema.required) {
         if (!(requiredParam in parameters)) {
-          throw new MCPToolException(
-            tool.name,
-            `Missing required parameter: ${requiredParam}`,
-          );
+          throw new MCPToolException(tool.name, `Missing required parameter: ${requiredParam}`);
         }
       }
     }
@@ -152,7 +147,10 @@ export class MCPToolInvokerService {
           const actualType = Array.isArray(paramValue) ? 'array' : typeof paramValue;
           const expectedType = paramSchema.type;
 
-          if (actualType !== expectedType && !(actualType === 'object' && expectedType === 'object')) {
+          if (
+            actualType !== expectedType &&
+            !(actualType === 'object' && expectedType === 'object')
+          ) {
             throw new MCPToolException(
               tool.name,
               `Invalid type for parameter '${paramName}': expected ${expectedType}, got ${actualType}`,