fix(compiler): handle symbol properties for dynamic values

[seckardt]

Details

Fix for issue #3892.

Does this pull request introduce a breaking change?

• ✅ No, it does not introduce a breaking change.

Does this pull request introduce an observable change?

• ✅ No, it does not introduce an observable change.

GUS work item

[pmdartus]

All the Karma tests are timing out since @seckardt isn't an LWC collaborator.
Instead of failing with a timeout, it would be great to make those actions require collaborators' manual approval.

[nolanlawson]

@pmdartus I had hoped GitHub Actions had a generic solution for this, but if they do, I haven't found it yet. I just open duplicate PRs to force CI to run. 😞

[nolanlawson]

LGTM overall. Just a few minor suggestions, see comments above.

[nolanlawson]

Looking into it further, computed properties actually support all kinds of literals:

{ [1]: 2 }
{ [/foo/]: 2 }
{ [null]: 2 }

We don't need to be pedantic about this, though. I'll open a separate PR to handle these in @wire.

[nolanlawson]

Thanks a lot!