Fix adjacent tracking across adjacent scopes - #1985

[evs-chris]

This sets deps of a reference as mutated along with the reference so that update code is not skipped for the reference deps if they also happen to appear in an adjacent scope.

This is an area that could probably use a little bit of improvement down the road. Right now, deps on any given expression don't really indicate which parts are affected by mutation e.g. in {#each foo[bar] as whatever}<button on:click={() => whatever.sure += 'yes'}>mutate foo, but not bar</button>{/each}. With this PR, bar will also be flagged as mutated, which isn't exactly ideal, as it's not in this case.

[Rich-Harris]

👍

Yeah, I was thinking about that foo/bar problem. I think it's solvable. Definitely something we can worry about later though

[evs-chris]

Also, I just checked and it looks like this fixes #1917 too.

[Rich-Harris]

nice! closed