Invalid JS Doc causes invalid destructuring optimizations

[ChadKillingsworth]

Invalid JS Doc causes destructuring to produce invalid code:

/**
 * @param {{foo: string[]}} obj
 * @param {string} bar
 */
function someFn({ foo }, bar) {
  console.log(foo, bar);
}

window['someFn'] = someFn;

https://closure-compiler.appspot.com/home#code%3D%252F%252F%2520%253D%253DClosureCompiler%253D%253D%250A%252F%252F%2520%2540output_file_name%2520default.js%250A%252F%252F%2520%253D%253D%252FClosureCompiler%253D%253D%250A%250A%252F**%250A%2520*%2520%2540param%2520%257B%257Bfoo%253A%2520string%255B%255D%257D%257D%2520obj%250A%2520*%2520%2540param%2520%257Bstring%257D%2520bar%250A%2520*%252F%250Afunction%2520someFn(%257B%2520foo%2520%257D%252C%2520bar)%2520%257B%250A%2520%2520console.log(foo%252C%2520bar)%253B%250A%257D%250A%250Awindow%255B'someFn'%255D%2520%253D%2520someFn%253B

[ChadKillingsworth]

As a side note, the JSDoc array format of string[] is becoming ever more popular. We might want to consider supporting it.

[blickly]

Wow, that's quite surprising!

Looks to be an issue during transpilation. Here's a repro on the debugger:
https://closure-compiler-debugger.appspot.com/#input0%3D%252F**%250A%2520*%2520%2540param%2520%257B%257Bfoo%253A%2520string%255B%255D%257D%257D%2520obj%250A%2520*%2520%2540param%2520%257Bstring%257D%2520bar%250A%2520*%252F%250Afunction%2520someFn(%257B%2520foo%2520%257D%252C%2520bar)%2520%257B%250A%2520%2520console.log(foo%252C%2520bar)%253B%250A%257D%250A%250Awindow%255B'someFn'%255D%2520%253D%2520someFn%253B%26input1%26conformanceConfig%26externs%26refasterjs-template%26includeDefaultExterns%3Dtrue%26CHECK_TYPES%3Dtrue%26TRANSPILE%3Dtrue%26CLOSURE_PASS%3Dtrue%26PRESERVE_TYPE_ANNOTATIONS%3Dtrue%26PRETTY_PRINT%3Dtrue

[blickly]

Created internal bug b/121271660 to track