panels-origin/api/pg.ts
Cursor Agent 493829d028
api: migrar todo el backend de SQLite a Postgres + Redis (fase 2-4e)
Fase 2 (driver):
- api/pg.ts: adaptador delgado sobre postgres.js (prepare/get/all/run,
  placeholders ? -> $n, withTenant con set_config para RLS), con parsers
  de tipo custom (numeric/date/timestamp(tz)/bigint) para que el resto
  del codigo heredado de SQLite (fechas/montos como string, ids como
  number) siga funcionando sin reescribir cada call-site a mano.
- api/platform_db.ts, api/iam_db.ts (nuevo), api/db.ts: pools separados
  por base/esquema (panels_platform, panels_product.iam,
  panels_product.core), owner pool para bootstrap/scripts/lookups
  administrativos que cruzan tenant a proposito.
- api/redis.ts: clientes iam/core separados (ACL panels_iam_redis /
  panels_core_redis).
- api/sessions.ts + auth.ts: sesiones ahora en Redis (cookie = id opaco,
  no HMAC autocontenido); revocacion real (logout, cambio de password).
- api/storage.ts (Fase 4c): documentos/PDFs via Contabo Object Storage
  (S3), con fallback a disco local si no hay credenciales S3 (dev).
- api/scope.ts: middleware withCoreScope/requireCoreAuth que abre la
  transaccion con app.tenant_id fijado (RLS) para cada request.
- api/cache.ts (Fase 4e): cache Redis con tenant_id obligatorio en la
  llave; aplicado a /v1/catalogs.

Fase 3 (reescritura SQL, ~80 endpoints en main.ts/companies.ts/budget.ts/
payroll.ts/payroll_http.ts/excel.ts/saas.ts/smtp.ts):
- Todo async/await, sintaxis Postgres (COALESCE, ~ regex, ON CONFLICT,
  now()/current_date, booleanos reales, RETURNING via lastInsertId()).
- IDOR cross-tenant cerrado: GET/PATCH /v1/projects/:id, /v1/workers/:id
  ya no dependen de que el handler recuerde el WHERE tenant_id -- Row
  Level Security lo hace estructuralmente (verificado con un segundo
  tenant real: 404 en vez de fuga de datos).
- API key ya no ve todos los tenants: ahora exige X-Tenant-Id explicito.

Fase 3b (tests): api/test_helpers.ts corre cada test en una transaccion
que siempre se revierte, contra el mismo baseline de Liquibase que
produccion (ya no un esquema SQLite escrito a mano). payroll_test.ts
reescrito con fixtures reales; 11/11 pasan contra Postgres.

Fase 4 (IAM/RBAC): iam.roles/permissions/role_permissions formalizados
(ver db/iam ya en fase 1); uploaded_by/created_by ahora son snapshot
desnormalizado (uploaded_by_id/name); seed() en runtime eliminado,
reemplazado por scripts/bootstrap-admin.ts (one-shot).

Fase 4d (zona horaria): nuevo endpoint /v1/configuracion (GET/PUT),
PAYROLL_TZ hardcodeado reemplazado por tenant_settings.timezone,
document_validity.ts ya no usa new Date() crudo.

Verificado end-to-end contra Postgres+Redis reales: login, sesiones,
catalogos con cache, alta de trabajador, subida/descarga de documento
cifrado, y el fix de IDOR probado con un segundo tenant real (403/404
en vez de fuga de datos).

Co-authored-by: alberto.martinez <alberto.martinez@mrdev.mx>
2026-09-02 20:47:45 +00:00

138 lines
5.7 KiB
TypeScript

import postgres from "npm:postgres@3";
/**
* Adaptador delgado sobre postgres.js que imita la forma
* `db.prepare(sql).get/all/run(...params)` que tenía el driver SQLite
* (@db/sqlite), para poder migrar ~80 endpoints sin reescribir cada query
* a mano en el mismo cambio que separa las bases/esquemas. Sigue siendo
* 100% parametrizado (nunca concatena valores en el texto SQL) -- lo único
* que cambia es la forma superficial de la llamada, no la seguridad.
*
* Traduce automáticamente placeholders `?` (estilo SQLite) a `$1, $2, ...`
* (estilo Postgres). Todo el acceso es async (a diferencia de better-sqlite3
* style), así que cada call-site necesita `await`.
*/
export type PgRow = Record<string, unknown>;
function toPositional(text: string): string {
let i = 0;
return text.replace(/\?/g, () => `$${++i}`);
}
export class PreparedStatement {
constructor(private sql: postgres.Sql, private text: string) {}
async get(...params: unknown[]): Promise<PgRow | undefined> {
const rows = await this.sql.unsafe(toPositional(this.text), params as never[]);
return rows[0] as PgRow | undefined;
}
async all(...params: unknown[]): Promise<PgRow[]> {
const rows = await this.sql.unsafe(toPositional(this.text), params as never[]);
return rows as unknown as PgRow[];
}
/** Igual que .all/.get pero no espera filas de vuelta (INSERT/UPDATE/DELETE). */
async run(...params: unknown[]): Promise<{ changes: number }> {
const rows = await this.sql.unsafe(toPositional(this.text), params as never[]);
return { changes: rows.count ?? rows.length };
}
}
/** Wrapper con la forma db.prepare()/db.exec() de @db/sqlite, sobre una
* conexión (o conexión reservada) de postgres.js. */
export class PgDb {
constructor(public raw: postgres.Sql) {}
prepare(text: string): PreparedStatement {
return new PreparedStatement(this.raw, text);
}
async exec(text: string): Promise<void> {
await this.raw.unsafe(text);
}
/** Equivalente a last_insert_rowid(): dentro de la MISMA conexión/
* transacción, lastval() devuelve el último valor de secuencia obtenido
* en esta sesión. Solo es seguro si se llama justo después del INSERT
* correspondiente, en la misma conexión reservada por request. */
async lastInsertId(): Promise<number> {
const rows = await this.raw.unsafe("SELECT lastval()::bigint AS id");
return Number((rows[0] as unknown as { id: number | bigint }).id);
}
}
export function createPool(url: string, options: postgres.Options<Record<string, never>> = {}) {
return postgres(url, {
max: 10,
idle_timeout: 30,
connect_timeout: 10,
types: {
// postgres.js devuelve NUMERIC como string por defecto (evita perder
// precisión en valores gigantes, tipo BigDecimal). Aquí los montos
// son jornales/presupuestos -- ya vivían como REAL/float en SQLite,
// así que parseFloat no introduce una regresión de precisión nueva,
// y evita tener que tocar cada call-site que hace aritmética sobre
// columnas NUMERIC (daily_wage, gross, balance, amount, etc.).
numeric: {
to: 1700,
from: [1700],
serialize: (x: number) => String(x),
parse: (x: string) => Number.parseFloat(x),
},
// postgres.js devuelve date/timestamp(tz) como objetos Date por
// defecto. Todo el código heredado de SQLite trata fechas como
// strings (comparaciones lexicográficas, .slice(0,10), template
// literals) -- se fuerzan a texto aquí para no reescribir cada
// call-site de fechas en el mismo cambio que separa las bases.
date: { to: 1082, from: [1082], serialize: (x: string) => x, parse: (x: string) => x },
timestamp: { to: 1114, from: [1114], serialize: (x: string) => x, parse: (x: string) => x },
timestamptz: { to: 1184, from: [1184], serialize: (x: string) => x, parse: (x: string) => x },
// Todas las PK son BIGINT GENERATED ALWAYS AS IDENTITY (pensando en
// escala futura), pero postgres.js devuelve bigint como string por
// defecto (evita perder precisión más allá de Number.MAX_SAFE_INTEGER).
// A esta escala de negocio los ids nunca se acercan a ese límite, y
// el código heredado los trata como number en todas partes (Map<number,...>,
// comparaciones ===, etc.), así que se parsean a Number aquí.
bigint: {
to: 20,
from: [20],
serialize: (x: number) => String(x),
parse: (x: string) => Number(x),
},
},
...options,
});
}
/**
* Ejecuta `fn` dentro de una transacción con `app.tenant_id` fijado vía
* set_config(..., is_local=true) -- así las políticas de Row Level Security
* (db/iam/changesets/002-rls.sql, db/core/changesets/005-rls.sql) filtran
* automáticamente por tenant, y el valor se limpia solo al terminar la
* transacción sin importar qué conexión del pool se reutilice después.
*
* tenantId = null significa "sin tenant" (p. ej. platform_admin o la
* API key legacy): las políticas de RLS son fail-closed, así que sin
* tenant_id fijado NO se ve ninguna fila con tenant_id NOT NULL. Las
* rutas que de verdad necesitan cruzar tenants deben pasar por el rol de
* soporte explícito (ver Fase 4/nota de seguridad sobre el bypass de
* X-API-Key), no por dejar tenantId en null "por si acaso".
*/
export async function withTenant<T>(
sql: postgres.Sql,
tenantId: number | null,
fn: (scoped: PgDb) => Promise<T>,
): Promise<T> {
const result = await sql.begin(async (tx) => {
if (tenantId != null) {
await tx`SELECT set_config('app.tenant_id', ${String(tenantId)}, true)`;
}
return [await fn(new PgDb(tx as unknown as postgres.Sql))] as const;
});
return (result as unknown as [T])[0];
}
export type { postgres };
export default postgres;