@bonnard/cli 0.2.3 → 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",
@@ -1810,7 +1841,7 @@ async function deployCommand(options = {}) {
 async function testAndSyncDatasources(cwd, options = {}) {
 	const { extractDatasourcesFromCubes } = await import("./cubes-9rklhdAJ.mjs");
 	const { loadLocalDatasources } = await Promise.resolve().then(() => local_exports);
-	const { pushDatasource } = await import("./push-mZujN1Ik.mjs");
+	const { pushDatasource } = await import("./push-Bv9AFGc2.mjs");
 	const references = extractDatasourcesFromCubes(cwd);
 	if (references.length === 0) return false;
 	console.log();
@@ -1833,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(`Use --push-datasources to auto-push missing datasources`));
-			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..."));
@@ -3818,7 +3814,7 @@ datasource.command("add").description("Add a data source to .bon/datasources.yam
 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("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);
 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);

package/dist/bin/{push-mZujN1Ik.mjs → push-Bv9AFGc2.mjs}

@@ -1,6 +1,6 @@
 import { n as resolveEnvVarsInCredentials, r as post, t as getLocalDatasource } from "./bon.mjs";
 import pc from "picocolors";
-import { confirm } from "@inquirer/prompts";
+import "@inquirer/prompts";
 
 //#region src/commands/datasource/push.ts
 /**

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

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

@@ -30,7 +30,7 @@ bon datasource add --name my_warehouse --type postgres \
 bon datasource add
 ```
 
-Supported types: `postgres` (also works for Redshift), `snowflake`, `bigquery`, `databricks`.
+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`.

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

@@ -65,7 +65,7 @@ bon datasource add --from-dbt
 bon datasource add
 ```
 
-Supported types: `postgres` (also works for Redshift), `snowflake`, `bigquery`, `databricks`.
+Supported types: `postgres`, `redshift`, `snowflake`, `bigquery`, `databricks`.
 
 The connection will be tested automatically during `bon deploy`.
 

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

@@ -29,7 +29,7 @@ bon datasource add --name my_warehouse --type postgres \
 bon datasource add
 ```
 
-Supported types: `postgres` (also works for Redshift), `snowflake`, `bigquery`, `databricks`.
+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`.

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

@@ -64,7 +64,7 @@ bon datasource add --from-dbt
 bon datasource add
 ```
 
-Supported types: `postgres` (also works for Redshift), `snowflake`, `bigquery`, `databricks`.
+Supported types: `postgres`, `redshift`, `snowflake`, `bigquery`, `databricks`.
 
 The connection will be tested automatically during `bon deploy`.
 

package/dist/templates/shared/bonnard.md

@@ -65,7 +65,7 @@ All tables are in the `contoso` schema. The datasource is named `contoso_demo`.
 | `bon datasource add --from-dbt` | Import from dbt profiles |
 | `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 |
@@ -117,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.3",
+  "version": "0.2.4",
   "type": "module",
   "bin": {
     "bon": "./dist/bin/bon.mjs"