refactor: simplify code by removing unused features

- Focus LANGUAGE_EXTENSIONS on web dev languages only
- Remove MVC pattern detection
- Replace spinner with simple logging
- Remove unused console utilities

close #33

Author: wishket-pjw

mcp-server/src/analyzer/code.sampler.ts

@@ -5,78 +5,31 @@ import { pathContainsSegment } from '../shared/path.utils';
 
 /**
  * Language detection by file extension
+ * Focused on web development languages commonly analyzed by this tool
  */
 export const LANGUAGE_EXTENSIONS: Record<string, string> = {
   // TypeScript
   '.ts': 'typescript',
   '.tsx': 'typescript',
-  '.mts': 'typescript',
-  '.cts': 'typescript',
 
   // JavaScript
   '.js': 'javascript',
   '.jsx': 'javascript',
-  '.mjs': 'javascript',
-  '.cjs': 'javascript',
 
   // Python
   '.py': 'python',
-  '.pyw': 'python',
 
-  // Java/Kotlin
+  // Java
   '.java': 'java',
-  '.kt': 'kotlin',
-  '.kts': 'kotlin',
-
-  // Go
-  '.go': 'go',
-
-  // Rust
-  '.rs': 'rust',
-
-  // C/C++
-  '.c': 'c',
-  '.h': 'c',
-  '.cpp': 'cpp',
-  '.hpp': 'cpp',
-  '.cc': 'cpp',
-
-  // C#
-  '.cs': 'csharp',
-
-  // Ruby
-  '.rb': 'ruby',
-
-  // PHP
-  '.php': 'php',
-
-  // Swift
-  '.swift': 'swift',
 
   // Styles
   '.css': 'css',
   '.scss': 'scss',
-  '.sass': 'sass',
-  '.less': 'less',
 
   // Data/Config
   '.json': 'json',
   '.yaml': 'yaml',
   '.yml': 'yaml',
-  '.toml': 'toml',
-  '.xml': 'xml',
-
-  // Shell
-  '.sh': 'shell',
-  '.bash': 'shell',
-  '.zsh': 'shell',
-
-  // SQL
-  '.sql': 'sql',
-
-  // GraphQL
-  '.graphql': 'graphql',
-  '.gql': 'graphql',
 
   // Framework-specific
   '.vue': 'vue',
@@ -85,25 +38,15 @@ export const LANGUAGE_EXTENSIONS: Record<string, string> = {
 
 /**
  * Extensions considered as code (for sampling)
+ * Matches LANGUAGE_EXTENSIONS for consistency
  */
 const CODE_EXTENSIONS = new Set([
   '.ts',
   '.tsx',
   '.js',
   '.jsx',
-  '.mjs',
-  '.cjs',
   '.py',
   '.java',
-  '.kt',
-  '.go',
-  '.rs',
-  '.c',
-  '.cpp',
-  '.cs',
-  '.rb',
-  '.php',
-  '.swift',
   '.css',
   '.scss',
   '.vue',

mcp-server/src/analyzer/directory.analyzer.ts

@@ -50,11 +50,6 @@ export const ARCHITECTURE_PATTERNS: PatternDefinition[] = [
     indicators: ['packages', 'apps', 'libs'],
     minIndicators: 1,
   },
-  {
-    name: 'MVC',
-    indicators: ['src/models', 'src/views', 'src/controllers'],
-    minIndicators: 2,
-  },
   {
     name: 'Clean Architecture',
     indicators: ['src/domain', 'src/application', 'src/infrastructure', 'src/presentation'],

mcp-server/src/cli/init/init.command.ts

@@ -62,12 +62,12 @@ export async function runInit(options: InitOptions): Promise<InitResult> {
     }
 
     // Step 1: Analyze project
-    console.spinner.start('Analyzing project...');
+    console.log.step('🔍', 'Analyzing project...');
 
     const analyzer = new AnalyzerService();
     const analysis = await analyzer.analyzeProject(options.projectRoot);
 
-    console.spinner.succeed('Project analysis complete');
+    console.log.success('Project analysis complete');
 
     // Log analysis summary
     if (analysis.packageInfo) {
@@ -79,21 +79,21 @@ export async function runInit(options: InitOptions): Promise<InitResult> {
     console.log.step('📁', `Files: ${analysis.directoryStructure.totalFiles}`);
 
     // Step 2: Generate config with AI
-    console.spinner.start('AI is generating configuration...');
+    console.log.step('🤖', 'AI is generating configuration...');
 
     const generator = new ConfigGenerator({ apiKey });
     const config = await generator.generate(analysis);
 
-    console.spinner.succeed('Configuration generated');
+    console.log.success('Configuration generated');
 
     // Step 3: Write config file
-    console.spinner.start('Writing configuration file...');
+    console.log.step('💾', 'Writing configuration file...');
 
     const configPath = await writeConfig(options.projectRoot, config, {
       format: options.format,
     });
 
-    console.spinner.succeed(`Configuration saved to ${configPath}`);
+    console.log.success(`Configuration saved to ${configPath}`);
 
     // Success message
     console.log.success('');
@@ -107,7 +107,7 @@ export async function runInit(options: InitOptions): Promise<InitResult> {
     };
   } catch (error) {
     const message = error instanceof Error ? error.message : String(error);
-    console.spinner.fail(`Error: ${message}`);
+    console.log.error(message);
 
     return {
       success: false,

mcp-server/src/cli/utils/console.spec.ts

@@ -62,64 +62,4 @@ describe('console utils', () => {
     });
   });
 
-  describe('spinner', () => {
-    it('should start spinner with message', () => {
-      consoleUtils.spinner.start('Loading...');
-
-      expect(stdoutWrite).toHaveBeenCalled();
-      const output = stdoutWrite.mock.calls[0][0] as string;
-      expect(output).toContain('Loading...');
-    });
-
-    it('should stop spinner and show success', () => {
-      consoleUtils.spinner.start('Loading...');
-      stdoutWrite.mockClear();
-
-      consoleUtils.spinner.succeed('Done!');
-
-      expect(stdoutWrite).toHaveBeenCalled();
-      const output = stdoutWrite.mock.calls.map((c) => c[0]).join('');
-      expect(output).toContain('Done!');
-      expect(output).toContain('✓');
-    });
-
-    it('should stop spinner and show failure', () => {
-      consoleUtils.spinner.start('Loading...');
-      stderrWrite.mockClear();
-
-      consoleUtils.spinner.fail('Failed!');
-
-      expect(stderrWrite).toHaveBeenCalled();
-      const output = stderrWrite.mock.calls.map((c) => c[0]).join('');
-      expect(output).toContain('Failed!');
-      expect(output).toContain('✗');
-    });
-
-    it('should stop spinner without message', () => {
-      consoleUtils.spinner.start('Loading...');
-      stdoutWrite.mockClear();
-
-      consoleUtils.spinner.stop();
-
-      // Should clear the spinner line
-      expect(stdoutWrite).toHaveBeenCalled();
-    });
-  });
-
-  describe('formatConfig', () => {
-    it('should format config object as readable output', () => {
-      const config = {
-        projectName: 'my-app',
-        techStack: {
-          frontend: ['React'],
-        },
-      };
-
-      const result = consoleUtils.formatConfig(config);
-
-      expect(result).toContain('projectName');
-      expect(result).toContain('my-app');
-      expect(result).toContain('React');
-    });
-  });
 });

mcp-server/src/cli/utils/console.ts

@@ -15,13 +15,6 @@ export interface ConsoleUtils {
     error(message: string): void;
     step(emoji: string, message: string): void;
   };
-  spinner: {
-    start(message: string): void;
-    succeed(message: string): void;
-    fail(message: string): void;
-    stop(): void;
-  };
-  formatConfig(config: unknown): string;
 }
 
 /**
@@ -33,26 +26,12 @@ const colors = {
   yellow: '\x1b[33m',
   red: '\x1b[31m',
   cyan: '\x1b[36m',
-  dim: '\x1b[2m',
 };
 
-/**
- * Spinner frames for animation
- */
-const spinnerFrames = ['⠋', '⠙', '⠹', '⠸', '⠼', '⠴', '⠦', '⠧', '⠇', '⠏'];
-
 /**
  * Create console utilities instance
  */
 export function createConsoleUtils(): ConsoleUtils {
-  let spinnerInterval: ReturnType<typeof setInterval> | null = null;
-  let spinnerMessage = '';
-  let frameIndex = 0;
-
-  const clearLine = () => {
-    process.stdout.write('\r\x1b[K');
-  };
-
   const log = {
     info(message: string): void {
       process.stdout.write(`${colors.cyan}ℹ${colors.reset} ${message}\n`);
@@ -75,58 +54,7 @@ export function createConsoleUtils(): ConsoleUtils {
     },
   };
 
-  const spinner = {
-    start(message: string): void {
-      spinnerMessage = message;
-      frameIndex = 0;
-
-      // Initial render
-      process.stdout.write(`${colors.cyan}${spinnerFrames[frameIndex]}${colors.reset} ${spinnerMessage}`);
-
-      // Start animation
-      spinnerInterval = setInterval(() => {
-        frameIndex = (frameIndex + 1) % spinnerFrames.length;
-        clearLine();
-        process.stdout.write(`${colors.cyan}${spinnerFrames[frameIndex]}${colors.reset} ${spinnerMessage}`);
-      }, 80);
-    },
-
-    succeed(message: string): void {
-      if (spinnerInterval) {
-        clearInterval(spinnerInterval);
-        spinnerInterval = null;
-      }
-      clearLine();
-      process.stdout.write(`${colors.green}✓${colors.reset} ${message}\n`);
-    },
-
-    fail(message: string): void {
-      if (spinnerInterval) {
-        clearInterval(spinnerInterval);
-        spinnerInterval = null;
-      }
-      clearLine();
-      process.stderr.write(`${colors.red}✗${colors.reset} ${message}\n`);
-    },
-
-    stop(): void {
-      if (spinnerInterval) {
-        clearInterval(spinnerInterval);
-        spinnerInterval = null;
-      }
-      clearLine();
-    },
-  };
-
-  const formatConfig = (config: unknown): string => {
-    return JSON.stringify(config, null, 2);
-  };
-
-  return {
-    log,
-    spinner,
-    formatConfig,
-  };
+  return { log };
 }
 
 /**

mcp-server/src/mcp/mcp.controller.ts

@@ -5,8 +5,8 @@ import { SSEServerTransport } from '@modelcontextprotocol/sdk/server/sse.js';
 
 @Controller()
 export class McpController {
-  // Note: This simple implementation assumes a single client connection for simplicity.
-  // For production with multiple clients, we would need session management.
+  // Single-client SSE implementation. New connections replace existing ones.
+  // For multi-client support, implement session-based transport management.
   private transport: SSEServerTransport | null = null;
 
   constructor(private mcpService: McpService) {}