@happyvertical/smrt-cli 0.40.23 → 0.40.25

package/README.md

@@ -57,7 +57,7 @@ batch.
 
 ### Audited PostgreSQL timestamp conversion
 
-`timestamp without time zone` values do not carry enough information for SMRT
+`timestamp without time zone` values do not carry enough information for s-m-r-t
 to infer their original instant. After auditing every historical writer,
 database default, trigger, and raw SQL path, an operator may confirm that the
 legacy values are UTC wall times and include the exact opt-in:
@@ -155,7 +155,7 @@ Custom methods on s-m-r-t objects are auto-discovered from manifests. Method par
 ## Usage
 
 ```bash
-# Discover what SMRT objects are available
+# Discover what s-m-r-t objects are available
 smrt introspect
 
 # Generate an MCP server

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.23",
+  "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.23",
-    "@happyvertical/smrt-config": "0.40.23",
-    "@happyvertical/smrt-dev-mcp": "0.40.23",
-    "@happyvertical/smrt-core": "0.40.23",
-    "@happyvertical/smrt-playground": "0.40.23",
-    "@happyvertical/smrt-types": "0.40.23"
+    "@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",