Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I just did this to experiment if lazy loading builtins would work and tryed it out with Array for now.
Using the js input
Before:
![image](https://private-user-images.githubusercontent.com/936006/356842149-31740581-54f3-4795-b287-d204f7f5a096.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkzNDk5NjcsIm5iZiI6MTczOTM0OTY2NywicGF0aCI6Ii85MzYwMDYvMzU2ODQyMTQ5LTMxNzQwNTgxLTU0ZjMtNDc5NS1iMjg3LWQyMDRmN2Y1YTA5Ni5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjEyJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIxMlQwODQxMDdaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1lODk2MzRjYmM0M2NjODMyZWFmY2IyMmEwMzdkZGJiMGNiYTkzMDhjMTNmZGEzZGRhNzg3Y2NlOWE4MTM0ZWM0JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.I16GWm1tTzomX9j0b-urX580ftEm10Hv-i6OrggVrRU)
After:
![image](https://private-user-images.githubusercontent.com/936006/356842159-54650fcd-0010-4bb2-9338-fdfdb4829249.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkzNDk5NjcsIm5iZiI6MTczOTM0OTY2NywicGF0aCI6Ii85MzYwMDYvMzU2ODQyMTU5LTU0NjUwZmNkLTAwMTAtNGJiMi05MzM4LWZkZmRiNDgyOTI0OS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjEyJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIxMlQwODQxMDdaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT05YzY0MjE0YjFlZjRmYWZjNmY0ZmVlZDExZjNhNThjZDg4OWQ1MzViNjU0MDEzZDY2ZWNiN2IwNjA5NWU1ZjU4JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.GPrWpTqKU6Gua-Hv52qxbgobGna2DgAT3gU2y7m9dJA)
@jedel1043 do you have any pointers on how is best to structure this, and is the
GetName
operationBindingLocatorEnvironment::GlobalObject
going to be a safe assumption for global builtin access?