arckode-framework 1.3.1 → 1.4.0

package/cli/analyze/report.ts

@@ -0,0 +1,107 @@
+import type { AnalysisResult, Violation } from './types'
+
+export function printAnalysis(result: AnalysisResult): void {
+  console.log('\n══════════════════════════════════════')
+  console.log('  Análisis de Arquitectura')
+  console.log('══════════════════════════════════════\n')
+
+  console.log(`📦 Módulos (${result.modules.length}):`)
+  for (const m of result.modules) {
+    const icon = m.hasIndex ? '✓' : '✗'
+    const parts = [
+      m.hasService ? 'service' : null,
+      m.hasController ? 'controller' : null,
+      m.hasTypes ? 'types' : null,
+      m.hasSockets ? 'sockets' : null,
+      m.hasValidators ? 'validators' : null,
+      m.hasTests ? 'tests' : null,
+    ].filter(Boolean)
+    console.log(`   ${icon} ${m.name} [${parts.join(', ')}]`)
+  }
+
+  console.log(`\n🔌 Conectores (${result.connectors.length}):`)
+  for (const c of result.connectors) console.log(`   ${c}`)
+
+  if (result.violations.length > 0) {
+    const byCategory: Record<string, Violation[]> = {
+      'Estructura': [], 'Acoplamiento': [], 'Diseño': [],
+      'Calidad': [], 'Seguridad': [], 'Performance': [], 'Portabilidad': [],
+    }
+
+    for (const v of result.violations) {
+      if (['MISSING_INDEX', 'MISSING_SERVICE', 'MISSING_TYPES', 'MISSING_SOCKETS', 'MISSING_VALIDATORS', 'MISSING_TESTS', 'MISSING_CONNECTORS', 'EMPTY_MODULE_DESCRIPTION', 'MODEL_IN_TYPES_FILE', 'MISSING_MODEL_TS'].includes(v.type)) {
+        byCategory['Estructura']!.push(v)
+      } else if (['DIRECT_MODULE_IMPORT', 'SERVICE_IMPORTS_OTHER_MODULE', 'CONNECTOR_BUSINESS_LOGIC', 'DUPLICATE_CONNECTOR', 'UNREGISTERED_CONNECTOR'].includes(v.type)) {
+        byCategory['Acoplamiento']!.push(v)
+      } else if (['BUSINESS_LOGIC_IN_CONTROLLER', 'GOD_SERVICE', 'INDEX_PENDING_CHANGES'].includes(v.type)) {
+        byCategory['Diseño']!.push(v)
+      } else if (['IDOR_RISK'].includes(v.type)) {
+        byCategory['Seguridad']!.push(v)
+      } else if (['N_PLUS_ONE_RISK'].includes(v.type)) {
+        byCategory['Performance']!.push(v)
+      } else if (['SERVICE_DEPENDS_ON_ORM'].includes(v.type)) {
+        byCategory['Portabilidad']!.push(v)
+      } else {
+        byCategory['Calidad']!.push(v)
+      }
+    }
+
+    console.log(`\n❌ Violaciones (${result.violations.length}):`)
+    for (const [cat, vs] of Object.entries(byCategory)) {
+      if (vs.length === 0) continue
+      console.log(`\n   [${cat}]`)
+      for (const v of vs) console.log(`     • ${v.message}`)
+    }
+  }
+
+  if (result.warnings.length > 0) {
+    console.log(`\n⚠️  Advertencias (${result.warnings.length}):`)
+    for (const w of result.warnings) console.log(`   ${w.message}`)
+  }
+
+  console.log(`\n📊 Resultado: ${result.valid ? '✅ VÁLIDO' : '❌ VIOLACIONES ENCONTRADAS'}`)
+  console.log('══════════════════════════════════════\n')
+}
+
+export function buildManifest(result: AnalysisResult): Record<string, unknown> {
+  return {
+    _arckode: '1.0.0',
+    _generated: new Date().toISOString(),
+    _instructions: 'Lee este archivo ANTES de escribir código. Contiene el estado completo del sistema: módulos, conectores, violaciones. Si está desactualizado, corré: bun run analyze --json',
+
+    system: {
+      healthy: result.valid,
+      violations: result.violations.length,
+      warnings: result.warnings.length,
+    },
+
+    modules: result.modules.map(m => ({
+      name: m.name,
+      structure: {
+        index: m.hasIndex, types: m.hasTypes, sockets: m.hasSockets,
+        service: m.hasService, controller: m.hasController,
+        validators: m.hasValidators, tests: m.hasTests,
+      },
+      complete: m.hasIndex && m.hasTypes && m.hasService && m.hasController && m.hasValidators && m.hasTests,
+    })),
+
+    connectors: result.connectors,
+
+    violations: result.violations.map(v => ({ module: v.module, type: v.type, message: v.message })),
+
+    warnings: result.warnings.map(w => ({ type: w.type, message: w.message })),
+
+    rules_summary: [
+      'REGLA 1: Módulos NO importan de otros módulos. Usar conectores.',
+      'REGLA 2: index.ts es APPEND-ONLY. No eliminar exports.',
+      'REGLA 3: Conectores NO tienen lógica de negocio. Solo delegan.',
+      'REGLA 4: Cada tabla pertenece a UN módulo.',
+      'REGLA 5: Si no está en composition-root.ts, no existe.',
+      'REGLA 6: Todo POST/PUT/PATCH requiere validateSchema().',
+      'REGLA 7: Controller NO llama al ORM. Llama al service.',
+      'REGLA 18: Service recibe RepositoryAdapter<T>, no ORM directamente.',
+      'REGLA 22: model.ts separado de types.ts. Schema DB ≠ contrato TS.',
+      'REGLA 23: Un DEFAULT por caso. Escalar SOLO cuando la condición lo justifica.',
+    ],
+  }
+}

package/cli/analyze/types.ts

@@ -0,0 +1,46 @@
+export interface AnalysisResult {
+  valid: boolean
+  modules: ModuleReport[]
+  connectors: string[]
+  violations: Violation[]
+  warnings: Warning[]
+}
+
+export interface ModuleReport {
+  name: string
+  hasIndex: boolean
+  hasService: boolean
+  hasController: boolean
+  hasTypes: boolean
+  hasSockets: boolean
+  hasValidators: boolean
+  hasTests: boolean
+}
+
+export interface Violation {
+  type:
+    | 'DIRECT_MODULE_IMPORT'
+    | 'MISSING_INDEX' | 'MISSING_CONTRACT' | 'MISSING_SERVICE' | 'MISSING_TYPES'
+    | 'MISSING_SOCKETS' | 'MISSING_VALIDATORS' | 'MISSING_TESTS'
+    | 'MISSING_CONNECTORS' | 'CONNECTOR_BUSINESS_LOGIC' | 'INDEX_PENDING_CHANGES'
+    | 'CONTROLLER_MISSING_VALIDATION'
+    | 'BUSINESS_LOGIC_IN_CONTROLLER'
+    | 'EMPTY_MODULE_DESCRIPTION'
+    | 'TESTS_WITHOUT_CASES'
+    | 'SERVICE_IMPORTS_OTHER_MODULE'
+    | 'GOD_SERVICE'
+    | 'N_PLUS_ONE_RISK'
+    | 'IDOR_RISK'
+    | 'SERVICE_DEPENDS_ON_ORM'
+    | 'MODEL_IN_TYPES_FILE'
+    | 'MISSING_MODEL_TS'
+    | 'DUPLICATE_CONNECTOR'
+    | 'UNREGISTERED_CONNECTOR'
+  module: string
+  message: string
+}
+
+export interface Warning {
+  type: 'NO_CONNECTORS' | 'SINGLE_MODULE' | 'NO_COMPOSITION_ROOT' | 'INDEX_PENDING_CHANGES' | 'LEGACY_ACTIONS_FOLDER'
+  message: string
+}

package/cli/analyze/utils.ts

@@ -0,0 +1,36 @@
+import { readdir, access } from 'node:fs/promises'
+import { join } from 'node:path'
+
+export async function fileExists(path: string): Promise<boolean> {
+  try { await access(path); return true } catch { return false }
+}
+
+export async function resolvePath(modPath: string, candidates: string[]): Promise<string | null> {
+  for (const candidate of candidates) {
+    const full = join(modPath, candidate)
+    if (await fileExists(full)) return full
+  }
+  return null
+}
+
+export function canonicalizeConnectorName(filename: string): string {
+  return filename.replace(/\.ts$/, '').replace(/[-_]/g, '').toLowerCase()
+}
+
+export async function getAllTsFiles(dir: string): Promise<string[]> {
+  const files: string[] = []
+  try {
+    const entries = await readdir(dir, { withFileTypes: true })
+    for (const entry of entries) {
+      const full = join(dir, entry.name)
+      if (entry.isDirectory()) {
+        if (entry.name !== 'node_modules' && entry.name !== 'tests' && entry.name !== '__tests__') {
+          files.push(...await getAllTsFiles(full))
+        }
+      } else if (entry.name.endsWith('.ts')) {
+        files.push(full)
+      }
+    }
+  } catch { /* ignore */ }
+  return files
+}