no-unused-vars errors with parameters when parameter marked used later

Tell us about your environment

• ESLint Version: 5.6.1
• Node Version: v9.4.0
• npm Version: 5.6.0

What parser (default, Babel-ESLint, etc.) are you using? babel-eslint

Please show your full configuration:

Configuration

{
	"parser": "babel-eslint",
	"plugins": [
		"babel",
		"react"
	],
	"extends": ["plugin:react/recommended"],
	"rules": {
		"no-unused-vars": 1
	}
}

What did you do? Please include the actual source code causing the issue, as well as the command that you used to run ESLint.

The eslint-plugin-react rule for jsx-uses-vars marks the Wrapped parameter as used. However, no-unused-vars doesn’t update the status of config. The default for no-unused-vars should not warn on parameters before the first used parameter.

import React from 'react';
import foo from 'foo';

export const AccessibilityDecorator = foo((config, Wrapped) => {
	return class t extends React.Component {
		render () {
			return <Wrapped />;
		}
	};
});

eslint .

What did you expect to happen?

No warnings.

What actually happened? Please include the actual, raw output from ESLint.

/Users/.../.../test.js
  4:44  warning  'config' is defined but never used  no-unused-vars

✖ 1 problem (0 errors, 1 warning)

Linked issue: yannickcr/eslint-plugin-react#2009

[platinumazure]

Thanks @webOS101 for digging further! I think I rewrote that code some time ago and this was an oversight on my part.

I’ve relabeled this as a bug, in accordance with your investigation results.

[webOS101]

I confirmed that works. I was able to create a test case that fails and passes with that fix. I still have never figured out how to use a locally linked eslint to test with either global or local installs. I guess I’ll figure it out someday. I’ll create a PR shortly.