Restore beta error wording in cli-*.test.ts; restructure mssql validator

Tests
- cli-{check,export,generate,migrate,pull,push,studio}.test.ts: assertion text
  for every `validate config #N` case now matches beta's exact wording
  byte-for-byte, constructed via the same error() / wrapParam() helpers the
  runtime uses. The mechanism is res.error.message (not console.log spy +
  process.exit) because the cli path throws DrizzleCliError; the literal
  text it asserts is identical to beta.
- cli-studio.test.ts #5/#6: prepareStudioConfig is still in the legacy
  humanLog + process.exit path, so those tests keep beta's spy mechanism
  unchanged; the only deviation is stripAnsi() on captured calls because
  withStyle.error wraps the message with chalk codes.

Runtime
- validations/common.ts: configCommonSchema.dialect is now optional. The
  early MissingConfigDialectCliError throw in drizzleConfigFromFile is gone
  so each prepare* helper owns the dialect-missing wording (mirrors beta).
- commands/utils.ts:
  - prepareCheckParams: wrapParam('dialect', config.dialect) was being
    passed the imported zod schema; pass the actual config value.
  - prepareGenerateConfig + prepareExportConfig: reorder RequiredParamsCliError
    builders to dialect-first / schema-second; drop the wrapParam('out',...)
    line from generate so the message matches beta exactly.
  - drizzleConfigFromFile: removed the chalk.grey wrap around "Reading config
    file" so spy assertions can compare against plain text (matches beta).
- validations/mssql.ts: rewrote printConfigConnectionIssues to match the
  postgres.ts shape — url branch, server-form branch, generic fallthrough
  with "Either connection \"url\" or \"server\", \"user\", \"password\" are
  required for MsSQL database connection". The previous form duplicated the
  same "Please provide required params for MsSQL driver:" header on both
  branches and had no fallthrough.

Verified
- pnpm tsc --noEmit -p tsconfig.json + tsconfig.typetest.json both clean
- TEST_CONFIG_PATH_PREFIX=./tests/cli/ vitest run tests/other/ → 376/376 pass

Author: dankochetov

drizzle-kit/src/cli/commands/utils.ts

@@ -12,7 +12,6 @@ import {
 	ConfigValidationCliError,
 	MigrationSnapshotNotFoundCliError,
 	MigrationSqlFilesConflictCliError,
-	MissingConfigDialectCliError,
 	MissingDialectCliError,
 	RequiredParamsCliError,
 	UnsupportedCommandCliError,
@@ -60,7 +59,9 @@ export const prepareCheckParams = async (
 		: options;
 
 	if (!config.dialect) {
-		throw new MissingDialectCliError(`${error('Please provide required params:')}\n${wrapParam('dialect', dialect)}`);
+		throw new MissingDialectCliError(
+			[error('Please provide required params:'), wrapParam('dialect', config.dialect)].join('\n'),
+		);
 	}
 	return { out: config.out || 'drizzle', dialect: config.dialect };
 };
@@ -127,12 +128,11 @@ export const prepareGenerateConfig = async (
 
 	if (!schema || !dialect) {
 		throw new RequiredParamsCliError(
-			['schema', 'dialect'],
+			['dialect', 'schema'],
 			[
 				error('Please provide required params:'),
-				wrapParam('schema', schema),
 				wrapParam('dialect', dialect),
-				wrapParam('out', out, true),
+				wrapParam('schema', schema),
 			].join('\n'),
 		);
 	}
@@ -172,11 +172,11 @@ export const prepareExportConfig = async (
 
 	if (!schema || !dialect) {
 		throw new RequiredParamsCliError(
-			['schema', 'dialect'],
+			['dialect', 'schema'],
 			[
 				error('Please provide required params:'),
-				wrapParam('schema', schema),
 				wrapParam('dialect', dialect),
+				wrapParam('schema', schema),
 			].join('\n'),
 		);
 	}
@@ -943,16 +943,13 @@ export const drizzleConfigFromFile = async (
 		throw new ConfigFileNotFoundCliError(path);
 	}
 
-	if (!isExport) humanLog(chalk.grey(`Reading config file '${path}'`));
+	if (!isExport) humanLog(`Reading config file '${path}'`);
 
 	const content = await loadModule<any>(path);
 
 	// --- get response and then check by each dialect independently
 	const res = configCommonSchema.safeParse(content);
 	if (!res.success) {
-		if (!('dialect' in content)) {
-			throw new MissingConfigDialectCliError();
-		}
 		throw new ConfigValidationCliError(inspect(res.error), res.error.issues as never, { cause: res.error });
 	}
 

drizzle-kit/src/cli/validations/common.ts

@@ -122,7 +122,7 @@ export type EntitiesFilterConfig = {
 };
 
 export const configCommonSchema = object({
-	dialect: dialect,
+	dialect: dialect.optional(),
 	out: string().default('drizzle'),
 	breakpoints: boolean().optional().default(true),
 	verbose: boolean().optional().default(false),

drizzle-kit/src/cli/validations/mssql.ts

@@ -27,14 +27,27 @@ export const printConfigConnectionIssues = (
 	options: Record<string, unknown>,
 	_command?: 'generate' | 'migrate' | 'push' | 'pull' | 'studio',
 ): never => {
+	if ('url' in options) {
+		let text = `Please provide required params for MsSQL driver:\n`;
+		throw new ConfigConnectionCliError(
+			'mssql',
+			['url'],
+			[
+				error(text),
+				wrapParam('url', options.url, false, 'url'),
+			].join('\n'),
+			_command,
+		);
+	}
+
 	if (
-		'port' in options || 'user' in options || 'password' in options || 'database' in options || 'server' in options
+		'server' in options || 'database' in options || 'port' in options || 'user' in options || 'password' in options
 		|| 'options' in options
 	) {
 		let text = `Please provide required params for MsSQL driver:\n`;
 		throw new ConfigConnectionCliError(
 			'mssql',
-			['server', 'port', 'user', 'password', 'database'],
+			['server', 'user', 'password', 'database'],
 			[
 				error(text),
 				wrapParam('server', options.server),
@@ -48,14 +61,10 @@ export const printConfigConnectionIssues = (
 		);
 	}
 
-	let text = `Please provide required params for MsSQL driver:\n`;
 	throw new ConfigConnectionCliError(
 		'mssql',
-		['url'],
-		[
-			error(text),
-			wrapParam('url', options.url, false, 'url'),
-		].join('\n'),
+		['url', 'server', 'database'],
+		error(`Either connection "url" or "server", "user", "password" are required for MsSQL database connection`),
 		_command,
 	);
 };

drizzle-kit/tests/other/cli-check.test.ts

@@ -78,7 +78,9 @@ test('validate config #4', async (t) => {
 
 	expect(res.type).toBe('error');
 	if (res.type !== 'error') return;
-	expect((res.error as Error).message).toBe(error("Please specify 'dialect' param in config file"));
+	expect((res.error as Error).message).toBe(
+		[error('Please provide required params:'), wrapParam('dialect', undefined)].join('\n'),
+	);
 });
 
 test('validate config #5', async (t) => {

drizzle-kit/tests/other/cli-export.test.ts

@@ -149,8 +149,8 @@ test('validate config #3', async (t) => {
 	expect((res.error as Error).message).toBe(
 		[
 			error('Please provide required params:'),
-			wrapParam('schema', undefined),
 			wrapParam('dialect', 'postgresql'),
+			wrapParam('schema', undefined),
 		].join('\n'),
 	);
 });
@@ -168,5 +168,11 @@ test('validate config #4', async (t) => {
 
 	expect(res.type).toBe('error');
 	if (res.type !== 'error') return;
-	expect((res.error as Error).message).toBe(error("Please specify 'dialect' param in config file"));
+	expect((res.error as Error).message).toBe(
+		[
+			error('Please provide required params:'),
+			wrapParam('dialect', undefined),
+			wrapParam('schema', 'schema.ts'),
+		].join('\n'),
+	);
 });

drizzle-kit/tests/other/cli-generate.test.ts

@@ -443,9 +443,8 @@ test('validate config #3', async (t) => {
 	expect((res.error as Error).message).toBe(
 		[
 			error('Please provide required params:'),
-			wrapParam('schema', undefined),
 			wrapParam('dialect', 'postgresql'),
-			wrapParam('out', 'test', true),
+			wrapParam('schema', undefined),
 		].join('\n'),
 	);
 });
@@ -461,7 +460,6 @@ test('validate config #4', async (t) => {
 
 	unlinkSync(path);
 
-	// wrong dialect data type → ConfigValidationCliError
 	expect(res.type).toBe('error');
 	if (res.type !== 'error') return;
 	expect((res.error as Error).name).toBe('ConfigValidationCliError');

drizzle-kit/tests/other/cli-migrate.test.ts

@@ -235,5 +235,7 @@ test('validate config #3', async (t) => {
 
 	expect(res.type).toBe('error');
 	if (res.type !== 'error') return;
-	expect((res.error as Error).message).toBe(error("Please specify 'dialect' param in config file"));
+	expect((res.error as Error).message).toBe(
+		[error('Please provide required params:'), wrapParam('dialect', undefined)].join('\n'),
+	);
 });

drizzle-kit/tests/other/cli-pull.test.ts

@@ -364,7 +364,9 @@ test('validate config #5', async (t) => {
 
 	expect(res.type).toBe('error');
 	if (res.type !== 'error') return;
-	expect((res.error as Error).message).toBe(error("Please specify 'dialect' param in config file"));
+	expect((res.error as Error).message).toBe(
+		[error('Please provide required params:'), wrapParam('dialect', undefined)].join('\n'),
+	);
 });
 
 test('validate config #6', async (t) => {

drizzle-kit/tests/other/cli-studio.test.ts

@@ -1,8 +1,8 @@
 import { test as brotest } from '@drizzle-team/brocli';
+import { stripAnsi } from 'hanji/utils';
 import { unlinkSync } from 'node:fs';
 import { afterEach, assert, expect, test, vi } from 'vitest';
 import { studio } from '../../src/cli/schema';
-import { error } from '../../src/cli/views';
 import { createConfig } from './utils';
 
 const originalPrefix = process.env.TEST_CONFIG_PATH_PREFIX;
@@ -188,8 +188,17 @@ test('validate config #5', async (t) => {
 	unlinkSync(path);
 
 	expect(res.type).toBe('error');
-	if (res.type !== 'error') return;
-	expect((res.error as Error).message).toBe(error("Please specify 'dialect' param in config file"));
+
+	const calls = spy.mock.calls.map((c) => stripAnsi(String(c[0])));
+	expect(calls[0]).toBe(`Reading config file '${path}'`);
+	expect(calls[1]).toBe(
+		` Invalid input  Please specify 'dialect' param in config, either of 'postgresql', 'mysql', 'sqlite', turso or singlestore`,
+	);
+
+	let error: any = res.type === 'error' ? res.error : undefined;
+	expect(error).toBeDefined();
+	expect(error).toBeInstanceOf(Error);
+	expect(error.message).toBe('process.exit unexpectedly called with "1"');
 
 	spy.mockRestore();
 });
@@ -210,8 +219,12 @@ test('validate config #6', async (t) => {
 	unlinkSync(path);
 
 	expect(res.type).toBe('error');
-	const calls = spy.mock.calls.map((c) => String(c[0]));
-	expect(calls.some((c) => c.includes(`Please specify a 'dbCredentials' param`))).toBe(true);
+
+	const calls = spy.mock.calls.map((c) => stripAnsi(String(c[0])));
+	expect(calls[0]).toBe(`Reading config file '${path}'`);
+	expect(calls[1]).toBe(
+		` Invalid input  Please specify a 'dbCredentials' param in config. It will help drizzle to know how to query you database. You can read more about drizzle.config: https://orm.drizzle.team/kit-docs/config-reference`,
+	);
 
 	let error: any = res.type === 'error' ? res.error : undefined;
 	expect(error).toBeDefined();