@happyvertical/smrt-cli 0.40.24 → 0.40.25

package/dist/{commands-CR1RZCYu.js → commands-gjGWLmwD.js}

@@ -691,6 +691,24 @@ function sqlShapeFingerprint(action) {
 	const normalized = parts.map((s) => s.replace(/\s+/g, " ").trim()).join(";");
 	return createHash("sha256").update(normalized).digest("hex").slice(0, 8);
 }
+/**
+* Keep type-upgrade tracker identities tied to the specific conversion, not
+* merely the table and column. A column may legitimately undergo multiple
+* upgrades over its lifetime; reusing the old bare id would make
+* MigrationTracker compare different SQL checksums against applied history.
+*
+* Include the differ's source/target types as well as the normalized SQL
+* shape. The type data keeps fallback identities meaningful when SQL is not
+* available, while the SQL fingerprint distinguishes conversions with the
+* same endpoints but different semantics (for example a UTC interpretation).
+*/
+function typeUpgradeFingerprint(action) {
+	const source = action.mismatch?.actual ?? "";
+	const target = action.mismatch?.expected ?? action.column?.type ?? "";
+	const conversion = sqlShapeFingerprint(action);
+	if (!source && !target && !conversion) return "";
+	return createHash("sha256").update(`source:${source};target:${target};conversion:${conversion}`).digest("hex").slice(0, 8);
+}
 function classifyTypeUpgradeSql(sql) {
 	const trimmed = sql?.trim();
 	if (!trimmed) return "manual";
@@ -711,7 +729,11 @@ function getSyntheticMigrationNameForAction(action) {
 			const fingerprint = sqlShapeFingerprint(action);
 			return fingerprint ? `drop_index_${action.indexName}_${fingerprint}` : `drop_index_${action.indexName}`;
 		}
-		case "type_upgrade": return action.column ? `type_upgrade_${action.tableName}_${action.column.name}` : null;
+		case "type_upgrade": {
+			if (!action.column) return null;
+			const fingerprint = typeUpgradeFingerprint(action);
+			return fingerprint ? `type_upgrade_${action.tableName}_${action.column.name}_${fingerprint}` : `type_upgrade_${action.tableName}_${action.column.name}`;
+		}
 		default: return null;
 	}
 }
@@ -736,10 +758,31 @@ function getSyntheticMigrationNameForChange(change) {
 			});
 			return fingerprint ? `drop_index_${change.name}_${fingerprint}` : `drop_index_${change.name}`;
 		}
-		case "type_upgrade": return change.name ? `type_upgrade_${change.table}_${change.name}` : null;
+		case "type_upgrade": {
+			if (!change.name) return null;
+			const fingerprint = typeUpgradeFingerprint(change);
+			return fingerprint ? `type_upgrade_${change.table}_${change.name}_${fingerprint}` : `type_upgrade_${change.table}_${change.name}`;
+		}
 		default: return null;
 	}
 }
+/**
+* Return every tracker name that can represent the current schema change.
+*
+* New type upgrades use a fingerprinted id, but releases before #2111 wrote
+* failures under the bare table-and-column id. Keep that old id as a
+* classification alias only: execution always uses the fingerprinted id, so
+* no applied migration record is rewritten or reused for a new conversion.
+*/
+function getSyntheticMigrationNamesForChange(change) {
+	const migrationName = getSyntheticMigrationNameForChange(change);
+	if (!migrationName) return [];
+	if (change.type === "type_upgrade" && change.name) {
+		const legacyName = `type_upgrade_${change.table}_${change.name}`;
+		return migrationName === legacyName ? [migrationName] : [migrationName, legacyName];
+	}
+	return [migrationName];
+}
 function shouldFailDbMigrate(state) {
 	return (state.tableErrorCount ?? 0) > 0 || (state.migrationErrorCount ?? 0) > 0 || (state.stiErrorCount ?? 0) > 0 || !state.dryRun && (state.manualInterventionCount ?? 0) > 0;
 }
@@ -755,8 +798,7 @@ function getUnresolvedGeneratedMigrationNames(changes) {
 	for (const change of changes) {
 		if (change.type !== "add_column" && change.type !== "add_index" && change.type !== "drop_index" && change.type !== "type_upgrade") continue;
 		if (change.type === "type_upgrade" && classifyTypeUpgradeSql(change.sql) === "noop") continue;
-		const migrationName = getSyntheticMigrationNameForChange(change);
-		if (migrationName) names.add(migrationName);
+		for (const migrationName of getSyntheticMigrationNamesForChange(change)) names.add(migrationName);
 	}
 	return names;
 }
@@ -891,18 +933,15 @@ function partitionSchemaChanges(changes, getClassForTable) {
 }
 //#endregion
 //#region src/commands/migration-failure-analysis.ts
-function getActionableGeneratedMigrationName(change) {
+function getActionableGeneratedMigrationNames(change) {
 	if (change.type === "type_upgrade" && change.name) {
-		if (classifyTypeUpgradeSql(change.sql) === "noop") return null;
+		if (classifyTypeUpgradeSql(change.sql) === "noop") return [];
 	}
-	return getSyntheticMigrationNameForChange(change);
+	return getSyntheticMigrationNamesForChange(change);
 }
 function getActionableFailedMigrationNames(diff) {
 	const names = /* @__PURE__ */ new Set();
-	for (const change of diff.changes) {
-		const name = getActionableGeneratedMigrationName(change);
-		if (name) names.add(name);
-	}
+	for (const change of diff.changes) for (const name of getActionableGeneratedMigrationNames(change)) names.add(name);
 	return names;
 }
 function classifyKind(name) {

package/dist/index.js

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@happyvertical/smrt-cli",
-  "version": "0.40.24",
+  "version": "0.40.25",
   "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.40.24",
-    "@happyvertical/smrt-config": "0.40.24",
-    "@happyvertical/smrt-core": "0.40.24",
-    "@happyvertical/smrt-dev-mcp": "0.40.24",
-    "@happyvertical/smrt-playground": "0.40.24",
-    "@happyvertical/smrt-types": "0.40.24"
+    "@happyvertical/smrt-agents": "0.40.25",
+    "@happyvertical/smrt-config": "0.40.25",
+    "@happyvertical/smrt-core": "0.40.25",
+    "@happyvertical/smrt-dev-mcp": "0.40.25",
+    "@happyvertical/smrt-types": "0.40.25",
+    "@happyvertical/smrt-playground": "0.40.25"
   },
   "devDependencies": {
     "@types/node": "24.13.2",