readOnly lazy attributes are not respected

[connor4312]

Testing #143572

1. Create a variable with only a getter, so that it contains readOnly in its presentationHint.attributes
2. Notice double clicking still allows setting it

[weinand]

Most likely this is not a specific problem of "lazy" variables...

[connor4312]

Looks like that's correct. I fixed this a short while ago in 989bb81 but it seems like it regressed.

[roblourens]

It seems like js-debug might not be sending a presentationHint on the resolved lazy property. I see 'readonly' and 'lazy' there before it's resolved but not after.

[weinand]

Yes, a "lazy" without a setter does not react on double click, so its value cannot be set.

However, if a setter is added, it still does not react on double click.
@connor @roblourens is this a problem of js-debug or VS Code?

[connor4312]

yep, will fix in js-debug