no-unused-var rule creates invalid code (2)

[laurent22]

Tell us about your environment

• eslint version: 6.1.0
• eslint-plugin-autofix version: 0.0.8
• eslint config:

It changes this:

sqliteErrorToJsError(error, sql = null, params = null) {
	return error;
}

to this:

sqliteErrorToJsError(error, , ) {
	return error;
}

[Nantris]

Also creates invalid code like:

From:

const someFunc = debounce(arg => {...})

To:

const someFunc = debounce( => {...})

Should change to:

const someFunc = debounce(() => {...})

[Nantris]

Also the super mysterious case of a used var being left alone, but needlessly having the comma afterward removed.

From:

const { x, y, width, height } = this.state;
const newX = parseInt((x / prevWindow.width) * winWidth);

To:

const { x y, width, height } = this.state;
const newX = parseInt((x / prevWindow.width) * winWidth);

Edit: Actually, this happens a lot... Doesn't seem to handle destructuring assignments whatsoever.

[Nantris]

There are a lot more edge cases too... like at least 4 more issues. They all seem to be variations on what is perhaps the same underlying bug in the rule's logic though.