Skip to content

chore(vendor): contribuir los SKIMA patches del adapter bun:sqlite al upstream #99

Description

@henfrydls

server/vendor/prisma-bun-sqlite-adapter/ (fork de @synapsenwerkstatt/prisma-bun-sqlite-adapter v1.2.2, MIT) acumula 4 parches marcados SKIMA PATCH que beneficiarían al upstream:

  1. Fechas como epoch-ms INTEGER en mapQueryArgs (paridad con timestampFormat: 'unixepoch-ms' del adapter oficial better-sqlite3) — sin esto, DBs creadas por el engine Rust se corrompen con formatos mixtos.
  2. Null guard al inicio de mapQueryArgsnull en args Int?/Float? se volvía NaN vía parseInt.
  3. Tablas schema-qualified en getTableNameFromQuery — el query compiler de Prisma emite FROM \main`.`Tabla`; el regex upstream capturaba main`, el PRAGMA devolvía vacío y todas las lecturas DateTime salían null (bug crítico cazado en la validación de v1.5.0-rc.4).
  4. PRAGMA con identificador quoted.

Tareas

  • Abrir PR(s) upstream (https://github.com/nogo/prisma-bun-sqlite-adapter) con los parches + los tests de src/__tests__/dbAdapter.test.js.
  • Si upstream los acepta y publica, evaluar des-vendorear (volver a la dependencia npm) — requiere que soporte Prisma 6 o que ya estemos en Prisma 7.
  • Nota de robustez pendiente (review M11): la detección de tipos por regex+PRAGMA sigue siendo heurística y puede fallar en joins/CTEs/agregados; el smoke de date-round-trip en CI lo vigila, pero una detección basada en metadata real del statement sería más sólida.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions