@stacksjs/buddy 0.70.201 → 0.70.203

package/dist/commands/migrate.js

@@ -5,6 +5,8 @@ import { Action } from "@stacksjs/enums";
 import { hasTTY, isCI } from "@stacksjs/env";
 import { appPath, frameworkPath, frameworkRuntimePath, projectPath } from "@stacksjs/path";
 import { ExitCode } from "@stacksjs/types";
+import { preflightDatabase } from "../database-preflight";
+import { DIALECT_OVERRIDE_ENV, auditMigrationCorpus, formatMigrationDialectError, stripSqlNoise } from "@stacksjs/database";
 let _runAction;
 async function runAction(...args) {
   if (!_runAction)
@@ -35,6 +37,15 @@ function acquireMigrationLock() {
     };
   }
 }
+async function ensureDatabaseOrExit() {
+  try {
+    const { ensureDatabaseReady } = await import("@stacksjs/database");
+    await ensureDatabaseReady();
+  } catch (error) {
+    log.syncError(error instanceof Error ? error.message : String(error));
+    process.exit(ExitCode.FatalError);
+  }
+}
 function readMigrateMarker() {
   const file = frameworkRuntimePath("last-migrate-result.json");
   if (!existsSync(file))
@@ -73,6 +84,17 @@ function validateModelsExist() {
     };
   return { valid: !0 };
 }
+function validateMigrationDialect() {
+  if (process.env[DIALECT_OVERRIDE_ENV] === "1")
+    return { valid: !0 };
+  const driver = String(process.env.DB_CONNECTION || "sqlite").toLowerCase(), target = driver === "singlestore" ? "mysql" : driver;
+  if (target !== "sqlite" && target !== "postgres" && target !== "mysql")
+    return { valid: !0 };
+  const dir = projectPath("database/migrations"), audit = auditMigrationCorpus({ dir, target });
+  if (audit.empty || audit.incompatible.length === 0)
+    return { valid: !0 };
+  return { valid: !1, error: formatMigrationDialectError(audit, target, "database/migrations") };
+}
 async function reportMissingForeignKeys() {
   try {
     const { auditForeignKeys } = await import("@stacksjs/database"), result = await auditForeignKeys();
@@ -189,12 +211,19 @@ export function migrate(buddy) {
     fromDb: "Diff against the live database schema instead of the snapshot (self-heal drift)",
     noRename: "Treat renamed columns as drop + add instead of a data-preserving rename"
   };
-  buddy.command("migrate", descriptions.migrate).alias("db:migrate").option("-d, --diff", "Show the SQL that would be run", { default: !1 }).option("-p, --project [project]", descriptions.project, { default: !1 }).option("-a, --auth", descriptions.auth, { default: !0 }).option("--no-auth", "Skip auth/oauth table migrations").option("-f, --force", descriptions.force, { default: !1 }).option("--from-db", descriptions.fromDb, { default: !1 }).option("--no-rename", descriptions.noRename).option("--verbose", descriptions.verbose, { default: !1 }).action(async (options) => {
+  buddy.command("migrate", descriptions.migrate).alias("db:migrate").option("-d, --diff", "Show the SQL that would be run", { default: !1 }).option("-p, --project [project]", descriptions.project, { default: !1 }).option("-a, --auth", descriptions.auth, { default: !0 }).option("--no-auth", "Skip auth/oauth table migrations").option("-f, --force", descriptions.force, { default: !1 }).option("--create-database", "Create the database if it does not exist, without asking", { default: !1 }).option("--from-db", descriptions.fromDb, { default: !1 }).option("--no-rename", descriptions.noRename).option("--verbose", descriptions.verbose, { default: !1 }).action(async (options) => {
     log.debug("Running `buddy migrate` ...", options);
     const perf = await intro("buddy migrate"), validation = validateModelsExist();
     if (!validation.valid) {
       console.error(`
 \u274C Error: ${validation.error}
+`);
+      process.exit(ExitCode.FatalError);
+    }
+    const dialectCheck = validateMigrationDialect();
+    if (!dialectCheck.valid) {
+      console.error(`
+\u274C Error: ${dialectCheck.error}
 `);
       process.exit(ExitCode.FatalError);
     }
@@ -219,6 +248,7 @@ export function migrate(buddy) {
       await outro("Diff complete \u2014 no changes applied.", { startTime: perf, useSeconds: !0 });
       process.exit(ExitCode.Success);
     }
+    await preflightDatabase({ createDatabase: options.createDatabase, command: "migrate" });
     if ((await resolveMigrationGuards()).confirmMigrate && !options.force)
       if (isCI || !hasTTY)
         log.debug("[migrate] confirmMigrate guard skipped \u2014 non-interactive environment.");
@@ -233,6 +263,7 @@ export function migrate(buddy) {
           process.exit(ExitCode.Success);
         }
       }
+    await ensureDatabaseOrExit();
     const lock = acquireMigrationLock();
     if (!lock.acquired) {
       log.syncError("Another migration is already running (storage/framework/runtime/migrations.lock exists). Wait for it to finish, or remove the lockfile if it is stale.");
@@ -293,12 +324,19 @@ export function migrate(buddy) {
     });
     process.exit(ExitCode.Success);
   });
-  buddy.command("migrate:fresh", descriptions.fresh).alias("db:fresh").option("-d, --diff", "Show the SQL that would be run", { default: !1 }).option("-p, --project [project]", descriptions.project, { default: !1 }).option("-s, --seed", "Run database seeders after migration", { default: !1 }).option("-a, --auth", descriptions.auth, { default: !0 }).option("--no-auth", "Skip auth/oauth table migrations").option("-f, --force", 'Skip the drop-database confirmation (only honored when the migrateFresh guard is "allow")', { default: !1 }).option("--verbose", descriptions.verbose, { default: !1 }).action(async (options) => {
+  buddy.command("migrate:fresh", descriptions.fresh).alias("db:fresh").option("-d, --diff", "Show the SQL that would be run", { default: !1 }).option("-p, --project [project]", descriptions.project, { default: !1 }).option("-s, --seed", "Run database seeders after migration", { default: !1 }).option("-a, --auth", descriptions.auth, { default: !0 }).option("--no-auth", "Skip auth/oauth table migrations").option("--create-database", "Create the database if it does not exist, without asking", { default: !1 }).option("-f, --force", 'Skip the drop-database confirmation (only honored when the migrateFresh guard is "allow")', { default: !1 }).option("--verbose", descriptions.verbose, { default: !1 }).action(async (options) => {
     log.debug("Running `buddy migrate:fresh` ...", options);
     const perf = await intro("buddy migrate:fresh"), validation = validateModelsExist();
     if (!validation.valid) {
       console.error(`
 \u274C Error: ${validation.error}
+`);
+      process.exit(ExitCode.FatalError);
+    }
+    const dialectCheck = validateMigrationDialect();
+    if (!dialectCheck.valid) {
+      console.error(`
+\u274C Error: ${dialectCheck.error}
 `);
       process.exit(ExitCode.FatalError);
     }
@@ -325,6 +363,8 @@ export function migrate(buddy) {
         process.exit(ExitCode.Success);
       }
     }
+    await preflightDatabase({ createDatabase: options.createDatabase, command: "migrate:fresh" });
+    await ensureDatabaseOrExit();
     const result = await runAction(Action.MigrateFresh, options);
     if (result.isErr)
       log.error("Model migrations failed \u2014 applying auth/notification/RBAC table guarantees before exiting.");
@@ -425,7 +465,7 @@ export function migrate(buddy) {
       if (existsSync(migrationsDir)) {
         const files = readdirSync(migrationsDir).filter((f) => f.endsWith(".sql"));
         for (const f of files) {
-          const content = readFileSync(`${migrationsDir}/${f}`, "utf-8").toLowerCase();
+          const content = stripSqlNoise(readFileSync(`${migrationsDir}/${f}`, "utf-8")).toLowerCase();
           if (/alter\s+table[\s\S]*add\s+constraint/.test(content))
             alterCount++;
           if (/^\s*create\s+unique\s+index/m.test(content))
@@ -433,7 +473,9 @@ export function migrate(buddy) {
         }
       }
     } catch {}
-    const sqliteFkNote = current === "sqlite" && alterCount > 0 ? `
+    const switchAudit = auditMigrationCorpus({ dir: projectPath("database/migrations"), target }), switchBlocked = switchAudit.incompatible.length > 0, blockedFiles = new Set(switchAudit.incompatible.map((m) => m.file)).size, dialectNote = switchBlocked ? `
+  \u26D4 ${blockedFiles} of ${switchAudit.total} migration files are ${switchAudit.inferred ?? "another dialect"}-flavoured and will NOT run on ${target}.
+     Regenerate them against ${target} before migrating, or this switch will fail on the first file.` : "", sqliteFkNote = current === "sqlite" && alterCount > 0 ? `
     (These were skipped on SQLite per stacksjs/stacks#1916 and survive on disk for replay.)` : "", boolNote = current === "sqlite" && (target === "postgres" || target === "mysql") ? `
   \u2022 Booleans land as 0/1 on SQLite; ${target} stores them as ${target === "postgres" ? "true/false" : "0/1 (compatible)"}.` : "", tzNote = target === "postgres" ? `
   \u2022 PostgreSQL uses timestamptz (with TZ) where SQLite/MySQL store plain TIMESTAMP. Existing rows do NOT auto-upgrade \u2014 they ride the column's stored type.` : "", envExtras = target !== "sqlite" ? ", plus DB_HOST / DB_PORT / DB_USERNAME / DB_PASSWORD / DB_DATABASE" : "", missingNote = missingEnv.length > 0 ? `
@@ -444,13 +486,13 @@ export function migrate(buddy) {
   \u2022 ${alterCount} ALTER TABLE ADD CONSTRAINT migration(s) will be applied against ${target}.${sqliteFkNote}
   \u2022 ${uniqueIdxCount} CREATE UNIQUE INDEX migration(s) will be applied against ${target}.
   \u2022 Auth tables (oauth_clients, oauth_access_tokens, oauth_refresh_tokens, password_resets) will be CREATE TABLE IF NOT EXISTS \u2014 they re-create cleanly under the new dialect.${boolNote}${tzNote}
-  \u2022 Existing row data does NOT auto-migrate. Use \`mysqldump\` / \`pg_dump\` (or your own export) to move it.${missingNote}
+  \u2022 Existing row data does NOT auto-migrate. Use \`mysqldump\` / \`pg_dump\` (or your own export) to move it.${missingNote}${dialectNote}
   \u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500
 
   Next steps:
     1. Update .env:  DB_CONNECTION=${target}${envExtras}
     2. (Optional) Export data from the current ${current} database.
-    3. Run \`./buddy migrate\` (or \`migrate:fresh\` to start clean).
+    3. ${switchBlocked ? `Regenerate the migration files for ${target} FIRST \u2014 \`./buddy migrate\` will refuse until then.` : "Run `./buddy migrate` (or `migrate:fresh` to start clean)."}
     4. The post-migrate FK audit will report any constraints that didn't replay.
 `);
     await outro("Plan rendered. Re-run after updating .env to actually switch.", {

package/dist/commands/serve.d.ts

@@ -32,3 +32,17 @@ export declare function serve(buddy: CLI): void;
  * command — `./buddy serve:api` resolves the framework wherever it lives.
  */
 export declare function serveApi(buddy: CLI): void;
+/**
+ * `buddy preview` — serve a finished static build locally.
+ *
+ * The twin of `buddy serve`, for the other kind of output: where that renders
+ * stx pages and proxies an API, this hands over pre-built files exactly as a
+ * static host would. It exists so checking a build before shipping it does not
+ * require a `preview.ts` in every project's root, hand-rolled per app and
+ * quietly drifting from how the host actually behaves.
+ *
+ * Extensionless paths resolve to `.html`, matching the "pretty URLs" behaviour
+ * of Netlify, Vercel and Cloudflare Pages, so a link that works here works
+ * there.
+ */
+export declare function preview(buddy: CLI): void;

package/dist/commands/serve.js

@@ -1,3 +1,4 @@
+import { ExitCode } from "@stacksjs/types";
 import { existsSync } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, join } from "node:path";
@@ -256,3 +257,30 @@ async function loadStxSiteConfig() {
   } catch {}
   return {};
 }
+export function preview(buddy) {
+  buddy.command("preview [dir]", "Serve a static build locally, the way a static host would").option("-p, --port <port>", "Port to listen on", { default: "3001" }).example("buddy preview").example("buddy preview dist --port 4000").action(async (dir, options) => {
+    const root = dir || "dist", port = Number(options?.port) || 3001;
+    if (!existsSync(root)) {
+      log.error(`No \`${root}\` directory to preview. Run \`buddy build\` first, or pass the directory to serve.`);
+      process.exit(ExitCode.FatalError);
+    }
+    Bun.serve({
+      port,
+      async fetch(req) {
+        let pathname = new URL(req.url).pathname;
+        if (pathname === "/" || pathname === "")
+          pathname = "/index.html";
+        else if (!pathname.includes("."))
+          pathname = `${pathname.replace(/\/$/, "")}.html`;
+        const file = Bun.file(join(root, pathname));
+        if (await file.exists())
+          return new Response(file);
+        const notFound = Bun.file(join(root, "404.html"));
+        if (await notFound.exists())
+          return new Response(notFound, { status: 404 });
+        return new Response("Not Found", { status: 404 });
+      }
+    });
+    log.success(`Previewing ./${root} at http://localhost:${port}`);
+  });
+}

package/dist/database-preflight.d.ts

@@ -0,0 +1,30 @@
+/**
+ * Whether we may raise a blocking question right now.
+ *
+ * Computed per call rather than read from `@stacksjs/env`'s `isCI` / `hasTTY`,
+ * which are module-level constants frozen at first import: anything that sets
+ * `CI` after the env package loads (a wrapper script, a test harness, a config
+ * file evaluated during CLI boot) would otherwise get no protection at all.
+ *
+ * Both stdin and stdout are required. stdout alone is not enough: `buddy
+ * migrate < /dev/null` keeps a terminal on stdout while stdin is at EOF.
+ */
+export declare function canPromptInteractively(): boolean;
+/**
+ * Resolve the standing policy, if the user set one.
+ * `always` and `never` skip the question entirely in both directions.
+ */
+export declare function resolveCreatePolicy(): 'always' | 'never' | 'prompt';
+/**
+ * Offer to create the target database when it is missing.
+ *
+ * Returns normally when the command may proceed. Exits the process when the
+ * user declines or when there is no safe way to continue. Never creates
+ * anything itself: it only records the decision, and the bootstrap in
+ * `@stacksjs/database` performs the create at the usual point.
+ */
+export declare function preflightDatabase(options?: DatabasePreflightOptions): Promise<void>;
+export declare interface DatabasePreflightOptions {
+  createDatabase?: boolean
+  command?: string
+}

package/dist/database-preflight.js

@@ -0,0 +1,88 @@
+import process from "node:process";
+import { confirmOrNull, log } from "@stacksjs/cli";
+import { ExitCode } from "@stacksjs/types";
+export function canPromptInteractively() {
+  if (process.env.CI || process.env.CONTINUOUS_INTEGRATION || process.env.BUILD_NUMBER || process.env.RUN_ID)
+    return !1;
+  return Boolean(process.stdin.isTTY && process.stdout.isTTY);
+}
+export function resolveCreatePolicy() {
+  const value = String(process.env.DB_CREATE_DATABASE || "").toLowerCase();
+  if (value === "always" || value === "true" || value === "1")
+    return "always";
+  if (value === "never" || value === "false" || value === "0")
+    return "never";
+  return "prompt";
+}
+function decide(create) {
+  process.env.STACKS_CREATE_DATABASE = create ? "1" : "0";
+}
+const UNANSWERED = Symbol("unanswered");
+async function confirmWithTimeout(message, timeoutMs) {
+  let timer;
+  try {
+    return await Promise.race([
+      confirmOrNull({ message, initial: !0 }).then((v) => v === null ? UNANSWERED : v),
+      new Promise((resolve) => {
+        timer = setTimeout(() => resolve(UNANSWERED), timeoutMs);
+      })
+    ]);
+  } finally {
+    if (timer)
+      clearTimeout(timer);
+  }
+}
+export async function preflightDatabase(options = {}) {
+  const commandName = options.command || "migrate", {
+    canCreateDatabases,
+    describeTarget,
+    manualCreateHint,
+    probeTargetDatabase,
+    resolveConnectionTarget
+  } = await import("@stacksjs/database"), target = resolveConnectionTarget();
+  if (!target)
+    return;
+  const probe = await probeTargetDatabase(target);
+  if (probe.ok || probe.kind !== "missing-database")
+    return;
+  const where = describeTarget(target), standing = resolveCreatePolicy();
+  if (standing === "never") {
+    decide(!1);
+    return;
+  }
+  if (options.createDatabase || standing === "always") {
+    decide(!0);
+    return;
+  }
+  if (!canPromptInteractively()) {
+    log.syncError(`The database "${target.database}" does not exist on ${where}.`);
+    log.syncError("Refusing to create it in a non-interactive shell.");
+    log.syncError("  Create it ahead of time, re-run with --create-database, or set DB_CREATE_DATABASE=always.");
+    log.syncError(`  ${manualCreateHint(target)}`);
+    process.exit(ExitCode.FatalError);
+  }
+  if (await canCreateDatabases(target) === !1) {
+    log.syncError(`The database "${target.database}" does not exist on ${where}.`);
+    log.syncError(`The user "${target.username}" is not allowed to create databases, so I cannot create it for you.`);
+    log.syncError(`  Grant it with:  ALTER ROLE "${target.username}" CREATEDB;`);
+    log.syncError(`  Or create the database yourself with:  ${manualCreateHint(target)}`);
+    process.exit(ExitCode.FatalError);
+  }
+  log.warn(`The database "${target.database}" does not exist on ${where}.`);
+  await log.flush();
+  const timeoutMs = Number(process.env.DB_CREATE_DATABASE_TIMEOUT_MS || 120000), answer = await confirmWithTimeout("Would you like to create it?", timeoutMs);
+  if (answer === UNANSWERED) {
+    log.syncError(`No answer after ${Math.round(timeoutMs / 1000)}s, so nothing was created.`);
+    log.syncError("  Re-run with --create-database, or set DB_CREATE_DATABASE=always, to skip this question.");
+    log.syncError(`  ${manualCreateHint(target)}`);
+    process.exit(ExitCode.FatalError);
+  }
+  if (!answer) {
+    decide(!1);
+    log.info("Nothing was changed. Create it yourself with:");
+    log.info(`  ${manualCreateHint(target)}`);
+    log.info(`Then re-run \`buddy ${commandName}\`, or re-run with --create-database to skip this question.`);
+    process.exit(ExitCode.Success);
+  }
+  decide(!0);
+}

package/dist/lazy-commands.js

@@ -101,6 +101,7 @@ const commandRegistry = {
   seed: { path: "./commands/seed.js", exportName: "seed" },
   "seed:roles": { path: "./commands/seed.js", exportName: "seed" },
   "roles:seed": { path: "./commands/seed.js", exportName: "seed" },
+  preview: { path: "./commands/serve.js", exportName: "preview" },
   serve: { path: "./commands/serve.js", exportName: "serve" },
   "serve:api": { path: "./commands/serve.js", exportName: "serveApi" },
   setup: { path: "./commands/setup.js", exportName: "setup" },

package/package.json

@@ -2,7 +2,7 @@
   "name": "@stacksjs/buddy",
   "type": "module",
   "sideEffects": false,
-  "version": "0.70.201",
+  "version": "0.70.203",
   "description": "Meet Buddy. The Stacks runtime.",
   "author": "Chris Breuer",
   "contributors": [