arckode-framework 1.1.2 → 1.2.0

package/cli/analyze.ts

@@ -42,6 +42,9 @@ export interface Violation {
     | 'IDOR_RISK'                      // findById sin assertOwnership posterior
     // ── Portabilidad ──
     | 'SERVICE_DEPENDS_ON_ORM'         // service inyecta ORM en lugar de RepositoryAdapter
+    // ── Estructura de archivos (Regla #22) ──
+    | 'MODEL_IN_TYPES_FILE'            // ModelDefinition en types.ts en vez de model.ts
+    | 'MISSING_MODEL_TS'               // módulo sin model.ts
     // ── Conectores ──
     | 'DUPLICATE_CONNECTOR'            // mismo nombre lógico con distinto casing (foo-bar.ts y fooBar.ts)
     | 'UNREGISTERED_CONNECTOR'         // archivo existe pero no está en composition-root.ts
@@ -427,7 +430,6 @@ export async function analyzeProject(basePath: string): Promise<AnalysisResult>
     // 9. SERVICE DEPENDS ON ORM — service inyecta ORM en lugar de RepositoryAdapter (Regla #18)
     try {
       const serviceContent = await readFile(servicePath, 'utf-8')
-      // Detectar: private orm: ORM  o  private readonly orm: ORM  en el constructor
       if (/private\s+(readonly\s+)?\w+\s*:\s*ORM\b/.test(serviceContent)) {
         violations.push({
           type: 'SERVICE_DEPENDS_ON_ORM',
@@ -436,6 +438,28 @@ export async function analyzeProject(basePath: string): Promise<AnalysisResult>
         })
       }
     } catch { /* no service */ }
+
+    // 10. MODEL IN TYPES — ModelDefinition en types.ts en vez de model.ts (Regla #22)
+    const typesFilePath = join(modPath, 'types.ts')
+    try {
+      const typesContent = await readFile(typesFilePath, 'utf-8')
+      if (/\bModelDefinition\b/.test(typesContent)) {
+        violations.push({
+          type: 'MODEL_IN_TYPES_FILE',
+          module: module.name,
+          message: `"${module.name}/types.ts" contiene ModelDefinition o schema de DB. Moverlo a model.ts — son conceptos distintos. (CLAUDE #22)`,
+        })
+      }
+    } catch { /* no types.ts */ }
+
+    // 11. MISSING MODEL.TS — módulo sin model.ts (Regla #22)
+    if (!await fileExists(join(modPath, 'model.ts'))) {
+      violations.push({
+        type: 'MISSING_MODEL_TS',
+        module: module.name,
+        message: `"${module.name}" no tiene model.ts. El schema de DB debe vivir separado de types.ts. (CLAUDE #22)`,
+      })
+    }
   }
 
   return {
@@ -525,7 +549,7 @@ export function printAnalysis(result: AnalysisResult): void {
     }
 
     for (const v of result.violations) {
-      if (['MISSING_INDEX', 'MISSING_SERVICE', 'MISSING_TYPES', 'MISSING_SOCKETS', 'MISSING_VALIDATORS', 'MISSING_TESTS', 'MISSING_CONNECTORS', 'EMPTY_MODULE_DESCRIPTION'].includes(v.type)) {
+      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'].includes(v.type)) {
         byCategory['Acoplamiento']!.push(v)
@@ -616,6 +640,8 @@ export function buildManifest(result: AnalysisResult): Record<string, unknown> {
       '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. Nivel 1: OrmRepository / service.ts / Conector. Nivel 2+ solo si aplica.',
     ],
   }
 }

package/cli/index.ts

@@ -168,12 +168,12 @@ async function main() {
         version: '1.0.0',
         type: 'module',
         scripts: {
-          dev:           `bun --watch src/composition-root.ts`,
-          start:         `bun run src/composition-root.ts`,
-          test:          `bun test`,
-          'test:watch':  `bun test --watch`,
-          analyze:       `bun run node_modules/arckode-framework/cli/index.ts analyze`,
-          'analyze:json': `bun run node_modules/arckode-framework/cli/index.ts analyze --json`,
+          dev:          `bun --watch src/composition-root.ts`,
+          start:        `bun run src/composition-root.ts`,
+          test:         `bun test`,
+          'test:watch': `bun test --watch`,
+          analyze:      `bun run node_modules/arckode-framework/cli/index.ts analyze`,
+          'analyze:ci': `bun run node_modules/arckode-framework/cli/index.ts analyze --ci`,
         },
         dependencies: {
           'arckode-framework': 'latest',
@@ -260,14 +260,18 @@ data/
       const basePath = process.cwd()
       const result = await analyzeProject(basePath)
 
-      if (args.includes('--json')) {
-        const manifest = buildManifest(result)
-        const dest = join(basePath, 'arckode.json')
-        await writeFile(dest, JSON.stringify(manifest, null, 2))
-        const status = result.valid ? '✅ válido' : `❌ ${result.violations.length} violaciones`
-        console.log(`arckode.json generado — ${status}`)
+      // arckode.json siempre se regenera — es el snapshot que la IA lee primero
+      const manifest = buildManifest(result)
+      const dest = join(basePath, 'arckode.json')
+      await writeFile(dest, JSON.stringify(manifest, null, 2))
+
+      if (args.includes('--ci')) {
+        // Modo CI: sin output de consola, exit 1 si hay violaciones
+        if (!result.valid) process.exit(1)
       } else {
         printAnalysis(result)
+        const status = result.valid ? '✅ sin violaciones' : `❌ ${result.violations.length} violación(es)`
+        console.log(`→ arckode.json actualizado — ${status}\n`)
       }
       break
     }

package/cli/stubs/claude-md-stub.ts

@@ -62,7 +62,8 @@ src/modules/mi-modulo/
 | 19 | Transactor para atomicidad multi-tabla — \`transactor.run(async (repos) => ...)\` | manual |
 | 20 | Conectores limpios: kebab-case, con prefijo \`connect\`, registrados en composition-root | \`arckode analyze\` |
 | 21 | Si el service repite filtros 3+ veces o necesita JOIN/IN/LIKE → crear \`repository.ts\` | manual |
-| 22 | \`model.ts\` separado de \`types.ts\` (schema DB ≠ contrato TS) | manual |
+| 22 | \`model.ts\` separado de \`types.ts\` (schema DB ≠ contrato TS) | \`arckode analyze\` |
+| 23 | Un DEFAULT por caso. Escalar SOLO cuando la condición lo justifica | \`arckode analyze\` |
 
 ---
 
@@ -152,11 +153,22 @@ seeds/                   ← datos de prueba
 migrations/              ← migraciones explícitas de DB
 \`\`\`
 
+## Escalation ladder — un DEFAULT por caso
+
+\`\`\`
+Capa de datos:       OrmRepository<T>  →  repository.ts custom  →  Transactor
+Lógica de negocio:   service.ts        →  usecases/{caso}.ts (nunca para CRUD)
+Entre módulos:       Conector          →  EventBus (2+ módulos reaccionan)
+DB Adapter:          SqliteAdapter     →  MySQL / Postgres (concurrencia alta)
+\`\`\`
+
+Escalar SOLO cuando la condición lo justifica. Si no hay condición → nivel 1.
+
 ## Comandos útiles
 \`\`\`bash
 bun run dev                              # desarrollo con hot reload
-bun run analyze                          # verificar arquitectura (consola)
-bun run analyze:json                     # generar/actualizar arckode.json
+bun run analyze                          # verificar arquitectura + actualiza arckode.json
+bun run analyze:ci                       # modo CI: exit 1 si hay violaciones
 bun arckode make:module Nombre           # generar módulo completo
 bun arckode make:connector nombre-x mod1 mod2  # conectar dos módulos (kebab-case)
 bun test                                 # correr todos los tests
@@ -166,9 +178,10 @@ bun test                                 # correr todos los tests
 
 ## Notas para la IA
 
-- Si el módulo es CRUD simple: NO crees \`repository.ts\` ni \`usecases/\`. Pasás \`OrmRepository<T>\` directo al service.
-- Si el módulo tiene 3+ tablas: definí TODOS los modelos en \`model.ts\` y registralos en \`registerXxxModels(orm)\` (helper único). El \`index.ts\` queda limpio.
-- Si necesitás atomicidad: inyectá \`Transactor\` al service, no el ORM. Mantiene la dependency inversion.
-- Si el analyzer muestra \`LEGACY_ACTIONS_FOLDER\`, \`DUPLICATE_CONNECTOR\` o \`UNREGISTERED_CONNECTOR\`: arreglarlos ANTES de pushear código nuevo.
+- CRUD simple: usá \`OrmRepository<T>\` directo. NO crees \`repository.ts\` ni \`usecases/\`.
+- 3+ tablas en un módulo: definí todos los modelos en \`model.ts\`, registralos en \`registerXxxModels(orm)\`.
+- Atomicidad multi-tabla: inyectá \`Transactor\` al service, no el ORM.
+- Antes de escribir código: revisá \`arckode.json\` (violations) y \`composition-root.ts\` (estado actual).
+- Si el analyzer muestra \`LEGACY_ACTIONS_FOLDER\`, \`DUPLICATE_CONNECTOR\`, \`UNREGISTERED_CONNECTOR\`, \`MODEL_IN_TYPES_FILE\` o \`MISSING_MODEL_TS\`: arreglarlos ANTES de agregar código nuevo.
 `
 }

package/cli/stubs/module-stub.ts

@@ -487,9 +487,7 @@ ${sampleData.replace(`'${p.name} de ejemplo'`, `'${p.name} de ejemplo 2'`).repla
     },
   ]
 
-  for (const item of items) {
-    await orm.create('${p.name}', item)
-  }
+  await Promise.all(items.map(item => orm.create('${p.name}', item)))
 
   console.log('  ✓ ${p.name} seeded: ' + items.length + ' items')
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "arckode-framework",
-  "version": "1.1.2",
+  "version": "1.2.0",
   "description": "AI-first TypeScript/Bun framework. Modular, SOLID, zero magic. The AI reads the composition root and knows everything.",
   "type": "module",
   "main": "./kernel/framework.ts",
@@ -60,7 +60,7 @@
   "peerDependencies": {
     "mysql2": ">=3.0.0",
     "pg": "^8.0.0",
-    "redis": ">=4.0.0",
+    "redis": "",
     "nodemailer": ">=6.0.0"
   },
   "peerDependenciesMeta": {
@@ -83,6 +83,7 @@
     "@types/pg": "^8.11.0",
     "bun-types": "^1.3.14",
     "pg": "^8.0.0",
+    "redis": ">=4.0.0",
     "typescript": "^5.6.0"
   },
   "keywords": [