arckode-framework 1.1.0 → 1.1.2

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/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
@@ -1252,6 +1269,55 @@ export interface ServerAdapter {
   stop(): Promise<void>
 }
 
+// ── Envelope estándar de respuesta API ──────────────────────────
+// Todas las respuestas siguen este contrato. Un solo lugar para cambiar el formato.
+// Inspirado en JSend + convenciones de GitHub/Laravel:
+//   success → { success: true,  data: T,    meta: { pagination? } | null, error: null }
+//   error   → { success: false, data: null, meta: null,               error: { code, message, details } }
+export interface ApiResponse<T = unknown> {
+  success: boolean
+  data: T | null
+  meta: { pagination?: unknown } | null
+  error: { code: string; message: string; details?: unknown } | null
+}
+
+function buildEnvelope(status: number, body: unknown): string {
+  // Errores 4xx / 5xx
+  if (status >= 400) {
+    const b = (body ?? {}) as Record<string, unknown>
+    return JSON.stringify({
+      success: false,
+      data: null,
+      meta: null,
+      error: {
+        code:    b.code    ?? 'ERROR',
+        message: b.error   ?? 'Error',
+        details: b.details ?? null,
+      },
+    } satisfies ApiResponse)
+  }
+
+  // Sin contenido (DELETE 204)
+  if (body === null || body === undefined) {
+    return JSON.stringify({ success: true, data: null, meta: null, error: null } satisfies ApiResponse)
+  }
+
+  const b = body as Record<string, unknown>
+
+  // Lista paginada — el service retorna { data: [], pagination: {} }
+  if (Array.isArray(b.data) && b.pagination !== undefined) {
+    return JSON.stringify({
+      success: true,
+      data: b.data,
+      meta: { pagination: b.pagination },
+      error: null,
+    } satisfies ApiResponse)
+  }
+
+  // Recurso único o cualquier otro payload exitoso
+  return JSON.stringify({ success: true, data: body, meta: null, error: null } satisfies ApiResponse)
+}
+
 export class NodeServer implements ServerAdapter {
   private server?: ReturnType<typeof createNodeServer>
   private maxBodyBytes: number
@@ -1314,8 +1380,10 @@ export class NodeServer implements ServerAdapter {
 
             // ── Respuesta normal (JSON o binario comprimido) ──
             const isBuffer = Buffer.isBuffer(res.body)
-            const responseBody = isBuffer ? res.body : JSON.stringify(res.body)
-            nodeRes.writeHead(res.status, {
+            // 204 prohíbe body por spec HTTP — con envelope usamos 200 para "no content"
+            const effectiveStatus = res.status === 204 ? 200 : res.status
+            const responseBody = isBuffer ? res.body : buildEnvelope(effectiveStatus, res.body)
+            nodeRes.writeHead(effectiveStatus, {
               'Content-Type': 'application/json',
               'X-Request-Id': req.id,
               ...res.headers,
@@ -1325,13 +1393,13 @@ export class NodeServer implements ServerAdapter {
             const httpStatus = (error as any)?.httpStatus
             if (httpStatus) {
               nodeRes.writeHead(httpStatus, { 'Content-Type': 'application/json' })
-              nodeRes.end(JSON.stringify({ error: (error as Error).message, code: 'REQUEST_ERROR' }))
+              nodeRes.end(buildEnvelope(httpStatus, { error: (error as Error).message, code: 'REQUEST_ERROR' }))
               return
             }
             const stack = error instanceof Error ? error.stack : String(error)
             this.logger.error('Error no manejado en HTTP', { error: String(error), stack })
             nodeRes.writeHead(500, { 'Content-Type': 'application/json' })
-            nodeRes.end(JSON.stringify({ error: 'Error interno', code: 'INTERNAL_ERROR' }))
+            nodeRes.end(buildEnvelope(500, { error: 'Error interno', code: 'INTERNAL_ERROR' }))
           } finally {
             this.activeRequests--
           }
@@ -1551,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,
     )
@@ -1968,6 +2036,6 @@ export default {
   // Seeds
   SeedRunner,
 
-  // Types (re-exported for convenience)
-  // All interfaces are exported above
+  // Response envelope
+  // ApiResponse interface is exported as named export above
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "arckode-framework",
-  "version": "1.1.0",
+  "version": "1.1.2",
   "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",
     "nodemailer": ">=6.0.0"
   },
   "peerDependenciesMeta": {
-    "better-sqlite3": {
-      "optional": true
-    },
     "mysql2": {
       "optional": true
     },
@@ -82,7 +78,6 @@
     }
   },
   "devDependencies": {
-    "@types/better-sqlite3": "^7.6.12",
     "@types/jsonwebtoken": "^9.0.7",
     "@types/nodemailer": "^6.4.0",
     "@types/pg": "^8.11.0",