feat(es/ast): Add node id to Ident

[bvanjoi]

No description provided.

[changeset-bot]

⚠️ No Changeset found

Latest commit: d69aaaa

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[Austaras]

Intersting. But why?

[codspeed-hq]

CodSpeed Performance Report

Merging #10973 will not alter performance

_{Comparing bvanjoi:main (d69aaaa) with main (c8a2668)}

Summary

✅ 140 untouched benchmarks

[kdy1]

I think this is for reimplementing variable management, right?

[kdy1]

I used dev/rust because this is a Wasm-breaking change

[mischnic]

What’s the usecase here?
And why only identifier and not every node type?
In particular: what benefit does the nodeid have over the atom, context tuple which is also unique for the variable?

[kdy1]

@mischnic Maybe #10998 (review)?

I assumed that he's going to use sequential node IDs to reimplement variable management, but it seems like he is trying to have a faster version of Id.