Exclude completions of binding pattern variable initializers

[zardoy]

Bug Report

Completions from binding pattern variable initializers are included, but they shouldn't

🔎 Search Terms

no suggestions const destruct destructure

🕗 Version & Regression Information

Copied from available fix #52723 :

const {a1} = a/* should not complete a1 */

Real world case:

let pageToReturn

const { data: pageData } = await octokit.repos.listTags({
    page: page /* pageData is suggested first */
})

Some kind of follow up for #42087

[RyanCavanaugh]

Care should be taken to not cause false negatives, e.g.

const { tick } = {
  tick() {
    window.setTimeout(tick, 100);
    //                ^^^^ valid
  }
}

In general "Did not offer a thing that was legal" is a much worse failure mode for UX than "Offered a thing that was TDZ"