Remove hash as well when unlinking

Author: msfstef

packages/sync-service/lib/electric/shape_cache/shape_status.ex

@@ -153,6 +153,13 @@ defmodule Electric.ShapeCache.ShapeStatus do
   def unlink_handle_from_shape(stack_ref, shape_handle) do
     stack_id = extract_stack_id(stack_ref)
     shape = ShapeDb.unlink_handle_from_shape!(stack_id, shape_handle)
+
+    :ets.update_element(
+      shape_meta_table(stack_ref),
+      shape_handle,
+      {@shape_meta_shape_hash_pos, nil}
+    )
+
     {:ok, shape}
   rescue
     ArgumentError ->

packages/sync-service/test/electric/shape_cache/shape_status_test.exs

@@ -114,6 +114,7 @@ defmodule Electric.ShapeCache.ShapeStatusTest do
 
     # After unlinking, the shape should not be findable via get_existing_shape
     refute ShapeStatus.get_existing_shape(state, shape)
+    refute ShapeStatus.has_shape_handle?(state, shape_handle)
 
     # But the shape should still be fetchable by handle (metadata still exists)
     assert {:ok, ^shape} = ShapeStatus.fetch_shape_by_handle(state, shape_handle)