Bindings where @scope has same name as module can result in runtime errors

[mediremi]

When a binding's @scope has the same name as a module, the binding's scope is shadowed by the module:

module URL = {  
  @val @scope("URL")
  external revokeObjectURL: string => unit = "revokeObjectURL"
}

URL.revokeObjectURL("some url")

let URL = {};

URL.revokeObjectURL("some url");

[cknitt]

I think this is not a v12 issue per se, it's just that v12 doesn't escape URL with $$ anymore.

If the module name is different, the same problem exists in v11:

module URRL = {  
  @val @scope("URRL")
  external revokeObjectURL: string => unit = "revokeObjectURL"
}

URRL.revokeObjectURL("some url")

https://rescript-lang.org/try?version=v11.1.4&module=esmodule&code=LYewJgrgNgpgBAVQEpIDJwLxwN5zgKDwAEA3AQyjiIGcBjEABxgAoAiZNVgSkLhgA8ALjABOAOwpwRMEiADWMAPIAjAFYxag5KgBccaoJEBLMQHNMAPjgQxRwZjitpshSvWbtrfAF98+DqgAdM7ySmoaWmhs1CDA8BAiUNz4QA

[mediremi]

Ah I didn't read the breaking changes section of v12.0.0-alpha.1 carefully enough - I see now that since #6831 we're not escaping browser globals anymore.

I'll update this issue title and description to not be v12 specific 👍

[cometkim]

I see. It should be something like:

let URL$1 = {};

URL.revokeObjectURL("some url");

export {
  URL$1 as URL,
}

[cometkim]

Workaround (IMO the correct resolution):

module URL = {  
  @val @scope("globalThis.URL")
  external revokeObjectURL: string => unit = "revokeObjectURL"
}

URL.revokeObjectURL("some url")

[cknitt]

Although this can be broken in basically the same way:

module URL = {  
  @val @scope("globalThis.URL")
  external revokeObjectURL: string => unit = "revokeObjectURL"
}

let globalThis = 0

URL.revokeObjectURL("some url")

[cometkim]

globalThis should be mangled by default.