fix: fix declarator removal only deletes one binding

[nerodesu017]

Q	A
Fixed Issues?	Fixes #16583
Patch: Bug Fix?	Yes, hotfix
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass?	No
Documentation PR Link
Any Dependency Changes?
License	MIT

As specified above, this PR fixes a problem where removing the only VariableDeclarator of a VariableDeclaration node leads to removal (or at least attempt) of 2 Bindings with the same name from different scopes (the removal of the declarator removes the correct binding, but the removal of the declaration attempts to remove it again, and if it finds it in a parent scope, it deletes it from there too)

{
  let a = 1; // removing the VariableDeclarator removes the binding of `a` from this scope
}
let a = 2; // when the first removal hook gets called, the binding of `a` from this scope gets removed
             // this is because the `getBindingIdentifiers` function was giving the same bindings even
             // after having them removed

The current solution is a HOTFIX and documented as such in the comments of the typescript code. I am open to suggestions for making it better.

[babel-bot]

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/57194