arckode-framework 1.1.1 → 1.2.0

package/adapters/sqlite.ts

@@ -1,8 +1,8 @@
-// adapters/sqlite.ts — Adapter SQLite
-// Responsabilidad ÚNICA: traducir llamadas de DbAdapter a better-sqlite3
+// adapters/sqlite.ts — Adapter SQLite (bun:sqlite)
+// Responsabilidad ÚNICA: traducir llamadas de DbAdapter a bun:sqlite
 // SOLID: implementa DbAdapter, puede reemplazarse por PostgresAdapter sin cambiar el ORM
 
-import Database from 'better-sqlite3'
+import { Database } from 'bun:sqlite'
 import type { DbAdapter } from '../kernel/framework'
 
 interface SqliteConfig {
@@ -11,7 +11,6 @@ interface SqliteConfig {
   foreignKeys?: boolean
 }
 
-// Configuración con valores por defecto
 const DEFAULT_CONFIG: SqliteConfig = {
   path: './data/db.sqlite',
   wal: true,
@@ -19,7 +18,7 @@ const DEFAULT_CONFIG: SqliteConfig = {
 }
 
 export class SqliteAdapter implements DbAdapter {
-  private db!: Database.Database
+  private db!: Database
   private config: SqliteConfig
 
   constructor(config?: Partial<SqliteConfig>) {
@@ -29,19 +28,21 @@ export class SqliteAdapter implements DbAdapter {
   async connect(): Promise<void> {
     this.db = new Database(this.config.path)
 
-    if (this.config.wal) this.db.pragma('journal_mode = WAL')
-    if (this.config.foreignKeys) this.db.pragma('foreign_keys = ON')
+    if (this.config.wal) this.db.exec('PRAGMA journal_mode = WAL')
+    if (this.config.foreignKeys) this.db.exec('PRAGMA foreign_keys = ON')
   }
 
   query(sql: string, params: unknown[] = []): Promise<unknown[]> {
-    const stmt = this.db.prepare(sql)
-    const rows = params.length > 0 ? stmt.all(...params) : stmt.all()
-    return Promise.resolve(rows)
+    const stmt = this.db.query(sql)
+    const rows = params.length > 0 ? stmt.all(...(params as Parameters<typeof stmt.all>)) : stmt.all()
+    return Promise.resolve(rows as unknown[])
   }
 
   run(sql: string, params: unknown[] = []): Promise<{ changes: number; lastId?: string }> {
-    const stmt = this.db.prepare(sql)
-    const result = params.length > 0 ? stmt.run(...params) : stmt.run()
+    const stmt = this.db.query(sql)
+    const result = params.length > 0
+      ? stmt.run(...(params as Parameters<typeof stmt.run>))
+      : stmt.run()
     return Promise.resolve({
       changes: result.changes,
       lastId: result.lastInsertRowid ? String(result.lastInsertRowid) : undefined,
@@ -49,21 +50,15 @@ export class SqliteAdapter implements DbAdapter {
   }
 
   async transaction<T>(fn: (adapter: DbAdapter) => Promise<T>): Promise<T> {
-    // better-sqlite3 usa transacciones síncronas — necesitamos un wrapper async
-    return new Promise((resolve, reject) => {
-      const txFn = this.db.transaction((args: unknown[]) => args[0])
-      // Ejecutar la función dentro de BEGIN/COMMIT de better-sqlite3
-      this.db.exec('BEGIN')
-      fn(this)
-        .then((result) => {
-          this.db.exec('COMMIT')
-          resolve(result)
-        })
-        .catch((err) => {
-          try { this.db.exec('ROLLBACK') } catch { /* ya hizo rollback */ }
-          reject(err)
-        })
-    })
+    this.db.exec('BEGIN')
+    try {
+      const result = await fn(this)
+      this.db.exec('COMMIT')
+      return result
+    } catch (err) {
+      try { this.db.exec('ROLLBACK') } catch { /* ya hizo rollback */ }
+      throw err
+    }
   }
 
   close(): Promise<void> {

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/kernel/framework.ts

@@ -896,6 +896,23 @@ export class ORM {
         } catch {
           // Error esperado: la columna ya existe — ignorar
         }
+
+        // 3c. Corregir drift de nullability: si el modelo dice nullable pero la columna es NOT NULL, quitarlo
+        // Solo PostgreSQL soporta ALTER COLUMN ... DROP NOT NULL de forma confiable
+        if (field.nullable === true) {
+          try {
+            await this.db.run(`ALTER TABLE ${def.table} ALTER COLUMN ${name} DROP NOT NULL`)
+          } catch { /* SQLite no soporta esto — ignorar */ }
+        }
+      }
+
+      // 3b. Agregar columnas de timestamps/softDelete si el modelo las requiere pero no existen aún
+      if (def.timestamps) {
+        try { await this.db.run(`ALTER TABLE ${def.table} ADD COLUMN createdAt TEXT`) } catch { /* ya existe */ }
+        try { await this.db.run(`ALTER TABLE ${def.table} ADD COLUMN updatedAt TEXT`) } catch { /* ya existe */ }
+      }
+      if (def.softDelete) {
+        try { await this.db.run(`ALTER TABLE ${def.table} ADD COLUMN deletedAt TEXT`) } catch { /* ya existe */ }
       }
 
       // 4. Detectar drift de schema: columnas en la BD que ya no están en el modelo
@@ -1602,7 +1619,7 @@ export class Auth {
 
   createRefreshToken(payload: { id: string; role: string }): string {
     const token = this.jwt.sign(
-      { id: payload.id, role: payload.role, type: 'refresh' },
+      { id: payload.id, role: payload.role, type: 'refresh', jti: crypto.randomUUID() },
       this.refreshSecret,
       this.refreshExpiresIn,
     )

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "arckode-framework",
-  "version": "1.1.1",
+  "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",
@@ -58,16 +58,12 @@
     "jsonwebtoken": "^9.0.0"
   },
   "peerDependencies": {
-    "better-sqlite3": ">=11.0.0",
     "mysql2": ">=3.0.0",
     "pg": "^8.0.0",
-    "redis": ">=4.0.0",
+    "redis": "",
     "nodemailer": ">=6.0.0"
   },
   "peerDependenciesMeta": {
-    "better-sqlite3": {
-      "optional": true
-    },
     "mysql2": {
       "optional": true
     },
@@ -82,12 +78,12 @@
     }
   },
   "devDependencies": {
-    "@types/better-sqlite3": "^7.6.12",
     "@types/jsonwebtoken": "^9.0.7",
     "@types/nodemailer": "^6.4.0",
     "@types/pg": "^8.11.0",
     "bun-types": "^1.3.14",
     "pg": "^8.0.0",
+    "redis": ">=4.0.0",
     "typescript": "^5.6.0"
   },
   "keywords": [