fix(SyncProcess): Fix concurrentOriginTargetRemoval

Only inject moves whose old position is itself swept up by the same target removal

Assisted-by: ClaudeCode:claude-opus-4-7
Signed-off-by: Marcel Klehr <mklehr@gmx.net>

Author: marcelklehr

src/lib/strategies/Default.ts

@@ -934,7 +934,24 @@ export default class SyncProcess {
               })
             }
 
-            const pendingMoves = sourceMoves.filter(move => move !== action)
+            // Only inject moves whose old position is itself swept up by the same
+            // target removal — those items lose their existing target counterpart
+            // and need to be recreated alongside us. Moves with an intact old
+            // counterpart must NOT be injected, otherwise the bulk-import below
+            // would create a duplicate on the target and the sub-scanner's
+            // addMapping would silently evict the existing mapping.
+            const pendingMoveChainCache = {}
+            const pendingMoves = sourceMoves.filter(move =>
+              move !== action &&
+              Diff.findChain(
+                mappingsSnapshot,
+                allCreateAndMoveActions,
+                sourceTree,
+                move.oldItem,
+                concurrentTargetOriginRemoval,
+                pendingMoveChainCache
+              )
+            )
 
             let insertedMove = true
             while (insertedMove) {