sqlbuild-cli 0.182.0__tar.gz → 0.184.0__tar.gz

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/Cargo.lock

@@ -1175,6 +1175,26 @@ version = "2.0.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "f8fadd59c855ef2080decdef8ff161eb6661b86933c9d82e5ba29dc602a55aba"
 
+[[package]]
+name = "signal-hook"
+version = "0.3.18"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "d881a16cf4426aa584979d30bd82cb33429027e42122b169753d6ef1085ed6e2"
+dependencies = [
+ "libc",
+ "signal-hook-registry",
+]
+
+[[package]]
+name = "signal-hook-registry"
+version = "1.4.8"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "c4db69cba1110affc0e9f7bcd48bbf87b3f4fc7c61fc9155afd4c469eb3d6c1b"
+dependencies = [
+ "errno",
+ "libc",
+]
+
 [[package]]
 name = "siphasher"
 version = "1.0.4"
@@ -1189,7 +1209,7 @@ checksum = "5b3dc8af474f516a851ff4bd12db780f948b9250ad37211e4eec0bccea54e01b"
 
 [[package]]
 name = "sqlbuild-analysis"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "memchr",
  "polyglot-sql",
@@ -1212,7 +1232,7 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-attachments"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "sqlbuild-core",
  "sqlbuild-sqltext",
@@ -1220,22 +1240,24 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-build-features"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "arrayvec",
  "bstr",
  "either",
+ "errno",
  "memchr",
  "once_cell",
  "regex-automata",
  "syn 2.0.119",
  "syn 3.0.3",
  "thiserror 2.0.21",
+ "windows-sys",
 ]
 
 [[package]]
 name = "sqlbuild-cache"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "blake3",
  "rayon",
@@ -1248,7 +1270,7 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-cli"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "blake3",
  "ctrlc",
@@ -1266,7 +1288,7 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-config"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "saphyr-parser",
  "sqlbuild-core",
@@ -1277,7 +1299,7 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-core"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "regex",
  "sha2",
@@ -1285,14 +1307,14 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-cursor"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "sqlbuild-core",
 ]
 
 [[package]]
 name = "sqlbuild-discovery"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "junction",
  "rayon",
@@ -1306,14 +1328,14 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-host"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "sqlbuild-core",
 ]
 
 [[package]]
 name = "sqlbuild-model-config"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "regex",
  "sqlbuild-core",
@@ -1321,7 +1343,7 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-project"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "sqlbuild-analysis",
  "sqlbuild-core",
@@ -1330,7 +1352,7 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-pysource"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "ruff_python_ast",
  "ruff_python_parser",
@@ -1341,7 +1363,7 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-python"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "fensu-policy",
  "mimalloc",
@@ -1350,6 +1372,7 @@ dependencies = [
  "rustc-hash",
  "serde",
  "serde_json",
+ "signal-hook",
  "sqlbuild-analysis",
  "sqlbuild-attachments",
  "sqlbuild-cache",
@@ -1374,7 +1397,7 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-recall"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "blake3",
  "rusqlite",
@@ -1388,7 +1411,7 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-refactor"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "serde",
  "serde_json",
@@ -1403,7 +1426,7 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-render"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "serde_json",
  "sqlbuild-cache",
@@ -1413,7 +1436,7 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-rules"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "blake3",
  "fensu-policy",
@@ -1438,7 +1461,7 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-scopes"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "serde_json",
  "sha2",
@@ -1451,7 +1474,7 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-sqltext"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "rayon",
  "serde",
@@ -1460,7 +1483,7 @@ dependencies = [
 
 [[package]]
 name = "sqlbuild-state"
-version = "0.182.0"
+version = "0.184.0"
 dependencies = [
  "serde_json",
  "sha2",

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/Cargo.toml

@@ -33,7 +33,7 @@ python-boundary = "sqlbuild-python"
 polyglot-floor = "sqlbuild-analysis"
 
 [workspace.package]
-version = "0.182.0"
+version = "0.184.0"
 edition = "2024"
 license = "Apache-2.0"
 publish = false
@@ -83,8 +83,13 @@ ruff_text_size = "=0.0.4"
 # Unix only: `sqb` shares the compute log prune lock with Python's `filelock` through `flock`.
 # Fast deterministic hasher for hot in-memory maps whose iteration order never reaches output.
 rustc-hash = "2"
+# The native SIGTERM handler: sets a flag the CLI watches, without unsafe code in the workspace.
+signal-hook = { version = "0.3", default-features = false }
+# Features signal-hook's registry enables, matched in the sqb build through sqlbuild-build-features.
+errno = "0.3"
+windows-sys = { version = "0.61", default-features = false, features = ["Win32_System_Diagnostics_Debug"] }
 # Recall's local store: SQLite compiled in, so every build has the same engine version.
-rusqlite = { version = "0.37", features = ["bundled"] }
+rusqlite = { version = "0.37", features = ["bundled", "column_decltype"] }
 rustix = { version = "1.1.4", default-features = false, features = ["fs", "std", "termios"] }
 saphyr-parser = "0.1"
 regex = "1"

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sqlbuild-cli
-Version: 0.182.0
+Version: 0.184.0
 License-File: LICENSE
 Summary: The native sqb and sqlbuild executables of SQLBuild
 License-Expression: Apache-2.0

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-build-features/Cargo.toml

@@ -22,6 +22,13 @@ once_cell.workspace = true
 regex-automata.workspace = true
 thiserror.workspace = true
 
+# signal-hook's registry, in the extension only, enables these on the platforms that use them.
+[target.'cfg(unix)'.dependencies]
+errno.workspace = true
+
+[target.'cfg(windows)'.dependencies]
+windows-sys.workspace = true
+
 # Proc-macro dependencies resolve with build dependencies, so the syn features PyO3's macros use
 # are enabled here. The empty build script makes Cargo resolve them.
 [build-dependencies]

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-cli/src/argv/_helpers/table.rs

@@ -131,6 +131,15 @@ impl TableReader {
                     help: help(choice.get("help").unwrap_or(&Json::Null)),
                 })
                 .collect(),
+            removed_choices: items(json, "removed_choices")
+                .iter()
+                .map(|removed| {
+                    (
+                        text(removed, "value").unwrap_or_default(),
+                        text(removed, "instead").unwrap_or_default(),
+                    )
+                })
+                .collect(),
         }
     }
 }

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-cli/src/argv/_helpers/values.rs

@@ -114,6 +114,16 @@ impl Run<'_> {
         if choices.iter().any(|choice| choice == text) {
             return Ok(());
         }
+        if let Some((_, instead)) = action
+            .removed_choices
+            .iter()
+            .find(|(removed, _)| removed == text)
+        {
+            return Err(argument_error(
+                action,
+                &format!("{} was removed; {instead}", self.host.repr(text)),
+            ));
+        }
         let listed: Vec<String> = choices
             .iter()
             .map(|choice| python_str_repr(choice))

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-cli/src/argv/command_table.json

@@ -4397,13 +4397,21 @@
          "--format"
         ],
         "dest": "query_format",
-        "default": "long",
         "choices": [
          "long",
-         "table",
-         "json",
-         "csv"
+         "json"
+        ],
+        "removed_choices": [
+         {
+          "value": "table",
+          "instead": "the default output prints a table when the result fits; use --format long for one record per row"
+         },
+         {
+          "value": "csv",
+          "instead": "use --json for programs, or the warehouse's own export for bulk data"
+         }
         ],
+        "help": "by default a table when the result fits, else one record per row; long always prints records; json for programs",
         "mutex_group": 0,
         "group": 1
        },
@@ -7183,14 +7191,21 @@
               "--format"
              ],
              "dest": "recall_format",
-             "default": "long",
              "choices": [
               "long",
-              "table",
-              "json",
-              "csv"
-             ],
-             "help": "long (default) for reading, json for programs, csv for export; table is deprecated",
+              "json"
+             ],
+             "removed_choices": [
+              {
+               "value": "table",
+               "instead": "the default output prints a table when the result fits; use --format long for one record per row"
+              },
+              {
+               "value": "csv",
+               "instead": "use --json for programs"
+              }
+             ],
+             "help": "by default a table when the result fits, else one record per row; long always prints records; json for programs",
              "mutex_group": 0,
              "group": 1
             },
@@ -7374,14 +7389,21 @@
               "--format"
              ],
              "dest": "recall_format",
-             "default": "long",
              "choices": [
               "long",
-              "table",
-              "json",
-              "csv"
-             ],
-             "help": "long (default) for reading, json for programs, csv for export; table is deprecated",
+              "json"
+             ],
+             "removed_choices": [
+              {
+               "value": "table",
+               "instead": "use the default long output"
+              },
+              {
+               "value": "csv",
+               "instead": "use --json for programs"
+              }
+             ],
+             "help": "long (default) for reading, json for programs",
              "mutex_group": 0,
              "group": 1
             },

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-cli/src/argv/constants.rs

@@ -159,7 +159,7 @@ pub(crate) const CLI_NAMESPACE_DEFAULTS: [(&str, NamespaceDefault); 184] = [
     ("max_columns", NamespaceDefault::None),
     ("query_sql", NamespaceDefault::None),
     ("query_file", NamespaceDefault::None),
-    ("query_format", NamespaceDefault::Text("long")),
+    ("query_format", NamespaceDefault::None),
     ("query_limit", NamespaceDefault::Int(20)),
     ("query_no_limit", NamespaceDefault::Bool(false)),
     ("cost_selector", NamespaceDefault::Text("latest")),
@@ -228,7 +228,7 @@ pub(crate) const CLI_NAMESPACE_DEFAULTS: [(&str, NamespaceDefault); 184] = [
     ("recall_command", NamespaceDefault::None),
     ("recall_sql", NamespaceDefault::None),
     ("recall_backend", NamespaceDefault::Text("logs")),
-    ("recall_format", NamespaceDefault::Text("long")),
+    ("recall_format", NamespaceDefault::None),
     ("recall_limit", NamespaceDefault::Int(100)),
     ("recall_no_limit", NamespaceDefault::Bool(false)),
     ("recall_max_cell_bytes", NamespaceDefault::None),

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-cli/src/argv/models.rs

@@ -98,6 +98,8 @@ pub struct ActionSpec {
     /// Sub-command names and their parsers, in order.
     pub parsers: Vec<(String, ParserSpec)>,
     pub choices_help: Vec<ChoiceHelp>,
+    /// Values no longer accepted, each with what to use instead; never listed in help.
+    pub removed_choices: Vec<(String, String)>,
 }
 
 /// One argument group's title and description.

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-cli/src/dispatch/constants.rs

@@ -12,6 +12,8 @@ pub(crate) const DBT_INIT_SUBCOMMAND: &str = "init";
 pub(crate) const DIFF_COMMAND_NAME: &str = "diff";
 /// `sqb recall` reads history and is never recorded itself.
 pub(crate) const RECALL_COMMAND_NAME: &str = "recall";
+/// A `--select-file` line starting with this is a comment.
+pub(crate) const SELECTOR_COMMENT_PREFIX: &str = "#";
 
 const PROJECT_DIR: Field = field("project_dir", Source::ProjectDir);
 const SELECT: Field = field("select", Source::Select);

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-cli/src/dispatch/main/mod.rs

@@ -3,4 +3,5 @@ pub mod command_dispatch;
 pub mod event_export_warning;
 pub mod failure_report;
 pub mod invocation_policy;
+pub mod selector_file_selectors;
 pub mod unavailable_handler;

sqlbuild_cli-0.184.0/crates/sqlbuild-cli/src/dispatch/main/selector_file_selectors.rs

@@ -0,0 +1,17 @@
+//! The selectors a `--select-file` lists.
+
+use sqlbuild_core::text::main::python_splitlines::python_splitlines;
+use sqlbuild_core::text::main::python_strip::python_strip;
+
+use crate::dispatch::constants::SELECTOR_COMMENT_PREFIX;
+
+/// One selector per stripped line of `text`, without blank lines and `#` comments.
+#[must_use]
+pub fn selector_file_selectors(text: &str) -> Vec<String> {
+    python_splitlines(text)
+        .into_iter()
+        .map(python_strip)
+        .filter(|line| !line.is_empty() && !line.starts_with(SELECTOR_COMMENT_PREFIX))
+        .map(str::to_owned)
+        .collect()
+}

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-config/src/recall/constants.rs

@@ -2,7 +2,8 @@
 
 pub const RECALL_SECTION: &str = "recall";
 /// Keys `[recall]` and `targets.<name>.recall` accept, sorted.
-pub const RECALL_KEYS: [&str; 7] = [
+pub const RECALL_KEYS: [&str; 8] = [
+    "audit_evidence",
     "diagnostics_min_severity",
     "enabled",
     "max_output_bytes",
@@ -15,6 +16,8 @@ pub const RECALL_KEYS: [&str; 7] = [
 pub const REMOVED_CONNECTION_KEY: &str = "connection";
 /// The diagnostic severities a Recall store can keep from, least severe first.
 pub const RECALL_SEVERITIES: [&str; 6] = ["trace", "debug", "info", "warning", "error", "critical"];
+/// Which audit outcomes keep their evidence rows in Recall, the default first.
+pub const AUDIT_EVIDENCE_MODES: [&str; 3] = ["warn-and-error", "error", "off"];
 /// The longest shutdown flush budget, in milliseconds.
 pub const MAX_SHUTDOWN_TIMEOUT_MS: u64 = 600_000;
 /// The longest retention, in days.

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-config/src/recall/main/recall_section.rs

@@ -3,7 +3,8 @@
 use crate::models::ConfigValue;
 use crate::recall::_helpers::values::{non_empty, output_bytes, retention_days, timeout_ms};
 use crate::recall::constants::{
-    MAX_RETENTION_DAYS, RECALL_KEYS, RECALL_SEVERITIES, REMOVED_CONNECTION_KEY,
+    AUDIT_EVIDENCE_MODES, MAX_RETENTION_DAYS, RECALL_KEYS, RECALL_SEVERITIES,
+    REMOVED_CONNECTION_KEY,
 };
 use crate::recall::models::RecallSection;
 
@@ -65,6 +66,16 @@ pub fn recall_section(value: &ConfigValue, label: &str) -> Result<RecallSection,
                 }
                 section.diagnostics_min_severity = Some(text.to_owned());
             }
+            "audit_evidence" => {
+                let text = non_empty(value, label, key)?;
+                if !AUDIT_EVIDENCE_MODES.contains(&text) {
+                    return Err(format!(
+                        "{label}.audit_evidence must be one of: {}",
+                        AUDIT_EVIDENCE_MODES.join(", ")
+                    ));
+                }
+                section.audit_evidence = Some(text.to_owned());
+            }
             other => {
                 return Err(format!(
                     "{label} contains unknown key(s): {other}. Allowed keys: {}",

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-config/src/recall/models.rs

@@ -13,4 +13,6 @@ pub struct RecallSection {
     pub diagnostics_min_severity: Option<String>,
     /// Bytes of each output stream to record; `Some(0)` records everything.
     pub max_output_bytes: Option<u64>,
+    /// Which audit outcomes keep their evidence rows: one of [`AUDIT_EVIDENCE_MODES`](crate::recall::constants::AUDIT_EVIDENCE_MODES).
+    pub audit_evidence: Option<String>,
 }

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-config/src/recall/tests/test_recall_section.rs

@@ -17,6 +17,7 @@ fn given_valid_sections_when_validated_then_settings_are_typed() {
                 ("shutdown_timeout", text("1500ms")),
                 ("diagnostics_min_severity", text("debug")),
                 ("max_output_bytes", ConfigValue::Integer(4_194_304)),
+                ("audit_evidence", text("error")),
             ]),
             expected_section: RecallSection {
                 enabled: Some(false),
@@ -26,6 +27,7 @@ fn given_valid_sections_when_validated_then_settings_are_typed() {
                 shutdown_timeout_ms: Some(1500),
                 diagnostics_min_severity: Some("debug".to_owned()),
                 max_output_bytes: Some(4_194_304),
+                audit_evidence: Some("error".to_owned()),
             },
         },
         ValidSectionTestCase {
@@ -64,6 +66,13 @@ fn given_valid_sections_when_validated_then_settings_are_typed() {
 #[test]
 fn given_invalid_sections_when_validated_then_the_message_names_the_key() {
     let test_cases = [
+        InvalidSectionTestCase {
+            description: "audit evidence names one of its modes",
+            value: table(&[("audit_evidence", text("always"))]),
+            label: "targets.prod.recall",
+            expected_message_prefix: "targets.prod.recall.audit_evidence must be one of: \
+                                      warn-and-error, error, off",
+        },
         InvalidSectionTestCase {
             description: "output bytes must not be negative",
             value: table(&[("max_output_bytes", ConfigValue::Integer(-1))]),

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-core/src/lifecycle_events/constants.rs

@@ -79,7 +79,7 @@ pub(crate) const FORBIDDEN_STATEMENT_PAYLOAD_FIELDS: [&str; 8] = [
     "sql",
     "statement_sql",
 ];
-pub(crate) const STRING_PAYLOAD_FIELDS: [&str; 55] = [
+pub(crate) const STRING_PAYLOAD_FIELDS: [&str; 56] = [
     "adapter",
     "attached_column_name",
     "attached_target_kind",
@@ -92,6 +92,7 @@ pub(crate) const STRING_PAYLOAD_FIELDS: [&str; 55] = [
     "binding_key",
     "command",
     "configured_batch_size",
+    "cost_status",
     "cursor_end_exclusive",
     "cursor_start",
     "definition_fingerprint",
@@ -653,7 +654,7 @@ pub(crate) const LIFECYCLE_EVENT_CATALOG_V2: [EventDefinition; 23] = [
         event_type: "invocation_completed",
         required_correlations: &[],
         required_payload: &[],
-        allowed_payload: &["command", "duration_ms", "exit_code"],
+        allowed_payload: &["command", "cost_status", "duration_ms", "exit_code"],
         terminal: true,
     },
     EventDefinition {
@@ -662,11 +663,13 @@ pub(crate) const LIFECYCLE_EVENT_CATALOG_V2: [EventDefinition; 23] = [
         required_payload: &[],
         allowed_payload: &[
             "command",
+            "cost_status",
             "duration_ms",
             "error_code",
             "error_message",
             "error_type",
             "exit_code",
+            "signal_number",
         ],
         terminal: true,
     },
@@ -1010,6 +1013,7 @@ pub(crate) const LIFECYCLE_EVENT_CATALOG_V2: [EventDefinition; 23] = [
             "intent",
             "job_id",
             "metadata",
+            "outcome",
             "purpose",
             "query_id",
             "row_count",

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-core/src/progress/constants.rs

@@ -113,7 +113,6 @@ pub(crate) const QUERY_COMMAND: &str = "query";
 pub(crate) const DBT_COMMAND: &str = "dbt";
 pub(crate) const DBT_JSON_OPTION: &str = "--json";
 pub(crate) const JSON_FORMAT: &str = "json";
-pub(crate) const CSV_FORMAT: &str = "csv";
 /// What replaces the cut tail of a transient status message.
 pub(crate) const STATUS_ELLIPSIS: &str = "...";
 /// The resource kind of audits and the prefix of their resource identifiers.

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-core/src/progress/main/machine_output_mode.rs

@@ -1,11 +1,11 @@
 //! Whether a command's stdout carries machine output, so human progress must use stderr.
 
 use crate::progress::constants::{
-    CSV_FORMAT, DBT_COMMAND, DBT_JSON_OPTION, JSON_FORMAT, LINEAGE_COMMAND, QUERY_COMMAND,
+    DBT_COMMAND, DBT_JSON_OPTION, JSON_FORMAT, LINEAGE_COMMAND, QUERY_COMMAND,
 };
 use crate::progress::models::OutputModeFacts;
 
-/// `--json`, `sqb dbt ... --json`, `lineage --format json` or `query --format csv|json`.
+/// `--json`, `sqb dbt ... --json`, `lineage --format json` or `query --format json`.
 #[must_use]
 pub fn machine_output_mode(facts: &OutputModeFacts) -> bool {
     let format = |value: &Option<String>| value.as_deref().unwrap_or_default().to_owned();
@@ -13,9 +13,7 @@ pub fn machine_output_mode(facts: &OutputModeFacts) -> bool {
         _ if facts.json => true,
         Some(DBT_COMMAND) => facts.dbt_args.iter().any(|arg| arg == DBT_JSON_OPTION),
         Some(LINEAGE_COMMAND) => format(&facts.lineage_format) == JSON_FORMAT,
-        Some(QUERY_COMMAND) => {
-            [CSV_FORMAT, JSON_FORMAT].contains(&format(&facts.query_format).as_str())
-        }
+        Some(QUERY_COMMAND) => format(&facts.query_format) == JSON_FORMAT,
         _ => false,
     }
 }

{sqlbuild_cli-0.182.0 → sqlbuild_cli-0.184.0}/crates/sqlbuild-core/src/progress/tests/test_progress_text.rs

@@ -269,9 +269,9 @@ fn given_parsed_options_when_choosing_output_mode_then_machine_output_is_detecte
             expected_machine_output: true,
         },
         OutputModeTestCase {
-            description: "query csv",
+            description: "query json",
             facts: OutputModeFacts {
-                query_format: Some("csv".to_owned()),
+                query_format: Some("json".to_owned()),
                 ..facts("query", &[])
             },
             expected_machine_output: true,