@drzl/cli 4.35.0 → 4.36.0

package/dist/cli.js

@@ -2182,6 +2182,24 @@ function sqlLiterals(values) {
 function derivedName(table, column) {
   return `${table}_${column}_check`;
 }
+var ADD_CONSTRAINT_DIALECTS = /* @__PURE__ */ new Set([
+  "postgres",
+  "cockroach",
+  "gel",
+  "mysql",
+  "singlestore",
+  "mssql"
+]);
+function fixFor(dialect, table, column, values) {
+  if (!ADD_CONSTRAINT_DIALECTS.has(dialect)) return void 0;
+  return `ALTER TABLE ${table} ADD CONSTRAINT ${derivedName(table, column)} CHECK (${column} IN (${sqlLiterals(values)}));`;
+}
+function noFixReason(dialect) {
+  if (dialect === "sqlite") {
+    return "SQLite cannot add a CHECK to an existing column: ALTER TABLE ... ADD CONSTRAINT is a syntax error there. Closing this means rebuilding the table, which is the twelve-step procedure SQLite documents: create the new table with the constraint, copy the rows, drop the old one, rename.";
+  }
+  return `no statement is emitted for the "${dialect}" dialect, which this report cannot name a fix for`;
+}
 function buildConstraintDriftReport(analysis, schemaPath) {
   const entries = [];
   for (const table of analysis.tables) {
@@ -2217,7 +2235,10 @@ function buildConstraintDriftReport(analysis, schemaPath) {
         columns: [column.name],
         rule: `${column.name} IN (${sqlLiterals(values)})`,
         reason: `the column is declared ${column.sqlType}, which holds any text; the set exists only in the generated schemas`,
-        fix: `ALTER TABLE ${table.name} ADD CONSTRAINT ${derivedName(table.name, column.name)} CHECK (${column.name} IN (${sqlLiterals(values)}));`
+        ...(() => {
+          const fix = fixFor(analysis.dialect, table.name, column.name, values);
+          return fix ? { fix } : { noFix: noFixReason(analysis.dialect) };
+        })()
       });
     }
   }
@@ -2292,6 +2313,8 @@ function renderConstraintDriftReport(report, style = PLAIN3) {
       if (e.fix) {
         out.push(chalk.dim("      Close it with:"));
         out.push(`      ${e.fix}`);
+      } else if (e.noFix) {
+        out.push(chalk.dim(wrap3(e.noFix, "      ", "      Not one statement here: ")));
       }
       out.push("");
     }
@@ -2323,6 +2346,43 @@ function renderConstraintDriftReport(report, style = PLAIN3) {
   );
   return out.join("\n");
 }
+function renderConstraintDriftSql(report) {
+  const gaps = report.entries.filter((e) => e.side === "schema-only");
+  if (!gaps.length) return "";
+  const out = [];
+  const runnable = gaps.filter((g) => g.fix);
+  const unrunnable = gaps.filter((g) => !g.fix);
+  out.push(`-- Generated by drzl doctor --constraints --sql`);
+  out.push(`-- ${report.schema}, ${report.dialect}`);
+  out.push(
+    `-- ${runnable.length} constraint(s) your schemas enforce and the database does not.`
+  );
+  if (unrunnable.length) {
+    out.push(
+      `-- ${unrunnable.length} more cannot be closed by one statement on this dialect; see below.`
+    );
+  }
+  out.push("");
+  const byPlace = (a, b) => a.table.localeCompare(b.table) || a.columns.join().localeCompare(b.columns.join());
+  for (const g of [...runnable].sort(byPlace)) {
+    out.push(`-- ${g.table}.${g.columns.join(", ")}: ${g.reason}`);
+    out.push(g.fix);
+    out.push("");
+  }
+  if (unrunnable.length) {
+    out.push("-- The rest, which this dialect cannot do in one statement:");
+    for (const g of [...unrunnable].sort(byPlace)) {
+      out.push(`--`);
+      out.push(`--   ${g.table}.${g.columns.join(", ")}`);
+      out.push(`--   ${g.rule}`);
+      for (const line of (g.noFix ?? "").match(/.{1,88}(\s|$)/g) ?? []) {
+        out.push(`--   ${line.trim()}`);
+      }
+    }
+    out.push("");
+  }
+  return out.join("\n");
+}
 
 // src/drift.ts
 import { promises as fs2 } from "fs";
@@ -3249,7 +3309,7 @@ withOutputFlags(
   program.command("doctor").description("Report what DRZL cannot type or enforce in your schema, and why").argument("[schema]", "path to drizzle schema (TS); defaults to the schema in drzl.config").option("-c, --config <path>", "path to drzl.config, read when no schema argument is given").option("--strict", "exit 2 when anything is reported", false).option(
     "--constraints",
     "report what each side enforces that the other does not, instead of the usual findings"
-  )
+  ).option("--sql", "with --constraints, emit the statements alone, for redirecting to a migration")
 ).action(async (schema, opts) => {
   const out = outputFor(opts);
   {
@@ -3272,6 +3332,13 @@ withOutputFlags(
         validateConstraints: true
       });
       const schemaLabel = Array.isArray(target) ? target.join(", ") : target;
+      if (opts.sql && !opts.constraints) {
+        const msg = "--sql reports the constraint drift as statements, so it needs --constraints.";
+        if (opts.json) out.jsonData(jsonFailure("doctor", "DRZL_CLI_DOCTOR", msg));
+        else out.error("Doctor failed (DRZL_CLI_DOCTOR):", msg);
+        process.exit(EXIT_FAILED);
+        return;
+      }
       if (opts.constraints) {
         const preflight = buildDoctorReport(analysis, schemaLabel);
         const fatal = preflight.findings.filter((f) => f.level === "error");
@@ -3290,6 +3357,11 @@ withOutputFlags(
         }
         const drift = buildConstraintDriftReport(analysis, schemaLabel);
         const driftCode = opts.strict && drift.counts.schemaOnly ? EXIT_FINDINGS : EXIT_OK;
+        if (opts.sql) {
+          out.data(renderConstraintDriftSql(drift));
+          process.exit(driftCode);
+          return;
+        }
         if (opts.json)
           out.data(
             JSON.stringify({ command: "doctor", exitCode: driftCode, ...drift }, null, 2)