no-unused-variable: Not detecting unused in object deconstruction

[OliverJAsh]

Bug Report

• TSLint version: 3.15.1
• TypeScript version: 2.0.2
• Running TSLint via: Visual Studio Code

TypeScript code being linted

const foo = () => ({ y: 1 });

(() => {
    const { y } = foo();
})()

(() => {
    const { y } = foo();
})()

with tslint.json configuration:

{
    "rules": {
        "no-unused-variable": true
    }
}

Actual behavior

No error

Expected behavior

y is unused in both IIFEs.

[adidahiya]

we're deprecating this rule in v4.0, so I'm marking this as "won't fix". check out #1481 for reasoning & alternatives.

[ghost]

Should be fixed by #2235.

[ajafff]

This was fixed in the v5.0.0 release