Support kebab-case namespace alias

[alexdao3]

I think this is a bug, as I don't see any reason why this wouldn't be supported

;; cljs
["firebase-admin$default" :as fb-admin]

emits

;; mjs
import fb-admin from 'firebase-admin';

fb-admin is invalid as a Javascript variable due to hyphens

Error:
SyntaxError: Unexpected token '-'

[borkdude]

Not sure why this should be supported: CLJS and shadow-cljs don't support it either right?

[borkdude]

Sorry, I think I read the report wrong.

Do you mean:

emits fb-admin instead of emits fb_admin ?

[alexdao3]

Yes, sorry about that. I meant "emits fb-admin"

[alexdao3]

If you point me roughly to where this logic lives, I could try to PR a fix if you agree

[borkdude]

@alexdao3 Ah yes, sorry for the pre-mature conversion into a Discussion. I created an issue from this now again.

[martinklepsch]

This was fixed here: #71

(Posting for anyone else who might be wondering)