fix: filter session struct

Author: zoedsoupe

config/config.exs

@@ -10,6 +10,17 @@ config :logger, :default_formatter,
   format: "[$level] $message $metadata\n",
   metadata: [:mcp_server, :mcp_client, :mcp_client_name, :mcp_transport]
 
+if System.get_env("ENABLE_SESSION_STORE") == "true" do
+  config :anubis_mcp, :session_store,
+    enabled: true,
+    adapter: Anubis.Server.Session.Store.Redis,
+    redis_url: System.get_env("REDIS_URL", "redis://localhost:6379"),
+    pool_size: String.to_integer(System.get_env("REDIS_POOL_SIZE", "10")),
+    ttl: String.to_integer(System.get_env("SESSION_TTL", "1800000")),
+    namespace: System.get_env("SESSION_NAMESPACE", "anubis:sessions"),
+    connection_name: :anubis_redis
+end
+
 # Session store configuration - disabled by default
 # To enable Redis persistence, uncomment and configure:
 # config :anubis_mcp, :session_store,

config/dev.exs

@@ -213,12 +213,11 @@ defmodule Anubis.Server.Session do
     if store = Anubis.get_session_store_adapter() do
       Logging.log(:debug, "Persisting session #{inspect(state.id)} to store", [])
 
-      # Convert struct to mapand remove runtime fields
       state_map =
         state
         |> Map.from_struct()
-        # Don't persist process names
         |> Map.delete(:name)
+        |> dbg()
 
       case store.save(state.id, state_map, []) do
         :ok ->

lib/anubis/server/session.ex

@@ -108,20 +108,17 @@ if Code.ensure_loaded?(Redix) do
       pool_size = Keyword.get(opts, :pool_size, 10)
       namespace = Keyword.get(opts, :namespace, @default_namespace)
       ttl = Keyword.get(opts, :ttl, @default_ttl)
-      # Strip :name from custom opts to preserve internal pool naming
+
       custom_redix_opts =
         opts
         |> Keyword.get(:redix_opts, [])
         |> validate_redix_opts()
         |> Keyword.delete(:name)
 
-      # Start Redix connection pool with anubis_ prefix to avoid conflicts
       children =
         for i <- 1..pool_size do
           child_id = :"anubis_#{conn_name}_#{i}"
 
-          # Default Redix options, merged with custom options (custom takes precedence)
-          # Note: :name is always set internally to maintain pool integrity
           redix_opts =
             Keyword.merge([name: child_id, sync_connect: false, exit_on_disconnection: false], custom_redix_opts)
 
@@ -131,10 +128,8 @@ if Code.ensure_loaded?(Redix) do
           }
         end
 
-      # Use anubis_ prefix for supervisor name
       supervisor_name = :"anubis_#{conn_name}_supervisor"
 
-      # Start connections under a supervisor
       case Supervisor.start_link(children, strategy: :one_for_one, name: supervisor_name) do
         {:ok, _pid} ->
           state = %State{
@@ -335,9 +330,16 @@ if Code.ensure_loaded?(Redix) do
       div(milliseconds, 1000)
     end
 
+    defp filter_data(data) do
+      data
+      |> Map.from_struct()
+      |> Map.delete(:name)
+    end
+
     defp encode_and_save(state, key, data, ttl) do
       conn = get_connection(state)
       ttl_seconds = ms_to_seconds(ttl)
+      data = filter_data(data)
 
       case json_encode(data) do
         {:ok, json} ->