fix: retract all nodes before updating the map

Author: Christopher-Chianelli

core/src/main/java/ai/timefold/solver/core/impl/bavet/uni/StaticDataUniNode.java

@@ -68,6 +68,7 @@ public void insert(@Nullable Object a) {
         if (tupleMap.containsKey(a)) {
             return;
         }
+        tupleMap.values().stream().flatMap(List::stream).forEach(this::retractExisting);
         tupleMap.put(a, new ArrayList<>());
         insertIntoNodeNetwork(a);
         recalculateTuples();
@@ -96,6 +97,7 @@ public void retract(@Nullable Object a) {
         if (!tupleMap.containsKey(a)) {
             return;
         }
+        tupleMap.values().stream().flatMap(List::stream).forEach(this::retractExisting);
         tupleMap.remove(a);
         retractFromNodeNetwork(a);
         recalculateTuples();
@@ -135,8 +137,6 @@ private void retractFromNodeNetwork(Object toRetract) {
     }
 
     private void recalculateTuples() {
-        tupleMap.values().stream().flatMap(List::stream).forEach(this::retractExisting);
-
         var recorder = recordingTupleNode.getTupleRecorder();
         recorder.reset();
         for (var mappedTupleEntry : tupleMap.entrySet()) {