Fix issue with incorrectly finding and triggering certain remappings

[xconverge]

fixes #2888

[wcauchois]

Thank you @xconverge!

[jpoon]

Err. sorry. I don't get this change. ELI5?

[xconverge]

@jpoon

Running this:

var remappings: { [key: string]: number } = {};

remappings["test"] = 4;

if(remappings["constructor"]){
	console.log("matched constructor");
}

if(remappings["test"]){
	console.log("matched matched test");
}

if(remappings["test123"]){
	console.log("matched matched test123");
}

The output is:

console: matched constructor
console: matched matched test

Any function name in the prototype of the object counts as a key in the map. There are ~10-20 functions in here and if anyone is writing javascript or typescript they would run into this problem a lot with our remapping

The fix here was to remove the prototype and have a plain map.

This was super enlightening to me and does not make sense to me coming from C++

[jpoon]

Lol. Good catch