From 6a5d6312871ccdbdeaf7790e438f87568e7d9a50 Mon Sep 17 00:00:00 2001
From: Per Stark <per@stark.pub>
Date: Mon, 25 May 2026 14:02:53 +0200
Subject: [PATCH] chore: remove unused clap dep and fix test_session_table name

- Remove clap dependency from retrieval-pipeline (RetrievalStrategy
  already has FromStr/Display; evaluations uses clap directly)
- Rename session table from test_session_table to session
---
 Cargo.lock                                | 1 -
 common/src/storage/db.rs                  | 2 +-
 retrieval-pipeline/Cargo.toml             | 4 ----
 retrieval-pipeline/src/pipeline/config.rs | 2 +-
 4 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/Cargo.lock b/Cargo.lock
index 1d043f0..8290cb9 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -5422,7 +5422,6 @@ dependencies = [
  "async-openai",
  "async-trait",
  "axum",
- "clap",
  "common",
  "fastembed",
  "futures",
diff --git a/common/src/storage/db.rs b/common/src/storage/db.rs
index 8aa9776..30b15b4 100644
--- a/common/src/storage/db.rs
+++ b/common/src/storage/db.rs
@@ -75,7 +75,7 @@ impl SurrealDbClient {
         SessionStore::new(
             Some(self.client.clone().into()),
             SessionConfig::default()
-                .with_table_name("test_session_table")
+                .with_table_name("session")
                 .with_secure(true),
         )
         .await
diff --git a/retrieval-pipeline/Cargo.toml b/retrieval-pipeline/Cargo.toml
index 8478ee1..c2fd692 100644
--- a/retrieval-pipeline/Cargo.toml
+++ b/retrieval-pipeline/Cargo.toml
@@ -21,9 +21,5 @@ async-openai = { workspace = true }
 async-trait = { workspace = true }
 uuid = { workspace = true }
 fastembed = { workspace = true }
-clap = { version = "4.4", features = ["derive"] }
- 
-common = { path = "../common" }
 
-[dev-dependencies]
 common = { path = "../common", features = ["test-utils"] }
diff --git a/retrieval-pipeline/src/pipeline/config.rs b/retrieval-pipeline/src/pipeline/config.rs
index b698f15..12d3320 100644
--- a/retrieval-pipeline/src/pipeline/config.rs
+++ b/retrieval-pipeline/src/pipeline/config.rs
@@ -3,7 +3,7 @@ use std::fmt;
 
 use crate::scoring::FusionWeights;
 
-#[derive(Debug, Clone, Copy, PartialEq, Eq, Serialize, Deserialize, clap::ValueEnum)]
+#[derive(Debug, Clone, Copy, PartialEq, Eq, Serialize, Deserialize)]
 #[serde(rename_all = "snake_case")]
 pub enum RetrievalStrategy {
     /// Primary hybrid chunk retrieval for search/chat (formerly Revised)