refactor: bring back stac-validate (#765)

Closes #748

Author: gadomski

Cargo.toml

@@ -10,15 +10,17 @@ members = [
     "crates/io",
     "crates/pgstac",
     "crates/server",
+    "crates/validate",
     "crates/wasm",
 ]
 default-members = [
     "crates/api",
     "crates/core",
-    "crates/io",
     "crates/derive",
     "crates/extensions",
+    "crates/io",
     "crates/server",
+    "crates/validate",
 ]
 
 [workspace.package]
@@ -85,6 +87,7 @@ stac-duckdb = { version = "0.1.1", path = "crates/duckdb" }
 stac-extensions = { version = "0.1.0", path = "crates/extensions" }
 stac-io = { version = "0.1.0", path = "crates/io" }
 stac-server = { version = "0.3.2", path = "crates/server" }
+stac-validate = { version = "0.4.0", path = "crates/validate" }
 syn = "2.0"
 tempfile = "3.16"
 thiserror = "2.0"

crates/cli/Cargo.toml

@@ -30,10 +30,10 @@ stac-duckdb.workspace = true
 stac-io = { workspace = true, features = [
     "store-all",
     "reqwest",
-    "validate",
     "geoparquet-compression",
 ] }
 stac-server = { workspace = true, features = ["axum", "duckdb"] }
+stac-validate.workspace = true
 tokio = { workspace = true, features = [
     "macros",
     "io-std",

crates/cli/src/lib.rs

@@ -9,8 +9,9 @@ use futures_core::TryStream;
 use futures_util::{TryStreamExt, pin_mut};
 use stac::{Assets, Collection, Item, Links, Migrate, SelfHref, geoparquet::Compression};
 use stac_api::{GetItems, GetSearch, Search};
-use stac_io::{Format, StacStore, Validate};
+use stac_io::{Format, StacStore};
 use stac_server::Backend;
+use stac_validate::Validate;
 use std::{
     collections::{HashMap, VecDeque},
     io::Write,
@@ -492,7 +493,7 @@ impl Rustac {
                     .spawn_blocking(move || value.validate())
                     .await?;
                 if let Err(error) = result {
-                    if let stac_io::Error::Validation(errors) = error {
+                    if let stac_validate::Error::Validation(errors) = error {
                         if let Some(format) = self.output_format {
                             if let Format::Json(_) = format {
                                 let value = errors

crates/core/Cargo.toml

@@ -55,6 +55,7 @@ url = { workspace = true, features = ["serde"] }
 assert-json-diff.workspace = true
 bytes.workspace = true
 rstest.workspace = true
+stac-validate.workspace = true
 tokio = { workspace = true, features = ["macros"] }
 tokio-test.workspace = true
 

crates/core/src/lib.rs

@@ -290,6 +290,7 @@ pub fn version() -> &'static str {
 #[cfg(test)]
 mod tests {
     use rstest as _;
+    use stac_validate as _;
     use tokio as _;
     use tokio_test as _;
 

crates/io/tests/examples.rs renamed to crates/core/tests/examples.rs

@@ -1,6 +1,6 @@
 use rstest::rstest;
 use stac::Value;
-use stac_io::Validate;
+use stac_validate::Validate;
 use std::path::PathBuf;
 
 #[rstest]

crates/io/tests/migrate.rs renamed to crates/core/tests/migrate.rs

@@ -1,6 +1,6 @@
 use rstest::rstest;
 use stac::{Migrate, Value, Version};
-use stac_io::Validate;
+use stac_validate::Validate;
 use std::path::PathBuf;
 
 #[rstest]

crates/duckdb/Cargo.toml

@@ -33,4 +33,4 @@ thiserror.workspace = true
 [dev-dependencies]
 geo.workspace = true
 rstest.workspace = true
-stac-io = { workspace = true, features = ["validate"] }
+stac-validate.workspace = true

crates/duckdb/src/client.rs

@@ -316,7 +316,7 @@ impl Client {
         if let Some(filter) = search.items.filter {
             let expr: Expr = filter.try_into()?;
             if expr_properties_match(&expr, &column_names) {
-                let sql = expr.to_ducksql()?;
+                let sql = expr.to_ducksql().map_err(Box::new)?;
                 wheres.push(sql);
             } else {
                 return Ok(Vec::new());
@@ -425,7 +425,7 @@ mod tests {
     use rstest::{fixture, rstest};
     use stac::Bbox;
     use stac_api::{Search, Sortby};
-    use stac_io::Validate;
+    use stac_validate::Validate;
 
     #[fixture]
     #[once]

crates/duckdb/src/error.rs

@@ -10,7 +10,7 @@ pub enum Error {
 
     /// [cql2::Error]
     #[error(transparent)]
-    Cql2(#[from] cql2::Error),
+    Cql2(#[from] Box<cql2::Error>),
 
     /// [duckdb::Error]
     #[error(transparent)]