@soulbatical/tetra-dev-toolkit 1.6.1 → 1.7.0

package/lib/checks/codeQuality/file-size.js

@@ -0,0 +1,140 @@
+/**
+ * Check: File Size / God File Detection
+ *
+ * Prevents "god files" — overly large source files that:
+ * - Are hard to review, test, and maintain
+ * - Indicate missing separation of concerns
+ * - Grow silently until they become unmovable
+ *
+ * Uses config.codeQuality.maxFileLines (default: 500).
+ * Scans backend + frontend source directories.
+ */
+
+import { glob } from 'glob'
+import { readFileSync } from 'fs'
+
+export const meta = {
+  id: 'file-size',
+  name: 'File Size / God File Detection',
+  category: 'codeQuality',
+  severity: 'high',
+  description: 'Detects source files exceeding maxFileLines — god files that need splitting'
+}
+
+export async function run(config, projectRoot) {
+  const maxLines = config.codeQuality?.maxFileLines || 500
+
+  const results = {
+    passed: true,
+    findings: [],
+    summary: { total: 0, critical: 0, high: 0, medium: 0, low: 0 },
+    info: {
+      maxFileLines: maxLines,
+      filesScanned: 0,
+      violations: 0,
+      largest: null
+    }
+  }
+
+  // Scan all TypeScript/JavaScript source files
+  const patterns = [
+    'backend/src/**/*.ts',
+    'frontend/src/**/*.ts',
+    'frontend/src/**/*.tsx',
+    'backend-mcp/src/**/*.ts',
+    'src/**/*.ts',
+    'src/**/*.tsx',
+    'bot/**/*.ts',
+  ]
+
+  let files = []
+  for (const pattern of patterns) {
+    const found = await glob(pattern, {
+      cwd: projectRoot,
+      ignore: [
+        ...(config.ignore || []),
+        'node_modules/**',
+        '**/node_modules/**',
+        'dist/**',
+        '**/dist/**',
+        'build/**',
+        '**/build/**',
+        '**/*.test.*',
+        '**/*.spec.*',
+        '**/*.d.ts',
+        '**/*.js.map',
+        '**/generated/**',
+      ]
+    })
+    files.push(...found)
+  }
+
+  // Deduplicate
+  files = [...new Set(files)]
+
+  if (files.length === 0) {
+    results.skipped = true
+    results.skipReason = 'No source files found'
+    return results
+  }
+
+  let largestFile = null
+  let largestLines = 0
+
+  for (const file of files) {
+    const filePath = `${projectRoot}/${file}`
+    let content
+    try {
+      content = readFileSync(filePath, 'utf-8')
+    } catch {
+      continue
+    }
+
+    results.info.filesScanned++
+
+    const lineCount = content.split('\n').length
+
+    // Track largest file
+    if (lineCount > largestLines) {
+      largestLines = lineCount
+      largestFile = file
+    }
+
+    if (lineCount <= maxLines) continue
+
+    // Determine severity based on how far over the limit
+    const ratio = lineCount / maxLines
+    let severity
+    if (ratio >= 5) {
+      severity = 'critical' // 5x over limit (e.g. 2500+ lines with 500 limit)
+    } else if (ratio >= 3) {
+      severity = 'high'     // 3x over limit (e.g. 1500+ lines)
+    } else if (ratio >= 2) {
+      severity = 'medium'   // 2x over limit (e.g. 1000+ lines)
+    } else {
+      severity = 'low'      // Just over limit
+    }
+
+    results.findings.push({
+      file,
+      line: 1,
+      type: 'GOD_FILE',
+      severity,
+      message: `${file} has ${lineCount} lines (max: ${maxLines}) — split into smaller modules`,
+      fix: `Break this file into focused modules of <${maxLines} lines each`
+    })
+
+    results.summary[severity]++
+    results.summary.total++
+    results.info.violations++
+  }
+
+  results.info.largest = largestFile ? { file: largestFile, lines: largestLines } : null
+
+  // Fail on critical or high findings
+  results.passed = results.findings.filter(f =>
+    f.severity === 'critical' || f.severity === 'high'
+  ).length === 0
+
+  return results
+}

package/lib/runner.js

@@ -15,6 +15,7 @@ import * as huskyHooks from './checks/stability/husky-hooks.js'
 import * as ciPipeline from './checks/stability/ci-pipeline.js'
 import * as npmAudit from './checks/stability/npm-audit.js'
 import * as apiResponseFormat from './checks/codeQuality/api-response-format.js'
+import * as fileSize from './checks/codeQuality/file-size.js'
 import * as gitignoreValidation from './checks/security/gitignore-validation.js'
 import * as rlsPolicyAudit from './checks/supabase/rls-policy-audit.js'
 import * as rpcParamMismatch from './checks/supabase/rpc-param-mismatch.js'
@@ -36,7 +37,8 @@ const ALL_CHECKS = {
     npmAudit
   ],
   codeQuality: [
-    apiResponseFormat
+    apiResponseFormat,
+    fileSize
   ],
   supabase: [
     rlsPolicyAudit,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@soulbatical/tetra-dev-toolkit",
-  "version": "1.6.1",
+  "version": "1.7.0",
   "publishConfig": {
     "access": "restricted"
   },