tech-debt-score 0.1.5 → 0.1.7

package/src/domain/rules/SizeRule.ts

@@ -1,98 +0,0 @@
-/**
- * Domain Rule: Size Rule
- * Evaluates file and function size metrics
- */
-
-import { BaseRule } from '../entities/Rule.js';
-import type { Metric } from '../entities/Metric.js';
-import type { Finding } from '../entities/Finding.js';
-import { FindingBuilder } from '../entities/Finding.js';
-
-export class SizeRule extends BaseRule {
-  // Thresholds (configurable in future versions)
-  private readonly MAX_FILE_LINES = 300;
-  private readonly MAX_FUNCTION_LINES = 50;
-  private readonly MAX_PARAMETERS = 5;
-
-  constructor() {
-    super(
-      'size',
-      'Size Rule',
-      'Evaluates file and function size to identify overly large code units'
-    );
-  }
-
-  evaluate(metrics: Metric[]): Finding[] {
-    const findings: Finding[] = [];
-
-    const sizeMetrics = metrics.filter(
-      m => m.name === 'file-length' || m.name === 'function-length' || m.name === 'parameter-count'
-    );
-
-    for (const metric of sizeMetrics) {
-      if (metric.name === 'file-length' && metric.value > this.MAX_FILE_LINES) {
-        findings.push(
-          new FindingBuilder()
-            .withRuleId(this.id)
-            .withSeverity(metric.value > 500 ? 'high' : 'medium')
-            .withMessage(`Large file (${metric.value} lines)`)
-            .withFilePath(metric.filePath)
-            .withSuggestion('Consider splitting this file into smaller, more focused modules')
-            .build()
-        );
-      }
-
-      if (metric.name === 'function-length' && metric.value > this.MAX_FUNCTION_LINES) {
-        const builder = new FindingBuilder()
-          .withRuleId(this.id)
-          .withSeverity(metric.value > 100 ? 'high' : 'medium')
-          .withMessage(
-            `Large function (${metric.value} lines) in ${metric.context || 'unknown'}`
-          )
-          .withFilePath(metric.filePath)
-          .withSuggestion('Break this function into smaller, single-purpose functions');
-        
-        if (metric.location) {
-          builder.withLocation(metric.location);
-        }
-        
-        findings.push(builder.build());
-      }
-
-      if (metric.name === 'parameter-count' && metric.value > this.MAX_PARAMETERS) {
-        const builder = new FindingBuilder()
-          .withRuleId(this.id)
-          .withSeverity('medium')
-          .withMessage(
-            `Too many parameters (${metric.value}) in ${metric.context || 'function'}`
-          )
-          .withFilePath(metric.filePath)
-          .withSuggestion('Consider using an options object or builder pattern');
-        
-        if (metric.location) {
-          builder.withLocation(metric.location);
-        }
-        
-        findings.push(builder.build());
-      }
-    }
-
-    return findings;
-  }
-
-  calculateScore(findings: Finding[]): number {
-    if (findings.length === 0) return 100;
-
-    const penalty = findings.reduce((sum, finding) => {
-      switch (finding.severity) {
-        case 'high': return sum + 8;
-        case 'medium': return sum + 4;
-        case 'low': return sum + 2;
-        default: return sum;
-      }
-    }, 0);
-
-    const score = Math.max(0, 100 - penalty);
-    return Math.round(score * 100) / 100;
-  }
-}

package/src/domain/rules/TypeSafetyRule.ts

@@ -1,63 +0,0 @@
-/**
- * Domain Rule: Type Safety Rule
- * Evaluates TypeScript type safety indicators
- */
-
-import { BaseRule } from '../entities/Rule.js';
-import type { Metric } from '../entities/Metric.js';
-import type { Finding } from '../entities/Finding.js';
-import { FindingBuilder } from '../entities/Finding.js';
-
-export class TypeSafetyRule extends BaseRule {
-  constructor() {
-    super(
-      'type-safety',
-      'Type Safety Rule',
-      'Evaluates TypeScript type safety by detecting usage of the `any` type'
-    );
-  }
-
-  evaluate(metrics: Metric[]): Finding[] {
-    const findings: Finding[] = [];
-
-    const anyUsageMetrics = metrics.filter(m => m.name === 'any-usage');
-
-    for (const metric of anyUsageMetrics) {
-      if (metric.value > 0) {
-        const builder = new FindingBuilder()
-          .withRuleId(this.id)
-          .withSeverity(metric.value > 5 ? 'high' : 'medium')
-          .withMessage(
-            `Found ${metric.value} usage(s) of 'any' type in ${metric.context || 'code'}`
-          )
-          .withFilePath(metric.filePath)
-          .withSuggestion('Replace `any` with specific types or use `unknown` for better type safety');
-        
-        if (metric.location) {
-          builder.withLocation(metric.location);
-        }
-        
-        findings.push(builder.build());
-      }
-    }
-
-    return findings;
-  }
-
-  calculateScore(findings: Finding[]): number {
-    if (findings.length === 0) return 100;
-
-    // Each 'any' usage reduces the score
-    const totalAnyUsages = findings.reduce((sum, finding) => {
-      // Extract count from message (hacky, but works for v1)
-      const match = finding.message.match(/Found (\d+) usage/);
-      return sum + (match ? parseInt(match[1] ?? '0', 10) : 1);
-    }, 0);
-
-    // Each 'any' costs 3 points
-    const penalty = Math.min(100, totalAnyUsages * 3);
-    const score = Math.max(0, 100 - penalty);
-
-    return Math.round(score * 100) / 100;
-  }
-}

package/src/shared/types.ts

@@ -1,18 +0,0 @@
-/**
- * Shared type definitions used across the application
- */
-
-export interface SourceLocation {
-  startLine: number;
-  endLine: number;
-  startColumn: number;
-  endColumn: number;
-}
-
-export type Severity = 'low' | 'medium' | 'high';
-
-export interface FileMetadata {
-  path: string;
-  size: number;
-  linesOfCode: number;
-}

package/tests/application/index.test.ts

@@ -1,12 +0,0 @@
-/**
- * Application Layer Tests
- * 
- * Integration tests using mock adapters
- */
-
-// TODO: Add tests for:
-// - AnalysisService orchestration
-// - Integration between services and ports
-// - Configuration handling
-
-export {};

package/tests/domain/index.test.ts

@@ -1,14 +0,0 @@
-/**
- * Domain Layer Tests
- * 
- * Unit tests for pure business logic (no I/O, no mocks needed)
- */
-
-// TODO: Add tests for:
-// - Score calculation
-// - Metric builder
-// - Finding builder
-// - Rule evaluation (ComplexityRule, SizeRule, TypeSafetyRule)
-// - ScoreCalculator normalization
-
-export {};

package/tests/e2e/index.test.ts

@@ -1,13 +0,0 @@
-/**
- * End-to-End Tests
- * 
- * Test with real codebases
- */
-
-// TODO: Add tests for:
-// - Full analysis workflow
-// - Testing against sample codebases
-// - Performance benchmarks
-// - Output validation
-
-export {};

package/tsconfig.json

@@ -1,31 +0,0 @@
-{
-  "compilerOptions": {
-    /* File Layout */
-    "rootDir": "./src",
-    "outDir": "./dist",
-
-    /* Environment */
-    "target": "ES2020",
-    "module": "NodeNext",
-    "moduleResolution": "NodeNext",
-    "types": ["node"],
-
-    /* Output */
-    "sourceMap": true,
-    "declaration": true,
-    "declarationMap": true,
-
-    /* Type Safety */
-    "strict": true,
-    "noUncheckedIndexedAccess": true,
-    "exactOptionalPropertyTypes": true,
-
-    /* Module / Build Safety */
-    "verbatimModuleSyntax": true,
-    "isolatedModules": true,
-    "moduleDetection": "force",
-    "skipLibCheck": true
-  },
-  "include": ["src"],
-  "exclude": ["node_modules", "dist"]
-}