@happyvertical/smrt-cli 0.43.0 → 0.43.2

package/dist/{commands-C1bK8bv9.js → commands-BAKZil1v.js}

@@ -493,6 +493,10 @@ function getSyntheticMigrationNameForAction(action) {
 			const fingerprint = sqlShapeFingerprint(action);
 			return fingerprint ? `add_foreign_key_${action.tableName}_${fingerprint}` : null;
 		}
+		case "drop_foreign_key": {
+			const fingerprint = sqlShapeFingerprint(action);
+			return fingerprint ? `drop_foreign_key_${action.tableName}_${fingerprint}` : null;
+		}
 		case "drop_index": {
 			if (!action.indexName) return null;
 			const fingerprint = sqlShapeFingerprint(action);
@@ -525,6 +529,10 @@ function getSyntheticMigrationNameForChange(change) {
 			const fingerprint = sqlShapeFingerprint(change);
 			return fingerprint ? `add_foreign_key_${change.table}_${fingerprint}` : null;
 		}
+		case "drop_foreign_key": {
+			const fingerprint = sqlShapeFingerprint(change);
+			return fingerprint ? `drop_foreign_key_${change.table}_${fingerprint}` : null;
+		}
 		case "drop_index": {
 			if (!change.name) return null;
 			const fingerprint = sqlShapeFingerprint({
@@ -565,13 +573,13 @@ function shouldApplySchemaMigrations(state) {
 	return !state.dryRun;
 }
 function classifyFailedMigration(migrationName, unresolvedSyntheticMigrationNames) {
-	if (!migrationName.startsWith("add_column_") && !migrationName.startsWith("drop_column_") && !migrationName.startsWith("alter_column_") && !migrationName.startsWith("add_foreign_key_") && !migrationName.startsWith("add_index_") && !migrationName.startsWith("drop_index_") && !migrationName.startsWith("type_upgrade_")) return "other";
+	if (!migrationName.startsWith("add_column_") && !migrationName.startsWith("drop_column_") && !migrationName.startsWith("alter_column_") && !migrationName.startsWith("add_foreign_key_") && !migrationName.startsWith("drop_foreign_key_") && !migrationName.startsWith("add_index_") && !migrationName.startsWith("drop_index_") && !migrationName.startsWith("type_upgrade_")) return "other";
 	return unresolvedSyntheticMigrationNames.has(migrationName) ? "unresolved" : "superseded";
 }
 function getUnresolvedGeneratedMigrationNames(changes) {
 	const names = /* @__PURE__ */ new Set();
 	for (const change of changes) {
-		if (change.type !== "add_column" && change.type !== "drop_column" && change.type !== "alter_column" && change.type !== "add_foreign_key" && change.type !== "add_index" && change.type !== "drop_index" && change.type !== "type_upgrade") continue;
+		if (change.type !== "add_column" && change.type !== "drop_column" && change.type !== "alter_column" && change.type !== "add_foreign_key" && change.type !== "drop_foreign_key" && change.type !== "add_index" && change.type !== "drop_index" && change.type !== "type_upgrade") continue;
 		if (change.type === "type_upgrade" && classifyTypeUpgradeSql(change.sql) === "noop") continue;
 		if (change.type === "alter_column" && isAdvisoryOnlyChangeLike(change)) continue;
 		for (const migrationName of getSyntheticMigrationNamesForChange(change)) names.add(migrationName);
@@ -714,9 +722,10 @@ function partitionSchemaChanges(changes, getClassForTable) {
 				});
 				break;
 			}
-			case "add_foreign_key": {
+			case "add_foreign_key":
+			case "drop_foreign_key": {
 				const action = {
-					type: "add_foreign_key",
+					type: change.type,
 					tableName: change.table,
 					className,
 					sql: change.sql,
@@ -986,7 +995,8 @@ var dbDiffCommand = {
 				}, null, 2));
 				return;
 			}
-			const { advisories } = partitionSchemaChanges(diff.changes, (tableName) => tableName);
+			const { advisories, manualInterventions } = partitionSchemaChanges(diff.changes, (tableName) => tableName);
+			const manualForeignKeyChanges = manualInterventions.filter((change) => change.type === "add_foreign_key" || change.type === "drop_foreign_key");
 			if (!diff.has_changes) {
 				console.log("✅ Database schema is up to date - no changes detected\n");
 				printSchemaAdvisories(advisories, {
@@ -995,7 +1005,7 @@ var dbDiffCommand = {
 				});
 				return;
 			}
-			if (diff.changes.filter((c) => !(c.advisory && !c.sql && !c.sqlStatements)).length === 0 && diff.added_tables.length === 0 && diff.dropped_tables.length === 0) {
+			if (diff.changes.filter((c) => !(c.advisory && !c.sql && !c.sqlStatements)).length === 0 && manualForeignKeyChanges.length === 0 && diff.added_tables.length === 0 && diff.dropped_tables.length === 0) {
 				console.log("✅ Database schema is up to date - no migrations needed (see notes below)\n");
 				printSchemaAdvisories(advisories, {
 					orphanTables: diff.orphan_tables,
@@ -1076,6 +1086,16 @@ var dbDiffCommand = {
 				for (const change of typeMismatches) console.log(`     ! ${change.table}.${change.name}: ${change.mismatch?.expected} vs ${change.mismatch?.actual}`);
 				console.log("     (Type changes require manual migration)\n");
 			}
+			if (manualForeignKeyChanges.length > 0) {
+				console.log(`  ⚠️  Foreign-key drift needing a manual step (${manualForeignKeyChanges.length}):`);
+				for (const change of manualForeignKeyChanges) {
+					const operation = change.type === "drop_foreign_key" ? "remove" : "add";
+					console.log(`     ! ${change.tableName}: ${operation} constraint`);
+					console.log(`       ${change.advisory?.message ?? "foreign-key constraint requires manual repair"}`);
+					for (const sql of change.advisory?.suggestedSql ?? []) console.log(`       ↳ ${sql}`);
+				}
+				console.log();
+			}
 			printSchemaAdvisories(advisories, {
 				orphanTables: diff.orphan_tables,
 				verbose: options.verbose
@@ -2382,6 +2402,17 @@ function summarizeSchemaDiff(diff) {
 				recommendation: "Run `smrt db:migrate` to add the missing index and reconcile the live schema."
 			});
 			break;
+		case "add_foreign_key":
+		case "drop_foreign_key": {
+			const relationship = change.foreignKey ? `${change.table}.${change.foreignKey.column} -> ${change.foreignKey.referencesTable}.${change.foreignKey.referencesColumn}` : `${change.table}.${change.name ?? "(unknown)"}`;
+			const isAddition = change.type === "add_foreign_key";
+			drift.push({
+				name: relationship,
+				type: isAddition ? "missing_foreign_key" : "disabled_foreign_key",
+				recommendation: change.advisory?.message ?? (isAddition ? "Run `smrt db:migrate` to add the missing foreign key and reconcile the live schema." : "Run `smrt db:migrate` to remove the disabled foreign key and reconcile the live schema.")
+			});
+			break;
+		}
 		case "type_upgrade":
 			switch (classifyTypeUpgradeSql(change.sql)) {
 				case "executable":
@@ -8312,7 +8343,7 @@ export default testManifest;
 				if (manualInterventions.length > 0) {
 					console.log("⚠️  Schema drift detected that requires manual intervention:\n");
 					for (const change of manualInterventions) {
-						if (change.type === "add_foreign_key") {
+						if (change.type === "add_foreign_key" || change.type === "drop_foreign_key") {
 							console.log(`   ${change.tableName}: ${change.advisory?.message ?? change.sql?.replace(/^--\s*/, "") ?? "foreign-key constraint requires manual repair"}`);
 							for (const sql of change.advisory?.suggestedSql ?? []) console.log(`      ${sql}`);
 							continue;
@@ -8347,6 +8378,7 @@ export default testManifest;
 						const columnMigrations = migrations.filter((m) => m.type === "add_column");
 						const indexMigrations = migrations.filter((m) => m.type === "add_index");
 						const foreignKeyMigrations = migrations.filter((m) => m.type === "add_foreign_key");
+						const foreignKeyDrops = migrations.filter((m) => m.type === "drop_foreign_key");
 						const indexDrops = migrations.filter((m) => m.type === "drop_index");
 						const columnAlterations = migrations.filter((m) => m.type === "alter_column");
 						const columnDrops = migrations.filter((m) => m.type === "drop_column");
@@ -8380,6 +8412,11 @@ export default testManifest;
 							for (const m of foreignKeyMigrations) console.log(`     ${m.tableName}`);
 							console.log();
 						}
+						if (foreignKeyDrops.length > 0) {
+							console.log(`  🔗 Foreign-key constraints to drop: ${foreignKeyDrops.length}`);
+							for (const m of foreignKeyDrops) console.log(`     ${m.tableName}`);
+							console.log();
+						}
 						console.log("  SQL Statements:\n");
 						for (const schema of tablePlan.schemas) {
 							const ddl = plannedTableDDL.get(schema.tableName);
@@ -8473,11 +8510,14 @@ export default testManifest;
 						} else if (migration.type === "add_foreign_key") {
 							migrationSql = migration.sql || "";
 							actionDesc = `Added foreign-key constraint on ${migration.tableName}`;
+						} else if (migration.type === "drop_foreign_key") {
+							migrationSql = migration.sql || "";
+							actionDesc = `Dropped foreign-key constraint on ${migration.tableName}`;
 						} else continue;
 						const migrationSqlStatements = migration.sqlStatements ?? (migrationSql ? [migrationSql] : []);
 						migrationDefs.push({
 							id: migrationName,
-							description: migration.type === "add_column" ? `Add column ${migration.column?.name} to ${migration.tableName}` : migration.type === "alter_column" ? `Alter column ${migration.columnName ?? migration.column?.name} on ${migration.tableName} (${migration.alteration ?? "alter"}: ${migration.mismatch?.actual ?? "?"} → ${migration.mismatch?.expected ?? "?"})` : migration.type === "drop_column" ? `Drop column ${migration.columnName} from ${migration.tableName}` : migration.type === "type_upgrade" ? `Upgrade column ${migration.column?.name} on ${migration.tableName} from ${migration.mismatch?.actual} to ${migration.mismatch?.expected}` : migration.type === "drop_index" ? `Drop index ${migration.indexName} on ${migration.tableName}` : migration.type === "add_foreign_key" ? `Add foreign-key constraint on ${migration.tableName}` : `Add index ${migration.index?.name} on ${migration.tableName}`,
+							description: migration.type === "add_column" ? `Add column ${migration.column?.name} to ${migration.tableName}` : migration.type === "alter_column" ? `Alter column ${migration.columnName ?? migration.column?.name} on ${migration.tableName} (${migration.alteration ?? "alter"}: ${migration.mismatch?.actual ?? "?"} → ${migration.mismatch?.expected ?? "?"})` : migration.type === "drop_column" ? `Drop column ${migration.columnName} from ${migration.tableName}` : migration.type === "type_upgrade" ? `Upgrade column ${migration.column?.name} on ${migration.tableName} from ${migration.mismatch?.actual} to ${migration.mismatch?.expected}` : migration.type === "drop_index" ? `Drop index ${migration.indexName} on ${migration.tableName}` : migration.type === "add_foreign_key" ? `Add foreign-key constraint on ${migration.tableName}` : migration.type === "drop_foreign_key" ? `Drop foreign-key constraint on ${migration.tableName}` : `Add index ${migration.index?.name} on ${migration.tableName}`,
 							version: "1.0.0",
 							up: migrationSqlStatements,
 							down: []

package/dist/index.js

@@ -49,63 +49,63 @@ var _playgroundCommands = null;
 var _workbenchCommands = null;
 async function getGnodeCommands() {
 	if (!_gnodeCommands) {
-		const { gnodeCommands } = await import("./commands-C1bK8bv9.js");
+		const { gnodeCommands } = await import("./commands-BAKZil1v.js");
 		_gnodeCommands = gnodeCommands;
 	}
 	return _gnodeCommands;
 }
 async function getGitCommands() {
 	if (!_gitCommands) {
-		const { gitCommands } = await import("./commands-C1bK8bv9.js");
+		const { gitCommands } = await import("./commands-BAKZil1v.js");
 		_gitCommands = gitCommands;
 	}
 	return _gitCommands;
 }
 async function getGenerateCommands() {
 	if (!_generateCommands) {
-		const { generateCommands } = await import("./commands-C1bK8bv9.js");
+		const { generateCommands } = await import("./commands-BAKZil1v.js");
 		_generateCommands = generateCommands;
 	}
 	return _generateCommands;
 }
 async function getInitCommands() {
 	if (!_initCommands) {
-		const { initCommands } = await import("./commands-C1bK8bv9.js");
+		const { initCommands } = await import("./commands-BAKZil1v.js");
 		_initCommands = initCommands;
 	}
 	return _initCommands;
 }
 async function getUtilityCommands() {
 	if (!_utilityCommands) {
-		const { utilityCommands } = await import("./commands-C1bK8bv9.js");
+		const { utilityCommands } = await import("./commands-BAKZil1v.js");
 		_utilityCommands = utilityCommands;
 	}
 	return _utilityCommands;
 }
 async function getDispatchCommands() {
 	if (!_dispatchCommands) {
-		const { dispatchCommands } = await import("./commands-C1bK8bv9.js");
+		const { dispatchCommands } = await import("./commands-BAKZil1v.js");
 		_dispatchCommands = dispatchCommands;
 	}
 	return _dispatchCommands;
 }
 async function getDocsCommands() {
 	if (!_docsCommands) {
-		const { docsCommands } = await import("./commands-C1bK8bv9.js");
+		const { docsCommands } = await import("./commands-BAKZil1v.js");
 		_docsCommands = docsCommands;
 	}
 	return _docsCommands;
 }
 async function getPlaygroundCommands() {
 	if (!_playgroundCommands) {
-		const { playgroundCommands } = await import("./commands-C1bK8bv9.js");
+		const { playgroundCommands } = await import("./commands-BAKZil1v.js");
 		_playgroundCommands = playgroundCommands;
 	}
 	return _playgroundCommands;
 }
 async function getWorkbenchCommands() {
 	if (!_workbenchCommands) {
-		const { workbenchCommands } = await import("./commands-C1bK8bv9.js");
+		const { workbenchCommands } = await import("./commands-BAKZil1v.js");
 		_workbenchCommands = workbenchCommands;
 	}
 	return _workbenchCommands;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@happyvertical/smrt-cli",
-  "version": "0.43.0",
+  "version": "0.43.2",
   "description": "Developer CLI for SMRT framework - introspection, testing, and project management",
   "type": "module",
   "main": "./dist/index.js",
@@ -32,12 +32,12 @@
     "acorn": "^8.17.0",
     "fast-glob": "3.3.3",
     "tar": "^7.5.19",
-    "@happyvertical/smrt-agents": "0.43.0",
-    "@happyvertical/smrt-config": "0.43.0",
-    "@happyvertical/smrt-core": "0.43.0",
-    "@happyvertical/smrt-playground": "0.43.0",
-    "@happyvertical/smrt-types": "0.43.0",
-    "@happyvertical/smrt-dev-mcp": "0.43.0"
+    "@happyvertical/smrt-agents": "0.43.2",
+    "@happyvertical/smrt-core": "0.43.2",
+    "@happyvertical/smrt-config": "0.43.2",
+    "@happyvertical/smrt-dev-mcp": "0.43.2",
+    "@happyvertical/smrt-playground": "0.43.2",
+    "@happyvertical/smrt-types": "0.43.2"
   },
   "devDependencies": {
     "@types/node": "24.13.2",