destructuring an object within an array method

[eballeste]

const { stuff } = {stuff: true};

[{stuff: true}, {stuff: false}].forEach(item => {
  const { stuff } = item;
});

looks like this:

first stuff variable:

source.js 
meta.declaration.variable.js.fjsx15 
meta.binding.destructuring.mapping.js.fjsx15 
meta.block.js.fjsx15 
entity.name.variable.js.fjsx15

second one:

source.js 
meta.function-call.js.fjsx15 
meta.group.js.fjsx15 
meta.function.js.fjsx15 
meta.block.js.fjsx15 
meta.declaration.variable.js.fjsx15 
meta.binding.destructuring.mapping.js.fjsx15 
meta.block.js.fjsx15 
entity.name.variable.js.fjsx15

[borela]

The scopes looks correct but it is strange that material color scheme is acting this way, I'll take a look at it.

[eballeste]

this one i checked with several themes and it happens it most, not all of them

[borela]

Found the issue, the color schemes don't have a rule for entity.name.variable.

Variables are using 2 scopes:

• entity.name.variable when a variable is declared;
• variable.other.readwrite for any other uses;

I was using this distinction to enable the indexer to pick variables without showing duplicates in the list; I'll do some tests if reverting it to just variable will cause any issues.

[borela]

Updated the variables' scopes to use variable.other.readwrite and it caused no issues; https://github.com/borela/naomi/releases/tag/v3.6.1

It'll probably solve this issue.

[eballeste]

works! awesome!