fix(javascript): retain destructured intrinsic writes

Share one pattern-target walker across lexical definitions, member mutation
tracking and Electron identity writes. Keep overwritten globals unresolved
without treating property keys or shadowed locals as global replacements.

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
Co-authored-by: Kaoru0822 <208281322+kaoru0822-kitauji@users.noreply.github.com>

src/domain/javascript/javascriptSemanticMemberMutations.ts

@@ -33,7 +33,7 @@ import { compareUnicodeCodePoints } from "../unicodeCodePointOrder.js";
 import { traverseJavaScriptAstSteps } from "./javascriptSemanticTraversal.js";
 import {
   semanticStaticPropertyKey,
-  semanticMemberWriteTargets,
+  semanticWriteTargets,
   unwrapJavaScriptExpression,
 } from "./javascriptAstValues.js";
 
@@ -401,14 +401,15 @@ export function* collectSemanticMemberMutationsSteps(
       markEscaped(expression.object, mutation);
   };
   const markTarget = (node: t.Node, mutation: t.Node): void => {
-    for (const target of semanticMemberWriteTargets(node))
-      markValue(
-        target.object,
-        [semanticStaticPropertyKey(target.property, target.computed)],
-        new Set(),
-        "write",
-        mutation,
-      );
+    for (const target of semanticWriteTargets(node))
+      if (!t.isIdentifier(target))
+        markValue(
+          target.object,
+          [semanticStaticPropertyKey(target.property, target.computed)],
+          new Set(),
+          "write",
+          mutation,
+        );
   };
   yield* traverseJavaScriptAstSteps(program, {
     enter: (node, parent) => {