ScriptEngineManager

[Tanguygab]

Hey 🖐
I was looking through the code to try to understand why in the world wouldn't the expansion detect Graal.js nor the JSEngine plugin and I've noticed that the expansion just created a new instance of the ScriptEngineManager while it could (should?) use the Bukkit ServicesManager.
I've realized that it might have been the cause JSEngine states it on its Spigot page

So I tried to use that, and it turns out it detected Graal.js correctly!

Idk with JSEngine since I was lazy to try x) but I guess it should work as well.
Anyways, hopefully this helps =)

[Tanguygab]

I've just tested and it doesn't seem to detect JSEngnie anyway, I'll try to search a bit more

[Tanguygab]

welp, it does work in the end, you just have to use JavaScript as engine, case sensitive, idk why js didn't work while it was listed nor why it doesn't want to show up under the Displaying all script engine factories. message...

[darbyjack]

Does this work on Java 16?

[Tanguygab]

that's what I tried on

[darbyjack]

Alrighty well thank you for your time and consideration but internally we have already discussed how we plan to move forward with this expansion and our changes will be implemented in the next week or so.

[Tanguygab]

Alright! I'll just close the pr then, looking forward to that update 👀