@bonnard/cli 0.2.2 → 0.2.4

package/dist/bin/bon.mjs

@@ -10,7 +10,6 @@ import os from "node:os";
 import http from "node:http";
 import crypto from "node:crypto";
 import { execFileSync } from "node:child_process";
-import { confirm } from "@inquirer/prompts";
 import { encode } from "@toon-format/toon";
 
 //#region rolldown:runtime
@@ -82,6 +81,7 @@ function mapDbtType(dbtType) {
 		snowflake: "snowflake",
 		postgres: "postgres",
 		postgresql: "postgres",
+		redshift: "redshift",
 		bigquery: "bigquery",
 		databricks: "databricks"
 	}[dbtType.toLowerCase()] ?? null;
@@ -338,6 +338,7 @@ function extractWarehouseFromEnv(cwd) {
 			const type = {
 				snowflake: "snowflake",
 				postgres: "postgres",
+				redshift: "redshift",
 				bigquery: "bigquery",
 				databricks: "databricks"
 			}[cubeDbType[1].trim().toLowerCase()];
@@ -787,7 +788,6 @@ async function logoutCommand() {
 var api_exports = /* @__PURE__ */ __exportAll({
 	del: () => del,
 	get: () => get,
-	getRemoteDatasources: () => getRemoteDatasources,
 	post: () => post
 });
 const APP_URL = process.env.BON_APP_URL || "https://app.bonnard.dev";
@@ -829,12 +829,6 @@ function post(path, body) {
 function del(path) {
 	return request("DELETE", path);
 }
-/**
-* Fetch remote datasources from Bonnard server
-*/
-async function getRemoteDatasources() {
-	return (await get("/api/datasources")).dataSources || [];
-}
 
 //#endregion
 //#region src/commands/whoami.ts
@@ -1221,6 +1215,43 @@ const WAREHOUSE_CONFIGS = [
 			required: true
 		}]
 	},
+	{
+		value: "redshift",
+		label: "Redshift",
+		configFields: [
+			{
+				name: "host",
+				message: "Host (cluster endpoint)",
+				required: true
+			},
+			{
+				name: "port",
+				message: "Port",
+				default: "5439"
+			},
+			{
+				name: "database",
+				message: "Database name",
+				required: true
+			},
+			{
+				name: "schema",
+				message: "Schema",
+				default: "public"
+			}
+		],
+		credentialFields: [{
+			name: "username",
+			flag: "user",
+			message: "Username",
+			required: true
+		}, {
+			name: "password",
+			message: "Password",
+			secret: true,
+			required: true
+		}]
+	},
 	{
 		value: "bigquery",
 		label: "BigQuery",
@@ -1490,7 +1521,7 @@ async function addManual(options) {
 	console.log();
 	console.log(pc.green(`✓ Datasource "${name}" saved to .bon/datasources.yaml`));
 	console.log();
-	console.log(pc.dim(`Test connection: bon datasource test ${name}`));
+	console.log(pc.dim(`Connection will be tested during \`bon deploy\``));
 }
 /**
 * Add the Contoso demo datasource (read-only retail dataset)
@@ -1523,7 +1554,7 @@ async function addDemo(options) {
 	console.log(pc.dim("Contoso is a read-only retail dataset with tables like:"));
 	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(`Connection will be tested during \`bon deploy\``));
 }
 /**
 * Main datasource add command
@@ -1630,34 +1661,6 @@ async function datasourceListCommand(options = {}) {
 	if (showRemote) await listRemoteDatasources();
 }
 
-//#endregion
-//#region src/commands/datasource/test.ts
-async function datasourceTestCommand(name) {
-	if (!loadCredentials()) {
-		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...`));
-	console.log();
-	try {
-		const result = await post("/api/datasources/test", { name });
-		if (result.success) {
-			console.log(pc.green(result.message));
-			if (result.details) {
-				if (result.details.warehouse) console.log(pc.dim(`  Warehouse: ${result.details.warehouse}`));
-				if (result.details.account) console.log(pc.dim(`  Account:   ${result.details.account}`));
-				if (result.details.latencyMs != null) console.log(pc.dim(`  Latency:   ${result.details.latencyMs}ms`));
-			}
-		} else {
-			console.log(pc.red(result.message));
-			process.exit(1);
-		}
-	} catch (err) {
-		console.error(pc.red(`Failed to test data source: ${err.message}`));
-		process.exit(1);
-	}
-}
-
 //#endregion
 //#region src/commands/datasource/remove.ts
 async function datasourceRemoveCommand(name, options = {}) {
@@ -1692,93 +1695,6 @@ async function removeRemote(name) {
 	}
 }
 
-//#endregion
-//#region src/commands/datasource/push.ts
-var push_exports = /* @__PURE__ */ __exportAll({
-	datasourcePushCommand: () => datasourcePushCommand,
-	pushDatasource: () => pushDatasource
-});
-/**
-* Push a local datasource to Bonnard server
-*/
-async function datasourcePushCommand(name, options = {}) {
-	if (!loadCredentials()) {
-		console.error(pc.red("Not logged in. Run `bon login` first."));
-		process.exit(1);
-	}
-	const datasource = getLocalDatasource(name);
-	if (!datasource) {
-		console.error(pc.red(`Datasource "${name}" not found in .bon/datasources.yaml`));
-		console.log(pc.dim("Run `bon datasource list --local` to see available datasources."));
-		process.exit(1);
-	}
-	const { resolved, missing } = resolveEnvVarsInCredentials(datasource.credentials);
-	if (missing.length > 0) {
-		console.error(pc.red(`Missing environment variables: ${missing.join(", ")}`));
-		console.log(pc.dim("Set them in your environment or use plain values in .bon/datasources.yaml"));
-		process.exit(1);
-	}
-	try {
-		if ((await getRemoteDatasources()).some((ds) => ds.name === name) && !options.force) {
-			if (!await confirm({
-				message: `Datasource "${name}" already exists on remote. Overwrite?`,
-				default: false
-			})) {
-				console.log(pc.dim("Aborted."));
-				process.exit(0);
-			}
-		}
-	} catch (err) {
-		console.log(pc.dim(`Note: Could not check remote datasources: ${err.message}`));
-	}
-	console.log(pc.dim(`Pushing "${name}"...`));
-	try {
-		await post("/api/datasources", {
-			name: datasource.name,
-			warehouse_type: datasource.type,
-			config: datasource.config,
-			credentials: resolved
-		});
-		console.log(pc.green(`✓ Datasource "${name}" pushed to Bonnard`));
-	} catch (err) {
-		const message = err.message;
-		if (message.includes("already exists")) {
-			console.error(pc.red(`Datasource "${name}" already exists on remote.`));
-			console.log(pc.dim("Use --force to overwrite."));
-			process.exit(1);
-		}
-		console.error(pc.red(`Failed to push datasource: ${message}`));
-		process.exit(1);
-	}
-}
-/**
-* Push a datasource programmatically (for use by deploy command)
-* Returns true on success, false on failure
-*/
-async function pushDatasource(name, options = {}) {
-	const datasource = getLocalDatasource(name);
-	if (!datasource) {
-		if (!options.silent) console.error(pc.red(`Datasource "${name}" not found locally`));
-		return false;
-	}
-	const { resolved, missing } = resolveEnvVarsInCredentials(datasource.credentials);
-	if (missing.length > 0) {
-		if (!options.silent) console.error(pc.red(`Missing env vars for "${name}": ${missing.join(", ")}`));
-		return false;
-	}
-	try {
-		await post("/api/datasources", {
-			name: datasource.name,
-			warehouse_type: datasource.type,
-			config: datasource.config,
-			credentials: resolved
-		});
-		return true;
-	} catch {
-		return false;
-	}
-}
-
 //#endregion
 //#region src/commands/validate.ts
 async function validateCommand() {
@@ -1788,7 +1704,7 @@ async function validateCommand() {
 		console.log(pc.red("No bon.yaml found. Are you in a Bonnard project?"));
 		process.exit(1);
 	}
-	const { validate } = await import("./validate-DEh1XQnH.mjs");
+	const { validate } = await import("./validate-BdqZBH2n.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/.`));
@@ -1854,7 +1770,7 @@ async function deployCommand(options = {}) {
 		process.exit(1);
 	}
 	console.log(pc.dim("Validating cubes and views..."));
-	const { validate } = await import("./validate-DEh1XQnH.mjs");
+	const { validate } = await import("./validate-BdqZBH2n.mjs");
 	const result = await validate(cwd);
 	if (!result.valid) {
 		console.log(pc.red("Validation failed:\n"));
@@ -1923,9 +1839,9 @@ async function deployCommand(options = {}) {
 * Returns true if any connection failed (strict mode)
 */
 async function testAndSyncDatasources(cwd, options = {}) {
-	const { extractDatasourcesFromCubes } = await import("./cubes-Bf0IPYd7.mjs");
+	const { extractDatasourcesFromCubes } = await import("./cubes-9rklhdAJ.mjs");
 	const { loadLocalDatasources } = await Promise.resolve().then(() => local_exports);
-	const { pushDatasource } = await Promise.resolve().then(() => push_exports);
+	const { pushDatasource } = await import("./push-Bv9AFGc2.mjs");
 	const references = extractDatasourcesFromCubes(cwd);
 	if (references.length === 0) return false;
 	console.log();
@@ -1948,51 +1864,16 @@ async function testAndSyncDatasources(cwd, options = {}) {
 		console.log(pc.red("Missing datasources. Fix issues before deploying."));
 		return true;
 	}
-	console.log(pc.dim("Checking remote datasources..."));
-	let remoteDatasources;
-	try {
-		remoteDatasources = await getRemoteDatasources();
-	} catch (err) {
-		console.log(pc.red(`Failed to fetch remote datasources: ${err.message}`));
-		return true;
+	console.log(pc.dim("Syncing datasources..."));
+	for (const name of foundDatasources) if (await pushDatasource(name, { silent: true })) console.log(pc.green(`✓ ${name} synced`));
+	else {
+		console.log(pc.red(`✗ Failed to sync "${name}"`));
+		failed = true;
 	}
-	const remoteNames = new Set(remoteDatasources.map((ds) => ds.name));
-	const missingRemote = foundDatasources.filter((name) => !remoteNames.has(name));
-	if (missingRemote.length > 0) {
-		console.log();
-		console.log(pc.yellow(`⚠ Missing remote datasource${missingRemote.length > 1 ? "s" : ""}: ${missingRemote.join(", ")}`));
+	if (failed) {
 		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 {
-				console.log(pc.red(`✗ Failed to push "${name}"`));
-				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(pc.red("Datasource sync failed. Check .bon/datasources.yaml and credentials."));
+		return true;
 	}
 	console.log();
 	console.log(pc.dim("Testing datasource connections..."));
@@ -3931,11 +3812,9 @@ 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 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("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("deploy").description("Deploy cubes and views to Bonnard. Requires login, validates, syncs datasources").option("--ci", "Non-interactive mode").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);
@@ -3949,4 +3828,4 @@ metabase.command("analyze").description("Analyze Metabase instance and generate
 program.parse();
 
 //#endregion
-export { getProjectPaths as t };
+export { getProjectPaths as i, resolveEnvVarsInCredentials as n, post as r, getLocalDatasource as t };

package/dist/bin/{cubes-Bf0IPYd7.mjs → cubes-9rklhdAJ.mjs}

@@ -1,4 +1,4 @@
-import { t as getProjectPaths } from "./bon.mjs";
+import { i as getProjectPaths } from "./bon.mjs";
 import fs from "node:fs";
 import path from "node:path";
 import YAML from "yaml";

package/dist/bin/push-Bv9AFGc2.mjs

@@ -0,0 +1,35 @@
+import { n as resolveEnvVarsInCredentials, r as post, t as getLocalDatasource } from "./bon.mjs";
+import pc from "picocolors";
+import "@inquirer/prompts";
+
+//#region src/commands/datasource/push.ts
+/**
+* Push a datasource programmatically (for use by deploy command)
+* Returns true on success, false on failure
+*/
+async function pushDatasource(name, options = {}) {
+	const datasource = getLocalDatasource(name);
+	if (!datasource) {
+		if (!options.silent) console.error(pc.red(`Datasource "${name}" not found locally`));
+		return false;
+	}
+	const { resolved, missing } = resolveEnvVarsInCredentials(datasource.credentials);
+	if (missing.length > 0) {
+		if (!options.silent) console.error(pc.red(`Missing env vars for "${name}": ${missing.join(", ")}`));
+		return false;
+	}
+	try {
+		await post("/api/datasources", {
+			name: datasource.name,
+			warehouse_type: datasource.type,
+			config: datasource.config,
+			credentials: resolved
+		});
+		return true;
+	} catch {
+		return false;
+	}
+}
+
+//#endregion
+export { pushDatasource };

package/dist/bin/{validate-DEh1XQnH.mjs → validate-BdqZBH2n.mjs}

@@ -1,4 +1,4 @@
-import { t as getProjectPaths } from "./bon.mjs";
+import { i as getProjectPaths } from "./bon.mjs";
 import fs from "node:fs";
 import path from "node:path";
 import YAML from "yaml";

package/dist/docs/topics/features.cli.md

@@ -35,10 +35,10 @@ bon docs cubes.measures           # Read modeling docs in terminal
 Deploy from GitHub Actions, GitLab CI, or any pipeline:
 
 ```bash
-bon deploy --ci --push-datasources -m "CI deploy"
+bon deploy --ci -m "CI deploy"
 ```
 
-Non-interactive mode with automatic datasource sync. Fails fast if anything is misconfigured.
+Non-interactive mode. Datasources are synced automatically. Fails fast if anything is misconfigured.
 
 ## Deployment versioning
 

package/dist/docs/topics/workflow.deploy.md

@@ -19,15 +19,16 @@ A `-m` message is **required** — it describes what changed in this deployment.
 | Flag | Description |
 |------|-------------|
 | `-m "message"` | **Required.** Deployment description |
-| `--ci` | Non-interactive mode (fails on missing datasources) |
-| `--push-datasources` | Auto-push missing datasources to Bonnard |
+| `--ci` | Non-interactive mode |
+
+Datasources are always synced automatically during deploy.
 
 ### CI/CD
 
-For automated pipelines, combine `--ci` with `--push-datasources`:
+For automated pipelines, use `--ci` for non-interactive mode:
 
 ```bash
-bon deploy --ci --push-datasources -m "CI deploy"
+bon deploy --ci -m "CI deploy"
 ```
 
 ## Prerequisites
@@ -148,7 +149,7 @@ Deploy aborted. Fix validation errors first.
 Deploy aborted. Fix connection issues:
   - Check credentials in .bon/datasources.yaml
   - Verify network access to database
-  - Run: bon datasource test analytics
+  - Run: bon datasource add (to reconfigure)
 ```
 
 ### Auth Errors

package/dist/docs/topics/workflow.md

@@ -146,7 +146,6 @@ bonnard/cubes/
 | `bon datasource add --demo` | Add demo dataset (no warehouse needed) |
 | `bon datasource add --from-dbt` | Import from dbt profiles |
 | `bon datasource list` | List configured sources |
-| `bon datasource test <name>` | Test connection (requires login) |
 | `bon validate` | Check cube and view syntax |
 | `bon deploy -m "message"` | Deploy to Bonnard (message required) |
 | `bon deploy --ci` | Non-interactive deploy |

package/dist/docs/topics/workflow.validate.md

@@ -116,7 +116,7 @@ measures:
 1. **Run before every deploy** — `bon validate && bon deploy`
 2. **Add to CI/CD** — validate on pull requests
 3. **Fix errors first** — don't deploy with validation errors
-4. **Test connections** — use `bon datasource test <name>` to check connectivity
+4. **Test connections** — connections are tested automatically during `bon deploy`
 
 ## See Also
 

package/dist/templates/claude/skills/bonnard-get-started/SKILL.md

@@ -15,29 +15,27 @@ confirming progress before moving on.
 Ask the user if they have a warehouse to connect, or want to try a demo dataset first:
 
 ```bash
-# Option A: Import from dbt (if they use it)
+# Option A: Use demo data (no warehouse needed)
+bon datasource add --demo
+
+# Option B: Import from dbt (if they use it)
 bon datasource add --from-dbt
 
-# Option B: Add manually (interactive)
-bon datasource add
+# Option C: Add manually, non-interactive (preferred for agents)
+bon datasource add --name my_warehouse --type postgres \
+  --host db.example.com --port 5432 --database mydb --schema public \
+  --user myuser --password mypassword
 
-# Option C: Use demo data (no warehouse needed)
-bon datasource add --demo
+# Option D: Add manually, interactive (in user's terminal)
+bon datasource add
 ```
 
+Supported types: `postgres`, `redshift`, `snowflake`, `bigquery`, `databricks`.
+
 The demo option adds a read-only Contoso retail dataset with tables like
 `fact_sales`, `dim_product`, `dim_store`, and `dim_customer`.
 
-Then verify the connection works:
-
-```bash
-bon datasource test <name>
-```
-
-If the test fails, common issues:
-- Wrong credentials — re-run `bon datasource add`
-- Network/firewall — check warehouse allows connections from this machine
-- SSL issues (Postgres) — may need `sslmode` in connection config
+The connection will be tested automatically during `bon deploy`.
 
 ## Phase 2: Explore the Data
 

package/dist/templates/claude/skills/bonnard-metabase-migrate/SKILL.md

@@ -48,24 +48,26 @@ drives every decision in the remaining phases.
 
 ## Phase 3: Connect the Data Warehouse
 
-Add a datasource pointing to the same database that Metabase queries:
+Add a datasource pointing to the same database that Metabase queries.
+The database connection details can often be found in Metabase under
+Admin > Databases, or in the analysis report header.
 
 ```bash
-# Interactive setup
-bon datasource add
+# Non-interactive (preferred for agents)
+bon datasource add --name my_warehouse --type postgres \
+  --host db.example.com --port 5432 --database mydb --schema public \
+  --user myuser --password mypassword
 
-# Or import from dbt if available
+# Import from dbt if available
 bon datasource add --from-dbt
-```
-
-Then verify the connection:
 
-```bash
-bon datasource test <name>
+# Interactive setup (in user's terminal)
+bon datasource add
 ```
 
-The database connection details can often be found in Metabase under
-Admin > Databases, or in the analysis report header.
+Supported types: `postgres`, `redshift`, `snowflake`, `bigquery`, `databricks`.
+
+The connection will be tested automatically during `bon deploy`.
 
 ## Phase 4: Explore Key Tables
 

package/dist/templates/cursor/rules/bonnard-get-started.mdc

@@ -14,29 +14,27 @@ confirming progress before moving on.
 Ask the user if they have a warehouse to connect, or want to try a demo dataset first:
 
 ```bash
-# Option A: Import from dbt (if they use it)
+# Option A: Use demo data (no warehouse needed)
+bon datasource add --demo
+
+# Option B: Import from dbt (if they use it)
 bon datasource add --from-dbt
 
-# Option B: Add manually (interactive)
-bon datasource add
+# Option C: Add manually, non-interactive (preferred for agents)
+bon datasource add --name my_warehouse --type postgres \
+  --host db.example.com --port 5432 --database mydb --schema public \
+  --user myuser --password mypassword
 
-# Option C: Use demo data (no warehouse needed)
-bon datasource add --demo
+# Option D: Add manually, interactive (in user's terminal)
+bon datasource add
 ```
 
+Supported types: `postgres`, `redshift`, `snowflake`, `bigquery`, `databricks`.
+
 The demo option adds a read-only Contoso retail dataset with tables like
 `fact_sales`, `dim_product`, `dim_store`, and `dim_customer`.
 
-Then verify the connection works:
-
-```bash
-bon datasource test <name>
-```
-
-If the test fails, common issues:
-- Wrong credentials — re-run `bon datasource add`
-- Network/firewall — check warehouse allows connections from this machine
-- SSL issues (Postgres) — may need `sslmode` in connection config
+The connection will be tested automatically during `bon deploy`.
 
 ## Phase 2: Explore the Data
 

package/dist/templates/cursor/rules/bonnard-metabase-migrate.mdc

@@ -47,24 +47,26 @@ drives every decision in the remaining phases.
 
 ## Phase 3: Connect the Data Warehouse
 
-Add a datasource pointing to the same database that Metabase queries:
+Add a datasource pointing to the same database that Metabase queries.
+The database connection details can often be found in Metabase under
+Admin > Databases, or in the analysis report header.
 
 ```bash
-# Interactive setup
-bon datasource add
+# Non-interactive (preferred for agents)
+bon datasource add --name my_warehouse --type postgres \
+  --host db.example.com --port 5432 --database mydb --schema public \
+  --user myuser --password mypassword
 
-# Or import from dbt if available
+# Import from dbt if available
 bon datasource add --from-dbt
-```
-
-Then verify the connection:
 
-```bash
-bon datasource test <name>
+# Interactive setup (in user's terminal)
+bon datasource add
 ```
 
-The database connection details can often be found in Metabase under
-Admin > Databases, or in the analysis report header.
+Supported types: `postgres`, `redshift`, `snowflake`, `bigquery`, `databricks`.
+
+The connection will be tested automatically during `bon deploy`.
 
 ## Phase 4: Explore Key Tables
 

package/dist/templates/shared/bonnard.md

@@ -63,10 +63,9 @@ All tables are in the `contoso` schema. The datasource is named `contoso_demo`.
 | `bon datasource add` | Add warehouse connection |
 | `bon datasource add --demo` | Add demo dataset (no warehouse needed) |
 | `bon datasource add --from-dbt` | Import from dbt profiles |
-| `bon datasource test <name>` | Test connection (requires login) |
 | `bon validate` | Validate YAML syntax, warn on missing descriptions and `data_source` |
 | `bon deploy -m "message"` | Deploy to Bonnard (requires login, message required) |
-| `bon deploy --ci` | Non-interactive deploy (fails on missing datasources) |
+| `bon deploy --ci` | Non-interactive deploy |
 | `bon deployments` | List recent deployments (add `--all` for full history) |
 | `bon diff <deployment-id>` | Show changes in a deployment (`--breaking` for breaking only) |
 | `bon annotate <deployment-id>` | Add reasoning/context to deployment changes |
@@ -118,7 +117,7 @@ Every deploy creates a versioned deployment with change detection:
 - **Diff**: `bon diff <id>` shows all changes; `bon diff <id> --breaking` filters to breaking only
 - **Annotate**: `bon annotate <id> --data '{"object": "note"}'` adds context to changes
 
-For CI/CD pipelines, use `bon deploy --ci -m "message"` (non-interactive, fails on issues) or `bon deploy --push-datasources -m "message"` to auto-push missing datasources.
+For CI/CD pipelines, use `bon deploy --ci -m "message"` (non-interactive, fails on issues). Datasources are always synced automatically during deploy.
 
 ## Best Practices
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bonnard/cli",
-  "version": "0.2.2",
+  "version": "0.2.4",
   "type": "module",
   "bin": {
     "bon": "./dist/bin/bon.mjs"