dlin-cli 0.2.4b1__tar.gz → 0.2.4rc2__tar.gz

{dlin_cli-0.2.4b1 → dlin_cli-0.2.4rc2}/Cargo.lock

@@ -87,9 +87,9 @@ dependencies = [
 
 [[package]]
 name = "anyhow"
-version = "1.0.103"
+version = "1.0.104"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "2a4385e2e34eb35d6b3efe798b9eb88096925d87726c0798709bf56d9ed84af3"
+checksum = "330a5ed07fa54e4702c9d6c4174f74427fc0ef6e214bbd677ae50a5099946470"
 
 [[package]]
 name = "arraydeque"
@@ -139,9 +139,9 @@ checksum = "9330f8b2ff13f34540b44e946ef35111825727b38d33286ef986142615121801"
 
 [[package]]
 name = "clap"
-version = "4.6.1"
+version = "4.6.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "1ddb117e43bbf7dacf0a4190fef4d345b9bad68dfc649cb349e7d17d28428e51"
+checksum = "d91e0c145792ef73a6ad36d27c75ac09f1832222a3c209689d90f534685ee5b7"
 dependencies = [
  "clap_builder",
  "clap_derive",
@@ -149,9 +149,9 @@ dependencies = [
 
 [[package]]
 name = "clap_builder"
-version = "4.6.0"
+version = "4.6.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "714a53001bf66416adb0e2ef5ac857140e7dc3a0c48fb28b2f10762fc4b5069f"
+checksum = "f09628afdcc538b57f3c6341e9c8e9970f18e4a481690a64974d7023bd33548b"
 dependencies = [
  "anstream",
  "anstyle",
@@ -161,14 +161,14 @@ dependencies = [
 
 [[package]]
 name = "clap_derive"
-version = "4.6.1"
+version = "4.6.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "f2ce8604710f6733aa641a2b3731eaa1e8b3d9973d5e3565da11800813f997a9"
+checksum = "d012d2b9d65aca7f18f4d9878a045bc17899bba951561ba5ec3c2ba1eed9a061"
 dependencies = [
  "heck",
  "proc-macro2",
  "quote",
- "syn",
+ "syn 3.0.3",
 ]
 
 [[package]]
@@ -230,7 +230,7 @@ checksum = "d0a5c400df2834b80a4c3327b3aad3a4c4cd4de0629063962b03235697506a28"
 
 [[package]]
 name = "dlin"
-version = "0.2.4-beta.1"
+version = "0.2.4-rc.2"
 dependencies = [
  "anyhow",
  "clap",
@@ -247,7 +247,7 @@ dependencies = [
 
 [[package]]
 name = "dlin-core"
-version = "0.2.4-beta.1"
+version = "0.2.4-rc.2"
 dependencies = [
  "anyhow",
  "clap",
@@ -266,7 +266,7 @@ dependencies = [
  "serde_json",
  "serial_test",
  "tempfile",
- "thiserror 2.0.18",
+ "thiserror 2.0.19",
  "walkdir",
 ]
 
@@ -385,9 +385,9 @@ dependencies = [
 
 [[package]]
 name = "globset"
-version = "0.4.18"
+version = "0.4.19"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "52dfc19153a48bde0cbd630453615c8151bce3a5adfac7a0aebfbf0a1e1f57e3"
+checksum = "e47d37d2ae4464254884b60ab7071be2b876a9c35b696bd018ddcc76847309cd"
 dependencies = [
  "aho-corasick",
  "bstr",
@@ -473,9 +473,9 @@ dependencies = [
 
 [[package]]
 name = "libc"
-version = "0.2.186"
+version = "0.2.189"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "68ab91017fe16c622486840e4c83c9a37afeff978bd239b5293d61ece587de66"
+checksum = "3eaf3ede3fee6db1a4c2ee091bf8a8b4dccdc6d17f656fb07896ee72867612f2"
 
 [[package]]
 name = "linux-raw-sys"
@@ -597,9 +597,9 @@ checksum = "a89322df9ebe1c1578d689c92318e070967d1042b512afbe49518723f4e6d5cd"
 
 [[package]]
 name = "polyglot-sql"
-version = "0.6.2"
+version = "0.8.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "009cbd62f8c82de46697221ee66b0cf3a6dacbda16f8c5a2ce25853940073bd5"
+checksum = "6b7dd6da530087bffbf2b8b791914093b669f70eac07dbc66cf08f6954a1902e"
 dependencies = [
  "polyglot-sql-ast-derive",
  "serde",
@@ -610,13 +610,13 @@ dependencies = [
 
 [[package]]
 name = "polyglot-sql-ast-derive"
-version = "0.6.2"
+version = "0.8.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "e3250a6ffcd1d39670eb532e2b1877ad430799c49cb91b243d63c9425b868161"
+checksum = "c72cee7fdad5082a0cc73d7a9064914311fcd095b63bf5d85bd4c5b8dd89249e"
 dependencies = [
  "proc-macro2",
  "quote",
- "syn",
+ "syn 2.0.117",
 ]
 
 [[package]]
@@ -737,9 +737,9 @@ checksum = "94143f37725109f92c262ed2cf5e59bce7498c01bcc1502d7b9afe439a4e9f49"
 
 [[package]]
 name = "serde"
-version = "1.0.228"
+version = "1.0.229"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9a8e94ea7f378bd32cbbd37198a4a91436180c5bb472411e48b5ec2e2124ae9e"
+checksum = "4148590afebada386688f18773da617792bf2ef03ffc1e4cbd2b1d45b023e0ba"
 dependencies = [
  "serde_core",
  "serde_derive",
@@ -766,29 +766,29 @@ dependencies = [
 
 [[package]]
 name = "serde_core"
-version = "1.0.228"
+version = "1.0.229"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "41d385c7d4ca58e59fc732af25c3983b67ac852c1a25000afe1175de458b67ad"
+checksum = "67dca2c9c51e58a4791a4b1ed58308b39c64224d349a935ab5039aa360942a48"
 dependencies = [
  "serde_derive",
 ]
 
 [[package]]
 name = "serde_derive"
-version = "1.0.228"
+version = "1.0.229"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "d540f220d3187173da220f885ab66608367b6574e925011a9353e4badda91d79"
+checksum = "e7a5d71263a5a7d47b41f6b3f06ba276f10cc18b0931f1799f710578e2309348"
 dependencies = [
  "proc-macro2",
  "quote",
- "syn",
+ "syn 3.0.3",
 ]
 
 [[package]]
 name = "serde_json"
-version = "1.0.150"
+version = "1.0.151"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "e8014e44b4736ed0538adeecded0fce2a272f22dc9578a7eb6b2d9993c74cfb9"
+checksum = "c841b55ecdae098c80dcae9cf767f6f8a0c2cdb3416bbef72181df4d0fe73f14"
 dependencies = [
  "itoa",
  "memchr",
@@ -819,7 +819,7 @@ checksum = "94e153fc76e1c6a068703d6d29c508a0b15c061c4b7e43da59cc097bc342673c"
 dependencies = [
  "proc-macro2",
  "quote",
- "syn",
+ "syn 2.0.117",
 ]
 
 [[package]]
@@ -857,6 +857,17 @@ dependencies = [
  "unicode-ident",
 ]
 
+[[package]]
+name = "syn"
+version = "3.0.3"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "53e9bae58849f64dfa4f5d5ae372c8341f7305f82a3868709269343628b659a3"
+dependencies = [
+ "proc-macro2",
+ "quote",
+ "unicode-ident",
+]
+
 [[package]]
 name = "tempfile"
 version = "3.27.0"
@@ -881,11 +892,11 @@ dependencies = [
 
 [[package]]
 name = "thiserror"
-version = "2.0.18"
+version = "2.0.19"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "4288b5bcbc7920c07a1149a35cf9590a2aa808e0bc1eafaade0b80947865fbc4"
+checksum = "09a43598840e33d5b0331f38c5e30d13bb11c11210a4b58f0d9b18a5a5eefcd9"
 dependencies = [
- "thiserror-impl 2.0.18",
+ "thiserror-impl 2.0.19",
 ]
 
 [[package]]
@@ -896,18 +907,18 @@ checksum = "4fee6c4efc90059e10f81e6d42c60a18f76588c3d74cb83a0b242a2b6c7504c1"
 dependencies = [
  "proc-macro2",
  "quote",
- "syn",
+ "syn 2.0.117",
 ]
 
 [[package]]
 name = "thiserror-impl"
-version = "2.0.18"
+version = "2.0.19"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "ebc4ee7f67670e9b64d05fa4253e753e016c6c95ff35b89b7941d6b856dec1d5"
+checksum = "43cbfe0cf76104d42a574802844187e84a305e531ed54455f11fbde0f10541cd"
 dependencies = [
  "proc-macro2",
  "quote",
- "syn",
+ "syn 3.0.3",
 ]
 
 [[package]]
@@ -991,7 +1002,7 @@ dependencies = [
  "bumpalo",
  "proc-macro2",
  "quote",
- "syn",
+ "syn 2.0.117",
  "wasm-bindgen-shared",
 ]
 
@@ -1051,7 +1062,7 @@ checksum = "0e8bc7269b54418e7aeeef514aa68f8690b8c0489a06b0136e5f57c4c5ccab89"
 dependencies = [
  "proc-macro2",
  "quote",
- "syn",
+ "syn 2.0.117",
 ]
 
 [[package]]

{dlin_cli-0.2.4b1 → dlin_cli-0.2.4rc2}/Cargo.toml

@@ -3,7 +3,7 @@ members = ["crates/*"]
 resolver = "3"
 
 [workspace.package]
-version = "0.2.4-beta.1"
+version = "0.2.4-rc.2"
 edition = "2024"
 license = "MIT"
 repository = "https://github.com/eitsupi/dlin"
@@ -23,7 +23,7 @@ indexmap = "2"
 minijinja = "2"
 rayon = "1"
 globset = "0.4"
-polyglot-sql = { version = "0.6.2", default-features = false, features = ["all-dialects", "semantic"] }
+polyglot-sql = { version = "0.8.0", default-features = false, features = ["all-dialects", "semantic"] }
 path-slash = "0.2.1"
 clap = { version = "4", features = ["derive", "env"] }
 libc = "0.2"
@@ -34,7 +34,7 @@ insta = "1"
 serial_test = "3.4.0"
 
 # internal
-dlin-core = { version = "0.2.4-beta.1", path = "crates/dlin-core" }
+dlin-core = { version = "0.2.4-rc.2", path = "crates/dlin-core" }
 
 [workspace.lints.rust]
 unexpected_cfgs = { level = "warn", check-cfg = ['cfg(tarpaulin_include)'] }

{dlin_cli-0.2.4b1 → dlin_cli-0.2.4rc2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dlin-cli
-Version: 0.2.4b1
+Version: 0.2.4rc2
 Classifier: Development Status :: 4 - Beta
 Classifier: Environment :: Console
 Classifier: Intended Audience :: Developers

{dlin_cli-0.2.4b1 → dlin_cli-0.2.4rc2}/crates/dlin-core/src/graph/column_lineage/cross_model.rs

@@ -5,6 +5,7 @@ use polyglot_sql::DialectType;
 
 use crate::parser::manifest::Manifest;
 
+use super::manifest_schema::make_fq_table_name;
 use super::{
     ColumnLineageCache, ColumnLineageError, ColumnSource, ModelColumnLineage, TransformationType,
     find_model_by_name,
@@ -254,11 +255,3 @@ fn compute_single_column_lineage(
         .ok()
         .map(|r| (r.sources, r.transformation))
 }
-
-fn make_fq_table_name(database: Option<&str>, schema: Option<&str>, name: &str) -> String {
-    match (database, schema) {
-        (Some(db), Some(s)) => format!("{}.{}.{}", db, s, name),
-        (None, Some(s)) => format!("{}.{}", s, name),
-        _ => name.to_string(),
-    }
-}

dlin_cli-0.2.4rc2/crates/dlin-core/src/graph/column_lineage/manifest_schema.rs

@@ -0,0 +1,302 @@
+//! Manifest helpers that are safe to use for ordinal-aware analysis.
+//!
+//! The legacy YAML schema builders remain in `schema.rs`.  The builder in this
+//! module has a stricter contract: only a compiled query whose output order is
+//! proven by polyglot-sql may become a `MappingSchema` relation.
+
+use std::collections::HashSet;
+
+use polyglot_sql::{DialectType, MappingSchema, Schema};
+
+use crate::parser::cache::hash_str;
+use crate::parser::manifest::{Manifest, ManifestNode};
+
+use super::semantic_model::build_query_model_from_sql;
+
+pub(super) fn make_fq_table_name(
+    database: Option<&str>,
+    schema: Option<&str>,
+    name: &str,
+) -> String {
+    match (database, schema) {
+        (Some(db), Some(s)) => format!("{}.{}.{}", db, s, name),
+        (None, Some(s)) => format!("{}.{}", s, name),
+        _ => name.to_string(),
+    }
+}
+
+/// Build a schema only from dependency models with a proven SQL output order.
+///
+/// Manifest YAML columns are intentionally not consulted for the relation's
+/// column order (or for adding a relation).  A source, YAML-only model,
+/// unnamed output, duplicate output name, or unresolved wildcard is omitted.
+pub fn build_proven_ordered_schema(
+    manifest: &Manifest,
+    node: &ManifestNode,
+    dialect: DialectType,
+) -> Option<MappingSchema> {
+    let mut schema = MappingSchema::with_dialect(dialect);
+    let mut has_entries = false;
+
+    for dependency_id in &node.depends_on.nodes {
+        let Some(dependency) = manifest.nodes.get(dependency_id) else {
+            // Sources and other YAML-only relations do not prove an ordinal
+            // schema for this stage.
+            continue;
+        };
+        let Some(sql) = dependency.compiled_code.as_deref() else {
+            continue;
+        };
+        let Ok(query_model) = build_query_model_from_sql(sql, dialect, None) else {
+            continue;
+        };
+        let Some(column_names) = query_model.outputs.proven_ordered_named_outputs() else {
+            continue;
+        };
+
+        let columns = column_names
+            .into_iter()
+            .map(|name| (name, polyglot_sql::expressions::DataType::Unknown))
+            .collect::<Vec<_>>();
+        let qualified_name = make_fq_table_name(
+            dependency.database.as_deref(),
+            dependency.schema.as_deref(),
+            &dependency.name,
+        );
+        if schema.add_table(&qualified_name, &columns, None).is_ok() {
+            has_entries = true;
+        }
+        if qualified_name != dependency.name {
+            let _ = schema.add_table(&dependency.name, &columns, None);
+        }
+    }
+
+    has_entries.then_some(schema)
+}
+
+pub(super) fn compute_manifest_columns_hash(manifest: &Manifest, node: &ManifestNode) -> u64 {
+    let mut visited = HashSet::new();
+    hash_node_columns_transitive(manifest, node, &mut visited)
+}
+
+fn hash_node_columns_transitive(
+    manifest: &Manifest,
+    node: &ManifestNode,
+    visited: &mut HashSet<String>,
+) -> u64 {
+    let mut parts = Vec::new();
+
+    let mut own_cols = node.columns.keys().collect::<Vec<_>>();
+    own_cols.sort_unstable();
+    for column in own_cols {
+        parts.push(column.clone());
+    }
+    if let Some(code) = &node.compiled_code {
+        parts.push(format!("sql:{}", hash_str(code)));
+    }
+    parts.push("|".to_string());
+
+    let mut dependency_ids = node.depends_on.nodes.iter().collect::<Vec<_>>();
+    dependency_ids.sort_unstable();
+    for dependency_id in dependency_ids {
+        parts.push(dependency_id.clone());
+        if !visited.insert(dependency_id.clone()) {
+            continue;
+        }
+        if let Some(dependency) = manifest.nodes.get(dependency_id) {
+            let dependency_hash = hash_node_columns_transitive(manifest, dependency, visited);
+            parts.push(format!("node:{}", dependency_hash));
+        } else if let Some(source) = manifest.sources.get(dependency_id) {
+            let mut columns = source.columns.keys().collect::<Vec<_>>();
+            columns.sort_unstable();
+            for column in columns {
+                parts.push(column.clone());
+            }
+            if let Some(database) = &source.database {
+                parts.push(format!("db:{}", database));
+            }
+            if let Some(schema) = &source.schema {
+                parts.push(format!("schema:{}", schema));
+            }
+            if let Some(identifier) = &source.identifier {
+                parts.push(format!("id:{}", identifier));
+            }
+        }
+    }
+
+    hash_str(&parts.join("\0"))
+}
+
+#[cfg(test)]
+mod tests {
+    use std::collections::HashMap;
+
+    use super::*;
+    use crate::parser::manifest::{DependsOn, ManifestColumn, ManifestConfig};
+
+    fn node(
+        unique_id: &str,
+        name: &str,
+        compiled_code: Option<&str>,
+        columns: &[&str],
+        dependencies: &[&str],
+    ) -> ManifestNode {
+        let columns = columns
+            .iter()
+            .map(|name| {
+                (
+                    (*name).to_string(),
+                    ManifestColumn {
+                        name: (*name).to_string(),
+                    },
+                )
+            })
+            .collect();
+        ManifestNode {
+            unique_id: unique_id.to_string(),
+            name: name.to_string(),
+            resource_type: "model".to_string(),
+            depends_on: DependsOn {
+                nodes: dependencies.iter().map(|id| (*id).to_string()).collect(),
+            },
+            config: ManifestConfig::default(),
+            description: None,
+            path: None,
+            original_file_path: None,
+            columns,
+            compiled_code: compiled_code.map(str::to_string),
+            database: None,
+            schema: None,
+        }
+    }
+
+    fn manifest(nodes: Vec<ManifestNode>) -> Manifest {
+        Manifest {
+            nodes: nodes
+                .into_iter()
+                .map(|node| (node.unique_id.clone(), node))
+                .collect::<HashMap<_, _>>(),
+            ..Manifest::default()
+        }
+    }
+
+    #[test]
+    fn yaml_map_order_does_not_change_proven_ordinal_schema() {
+        let upstream_id = "model.test.upstream";
+        let downstream_id = "model.test.downstream";
+        let upstream_sql = Some("SELECT first_col, second_col FROM source_table");
+        let downstream = node(
+            downstream_id,
+            "downstream",
+            Some("SELECT first_col FROM upstream"),
+            &[],
+            &[upstream_id],
+        );
+
+        let first = build_proven_ordered_schema(
+            &manifest(vec![
+                node(
+                    upstream_id,
+                    "upstream",
+                    upstream_sql,
+                    &["first_col", "second_col"],
+                    &[],
+                ),
+                node(
+                    downstream_id,
+                    "downstream",
+                    Some("SELECT first_col FROM upstream"),
+                    &[],
+                    &[upstream_id],
+                ),
+            ]),
+            &downstream,
+            DialectType::Generic,
+        )
+        .unwrap();
+        assert_eq!(
+            first.column_names("upstream").unwrap(),
+            vec!["first_col", "second_col"]
+        );
+        // The second manifest has the same SQL but a different YAML map order.
+        // Rebuild the node reference from that manifest so only the SQL proof
+        // participates in the schema.
+        let second_manifest = manifest(vec![
+            node(
+                upstream_id,
+                "upstream",
+                upstream_sql,
+                &["second_col", "first_col"],
+                &[],
+            ),
+            node(
+                downstream_id,
+                "downstream",
+                Some("SELECT first_col FROM upstream"),
+                &[],
+                &[upstream_id],
+            ),
+        ]);
+        let second = build_proven_ordered_schema(
+            &second_manifest,
+            second_manifest.nodes.get(downstream_id).unwrap(),
+            DialectType::Generic,
+        )
+        .unwrap();
+        assert_eq!(
+            second.column_names("upstream").unwrap(),
+            first.column_names("upstream").unwrap()
+        );
+    }
+
+    #[test]
+    fn only_proven_compiled_named_outputs_enter_the_schema() {
+        let good_id = "model.test.good";
+        let unnamed_id = "model.test.unnamed";
+        let star_id = "model.test.star";
+        let downstream = node(
+            "model.test.downstream",
+            "downstream",
+            Some("SELECT value FROM good"),
+            &[],
+            &[good_id, unnamed_id, star_id],
+        );
+        let manifest = manifest(vec![
+            node(
+                good_id,
+                "good",
+                Some("SELECT first_col, second_col FROM t"),
+                &[],
+                &[],
+            ),
+            node(
+                unnamed_id,
+                "unnamed",
+                Some("SELECT value * 2 FROM t"),
+                &["yaml_name"],
+                &[],
+            ),
+            node(
+                star_id,
+                "star",
+                Some("SELECT * FROM unknown"),
+                &["yaml_name"],
+                &[],
+            ),
+            downstream,
+        ]);
+
+        let schema = build_proven_ordered_schema(
+            &manifest,
+            manifest.nodes.get("model.test.downstream").unwrap(),
+            DialectType::Generic,
+        )
+        .unwrap();
+        assert_eq!(
+            schema.column_names("good").unwrap(),
+            vec!["first_col", "second_col"]
+        );
+        assert!(schema.column_names("unnamed").is_err());
+        assert!(schema.column_names("star").is_err());
+    }
+}

{dlin_cli-0.2.4b1 → dlin_cli-0.2.4rc2}/crates/dlin-core/src/graph/column_lineage/mod.rs

@@ -1,7 +1,7 @@
 use std::collections::HashSet;
 use std::path::Path;
 
-use polyglot_sql::DialectType;
+use polyglot_sql::{ColumnResolutionReason, DialectType, Error};
 use rayon::prelude::*;
 
 use crate::parser::manifest::Manifest;
@@ -9,8 +9,11 @@ use crate::parser::manifest::Manifest;
 mod cache;
 mod cross_model;
 mod impact;
+pub mod manifest_schema;
 mod schema;
+pub mod semantic_model;
 mod single_model;
+mod star_guard;
 #[cfg(test)]
 mod tests;
 mod types;
@@ -23,8 +26,10 @@ pub use impact::{
     ColumnImpactReport, ImpactedColumn, compute_column_impact,
     compute_column_impact_with_manifest_path,
 };
-use schema::{build_yaml_schema_for_node, compute_manifest_columns_hash, infer_output_columns};
-use single_model::{has_unresolved_stars, prepare_lineage_context_with_expr, run_column_lineage};
+use manifest_schema::compute_manifest_columns_hash;
+use schema::{build_yaml_schema_for_node, infer_output_columns};
+use single_model::{prepare_lineage_context_with_expr, run_column_lineage};
+use star_guard::has_unresolved_stars;
 pub use types::{
     ColumnLineageEntry, ColumnLineageError, ColumnLineageErrorKind, ColumnSource,
     ModelColumnLineage, TransformationType,
@@ -174,14 +179,19 @@ pub fn compute_column_lineage_with_manifest_path(
                 sources: result.sources,
             }),
             Err(e) => {
-                let hint = (has_star_columns && e.contains("Cannot find column")).then(|| {
-                    "column may be introduced via SELECT * that could not be expanded; \
+                let hint =
+                    (has_star_columns && is_indeterminate_column_resolution(&e)).then(|| {
+                        "column may be introduced via SELECT * that could not be expanded; \
                      define upstream columns in the model YAML to enable full resolution"
-                        .to_string()
-                });
+                            .to_string()
+                    });
                 Err(ColumnLineageError {
                     kind: ColumnLineageErrorKind::ColumnNotFound,
-                    what: format!("column '{}': {}", col_name, e),
+                    what: format!(
+                        "column '{}': {}",
+                        col_name,
+                        single_model::format_lineage_error(&e)
+                    ),
                     why: None,
                     hint,
                 })
@@ -218,6 +228,16 @@ pub fn compute_column_lineage_with_manifest_path(
     result
 }
 
+fn is_indeterminate_column_resolution(error: &Error) -> bool {
+    matches!(
+        error,
+        Error::ColumnResolution {
+            reason: ColumnResolutionReason::Indeterminate,
+            ..
+        }
+    )
+}
+
 pub(super) fn find_model_by_name<'a>(
     manifest: &'a Manifest,
     name: &str,