Split transform into seperate crate

Cargo.toml

@@ -4,6 +4,7 @@ members = [
  "test-helpers",
  "tokio-bin-process",
  "custom-transforms-example",
+ "redis-get-rewrite"
 ]
 
 # https://deterministic.space/high-performance-rust.html

custom-transforms-example/Cargo.toml

@@ -11,12 +11,9 @@ license = "Apache-2.0"
 shotover-proxy = { path = "../shotover-proxy" }
 anyhow = "1.0"
 clap = "4.0.4"
-serde = "1.0"
-async-trait = "0.1.30"
-tracing = "0.1"
-typetag = "0.2.6"
+redis-get-rewrite = { path = "../redis-get-rewrite" }
 
 [dev-dependencies]
 tokio = "1.25.0"
-test-helpers = {path = "../test-helpers"}
+test-helpers = { path = "../test-helpers"}
 redis = "0.22.0"

custom-transforms-example/src/main.rs

@@ -1,10 +1,12 @@
 use anyhow::Result;
 use clap::Parser;
-
+// We need to import this to get the typetag to register.
+// TODO: I suspect hiding it behind a `shotover_proxy::import_transform!(redis_get_rewrite::RedisGetRewriteConfig)` would look nicer.
+// We could also have the macro check that it implements the TransformConfig trait
+#[allow(unused_imports)]
+use redis_get_rewrite::RedisGetRewriteConfig;
 use shotover_proxy::runner::{ConfigOpts, Runner};
 
-mod redis_get_rewrite;
-
 fn main() -> Result<()> {
  Runner::new(ConfigOpts::parse())?
  .with_observability_interface()?

redis-get-rewrite/Cargo.toml

@@ -0,0 +1,17 @@
+[package]
+name = "redis-get-rewrite"
+version = "0.0.1"
+authors = ["Ben <ben@instaclustr.com>"]
+edition = "2021"
+license = "Apache-2.0"
+
+# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
+
+[dependencies]
+shotover-proxy = { path = "../shotover-proxy" }
+anyhow = "1.0"
+clap = "4.0.4"
+serde = "1.0"
+async-trait = "0.1.30"
+tracing = "0.1"
+typetag = "0.2.6"