Update js asset registration

[jayson-temporas]

Fix for Issue #86

[nathanheffley]

This seems like an unusual way to fix this issue, I'm not sure I've really ever seen a package use the iife format as modern packages are usually isolated by default anyway. Is it possible to figure out what is conflicting and fix the way we are registering it in the code instead?

[jayson-temporas]

Current asset registration use this:
Js::make('commentions-scripts', DIR . '/../resources/dist/commentions.js')

Which will produce <script src=".."> and this is not isolated. all variables and functions are in global namespace.

Fix is to either use iife or update the asset registration to use module
Js::make('commentions-scripts', DIR . '/../resources/dist/commentions.js')->module()

This will produce <script src=".." type="module"> which is isolated.

Remove the iife format now.

Thank you!