Fix test failures and compiler warnings

- Remove unused alias SanitizeFilter from checkend.ex
- Remove unused alias Configuration from notice_builder.ex
- Convert atom keys to strings in SanitizeFilter for consistent JSON output
- Fix deep nesting test expectation (Enum.reduce produces outermost level=14)
- Update atom test to use string key access

Author: geetfun

lib/checkend.ex

@@ -28,7 +28,7 @@ defmodule Checkend do
   @version "0.1.0"
 
   alias Checkend.{Configuration, Notice, NoticeBuilder, Client, Worker, Testing}
-  alias Checkend.Filters.{SanitizeFilter, IgnoreFilter}
+  alias Checkend.Filters.IgnoreFilter
 
   @doc """
   Returns the SDK version.

lib/checkend/filters/sanitize.ex

@@ -48,12 +48,13 @@ defmodule Checkend.Filters.SanitizeFilter do
 
       data
       |> Enum.map(fn {key, value} ->
+        # Convert atom keys to strings for consistent JSON output
         key_str = to_string(key)
 
         if should_filter?(filter, key_str) do
-          {key, @filtered_value}
+          {key_str, @filtered_value}
         else
-          {key, filter_value(filter, value, depth + 1, new_seen)}
+          {key_str, filter_value(filter, value, depth + 1, new_seen)}
         end
       end)
       |> Enum.into(%{})

lib/checkend/notice_builder.ex

@@ -3,7 +3,7 @@ defmodule Checkend.NoticeBuilder do
   Builds Notice structs from exceptions.
   """
 
-  alias Checkend.{Configuration, Notice}
+  alias Checkend.Notice
   alias Checkend.Filters.SanitizeFilter
 
   @max_backtrace_lines 100

test/sanitize_filter_test.exs

@@ -114,14 +114,15 @@ defmodule Checkend.Filters.SanitizeFilterTest do
 
     test "handles deep nesting", %{filter: filter} do
       # Create deeply nested structure
+      # Enum.reduce wraps from inside out, so final outermost level is 14
       data =
         Enum.reduce(0..14, %{"level" => 15}, fn i, acc ->
           %{"level" => i, "nested" => acc}
         end)
 
       # Should not raise, should handle max depth
       result = SanitizeFilter.filter(filter, data)
-      assert result["level"] == 0
+      assert result["level"] == 14
     end
 
     test "handles empty map", %{filter: filter} do
@@ -133,8 +134,9 @@ defmodule Checkend.Filters.SanitizeFilterTest do
       data = %{status: :active, password: :secret}
       result = SanitizeFilter.filter(filter, data)
 
-      assert result[:status] == "active"
-      assert result[:password] == "[FILTERED]"
+      # Atom keys are converted to strings, atom values are converted to strings
+      assert result["status"] == "active"
+      assert result["password"] == "[FILTERED]"
     end
   end
 end