@bonnard/cli 0.2.0 → 0.2.1

package/dist/bin/bon.mjs

@@ -1519,7 +1519,6 @@ async function addDemo(options) {
 	console.log(pc.dim("  fact_sales, dim_product, dim_store, dim_customer"));
 	console.log();
 	console.log(pc.dim(`Test connection: bon datasource test ${name}`));
-	console.log(pc.dim(`Explore tables:  bon preview ${name} "SELECT table_name FROM information_schema.tables WHERE table_schema = 'contoso'"`));
 }
 /**
 * Main datasource add command
@@ -1626,464 +1625,11 @@ async function datasourceListCommand(options = {}) {
 	if (showRemote) await listRemoteDatasources();
 }
 
-//#endregion
-//#region src/lib/connection/snowflake.ts
-/**
-* Snowflake connection testing and querying
-*/
-const require$4 = createRequire(import.meta.url);
-function loadSnowflake() {
-	try {
-		const snowflake = require$4("snowflake-sdk");
-		snowflake.configure({ logLevel: "ERROR" });
-		return snowflake;
-	} catch {
-		return null;
-	}
-}
-async function testSnowflakeConnection(config, credentials) {
-	const snowflake = loadSnowflake();
-	if (!snowflake) return {
-		success: false,
-		message: "Snowflake driver not installed",
-		error: "Run: pnpm add snowflake-sdk"
-	};
-	const startTime = Date.now();
-	return new Promise((resolve) => {
-		const connection = snowflake.createConnection({
-			account: config.account,
-			username: credentials.username,
-			password: credentials.password,
-			database: config.database,
-			warehouse: config.warehouse,
-			schema: config.schema,
-			role: config.role
-		});
-		connection.connect((err) => {
-			if (err) {
-				resolve({
-					success: false,
-					message: "Connection failed",
-					error: err.message
-				});
-				return;
-			}
-			connection.execute({
-				sqlText: "SELECT 1",
-				complete: (queryErr) => {
-					const latencyMs = Date.now() - startTime;
-					connection.destroy(() => {});
-					if (queryErr) resolve({
-						success: false,
-						message: "Query failed",
-						error: queryErr.message,
-						latencyMs
-					});
-					else resolve({
-						success: true,
-						message: "Connection successful",
-						latencyMs
-					});
-				}
-			});
-		});
-	});
-}
-async function querySnowflake(config, credentials, sql, options = {}) {
-	const snowflake = loadSnowflake();
-	if (!snowflake) return {
-		columns: [],
-		rows: [],
-		rowCount: 0,
-		truncated: false,
-		error: "Snowflake driver not installed. Run: pnpm add snowflake-sdk"
-	};
-	const limit = options.limit ?? 1e3;
-	return new Promise((resolve) => {
-		const connection = snowflake.createConnection({
-			account: config.account,
-			username: credentials.username,
-			password: credentials.password,
-			database: options.database || config.database,
-			warehouse: config.warehouse,
-			schema: options.schema || config.schema,
-			role: config.role
-		});
-		connection.connect((err) => {
-			if (err) {
-				resolve({
-					columns: [],
-					rows: [],
-					rowCount: 0,
-					truncated: false,
-					error: err.message
-				});
-				return;
-			}
-			connection.execute({
-				sqlText: sql,
-				complete: (queryErr, _stmt, rows) => {
-					connection.destroy(() => {});
-					if (queryErr) {
-						resolve({
-							columns: [],
-							rows: [],
-							rowCount: 0,
-							truncated: false,
-							error: queryErr.message
-						});
-						return;
-					}
-					const allRows = rows || [];
-					const truncated = allRows.length > limit;
-					const resultRows = truncated ? allRows.slice(0, limit) : allRows;
-					resolve({
-						columns: resultRows.length > 0 ? Object.keys(resultRows[0]) : [],
-						rows: resultRows,
-						rowCount: resultRows.length,
-						truncated
-					});
-				}
-			});
-		});
-	});
-}
-
-//#endregion
-//#region src/lib/connection/postgres.ts
-/**
-* Postgres connection testing and querying
-*/
-const require$3 = createRequire(import.meta.url);
-function loadPg() {
-	try {
-		return require$3("pg");
-	} catch {
-		return null;
-	}
-}
-function createClient(config, credentials, pg) {
-	return new pg.Client({
-		host: config.host,
-		port: config.port ? parseInt(config.port, 10) : 5432,
-		database: config.database,
-		user: credentials.username,
-		password: credentials.password,
-		ssl: config.sslmode === "require" ? { rejectUnauthorized: false } : void 0
-	});
-}
-async function testPostgresConnection(config, credentials) {
-	const pg = loadPg();
-	if (!pg) return {
-		success: false,
-		message: "Postgres driver not installed",
-		error: "Run: pnpm add pg"
-	};
-	const startTime = Date.now();
-	const client = createClient(config, credentials, pg);
-	try {
-		await client.connect();
-		await client.query("SELECT 1");
-		const latencyMs = Date.now() - startTime;
-		await client.end();
-		return {
-			success: true,
-			message: "Connection successful",
-			latencyMs
-		};
-	} catch (err) {
-		try {
-			await client.end();
-		} catch {}
-		return {
-			success: false,
-			message: "Connection failed",
-			error: err.message
-		};
-	}
-}
-async function queryPostgres(config, credentials, sql, options = {}) {
-	const pg = loadPg();
-	if (!pg) return {
-		columns: [],
-		rows: [],
-		rowCount: 0,
-		truncated: false,
-		error: "Postgres driver not installed. Run: pnpm add pg"
-	};
-	const limit = options.limit ?? 1e3;
-	const client = createClient(config, credentials, pg);
-	try {
-		await client.connect();
-		const schema = options.schema || config.schema;
-		if (schema) {
-			if (!/^[a-zA-Z_][a-zA-Z0-9_]*$/.test(schema)) throw new Error("Invalid schema name");
-			await client.query(`SET search_path TO "${schema}"`);
-		}
-		const result = await client.query(sql);
-		await client.end();
-		const columns = result.fields?.map((f) => f.name) || [];
-		const allRows = result.rows || [];
-		const truncated = allRows.length > limit;
-		const rows = truncated ? allRows.slice(0, limit) : allRows;
-		return {
-			columns,
-			rows,
-			rowCount: rows.length,
-			truncated
-		};
-	} catch (err) {
-		try {
-			await client.end();
-		} catch {}
-		return {
-			columns: [],
-			rows: [],
-			rowCount: 0,
-			truncated: false,
-			error: err.message
-		};
-	}
-}
-
-//#endregion
-//#region src/lib/connection/bigquery.ts
-/**
-* BigQuery connection testing
-*/
-const require$2 = createRequire(import.meta.url);
-async function testBigQueryConnection(config, credentials) {
-	let BigQuery;
-	try {
-		BigQuery = require$2("@google-cloud/bigquery").BigQuery;
-	} catch {
-		return {
-			success: false,
-			message: "BigQuery driver not installed",
-			error: "Run: pnpm add @google-cloud/bigquery"
-		};
-	}
-	const startTime = Date.now();
-	try {
-		const options = { projectId: config.project_id };
-		if (config.location) options.location = config.location;
-		if (credentials.service_account_json) options.credentials = JSON.parse(credentials.service_account_json);
-		else if (credentials.keyfile_path) options.keyFilename = credentials.keyfile_path;
-		await new BigQuery(options).query("SELECT 1");
-		return {
-			success: true,
-			message: "Connection successful",
-			latencyMs: Date.now() - startTime
-		};
-	} catch (err) {
-		return {
-			success: false,
-			message: "Connection failed",
-			error: err.message
-		};
-	}
-}
-
-//#endregion
-//#region src/lib/connection/databricks.ts
-/**
-* Databricks connection testing
-*/
-const require$1 = createRequire(import.meta.url);
-async function testDatabricksConnection(config, credentials) {
-	let DBSQLClient;
-	try {
-		const module = require$1("@databricks/sql");
-		DBSQLClient = module.default || module;
-	} catch {
-		return {
-			success: false,
-			message: "Databricks driver not installed",
-			error: "Run: pnpm add @databricks/sql"
-		};
-	}
-	const startTime = Date.now();
-	const client = new DBSQLClient();
-	try {
-		const connection = await client.connect({
-			host: config.hostname,
-			path: config.http_path,
-			token: credentials.token
-		});
-		const session = await connection.openSession({
-			initialCatalog: config.catalog,
-			initialSchema: config.schema
-		});
-		const operation = await session.executeStatement("SELECT 1");
-		await operation.fetchAll();
-		await operation.close();
-		const latencyMs = Date.now() - startTime;
-		await session.close();
-		await connection.close();
-		return {
-			success: true,
-			message: "Connection successful",
-			latencyMs
-		};
-	} catch (err) {
-		try {
-			await client.close();
-		} catch {}
-		return {
-			success: false,
-			message: "Connection failed",
-			error: err.message
-		};
-	}
-}
-
-//#endregion
-//#region src/lib/connection/index.ts
-var connection_exports = /* @__PURE__ */ __exportAll({
-	executeQuery: () => executeQuery,
-	testConnection: () => testConnection
-});
-/**
-* Test connection to a datasource
-*/
-async function testConnection(datasource) {
-	const { type, config, credentials } = datasource;
-	switch (type) {
-		case "snowflake": return testSnowflakeConnection({
-			account: config.account,
-			database: config.database,
-			warehouse: config.warehouse,
-			schema: config.schema,
-			role: config.role
-		}, {
-			username: credentials.username,
-			password: credentials.password
-		});
-		case "postgres": return testPostgresConnection({
-			host: config.host,
-			port: config.port,
-			database: config.database,
-			schema: config.schema,
-			sslmode: config.sslmode
-		}, {
-			username: credentials.username,
-			password: credentials.password
-		});
-		case "bigquery": return testBigQueryConnection({
-			project_id: config.project_id,
-			dataset: config.dataset,
-			location: config.location
-		}, {
-			service_account_json: credentials.service_account_json,
-			keyfile_path: credentials.keyfile_path
-		});
-		case "databricks": return testDatabricksConnection({
-			hostname: config.hostname,
-			http_path: config.http_path,
-			catalog: config.catalog,
-			schema: config.schema
-		}, { token: credentials.token });
-		default: return {
-			success: false,
-			message: `Unsupported warehouse type: ${type}`
-		};
-	}
-}
-/**
-* Execute a query against a datasource
-*/
-async function executeQuery(datasource, sql, options = {}) {
-	const { type, config, credentials } = datasource;
-	switch (type) {
-		case "snowflake": return querySnowflake({
-			account: config.account,
-			database: config.database,
-			warehouse: config.warehouse,
-			schema: config.schema,
-			role: config.role
-		}, {
-			username: credentials.username,
-			password: credentials.password
-		}, sql, options);
-		case "postgres": return queryPostgres({
-			host: config.host,
-			port: config.port,
-			database: config.database,
-			schema: config.schema,
-			sslmode: config.sslmode
-		}, {
-			username: credentials.username,
-			password: credentials.password
-		}, sql, options);
-		case "bigquery": return {
-			columns: [],
-			rows: [],
-			rowCount: 0,
-			truncated: false,
-			error: "BigQuery local querying not yet implemented"
-		};
-		case "databricks": return {
-			columns: [],
-			rows: [],
-			rowCount: 0,
-			truncated: false,
-			error: "Databricks local querying not yet implemented"
-		};
-		default: return {
-			columns: [],
-			rows: [],
-			rowCount: 0,
-			truncated: false,
-			error: `Unsupported warehouse type: ${type}`
-		};
-	}
-}
-
 //#endregion
 //#region src/commands/datasource/test.ts
-async function datasourceTestCommand(name, options = {}) {
-	const localDs = getLocalDatasource(name);
-	if (options.remote || !localDs) {
-		await testRemote(name, !localDs);
-		return;
-	}
-	await testLocal(name, localDs);
-}
-/**
-* Test datasource locally using direct connection
-*/
-async function testLocal(name, ds) {
-	console.log(pc.dim(`Testing ${name} locally...`));
-	console.log();
-	const { resolved, missing } = resolveEnvVarsInCredentials(ds.credentials);
-	if (missing.length > 0) {
-		console.log(pc.red(`Missing environment variables: ${missing.join(", ")}`));
-		console.log(pc.dim("Set these env vars or update .bon/datasources.yaml with actual values."));
-		process.exit(1);
-	}
-	const result = await testConnection({
-		type: ds.type,
-		config: ds.config,
-		credentials: resolved
-	});
-	if (result.success) {
-		console.log(pc.green(`✓ ${result.message}`));
-		if (result.latencyMs) console.log(pc.dim(`  Latency: ${result.latencyMs}ms`));
-	} else {
-		console.log(pc.red(`✗ ${result.message}`));
-		if (result.error) console.log(pc.dim(`  ${result.error}`));
-		process.exit(1);
-	}
-}
-/**
-* Test datasource via remote API (requires login)
-*/
-async function testRemote(name, notFoundLocally) {
+async function datasourceTestCommand(name) {
 	if (!loadCredentials()) {
-		if (notFoundLocally) {
-			console.log(pc.red(`Datasource "${name}" not found locally.`));
-			console.log(pc.dim("Run `bon datasource add` to create it, or `bon login` to test remote datasources."));
-		} else console.log(pc.red("Not logged in. Run `bon login` to test remote datasources."));
+		console.log(pc.red("Not logged in. Run `bon login` to test datasources."));
 		process.exit(1);
 	}
 	console.log(pc.dim(`Testing ${name} via remote API...`));
@@ -2228,58 +1774,16 @@ async function pushDatasource(name, options = {}) {
 	}
 }
 
-//#endregion
-//#region src/commands/preview.ts
-async function previewCommand(datasourceName, sql, options) {
-	const limit = options.limit ? parseInt(options.limit, 10) : 1e3;
-	const format = options.format ?? "toon";
-	const ds = getLocalDatasource(datasourceName);
-	if (!ds) {
-		console.error(pc.red(`Datasource "${datasourceName}" not found in .bon/datasources.yaml`));
-		console.log(pc.dim("Run `bon datasource add` to create it."));
-		process.exit(1);
-	}
-	const { resolved, missing } = resolveEnvVarsInCredentials(ds.credentials);
-	if (missing.length > 0) {
-		console.error(pc.red(`Missing environment variables: ${missing.join(", ")}`));
-		console.log(pc.dim("Set these env vars or update .bon/datasources.yaml with actual values."));
-		process.exit(1);
-	}
-	const result = await executeQuery({
-		type: ds.type,
-		config: ds.config,
-		credentials: resolved
-	}, sql, {
-		limit,
-		schema: options.schema,
-		database: options.database
-	});
-	if (result.error) {
-		console.error(pc.red(result.error));
-		process.exit(1);
-	}
-	if (result.rowCount === 0) {
-		console.log("No rows returned.");
-		return;
-	}
-	if (format === "json") console.log(JSON.stringify(result, null, 2));
-	else {
-		const toon = encode({ results: result.rows });
-		console.log(toon);
-	}
-	if (result.truncated) console.log(pc.dim(`(truncated to ${result.rowCount} rows)`));
-}
-
 //#endregion
 //#region src/commands/validate.ts
-async function validateCommand(options = {}) {
+async function validateCommand() {
 	const cwd = process.cwd();
 	const paths = getProjectPaths(cwd);
 	if (!fs.existsSync(paths.config)) {
 		console.log(pc.red("No bon.yaml found. Are you in a Bonnard project?"));
 		process.exit(1);
 	}
-	const { validate } = await import("./validate-C31hmPk8.mjs");
+	const { validate } = await import("./validate-DEh1XQnH.mjs");
 	const result = await validate(cwd);
 	if (result.cubes.length === 0 && result.views.length === 0 && result.valid) {
 		console.log(pc.yellow(`No cube or view files found in ${BONNARD_DIR}/cubes/ or ${BONNARD_DIR}/views/.`));
@@ -2314,62 +1818,6 @@ async function validateCommand(options = {}) {
 		console.log(pc.dim("  This can cause issues when multiple warehouses are configured."));
 		console.log(pc.dim(`  ${result.cubesMissingDataSource.join(", ")}`));
 	}
-	if (options.testConnection) {
-		console.log();
-		await testReferencedConnections(cwd);
-	}
-}
-/**
-* Test connections for datasources referenced by cubes and views
-* Lenient: warns but doesn't fail validation
-*/
-async function testReferencedConnections(cwd) {
-	const { extractDatasourcesFromCubes } = await import("./cubes-De1_2_YJ.mjs");
-	const { loadLocalDatasources, resolveEnvVarsInCredentials } = await Promise.resolve().then(() => local_exports);
-	const { testConnection } = await Promise.resolve().then(() => connection_exports);
-	const references = extractDatasourcesFromCubes(cwd);
-	if (references.length === 0) {
-		console.log(pc.dim("No datasource references found in cubes."));
-		return;
-	}
-	console.log(pc.bold("Testing connections..."));
-	console.log();
-	const localDatasources = loadLocalDatasources(cwd);
-	let warnings = 0;
-	for (const ref of references) {
-		const ds = localDatasources.find((d) => d.name === ref.name);
-		if (!ds) {
-			console.log(pc.yellow(`⚠ ${ref.name}: not found in .bon/datasources.yaml`));
-			console.log(pc.dim(`    Used by: ${ref.cubes.join(", ")}`));
-			warnings++;
-			continue;
-		}
-		const { resolved, missing } = resolveEnvVarsInCredentials(ds.credentials);
-		if (missing.length > 0) {
-			console.log(pc.yellow(`⚠ ${ref.name}: missing env vars: ${missing.join(", ")}`));
-			console.log(pc.dim(`    Used by: ${ref.cubes.join(", ")}`));
-			warnings++;
-			continue;
-		}
-		const result = await testConnection({
-			type: ds.type,
-			config: ds.config,
-			credentials: resolved
-		});
-		if (result.success) {
-			const latency = result.latencyMs ? pc.dim(` (${result.latencyMs}ms)`) : "";
-			console.log(pc.green(`✓ ${ref.name}${latency}`));
-		} else {
-			console.log(pc.yellow(`⚠ ${ref.name}: ${result.error || result.message}`));
-			console.log(pc.dim(`    Used by: ${ref.cubes.join(", ")}`));
-			warnings++;
-		}
-	}
-	if (warnings > 0) {
-		console.log();
-		console.log(pc.yellow(`${warnings} connection warning(s)`));
-		console.log(pc.dim("Connection issues won't block file validation, but will fail at deploy."));
-	}
 }
 
 //#endregion
@@ -2401,7 +1849,7 @@ async function deployCommand(options = {}) {
 		process.exit(1);
 	}
 	console.log(pc.dim("Validating cubes and views..."));
-	const { validate } = await import("./validate-C31hmPk8.mjs");
+	const { validate } = await import("./validate-DEh1XQnH.mjs");
 	const result = await validate(cwd);
 	if (!result.valid) {
 		console.log(pc.red("Validation failed:\n"));
@@ -2470,51 +1918,29 @@ async function deployCommand(options = {}) {
 * Returns true if any connection failed (strict mode)
 */
 async function testAndSyncDatasources(cwd, options = {}) {
-	const { extractDatasourcesFromCubes } = await import("./cubes-De1_2_YJ.mjs");
-	const { loadLocalDatasources, resolveEnvVarsInCredentials } = await Promise.resolve().then(() => local_exports);
-	const { testConnection } = await Promise.resolve().then(() => connection_exports);
+	const { extractDatasourcesFromCubes } = await import("./cubes-Bf0IPYd7.mjs");
+	const { loadLocalDatasources } = await Promise.resolve().then(() => local_exports);
 	const { pushDatasource } = await Promise.resolve().then(() => push_exports);
 	const references = extractDatasourcesFromCubes(cwd);
 	if (references.length === 0) return false;
 	console.log();
-	console.log(pc.dim("Testing datasource connections..."));
+	console.log(pc.dim("Checking datasources..."));
 	const localDatasources = loadLocalDatasources(cwd);
 	let failed = false;
-	const validatedDatasources = [];
+	const foundDatasources = [];
 	for (const ref of references) {
-		const ds = localDatasources.find((d) => d.name === ref.name);
-		if (!ds) {
+		if (!localDatasources.find((d) => d.name === ref.name)) {
 			console.log(pc.red(`✗ ${ref.name}: not found in .bon/datasources.yaml`));
 			console.log(pc.dim(`    Used by: ${ref.cubes.join(", ")}`));
 			console.log(pc.dim(`    Run: bon datasource add --from-dbt`));
 			failed = true;
 			continue;
 		}
-		const { resolved, missing } = resolveEnvVarsInCredentials(ds.credentials);
-		if (missing.length > 0) {
-			console.log(pc.red(`✗ ${ref.name}: missing env vars: ${missing.join(", ")}`));
-			console.log(pc.dim(`    Used by: ${ref.cubes.join(", ")}`));
-			failed = true;
-			continue;
-		}
-		const result = await testConnection({
-			type: ds.type,
-			config: ds.config,
-			credentials: resolved
-		});
-		if (result.success) {
-			const latency = result.latencyMs ? pc.dim(` (${result.latencyMs}ms)`) : "";
-			console.log(pc.green(`✓ ${ref.name}${latency}`));
-			validatedDatasources.push(ref.name);
-		} else {
-			console.log(pc.red(`✗ ${ref.name}: ${result.error || result.message}`));
-			console.log(pc.dim(`    Used by: ${ref.cubes.join(", ")}`));
-			failed = true;
-		}
+		foundDatasources.push(ref.name);
 	}
-	console.log();
 	if (failed) {
-		console.log(pc.red("Connection tests failed. Fix datasource issues before deploying."));
+		console.log();
+		console.log(pc.red("Missing datasources. Fix issues before deploying."));
 		return true;
 	}
 	console.log(pc.dim("Checking remote datasources..."));
@@ -2526,22 +1952,17 @@ async function testAndSyncDatasources(cwd, options = {}) {
 		return true;
 	}
 	const remoteNames = new Set(remoteDatasources.map((ds) => ds.name));
-	const missingRemote = validatedDatasources.filter((name) => !remoteNames.has(name));
-	if (missingRemote.length === 0) {
-		console.log(pc.green("✓ All datasources exist on remote"));
+	const missingRemote = foundDatasources.filter((name) => !remoteNames.has(name));
+	if (missingRemote.length > 0) {
 		console.log();
-		return false;
-	}
-	console.log();
-	console.log(pc.yellow(`⚠ Missing remote datasource${missingRemote.length > 1 ? "s" : ""}: ${missingRemote.join(", ")}`));
-	console.log();
-	if (options.ci) {
-		console.log(pc.red("Deploy aborted (--ci mode)."));
-		console.log(pc.dim(`Run: bon datasource push <name>`));
-		return true;
-	}
-	if (options.pushDatasources) {
-		for (const name of missingRemote) {
+		console.log(pc.yellow(`⚠ Missing remote datasource${missingRemote.length > 1 ? "s" : ""}: ${missingRemote.join(", ")}`));
+		console.log();
+		if (options.ci) {
+			console.log(pc.red("Deploy aborted (--ci mode)."));
+			console.log(pc.dim(`Run: bon datasource push <name>`));
+			return true;
+		}
+		if (options.pushDatasources) for (const name of missingRemote) {
 			console.log(pc.dim(`Pushing "${name}"...`));
 			if (await pushDatasource(name, { silent: true })) console.log(pc.green(`✓ Pushed "${name}"`));
 			else {
@@ -2549,25 +1970,46 @@ async function testAndSyncDatasources(cwd, options = {}) {
 				return true;
 			}
 		}
-		console.log();
-		return false;
-	}
-	if (!await confirm({
-		message: `Push ${missingRemote.length > 1 ? "these datasources" : `"${missingRemote[0]}"`} to Bonnard? (credentials will be encrypted)`,
-		default: true
-	})) {
-		console.log(pc.dim("Deploy aborted."));
-		return true;
+		else {
+			if (!await confirm({
+				message: `Push ${missingRemote.length > 1 ? "these datasources" : `"${missingRemote[0]}"`} to Bonnard? (credentials will be encrypted)`,
+				default: true
+			})) {
+				console.log(pc.dim("Deploy aborted."));
+				return true;
+			}
+			console.log();
+			for (const name of missingRemote) {
+				console.log(pc.dim(`Pushing "${name}"...`));
+				if (await pushDatasource(name, { silent: true })) console.log(pc.green(`✓ Pushed "${name}"`));
+				else {
+					console.log(pc.red(`✗ Failed to push "${name}"`));
+					return true;
+				}
+			}
+		}
 	}
 	console.log();
-	for (const name of missingRemote) {
-		console.log(pc.dim(`Pushing "${name}"...`));
-		if (await pushDatasource(name, { silent: true })) console.log(pc.green(`✓ Pushed "${name}"`));
-		else {
-			console.log(pc.red(`✗ Failed to push "${name}"`));
-			return true;
+	console.log(pc.dim("Testing datasource connections..."));
+	for (const name of foundDatasources) try {
+		const result = await post("/api/datasources/test", { name });
+		if (result.success) {
+			const latency = result.details?.latencyMs ? pc.dim(` (${result.details.latencyMs}ms)`) : "";
+			console.log(pc.green(`✓ ${name}${latency}`));
+		} else {
+			console.log(pc.red(`✗ ${name}: ${result.message}`));
+			failed = true;
 		}
+	} catch (err) {
+		console.log(pc.red(`✗ ${name}: ${err.message}`));
+		failed = true;
+	}
+	console.log();
+	if (failed) {
+		console.log(pc.red("Connection tests failed. Fix datasource issues before deploying."));
+		return true;
 	}
+	console.log(pc.green("✓ All datasources connected"));
 	console.log();
 	return false;
 }
@@ -3093,12 +2535,11 @@ program.command("whoami").description("Show current login status").option("--ver
 const datasource = program.command("datasource").description("Manage warehouse data source connections");
 datasource.command("add").description("Add a data source to .bon/datasources.yaml. Use --name and --type together for non-interactive mode").option("--demo", "Add a read-only demo datasource (Contoso retail dataset) for testing").option("--from-dbt [profile]", "Import from dbt profiles.yml (optionally specify profile/target)").option("--target <target>", "Target name when using --from-dbt").option("--all", "Import all connections from dbt profiles").option("--default-targets", "Import only default targets from dbt profiles (non-interactive)").option("--name <name>", "Datasource name (required for non-interactive mode)").option("--type <type>", "Warehouse type: snowflake, postgres, bigquery, databricks (required for non-interactive mode)").option("--account <account>", "Snowflake account identifier").option("--database <database>", "Database name").option("--schema <schema>", "Schema name").option("--warehouse <warehouse>", "Warehouse name (Snowflake)").option("--role <role>", "Role (Snowflake)").option("--host <host>", "Host (Postgres)").option("--port <port>", "Port (Postgres, default: 5432)").option("--project-id <projectId>", "GCP Project ID (BigQuery)").option("--dataset <dataset>", "Dataset name (BigQuery)").option("--location <location>", "Location (BigQuery)").option("--hostname <hostname>", "Server hostname (Databricks)").option("--http-path <httpPath>", "HTTP path (Databricks)").option("--catalog <catalog>", "Catalog name (Databricks)").option("--user <user>", "Username").option("--password <password>", "Password (use --password-env for env var reference)").option("--token <token>", "Access token (use --token-env for env var reference)").option("--service-account-json <json>", "Service account JSON (BigQuery)").option("--keyfile <path>", "Path to service account key file (BigQuery)").option("--password-env <varName>", "Env var name for password, stores as {{ env_var('NAME') }}").option("--token-env <varName>", "Env var name for token, stores as {{ env_var('NAME') }}").option("--force", "Overwrite existing datasource without prompting").action(datasourceAddCommand);
 datasource.command("list").description("List data sources (shows both local and remote by default)").option("--local", "Show only local data sources from .bon/datasources.yaml").option("--remote", "Show only remote data sources from Bonnard server (requires login)").action(datasourceListCommand);
-datasource.command("test").description("Test data source connectivity by connecting directly to the warehouse").argument("<name>", "Data source name from .bon/datasources.yaml").option("--remote", "Test via Bonnard API instead of direct connection (requires login)").action(datasourceTestCommand);
+datasource.command("test").description("Test data source connectivity via Bonnard API (requires login)").argument("<name>", "Data source name from .bon/datasources.yaml").action(datasourceTestCommand);
 datasource.command("remove").description("Remove a data source from .bon/datasources.yaml (local by default)").argument("<name>", "Data source name").option("--remote", "Remove from Bonnard server instead of local (requires login)").action(datasourceRemoveCommand);
 datasource.command("push").description("Push a local data source to Bonnard server (requires login)").argument("<name>", "Data source name from .bon/datasources.yaml").option("--force", "Overwrite if already exists on remote").action(datasourcePushCommand);
-program.command("preview").description("Preview data from a local warehouse using raw SQL (for development/exploration)").argument("<datasource>", "Data source name from .bon/datasources.yaml").argument("<sql>", "SQL query to execute").option("--schema <schema>", "Override schema").option("--database <database>", "Override database").option("--limit <limit>", "Max rows to return", "1000").option("--format <format>", "Output format: toon or json", "toon").action(previewCommand);
-program.command("validate").description("Validate YAML syntax in bonnard/cubes/ and bonnard/views/").option("--test-connection", "Also test datasource connections (warns on failure, doesn't block)").action(validateCommand);
-program.command("deploy").description("Deploy cubes and views to Bonnard. Requires login, validates, tests connections (fails on error)").option("--ci", "Non-interactive mode (fail if missing datasources)").option("--push-datasources", "Auto-push missing datasources without prompting").requiredOption("-m, --message <text>", "Deploy message describing your changes").action(deployCommand);
+program.command("validate").description("Validate YAML syntax in bonnard/cubes/ and bonnard/views/").action(validateCommand);
+program.command("deploy").description("Deploy cubes and views to Bonnard. Requires login, validates, syncs datasources").option("--ci", "Non-interactive mode (fail if missing datasources)").option("--push-datasources", "Auto-push missing datasources without prompting").requiredOption("-m, --message <text>", "Deploy message describing your changes").action(deployCommand);
 program.command("deployments").description("List deployment history").option("--all", "Show all deployments (default: last 10)").option("--format <format>", "Output format: table or json", "table").action(deploymentsCommand);
 program.command("diff").description("Show changes in a deployment").argument("<id>", "Deployment ID").option("--format <format>", "Output format: table or json", "table").option("--breaking", "Show only breaking changes").action(diffCommand);
 program.command("annotate").description("Annotate deployment changes with reasoning").argument("<id>", "Deployment ID").option("--data <json>", "Annotations JSON").action(annotateCommand);