@girardelli/architect 5.0.0 → 8.1.0

package/tests/framework-detector.test.ts

@@ -1,1172 +0,0 @@
-import { FrameworkDetector } from '../src/agent-generator/framework-detector.js';
-import { AnalysisReport } from '../src/types.js';
-import { existsSync, mkdtempSync, writeFileSync, rmSync } from 'fs';
-import { join } from 'path';
-
-// ── Test Data Factories ──
-
-/**
- * Creates a mock AnalysisReport for testing purposes.
- */
-function makeReport(overrides: Partial<AnalysisReport> = {}): AnalysisReport {
-  return {
-    timestamp: new Date().toISOString(),
-    projectInfo: {
-      path: '/test',
-      name: 'test-project',
-      frameworks: [],
-      totalFiles: 50,
-      totalLines: 5000,
-      primaryLanguages: ['Unknown'],
-    },
-    score: {
-      overall: 72,
-      components: [],
-      breakdown: { modularity: 80, coupling: 65, cohesion: 70, layering: 75 },
-    },
-    antiPatterns: [],
-    layers: [],
-    dependencyGraph: {
-      nodes: ['src/index.ts'],
-      edges: [],
-    },
-    suggestions: [],
-    diagram: { mermaid: '', type: 'layer' },
-    ...overrides,
-  };
-}
-
-// ── Test Suite ──
-
-describe('FrameworkDetector', () => {
-  const detector = new FrameworkDetector();
-  let tempDir: string;
-
-  beforeEach(() => {
-    tempDir = mkdtempSync(join(process.cwd(), '__test_framework__'));
-  });
-
-  afterEach(() => {
-    if (existsSync(tempDir)) {
-      rmSync(tempDir, { recursive: true, force: true });
-    }
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // PYTHON DETECTION
-  // ═══════════════════════════════════════════════════════════════════════
-
-  describe('Python detection', () => {
-    it('should detect FastAPI, Django, Flask from requirements.txt with versions', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Python'] } });
-      writeFileSync(join(tempDir, 'requirements.txt'), 'fastapi==0.109.0\ndjango>=4.0\nflask~=2.3.0\n');
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.length).toBeGreaterThanOrEqual(3);
-      expect(result.frameworks.map(f => f.name)).toContain('FastAPI');
-      expect(result.frameworks.map(f => f.name)).toContain('Django');
-      expect(result.frameworks.map(f => f.name)).toContain('Flask');
-      expect(result.primaryFramework).toBeDefined();
-      expect(result.primaryFramework?.category).toBe('web');
-    });
-
-    it('should detect dependencies from pyproject.toml PEP 621 format with [project] section', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Python'] } });
-      writeFileSync(
-        join(tempDir, 'pyproject.toml'),
-        `[project]
-dependencies = [
-  "fastapi>=0.109.0",
-  "sqlalchemy>=2.0.0"
-]
-`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.map(f => f.name)).toContain('FastAPI');
-      expect(result.frameworks.map(f => f.name)).toContain('SQLAlchemy');
-    });
-
-    it('should detect dependencies from pyproject.toml with [project.optional-dependencies]', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Python'] } });
-      writeFileSync(
-        join(tempDir, 'pyproject.toml'),
-        `[project]
-dependencies = ["fastapi>=0.109.0"]
-
-[project.optional-dependencies]
-dev = ["pytest>=7.0", "ruff>=0.1.0"]
-test = ["hypothesis>=6.0"]
-`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.map(f => f.name)).toContain('FastAPI');
-      expect(result.frameworks.map(f => f.name)).toContain('pytest');
-      expect(result.frameworks.map(f => f.name)).toContain('Ruff');
-      expect(result.frameworks.map(f => f.name)).toContain('Hypothesis');
-    });
-
-    it('should detect dependencies from pyproject.toml with [tool.poetry.dependencies]', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Python'] } });
-      writeFileSync(
-        join(tempDir, 'pyproject.toml'),
-        `[tool.poetry.dependencies]
-python = "^3.11"
-django = "~4.2.0"
-sqlalchemy = "^2.0"
-`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.map(f => f.name)).toContain('Django');
-      expect(result.frameworks.map(f => f.name)).toContain('SQLAlchemy');
-    });
-
-    it('should detect setup.py as fallback Python dependencies', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Python'] } });
-      // parsePythonRequirements parses line-by-line, so setup.py needs one dep per line
-      writeFileSync(
-        join(tempDir, 'setup.py'),
-        `flask>=2.0
-sqlalchemy>=2.0
-`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.map(f => f.name)).toContain('Flask');
-      expect(result.frameworks.map(f => f.name)).toContain('SQLAlchemy');
-    });
-
-    it('should detect Pipfile as fallback Python dependencies', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Python'] } });
-      writeFileSync(
-        join(tempDir, 'Pipfile'),
-        `[packages]
-tornado = "*"
-peewee = ">=3.0"
-
-[dev-packages]
-pytest = "*"
-`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.map(f => f.name)).toContain('Tornado');
-      expect(result.frameworks.map(f => f.name)).toContain('Peewee');
-      expect(result.frameworks.map(f => f.name)).toContain('pytest');
-    });
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // NODE.JS DETECTION
-  // ═══════════════════════════════════════════════════════════════════════
-
-  describe('Node.js detection', () => {
-    it('should detect Express, NestJS, Jest, ESLint from package.json', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['TypeScript'] } });
-      writeFileSync(
-        join(tempDir, 'package.json'),
-        JSON.stringify({
-          name: 'test-app',
-          dependencies: {
-            express: '^4.18.0',
-            '@nestjs/core': '^10.0.0',
-          },
-          devDependencies: {
-            jest: '^29.0.0',
-            eslint: '^8.0.0',
-          },
-        }),
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.map(f => f.name)).toContain('Express');
-      expect(result.frameworks.map(f => f.name)).toContain('NestJS');
-      expect(result.frameworks.map(f => f.name)).toContain('Jest');
-      expect(result.frameworks.map(f => f.name)).toContain('ESLint');
-    });
-
-    it('should handle invalid JSON gracefully', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['TypeScript'] } });
-      writeFileSync(join(tempDir, 'package.json'), 'invalid json {]');
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.length).toBe(0);
-      expect(result.primaryFramework).toBeNull();
-    });
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // JAVA DETECTION
-  // ═══════════════════════════════════════════════════════════════════════
-
-  describe('Java detection', () => {
-    it('should detect Spring Boot from pom.xml with spring-boot-starter-web artifactId', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Java'] } });
-      writeFileSync(
-        join(tempDir, 'pom.xml'),
-        `<project>
-  <dependencies>
-    <dependency>
-      <artifactId>spring-boot-starter-web</artifactId>
-    </dependency>
-  </dependencies>
-</project>`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.map(f => f.name)).toContain('Spring Boot');
-    });
-
-    it('should detect Spring Boot, Quarkus, Micronaut, Ktor from build.gradle', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Java'] } });
-      writeFileSync(
-        join(tempDir, 'build.gradle'),
-        `plugins {
-  id 'java'
-}
-
-dependencies {
-  implementation 'io.quarkus:quarkus-core'
-}`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.map(f => f.name)).toContain('Quarkus');
-    });
-
-    it('should detect frameworks from build.gradle.kts', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Kotlin'] } });
-      writeFileSync(
-        join(tempDir, 'build.gradle.kts'),
-        `dependencies {
-  implementation("io.micronaut:micronaut-core")
-  implementation("io.ktor:ktor-server-core")
-}`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.map(f => f.name)).toContain('Micronaut');
-      expect(result.frameworks.map(f => f.name)).toContain('Ktor');
-    });
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // PHP DETECTION
-  // ═══════════════════════════════════════════════════════════════════════
-
-  describe('PHP detection', () => {
-    it('should detect Laravel from composer.json', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['PHP'] } });
-      writeFileSync(
-        join(tempDir, 'composer.json'),
-        JSON.stringify({
-          name: 'myapp',
-          require: {
-            'laravel/framework': '^10.0',
-            php: '^8.1',
-          },
-        }),
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.map(f => f.name)).toContain('Laravel');
-    });
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // GO DETECTION
-  // ═══════════════════════════════════════════════════════════════════════
-
-  describe('Go detection', () => {
-    it('should detect Gin from go.mod', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Go'] } });
-      writeFileSync(
-        join(tempDir, 'go.mod'),
-        `module myapp
-
-go 1.21
-
-require (
-  github.com/gin-gonic/gin v1.9.0
-  github.com/labstack/echo v3.3.0
-)`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.map(f => f.name)).toContain('Gin');
-      expect(result.frameworks.map(f => f.name)).toContain('Echo');
-    });
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // RUBY DETECTION
-  // ═══════════════════════════════════════════════════════════════════════
-
-  describe('Ruby detection', () => {
-    it('should detect Rails, Sinatra, RSpec from Gemfile', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Ruby'] } });
-      writeFileSync(
-        join(tempDir, 'Gemfile'),
-        `source "https://rubygems.org"
-
-gem "rails", "~> 7.0.0"
-gem "sinatra"
-gem "rspec"`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.map(f => f.name)).toContain('Ruby on Rails');
-      expect(result.frameworks.map(f => f.name)).toContain('Sinatra');
-      expect(result.frameworks.map(f => f.name)).toContain('RSpec');
-    });
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // DART DETECTION
-  // ═══════════════════════════════════════════════════════════════════════
-
-  describe('Dart detection', () => {
-    it('should detect Flutter from pubspec.yaml with flutter: section', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Dart'] } });
-      writeFileSync(
-        join(tempDir, 'pubspec.yaml'),
-        `name: myapp
-description: A Flutter app
-
-flutter:
-  uses-material-design: true
-
-dev_dependencies:
-  flutter_test:
-    sdk: flutter`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.map(f => f.name)).toContain('Flutter');
-    });
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // RUST DETECTION
-  // ═══════════════════════════════════════════════════════════════════════
-
-  describe('Rust detection', () => {
-    it('should detect Actix Web and Axum from Cargo.toml', () => {
-      const report = makeReport({ projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Rust'] } });
-      writeFileSync(
-        join(tempDir, 'Cargo.toml'),
-        `[package]
-name = "myapp"
-version = "0.1.0"
-
-[dependencies]
-actix-web = "4.4"
-axum = "0.7"`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.map(f => f.name)).toContain('Actix Web');
-      expect(result.frameworks.map(f => f.name)).toContain('Axum');
-    });
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // TOOLCHAIN DETECTION
-  // ═══════════════════════════════════════════════════════════════════════
-
-  describe('Toolchain detection', () => {
-    it('should detect Python + FastAPI toolchain with uvicorn, pytest, ruff', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Python'] },
-        dependencyGraph: { nodes: ['src/main.py'], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'pyproject.toml'),
-        `[project]
-dependencies = [
-  "fastapi>=0.109.0",
-  "uvicorn[standard]>=0.27.0"
-]
-
-[project.optional-dependencies]
-dev = ["pytest>=7.0", "ruff>=0.1.0"]`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.runCmd).toContain('uvicorn');
-      expect(result.toolchain.testCmd).toBe('pytest');
-      expect(result.toolchain.lintCmd).toContain('ruff');
-      // depsFile defaults to requirements.txt unless poetry.lock or Pipfile.lock exists
-      expect(result.toolchain.depsFile).toBe('requirements.txt');
-    });
-
-    it('should detect Python + Django toolchain with manage.py commands', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Python'] },
-        dependencyGraph: { nodes: ['manage.py', 'app/models.py'], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'requirements.txt'),
-        `django>=4.0
-pytest>=7.0`,
-      );
-      writeFileSync(join(tempDir, 'manage.py'), 'import django\n');
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.runCmd).toBe('python manage.py runserver');
-      expect(result.toolchain.testCmd).toBe('pytest');
-      expect(result.toolchain.migrateCmd).toBe('python manage.py migrate');
-    });
-
-    it('should detect TypeScript with npm by default', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['TypeScript'] },
-        dependencyGraph: { nodes: ['src/index.ts'], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'package.json'),
-        JSON.stringify({
-          name: 'test',
-          dependencies: { express: '^4.18.0' },
-          devDependencies: { jest: '^29.0.0' },
-        }),
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.buildCmd).toContain('npm');
-      expect(result.toolchain.testCmd).toContain('npm');
-      expect(result.toolchain.runCmd).toContain('npm');
-    });
-
-    it('should detect TypeScript with yarn when yarn.lock exists', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['TypeScript'] },
-        dependencyGraph: { nodes: ['src/index.ts'], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'package.json'),
-        JSON.stringify({
-          name: 'test',
-          dependencies: { express: '^4.18.0' },
-        }),
-      );
-      writeFileSync(join(tempDir, 'yarn.lock'), '# yarn lockfile v1\n');
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.buildCmd).toContain('yarn');
-      expect(result.toolchain.testCmd).toContain('yarn');
-    });
-
-    it('should detect TypeScript with pnpm when pnpm-lock.yaml exists', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['TypeScript'] },
-        dependencyGraph: { nodes: ['src/index.ts'], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'package.json'),
-        JSON.stringify({
-          name: 'test',
-          dependencies: { express: '^4.18.0' },
-        }),
-      );
-      writeFileSync(join(tempDir, 'pnpm-lock.yaml'), 'lockfileVersion: 5.4\n');
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.buildCmd).toContain('pnpm');
-      expect(result.toolchain.testCmd).toContain('pnpm');
-    });
-
-    it('should detect Java + pom.xml toolchain with mvn commands', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Java'] },
-        dependencyGraph: { nodes: ['src/main/java/App.java'], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'pom.xml'),
-        `<project>
-  <dependencies>
-    <dependency>
-      <artifactId>spring-boot-starter-web</artifactId>
-    </dependency>
-  </dependencies>
-</project>`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.buildCmd).toBe('mvn clean package');
-      expect(result.toolchain.testCmd).toBe('mvn test');
-      expect(result.toolchain.runCmd).toBe('mvn spring-boot:run');
-      expect(result.toolchain.depsFile).toBe('pom.xml');
-    });
-
-    it('should detect Java + build.gradle toolchain with gradlew commands', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Java'] },
-        dependencyGraph: { nodes: ['src/main/java/App.java'], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'build.gradle'),
-        `plugins { id 'java' }
-dependencies { implementation 'org.springframework.boot:spring-boot' }`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.buildCmd).toBe('./gradlew build');
-      expect(result.toolchain.testCmd).toBe('./gradlew test');
-      expect(result.toolchain.runCmd).toBe('./gradlew bootRun');
-      expect(result.toolchain.depsFile).toBe('build.gradle');
-    });
-
-    it('should detect PHP + Laravel toolchain with artisan commands', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['PHP'] },
-        dependencyGraph: { nodes: ['app/Http/Controllers/AppController.php'], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'composer.json'),
-        JSON.stringify({
-          require: { 'laravel/framework': '^10.0' },
-        }),
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.runCmd).toBe('php artisan serve');
-      expect(result.toolchain.testCmd).toBe('php artisan test');
-      expect(result.toolchain.migrateCmd).toBe('php artisan migrate');
-    });
-
-    it('should detect Go toolchain with go commands', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Go'] },
-        dependencyGraph: { nodes: ['main.go'], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'go.mod'),
-        `module myapp
-go 1.21
-require github.com/gin-gonic/gin v1.9.0`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.buildCmd).toBe('go build ./...');
-      expect(result.toolchain.testCmd).toBe('go test ./...');
-      expect(result.toolchain.lintCmd).toBe('golangci-lint run');
-      expect(result.toolchain.runCmd).toBe('go run .');
-      expect(result.toolchain.depsFile).toBe('go.mod');
-    });
-
-    it('should detect Ruby + Rails toolchain with rails commands', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Ruby'] },
-        dependencyGraph: { nodes: ['app/controllers/application_controller.rb'], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'Gemfile'),
-        `source "https://rubygems.org"
-gem "rails", "~> 7.0.0"
-gem "rspec"`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.runCmd).toBe('rails server');
-      expect(result.toolchain.testCmd).toBe('bundle exec rspec');
-      expect(result.toolchain.migrateCmd).toBe('rails db:migrate');
-      expect(result.toolchain.depsFile).toBe('Gemfile');
-    });
-
-    it('should detect Dart + Flutter toolchain with flutter commands', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Dart'] },
-        dependencyGraph: { nodes: ['lib/main.dart'], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'pubspec.yaml'),
-        `name: myapp
-flutter:
-  uses-material-design: true`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.buildCmd).toBe('flutter build');
-      expect(result.toolchain.testCmd).toBe('flutter test');
-      expect(result.toolchain.runCmd).toBe('flutter run');
-      expect(result.toolchain.depsFile).toBe('pubspec.yaml');
-    });
-
-    it('should detect Rust toolchain with cargo commands', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Rust'] },
-        dependencyGraph: { nodes: ['src/main.rs'], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'Cargo.toml'),
-        `[package]
-name = "myapp"
-[dependencies]
-actix-web = "4.4"`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.buildCmd).toBe('cargo build');
-      expect(result.toolchain.testCmd).toBe('cargo test');
-      expect(result.toolchain.lintCmd).toBe('cargo clippy');
-      expect(result.toolchain.runCmd).toBe('cargo run');
-      expect(result.toolchain.depsFile).toBe('Cargo.toml');
-    });
-
-    it('should fallback to Makefile commands when available', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Unknown'] },
-        dependencyGraph: { nodes: [], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'Makefile'),
-        `build:
-	@echo "Building..."
-
-test:
-	@echo "Testing..."
-
-lint:
-	@echo "Linting..."
-
-run:
-	@echo "Running..."`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.buildCmd).toBe('make build');
-      expect(result.toolchain.testCmd).toBe('make test');
-      expect(result.toolchain.lintCmd).toBe('make lint');
-      expect(result.toolchain.runCmd).toBe('make run');
-    });
-
-    it('should fallback to generic commands when no build tool detected', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Unknown'] },
-        dependencyGraph: { nodes: [], edges: [] },
-      });
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.buildCmd).toContain('No build command detected');
-      expect(result.toolchain.testCmd).toContain('No test command detected');
-      expect(result.toolchain.depsFile).toBe('unknown');
-    });
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // PROJECT STRUCTURE DETECTION
-  // ═══════════════════════════════════════════════════════════════════════
-
-  describe('Project structure detection', () => {
-    it('should detect clean-architecture with domain/ and infrastructure/ directories', () => {
-      const report = makeReport({
-        dependencyGraph: {
-          nodes: [
-            'src/domain/entities/user.ts',
-            'src/domain/repositories/user.repository.ts',
-            'src/infrastructure/database/connection.ts',
-            'src/application/services/user.service.ts',
-            'src/presentation/controllers/user.controller.ts',
-          ],
-          edges: [],
-        },
-      });
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.projectStructure).toBe('clean-architecture');
-    });
-
-    it('should detect mvc with models/, views/, controllers/ directories', () => {
-      const report = makeReport({
-        dependencyGraph: {
-          nodes: [
-            'app/models/user.php',
-            'app/views/user/show.html',
-            'app/controllers/user_controller.php',
-          ],
-          edges: [],
-        },
-      });
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.projectStructure).toBe('mvc');
-    });
-
-    it('should detect modular with modules/ or features/ directories', () => {
-      const report = makeReport({
-        dependencyGraph: {
-          nodes: [
-            'src/modules/auth/auth.module.ts',
-            'src/modules/users/users.module.ts',
-            'src/modules/auth/controllers/auth.controller.ts',
-          ],
-          edges: [],
-        },
-      });
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.projectStructure).toBe('modular');
-    });
-
-    it('should detect modular with features directory', () => {
-      const report = makeReport({
-        dependencyGraph: {
-          nodes: [
-            'src/features/auth/auth.ts',
-            'src/features/users/users.ts',
-          ],
-          edges: [],
-        },
-      });
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.projectStructure).toBe('modular');
-    });
-
-    it('should detect monorepo with packages/ or apps/ directories', () => {
-      const report = makeReport({
-        dependencyGraph: {
-          nodes: [
-            'src/packages/api/src/index.ts',
-            'src/packages/web/src/index.ts',
-            'src/packages/shared/src/types.ts',
-          ],
-          edges: [],
-        },
-      });
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.projectStructure).toBe('monorepo');
-    });
-
-    it('should detect monorepo with apps directory', () => {
-      const report = makeReport({
-        dependencyGraph: {
-          nodes: [
-            'src/apps/api/src/index.ts',
-            'src/apps/web/src/index.ts',
-          ],
-          edges: [],
-        },
-      });
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.projectStructure).toBe('monorepo');
-    });
-
-    it('should detect flat structure with shallow directory depth', () => {
-      const report = makeReport({
-        dependencyGraph: {
-          nodes: [
-            'src/index.ts',
-            'src/util.ts',
-            'src/config.ts',
-          ],
-          edges: [],
-        },
-      });
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.projectStructure).toBe('flat');
-    });
-
-    it('should return unknown for unmatched structures', () => {
-      const report = makeReport({
-        dependencyGraph: {
-          nodes: [
-            'src/a/b/c/d/e/f/component.ts',
-            'src/x/y/z/util.ts',
-          ],
-          edges: [],
-        },
-      });
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.projectStructure).toBe('unknown');
-    });
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // EDGE CASES
-  // ═══════════════════════════════════════════════════════════════════════
-
-  describe('Edge cases', () => {
-    it('should handle empty project with no dependency files', () => {
-      const report = makeReport();
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.length).toBe(0);
-      expect(result.primaryFramework).toBeNull();
-      expect(result.toolchain).toBeDefined();
-    });
-
-    it('should deduplicate frameworks detected from multiple sources', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['TypeScript'] },
-        dependencyGraph: { nodes: ['package.json', 'yarn.lock'], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'package.json'),
-        JSON.stringify({
-          name: 'test',
-          dependencies: { jest: '^29.0.0', express: '^4.18.0' },
-          devDependencies: { jest: '^29.0.0' }, // Jest in both deps and devDeps
-        }),
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      const jestCount = result.frameworks.filter(f => f.name === 'Jest').length;
-      expect(jestCount).toBe(1); // Should be deduplicated
-    });
-
-    it('should pick web framework as primaryFramework', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['TypeScript'] },
-        dependencyGraph: { nodes: [], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'package.json'),
-        JSON.stringify({
-          name: 'test',
-          dependencies: {
-            express: '^4.18.0',
-            jest: '^29.0.0',
-            eslint: '^8.0.0',
-          },
-        }),
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.primaryFramework?.name).toBe('Express');
-      expect(result.primaryFramework?.category).toBe('web');
-    });
-
-    it('should sort frameworks: web first, then by confidence', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Python'] },
-        dependencyGraph: { nodes: [], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'requirements.txt'),
-        `pytest>=7.0
-fastapi>=0.100.0
-flask>=2.0
-black>=23.0`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      const frameworks = result.frameworks;
-      // Web frameworks (FastAPI, Flask) should come before test/lint (pytest, Black)
-      const webFrameworksIndices = frameworks
-        .map((f, i) => (f.category === 'web' ? i : -1))
-        .filter(i => i !== -1);
-      const nonWebIndices = frameworks
-        .map((f, i) => (f.category !== 'web' ? i : -1))
-        .filter(i => i !== -1);
-
-      if (webFrameworksIndices.length > 0 && nonWebIndices.length > 0) {
-        expect(Math.max(...webFrameworksIndices)).toBeLessThan(Math.min(...nonWebIndices));
-      }
-    });
-
-    it('should handle missing primaryLanguages gracefully', () => {
-      const report = makeReport({
-        projectInfo: {
-          ...makeReport().projectInfo,
-          primaryLanguages: [],
-        },
-        dependencyGraph: { nodes: [], edges: [] },
-      });
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain).toBeDefined();
-    });
-
-    it('should handle files in subdirectories (e.g., requirements/prod.txt)', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Python'] },
-        dependencyGraph: { nodes: [], edges: [] },
-      });
-
-      // Create subdirectory
-      const reqDir = join(tempDir, 'requirements');
-      writeFileSync(
-        join(tempDir, 'requirements'),
-        'placeholder',
-      );
-      rmSync(join(tempDir, 'requirements'), { force: true });
-
-      // Instead, just test that reading non-existent file doesn't crash
-      const result = detector.detect(tempDir, report);
-
-      expect(result).toBeDefined();
-    });
-
-    it('should detect Vitest over Jest when both are present', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['TypeScript'] },
-        dependencyGraph: { nodes: [], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'package.json'),
-        JSON.stringify({
-          name: 'test',
-          devDependencies: {
-            jest: '^29.0.0',
-            vitest: '^1.0.0',
-          },
-        }),
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      const frameworks = result.frameworks.map(f => f.name);
-      expect(frameworks).toContain('Jest');
-      expect(frameworks).toContain('Vitest');
-    });
-
-    it('should handle poetry.lock indicator for Poetry package manager', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Python'] },
-        dependencyGraph: { nodes: [], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'pyproject.toml'),
-        `[project]
-dependencies = ["fastapi>=0.100.0"]`,
-      );
-      writeFileSync(join(tempDir, 'poetry.lock'), '# poetry lock file\n');
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.installCmd).toBe('poetry install');
-      expect(result.toolchain.depsFile).toBe('pyproject.toml');
-    });
-
-    it('should handle Pipenv Pipfile.lock indicator', () => {
-      const report = makeReport({
-        projectInfo: { ...makeReport().projectInfo, primaryLanguages: ['Python'] },
-        dependencyGraph: { nodes: [], edges: [] },
-      });
-      writeFileSync(
-        join(tempDir, 'requirements.txt'),
-        'flask>=2.0',
-      );
-      writeFileSync(join(tempDir, 'Pipfile.lock'), '{}');
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.toolchain.installCmd).toBe('pipenv install');
-    });
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // INTEGRATION TESTS
-  // ═══════════════════════════════════════════════════════════════════════
-
-  describe('Integration tests', () => {
-    it('should detect full Python FastAPI stack with clean-architecture structure', () => {
-      const report = makeReport({
-        projectInfo: {
-          ...makeReport().projectInfo,
-          primaryLanguages: ['Python'],
-          name: 'fastapi-app',
-        },
-        dependencyGraph: {
-          nodes: [
-            'src/domain/entities/user.py',
-            'src/infrastructure/database.py',
-            'src/application/services/user.py',
-            'src/main.py',
-          ],
-          edges: [],
-        },
-      });
-      writeFileSync(
-        join(tempDir, 'pyproject.toml'),
-        `[project]
-dependencies = [
-  "fastapi>=0.109.0",
-  "sqlalchemy>=2.0",
-  "uvicorn[standard]>=0.27"
-]
-
-[project.optional-dependencies]
-dev = ["pytest>=7.0", "ruff>=0.1.0"]`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.frameworks.map(f => f.name)).toContain('FastAPI');
-      expect(result.frameworks.map(f => f.name)).toContain('SQLAlchemy');
-      expect(result.primaryFramework?.name).toBe('FastAPI');
-      expect(result.toolchain.runCmd).toContain('uvicorn');
-      expect(result.toolchain.testCmd).toBe('pytest');
-      expect(result.projectStructure).toBe('clean-architecture');
-    });
-
-    it('should detect full TypeScript NestJS stack with modular structure', () => {
-      const report = makeReport({
-        projectInfo: {
-          ...makeReport().projectInfo,
-          primaryLanguages: ['TypeScript'],
-          name: 'nestjs-app',
-        },
-        dependencyGraph: {
-          nodes: [
-            'src/modules/auth/auth.module.ts',
-            'src/modules/users/users.module.ts',
-            'src/modules/auth/services/auth.service.ts',
-            'src/app.module.ts',
-            'src/main.ts',
-          ],
-          edges: [],
-        },
-      });
-      writeFileSync(
-        join(tempDir, 'package.json'),
-        JSON.stringify({
-          name: 'nestjs-app',
-          dependencies: {
-            '@nestjs/core': '^10.0.0',
-            '@nestjs/common': '^10.0.0',
-            typeorm: '^0.3.0',
-          },
-          devDependencies: {
-            jest: '^29.0.0',
-            eslint: '^8.0.0',
-          },
-        }),
-      );
-      writeFileSync(
-        join(tempDir, 'yarn.lock'),
-        '# yarn lockfile v1\n',
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.primaryFramework?.name).toBe('NestJS');
-      expect(result.frameworks.map(f => f.name)).toContain('TypeORM');
-      expect(result.toolchain.buildCmd).toContain('yarn');
-      expect(result.projectStructure).toBe('modular');
-    });
-
-    it('should detect full Java Spring Boot stack', () => {
-      const report = makeReport({
-        projectInfo: {
-          ...makeReport().projectInfo,
-          primaryLanguages: ['Java'],
-          name: 'spring-app',
-        },
-        dependencyGraph: {
-          nodes: [
-            'src/main/java/com/example/controller/UserController.java',
-            'src/main/java/com/example/service/UserService.java',
-            'src/main/java/com/example/repository/UserRepository.java',
-            'pom.xml',
-          ],
-          edges: [],
-        },
-      });
-      writeFileSync(
-        join(tempDir, 'pom.xml'),
-        `<project>
-  <dependencies>
-    <dependency>
-      <artifactId>spring-boot-starter-web</artifactId>
-    </dependency>
-    <dependency>
-      <artifactId>spring-boot-starter-data-jpa</artifactId>
-    </dependency>
-  </dependencies>
-</project>`,
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.primaryFramework?.name).toBe('Spring Boot');
-      expect(result.toolchain.buildCmd).toBe('mvn clean package');
-      expect(result.toolchain.runCmd).toBe('mvn spring-boot:run');
-      expect(result.projectStructure).not.toBe('clean-architecture');
-    });
-
-    it('should detect full PHP Laravel stack with mvc structure', () => {
-      const report = makeReport({
-        projectInfo: {
-          ...makeReport().projectInfo,
-          primaryLanguages: ['PHP'],
-          name: 'laravel-app',
-        },
-        dependencyGraph: {
-          nodes: [
-            'app/models/User.php',
-            'app/views/users/show.blade.php',
-            'app/Http/Controllers/UserController.php',
-          ],
-          edges: [],
-        },
-      });
-      writeFileSync(
-        join(tempDir, 'composer.json'),
-        JSON.stringify({
-          require: {
-            'laravel/framework': '^10.0',
-          },
-        }),
-      );
-
-      const result = detector.detect(tempDir, report);
-
-      expect(result.primaryFramework?.name).toBe('Laravel');
-      expect(result.toolchain.runCmd).toBe('php artisan serve');
-      expect(result.toolchain.migrateCmd).toBe('php artisan migrate');
-      expect(result.projectStructure).toBe('mvc');
-    });
-  });
-});