@bonnard/cli 0.2.2 → 0.2.3

package/dist/bin/bon.mjs

@@ -1490,7 +1490,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 +1523,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 +1630,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 +1664,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 +1673,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 +1739,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 +1808,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-mZujN1Ik.mjs");
 	const references = extractDatasourcesFromCubes(cwd);
 	if (references.length === 0) return false;
 	console.log();
@@ -1964,7 +1849,7 @@ async function testAndSyncDatasources(cwd, options = {}) {
 		console.log();
 		if (options.ci) {
 			console.log(pc.red("Deploy aborted (--ci mode)."));
-			console.log(pc.dim(`Run: bon datasource push <name>`));
+			console.log(pc.dim(`Use --push-datasources to auto-push missing datasources`));
 			return true;
 		}
 		if (options.pushDatasources) for (const name of missingRemote) {
@@ -3931,9 +3816,7 @@ 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("deployments").description("List deployment history").option("--all", "Show all deployments (default: last 10)").option("--format <format>", "Output format: table or json", "table").action(deploymentsCommand);
@@ -3949,4 +3832,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-mZujN1Ik.mjs

@@ -0,0 +1,35 @@
+import { n as resolveEnvVarsInCredentials, r as post, t as getLocalDatasource } from "./bon.mjs";
+import pc from "picocolors";
+import { confirm } from "@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/workflow.deploy.md

@@ -148,7 +148,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` (also works for 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` (also works for 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` (also works for 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` (also works for 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,7 +63,6 @@ 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) |

package/package.json

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