@stacksjs/buddy 0.70.274 → 0.70.275

package/dist/commands/migrate.d.ts

@@ -1,4 +1,19 @@
 import { confirm } from '@stacksjs/cli';
 import type { CLI } from '@stacksjs/types';
+/**
+ * Refuse to replay a migration corpus against a database it was not written
+ * for.
+ *
+ * Stacks ships ONE set of migration files under `database/migrations`, emitted
+ * for a single dialect, and `buddy new` copies them verbatim into every new
+ * project. Setting DB_CONNECTION=postgres therefore used to fail with a raw
+ * `syntax error at or near "AUTOINCREMENT"` on file 1 of 121, which says
+ * nothing about the actual cause.
+ *
+ * Checked in the parent, before the migrate:fresh drop confirmation and before
+ * any action subprocess starts, so a mismatch cannot drop the framework tables
+ * and only then discover it has nothing to rebuild them with.
+ */
+export declare function validateMigrationDialect(cwd?: unknown): { valid: boolean, error?: string };
 export declare function migrate(buddy: CLI): void;
 declare type FreshGuard = 'allow' | 'confirm' | 'disabled';

package/dist/commands/migrate.js

@@ -1,4 +1,4 @@
-import{closeSync,existsSync,mkdirSync,openSync,readdirSync,readFileSync,rmSync}from"node:fs";import process from"node:process";import{confirm,intro,log,onUnknownSubcommand,outro,text}from"@stacksjs/cli";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{DDL_CONSTRAINT_OVERRIDE_ENV,DIALECT_OVERRIDE_ENV,auditDdlConstraints,auditMigrationCorpus,dialectCapabilities,formatDdlConstraintError,formatMigrationDialectError,stripSqlNoise}from"@stacksjs/database";import{resultFailed}from"../result";let _runAction;async function runAction(...args){if(!_runAction)_runAction=(await import("@stacksjs/actions")).runAction;return _runAction(...args)}function acquireMigrationLock(){const lockDir=frameworkRuntimePath(),lockFile=`${lockDir}/migrations.lock`;try{if(!existsSync(lockDir))mkdirSync(lockDir,{recursive:!0});const fd=openSync(lockFile,"wx");try{closeSync(fd)}catch{}return{acquired:!0,release:()=>{try{rmSync(lockFile,{force:!0})}catch{}}}}catch{return{acquired:!1,release:()=>{}}}}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))return null;try{const raw=readFileSync(file,"utf8"),parsed=JSON.parse(raw),n=typeof parsed.appliedCount==="number"?parsed.appliedCount:null;if(n===null||!Number.isFinite(n))return null;return{appliedCount:Math.max(0,Math.floor(n))}}catch{return null}finally{try{rmSync(file,{force:!0})}catch{}}}function countModelFiles(dir){if(!existsSync(dir))return 0;let count=0;const entries=readdirSync(dir,{withFileTypes:!0});for(const entry of entries)if(entry.isDirectory())count+=countModelFiles(`${dir}/${entry.name}`);else if(entry.name.endsWith(".ts")&&!entry.name.startsWith(".")&&!entry.name.startsWith("index"))count++;return count}function validateModelsExist(){const userModelsPath=appPath("Models"),defaultModelsPath=frameworkPath("defaults/app/Models"),userModelCount=countModelFiles(userModelsPath),defaultModelCount=countModelFiles(defaultModelsPath);if(userModelCount===0&&defaultModelCount===0)return{valid:!1,error:"No models found. Please create models in app/Models or ensure framework defaults exist."};return{valid:!0}}function validateMigrationDialect(){const driver=String(process.env.DB_CONNECTION||"sqlite").toLowerCase(),dir=projectPath("database/migrations");if(process.env[DIALECT_OVERRIDE_ENV]!=="1"){const caps=dialectCapabilities(driver),target=caps.wire==="mysql"?"mysql":caps.wire==="postgres"?"postgres":"sqlite";if(driver==="sqlite"||driver==="postgres"||caps.wire==="mysql"){const audit=auditMigrationCorpus({dir,target});if(!audit.empty&&audit.incompatible.length>0)return{valid:!1,error:formatMigrationDialectError(audit,target,"database/migrations")}}}if(process.env[DDL_CONSTRAINT_OVERRIDE_ENV]!=="1"){const constraints=auditDdlConstraints({dir,dialect:driver});if(!constraints.empty&&constraints.violations.length>0)return{valid:!1,error:formatDdlConstraintError(constraints,driver,"database/migrations")}}return{valid:!0}}async function reportMissingForeignKeys(){try{const{auditForeignKeys}=await import("@stacksjs/database"),result=await auditForeignKeys();if(result.missing.length===0)return;const sample=result.missing.slice(0,5).map((fk)=>`  \u2022 ${fk.fromTable}.${fk.fromColumn} \u2192 ${fk.toTable}.${fk.toColumn} (${fk.model})`).join(`
+import{closeSync,existsSync,mkdirSync,openSync,readdirSync,readFileSync,rmSync}from"node:fs";import process from"node:process";import{confirm,intro,log,onUnknownSubcommand,outro,text}from"@stacksjs/cli";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{DDL_CONSTRAINT_OVERRIDE_ENV,DIALECT_OVERRIDE_ENV,auditDdlConstraints,auditMigrationCorpus,dialectCapabilities,formatDdlConstraintError,formatMigrationDialectError,relativeMigrationDirectory,resolveMigrationDirectory,stripSqlNoise}from"@stacksjs/database";import{resultFailed}from"../result";let _runAction;async function runAction(...args){if(!_runAction)_runAction=(await import("@stacksjs/actions")).runAction;return _runAction(...args)}function acquireMigrationLock(){const lockDir=frameworkRuntimePath(),lockFile=`${lockDir}/migrations.lock`;try{if(!existsSync(lockDir))mkdirSync(lockDir,{recursive:!0});const fd=openSync(lockFile,"wx");try{closeSync(fd)}catch{}return{acquired:!0,release:()=>{try{rmSync(lockFile,{force:!0})}catch{}}}}catch{return{acquired:!1,release:()=>{}}}}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))return null;try{const raw=readFileSync(file,"utf8"),parsed=JSON.parse(raw),n=typeof parsed.appliedCount==="number"?parsed.appliedCount:null;if(n===null||!Number.isFinite(n))return null;return{appliedCount:Math.max(0,Math.floor(n))}}catch{return null}finally{try{rmSync(file,{force:!0})}catch{}}}function countModelFiles(dir){if(!existsSync(dir))return 0;let count=0;const entries=readdirSync(dir,{withFileTypes:!0});for(const entry of entries)if(entry.isDirectory())count+=countModelFiles(`${dir}/${entry.name}`);else if(entry.name.endsWith(".ts")&&!entry.name.startsWith(".")&&!entry.name.startsWith("index"))count++;return count}function validateModelsExist(){const userModelsPath=appPath("Models"),defaultModelsPath=frameworkPath("defaults/app/Models"),userModelCount=countModelFiles(userModelsPath),defaultModelCount=countModelFiles(defaultModelsPath);if(userModelCount===0&&defaultModelCount===0)return{valid:!1,error:"No models found. Please create models in app/Models or ensure framework defaults exist."};return{valid:!0}}export function validateMigrationDialect(cwd=process.cwd()){const driver=String(process.env.DB_CONNECTION||"sqlite").toLowerCase(),dir=resolveMigrationDirectory(driver,{cwd}),relativeDir=relativeMigrationDirectory(dir,cwd);if(process.env[DIALECT_OVERRIDE_ENV]!=="1"){const caps=dialectCapabilities(driver),target=caps.wire==="mysql"?"mysql":caps.wire==="postgres"?"postgres":"sqlite";if(driver==="sqlite"||driver==="postgres"||caps.wire==="mysql"){const audit=auditMigrationCorpus({dir,target});if(!audit.empty&&audit.incompatible.length>0)return{valid:!1,error:formatMigrationDialectError(audit,target,relativeDir)}}}if(process.env[DDL_CONSTRAINT_OVERRIDE_ENV]!=="1"){const constraints=auditDdlConstraints({dir,dialect:driver});if(!constraints.empty&&constraints.violations.length>0)return{valid:!1,error:formatDdlConstraintError(constraints,driver,relativeDir)}}return{valid:!0}}async function reportMissingForeignKeys(){try{const{auditForeignKeys}=await import("@stacksjs/database"),result=await auditForeignKeys();if(result.missing.length===0)return;const sample=result.missing.slice(0,5).map((fk)=>`  \u2022 ${fk.fromTable}.${fk.fromColumn} \u2192 ${fk.toTable}.${fk.toColumn} (${fk.model})`).join(`
 `),more=result.missing.length>5?`
   + ${result.missing.length-5} more \u2014 run \`./buddy doctor\` for the full list.`:"";log.warn(`${result.missing.length} of ${result.declared.length} declared foreign keys are missing from the live schema:
 ${sample}${more}
@@ -21,7 +21,7 @@ ${sample}${more}
   Unknown target driver "${driver}". Allowed: sqlite, mysql, vitess, postgres.
 `);await outro("Aborted.",{startTime:perf,useSeconds:!0});process.exit(ExitCode.FatalError)}const current=(process.env.DB_CONNECTION||"sqlite").toLowerCase();if(current===target){console.log(`
   DB_CONNECTION is already "${target}". Nothing to switch.
-`);await outro("No-op.",{startTime:perf,useSeconds:!0});process.exit(ExitCode.Success)}const missingEnv=({sqlite:["DB_DATABASE"],mysql:["DB_HOST","DB_PORT","DB_USERNAME","DB_PASSWORD","DB_DATABASE"],vitess:["DB_HOST","DB_PORT","DB_USERNAME","DB_PASSWORD","DB_DATABASE"],postgres:["DB_HOST","DB_PORT","DB_USERNAME","DB_PASSWORD","DB_DATABASE"]}[target]??[]).filter((k)=>!process.env[k]);let alterCount=0,uniqueIdxCount=0;try{const migrationsDir=projectPath("database/migrations");if(existsSync(migrationsDir)){const files=readdirSync(migrationsDir).filter((f)=>f.endsWith(".sql"));for(const f of files){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))uniqueIdxCount++}}}catch{}const auditTarget=target==="vitess"?"mysql":target,switchAudit=auditMigrationCorpus({dir:projectPath("database/migrations"),target:auditTarget}),switchBlocked=switchAudit.incompatible.length>0,blockedFiles=new Set(switchAudit.incompatible.map((m)=>m.file)).size,dialectNote=switchBlocked?`
+`);await outro("No-op.",{startTime:perf,useSeconds:!0});process.exit(ExitCode.Success)}const missingEnv=({sqlite:["DB_DATABASE"],mysql:["DB_HOST","DB_PORT","DB_USERNAME","DB_PASSWORD","DB_DATABASE"],vitess:["DB_HOST","DB_PORT","DB_USERNAME","DB_PASSWORD","DB_DATABASE"],postgres:["DB_HOST","DB_PORT","DB_USERNAME","DB_PASSWORD","DB_DATABASE"]}[target]??[]).filter((k)=>!process.env[k]);let alterCount=0,uniqueIdxCount=0;try{const migrationsDir=resolveMigrationDirectory(target);if(existsSync(migrationsDir)){const files=readdirSync(migrationsDir).filter((f)=>f.endsWith(".sql"));for(const f of files){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))uniqueIdxCount++}}}catch{}const auditTarget=target==="vitess"?"mysql":target,switchAudit=auditMigrationCorpus({dir:resolveMigrationDirectory(target),target:auditTarget}),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"||target==="vitess")?`

package/package.json

@@ -2,7 +2,7 @@
   "name": "@stacksjs/buddy",
   "type": "module",
   "sideEffects": false,
-  "version": "0.70.274",
+  "version": "0.70.275",
   "description": "Meet Buddy. The Stacks runtime.",
   "author": "Chris Breuer",
   "contributors": [
@@ -95,53 +95,53 @@
     "prepublishOnly": "bun run build"
   },
   "dependencies": {
-    "@stacksjs/actions": "^0.70.274",
-    "@stacksjs/ai": "^0.70.274",
-    "@stacksjs/alias": "^0.70.274",
-    "@stacksjs/arrays": "^0.70.274",
-    "@stacksjs/auth": "^0.70.274",
-    "@stacksjs/build": "^0.70.274",
-    "@stacksjs/cache": "^0.70.274",
-    "@stacksjs/cli": "^0.70.274",
+    "@stacksjs/actions": "^0.70.275",
+    "@stacksjs/ai": "^0.70.275",
+    "@stacksjs/alias": "^0.70.275",
+    "@stacksjs/arrays": "^0.70.275",
+    "@stacksjs/auth": "^0.70.275",
+    "@stacksjs/build": "^0.70.275",
+    "@stacksjs/cache": "^0.70.275",
+    "@stacksjs/cli": "^0.70.275",
     "@stacksjs/clapp": "^0.2.12",
-    "@stacksjs/cloud": "^0.70.274",
-    "@stacksjs/collections": "^0.70.274",
-    "@stacksjs/config": "^0.70.274",
-    "@stacksjs/database": "^0.70.274",
-    "@stacksjs/desktop-build": "^0.70.274",
-    "@stacksjs/dns": "^0.70.274",
-    "@stacksjs/email": "^0.70.274",
-    "@stacksjs/enums": "^0.70.274",
-    "@stacksjs/error-handling": "^0.70.274",
-    "@stacksjs/events": "^0.70.274",
-    "@stacksjs/git": "^0.70.274",
+    "@stacksjs/cloud": "^0.70.275",
+    "@stacksjs/collections": "^0.70.275",
+    "@stacksjs/config": "^0.70.275",
+    "@stacksjs/database": "^0.70.275",
+    "@stacksjs/desktop-build": "^0.70.275",
+    "@stacksjs/dns": "^0.70.275",
+    "@stacksjs/email": "^0.70.275",
+    "@stacksjs/enums": "^0.70.275",
+    "@stacksjs/error-handling": "^0.70.275",
+    "@stacksjs/events": "^0.70.275",
+    "@stacksjs/git": "^0.70.275",
     "@stacksjs/gitit": "^0.2.5",
-    "@stacksjs/health": "^0.70.274",
+    "@stacksjs/health": "^0.70.275",
     "@stacksjs/dnsx": "^0.2.3",
     "@stacksjs/httx": "^0.1.10",
-    "@stacksjs/image": "^0.70.274",
-    "@stacksjs/lint": "^0.70.274",
-    "@stacksjs/logging": "^0.70.274",
-    "@stacksjs/notifications": "^0.70.274",
-    "@stacksjs/objects": "^0.70.274",
-    "@stacksjs/orm": "^0.70.274",
-    "@stacksjs/path": "^0.70.274",
-    "@stacksjs/skills": "^0.70.274",
-    "@stacksjs/payments": "^0.70.274",
-    "@stacksjs/realtime": "^0.70.274",
-    "@stacksjs/router": "^0.70.274",
+    "@stacksjs/image": "^0.70.275",
+    "@stacksjs/lint": "^0.70.275",
+    "@stacksjs/logging": "^0.70.275",
+    "@stacksjs/notifications": "^0.70.275",
+    "@stacksjs/objects": "^0.70.275",
+    "@stacksjs/orm": "^0.70.275",
+    "@stacksjs/path": "^0.70.275",
+    "@stacksjs/skills": "^0.70.275",
+    "@stacksjs/payments": "^0.70.275",
+    "@stacksjs/realtime": "^0.70.275",
+    "@stacksjs/router": "^0.70.275",
     "@stacksjs/rpx": "^0.11.42",
-    "@stacksjs/search-engine": "^0.70.274",
-    "@stacksjs/security": "^0.70.274",
-    "@stacksjs/server": "^0.70.274",
-    "@stacksjs/storage": "^0.70.274",
-    "@stacksjs/strings": "^0.70.274",
-    "@stacksjs/testing": "^0.70.274",
-    "@stacksjs/tunnel": "^0.70.274",
-    "@stacksjs/types": "^0.70.274",
-    "@stacksjs/ui": "^0.70.274",
-    "@stacksjs/utils": "^0.70.274",
-    "@stacksjs/validation": "^0.70.274",
+    "@stacksjs/search-engine": "^0.70.275",
+    "@stacksjs/security": "^0.70.275",
+    "@stacksjs/server": "^0.70.275",
+    "@stacksjs/storage": "^0.70.275",
+    "@stacksjs/strings": "^0.70.275",
+    "@stacksjs/testing": "^0.70.275",
+    "@stacksjs/tunnel": "^0.70.275",
+    "@stacksjs/types": "^0.70.275",
+    "@stacksjs/ui": "^0.70.275",
+    "@stacksjs/utils": "^0.70.275",
+    "@stacksjs/validation": "^0.70.275",
     "@stacksjs/ts-cloud": "^0.7.103",
     "ajv": "^8.20.0",
     "ajv-formats": "^3.0.1",