@happyvertical/smrt-cli 0.41.0 → 0.42.0

package/AGENTS.md

@@ -14,6 +14,7 @@ smrt db:migrate              # Apply migrations
 smrt db:migrate --postgres-safe # PostgreSQL concurrent-index mode (see below)
 smrt db:migrate --force-migration <exact-id> [--force-migration <exact-id>...] # Force exact generated migrations in one atomic batch
 smrt db:migrate-uuid         # Convert schema-declared UUID text columns after data remap
+smrt db:migrate-int8         # Explicitly widen pre-#2373 int4 columns after preflight
 smrt db:diff                 # Show schema differences without generating migration files
 smrt db:rollback             # Roll back migrations by executing their recorded DOWN
 smrt db:rollback --mark-only # Record-only flip; schema deliberately untouched
@@ -95,6 +96,12 @@ not from the schema definition.
   (`indexIntrospection: 'unavailable'`) rather than inventing missing indexes.
 - The check is read-only and lives in a new core module; it does not share code
   with `migrations/differ.ts`.
+- PostgreSQL/DuckDB `int4` columns created before #2373 are advisory warnings,
+  not type drift: the differ intentionally treats int4/int8 as equivalent.
+  Run `smrt db:migrate-int8 --dry-run`, schedule the reported table rewrites,
+  then run `smrt db:migrate-int8`; SQLite is already 64-bit and is a no-op.
+  PostgreSQL uses the same bounded `migrations.postgres.lockTimeout` and
+  `statementTimeout` settings as ordinary schema migration.
 
 ## `db:migrate` on SQLite: type changes rebuild the table
 

package/dist/{commands-DjgCD-0o.js → commands-BOw91sjd.js}

@@ -20,6 +20,7 @@ import https from "node:https";
 import { homedir, tmpdir } from "node:os";
 import { extract } from "tar";
 import { importWorkspaceModule } from "@happyvertical/smrt-core/utils/import-workspace-module";
+import { buildIntegerWidthTableStatements, collectIntegerWidthTargets, parsePostgresTimeoutMs, preflightIntegerWidthWidening, widenIntegerColumnsToBigInt } from "@happyvertical/smrt-core/migrations";
 import { buildArchitectureContext, buildKnowledgeIndex, buildReviewContext, checkKnowledgeFreshness, diffKnowledgeIndex, renderFreshnessResult, renderKnowledgeIndexMarkdown } from "@happyvertical/smrt-dev-mcp/knowledge";
 //#region src/commands/db-command-utils.ts
 var SENSITIVE_QUERY_PARAMS = /* @__PURE__ */ new Set([
@@ -6035,6 +6036,100 @@ var playgroundCommands = {
 	}
 };
 //#endregion
+//#region src/commands/db-migrate-int8.ts
+/**
+* db:migrate-int8 Command
+*
+* Explicitly widens legacy PostgreSQL/DuckDB int4 columns created before
+* #2373. This is deliberately separate from `db:migrate`: PostgreSQL rewrites
+* each table, so an operator must review the row-count preflight and schedule
+* a maintenance window before opting in.
+*/
+var BACKFILL_NAME = "@happyvertical/smrt-core:integer-width:v1";
+var DEFAULT_POSTGRES_LOCK_TIMEOUT_MS = 3e4;
+var DEFAULT_POSTGRES_STATEMENT_TIMEOUT_MS = 6e4;
+var dbMigrateInt8Command = {
+	name: "db:migrate-int8",
+	description: "Widen legacy SMRT int4 columns to BIGINT after reviewing the maintenance-window preflight. Run after db:migrate.",
+	aliases: ["migrate-int8", "db-migrate-int8"],
+	args: [],
+	options: {
+		"dry-run": {
+			type: "boolean",
+			description: "Print the preflight and ALTER statements without writing.",
+			default: false
+		},
+		verbose: {
+			type: "boolean",
+			description: "Print the full per-table preflight report.",
+			default: false,
+			short: "v"
+		}
+	},
+	handler: async (_args, options) => {
+		let db;
+		const dryRun = Boolean(options["dry-run"]);
+		try {
+			const { getPackageConfig } = await import("@happyvertical/smrt-config");
+			const { DEFAULT_CLI_CONFIG } = await import("./config-BwrFRL8L.js");
+			const config = getPackageConfig("cli", DEFAULT_CLI_CONFIG);
+			if (!config.database?.url || config.database.url === ":memory:") throw new Error("Database configuration required for db:migrate-int8. Configure database.url in smrt.config.ts (or DATABASE_URL).");
+			await autoDiscoverAndLoad();
+			const schemas = ObjectRegistry.getAllSchemasAsDefinitions();
+			if (Object.keys(schemas).length === 0) throw new Error("No SMRT application schemas were discovered. Run this command from the project root after generating manifests.");
+			const dbType = config.database.type || "sqlite";
+			const dbUrl = config.database.url;
+			const { getDatabase } = await import("@happyvertical/sql");
+			db = await getDatabase({
+				type: dbType,
+				url: dbUrl
+			});
+			console.log("\n↔️  Integer-width migration\n");
+			console.log(`✓ Connected to ${formatDatabaseDisplayUrl(dbType, dbUrl)}\n`);
+			const targets = collectIntegerWidthTargets(schemas, { includeSystemTables: true });
+			const preflight = await preflightIntegerWidthWidening(db, targets, { engineHint: dbType });
+			console.log(preflight.summary);
+			if (options.verbose && preflight.supported) {
+				console.log("\nFull preflight report:");
+				for (const table of preflight.tables) {
+					const columns = table.columns.map((column) => `${column.column}: ${column.declaredType ?? "missing"} (${column.state})`).join(", ");
+					console.log(`  ${table.table}: ${table.rowCount ?? "not counted"} row(s); ${columns}`);
+				}
+			}
+			if (!preflight.supported) {
+				console.log("\nNo widening is needed on this engine.\n");
+				return;
+			}
+			if (preflight.unexpectedColumns > 0) throw new Error("Some schema-declared integer columns have unexpected live types. Resolve ordinary schema drift before this widening pass.");
+			if (preflight.pendingColumns === 0) {
+				console.log("\nNo legacy int4 columns remain.\n");
+				return;
+			}
+			const statements = preflight.tables.flatMap((table) => buildIntegerWidthTableStatements(preflight.engine, table.table, table.columns.filter((column) => column.state === "pending").map((column) => column.column)));
+			console.log(`\n${dryRun ? "DRY RUN — would execute" : "Applying"} ${statements.length} lossless ALTER statement(s):`);
+			for (const statement of statements) console.log(`  ${statement};`);
+			if (dryRun) {
+				console.log("\nDry run complete — no changes applied.\n");
+				return;
+			}
+			const postgresMigrationConfig = config.migrations?.postgres;
+			const result = await widenIntegerColumnsToBigInt(db, targets, {
+				engineHint: dbType,
+				backfillName: BACKFILL_NAME,
+				packageName: "@happyvertical/smrt-core",
+				lockTimeout: parsePostgresTimeoutMs(postgresMigrationConfig?.lockTimeout, DEFAULT_POSTGRES_LOCK_TIMEOUT_MS),
+				statementTimeout: parsePostgresTimeoutMs(postgresMigrationConfig?.statementTimeout, DEFAULT_POSTGRES_STATEMENT_TIMEOUT_MS)
+			});
+			console.log(result.ran ? `\n✓ Widened ${result.widenedColumns.length} column(s) to BIGINT.\n` : "\nNo widening was applied; no legacy int4 columns remain.\n");
+		} catch (error) {
+			console.error(`\n❌ int8 migration failed: ${error instanceof Error ? error.message : String(error)}\n`);
+			process.exitCode = 1;
+		} finally {
+			await closeDatabaseConnection(db);
+		}
+	}
+};
+//#endregion
 //#region src/commands/db-migrate-uuid.ts
 /**
 * db:migrate-uuid Command
@@ -8714,6 +8809,7 @@ export default testManifest;
 	"db:rollback": dbRollbackCommand,
 	"db:generate": dbGenerateCommand,
 	"db:migrate-uuid": dbMigrateUuidCommand,
+	"db:migrate-int8": dbMigrateInt8Command,
 	"db:prune": dbPruneCommand,
 	"config:export": configExportCommand,
 	export: exportCommand

package/dist/index.js

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

package/package.json

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