@drzl/analyzer 1.21.1 → 1.21.3

package/dist/index.cjs

@@ -48,6 +48,12 @@ function qualifiedTableName(table) {
 function qualifiedForeignTable(fk) {
   return fk.foreignSchema ? `${fk.foreignSchema}.${fk.foreignTable}` : fk.foreignTable;
 }
+function temporalTextFormat(entityKind, kind) {
+  const pg = entityKind.startsWith("Pg");
+  const mysql = entityKind.startsWith("MySql");
+  if (kind === "stamp") return pg || mysql ? "temporalText" : void 0;
+  return pg ? "temporalText" : void 0;
+}
 function renderSqlLiteral(v) {
   if (v === null || v === void 0) return "NULL";
   if (typeof v === "number" || typeof v === "bigint") return String(v);
@@ -221,6 +227,7 @@ function describeV1Column(column) {
     case "date":
       out.tsType = js === "string" ? "string" : "Date";
       out.dbType = codec?.startsWith("timestamp") ? "TIMESTAMP" : "DATE";
+      if (out.tsType === "string") out.format = temporalTextFormat(entityKind, "stamp");
       break;
     case "timestamp":
     // `datetime` is the same fact under MySQL's name for it, and it had no arm, so every column
@@ -235,14 +242,17 @@ function describeV1Column(column) {
     case "datetime":
       out.tsType = js === "string" ? "string" : "Date";
       out.dbType = "TIMESTAMP";
+      if (out.tsType === "string") out.format = temporalTextFormat(entityKind, "stamp");
       break;
     case "time":
       out.tsType = "string";
       out.dbType = "TIME";
+      out.format = temporalTextFormat(entityKind, "time");
       break;
     case "interval":
       out.tsType = "string";
       out.dbType = "INTERVAL";
+      out.format = temporalTextFormat(entityKind, "interval");
       break;
     case "inet":
     case "cidr":
@@ -719,6 +729,10 @@ var _SchemaAnalyzer = class _SchemaAnalyzer {
       out.integer = true;
     }
     if (/^(Pg)?UUID$/i.test(ctor) || /Uuid$/i.test(ctor)) out.format = "uuid";
+    if (/^Pg(TimestampString|DateString|Time|Interval)$/.test(ctor)) out.format = "temporalText";
+    if (/^MySql(TimestampString|DateTimeString|DateString)$/.test(ctor)) {
+      out.format = "temporalText";
+    }
     return out;
   }
   mapColumnType(column) {

package/dist/index.d.cts

@@ -156,11 +156,34 @@ interface Column {
      * `declaredDecimalRange` reads both numbers now, the two are distinguishable, and each says what
      * its own server does.
      *
-     * Postgres and Gel. MySQL refuses all three on a `float`/`double`, and on a `decimal` refuses
-     * them outright rather than storing `0.00`: measured on MySQL 8.4.11 in `STRICT_TRANS_TABLES`,
-     * all three answer `Incorrect decimal value`. SQLite returns both infinities and silently turns
-     * `NaN` into NULL, which is real and is filed on its own: a column needs both halves of that
-     * answer or none.
+     * Postgres and Gel. MySQL is a `false` on every one of these, and the reason is worth writing
+     * down carefully, because the two client paths give different answers and only one of them is
+     * the path a driver takes.
+     *
+     * Measured on MySQL 8.4.11 in `STRICT_TRANS_TABLES`, through mysql2's `execute()`, which is the
+     * binary prepared path drizzle uses:
+     *
+     *   float, double     all three refused, `Out of range value`
+     *   decimal(10,2)     all three STORED AS 0.00, silently, `SHOW WARNINGS` empty
+     *   int               `NaN` stored as 0 silently; both infinities refused
+     *   bigint            `NaN` stored as the int64 minimum silently; both infinities refused
+     *
+     * A control rules out ordinary overflow as the explanation: a finite `1e308` into the same
+     * `decimal(10,2)` is refused. So on three of those five, the server takes the row and stores a
+     * number nobody sent. This comment used to say a decimal "refuses them outright", which is the
+     * *text* path's answer: through the `mysql` CLI the same value answers `Incorrect decimal value`.
+     * Both statements are true of their own path, and only one of them describes what a validator
+     * sits in front of.
+     *
+     * The flag stays `false` on all of them, so the emitted schemas refuse all three. That is
+     * deliberately not the "never be stricter than the server" rule the range work follows: the
+     * server is not accepting the value here, it is accepting the row and storing a different value,
+     * and a validator whose whole job is to say what the database will do with a write cannot call
+     * that acceptance. Nothing is lost either way, since every generator's plain number type already
+     * refuses the three.
+     *
+     * SQLite returns both infinities and silently turns `NaN` into NULL, which is real and is filed
+     * on its own: a column needs both halves of that answer or none.
      *
      * Gel joined on a measurement of its own rather than on being Postgres-backed: a live Gel 7.1
      * stored `nan`, `inf` and `-inf` in both `std::float32` and `std::float64` and handed all three
@@ -185,7 +208,7 @@ interface Column {
      * `'1_000'` included, and by MySQL as a decimal number it then rounds, so `'12.5'` is a row on
      * one server and an error on the other.
      */
-    format?: 'uuid' | 'numeric' | 'pgBigint' | 'mysqlBigint';
+    format?: 'uuid' | 'numeric' | 'pgBigint' | 'mysqlBigint' | 'temporalText';
     /**
      * The column's default, when it is a literal a schema can reproduce.
      *

package/dist/index.d.ts

@@ -156,11 +156,34 @@ interface Column {
      * `declaredDecimalRange` reads both numbers now, the two are distinguishable, and each says what
      * its own server does.
      *
-     * Postgres and Gel. MySQL refuses all three on a `float`/`double`, and on a `decimal` refuses
-     * them outright rather than storing `0.00`: measured on MySQL 8.4.11 in `STRICT_TRANS_TABLES`,
-     * all three answer `Incorrect decimal value`. SQLite returns both infinities and silently turns
-     * `NaN` into NULL, which is real and is filed on its own: a column needs both halves of that
-     * answer or none.
+     * Postgres and Gel. MySQL is a `false` on every one of these, and the reason is worth writing
+     * down carefully, because the two client paths give different answers and only one of them is
+     * the path a driver takes.
+     *
+     * Measured on MySQL 8.4.11 in `STRICT_TRANS_TABLES`, through mysql2's `execute()`, which is the
+     * binary prepared path drizzle uses:
+     *
+     *   float, double     all three refused, `Out of range value`
+     *   decimal(10,2)     all three STORED AS 0.00, silently, `SHOW WARNINGS` empty
+     *   int               `NaN` stored as 0 silently; both infinities refused
+     *   bigint            `NaN` stored as the int64 minimum silently; both infinities refused
+     *
+     * A control rules out ordinary overflow as the explanation: a finite `1e308` into the same
+     * `decimal(10,2)` is refused. So on three of those five, the server takes the row and stores a
+     * number nobody sent. This comment used to say a decimal "refuses them outright", which is the
+     * *text* path's answer: through the `mysql` CLI the same value answers `Incorrect decimal value`.
+     * Both statements are true of their own path, and only one of them describes what a validator
+     * sits in front of.
+     *
+     * The flag stays `false` on all of them, so the emitted schemas refuse all three. That is
+     * deliberately not the "never be stricter than the server" rule the range work follows: the
+     * server is not accepting the value here, it is accepting the row and storing a different value,
+     * and a validator whose whole job is to say what the database will do with a write cannot call
+     * that acceptance. Nothing is lost either way, since every generator's plain number type already
+     * refuses the three.
+     *
+     * SQLite returns both infinities and silently turns `NaN` into NULL, which is real and is filed
+     * on its own: a column needs both halves of that answer or none.
      *
      * Gel joined on a measurement of its own rather than on being Postgres-backed: a live Gel 7.1
      * stored `nan`, `inf` and `-inf` in both `std::float32` and `std::float64` and handed all three
@@ -185,7 +208,7 @@ interface Column {
      * `'1_000'` included, and by MySQL as a decimal number it then rounds, so `'12.5'` is a row on
      * one server and an error on the other.
      */
-    format?: 'uuid' | 'numeric' | 'pgBigint' | 'mysqlBigint';
+    format?: 'uuid' | 'numeric' | 'pgBigint' | 'mysqlBigint' | 'temporalText';
     /**
      * The column's default, when it is a literal a schema can reproduce.
      *

package/dist/index.js

@@ -5,6 +5,12 @@ function qualifiedTableName(table) {
 function qualifiedForeignTable(fk) {
   return fk.foreignSchema ? `${fk.foreignSchema}.${fk.foreignTable}` : fk.foreignTable;
 }
+function temporalTextFormat(entityKind, kind) {
+  const pg = entityKind.startsWith("Pg");
+  const mysql = entityKind.startsWith("MySql");
+  if (kind === "stamp") return pg || mysql ? "temporalText" : void 0;
+  return pg ? "temporalText" : void 0;
+}
 function renderSqlLiteral(v) {
   if (v === null || v === void 0) return "NULL";
   if (typeof v === "number" || typeof v === "bigint") return String(v);
@@ -178,6 +184,7 @@ function describeV1Column(column) {
     case "date":
       out.tsType = js === "string" ? "string" : "Date";
       out.dbType = codec?.startsWith("timestamp") ? "TIMESTAMP" : "DATE";
+      if (out.tsType === "string") out.format = temporalTextFormat(entityKind, "stamp");
       break;
     case "timestamp":
     // `datetime` is the same fact under MySQL's name for it, and it had no arm, so every column
@@ -192,14 +199,17 @@ function describeV1Column(column) {
     case "datetime":
       out.tsType = js === "string" ? "string" : "Date";
       out.dbType = "TIMESTAMP";
+      if (out.tsType === "string") out.format = temporalTextFormat(entityKind, "stamp");
       break;
     case "time":
       out.tsType = "string";
       out.dbType = "TIME";
+      out.format = temporalTextFormat(entityKind, "time");
       break;
     case "interval":
       out.tsType = "string";
       out.dbType = "INTERVAL";
+      out.format = temporalTextFormat(entityKind, "interval");
       break;
     case "inet":
     case "cidr":
@@ -676,6 +686,10 @@ var _SchemaAnalyzer = class _SchemaAnalyzer {
       out.integer = true;
     }
     if (/^(Pg)?UUID$/i.test(ctor) || /Uuid$/i.test(ctor)) out.format = "uuid";
+    if (/^Pg(TimestampString|DateString|Time|Interval)$/.test(ctor)) out.format = "temporalText";
+    if (/^MySql(TimestampString|DateTimeString|DateString)$/.test(ctor)) {
+      out.format = "temporalText";
+    }
     return out;
   }
   mapColumnType(column) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@drzl/analyzer",
-  "version": "1.21.1",
+  "version": "1.21.3",
   "private": false,
   "license": "Apache-2.0",
   "type": "module",