require-atomic-updates false positive

Tell us about your environment

• ESLint Version: 6.0.1
• Node Version: 12.4.0
• npm Version: 6.9.0

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

Default

Please show your full configuration:

https://github.com/medikoo/eslint-config-medikoo/blob/master/index.js

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

const fn = async someObj => {
  let { foo } = someObj;
  await Promise.resolve();
  // Possible race condition: `someObj.bar` might be reassigned based on an outdated value of `someObj.bar`
  someObj.bar = "whatever";
};

What did you expect to happen?

No error reported

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

Error reported:

Possible race condition: `someObj.bar` might be reassigned based on an outdated value of 

Are you willing to submit a pull request to fix this bug?

Not at this point

[stinovlas]

I would say that if nothing else, error message is pretty misleading.

someObj.bar might be reassigned based on an outdated value of someObj.bar

That is simply not true, because someObj.bar = "whatever"; assigns string "whatever" which is not depending on someObj.bar in any way.

Also, it’s not following the rule definition which clearly states that following must be true:

A variable or property is reassigned to a new value which is based on its old value.

[medikoo]

I think this is intentional behavior because that someObj object was read before await expression and written after await expression.

Ok, but that’s pretty aggressive assumption, as in many cases that can be an intentional well-thought construct

In place where it reported for me, it’s totally intentional and there’s no way to write it differently (aside of dropping async/await), due to that I needed to turn off this rule globally.

Still I’d love it to report for me cases as count += await getCount() which are error-prone obviously (and even if not, easy to write in a way they do not report)